Attachment #91536 for bug #22021

View | Details | Raw Unified | Return to bug 22021
Collapse All | Expand All




                    </td>
                [% END %]
                    <td>
                        [% IF ( itemloo.onloan ) %]
                            <span title="[% itemloo.date_due | html %]" class="checkedout">Due [% itemloo.date_due | $KohaDates  as_due_date => 1 %]</span>
                        [% ELSE %]
                            <span title="0000-00-00">
                                [% IF ( itemloo.transfertwhen ) %]
                                    In transit from [% Branches.GetName( itemloo.transfertfrom ) | html %],
                                    to [% Branches.GetName( itemloo.transfertto ) | html %], since [% itemloo.transfertwhen | html %]
                                [% END %]
                            </span>
                        [% END %]
                    </span>
                [% END %]
                [% IF ( itemloo.message ) %]
                   Unavailable (lost or missing)
                [% END %]

                        [% IF ( itemloo.reservedate ) %]
                            [% IF ( itemloo.nocancel ) %]
                                    Can't be cancelled when item is in transit
                            [% ELSE %]
                            [% IF ( itemloo.waitingdate ) %]Waiting[% ELSE %]On hold[% END %]
                            [% IF ( itemloo.canreservefromotherbranches ) %]for <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% itemloo.ReservedFor.borrowernumber | uri %]">[% itemloo.ReservedFor.firstname | html %] [% itemloo.ReservedFor.surname | html %]</a>[% END %] [% IF ( itemloo.waitingdate ) %]at[% ELSE %]expected at[% END %] [% Branches.GetName( itemloo.ExpectedAtLibrary ) | html %]
                            since
                            [% IF ( itemloo.waitingdate ) %][% itemloo.waitingdate | $KohaDates %][% ELSE %][% IF ( itemloo.reservedate ) %][% itemloo.reservedate | html %][% END %][% END %]. <a class="info" href="modrequest.pl?CancelBiblioNumber=[% itemloo.biblionumber | html %]&amp;CancelBorrowerNumber=[% itemloo.ReservedFor.borrowernumber | html %]&amp;CancelItemnumber=[% itemloo.itemnumber | html %]"  onclick="return confirmDelete(MSG_CONFIRM_DELETE_HOLD);">Cancel hold</a>
                            [% END %]
                        [% ELSE %]
                            Not on hold
                        [% END %]

                        [% IF itemloo.item_level_holds == "N" %]
                            <br/>Item level hold not allowed from OPAC
                        [% ELSIF itemloo.item_level_holds == "F" %]
                            <br/>Item level hold forced from OPAC
                        [% END %]
                    [% IF ( itemloo.canreservefromotherbranches ) %]for <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% itemloo.ReservedFor.borrowernumber | uri %]">[% itemloo.ReservedFor.firstname | html %] [% itemloo.ReservedFor.surname | html %]</a>[% END %] [% IF ( itemloo.waitingdate ) %]at[% ELSE %]expected at[% END %] [% Branches.GetName( itemloo.ExpectedAtLibrary ) | html %]
                    since
                    [% IF ( itemloo.waitingdate ) %][% itemloo.waitingdate | $KohaDates %][% ELSE %][% IF ( itemloo.reservedate ) %][% itemloo.reservedate | html %][% END %][% END %]. <a class="info" href="modrequest.pl?CancelBiblioNumber=[% itemloo.biblionumber | html %]&amp;CancelBorrowerNumber=[% itemloo.ReservedFor.borrowernumber | html %]&amp;CancelItemnumber=[% itemloo.itemnumber | html %]"  onclick="return confirmDelete(MSG_CONFIRM_DELETE_HOLD);">Cancel hold</a>

                        [% IF ( itemloo.itemlost ) %]
                           <span class="lost">[% AuthorisedValues.GetByCode( 'LOST', itemloo.itemlost ) | html %]</span>
                        [% END %]
                [% END %]

                        [% IF ( itemloo.damaged ) %]
                            <span class="dmg">[% AuthorisedValues.GetByCode( 'DAMAGED', itemloo.damaged ) | html %]</span>
                        [% END %]

                        [% IF ( itemloo.notforloan ) %]
                           <span class="nfl">Not for loan ([% AuthorisedValues.GetByCode( 'NOT_LOAN', itemloo.notforloan ) | html %])</span>
                        [% END %]
                [% END %]
                    </td>
                </tr>
            [% END %] <!--UNLESS item hide-->




          }
    };

    my $frameworkcode = GetFrameworkCode( $biblionumber );
    my @notforloan_avs = Koha::AuthorisedValues->search_by_koha_field({ kohafield => 'items.notforloan', frameworkcode => $frameworkcode });
    my $notforloan_label_of = { map { $_->authorised_value => $_->lib } @notforloan_avs };

    my @bibitemloop;

    my @available_itemtypes;

            # Management of the notforloan document
            if ( $item->{notforloan} ) {
                $item->{backgroundcolor} = 'other';
                $item->{notforloanvalue} =
                  $notforloan_label_of->{ $item->{notforloan} };
            }

            # Management of lost or long overdue items
            if ( $item->{itemlost} ) {

                # FIXME localized strings should never be in Perl code
                $item->{message} =
                  $item->{itemlost} == 1 ? "(lost)"
                    : $item->{itemlost} == 2 ? "(long overdue)"
                      : "";
                $item->{backgroundcolor} = 'other';
                if ($logged_in_patron->category->hidelostitems && !$showallitems) {
                    $item->{hide} = 1;
- 

Return to bug 22021

Lines 396-441 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt (-35 / +36 lines)
396	</td>	396	</td>
397	[% END %]	397	[% END %]
398	<td>	398	<td>
399	[% IF ( itemloo.onloan ) %]	399	[% IF ( itemloo.onloan ) %]
400	<span title="[% itemloo.date_due \| html %]" class="checkedout">Due [% itemloo.date_due \| $KohaDates as_due_date => 1 %]</span>	400	<span title="[% itemloo.date_due \| html %]" class="checkedout">Due [% itemloo.date_due \| $KohaDates as_due_date => 1 %]</span>
401	[% ELSE %]	401	[% ELSE %]
402	<span title="0000-00-00">	402	<span title="0000-00-00">
403	[% IF ( itemloo.transfertwhen ) %]	403	[% IF ( itemloo.transfertwhen ) %]
404	In transit from [% Branches.GetName( itemloo.transfertfrom ) \| html %],	404	In transit from [% Branches.GetName( itemloo.transfertfrom ) \| html %],
405	to [% Branches.GetName( itemloo.transfertto ) \| html %], since [% itemloo.transfertwhen \| html %]	405	to [% Branches.GetName( itemloo.transfertto ) \| html %], since [% itemloo.transfertwhen \| html %]
		406	[% END %]
		407	</span>
406	[% END %]	408	[% END %]
407	</span>
408	[% END %]
409	[% IF ( itemloo.message ) %]
410	Unavailable (lost or missing)
411	[% END %]
412		409
413	[% IF ( itemloo.notforloan ) %]	410	[% IF ( itemloo.reservedate ) %]
414	Not for loan ([% itemloo.notforloanvalue \| html %])	411	[% IF ( itemloo.nocancel ) %]
415	[% END %]	412	Can't be cancelled when item is in transit
		413	[% ELSE %]
		414	[% IF ( itemloo.waitingdate ) %]Waiting[% ELSE %]On hold[% END %]
		415	[% IF ( itemloo.canreservefromotherbranches ) %]for <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% itemloo.ReservedFor.borrowernumber \| uri %]">[% itemloo.ReservedFor.firstname \| html %] [% itemloo.ReservedFor.surname \| html %]</a>[% END %] [% IF ( itemloo.waitingdate ) %]at[% ELSE %]expected at[% END %] [% Branches.GetName( itemloo.ExpectedAtLibrary ) \| html %]
		416	since
		417	[% IF ( itemloo.waitingdate ) %][% itemloo.waitingdate \| $KohaDates %][% ELSE %][% IF ( itemloo.reservedate ) %][% itemloo.reservedate \| html %][% END %][% END %]. <a class="info" href="modrequest.pl?CancelBiblioNumber=[% itemloo.biblionumber \| html %]&CancelBorrowerNumber=[% itemloo.ReservedFor.borrowernumber \| html %]&CancelItemnumber=[% itemloo.itemnumber \| html %]" onclick="return confirmDelete(MSG_CONFIRM_DELETE_HOLD);">Cancel hold</a>
		418	[% END %]
		419	[% ELSE %]
		420	Not on hold
		421	[% END %]
416		422
417	[% IF ( itemloo.reservedate ) %]	423	[% IF itemloo.item_level_holds == "N" %]
418	[% IF ( itemloo.nocancel ) %]	424	<br/>Item level hold not allowed from OPAC
419	Can't be cancelled when item is in transit	425	[% ELSIF itemloo.item_level_holds == "F" %]
420	[% ELSE %]	426	<br/>Item level hold forced from OPAC
421	[% IF ( itemloo.waitingdate ) %]Waiting[% ELSE %]On hold[% END %]	427	[% END %]
422	[% IF ( itemloo.canreservefromotherbranches ) %]for <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% itemloo.ReservedFor.borrowernumber \| uri %]">[% itemloo.ReservedFor.firstname \| html %] [% itemloo.ReservedFor.surname \| html %]</a>[% END %] [% IF ( itemloo.waitingdate ) %]at[% ELSE %]expected at[% END %] [% Branches.GetName( itemloo.ExpectedAtLibrary ) \| html %]
423	since
424	[% IF ( itemloo.waitingdate ) %][% itemloo.waitingdate \| $KohaDates %][% ELSE %][% IF ( itemloo.reservedate ) %][% itemloo.reservedate \| html %][% END %][% END %]. <a class="info" href="modrequest.pl?CancelBiblioNumber=[% itemloo.biblionumber \| html %]&CancelBorrowerNumber=[% itemloo.ReservedFor.borrowernumber \| html %]&CancelItemnumber=[% itemloo.itemnumber \| html %]" onclick="return confirmDelete(MSG_CONFIRM_DELETE_HOLD);">Cancel hold</a>
425		428
426	[% END %]	429	[% IF ( itemloo.itemlost ) %]
427	[% ELSE %]	430	<span class="lost">[% AuthorisedValues.GetByCode( 'LOST', itemloo.itemlost ) \| html %]</span>
428	Not on hold	431	[% END %]
429	[% END %]
430		432
431	[% IF itemloo.item_level_holds == "N" %]	433	[% IF ( itemloo.damaged ) %]
432	<br/>Item level hold not allowed from OPAC	434	<span class="dmg">[% AuthorisedValues.GetByCode( 'DAMAGED', itemloo.damaged ) \| html %]</span>
433	[% ELSIF itemloo.item_level_holds == "F" %]	435	[% END %]
434	<br/>Item level hold forced from OPAC	436
435	[% END %]	437	[% IF ( itemloo.notforloan ) %]
436	[% IF ( itemloo.damaged ) %]	438	<span class="nfl">Not for loan ([% AuthorisedValues.GetByCode( 'NOT_LOAN', itemloo.notforloan ) \| html %])</span>
437	<br />Damaged	439	[% END %]
438	[% END %]
439	</td>	440	</td>
440	</tr>	441	</tr>
441	[% END %] <!--UNLESS item hide-->	442	[% END %] <!--UNLESS item hide-->

Lines 346-355 foreach my $biblionumber (@biblionumbers) { Link Here

(-)a/reserve/request.pl (-13 lines)
346	}	346	}
347	};	347	};
348		348
349	my $frameworkcode = GetFrameworkCode( $biblionumber );
350	my @notforloan_avs = Koha::AuthorisedValues->search_by_koha_field({ kohafield => 'items.notforloan', frameworkcode => $frameworkcode });
351	my $notforloan_label_of = { map { $_->authorised_value => $_->lib } @notforloan_avs };
352
353	my @bibitemloop;	349	my @bibitemloop;
354		350
355	my @available_itemtypes;	351	my @available_itemtypes;
Lines 436-453 foreach my $biblionumber (@biblionumbers) { Link Here
436	# Management of the notforloan document	432	# Management of the notforloan document
437	if ( $item->{notforloan} ) {	433	if ( $item->{notforloan} ) {
438	$item->{backgroundcolor} = 'other';	434	$item->{backgroundcolor} = 'other';
439	$item->{notforloanvalue} =
440	$notforloan_label_of->{ $item->{notforloan} };
441	}	435	}
442		436
443	# Management of lost or long overdue items	437	# Management of lost or long overdue items
444	if ( $item->{itemlost} ) {	438	if ( $item->{itemlost} ) {
445
446	# FIXME localized strings should never be in Perl code
447	$item->{message} =
448	$item->{itemlost} == 1 ? "(lost)"
449	: $item->{itemlost} == 2 ? "(long overdue)"
450	: "";
451	$item->{backgroundcolor} = 'other';	439	$item->{backgroundcolor} = 'other';
452	if ($logged_in_patron->category->hidelostitems && !$showallitems) {	440	if ($logged_in_patron->category->hidelostitems && !$showallitems) {
453	$item->{hide} = 1;	441	$item->{hide} = 1;
454	-