Attachment 55628 Details for Bug 17308 – Bug 17308 - 'New patron attribute type' button does not work

[patch] Bug 17308 - 'New patron attribute type' button does not work

Bug-17308---New-patron-attribute-type-button-does-.patch (text/plain), 1.49 KB, created by Marc Véron on 2016-09-16 18:34:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2016-09-16 18:34:57 UTC

Size: 1.49 KB

patch

obsolete

>From 7cd6d13485074f7e9b28fee3da05927b48fc4051 Mon Sep 17 00:00:00 2001
>From: Andreas Roussos <arouss1980@gmail.com>
>Date: Fri, 16 Sep 2016 21:01:58 +0300
>Subject: [PATCH] Bug 17308 - 'New patron attribute type' button does not work
>
>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.
>---
> admin/patron-attr-types.pl | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/admin/patron-attr-types.pl b/admin/patron-attr-types.pl
>index 9a5fce2..72c13a9 100755
>--- a/admin/patron-attr-types.pl
>+++ b/admin/patron-attr-types.pl
>@@ -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},
>         };
>     }
> 
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 17308: 55627 | 55628 | 55629 | 55643