Bug 17308 - 'New patron attribute type' button does not work
authorAndreas Roussos <arouss1980@gmail.com>
Fri, 16 Sep 2016 18:01:58 +0000 (21:01 +0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Sun, 25 Sep 2016 13:09:49 +0000 (13:09 +0000)
In the Staff client, the 'New patron attribute type' button in the Patron
attribute types page does not work. This patch fixes that.

Test plan:
1) In the Staff client, go to Home > Administration > Patron attribute types,
   and click on the 'New patron attribute type' button. You cannot create a
   new patron attribute type and you get the following Software error:
   Not a HASH reference at [...]
2) Apply the patch.
3) Repeat step 1). The 'New patron attribute type' button now works.

Followed test plan, works as expected.
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
admin/patron-attr-types.pl

index 9a5fce2..72c13a9 100755 (executable)
@@ -86,10 +86,10 @@ sub add_attribute_type_form {
 
     my $branches = Koha::Libraries->search( {}, { order_by => ['branchname'] } )->unblessed;
     my @branches_loop;
-    foreach my $branch (sort keys %$branches) {
+    foreach my $branch (@$branches) {
         push @branches_loop, {
-            branchcode => $$branches{$branch}{branchcode},
-            branchname => $$branches{$branch}{branchname},
+            branchcode => $branch->{branchcode},
+            branchname => $branch->{branchname},
         };
     }