[Koha-bugs] [Bug 7709] New: Cover Images Template plugin
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue Mar 13 20:32:52 CET 2012
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7709
Priority: P5 - low
Change sponsored?: ---
Bug ID: 7709
Assignee: oleonard at myacpl.org
Summary: Cover Images Template plugin
QA Contact: koha.sekjal at gmail.com
Severity: enhancement
Classification: Unclassified
OS: All
Reporter: koha.sekjal at gmail.com
Hardware: All
Status: NEW
Version: master
Component: Templates
Product: Koha
As we add more and more sources for cover images, and use them in more and more
places, it becomes increasingly difficult to keep the code synchronized. I
propose we move the logic of picking and displaying a cover image out of the
templates and into a T:T Plugin.
Usage would be like:
[% biblionumber | KohaCoverImage %]
I say biblionumber instead of normalised_isbn (which is common for most all
external services) because it will also work for local cover images. This does
add the overhead of having to look up the normalized ISBN again, even if we
already had it on the page. Anyone more familiar with passing variables
(particularly hashes) to T:T plugins, would be welcome to step up here.
This would simplify out templates by several dozen lines of code, and provide a
uniform logic for selection. New selection methods, as well as new sources,
would be easy to add, since they'd only need to be coded in one place.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list