[Bug 36511] New: Local cover images in admin interface after update
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Bug ID: 36511 Summary: Local cover images in admin interface after update Change sponsored?: --- Product: Koha Version: 23.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs@lists.koha-community.org Reporter: isabel.pineda@ds.edu.sv QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl Today I updated Koha from version 23.11.03 to 23.11.04 and now the local cover images of the books are not visible anymore in the admin interface. In the OPAC they are still visible normally. When I upload a new local cover image, I can see it in the OPAC interface too, but not in the admin interface. Is there some error in the update? -- 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=36511 boris.dovbnya <boris.dovbnya@kname.edu.ua> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |boris.dovbnya@kname.edu.ua --- Comment #1 from boris.dovbnya <boris.dovbnya@kname.edu.ua> --- My problem is described exactly here (Bug 36511). A speedy correction would be greatly appreciated. I'm waiting for help. -- 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=36511 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal Version|23.11 |master Summary|Local cover images in admin |Local cover images broken |interface after update |in admin interface after | |update --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- git bisect tells me this is caused by Bug 24879 -- 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=36511 --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- The error I get is: Undefined subroutine &CGI::Compile::ROOT::kohadevbox_koha_catalogue_image_2epl::check_cookie_auth called at /kohadevbox/koha/catalogue/image.pl line 36 -- 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=36511 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |oleonard@myacpl.org |ity.org | Status|NEW |ASSIGNED Summary|Local cover images broken |Some scripts missing a |in admin interface after |dependency following Bug |update |24879 -- 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=36511 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 164566 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164566&action=edit Bug 36511: Some scripts missing a dependency following Bug 24879 These files needed the addition of 'use C4::Auth qw( check_cookie_auth );'. To test, apply the patch and restart services. - If necessary, enable the LocalCoverImages system preference. - Open the browser console and then the "Network" tab. You can click "Images" to filter for the correct kind of request. - Perform a catalog search. After the search has loaded, check that there are no 500 errors in the Network tab. - Go to Cataloging -> Label creator. - If necessary, create a label batch and add some items. - Export your batch and test both the "Download as CSV" and "Download as XML" links. Both should trigger the correct download. - Go to Serials -> Claims, and select a vendor with late issues. - Select all late issues and click "Download selected claims" at the bottom of the page. - Your CSV file should download correctly. The file acqui/check_uniqueness.pl has been corrected as well but I'm not sure how to test it! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |24879 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24879 [Bug 24879] Add missing authentication checks -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Danyon Sewell <danyonsewell@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Danyon Sewell <danyonsewell@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164566|0 |1 is obsolete| | --- Comment #5 from Danyon Sewell <danyonsewell@catalyst.net.nz> --- Created attachment 164574 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164574&action=edit Bug 36511: Some scripts missing a dependency following Bug 24879 These files needed the addition of 'use C4::Auth qw( check_cookie_auth );'. To test, apply the patch and restart services. - If necessary, enable the LocalCoverImages system preference. - Open the browser console and then the "Network" tab. You can click "Images" to filter for the correct kind of request. - Perform a catalog search. After the search has loaded, check that there are no 500 errors in the Network tab. - Go to Cataloging -> Label creator. - If necessary, create a label batch and add some items. - Export your batch and test both the "Download as CSV" and "Download as XML" links. Both should trigger the correct download. - Go to Serials -> Claims, and select a vendor with late issues. - Select all late issues and click "Download selected claims" at the bottom of the page. - Your CSV file should download correctly. The file acqui/check_uniqueness.pl has been corrected as well but I'm not sure how to test it! Signed-off-by: danyonsewell <danyonsewell@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@gmail.com Severity|normal |major -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164574|0 |1 is obsolete| | --- Comment #6 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 164577 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164577&action=edit Bug 36511: Some scripts missing a dependency following Bug 24879 These files needed the addition of 'use C4::Auth qw( check_cookie_auth );'. To test, apply the patch and restart services. - If necessary, enable the LocalCoverImages system preference. - Open the browser console and then the "Network" tab. You can click "Images" to filter for the correct kind of request. - Perform a catalog search. After the search has loaded, check that there are no 500 errors in the Network tab. - Go to Cataloging -> Label creator. - If necessary, create a label batch and add some items. - Export your batch and test both the "Download as CSV" and "Download as XML" links. Both should trigger the correct download. - Go to Serials -> Claims, and select a vendor with late issues. - Select all late issues and click "Download selected claims" at the bottom of the page. - Your CSV file should download correctly. The file acqui/check_uniqueness.pl has been corrected as well but I'm not sure how to test it! Signed-off-by: danyonsewell <danyonsewell@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |24.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #8 from David Cook <dcook@prosentient.com.au> --- We just bumped into this as well for 23.11.04. Will need to backport this one pretty widely. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Hans Ulrich Frink <hansulrich.frink@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hansulrich.frink@gmail.com --- Comment #9 from Hans Ulrich Frink <hansulrich.frink@gmail.com> --- Our koha install is showing this error, too. I patched image.pl with the additional line. The error persisted: in admin page no local cover images are shown. GET https://gagldkohastaff.duckdns.org/cgi-bin/koha/catalogue/image.pl?thumbnail=1&imagenumber=1243 500 (Internal Server Error) Do I have to apply the patch to all 5 files? thanks Uli -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|24.05.00 |24.05.00,23.11.04 released in| | CC| |fridolin.somers@biblibre.co | |m --- Comment #10 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.11.x on top of v23.11.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|24.05.00,23.11.04 |24.05.00,23.11.04,23.05.10 released in| | CC| |lucas@bywatersolutions.com Status|Pushed to stable |Pushed to oldstable --- Comment #11 from Lucas Gass <lucas@bywatersolutions.com> --- Pushed to 23.05.10 and packaged for 23.05.10-02 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 wainuiwitikapark@catalyst.net.nz changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wainuiwitikapark@catalyst.n | |et.nz --- Comment #12 from wainuiwitikapark@catalyst.net.nz --- Pushed to 22.05.x for 22.05.20-3 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 --- Comment #13 from wainuiwitikapark@catalyst.net.nz --- (In reply to wainuiwitikapark from comment #12)
Pushed to 22.05.x for 22.05.20-3
*v22.05.20-2 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldstable |Pushed to oldoldstable Version(s)|24.05.00,23.11.04,23.05.10 |24.05.00,23.11.04,23.05.10, released in| |22.11.16,22.05.20 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldoldstable |Pushed to oldoldoldstable --- Comment #14 from Fridolin Somers <fridolin.somers@biblibre.com> --- I see it in in 22.11.16 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Bug 36511 depends on bug 24879, which changed state. Bug 24879 Summary: Add missing authentication checks https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24879 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldoldoldstable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36511 Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to oldoldoldstable |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org