search

hrehfeld / QuakeInjector

Download, install and play quake singleplayer maps from the quaddicted.com archive
https://www.quaddicted.com/tools/quake_injector
GNU General Public License v3.0
94 stars 15 forks source link

Support for tags #104

Open SpiritQuaddicted opened 9 years ago

SpiritQuaddicted commented 9 years ago

Similarly to https://www.quaddicted.com/reviews/ the QI should list the tags and allow filtering. This will require the "API" at Quaddicted to serve them first.

hrehfeld commented 9 years ago

Check the xml parsing, I think you should just be able to add new tags into the map section. Adding to the api would be a first step. What are the allowed chars in tags? how do you save them in the db?

hrehfeld commented 8 years ago

Any progress on this?

SpiritQuaddicted commented 7 years ago

I could simply add a new element like this:

<tags>
  <tag>this</tag>
  <tag>is</tag>
  <tag>a</tag>
  <tag>tag</tag>
</tags>

I use /^[a-z0-9_\-, ]*$ to sanitize tags. The table has zipname&tag&username in each row. Usernames are not really used, I had other plans back then. Messy spaghetti ahead: https://github.com/SpiritQuaddicted/Quaddicted-reviews/blob/master/details.php#L67

SpiritQuaddicted commented 3 years ago

Tags are now included in the XML.