Bug 15407: Uniformise calls in patron-attr-types
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 12 Aug 2016 11:56:22 +0000 (12:56 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Thu, 8 Sep 2016 13:29:23 +0000 (13:29 +0000)
Does not make sense to call first in list context and later in scalar
context.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
admin/patron-attr-types.pl

index 8d44f70..1fcbc7c 100755 (executable)
@@ -271,12 +271,12 @@ sub edit_attribute_type_form {
         category_description => $attr_type->category_description,
     );
 
-    my @patron_categories = Koha::Patron::Categories->search({}, {order_by => ['description']});
+    my $patron_categories = Koha::Patron::Categories->search({}, {order_by => ['description']});
     $template->param(
         attribute_type_form => 1,
         edit_attribute_type => 1,
         confirm_op => 'edit_attribute_type_confirmed',
-        categories => \@patron_categories,
+        categories => $patron_categories,
     );
 
 }