QuakeInjector icon indicating copy to clipboard operation
QuakeInjector copied to clipboard

Support for tags

Open SpiritQuaddicted opened this issue 9 years ago • 4 comments

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.

SpiritQuaddicted avatar Aug 16 '15 13:08 SpiritQuaddicted

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 avatar Aug 23 '15 19:08 hrehfeld

Any progress on this?

hrehfeld avatar Oct 18 '16 11:10 hrehfeld

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 avatar Dec 04 '16 14:12 SpiritQuaddicted

Tags are now included in the XML.

SpiritQuaddicted avatar Aug 22 '21 10:08 SpiritQuaddicted