http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8311
Paul Poulain <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #4 from Paul Poulain <[email protected]> --- QA comment: Why did you put the line require C4::Members; outside from if (!defined($borrowernumber) && defined($user)) { ? The way you made it, C4::Members will be loaded on everypage, that will impact performances ! Written: if (!defined($borrowernumber) && defined($user)) { require C4::Members; my $borrower = C4::Members::GetMember(borrowernumber => $user); will load the C4::Members only if the getborrowernumber does not return the user (which should never happen) waiting for your feedback, not pushing not marking failed QA -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ 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/
