http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6673
--- Comment #8 from Ian Walls <[email protected]> 2011-09-21 02:29:25 UTC --- Further evidence: all other invocations of the variable LoginBranchname work like this: get_branch_code_from_name($template->{VARS}->{'LoginBranchname'}) Whereas (at least before the attached patch), manage-marc-import.pl does this: get_branch_code_from_name($template->param('LoginBranchname')) By default, LoginBranchname is defined as: C4::Context->userenv?C4::Context->userenv->{"branchname"}:"insecure" or C4::Context->userenv?C4::Context->userenv->{"branchname"}:"" depending on what part of C4/Auth.pm you look at. Since 'insecure' and '' are both invalid entries, it makes more sense to bypass the code=>name=>code jump, and just invoke the code in the standard fashion, as submitted in this patch. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
