Bug 34547: Add transfer reason to checkedintable on returns.tt

To test:
1. Apply patch, restart_all
2. Check some items out and prepare some reasons that would initiate a branchtransfer. According the schema it can be any of the following: 'Manual', 'StockrotationAdvance', 'StockrotationRepatriation', 'ReturnToHome', 'ReturnToHolding', 'RotatingCollection', 'Reserve', 'LostReserve', 'CancelReserve', 'TransferCancellation', 'Recall', 'RecallCancellation.
3. CHeck in some the items and notice the 'Transfer reason' column in the table. Make sure the reasons are acurate and make sense.
4. To go Table settings and find the 'checkedintable'. Make sure you can properly hide the column from the display.

Signed-off-by: AndrewA <andrew.auld@ptfs-europe.com>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Lucas Gass 2023-08-16 21:20:36 +00:00 committed by Tomas Cohen Arazi
parent 5282a9c4ce
commit f25fa87198
No known key found for this signature in database
GPG Key ID: 0A272EA1B2F3C15F
3 changed files with 18 additions and 0 deletions

View File

@ -1462,6 +1462,8 @@ modules:
columnname: homelibrary
-
columnname: transferlibrary
-
columnname: transferreason
-
columnname: location
-

View File

@ -812,6 +812,7 @@ foreach ( sort { $a <=> $b } keys %returneditems ) {
$ri{homebranch} = $item->homebranch;
$ri{transferbranch} = $item->get_transfer ? $item->get_transfer->tobranch : '';
$ri{damaged} = $item->damaged;
$ri{transferreason} = $item->get_transfer ? $item->get_transfer->reason : '';
$ri{location} = $item->location;
my $shelfcode = $ri{'location'};

View File

@ -1206,6 +1206,7 @@
<th class="ci-barcode">Barcode</th>
<th class="ci-homelibrary">Home library</th>
<th class="ci-transferlibrary">Transfer to</th>
<th class="ci-transferreason">Transfer reason</th>
<th class="ci-shelvinglocation">Shelving location</th>
<th class="ci-callnumber">Call number</th>
<th class="ci-dateaccessioned">Date acquired</th>
@ -1255,6 +1256,20 @@
<td class="ci-transferlibrary">
[% Branches.GetName( riloo.transferbranch ) | html %]
</td>
<td class="ci-transferreason">
[%- SWITCH riloo.transferreason -%]
[%- CASE 'Manual' -%]<span>Manual</span>
[%- CASE 'StockrotationAdvance' -%]<span>Stock rotation advance</span>
[%- CASE 'StockrotationRepatriation' -%]<span>Stock rotation repatriation</span>
[%- CASE 'ReturnToHome' -%]<span>Return to home library</span>
[%- CASE 'ReturnToHolding' -%]<span>Return to holding library</span>
[%- CASE 'RotatingCollection' -%]<span>Rotating collection</span>
[%- CASE 'Reserve' -%]<span>Hold</span>
[%- CASE 'LostReserve' -%]<span>Lost hold</span>
[%- CASE 'CancelReserve' -%]<span>Cancelled hold</span>
[%- CASE 'TransferCancellation' -%]<span>Transfer was cancelled whilst in transit</span>
[%- END -%]
</td>
<td class="ci-shelvinglocation">
<span class="shelvingloc">[% riloo.location | html %]</span>
</td>