[Bug 13349] New: Allow library code to display against users pending changes
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Bug ID: 13349 Summary: Allow library code to display against users pending changes Change sponsored?: --- Product: Koha Version: 3.16 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Patrons Assignee: koha-bugs@lists.koha-community.org Reporter: david.roberts@ptfs-europe.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com When OPACPatronDetails is set to Allow in a multi-library environment, it is sometimes useful if the library of the user submitting the change is visible to the staff member approving the change, so that each library can approve or deny the request from their own patrons. The attached patch corrects that. To test: Ensure OPACPatronDetails is set to enabled Apply the patch Log into the OPAC and update your patron details (address, phone number etc) Log into the staff client and view pending patron changes. The branchcode of the library to which the patron belongs should display next to the user's name. -- 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=13349 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david.roberts@ptfs-europe.c | |om --- Comment #1 from David Roberts <david.roberts@ptfs-europe.com> --- Created attachment 33958 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33958&action=edit Allow patron's library to display after patron updates details -- 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=13349 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |david.roberts@ptfs-europe.c |ity.org |om -- 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=13349 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Christopher Brannon <cbrannon@cdalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cbrannon@cdalibrary.org --- Comment #2 from Christopher Brannon <cbrannon@cdalibrary.org> --- Traceback (most recent call last): File "/usr/local/bin/git-bz", line 2630, in <module> do_apply(*args) File "/usr/local/bin/git-bz", line 1755, in do_apply users = bug.server.get_xmlrpc_proxy().User.get({ 'names': [patch.attacher] })['users'] File "/usr/lib/python2.6/xmlrpclib.py", line 1199, in __call__ return self.__send(self.__name, args) File "/usr/lib/python2.6/xmlrpclib.py", line 1489, in __request verbose=self.__verbose File "/usr/lib/python2.6/xmlrpclib.py", line 1243, in request headers xmlrpclib.ProtocolError: <ProtocolError for bugs.koha-community.org/xmlrpc.cgi: 404 Not Found> Not sure if this is a problem on my end or what. Christopher -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply Version|3.16 |master --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- I get the same problem. It looks like the patch isn't formatted correctly? Please resubmit the patch, and while you're at it, add a description of the bug and a test plan. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #33958|0 |1 is obsolete| | --- Comment #4 from David Roberts <david.roberts@ptfs-europe.com> --- Created attachment 34074 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34074&action=edit Version 2 of the patch Sorry, I don't know why the patch wouldn't apply - I've tried to regenerate it so hopefully it will work OK this time. To test: 1) Ensure OPACPatronDetails is set to enabled 2) Log into the OPAC and update your patron details (address, phone number etc) 3) Log into the staff client and view pending patron changes. The branchcode of the library to which the patron belongs will not display. 4) Apply the patch 5) Log into the staff client again and view pending patron changes. The branchcode of the library to which the patron belongs should now display next to the user's name. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 David Roberts <david.roberts@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 --- Comment #5 from David Roberts <david.roberts@ptfs-europe.com> --- I've tested this patch against a sandbox and it compiles OK. There is already a description of the enhancement in comment 1. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 --- Comment #6 from Owen Leonard <oleonard@myacpl.org> --- Your second patch applied successfully using git-bz. Thanks for resubmitting. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 --- Comment #7 from Marc Véron <veron@veron.ch> --- Created attachment 34162 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34162&action=edit [Signed-off] Bug 13349 - Allow library code to display against users pending changes -update screen http://bugs.koha-community.org/show_bug.cgi?id=13349 To test: Ensure OPACPatronDetails is set to enabled Apply the patch Log into the OPAC and update your patron details (address, phone number etc) Log into the staff client and view pending patron changes. The branchcode of the library to which the patron belongs should display next to the user's name. Followed test plan, works as expected Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34074|0 |1 is obsolete| | CC| |veron@veron.ch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34162|0 |1 is obsolete| | --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 34349 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34349&action=edit [PASSED QA] Bug 13349 - Allow library code to display against users pending changes -update screen http://bugs.koha-community.org/show_bug.cgi?id=13349 To test: Ensure OPACPatronDetails is set to enabled Apply the patch Log into the OPAC and update your patron details (address, phone number etc) Log into the staff client and view pending patron changes. The branchcode of the library to which the patron belongs should display next to the user's name. Followed test plan, works as expected Signed-off-by: Marc Veron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 --- Comment #9 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 34350 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34350&action=edit [PASSED QA] Bug 13349 [QA Followup] I don't think we need the library label. Also switched to displaying the library name, rather than just the branchcode. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |12028 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |12352 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13349 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com Status|Passed QA |Pushed to Master --- Comment #10 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks David! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org