Bug 21460: (follow-up) Fix bugs found in QA
authorAndrew Isherwood <andrew.isherwood@ptfs-europe.com>
Wed, 24 Apr 2019 13:58:41 +0000 (14:58 +0100)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 25 Apr 2019 10:46:56 +0000 (10:46 +0000)
Fix silly bugs mentioned in comment #29

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Koha/REST/V1/Illrequests.pm
ill/ill-requests.pl
koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt

index f747390..907dc5c 100644 (file)
@@ -57,7 +57,7 @@ sub list {
     # Get all requests
     # If necessary, only get those from a specified patron
     my @requests = Koha::Illrequests->search({
-        args->{borrowernumber}
+        $args->{borrowernumber}
         ? ( borrowernumber => $args->{borrowernumber} )
         : ()
     })->as_list;
index 57b0692..010c61e 100755 (executable)
@@ -280,8 +280,8 @@ if ( $backends_available ) {
                     uri_escape_utf8(scalar $params->{$filter});
             }
         }
+        my @tpl_arr = ();
         if (keys %{$active_filters}) {
-            my @tpl_arr;
             foreach my $key (keys %{$active_filters}) {
                 push @tpl_arr, $key . "=" . $active_filters->{$key};
             }
index 8feeccb..88ee777 100644 (file)
     [% INCLUDE 'calendar.inc' %]
     [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
     <script>
-        var prefilters = '[% prefilters %]';
+        var prefilters = '[% prefilters | $raw %]';
         // Set column settings
         var columns_settings = [% ColumnsSettings.GetColumns( 'illrequests', 'ill-requests', 'ill-requests', 'json' ) %];
     </script>