[koha-commits] main Koha release repository branch master updated. v3.22.00-802-g9be221b

Git repo owner gitmaster at git.koha-community.org
Mon Mar 7 18:14:08 CET 2016


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  9be221b14eee137f4143e4a2243eadfb03fcfbbc (commit)
      from  59bef628b69ecea5edc8c0150fbddf9cf5cd5796 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9be221b14eee137f4143e4a2243eadfb03fcfbbc
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Mar 3 11:35:14 2016 +0000

     Bug 15968: Unnecessary loop in C4::Templates
    
    From C4::Templates::output
    
         # add variables set via param to $vars for processing
         for my $k ( keys %{ $self->{VARS} } ) {
             $vars->{$k} = $self->{VARS}->{$k};
         }
    
    This loop is not necessary, we could do the same with
    
         $vars = { %$vars, %{ $self->{VARS} } };
    
    After a quick benchmark, it gains 100 microseconds when we pass 170 vars
    to the template.
    
    Test plan:
    Do some clicks on the interface, everything should be ok.
    
    Signed-off-by: Frédéric Demians <f.demians at tamil.fr>
      Perl idiosyncratic way of merging hash, clearer, if not quicker (not
      verified)
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>

-----------------------------------------------------------------------

Summary of changes:
 C4/Templates.pm |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list