Attachment 170184 Details for Bug 37177 – Bug 37177: "item" should be "record" in list page

[patch] Bug 37177: "item" should be "record" in list page

Bug-37177-item-should-be-record-in-list-page.patch (text/plain), 6.20 KB, created by Owen Leonard on 2024-08-09 12:17:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2024-08-09 12:17:05 UTC

Size: 6.20 KB

patch

obsolete

>From 4b18ac510413702efbb0297c8835bce3a54f254a Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 9 Aug 2024 11:57:53 +0000
>Subject: [PATCH] Bug 37177: "item" should be "record" in list page
>
>This patch rewords areas of the lists and cart templates so that the
>term "record" is preferred over "item" when referring to a bibliographic
>record.
>
>To test, apply the patch and go to Lists in the staff interface.
>
>- If necessary, create a new list.
>- From the list contents view, click "Add records" in the toolbar.
>  - Submit some valid barcodes and biblionumbers
>  - After adding biblionumbers you should see the message "The record
>    ( [number] ) has been added to the list"
>  - After adding barcodes you should see the message "The record for
>    this item ( [barcode] ) has been added to the list"
>- Click "Add records" again
>  - Submit some of the same barcodes and biblionumbers
>    - You should see these messages:
>      - "The record for this item ( [barcode] ) has not
>         been added to the list. Please verify it is not already in the
>         list."
>      - "The record ( [number] ) has not been added to the
>        list. Please verify it is not already in the list.
>- Check the checkbox for one or more titles on your list and click
>  "Remove selected"
>  - You should see the message, "The record has been removed from the
>    list."
>- Perform a catalog search and from the results screen, select one or
>  more results and add them to the cart.
>  - Open the Cart and confirm that the controls at the top of the table
>    of titles reads "Selected records: ..."
>
>Sponsored-by: Athens County Public Libraries
>---
> .../prog/en/includes/virtualshelves-toolbar.inc        |  2 +-
> .../intranet-tmpl/prog/en/modules/basket/basket.tt     |  2 +-
> .../prog/en/modules/virtualshelves/shelves.tt          | 10 +++++-----
> 3 files changed, 7 insertions(+), 7 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/virtualshelves-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/virtualshelves-toolbar.inc
>index 2ce1ccd503..08d831158b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/virtualshelves-toolbar.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/virtualshelves-toolbar.inc
>@@ -4,7 +4,7 @@
>     [% IF ( can_add_biblios ) %]
>         <div class="btn-group">
>             <button type="button" class="btn btn-default" data-toggle="modal" data-target="#addToList">
>-                <i class="fa fa-plus"></i> Add items
>+                <i class="fa fa-plus"></i> Add records
>             </button>
>         </div>
>     [% END %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>index efd48a74d9..5660c30ef3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>@@ -7,7 +7,7 @@
> [% BLOCK controls %]
>     <p class="cart-controls">
>         <a id="CheckAll" href="#"><i class="fa fa-check"></i> Select all</a> <a id="CheckNone" href="#"><i class="fa fa-times"></i> Clear all</a>
>-        | <strong>Selected items:</strong>
>+        | <strong>Selected records:</strong>
>         <a href="#" id="remove_from_cart">Remove</a>
> 
>         [% IF ( loggedinusername ) %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>index 4a0d749579..cf59687db9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>@@ -151,7 +151,7 @@
>                         <span>An error occurred when deleting this list.</span>
>                     [% CASE 'error_on_add_biblio' %]
>                         [% IF m.item_barcode %]
>-                            <span>The item ([% m.item_barcode | html %]) has not been added to the list. Please verify it is not already in the list.</span>
>+                            <span>The record for this item ([% m.item_barcode | html %]) has not been added to the list. Please verify it is not already in the list.</span>
>                         [% ELSE %]
>                             <span>The record ([% m.bibnum | html %]) has not been added to the list. Please verify it is not already in the list.</span>
>                         [% END %]
>@@ -163,12 +163,12 @@
>                         <span>List deleted.</span>
>                     [% CASE 'success_on_add_biblio' %]
>                         [% IF m.item_barcode %]
>-                            <span>The item ([% m.item_barcode | html %]) has been added to the list.</span>
>+                            <span>The record for this item ([% m.item_barcode | html %]) has been added to the list.</span>
>                         [% ELSE %]
>                             <span>The record ([% m.bibnum | html %]) has been added to the list.</span>
>                         [% END %]
>                     [% CASE 'success_on_remove_biblios' %]
>-                        <span>The item has been removed from the list.</span>
>+                        <span>The record has been removed from the list.</span>
>                     [% CASE 'does_not_exist' %]
>                         <span>This list does not exist.</span>
>                     [% CASE 'item_does_not_exist' %]
>@@ -572,7 +572,7 @@
>                 <div class="modal-content">
>                     <div class="modal-header">
>                         <button type="button" class="closebtn" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
>-                        <h4 class="modal-title" id="addToListLabel">Add items</h4>
>+                        <h4 class="modal-title" id="addToListLabel">Add records</h4>
>                     </div>
>                     <div class="modal-body">
>                         <fieldset class="brief">
>@@ -624,7 +624,7 @@
>     [% END %]
>     <script>
>         var MSG_NO_ITEM_SELECTED = _("Nothing is selected.");
>-        var MSG_REMOVE_FROM_LIST = _("Are you sure you want to remove these items from the list?");
>+        var MSG_REMOVE_FROM_LIST = _("Are you sure you want to remove these records from the list?");
>         var MSG_CONFIRM_DELETE_LIST = _("Are you sure you want to remove this list?");
> 
>         [% IF op == 'list' %]
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 37177: 170184 | 170236 | 170239 | 170428 | 170901