After upgrading to 21.11 from 21.05, locally cover images stopped loading on the details page for an individual biblio. They still load in a listing of multiple biblios. Additionally, images from the locally running Coce server load without problems. Example of multiple listing where covers show as normal: https://katalogo.biblioteko.ca/cgi-bin/koha/opac-search.pl?q=au:%22Doyle%2C%20Arthur%20Conan%22 Example of individual biblio where locally stored cover doesn't load: https://katalogo.biblioteko.ca/cgi-bin/koha/opac-detail.pl?biblionumber=2189&query_desc=au%3A%22Doyle%2C%20Arthur%20Conan%22
The image is loading but it is hidden. I notice you have some CSS loading twice in your OPAC: <link href="/opac-tmpl/bootstrap/css/opac_21.1101000.css" type="text/css" rel="stylesheet"> <link rel="stylesheet" type="text/css" href="/opac-tmpl/bootstrap/css/opac.css" /> Do you have a custom stylesheet defined somewhere? In the OpacAdditionalStylesheet system preference perhaps?
Thanks for the reply. I do have some custom CSS which I had previously stripped out (from OpacAdditionalStylesheet and any other possible field), which didn't help at all. I do also have a custom Coce server, hosted on the same machine, and I have now discovered that if I disable that the local cover images display correctly. Is this a bug in 21.11 which causes a conflict between a locally hosted Coce server and locally stored cover images perhaps? It worked perfectly under 21.05 and the previous versions of Koha I had used.
Looks like it's working now?
Only if I disable Coce, and I do very much want/need that to be working as well. So I wouldn't want to close this bug until that issue is resolved in case it's not just my system.
(In reply to Owen Leonard from comment #1) > <link href="/opac-tmpl/bootstrap/css/opac_21.1101000.css" type="text/css" > rel="stylesheet"> > <link rel="stylesheet" type="text/css" > href="/opac-tmpl/bootstrap/css/opac.css" /> > > Do you have a custom stylesheet defined somewhere? In the > OpacAdditionalStylesheet system preference perhaps? The source for that page still has two links to opac.css. I think that's the cause.
I have now stripped out all customisation of the OPAC css and the source shows that <link rel="stylesheet" type="text/css" href="/opac-tmpl/bootstrap/css/opac.css" /> is now absent. Upon reenabling Coce the local cover image again fails to display. Will leave the catalogue like this while investigating.
I've tested this with BywaterSolutions Coce server and for me it's the other way around. When both Coce and local covers are available, only the local cover shows on the detail page. The result page shows both. I wonder if that has to do with bug 28180 - linking them just in case.
For testing: https://bywatersolutions.com/education/koha-tutorial-cover-images-koha-opac-coce-server
Created attachment 129247 [details] [review] Bug 29803: Prevent local cover image to be hidden if Coce is enabled If Coce is enabled by no image are returned by the service, then no other image will be displayed (actually un-hidden) The getURL is returning early if no image is returned by the service and the "done" flag is never set. Test plan: Add a local cover image to a bibliographic record (ktd biblionumber=14 is good) Set CoceHost = http://coce.bywatersolutions.com Select all CoceProviders Enable OpacCoce => If no image is returned by Coce then the local image should be displayed anyway
Created attachment 129248 [details] [review] Bug 29803: Do not wait for the images more than 5 seconds In case of the previous bug, the page was calling the callback function every 100ms, endlessly. We should add a test to prevent future similar problem. Test plan: Remove the previous patch "Prevent local cover image to be hidden if Coce is enabled" Recreate the test plan => You should see the local image displayed after 5 seconds, and a log in the JS console
Hello there Both patches applied, and that worked! Many thanks for your efforts on resolving this bug. Martin
I believe we have the same issue as Martin (although we're on 21.05). I tried the troubleshooting suggestions above and they didn't work for me. Wondering what I'm missing here. Covers show in opac-search.pl: https://catalog.helmlib.org/cgi-bin/koha/opac-search.pl?idx=ti&q=Careers+in+psychology+%26+behavioral+health.&limit=&weight_search=1 No covers in opac-detail.pl: https://catalog.helmlib.org/cgi-bin/koha/opac-detail.pl?biblionumber=1253757&query_desc=Careers%20in%20psychology%20%26%20behavioral%20health. The local covers do load in opac-detail.pl sometimes, adding to my confusion: https://catalog.helmlib.org/cgi-bin/koha/opac-detail.pl?biblionumber=1298655&query_desc=1298655 The ones that don't appear seem to blip in and out of existence as the page loads so they're being hidden by something. I've tried disabling JS which didn't solve the problem.
Good afternoon I've just upgraded to 21.11.02 and this problem has returned. Can I just reapply the patches which I previously applied for 21.11.01? Martin
(In reply to Martin Morris from comment #13) > Good afternoon > > I've just upgraded to 21.11.02 and this problem has returned. Can I just > reapply the patches which I previously applied for 21.11.01? > > Martin Can you add your Signed-off-by lines to them? That will make them move forward. https://wiki.koha-community.org/wiki/Sign_off_on_patches
(In reply to Martin Morris from comment #13) > Good afternoon > > I've just upgraded to 21.11.02 and this problem has returned. Can I just > reapply the patches which I previously applied for 21.11.01? > > Martin It will probably work, but you always have to be careful when applying manually locally. Better to get this moving into releases.
Created attachment 130579 [details] [review] Bug 29803: Prevent local cover image to be hidden if Coce is enabled If Coce is enabled by no image are returned by the service, then no other image will be displayed (actually un-hidden) The getURL is returning early if no image is returned by the service and the "done" flag is never set. Test plan: Add a local cover image to a bibliographic record (ktd biblionumber=14 is good) Set CoceHost = http://coce.bywatersolutions.com Select all CoceProviders Enable OpacCoce => If no image is returned by Coce then the local image should be displayed anyway Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 130580 [details] [review] Bug 29803: Do not wait for the images more than 5 seconds In case of the previous bug, the page was calling the callback function every 100ms, endlessly. We should add a test to prevent future similar problem. Test plan: Remove the previous patch "Prevent local cover image to be hidden if Coce is enabled" Recreate the test plan => You should see the local image displayed after 5 seconds, and a log in the JS console Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
I signed off, but you might want to have a look at the warning from QA test tools for QA: FAIL koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt FAIL forbidden patterns forbidden pattern: console.log (line 1568)
The console.log is expected, see the second patch.
(In reply to Jonathan Druart from comment #19) > The console.log is expected, see the second patch. I thought it might, but always better to get any questions out of the way :)
Created attachment 130866 [details] [review] Bug 29803: Prevent local cover image to be hidden if Coce is enabled If Coce is enabled by no image are returned by the service, then no other image will be displayed (actually un-hidden) The getURL is returning early if no image is returned by the service and the "done" flag is never set. Test plan: Add a local cover image to a bibliographic record (ktd biblionumber=14 is good) Set CoceHost = http://coce.bywatersolutions.com Select all CoceProviders Enable OpacCoce => If no image is returned by Coce then the local image should be displayed anyway Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 130867 [details] [review] Bug 29803: Do not wait for the images more than 5 seconds In case of the previous bug, the page was calling the callback function every 100ms, endlessly. We should add a test to prevent future similar problem. Test plan: Remove the previous patch "Prevent local cover image to be hidden if Coce is enabled" Recreate the test plan => You should see the local image displayed after 5 seconds, and a log in the JS console Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
Pushed to 21.11.x for 21.11.03
Could this be related to https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30234 ?
(In reply to Manos PETRIDIS from comment #25) > Could this be related to > https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30234 ? It could be a duplicate, but this bug has already been closed and pushed to stable versions, so best way to find out is to update :)
(In reply to Katrin Fischer from comment #26) > (In reply to Manos PETRIDIS from comment #25) > > Could this be related to > > https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30234 ? > > It could be a duplicate, but this bug has already been closed and pushed to > stable versions, so best way to find out is to update :) Unfortunately I'm already at 21.11.03.000 and apt-get reports there isn't anything new to install.
Missing dependency, not backported to 21.05