From b349fa10017e1dc397aeb32502296c7fd7e3c741 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Tue, 13 Aug 2019 15:47:08 +0000 Subject: [PATCH] Bug 23458: Clean up holds template in the staff client This patch reindents reserve/request.tt to use four spaces and to eliminate tabs, following coding guidelines. Trailing white space has been removed. To test, apply the patch and use your preferred method for checking the differences between files while ignoring whitespace. I use diff with the "-w" flag, but I'm no diffctator. Test the process of placing a hold in the staff client: A single hold, multiple holds, etc. Everything should work correctly. Signed-off-by: Maryse Simard Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize --- .../prog/en/modules/reserve/request.tt | 1193 +++++++++-------- 1 file changed, 622 insertions(+), 571 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt index 96fc32d7a4..d218ae5afe 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -22,9 +22,18 @@ [% INCLUDE 'circ-search.inc' %] [% UNLESS ( multi_hold ) %] - + [% ELSE %] - + [% END %]
@@ -34,639 +43,681 @@ [% ELSE %]
[% END %] -
- -[% IF ( noitems ) %] -
- [%IF (multi_hold) %] - Cannot place hold: one or more records without items attached. - [% ELSE %] - Cannot place hold: this record has no items attached. - [% END %] -
-[% END %] - - [% IF ( messagetransfert ) %] -
-

Hold found for ([% nextreservtitle | html %]), please transfer

-

Hold placed by : [% nextreservsurname | html %] [% nextreservfirstname | html %] at : [% branchname | html %] , Please transfer this item. -

-
- -
-
- [% END %] - - [% UNLESS ( multi_hold ) %] -

Place a hold on [% INCLUDE 'biblio-default-view.inc' %][% INCLUDE 'biblio-title.inc' %]

- [% ELSE %] -

Confirm holds

- [% END %] - - [% UNLESS patron OR patron.borrowernumber OR noitems %] - [% IF ( messageborrower ) %] -

Patron not found

No patron with this name, please, try another

- [% END %] -
-
- - [% UNLESS borrowers %] - -
Enter patron card number or partial name:
- - - [% IF multi_hold %] - - - [% ELSE %] - - [% END %] -
- [% ELSE %] - [% INCLUDE 'circ-patron-search-results.inc' destination = "holds" %] - - [% END %] - [% IF ( multi_hold ) %] - - - [% END %] -
- [% ELSIF NOT noitems %] - -[% IF ( checked_previously && !multi_hold ) %] -
-
    -
  • Patron has previously checked out this title
    • -
-
-[% END %] +
-[% IF ( exceeded_maxreserves || exceeded_holds_per_record || alreadyreserved || none_available || alreadypossession || ageRestricted ) %] -
- - [% UNLESS ( multi_hold ) %] -

Cannot place hold

- - [% ELSE %] -

Cannot place hold on some items

- [% IF ( exceeded_maxreserves ) %] -
  • Too many holds: [% patron.firstname | html %] [% patron.surname | html %] can place [% new_reserves_allowed | html %] of the requested [% new_reserves_count | html %] holds for a maximum of [% maxreserves | html %] total holds.
    • - [% ELSIF ( exceeded_holds_per_record ) %] - [% FOREACH biblioloo IN biblioloop %] - [% IF (biblioloo.tooManyHoldsForThisRecord) %] -
  • Too many holds for [% biblioloo.title | html %]: [% patron.firstname | html %] [% patron.surname | html %] can only place a maximum of [% max_holds_for_record | html %] hold(s) on this record.
    • + [% IF ( noitems ) %] +
    + [%IF (multi_hold) %] + Cannot place hold: one or more records without items attached. + [% ELSE %] + Cannot place hold: this record has no items attached. [% END %] +
    [% END %] - [% END %] - [% END %] - -
    -[% END %] - -[% IF ( expiry || diffbranch || patron.is_debarred || ( amount_outstanding && Koha.Preference('maxoutstanding') && amount_outstanding > Koha.Preference('maxoutstanding') ) ) %] -
    -[% END %] - - [% IF ( messageborrower ) %] -

    Patron not found:

    Name or barcode not found. Please try an other

    - [% END %] - -
    - Hold details -
    - - - - - [% IF ( multi_hold ) %] - - - - - [% FOREACH biblioloo IN biblioloop %] - - + [% IF ( messagetransfert ) %] +
    +

    Hold found for ([% nextreservtitle | html %]), please transfer

    +

    Hold placed by : [% nextreservsurname | html %] [% nextreservfirstname | html %] at : [% branchname | html %] , Please transfer this item. +

    + + + +
    [% END %] - [% ELSE %] - - - - [% END %] -
    1. Patron: - [% IF ( patron.borrowernumber ) %] - [% patron.firstname | html %] [% patron.surname | html %] ([% patron.cardnumber | html %]) - [% ELSE %] - Not defined yet - [% END %] -
      2. - [% UNLESS ( multi_hold ) %] -
    2. - Estimated priority: - [% fixedRank | html %] -
      3. - [% END %] -
    3. - - -
      4. -
    4. - - -
      5. - - [% UNLESS ( multi_hold ) %] - [% IF Koha.Preference('AllowHoldItemTypeSelection') %] -
    5. - - -
      6. - [% END %] - [% END %] - - [% IF ( reserve_in_future ) %] -
    6. - - - Clear date -
      7. - [% END %] - -
    7. - - - Clear date -
      8. - - [% UNLESS ( multi_hold ) %] -
    8. - [% IF force_hold_level == 'item' %] - - [% ELSIF force_hold_level == 'record' %] - - - [% ELSE %] - - [% END %] - - -
      9. - - [% IF remaining_holds_for_record > 1 %] -
    9. - - -
      10. + [% UNLESS ( multi_hold ) %] +

      Place a hold on [% INCLUDE 'biblio-default-view.inc' %][% INCLUDE 'biblio-title.inc' %]

      [% ELSE %] - +

      Confirm holds

      [% END %] - [% END %] -
    - [% UNLESS ( multi_hold ) %] -
    - [% IF ( patron.borrowernumber ) %] - [% IF ( override_required ) %] - - [% ELSIF ( none_available ) %] - - [% ELSE %] - + [% UNLESS patron OR patron.borrowernumber OR noitems %] + [% IF ( messageborrower ) %] +
    +

    Patron not found

    +

    No patron with this name, please, try another

    +
    [% END %] - [% END %] -
    - [% FOREACH bibitemloo IN bibitemloop %] -
      - [% UNLESS ( item_level_itypes ) %] -
    1. Item type: [% bibitemloo.description | html %]
      2. - [% END %] - [% IF ( bibitemloo.publicationyear ) %]
    2. Publication year: [% bibitemloo.publicationyear | html %]
      3. [% END %] -
    - -

    - Place a hold on a specific item - [% IF bibitemloo.force_hold_level == 'item' %] - (Required) - [% END %] -

    - - - - - [% IF ( item_level_itypes ) %] - - [% END %] - - - - [% IF itemdata_ccode %] - - [% END %] - - - [% IF itemdata_enumchron %] - - [% END %] - - - - - [% SET selected = 0 %] - [% FOREACH itemloo IN bibitemloo.itemloop %] - [% UNLESS ( itemloo.hide ) %] - - - [% IF ( item_level_itypes ) %] - - [% END %] + - - - - [% IF itemdata_ccode %] - - [% END %] - - - [% IF itemdata_enumchron %] - + [% ELSIF NOT noitems %] + + [% IF ( checked_previously && !multi_hold ) %] +
    +
      +
    • Patron has previously checked out this title
      • +
    +
    [% END %] - - - [% END %] - [% END %] - -
    HoldItem typeBarcodeHome libraryLast locationCollectionCall no.Copy numberVol no.Information
    - [% IF itemloo.force_hold_level == 'record' # Patron has placed a record level hold previously for this record %] - - - Hold must be record level - - [% ELSIF ( itemloo.available ) %] - - [% ELSIF ( itemloo.override ) %] - - - [% ELSE %] - - - [% IF itemloo.not_holdable %] - [% IF itemloo.not_holdable == 'damaged' %] - Item damaged - [% ELSIF itemloo.not_holdable == 'ageRestricted' %] - Age restricted - [% ELSIF itemloo.not_holdable == 'tooManyHoldsForThisRecord' %] - Exceeded max holds per record - [% ELSIF itemloo.not_holdable == 'tooManyReservesToday' %] - Daily hold limit reached for patron - [% ELSIF itemloo.not_holdable == 'tooManyReserves' %] - Too many holds - [% ELSIF itemloo.not_holdable == 'notReservable' %] - Not holdable - [% ELSIF itemloo.not_holdable == 'cannotReserveFromOtherBranches' %] - Patron is from different library - [% ELSIF itemloo.not_holdable == 'itemAlreadyOnHold' %] - Patron already has hold for this item - [% ELSIF itemloo.not_holdable == 'cannotBeTransferred' %] - Cannot be transferred to pickup library +
    +
    + [% UNLESS borrowers %] + +
    Enter patron card number or partial name:
    + + + [% IF multi_hold %] + + [% ELSE %] - [% itemloo.not_holdable | html %] + [% END %] + [% ELSE %] + [% INCLUDE 'circ-patron-search-results.inc' destination = "holds" %] [% END %] - - [% END %] -
    		- [% UNLESS ( noItemTypeImages ) %] - [% IF ( itemloo.imageurl ) %]
    [% END %] + + + [% IF ( multi_hold ) %] + + [% END %] - [% itemloo.itypename | html %] -     		- [% itemloo.barcode | html %] - - [% Branches.GetName( itemloo.homebranch ) | html %] - - [% Branches.GetName( itemloo.holdingbranch ) | html %] - - [% IF ( itemloo.ccode ) %][% AuthorisedValues.GetByCode( 'CCODE', itemloo.ccode ) | html %][% END %] - - [% itemloo.itemcallnumber | html %] - - [% IF ( itemloo.copynumber ) %][% itemloo.copynumber | html %][% ELSE %] [% END %] - - [% itemloo.enumchron | html %] - - [% IF ( itemloo.onloan ) %] - Due [% itemloo.date_due | $KohaDates as_due_date => 1 %] + + [% IF ( exceeded_maxreserves || exceeded_holds_per_record || alreadyreserved || none_available || alreadypossession || ageRestricted ) %] +
    + + [% UNLESS ( multi_hold ) %] +

    Cannot place hold

    + [% ELSE %] - - [% IF ( itemloo.transfertwhen ) %] - In transit from [% Branches.GetName( itemloo.transfertfrom ) | html %], - to [% Branches.GetName( itemloo.transfertto ) | html %], since [% itemloo.transfertwhen | html %] +

    Cannot place hold on some items

    + [% IF ( exceeded_maxreserves ) %] +
  • Too many holds: [% patron.firstname | html %] [% patron.surname | html %] can place [% new_reserves_allowed | html %] of the requested [% new_reserves_count | html %] holds for a maximum of [% maxreserves | html %] total holds.
    • + [% ELSIF ( exceeded_holds_per_record ) %] + [% FOREACH biblioloo IN biblioloop %] + [% IF (biblioloo.tooManyHoldsForThisRecord) %] +
  • Too many holds for [% biblioloo.title | html %]: [% patron.firstname | html %] [% patron.surname | html %] can only place a maximum of [% max_holds_for_record | html %] hold(s) on this record.
    • + [% END %] [% END %] -     - [% 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 [% itemloo.ReservedFor.firstname | html %] [% itemloo.ReservedFor.surname | html %][% 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 %]. Cancel hold [% END %] - [% ELSE %] - Not on hold - [% END %] - - [% IF itemloo.item_level_holds == "N" %] -
    Item level hold not allowed from OPAC - [% ELSIF itemloo.item_level_holds == "F" %] -
    Item level hold forced from OPAC - [% END %] - - [% IF ( itemloo.itemlost ) %] - [% AuthorisedValues.GetByCode( 'LOST', itemloo.itemlost ) | html %] - [% END %] - - [% IF ( itemloo.damaged ) %] - [% AuthorisedValues.GetByCode( 'DAMAGED', itemloo.damaged ) | html %] [% END %] +
    + [% END %] - [% IF ( itemloo.notforloan ) %] - Not for loan ([% AuthorisedValues.GetByCode( 'NOT_LOAN', itemloo.notforloan ) | html %]) - [% END %] -
    - [% IF ( bibitemloo.hiddencount ) %] -
    -

    Show all items ([% bibitemloo.hiddencount | html %] hidden)

    -
    - [% END %] - [% END %] - - [% ELSE %] - - - - - - [% UNLESS ( item_level_itypes ) %] - - [% END %] - - - - [% FOREACH biblioloo IN biblioloop %] - [% IF ( biblioloo.warn ) %] - - [% ELSE %] - - [% END %] - - [% END %] - - [% UNLESS ( item_level_itypes ) %] - - [% END %] - - - - [% END %] -
     TitleItem typePriorityInformation
    - [% UNLESS ( biblioloo.warn ) %] - -
      -
    • [% biblioloo.title | html %]
      • - [% IF ( biblioloo.publicationyear ) %] -
    • Publication year: [% biblioloo.publicationyear | html %]
      • - [% END %] -
    - [% IF ( biblioloo.warn ) %] - - [% END %] -     		- [% biblioloo.itypename | html %] - [% biblioloo.rank | html %] - [% IF ( biblioloo.checked_previously ) %] - Patron has previously checked out this title
    - [% END %] - [% IF ( biblioloo.alreadyres ) %] -
    + [% IF ( diffbranch ) %] +
  • Pickup library is different. Patron: [% patron.firstname | html %] [% patron.surname | html %] Patron's home library: ([% Branches.GetName(patron.branchcode) | html %] / [% patron.branchcode | html %] )
    • + [% END %] - [% END %] + +
    + [% END %] -
    - [% IF ( patron AND patron.borrowernumber ) %] - [% IF ( override_required ) %] - - [% ELSIF ( none_available ) %] - - [% ELSE %] - [% IF ( multi_hold ) %] - - [% ELSE %] - + [% IF ( messageborrower ) %] +
    +

    Patron not found:

    +

    Name or barcode not found. Please try an other

    +
    [% END %] - [% END %] - [% END %] -
    - - -[% END %] -[% UNLESS ( patron ) %] - [% IF ( reserveloop ) %] -
    - [% IF ( multi_hold ) %] - - - [% END %] +
    + Hold details + -
    - Existing holds + + - [% FOREACH biblioloo IN biblioloop %] - [% IF ( biblioloo.reserveloop ) %] [% IF ( multi_hold ) %] -

    - - [% biblioloo.title | html %] - -

    + + + + + [% FOREACH biblioloo IN biblioloop %] + + + [% END %] + [% ELSE %] + + + [% END %] - [% IF Koha.Preference('HoldsSplitQueue') == 'branch' %] - [% SET branchcodes = [] %] +
      +
    1. + Patron: + [% IF ( patron.borrowernumber ) %] + [% patron.firstname | html %] [% patron.surname | html %] ([% patron.cardnumber | html %]) + [% ELSE %] + Not defined yet + [% END %] +
      2. - [% FOREACH h IN biblioloo.reserveloop %] - [% branchcodes.push( h.branchcode ) %] + [% UNLESS ( multi_hold ) %] +
    2. + Estimated priority: + [% fixedRank | html %] +
      3. [% END %] - [% branchcodes = branchcodes.unique %] - [% FOREACH b IN branchcodes.sort %] - [% SET holds_by_branch = [] %] - [% FOREACH h IN biblioloo.reserveloop %] - [% IF h.branchcode == b %] - [% holds_by_branch.push( h ) %] - [% END %] +
    3. + + +
      4. +
    4. + + +
      5. + + [% UNLESS ( multi_hold ) %] + [% IF Koha.Preference('AllowHoldItemTypeSelection') %] +
    5. + + +
      6. [% END %] -
      - [% Branches.GetName( b ) | html %] - [% INCLUDE holds_table.inc holds=holds_by_branch %] -
      [% END %] - [% ELSIF Koha.Preference('HoldsSplitQueue') == 'itemtype' %] - [% SET itemtypes = [] %] - [% FOREACH h IN biblioloo.reserveloop %] - [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] - [% itemtypes.push( hold_itemtype ) %] + [% IF ( reserve_in_future ) %] +
    6. + + + Clear date +
      7. [% END %] - [% itemtypes = itemtypes.unique %] - - [% FOREACH i IN itemtypes.sort %] - [% SET holds_by_itemtype = [] %] - [% FOREACH h IN biblioloo.reserveloop %] - [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] - [% IF hold_itemtype == i %] - [% holds_by_itemtype.push( h ) %] - [% END %] - [% END %] -
      - [% IF i %] - [% ItemTypes.GetDescription( i ) | html %] +
    7. + + + Clear date +
      8. + + [% UNLESS ( multi_hold ) %] +
    8. + + [% IF force_hold_level == 'item' %] + + [% ELSIF force_hold_level == 'record' %] + + [% ELSE %] - Any item type + [% END %] - [% INCLUDE holds_table.inc holds=holds_by_itemtype %] -
      9. + + + + + [% IF remaining_holds_for_record > 1 %] +
    9. + + +
      10. + [% ELSE %] + + [% END %] [% END %] - [% ELSIF Koha.Preference('HoldsSplitQueue') == 'branch_itemtype' %] - [% SET branchcodes = [] %] +
    + + [% UNLESS ( multi_hold ) %] +
    + [% IF ( patron.borrowernumber ) %] + [% IF ( override_required ) %] + + [% ELSIF ( none_available ) %] + + [% ELSE %] + + [% END %] + [% END %] +
    + + [% FOREACH bibitemloo IN bibitemloop %] +
      + [% UNLESS ( item_level_itypes ) %] +
    1. + Item type: + [% bibitemloo.description | html %] +
      2. + [% END %] - [% FOREACH h IN biblioloo.reserveloop %] - [% branchcodes.push( h.branchcode ) %] - [% END %] - [% branchcodes = branchcodes.unique %] - - [% FOREACH b IN branchcodes.sort %] -
      - [% Branches.GetName( b ) | html %] - [% SET holds_by_branch = [] %] - [% FOREACH h IN biblioloo.reserveloop %] - [% IF h.branchcode == b %] - [% holds_by_branch.push( h ) %] - [% END %] + [% IF ( bibitemloo.publicationyear ) %] +
    2. + Publication year: + [% bibitemloo.publicationyear | html %] +
      3. [% END %] +
    - [% SET itemtypes = [] %] - [% FOREACH h IN holds_by_branch %] - [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] - [% itemtypes.push( hold_itemtype ) %] +

    + Place a hold on a specific item + [% IF bibitemloo.force_hold_level == 'item' %] + (Required) [% END %] - [% itemtypes = itemtypes.unique %] +

    + + + + + + [% IF ( item_level_itypes ) %] + + [% END %] + + + + [% IF itemdata_ccode %] + + [% END %] + + + [% IF itemdata_enumchron %] + + [% END %] + + + + + [% SET selected = 0 %] + [% FOREACH itemloo IN bibitemloo.itemloop %] + [% UNLESS ( itemloo.hide ) %] + + + [% IF ( item_level_itypes ) %] + + [% END %] + + + + [% IF itemdata_ccode %] + + [% END %] + + + [% IF itemdata_enumchron %] + + [% END %] + + + [% END %] + [% END %] + +
    HoldItem typeBarcodeHome libraryLast locationCollectionCall no.Copy numberVol no.Information
    + [% IF itemloo.force_hold_level == 'record' # Patron has placed a record level hold previously for this record %] + + + Hold must be record level + + [% ELSIF ( itemloo.available ) %] + + [% ELSIF ( itemloo.override ) %] + + + [% ELSE %] + + + [% IF itemloo.not_holdable %] + [% IF itemloo.not_holdable == 'damaged' %] + Item damaged + [% ELSIF itemloo.not_holdable == 'ageRestricted' %] + Age restricted + [% ELSIF itemloo.not_holdable == 'tooManyHoldsForThisRecord' %] + Exceeded max holds per record + [% ELSIF itemloo.not_holdable == 'tooManyReservesToday' %] + Daily hold limit reached for patron + [% ELSIF itemloo.not_holdable == 'tooManyReserves' %] + Too many holds + [% ELSIF itemloo.not_holdable == 'notReservable' %] + Not holdable + [% ELSIF itemloo.not_holdable == 'cannotReserveFromOtherBranches' %] + Patron is from different library + [% ELSIF itemloo.not_holdable == 'itemAlreadyOnHold' %] + Patron already has hold for this item + [% ELSIF itemloo.not_holdable == 'cannotBeTransferred' %] + Cannot be transferred to pickup library + [% ELSE %] + [% itemloo.not_holdable | html %] + [% END %] + [% END %] + + [% END %] + + [% UNLESS ( noItemTypeImages ) %] + [% IF ( itemloo.imageurl ) %]
    [% END %] + [% END %] + [% itemloo.itypename | html %] +     		+ [% itemloo.barcode | html %] + + [% Branches.GetName( itemloo.homebranch ) | html %] + + [% Branches.GetName( itemloo.holdingbranch ) | html %] + + [% IF ( itemloo.ccode ) %][% AuthorisedValues.GetByCode( 'CCODE', itemloo.ccode ) | html %][% END %] + + [% itemloo.itemcallnumber | html %] + + [% IF ( itemloo.copynumber ) %][% itemloo.copynumber | html %][% ELSE %] [% END %] + + [% itemloo.enumchron | html %] + + [% IF ( itemloo.onloan ) %] + Due [% itemloo.date_due | $KohaDates as_due_date => 1 %] + [% ELSE %] + + [% IF ( itemloo.transfertwhen ) %] + In transit from [% Branches.GetName( itemloo.transfertfrom ) | html %], + to [% Branches.GetName( itemloo.transfertto ) | html %], since [% itemloo.transfertwhen | html %] + [% END %] + + [% 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 [% itemloo.ReservedFor.firstname | html %] [% itemloo.ReservedFor.surname | html %] + [% 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 %]. + Cancel hold + [% END %] + [% ELSE %] + Not on hold + [% END %] + + [% IF itemloo.item_level_holds == "N" %] +
    Item level hold not allowed from OPAC + [% ELSIF itemloo.item_level_holds == "F" %] +
    Item level hold forced from OPAC + [% END %] + + [% IF ( itemloo.itemlost ) %] + [% AuthorisedValues.GetByCode( 'LOST', itemloo.itemlost ) | html %] + [% END %] + + [% IF ( itemloo.damaged ) %] + [% AuthorisedValues.GetByCode( 'DAMAGED', itemloo.damaged ) | html %] + [% END %] + + [% IF ( itemloo.notforloan ) %] + Not for loan ([% AuthorisedValues.GetByCode( 'NOT_LOAN', itemloo.notforloan ) | html %]) + [% END %] +
    + + [% IF ( bibitemloo.hiddencount ) %] + +

    + Show all items ([% bibitemloo.hiddencount | html %] hidden) +

    + + [% END %] + [% END %] + + [% ELSE %] + + + + + + [% UNLESS ( item_level_itypes ) %] + + [% END %] + + + + [% FOREACH biblioloo IN biblioloop %] + [% IF ( biblioloo.warn ) %] + + [% ELSE %] + + [% END %] + + [% END %] + + [% UNLESS ( item_level_itypes ) %] + + [% END %] + + + + [% END %] +
     TitleItem typePriorityInformation
    + [% UNLESS ( biblioloo.warn ) %] + + +
      +
    • + [% biblioloo.title | html %] +
      • + [% IF ( biblioloo.publicationyear ) %] +
    • + Publication year: [% biblioloo.publicationyear | html %] +
      • + [% END %] +
    + [% IF ( biblioloo.warn ) %] + + [% END %] +     		+ [% biblioloo.itypename | html %] + [% biblioloo.rank | html %] + [% IF ( biblioloo.checked_previously ) %] + Patron has previously checked out this title
    + [% END %] + [% IF ( biblioloo.alreadyres ) %] +
      + [% ELSE %] + [% IF ( biblioloo.none_avail ) %] +
        + [% END %] + [% END %] - [% FOREACH i IN itemtypes.sort %] -
        - [% IF i %] - [% ItemTypes.GetDescription( i ) | html %] + [% IF ( biblioloo.alreadyres ) %] +
      • + [% patron.firstname | html %] [% patron.surname | html %] + already has a hold on this item +
        • + [% END %] + [% IF ( biblioloo.none_avail ) %] +
      • No items are available to be placed on hold
        • + [% END %] + + [% IF ( biblioloo.alreadyres ) %] +
      [% ELSE %] - Any item type + [% IF ( biblioloo.none_avail ) %] +
    + [% END %] [% END %] +
    + + [% END %] + +
    + [% IF ( patron AND patron.borrowernumber ) %] + [% IF ( override_required ) %] + + [% ELSIF ( none_available ) %] + + [% ELSE %] + [% IF ( multi_hold ) %] + + [% ELSE %] + + [% END %] + [% END %] + [% END %] +
    + +
    + [% END %] + + [% UNLESS ( patron ) %] + [% IF ( reserveloop ) %] +
    + [% IF ( multi_hold ) %] + + + [% END %] + +
    + Existing holds + + [% FOREACH biblioloo IN biblioloop %] + [% IF ( biblioloo.reserveloop ) %] + [% IF ( multi_hold ) %] +

    + + [% biblioloo.title | html %] + +

    + [% END %] + + [% IF Koha.Preference('HoldsSplitQueue') == 'branch' %] + [% SET branchcodes = [] %] + + [% FOREACH h IN biblioloo.reserveloop %] + [% branchcodes.push( h.branchcode ) %] + [% END %] + [% branchcodes = branchcodes.unique %] + + [% FOREACH b IN branchcodes.sort %] + [% SET holds_by_branch = [] %] + [% FOREACH h IN biblioloo.reserveloop %] + [% IF h.branchcode == b %] + [% holds_by_branch.push( h ) %] + [% END %] + [% END %] +
    + [% Branches.GetName( b ) | html %] + [% INCLUDE holds_table.inc holds=holds_by_branch %] +
    + [% END %] + [% ELSIF Koha.Preference('HoldsSplitQueue') == 'itemtype' %] + [% SET itemtypes = [] %] + + [% FOREACH h IN biblioloo.reserveloop %] + [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] + [% itemtypes.push( hold_itemtype ) %] + [% END %] + [% itemtypes = itemtypes.unique %] + [% FOREACH i IN itemtypes.sort %] [% SET holds_by_itemtype = [] %] - [% FOREACH h IN holds_by_branch %] + [% FOREACH h IN biblioloo.reserveloop %] [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] [% IF hold_itemtype == i %] [% holds_by_itemtype.push( h ) %] [% END %] [% END %] - [% INCLUDE holds_table.inc holds=holds_by_itemtype %] -
    + +
    + [% IF i %] + [% ItemTypes.GetDescription( i ) | html %] + [% ELSE %] + Any item type + [% END %] + [% INCLUDE holds_table.inc holds=holds_by_itemtype %] +
    + [% END %] + [% ELSIF Koha.Preference('HoldsSplitQueue') == 'branch_itemtype' %] + [% SET branchcodes = [] %] + + [% FOREACH h IN biblioloo.reserveloop %] + [% branchcodes.push( h.branchcode ) %] + [% END %] + [% branchcodes = branchcodes.unique %] + + [% FOREACH b IN branchcodes.sort %] +
    + [% Branches.GetName( b ) | html %] + [% SET holds_by_branch = [] %] + [% FOREACH h IN biblioloo.reserveloop %] + [% IF h.branchcode == b %] + [% holds_by_branch.push( h ) %] + [% END %] + [% END %] + + [% SET itemtypes = [] %] + [% FOREACH h IN holds_by_branch %] + [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] + [% itemtypes.push( hold_itemtype ) %] + [% END %] + [% itemtypes = itemtypes.unique %] + + [% FOREACH i IN itemtypes.sort %] +
    + [% IF i %] + [% ItemTypes.GetDescription( i ) | html %] + [% ELSE %] + Any item type + [% END %] + + [% SET holds_by_itemtype = [] %] + [% FOREACH h IN holds_by_branch %] + [% SET hold_itemtype = h.object.item.effective_itemtype || h.itemtype %] + [% IF hold_itemtype == i %] + [% holds_by_itemtype.push( h ) %] + [% END %] + [% END %] + [% INCLUDE holds_table.inc holds=holds_by_itemtype %] +
    + [% END %] +
    + [% END %] + [% ELSE %] + [% INCLUDE holds_table.inc holds=biblioloo.reserveloop %] [% END %] -
    - [% END %] - [% ELSE %] - [% INCLUDE holds_table.inc holds=biblioloo.reserveloop %] - [% END %] - [% END %] - [% END %] + [% END %] + [% END %] -
    - -
    - - - [% END %] -[% END %] +
    + +
    + + + [% END %] + [% END %] - + [% IF ( multi_hold ) # No sidebar menu when placing multiple holds %]
    -- 2.39.5