[Koha-bugs] [Bug 25007] AmazonCoverImages doesnt check for ISBN in details.tt

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Sun Mar 29 18:09:06 CEST 2020


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

--- Comment #7 from Owen Leonard <oleonard at myacpl.org> ---
Created attachment 102057
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102057&action=edit
Bug 25007: (follow-up) Fix the way image check alters layout

A question in the bug report asks: Why is is necessary to change the
Bootstrap class to col-xs-12? The JavaScript which checks for the
existence of an Amazon cover image is designed to remove the Amazon
cover's container if there is none. Changing the class of
cover image column is gone.

However, the check is incorrect because it doesn't account for Local
Covers. There might be both a 1 x 1 pixel Amazon image and a local cover
image, so the layout shouldn't be changed unless there are no images
present.

To test, apply the patch and view the bibliographic details page of
various titles including:
- Amazon cover present, no local cover
- No Amazon cover, local cover image present
- Both Amazon and local cover image

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


More information about the Koha-bugs mailing list