Attachment 173845 Details for Bug 31391 – Bug 31391: Interface and wording fixes

[patch] Bug 31391: Interface and wording fixes

Bug-31391-Interface-and-wording-fixes.patch (text/plain), 14.45 KB, created by Aleisha Amohia on 2024-10-31 23:30:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2024-10-31 23:30:53 UTC

Size: 14.45 KB

patch

obsolete

>From 1c4e490d09837ac5aa9753be117a93951b9ac853 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Thu, 2 Mar 2023 00:42:28 +0000
>Subject: [PATCH] Bug 31391: Interface and wording fixes
>
>This patch includes the following fixes:
>- Making the atomic updates executable
>- Preventing warns when running tests
>- Updating db updates
>- Fixing terminology
>- Make 'cancel selected recalls' button yellow
>- Fix HTML makeup of recalls request page
>- Show a warning when placing a recall if there is already a
>record-level hold by this patron
>- Show a link to mark a hold for conversion which selects the recall
>option on holds page
>- Include changes to patron search dropdown
>- add 'mark hold for conversion to recall' column to columns settings
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> admin/columns_settings.yml                    |  2 ++
> ..._CANCELLATION-RECALLED_authorised_value.pl |  4 +++-
> ...ug_31391_-_add_RecallsInterface_syspref.pl |  4 +++-
> .../prog/en/includes/holds_table.inc          |  6 ++++++
> .../admin/preferences/circulation.pref        | 18 +++++++++---------
> .../prog/en/modules/recalls/request.tt        | 19 +++++++++++++++----
> .../prog/en/modules/reserve/request.tt        |  6 +++++-
> recalls/request.pl                            |  2 ++
> t/db_dependent/Holds.t                        | 12 ++++++++++++
> 9 files changed, 57 insertions(+), 16 deletions(-)
> mode change 100644 => 100755 installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl
> mode change 100644 => 100755 installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl
>
>diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
>index 8ab19fdf96b..c9a07c923d9 100644
>--- a/admin/columns_settings.yml
>+++ b/admin/columns_settings.yml
>@@ -1832,6 +1832,8 @@ modules:
>               columnname: suspend
>             -
>               columnname: print_slip
>+            -
>+              columnname: convert_to_recall
> 
>     holdsratios:
>       holds-ratios:
>diff --git a/installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl b/installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl
>old mode 100644
>new mode 100755
>index dc2b8249052..57967eedb80
>--- a/installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl
>+++ b/installer/data/mysql/atomicupdate/bug_31391_-_add_HOLD_CANCELLATION-RECALLED_authorised_value.pl
>@@ -2,11 +2,13 @@ use Modern::Perl;
> 
> return {
>     bug_number => "31391",
>-    description => "Add new RECALLED value to HOLD_CANCELLATION authorised value category",
>+    description => "Staff-side recalls",
>     up => sub {
>         my ($args) = @_;
>         my ($dbh, $out) = @$args{qw(dbh out)};
> 
>         $dbh->do(q{ INSERT IGNORE INTO authorised_values (category, authorised_value, lib) VALUES ('HOLD_CANCELLATION','RECALLED','Hold was converted to a recall') });
>+
>+        say $out "Added RECALLED value to HOLD_CANCELLATION authorised value category";
>     },
> };
>diff --git a/installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl b/installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl
>old mode 100644
>new mode 100755
>index 48ca6d92bc3..15c301fa823
>--- a/installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl
>+++ b/installer/data/mysql/atomicupdate/bug_31391_-_add_RecallsInterface_syspref.pl
>@@ -2,11 +2,13 @@ use Modern::Perl;
> 
> return {
>     bug_number => "31391",
>-    description => "Add new system preference RecallsInterface",
>+    description => "Staff-side recalls",
>     up => sub {
>         my ($args) = @_;
>         my ($dbh, $out) = @$args{qw(dbh out)};
> 
>         $dbh->do(q{INSERT IGNORE INTO systempreferences (variable,value,options,explanation,type) VALUES ('RecallsInterface','opac','opac|staff|both','The interface that recalls can be placed through','Choice') });
>+
>+        say $out "Added system preference 'RecallsInterface'";
>     },
> };
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc
>index 20e4ab81d41..51e7b9e3a3a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc
>@@ -25,6 +25,7 @@
>             <th id="delete" data-colname="delete">Delete</th>
>             <th id="suspend" data-colname="suspend">Suspend</th>
>             <th id="print_slip" data-colname="print_slip">Print hold/transfer slip</th>
>+            <th id="convert_to_recall" data-colname="convert_to_recall">Convert hold to recall</th>
>         </tr>
>     </thead>
>     [%- SET first_priority = 0 -%]
>@@ -277,6 +278,11 @@
>                 [% ELSE %]
>                     <td></td>
>                 [% END %]
>+                <td>
>+                    [% IF Koha.Preference('UseRecalls') AND Koha.Preference('RecallsInterface') != 'opac' %]
>+                    <a class="convert-to-recall btn" title="Convert hold to recall" data-id="[% hold.reserve_id | html %]">Mark for conversion to recall</a>
>+                    [% END %]
>+                </td>
>             </tr>
>         [% END %]
>     </tbody>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>index 6fd08084f90..c7f2bc6647a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>@@ -1469,6 +1469,15 @@ Circulation:
>                   1: Use
>                   0: "Don't use"
>             - recalls. Make sure you configure <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a> for recalls once enabled.
>+        -
>+            - Recalls can be placed through
>+            - pref: RecallsInterface
>+              default: opac
>+              choices:
>+                  opac: the OPAC only
>+                  staff: the staff interface only
>+                  both: both the OPAC and the staff interface
>+            - . (Requires <a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=UseRecalls">UseRecalls</a>.)
> 
>     SIP2:
>         -
>@@ -1511,12 +1520,3 @@ Circulation:
>                   1: Enable
>                   0: Disable
>             - "the curbside pickup module."
>-        -
>-            - Recalls can be placed through
>-            - pref: RecallsInterface
>-              default: opac
>-              choices:
>-                  opac: the OPAC only
>-                  staff: the staff interface only
>-                  both: both the OPAC and the staff interface
>-            - . (Requires <a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=UseRecalls">UseRecalls</a>.)
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/recalls/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/recalls/request.tt
>index 2a93087cddc..9178c7fada2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/recalls/request.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/recalls/request.tt
>@@ -58,6 +58,12 @@
>             </div>
>         [% END %]
> 
>+        [% IF patron_holds_count %]
>+            <div class="dialog alert alert-info">
>+                Patron has placed one or more record-level holds. <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% biblio.biblionumber | html %]">Convert a hold to a recall</a>.
>+            </div>
>+        [% END %]
>+
>         <fieldset class="rows">
>             <legend>Recall details</legend>
>             <form id="recallform" action="/cgi-bin/koha/recalls/request.pl" method="post">
>@@ -111,7 +117,7 @@
>                         <th>Barcode</th>
>                         [% UNLESS ( single_branch_mode ) %]
>                             <th>Home library</th>
>-                            <th>Last location</th>
>+                            <th>Holding library</th>
>                         [% END %]
>                         <th>Collection</th>
>                         <th>Call number</th>
>@@ -148,7 +154,7 @@
>                                 <span class="intransit">In transit from [% Branches.GetName( item.get_transfer.frombranch ) | html %] to [% Branches.GetName( item.get_transfer.tobranch ) | html %] since [% item.get_transfer.datesent | $KohaDates %]</span>
>                             [% END %]
>                             [% IF ( item.itemlost || item.withdrawn ) %]
>-                                <span class="lost">Unavailable (lost or missing)</span>
>+                                <span class="lost">Unavailable (lost or withdrawn)</span>
>                             [% END %]
>                             [% IF ( item.notforloan ) %]
>                                 <span class="notforloan">Not for loan ([% item.notforloan | html %])</span>
>@@ -168,7 +174,7 @@
>                                 </span>
>                             [% ELSIF ( hold.borrowernumber == logged_in_user.borrowernumber ) %]
>                                 <span class="waiting">
>-                                    Patron has already placed a <a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% patron.borrowernumber | uri %]#reserves">reserve</a> on this item.
>+                                    Patron has already placed a <a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% patron.borrowernumber | uri %]#reserves">hold</a> on this item.
>                                 </span>
>                             [% END # / IF ( item.recall or hold ) %]
>                         </td>
>@@ -216,6 +222,8 @@
> 
>         [% END %]
> 
>+        <div class="page-section">
>+
>         <h2>Existing recalls</h2>
> 
>         [% IF Koha.Preference('UseRecalls') %]
>@@ -228,7 +236,7 @@
>                         <input type="checkbox" id="select_all"> <span id="select_all_text">Select all</span>
>                         [% INCLUDE 'recalls.inc' %]
>                         <fieldset class="action">
>-                            <button type="submit" id="cancel_selected" class="btn btn-default btn-sm">Cancel selected recalls</button>
>+                            <button type="submit" id="cancel_selected" class="btn btn-primary btn-sm">Cancel selected recalls</button>
>                         </fieldset>
>                     </form>
>                 </div> 
>@@ -258,10 +266,13 @@
>     [% INCLUDE 'datatables.inc' %]
>     [% INCLUDE 'columns_settings.inc' %]
>     [% INCLUDE 'calendar.inc' %]
>+    [% INCLUDE 'select2.inc' %]
>     [% SET url_biblio_params = "biblionumber=" _ biblio.biblionumber %]
>     [% PROCESS patron_search_js table_id => 'table_borrowers', categories => categories, libraries => libraries, extended_attribute_types => attribute_type_codes, columns => columns, open_on_row_click => 1, on_click_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_one_result => 1, redirect_url => '/cgi-bin/koha/recalls/request.pl?' _ url_biblio_params, redirect_if_attribute_equal => 'cardnumber' %]
>     <script>
>         table_settings = [% TablesSettings.GetColumns( 'circ', 'circulation', 'table_borrowers', 'json' ) | $raw %];
>+        $.fn.select2.defaults.set("width", "100%" );
>+        $.fn.select2.defaults.set("dropdownAutoWidth", true );
>         $(document).ready(function(){
>             $("#recalls_patronsearch").on("submit", filter);
>             [% UNLESS patron %]
>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 9bade1d1be2..1ad182f54e0 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
>@@ -259,7 +259,7 @@
>                 [% IF ( recallerror ) %]
>                     <div class="dialog alert">
>                         <h3>Hold cannot be converted to a recall</h3>
>-                        <p>A recall cannot be placed on one or more holds. Check your <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a>.</p>
>+                        <p>A recall cannot be placed for one or more holds. Check your <a href="/cgi-bin/koha/admin/smart-rules.pl">circulation and fine rules</a>.</p>
>                     </div>
>                 [% END %]
> 
>@@ -2021,6 +2021,10 @@
>             window.open("/cgi-bin/koha/circ/hold-transfer-slip.pl?reserve_id=" + reserve_id);
>             return false;
>         })
>+        $(".convert-to-recall").click(function(){
>+            var reserve_id = $(this).attr('data-id');
>+            $("select[class='rank-request'][data-hold-id='"+reserve_id+"']").val('recall');
>+        });
>     </script>
> [% END %]
> 
>diff --git a/recalls/request.pl b/recalls/request.pl
>index 37d356212be..8e71fad59d8 100755
>--- a/recalls/request.pl
>+++ b/recalls/request.pl
>@@ -122,9 +122,11 @@ if ( $borrowernumber ) {
>         if ( !$biblio->can_be_recalled({ patron => $patron }) ) {
>             $error = 1;
>         }
>+        my $patron_holds_count = Koha::Holds->search({ borrowernumber => $borrowernumber, biblionumber => $biblio->biblionumber, item_level_hold => 0 })->count;
> 
>         $template->param(
>             patron => $patron,
>+            patron_holds_count => $patron_holds_count,
>         );
>     }
> }
>diff --git a/t/db_dependent/Holds.t b/t/db_dependent/Holds.t
>index 8ba75a65397..cdaa2c485d8 100755
>--- a/t/db_dependent/Holds.t
>+++ b/t/db_dependent/Holds.t
>@@ -1897,6 +1897,7 @@ subtest 'ModReserve to convert a hold to a recall' => sub {
>             surname      => 'whatever surname',
>             categorycode => $category->{categorycode},
>             branchcode   => $branch,
>+            email        => 'name@email.com',
>         }
>     )->store;
>     my $borrowernumber = $patron->borrowernumber;
>@@ -1907,6 +1908,7 @@ subtest 'ModReserve to convert a hold to a recall' => sub {
>             surname      => 'whatever surname',
>             categorycode => $category->{categorycode},
>             branchcode   => $branch,
>+            email        => 'name2@email.com',
>         }
>     )->store;
> 
>@@ -1926,6 +1928,16 @@ subtest 'ModReserve to convert a hold to a recall' => sub {
>         },
>     });
> 
>+    my $notice = Koha::Notice::Template->new({
>+        name                   => 'Hold cancellation',
>+        module                 => 'reserves',
>+        code                   => 'HOLD_CANCELLATION',
>+        title                  => 'Hold cancelled',
>+        content                => 'Your hold was cancelled.',
>+        message_transport_type => 'email',
>+        branchcode             => q{},
>+    })->store();
>+
>     my $reserve_id = AddReserve(
>         {
>             branchcode     => $branch,
>-- 
>2.39.5

Actions: View | Diff | Splinter Review

Attachments on bug 31391: 139442 | 139443 | 139444 | 139446 | 139642 | 139643 | 139644 | 139645 | 141474 | 142594 | 143155 | 144104 | 144105 | 144131 | 144132 | 144133 | 144134 | 144135 | 144136 | 147581 | 147582 | 150485 | 150486 | 150487 | 150488 | 150489 | 150490 | 150491 | 156995 | 156996 | 156997 | 156998 | 156999 | 157000 | 157001 | 158691 | 158692 | 158693 | 158694 | 158695 | 158825 | 158826 | 158827 | 158828 | 158829 | 158830 | 158831 | 158832 | 172166 | 172167 | 172168 | 172169 | 172170 | 172171 | 172172 | 172173 | 172174 | 172175 | 173839 | 173840 | 173841 | 173842 | 173843 | 173844 | 173845 | 173846 | 173847 | 174396 | 174822 | 174823 | 174824 | 174825 | 174826 | 174827 | 174828 | 174829 | 174830 | 174831