[Bug 14830] New: anonymous purchase suggestions are invisible/inaccessible in staff interface
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14830 Bug ID: 14830 Summary: anonymous purchase suggestions are invisible/inaccessible in staff interface Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: mtj@kohaaloha.com QA Contact: testopia@bugs.koha-community.org this seems to be due to anon. suggestions being created with a NULL branchcode more info to come... -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14830 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=14839 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14830 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |14839 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14839 [Bug 14839] 'anonymous' purchase suggestions are created with 'Anon' user's branchcode ignored, (NULL branchcode) -- 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=14830 --- Comment #1 from Mason James <mtj@kohaaloha.com> --- Created attachment 43244 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=43244&action=edit Bug 14830 - anonymous purchase suggestions are invisible/inaccessible in staff interface To test bug... 1/ set various suggestion sysprefs like so mysql> select variable, value from systempreferences where variable like '%sugg%'; +-------------------------------------+-------+ | variable | value | +-------------------------------------+-------+ | AllowPurchaseSuggestionBranchChoice | 0 | | AnonSuggestions | 1 | | OpacSuggestionManagedBy | 1 | | OPACViewOthersSuggestions | 0 | | suggestion | 1 | +-------------------------------------+-------+ 2/ create an generic ANON borrower for testing 3/ set 'AnonymousPatron' syspref to ANON's borrowernumber 4/ add a suggestion from the OPAC, without being logged in (ie: an anonymous suggestion) 2/ go to staff interface , fail to see suggestion :'( https://demo-admin.yourkoha.foo/cgi-bin/koha/suggestion/suggestion.pl 3/ apply patch... 4/ go to staff interface , see suggestion! :) (pix attached) https://demo-admin.yourkoha.foo/cgi-bin/koha/suggestion/suggestion.pl 5/ reward yourself with cake (optional) -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14830 --- Comment #2 from Mason James <mtj@kohaaloha.com> --- this patch adds some SQL to display suggestion rows with a NULL branchcode, in the staff interface -- 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=14830 --- Comment #3 from Mason James <mtj@kohaaloha.com> --- Created attachment 43245 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=43245&action=edit pix, after 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=14830 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored Priority|P5 - low |P3 Patch complexity|--- |Trivial 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=14830 Mason James <mtj@kohaaloha.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=14830 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #43244|0 |1 is obsolete| | --- Comment #4 from Mason James <mtj@kohaaloha.com> --- Created attachment 43246 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=43246&action=edit Bug 14830 - anonymous purchase suggestions are invisible/inaccessible in staff interface To test bug... 1/ set various suggestion sysprefs like so mysql> select variable, value from systempreferences where variable like '%sugg%'; +-------------------------------------+-------+ | variable | value | +-------------------------------------+-------+ | AllowPurchaseSuggestionBranchChoice | 0 | | AnonSuggestions | 1 | | OpacSuggestionManagedBy | 1 | | OPACViewOthersSuggestions | 0 | | suggestion | 1 | +-------------------------------------+-------+ 2/ create an generic ANON borrower for testing 3/ set 'AnonymousPatron' syspref to ANON's borrowernumber 4/ add a suggestion from the OPAC, without being logged in (ie: an anonymous suggestion) 5/ go to staff interface , fail to see suggestion :'( https://demo-admin.yourkoha.foo/cgi-bin/koha/suggestion/suggestion.pl 6/ apply patch... 7/ go to staff interface , see suggestion! :) (pix attached) https://demo-admin.yourkoha.foo/cgi-bin/koha/suggestion/suggestion.pl 8/ reward yourself with cake (optional) -- 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=14830 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch --- Comment #5 from Marc Véron <veron@veron.ch> --- Testing: - Check with mysql: I did not have branchcode = NULL in the suggestions table. - I tried to have one by removing syspref AnonymousPatron and then submit a suggestion from OPAC, it resulted in an software error on Suggestions.pm line 108 (Column 'suggestedby' canot be null). - I set the branchcode of one of the suggestions to NULL usimg mysql - I could reproduce the issue (the one with NULL branchcode did not show up) - With patch, the NULL-suggestion shows up, that is OK so far. More testing: - In the mysgl table I have following branchcodes: FFL, FFL, FFL, MPL, NULL, all with status ASKED - On the home page, I have "Suggestions pending approval: 5" - But not all suggestions show up when managing suggestions, only the ones with the same branchcode as the staff user (branch code FFL, all rights set, superlibrarian), plus the NULL one if the patch is applied. That is not consistent. The question is: Should the librarian be able to manage the suggestions for all branches? - If yes, all sggestions should display on Home > Acquisitions > Suggestions management (I think including the NULL branchcode ones). I would expect that for IndependentBranches = 'Don't prevent...' and superlibrarians. (That's the behaviour of this syspref for other things). - If no, the counter on the home page needs some adjustement to show the count of pending suggestions that are manageable for the staff users branch I would expect that for IndependentBranches = 'Don't prevent...' current staff user is not a superlibrarian. -- 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=14830 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=13755 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14830 --- Comment #6 from Marc Véron <veron@veron.ch> --- One more finding: A suggestion ist not visible / manageable if created in staff client with Library set to "Any" The value for the library code in the database is en empty string in that case. It is not visible in staff client, not even for superlibrarians (with and without 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=14830 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org