Bug 4837 - Circulation Print Page uses incorrect heading / incorrect information
Summary: Circulation Print Page uses incorrect heading / incorrect information
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: unspecified
Hardware: PC Other
: P5 - low normal (vote)
Assignee: Guillaume Hatt
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-28 21:47 UTC by Christopher Brannon
Modified: 2013-12-05 20:01 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Screen shot of Print Page error (33.28 KB, image/jpeg)
2010-05-28 21:47 UTC, Christopher Brannon
Details
patch (1.21 KB, patch)
2010-06-02 22:55 UTC, Nicole C. Engard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Christopher Brannon 2010-05-28 21:47:00 UTC
Created attachment 2188 [details]
Screen shot of Print Page error

When using the print page feature in circulation, the column heading of "Format" is displayed over a column that appears to be Item Type.

Also, the information in the column is incorrect.  It is displaying the wrong type for the item being displayed.
Comment 1 Liz Rea 2010-05-28 21:52:59 UTC
This also seems to be broken in Master:

See:
 http://screencast.com/t/Y2U4ZTdhNzct Which shows the print page

http://screencast.com/t/MzNjZjYyZDYt Which shows that the bib level itype is AB

It looks like it's pulling the bib level itype instead of either the actual format or the item-level itype (Koha itype).
Comment 2 Nicole C. Engard 2010-06-02 22:55:28 UTC
Created attachment 2204 [details] [review]
patch
Comment 3 Owen Leonard 2010-12-14 14:15:49 UTC
Instead of showing itemtype by default, might it be better to show either itemtype or ccode depending on the value of the AdvancedSearchTypes system pref?
Comment 4 Nicole C. Engard 2011-03-04 15:15:24 UTC
That works, I'm not sure how to do it so you can feel free to post a follow up patch or a new patch altogether.
Comment 5 Chris Cormack 2011-04-06 07:36:42 UTC
Pushed please test