From f7abff91915afe103e7d7a37fe64dd6721401da2 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 14 Nov 2022 16:48:22 +0000 Subject: [PATCH] Bug 32042: Add page-section to catalog's item detail view This patch adds a couple of .page-section divs to the item detail view in the catalog. A "clearfix" class is also added in order to make sure the sections don't overlap. To test, apply the patch and locate a bibliographic record in the catalog. - In the sidebar menu of the bibliographic detail view, click "Items." - On the item detail page, confirm that the information is displayed well. Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../prog/en/modules/catalogue/moredetail.tt | 704 +++++++++--------- 1 file changed, 351 insertions(+), 353 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt index bfb6c4712d..765c87fef4 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt @@ -8,7 +8,7 @@ [% INCLUDE 'doc-head-open.inc' %] Item details for [% INCLUDE 'biblio-title-head.inc' %] › Catalog › Koha [% INCLUDE 'doc-head-close.inc' %] - + [% USE KohaDates %] @@ -47,55 +47,51 @@ [% INCLUDE 'cat-toolbar.inc' %]
- -

[% title | html %]

- [% IF ( subtitle ) %] -

[% FOREACH subtitl IN subtitle %] [% subtitl | html %][% END %]

- [% END %] - [% IF ( author ) %] -

by [% author | html %]

- [% END %] - -
-
-
    -
  1. Biblionumber: [% biblionumber | html %] 
  2. - [% UNLESS ( item_level_itypes ) %] -
  3. Item type: [% itemtypename | html %] 
  4. - [% END %] - [% IF ( rentalcharge ) %] -
  5. Rental charge:[% rentalcharge | $Price %] 
  6. - [% END %] - [% IF ( rentalcharge_daily ) %] -
  7. Daily rental charge:[% rentalcharge_daily | $Price %] 
  8. - [% END %] - [% IF ( rentalcharge_hourly ) %] -
  9. Hourly rental charge:[% rentalcharge_hourly | $Price %] 
  10. - [% END %] -
  11. ISBN: [% isbn | html %] 
  12. -
  13. Publication details:[% place | html %] [% publishercode | html %] [% publicationyear | html %] 
  14. - [% IF ( volumeddesc ) %] -
  15. Volume: [% volumeddesc | html %]
  16. - [% END %] -
  17. Physical details: [% pages | html %] [% illus | html %] [% size | html %] 
  18. - [% IF ( bnotes ) %] -
  19. Notes: [% bnotes | html %]
  20. - [% END %] -
  21. - No. of items: - [% count | html %] - [% IF ( hiddencount ) %] - total ([% showncount | html %] shown / [% hiddencount | html %] hidden) - Show all items +
    +

    [% title | html %]

    + [% IF ( subtitle ) %] +

    [% FOREACH subtitl IN subtitle %] [% subtitl | html %][% END %]

    + [% END %] + [% IF ( author ) %] +

    by [% author | html %]

    + [% END %] +
    +
    +
      +
    1. Biblionumber: [% biblionumber | html %] 
    2. + [% UNLESS ( item_level_itypes ) %] +
    3. Item type: [% itemtypename | html %] 
    4. [% END %] - -
    -
    -
    - - [% IF ONLY_ONE || updated_exclude_from_local_holds_priority.defined %] -
    - [% END %] + [% IF ( rentalcharge ) %] +
  22. Rental charge:[% rentalcharge | $Price %] 
  23. + [% END %] + [% IF ( rentalcharge_daily ) %] +
  24. Daily rental charge:[% rentalcharge_daily | $Price %] 
  25. + [% END %] + [% IF ( rentalcharge_hourly ) %] +
  26. Hourly rental charge:[% rentalcharge_hourly | $Price %] 
  27. + [% END %] +
  28. ISBN: [% isbn | html %] 
  29. +
  30. Publication details:[% place | html %] [% publishercode | html %] [% publicationyear | html %] 
  31. + [% IF ( volumeddesc ) %] +
  32. Volume: [% volumeddesc | html %]
  33. + [% END %] +
  34. Physical details: [% pages | html %] [% illus | html %] [% size | html %] 
  35. + [% IF ( bnotes ) %] +
  36. Notes: [% bnotes | html %]
  37. + [% END %] +
  38. + No. of items: + [% count | html %] + [% IF ( hiddencount ) %] + total ([% showncount | html %] shown / [% hiddencount | html %] hidden) + Show all items + [% END %] +
  39. +
+
+
+
[% IF ( ONLY_ONE ) %]
You are only viewing one item. View all
@@ -112,363 +108,365 @@ [% END %] [% FOREACH ITEM_DAT IN ITEM_DATA %] - [% SET not_for_loan = 0 %] - [% IF ITEM_DAT.notforloan || ITEM_DAT.effective_itemtype.notforloan %] - [% SET not_for_loan = 1 %] - [% SET not_for_loan_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.notforloan', authorised_value => item.notforloan ) %] - [% END %] -
-

- Barcode [% ITEM_DAT.barcode | html %] [% IF not_for_loan %][% not_for_loan_description | html %] [% END %] -

- -

- Item information - [% UNLESS ( ITEM_DAT.nomod ) %] - [% IF ( CAN_user_editcatalogue_edit_items ) %] - Edit item - [% END %] - [% END %] -

+
+ [% SET not_for_loan = 0 %] + [% IF ITEM_DAT.notforloan || ITEM_DAT.effective_itemtype.notforloan %] + [% SET not_for_loan = 1 %] + [% SET not_for_loan_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.notforloan', authorised_value => item.notforloan ) %] + [% END %] +
+

+ Barcode [% ITEM_DAT.barcode | html %] [% IF not_for_loan %][% not_for_loan_description | html %] [% END %] +

-
-
    -
  1. Home library: [% Branches.GetName( ITEM_DAT.homebranch ) | html %] 
  2. - [% IF ( item_level_itypes ) %] -
  3. Item type: [% ITEM_DAT.itype | html %] 
  4. - [% END %] - [% IF ( ITEM_DAT.ccode) %] -
  5. Collection: [% ITEM_DAT.ccode | html %]
  6. - [% END %] -
  7. Item callnumber: [% ITEM_DAT.itemcallnumber | html %] 
  8. - [% IF ( ITEM_DAT.displaycopy ) %] -
  9. Copy number: [% ITEM_DAT.copyvol | html %] 
  10. - [% END %] - [% IF ( ITEM_DAT.replacementprice ) %] -
  11. Replacement price: [% ITEM_DAT.replacementprice | $Price %] 
  12. - [% END %] - [% IF ITEM_DAT.materials %] -
  13. Materials specified: [% ITEM_DAT.materials | html %]
  14. +

    + Item information + [% UNLESS ( ITEM_DAT.nomod ) %] + [% IF ( CAN_user_editcatalogue_edit_items ) %] + Edit item + [% END %] [% END %] -

-
-
+ -
-

- Statuses - [% IF ( ITEM_DAT.status_advisory ) %] - ([% IF ( not_for_loan ) %] - [% IF not_for_loan_description %] - [% not_for_loan_description | html %] - [% ELSE %] - Not for loan +
+
    +
  1. Home library: [% Branches.GetName( ITEM_DAT.homebranch ) | html %] 
  2. + [% IF ( item_level_itypes ) %] +
  3. Item type: [% ITEM_DAT.itype | html %] 
  4. [% END %] - [% END %] - [% IF ( ITEM_DAT.itemlost ) %]Lost[% END %] - [% IF ( ITEM_DAT.damaged ) %]Damaged[% END %] - [% IF ( ITEM_DAT.withdrawn ) %]Withdrawn[% END %] - ) - [% END %] -

+ [% IF ( ITEM_DAT.ccode) %] +
  • Collection: [% ITEM_DAT.ccode | html %]
  • + [% END %] +
  • Item callnumber: [% ITEM_DAT.itemcallnumber | html %] 
  • + [% IF ( ITEM_DAT.displaycopy ) %] +
  • Copy number: [% ITEM_DAT.copyvol | html %] 
  • + [% END %] + [% IF ( ITEM_DAT.replacementprice ) %] +
  • Replacement price: [% ITEM_DAT.replacementprice | $Price %] 
  • + [% END %] + [% IF ITEM_DAT.materials %] +
  • Materials specified: [% ITEM_DAT.materials | html %]
  • + [% END %] + +
    +
    -
    -
      -
    1. Current library: [% Branches.GetName( ITEM_DAT.holdingbranch ) | html %] 
    2. -
    3. - Checkout status: - [% SET checkout = ITEM_DAT.object.checkout %] - [% IF ( checkout ) %] - Checked out to [% INCLUDE 'patron-title.inc' patron => checkout.patron hide_patron_infos_if_needed=1 %] - [% IF ( checkout.lastreneweddate ) %] - Last renewed [% checkout.lastreneweddate | $KohaDates %], - [% END %] - [% IF ( checkout.date_due ) %] - Due back on [% checkout.date_due | $KohaDates %] +
      +

      + Statuses + [% IF ( ITEM_DAT.status_advisory ) %] + ([% IF ( not_for_loan ) %] + [% IF not_for_loan_description %] + [% not_for_loan_description | html %] [% ELSE %] - Not checked out + Not for loan [% END %] [% END %] -

    4. -
    5. - Current renewals: - [% ITEM_DAT.renewals | html %] - [% IF ITEM_DAT.renewals %] - [ View ] - [% END %] -
    6. - [% IF itemlostloop %] + [% IF ( ITEM_DAT.itemlost ) %]Lost[% END %] + [% IF ( ITEM_DAT.damaged ) %]Damaged[% END %] + [% IF ( ITEM_DAT.withdrawn ) %]Withdrawn[% END %] + ) + [% END %] + + +
      +
        +
      1. Current library: [% Branches.GetName( ITEM_DAT.holdingbranch ) | html %] 
      2. - Lost status: - [% IF ( CAN_user_circulate ) %] - [% SET ClaimReturnedLostValue = Koha.Preference('ClaimReturnedLostValue') %] -
        - - - + Checkout status: + [% SET checkout = ITEM_DAT.object.checkout %] + [% IF ( checkout ) %] + Checked out to [% INCLUDE 'patron-title.inc' patron => checkout.patron hide_patron_infos_if_needed=1 %] + [% IF ( checkout.lastreneweddate ) %] + Last renewed [% checkout.lastreneweddate | $KohaDates %], + [% END %] + [% IF ( checkout.date_due ) %] + Due back on [% checkout.date_due | $KohaDates %] + [% ELSE %] + Not checked out + [% END %] + [% END %] +
      3. +
      4. + Current renewals: + [% ITEM_DAT.renewals | html %] + [% IF ITEM_DAT.renewals %] + [ View ] + [% END %] +
      5. + [% IF itemlostloop %] +
      6. + Lost status: + [% IF ( CAN_user_circulate ) %] + [% SET ClaimReturnedLostValue = Koha.Preference('ClaimReturnedLostValue') %] + + + + - [% IF ClaimReturnedLostValue && ITEM_DAT.itemlost == ClaimReturnedLostValue %] - - [% END %] - - [% FOREACH itemlostloo IN itemlostloop %] - [% IF ClaimReturnedLostValue && !ITEM_DAT.itemlost && itemlostloo.authorised_value == ClaimReturnedLostValue #Filter claims returned status from pulldown %] - - [% ELSIF itemlostloo.authorised_value == ITEM_DAT.itemlost %] - - [% ELSE %] - - [% END %] + [% IF ClaimReturnedLostValue && ITEM_DAT.itemlost == ClaimReturnedLostValue %] + [% END %] - - - + + [% FOREACH itemlostloo IN itemlostloop %] + [% IF ClaimReturnedLostValue && !ITEM_DAT.itemlost && itemlostloo.authorised_value == ClaimReturnedLostValue #Filter claims returned status from pulldown %] + + [% ELSIF itemlostloo.authorised_value == ITEM_DAT.itemlost %] + + [% ELSE %] + + [% END %] + [% END %] + + + - [% IF ClaimReturnedLostValue && ITEM_DAT.itemlost == ClaimReturnedLostValue %] - -

        Item has been claimed as returned.

        - [% ELSE %] - - - [% END %] - - [% ELSE %] - [% FOREACH itemlostloo IN itemlostloop %] - [% IF ( itemlostloo.selected ) %] - [% itemlostloo.lib | html %] + [% IF ClaimReturnedLostValue && ITEM_DAT.itemlost == ClaimReturnedLostValue %] + +

        Item has been claimed as returned.

        + [% ELSE %] + + + [% END %] + + [% ELSE %] + [% FOREACH itemlostloo IN itemlostloop %] + [% IF ( itemlostloo.selected ) %] + [% itemlostloo.lib | html %] + [% END %] [% END %] [% END %] +
      7. + [% IF ITEM_DAT.itemlost != "" && ITEM_DAT.itemlost_on %] +
      8. Lost on:[% ITEM_DAT.itemlost_on | $KohaDates %]
      9. [% END %] - - [% IF ITEM_DAT.itemlost != "" && ITEM_DAT.itemlost_on %] -
      10. Lost on:[% ITEM_DAT.itemlost_on | $KohaDates %]
      11. [% END %] - [% END %] - [% IF itemdamagedloop %] -
      12. - Damaged status: - [% IF ( CAN_user_circulate ) %] -
        - - - - + + + + + +
        + [% ELSE %] + [% FOREACH itemdamagedloo IN itemdamagedloop %] + [% IF ( itemdamagedloo.selected ) %] + [% itemdamagedloo.lib | html %] [% END %] - - - - - [% ELSE %] - [% FOREACH itemdamagedloo IN itemdamagedloop %] - [% IF ( itemdamagedloo.selected ) %] - [% itemdamagedloo.lib | html %] [% END %] [% END %] +
      13. + [% IF ITEM_DAT.damaged != "" && ITEM_DAT.damaged_on %] +
      14. Damaged on:[% ITEM_DAT.damaged_on | $KohaDates %]
      15. [% END %] - - [% IF ITEM_DAT.damaged != "" && ITEM_DAT.damaged_on %] -
      16. Damaged on:[% ITEM_DAT.damaged_on | $KohaDates %]
      17. [% END %] - [% END %] - [% IF itemwithdrawnloop %] -
      18. - Withdrawn status: - [% IF ( CAN_user_circulate ) %] -
        - - - - + + + + + +
        + [% ELSE %] + [% FOREACH itemwithdrawn IN itemwithdrawnloop %] + [% IF itemwithdrawn.authorised_value == ITEM_DAT.withdrawn %] + [% itemwithdrawn.lib | html %] [% END %] - - - - - [% ELSE %] - [% FOREACH itemwithdrawn IN itemwithdrawnloop %] - [% IF itemwithdrawn.authorised_value == ITEM_DAT.withdrawn %] - [% itemwithdrawn.lib | html %] [% END %] [% END %] +
      19. + [% IF ITEM_DAT.withdrawn != "" && ITEM_DAT.withdrawn_on %] +
      20. Withdrawn on:[% ITEM_DAT.withdrawn_on | $KohaDates %]
      21. [% END %] + [% END %] +
      +
      +
    + +
    +

    Priority

    +
    +
      +
    1. + Exclude from local holds priority: +
      + + + + + + +
    2. - [% IF ITEM_DAT.withdrawn != "" && ITEM_DAT.withdrawn_on %] -
    3. Withdrawn on:[% ITEM_DAT.withdrawn_on | $KohaDates %]
    4. +
    +
    +
    + +
    +

    History

    +
    +
      + + [% IF ITEM_DAT.basketno %] +
    1. + Order date: + [% IF ( CAN_user_acquisition_order_manage ) %] + [% ITEM_DAT.orderdate | $KohaDates %] + [% ELSE %] + [% ITEM_DAT.orderdate | $KohaDates %] + [% END %] +
    2. +
    3. + Vendor: + [% ITEM_DAT.vendor | html %] +
    4. [% END %] - [% END %] -
    -
    -
    -
    -

    Priority

    -
    -
      -
    1. - Exclude from local holds priority: -
      - - - - - - -
      -
    2. -
    -
    -
    + + [% END %] -
    -

    History

    -
    -
      + [% IF ( ITEM_DAT.invoicenumber ) %] +
    1. + Invoice number: + [% ITEM_DAT.invoicenumber | html %] +
    2. + [% END %] - [% IF ITEM_DAT.basketno %]
    3. - Order date: - [% IF ( CAN_user_acquisition_order_manage ) %] - [% ITEM_DAT.orderdate | $KohaDates %] + Total checkouts: + [% IF ( ITEM_DAT.issues ) %] + [% ITEM_DAT.issues | html %] [% ELSE %] - [% ITEM_DAT.orderdate | $KohaDates %] + 0 [% END %] + (View item's checkout history)
    4. -
    5. - Vendor: - [% ITEM_DAT.vendor | html %] -
    6. - [% END %] - [% IF ITEM_DAT.dateaccessioned %]
    7. - Accession date: - [% IF ( CAN_user_acquisition_order_receive && ITEM_DAT.invoiceid ) %] - [% ITEM_DAT.dateaccessioned | $KohaDates %] - [% ELSE %] - [% ITEM_DAT.dateaccessioned | $KohaDates %] - [% END %] + Last seen: + [% IF ( ITEM_DAT.datelastseen ) %] + [% ITEM_DAT.datelastseen | $KohaDates %] + [%END %]
    8. - [% END %] - - [% IF ( ITEM_DAT.invoicenumber ) %]
    9. - Invoice number: - [% ITEM_DAT.invoicenumber | html %] + Last borrowed: + [% IF (ITEM_DAT.datelastborrowed ) %][% ITEM_DAT.datelastborrowed | $KohaDates %][% END %]
    10. - [% END %] + [% IF Koha.Preference('StoreLastBorrower') && ITEM_DAT.object.last_returned_by %] +
    11. + Last returned by: + [% ITEM_DAT.object.last_returned_by.cardnumber | html %] +
    12. + [% END %] + [% IF ( ITEM_DAT.card0 ) %] +
    13. + Last borrower: + [% ITEM_DAT.card0 | html %]  +
    14. + [% END %] + [% IF ( ITEM_DAT.card1 ) %] +
    15. + Previous borrower: + [% ITEM_DAT.card1 | html %]  +
    16. + [% END %] + [% IF ( ITEM_DAT.card2 ) %] +
    17. + Previous borrower: + [% ITEM_DAT.card2 | html %]  +
    18. + [% END %] + [% IF ( ITEM_DAT.paidfor ) %] +
    19. + Paid for?: + [% INCLUDE 'patron-title.inc' patron=ITEM_DAT.paidfor.patron hide_patron_infos_if_needed=1 %] on [% ITEM_DAT.paidfor.created_on | $KohaDates %] +
    20. + [% END %] -
    21. - Total checkouts: - [% IF ( ITEM_DAT.issues ) %] - [% ITEM_DAT.issues | html %] - [% ELSE %] - 0 + [% IF ( ITEM_DAT.enumchron ) %] +
    22. + Serial enumeration: + [% ITEM_DAT.enumchron | html %]  +
    23. [% END %] - (View item's checkout history) - -
    24. - Last seen: - [% IF ( ITEM_DAT.datelastseen ) %] - [% ITEM_DAT.datelastseen | $KohaDates %] - [%END %] -
    25. -
    26. - Last borrowed: - [% IF (ITEM_DAT.datelastborrowed ) %][% ITEM_DAT.datelastborrowed | $KohaDates %][% END %] -
    27. - [% IF Koha.Preference('StoreLastBorrower') && ITEM_DAT.object.last_returned_by %]
    28. - Last returned by: - [% ITEM_DAT.object.last_returned_by.cardnumber | html %] -
    29. - [% END %] - [% IF ( ITEM_DAT.card0 ) %] -
    30. - Last borrower: - [% ITEM_DAT.card0 | html %]  -
    31. - [% END %] - [% IF ( ITEM_DAT.card1 ) %] -
    32. - Previous borrower: - [% ITEM_DAT.card1 | html %]  -
    33. - [% END %] - [% IF ( ITEM_DAT.card2 ) %] -
    34. - Previous borrower: - [% ITEM_DAT.card2 | html %]  -
    35. - [% END %] - [% IF ( ITEM_DAT.paidfor ) %] -
    36. - Paid for?: - [% INCLUDE 'patron-title.inc' patron=ITEM_DAT.paidfor.patron hide_patron_infos_if_needed=1 %] on [% ITEM_DAT.paidfor.created_on | $KohaDates %] + Public note: + [% IF ( CAN_user_editcatalogue_edit_items ) %] +
      + + + +
      + [% ELSE %] + [% ITEM_DAT.itemnotes | html %] + [% END %]
    37. - [% END %] - [% IF ( ITEM_DAT.enumchron ) %]
    38. - Serial enumeration: - [% ITEM_DAT.enumchron | html %]  + Non-public note: + [% IF ( CAN_user_editcatalogue_edit_items ) %] +
      + + + +
      + [% ELSE %] + [% ITEM_DAT.itemnotes_nonpublic | html %] + [% END %]
    39. - [% END %] - -
    40. - Public note: - [% IF ( CAN_user_editcatalogue_edit_items ) %] -
      - - - -
      - [% ELSE %] - [% ITEM_DAT.itemnotes | html %] - [% END %] -
    41. - -
    42. - Non-public note: - [% IF ( CAN_user_editcatalogue_edit_items ) %] -
      - - - -
      - [% ELSE %] - [% ITEM_DAT.itemnotes_nonpublic | html %] - [% END %] -
    43. -
    -
    -
    + +
    + + [% END # /FOREACH ITEM_DAT %] -- 2.39.5