https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30849

Nick Clemens (kidclamp) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |[email protected]

--- Comment #10 from Nick Clemens (kidclamp) <[email protected]> ---
The general count is:
my $pending_borrower_modifications   =
Koha::Patron::Modifications->pending_count($branch);


branch comes from:
 75 my $branch =
 76     (     
C4::Context->preference("IndependentBranchesPatronModifications")
 77         || C4::Context->preference("IndependentBranches") )
 78     && !$flags->{'superlibrarian'}
 79     ? C4::Context->userenv()->{'branch'}
 80     : undef;



Then in the template:
[% IF Koha.Preference('IndependentBranchesPatronModifications') &&
!CAN_user_superlibrarian %]

Seems like that search can drop the $branch and we can use
$local_pendingmodifications_count in the template

Otherwise for independent branches we are doing the same branch limited search
twice

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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/

Reply via email to