[Koha-patches] [PATCH 2/2] [SIGNED-OFF] Bug 3202: creating new 'child' categorytype patrons doesn't set branch properly

[Nicole C. Engard]

From: Ian Walls <ian.walls at bywatersolutions.com>

If a guarantor search is not being performed, the creation of any Child category_type patron will
not default to the currently logged in branch, but rather the first branch alphabetically.

Adding a test for the existence of any guarantor branch data before assigning it clears up this problem

Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
---
 members/memberentry.pl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/members/memberentry.pl b/members/memberentry.pl
index c665b6e..a8e764c 100755
--- a/members/memberentry.pl
+++ b/members/memberentry.pl
@@ -549,7 +549,7 @@ if(scalar(@select_branch) > 0){
 # --------------------------------------------------------------------------------------------------------
   #in modify mod :default value from $CGIbranch comes from borrowers table
   #in add mod: default value come from branches table (ip correspendence)
-$default=$data{'branchcode'}  if ($op eq 'modify' || ($op eq 'add' && $category_type eq 'C'));
+$default=$data{'branchcode'}  if ($op eq 'modify' || ($op eq 'add' && $category_type eq 'C' && $data{'branchcode'}));
 $CGIbranch = CGI::scrolling_list(-id    => 'branchcode',
             -name   => 'branchcode',
             -values => \@select_branch,
-- 
1.7.2.3