Bug 15574 - Better wording for error message when adding tags
Summary: Better wording for error message when adding tags
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-14 00:39 UTC by Aleisha Amohia
Modified: 2017-06-14 22:10 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 15574: Better wording for error message when adding a duplicate tag (1.60 KB, patch)
2016-01-14 00:43 UTC, Aleisha Amohia
Details | Diff | Splinter Review
[SIGNED-OFF]Bug 15574: Better wording for error message when adding a duplicate tag (1.70 KB, patch)
2016-01-15 01:22 UTC, Héctor Eduardo Castro Avalos
Details | Diff | Splinter Review
Bug 15574: Better wording for error message when adding a duplicate tag (1.74 KB, patch)
2016-01-15 15:55 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Aleisha Amohia 2016-01-14 00:39:56 UTC
When a user attempts to add a tag which already exists, the error message starts with: 
"Error! The add_tag operation failed on"
which is kind of developer speak and should be worded better.
Comment 1 Aleisha Amohia 2016-01-14 00:43:04 UTC Comment hidden (obsolete)
Comment 2 Héctor Eduardo Castro Avalos 2016-01-15 01:22:49 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2016-01-15 15:55:42 UTC
Created attachment 46706 [details] [review]
Bug 15574: Better wording for error message when adding a duplicate tag

To test:

1) Apply patch
2) Go to a record (ie opac-detail.pl) and add tags that already exist with that record
3) Confirm new error message

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Rewording Ok for common patrons

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Brendan Gallagher 2016-01-27 05:35:53 UTC
Pushed to Master - Should be in the May 2016 release. Thanks!
Comment 5 Julian Maurice 2016-02-04 21:50:52 UTC
Patch pushed to 3.22.x, will be in 3.22.3
Comment 6 Frédéric Demians 2016-02-20 08:54:13 UTC
This patch has been pushed to 3.20.x, will be in 3.20.9.