Summary: | Patron batch modification tool cannot use authorised value "0" | ||
---|---|---|---|
Product: | Koha | Reporter: | Emma Perks <emma.perks> |
Component: | Patrons | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | charles.farmer, dcook, emma.perks, fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 23217, 23218 | ||
Attachments: |
Bug 19673: Allow to set patron attributes to 0 with batch patron modification
Bug 19673: Allow to set patron attributes to 0 with batch patron modification Bug 19673: Allow to set patron attributes to 0 with batch patron modification |
Description
Emma Perks
2017-11-21 15:57:35 UTC
Created attachment 69984 [details] [review] Bug 19673: Allow to set patron attributes to 0 with batch patron modification This patch will have to be tested deeply to make sure it will not introduce regression! The idea is to display an empty option in the patron attributes select and ignore it. That way we can deal with false values 0 and "" which were skipped before. Test plan: Add several patron attributes Use the batch patron modification tool to add/update/remove them Play with empty "" and 0 values, as well as other values Modify several attributes in a row Created attachment 73223 [details] [review] Bug 19673: Allow to set patron attributes to 0 with batch patron modification This patch will have to be tested deeply to make sure it will not introduce regression! The idea is to display an empty option in the patron attributes select and ignore it. That way we can deal with false values 0 and "" which were skipped before. Test plan: Add several patron attributes Use the batch patron modification tool to add/update/remove them Play with empty "" and 0 values, as well as other values Modify several attributes in a row Signed-off-by: Charles Farmer <charles.farmer@inLibro.com> Could reproduce the original issue with newly created attributes. The values 0 and empty string ('') were not considered during update. Applied patch, and now I could update with both 0 and an empty field. Patch set to 'Signed off' Created attachment 73367 [details] [review] Bug 19673: Allow to set patron attributes to 0 with batch patron modification This patch will have to be tested deeply to make sure it will not introduce regression! The idea is to display an empty option in the patron attributes select and ignore it. That way we can deal with false values 0 and "" which were skipped before. Test plan: Add several patron attributes Use the batch patron modification tool to add/update/remove them Play with empty "" and 0 values, as well as other values Modify several attributes in a row Signed-off-by: Charles Farmer <charles.farmer@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 18.05, thanks to everybody involved! Pushed to stable for 17.11.06 Awesome work all! Pushed to 17.05.x for v17.05.12 I think that I may have found a regression caused by this patch: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23217 Basically I'm getting database errors when I try to do Batch Patron Modification and there is no borrower attribute provided, although there are borrower attribute codes on the system so the select menu appears. (In reply to David Cook from comment #8) > I think that I may have found a regression caused by this patch: > > https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23217 > > Basically I'm getting database errors when I try to do Batch Patron > Modification and there is no borrower attribute provided, although there are > borrower attribute codes on the system so the select menu appears. The key is using DEBUG=1 in your Apache configuration to turn on RaiseError in DBI. Also found this bug: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23218 Depending on user behaviour, it may or may not be an edge case |