Opened 16 years ago
Closed 15 years ago
#4498 closed defect (fixed)
tags not handled correctly
Reported by: | izzy | Owned by: | Radek Bartoň |
---|---|---|---|
Priority: | normal | Component: | ScreenshotsPlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: | 0.11 |
Description
When adding tags to a screenshot, the plugin not only inserts the specified tags, but also adds author, components, versions and name as tags - which is not what is usually done (neither tickets nor wiki pages show the same behaviour). Moreover, if one then decides to remove the tags again (by emptying the corresponding field), only the contents of that field get removed. To remove the other tags, one needs to drop the screenshot completely.
I created a patch to change that behaviour to be more "trac-like" (i.e. it will only add the specified tags, and not the other elements). You can simply apply this patch if you agree to handle tags like the other components do. Otherwise, you should add least make this optional (and turned off by default).
Attachments (1)
Change History (4)
Changed 16 years ago by
Attachment: | tags-only.diff added |
---|
comment:1 Changed 16 years ago by
Status: | new → assigned |
---|
That is matter of taste. I can add options to disable tagging of name, author, component and version but the claim that tickets don't do this is wrong. Opened tickets generate author, component and version tags. Just see Tags on this server. That's why I did it how it is.
comment:2 Changed 16 years ago by
That is what I compared: I tagged wiki pages, some tickets, and some screenshots. Then I went to the /tags
URL. From the wiki and ticket tags, only the tags have been taken - while the screenshots additionally displayed author etc. as tags.
But however: Having the possibility to switch that off (and only "tag the tags") would be fine. As it is now, you have the problem that you cannot even "untag" name and author without deleting the screenshot.
Generally, the idea to have tags on the version and component is not that bad - it's simply not always suitable. Just why to tag the name I cannot understand. Having these things optionally, however, cannot hurt anyone ;)
comment:3 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
r7486 adds additional_tags
config option.
when adding tags, only add the tags to the tags table (and not author, components, etc. as well)