[Koha-bugs] [Bug 22445] Ability to define a url for custom cover images

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22445

--- Comment #18 from Hayley Mapley <hayleymapley at catalyst.net.nz> ---
(In reply to Jonathan Druart from comment #17)
> (In reply to Hayley Mapley from comment #15)
> > These patches don't seem to work fully as expected for me, as outlined below.
> > 
> > When clicking Edit on the bibliographic record details page and uploading a
> > custom image, all test plans pass except for:
> > - OPAC reviews (Comment #11 rss - template is still broken it seems), 
> > - OPAC reviews (Comment #10 Recent Comments), 
> > - OPAC user (Comment #12 checkout list). 
> > In the case of the latter two scenarios, instead of displaying the image or
> > the 'no cover image' placeholder image, a link that says 'Cover image' is
> > displayed.
> > 
> > In terms of the url/local path options in the system prefs, is there
> > anything that needs to be done apart from allowing display, setting the
> > path/url and ensuring that the biblio has a url that is present in the
> > path/url? I'm having trouble testing this (I'm pretty new)
> > 
> > Additionally, in most places where the cover image is displayed (such as the
> > item detail page, etc.), the 'Cover image' link is also displayed
> > underneath. I'm assuming this is a bug?
> 
> Hello Hayley,
> Did you fill CustomCoverImagesURL in? What value?
> You are not supposed to attach an image to the record (this is the
> LocalCoverImages feature) neither add "an url to a biblio" (I did not
> understand that part in your comment).
> 
> To test easily you can use
> "https://covers.openlibrary.org/b/isbn/%isbn%-M.jpg"
> (and not .jpb as written in the test plan, sorry it's a typo!)
> 
> Then the feature will behave the same as OpenLibraryCovers (it will pick the
> same covers).

Gah! As confusingly written as my comment was (and excluding the fact that I
was mistakenly also testing the local cover images), I was simply missing the
-M part of that URL which was stopping it from working.

-- 
You are receiving this mail because:
You are watching all bug changes.