Bug 3202: creating new 'child' categorytype patrons doesn't set branch properly
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@bywatersolutions.com> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
parent
694f0fbeaf
commit
b08be41d41
|
|
@ -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,
|
||||
|
|
|
|||
Loading…
Reference in New Issue