Bug 5467

Summary: LDR pos 6 o = Kit should not show as visual material
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: OPACAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: trivial    
Priority: PATCH-Sent (DO NOT USE) CC: chris, jwagner
Version: rel_3_2   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: proposed patch

Description Katrin Fischer 2010-12-03 13:37:21 UTC
One of my libaries protested, because a language course (books and audio cds) showed up in OPAC as "visual material". The icon indicating that it was including some kind of film.

I looked at the XSLT stylesheets and I think kit would be better grouped together with mixed materials instead of visual materials.

Or perhaps use a totally different icon?

<xsl:when test="$leader6='g' or $leader6='k' or $leader6='o' or $leader6='r'">
<img src="/opac-tmpl/prog/famfamfam/silk/film.png" alt="visual material" title="visual material"/>
 Visual Material
</xsl:when>

<xsl:when test="$leader6='p'">
<img src="/opac-tmpl/prog/famfamfam/silk/report_disk.png" alt="mixed materials" title="mixed materials"/>
Mixed Materials
</xsl:when>

06 - Type of record
a - Language material
c - Notated music
d - Manuscript notated music
e - Cartographic material
f - Manuscript cartographic material
g - Projected medium
i - Nonmusical sound recording
j - Musical sound recording
k - Two-dimensional nonprojectable graphic
m - Computer file
o - Kit
p - Mixed materials
r - Three-dimensional artifact or naturally occurring object
t - Manuscript language material
Comment 1 Katrin Fischer 2010-12-06 13:18:09 UTC
Created attachment 2805 [details] [review]
proposed patch

Patch adds kit as description using the icon for mixed materials.

The mixed material icon referenced in opac result was not existing for detail pages of opac and intranet. Also added the missing file MX.png.
Comment 2 Chris Cormack 2010-12-11 18:54:57 UTC
Patch pushed please test
Comment 3 Katrin Fischer 2010-12-12 10:51:59 UTC
Successfully tested in HEAD.