Bug 6887 - Show "checked out from" column on current checkouts
Summary: Show "checked out from" column on current checkouts
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Ian Walls
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-20 15:08 UTC by Ian Walls
Modified: 2013-12-05 19:59 UTC (History)
3 users (show)

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


Attachments
[SIGNED-OFF] Enh 6887: Add 'checked out from' column to issued items display tables (10.03 KB, patch)
2011-09-20 16:07 UTC, Nicole C. Engard
Details | Diff | Splinter Review
[Signed Off] Enh 6887: Add 'checked out from' column to issued items display tables (12.29 KB, patch)
2011-09-20 19:49 UTC, Liz Rea
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Walls 2011-09-20 15:08:41 UTC
Add a "Checked Out From" column to the right of the "Checked Out On" column, on
both the Patron Checkout screen and Patron details screen, which displays the
HomeOrHoldingBranch branchname (either home or holding, depending), and have
that column be sortable.
Comment 1 Nicole C. Engard 2011-09-20 16:07:25 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2011-09-20 16:24:08 UTC
Tablesorter columns need to be adjusted.
Comment 3 Liz Rea 2011-09-20 19:49:11 UTC
Created attachment 5482 [details] [review]
[Signed Off] Enh 6887: Add 'checked out from' column to issued items display tables

Adds "checked out from" column to the right of "checked out on" on both
circ/circulation.pl and members/moremembers.pl.  Columns are sortable with tablesort
(though only on circ/circulation.pl if syspref enabled).

The branch name is sourced from issues.branchcode.

EDIT BY Owen Leonard: Adjusting table columns which are sorted by default
to accommodate the new column.

Things look good! Thanks!

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Liz Rea <lrea@nekls.org>
Comment 4 Ian Walls 2011-09-20 20:12:08 UTC
Now that more eyes have had a chance to review this (thanks for catching the tablesorter error, Owen!), I feel more comfortable marking it as Passed QA.
Comment 5 Chris Cormack 2011-09-20 21:49:01 UTC
Pushed, please test
Comment 6 Jared Camins-Esakov 2012-05-23 12:45:19 UTC
Seems to be working. Closing.