Bug 5265

Summary: error message on lists is misleading
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris
Version: Main   
Hardware: All   
OS: All   
URL: cgi-bin/koha/virtualshelves/shelves.pl
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: error
Proposed fix

Description Nicole C. Engard 2010-09-29 17:06:30 UTC
Created attachment 2655 [details]
error

When you try to add a barcode to a list on the staff client that doesn't exist your error reads "A record matching barcode 123456987 has already been added." -- it should instead say that this barcode doesn't exist.
Comment 1 Owen Leonard 2010-10-04 14:33:28 UTC
Created attachment 2662 [details] [review]
Proposed fix

The check to see if an item exists always evaluates as true.
Correcting this lets the script handle the error as it
was designed to.
Comment 2 Nicole C. Engard 2010-10-04 15:12:39 UTC
signed off on patch - much better!
Comment 3 MJ Ray (software.coop) 2010-11-19 16:33:23 UTC
This bug is mentioned in:
Fix for Bug 5265,	error message on lists is misleading http://lists.koha-community.org/pipermail/koha-patches/2010-October/012607.html
Fix for Bug 5265,	error message on lists is misleading http://lists.koha-community.org/pipermail/koha-patches/2010-October/012609.html
Comment 4 Chris Cormack 2010-11-21 20:16:57 UTC
This has been applied to master, please test and close