[Bug 32080] New: Public lists not available after recent upgrade
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Bug ID: 32080 Summary: Public lists not available after recent upgrade Change sponsored?: --- Product: Koha Version: 22.05 Hardware: All OS: Linux Status: NEW Severity: major Priority: P5 - low Component: Lists Assignee: koha-bugs@lists.koha-community.org Reporter: snicker1@maine.rr.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl I recently upgraded the Debian package version from 19.11.03.000 to the latest 22.05.04.000 (including the underlying OS upgrades as well) and now no one is able to view any of the "Public Lists" and get a 500 error when trying. Opac-error.log shows this ("USERID" is my userid and "WEBSITE" is the hostname in the messages below): Mon Oct 24 17:44:32.268569 2022] [cgi:error] [pid 22000] [client x.x.x.x:5322] AH01215: DBIx::Class::Storage::DBI::select_single(): Query returned more than one row. SQL that returns multiple rows is DEPRECATED for ->find and ->single at /usr/share/koha/lib/Koha/Objects.pm line 96: /usr/share/koha/opac/cgi-bin/opac/opac-suggestions.pl [Mon Oct 24 17:44:39.651936 2022] [cgi:error] [pid 21984] [client x.x.x.x:56417] AH01215: Use of inherited AUTOLOAD for non-method Koha::Virtualshelf::haspermission() is deprecated at /usr/share/koha/lib/Koha/Virtualshelf.pm line 247.: /usr/share/koha/opac/cgi-bin/opac/opac-shelves.pl, referer: http://WEBSITE/cgi-bin/koha/opac-shelves.pl?op=list&public=1 [Mon Oct 24 17:44:39.652426 2022] [cgi:error] [pid 21984] [client x.x.x.x:56417] AH01215: Template process failed: undef error - Can't locate object method "_columns" via package "USERID" (perhaps you forgot to load "USERID"?) at /usr/share/koha/lib/Koha/Object.pm line 858.: /usr/share/koha/opac/cgi-bin/opac/opac-shelves.pl, referer: http://WEBSITE/cgi-bin/koha/opac-shelves.pl?op=list&public=1 [Mon Oct 24 17:44:39.695408 2022] [cgi:error] [pid 21984] [client x.x.x.x:56417] End of script output before headers: opac-shelves.pl, referer: http://WEBSITE/cgi-bin/koha/opac-shelves.pl?op=list <http://WEBSITE/cgi-bin/koha/opac-shelves.pl?op=list&public= 1> &public=1 I did look in the database itself and do find a count of 93 items in the 'virtualshelves' table. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=29955 CC| |lucas@bywatersolutions.com --- Comment #1 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Steven Nickerson from comment #0)
[Mon Oct 24 17:44:39.651936 2022] [cgi:error] [pid 21984] [client x.x.x.x:56417] AH01215: Use of inherited AUTOLOAD for non-method Koha::Virtualshelf::haspermission() is deprecated at /usr/share/koha/lib/Koha/Virtualshelf.pm line 247.: /usr/share/koha/opac/cgi-bin/opac/opac-shelves.pl, referer: http://WEBSITE/cgi-bin/koha/opac-shelves.pl?op=list&public=1
return 1 if $self->is_public and haspermission( $patron->userid, { lists => 'edit_public_lists' } ); This is corrected in master with: return 1 if $self->is_public and C4::Auth::haspermission( $patron->userid, { lists => 'edit_public_lists' } ); Solved on Bug 29955: Fix method import issues This fix should be backported. Trying to catch the attention of the 22.05 RMaint: Lucas Gass. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_22_05_candidate --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Setting the keyword for his version - agreed with Lucas on that so that he can spot those requests easier. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 --- Comment #3 from Blou <philippe.blouin@inlibro.com> --- Created attachment 143108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143108&action=edit Bug 29955: Fix method import issues This patch fixes this: $ prove t/db_dependent/Utils/Datatables_Virtualshelves.t t/db_dependent/Utils/Datatables_Virtualshelves.t .. 2/13 Use of inherited AUTOLOAD for non-method Koha::Virtualshelf::haspermission() is no longer allowed at /kohadevbox/koha/Koha/Virtualshelf.pm line 248. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #143108|0 |1 is obsolete| | --- Comment #4 from Blou <philippe.blouin@inlibro.com> --- Created attachment 143109 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=143109&action=edit Bug 29955: Fix method import issues This patch fixes this: $ prove t/db_dependent/Utils/Datatables_Virtualshelves.t t/db_dependent/Utils/Datatables_Virtualshelves.t .. 2/13 Use of inherited AUTOLOAD for non-method Koha::Virtualshelf::haspermission() is no longer allowed at /kohadevbox/koha/Koha/Virtualshelf.pm line 248. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> https://bugs.koha-community.org/show_bug.cgi?id=32080 Signed-off-by: Blou <blou@inlibro.com> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff CC| |philippe.blouin@inlibro.com --- Comment #5 from Blou <philippe.blouin@inlibro.com> --- I attached the REBASED patch from Bug 29955 that fixed it for me in 22.05.06. Not sure if RM want to put it as-is, or rename it or ... So I signed it off, as it works for me. Marcel/QA can validate if it makes sense. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #6 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Blou from comment #5)
I attached the REBASED patch from Bug 29955 that fixed it for me in 22.05.06.
Not sure if RM want to put it as-is, or rename it or ... So I signed it off, as it works for me. Marcel/QA can validate if it makes sense.
Looks good to me. Will change status so that RMaint can pick it up. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable --- Comment #7 from Lucas Gass <lucas@bywatersolutions.com> --- Thanks all, this is backported to 22.05.x for upcoming 22.05.07 release -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32080 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_22_05_candidate | -- 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