[Bug 11282] New: Not possible to create multiple restrictions from patron details/new restriction overwrites existing
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Bug ID: 11282 Summary: Not possible to create multiple restrictions from patron details/new restriction overwrites existing Change sponsored?: --- Product: Koha Version: 3.14 Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Patrons Assignee: koha-bugs@lists.koha-community.org Reporter: katrin.fischer@bsz-bw.de QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com To test: - Create a patron - Create a MANUAL restriction (former debarred flag) - Check it displays in the patron record - Try adding another (Using one of the edit links - not the restriction tab on the checkouts page) - The new restriction overwrote the existing one instead of creating another - Try adding a second restriction from the restriction tab on the checkouts page, it should work ok there. Also MANUAL does look a bit like an untranslatable string... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=2720 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 --- Comment #1 from Liz Rea <liz@catalyst.net.nz> --- I think this has to do more with how existing (pre 3.14) debarments are shown alongside the new system for restrictions.
From what I can tell, the two systems are utterly independent, which can lead to displays like this: http://img593.imageshack.us/img593/2772/12la.png - where a borrower is simultaneously having no restrictions, and being restricted. That's a really confusing display.
Also, one cannot add multiple debarments from the edit page, as Katrin has shown. That is confusing. I have a few ideas of how to fix this - remove the debarring functionality from "edit borrower" and add it (if it has a purpose!) to the restrictions tab. Probably not best, could be confusing. Display the edit-generated restriction in the table as a restriction, and leave its management aside (might be a good in-between step) Migrate the old restrictions to the new system in a DB update, and eliminate them from the edit screen altogether. Thoughts? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- What we have to keep in mind here is importing debarred patrons - we actually have a library where that is wanted. Right now it's just a matter of having a date in debarred in your patron import file. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Liz Rea <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |liz@catalyst.net.nz --- Comment #3 from Liz Rea <liz@catalyst.net.nz> --- A bit more about this - further poking indicates that the cause for error on the Member Edit page is to do with the first existing debarment being updated with new information, rather than a new one being added. Being able to edit a debarment is a nice feature, but I think it's not actually intended here. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|major |blocker --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Upping severity - you can accidentally lose data here - and losing information about why a patron is debarred/restricted can be quite a problem. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 23080 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23080&action=edit Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing Test Plan: 1) Edit a patron, add a restriction 2) Edit the patron again, add a second restriction 3) Note the first restriction has disappeared! 4) Apply this patch 5) Edit the patron again, add another restriction 6) Note the previous restriction is not longer removed -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA Version|3.14 |master --- Comment #6 from Owen Leonard <oleonard@myacpl.org> --- I can add an additional restriction from the patron edit page, but it doesn't save the date that I specify. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #23080|0 |1 is obsolete| | --- Comment #7 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 23808 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23808&action=edit Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing Test Plan: 1) Edit a patron, add a restriction 2) Edit the patron again, add a second restriction 3) Note the first restriction has disappeared! 4) Apply this patch 5) Edit the patron again, add another restriction 6) Note the previous restriction is not longer removed Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #8 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to Owen Leonard from comment #6)
I can add an additional restriction from the patron edit page, but it doesn't save the date that I specify.
The patch works for me, the dates are saved correctly. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 --- Comment #9 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 23816 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23816&action=edit [PASSED QA] Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing Test Plan: 1) Edit a patron, add a restriction 2) Edit the patron again, add a second restriction 3) Note the first restriction has disappeared! 4) Apply this patch 5) Edit the patron again, add another restriction 6) Note the previous restriction is not longer removed Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Tested: - Adding and removing multiple restrictions from - the details tab - the checkouts tab - the edit patron form All works as expected. Patch passes all tests in t, xt, and QA script. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Patch complexity|--- |Small patch -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #23808|0 |1 is obsolete| | Attachment #23816|0 |1 is obsolete| | --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 23817 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23817&action=edit [PASSED QA] Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing Test Plan: 1) Edit a patron, add a restriction 2) Edit the patron again, add a second restriction 3) Note the first restriction has disappeared! 4) Apply this patch 5) Edit the patron again, add another restriction 6) Note the previous restriction is not longer removed Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Tested: - Adding and removing multiple restrictions from - the details tab - the checkouts tab - the edit patron form All works as expected. Patch passes all tests in t, xt, and QA script. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #11 from Galen Charlton <gmcharlt@gmail.com> --- Pushed to master. Thanks, Kyle! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11282 Fridolin SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |fridolyn.somers@biblibre.co | |m --- Comment #12 from Fridolin SOMERS <fridolyn.somers@biblibre.com> --- Patch pushed to 3.14.x, will be in 3.14.2. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org