https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17982
--- Comment #5 from Marcel de Rooy <[email protected]> --- (In reply to M. Tompsett from comment #4) > Comment on attachment 59630 [details] [review] > Bug 17982: Fix the use of uniq in sub themelanguage > > Review of attachment 59630 [details] [review]: > ----------------------------------------------------------------- > > ::: C4/Templates.pm > @@ +256,4 @@ > > my $where = $tmpl =~ /xsl$/ ? 'xslt' : 'modules'; > > for my $theme (@themes) { > > if ( -e "$htdocs/$theme/$lang/$where/$tmpl" ) { > > + return ( $theme, $lang, [ uniq(@themes) ] ); > > I'm not sure the parenthesis for uniq are necessary. Seems to be a more a matter of taste. My personal preference is parentheses, only omitting them with builtins like open or print. -- 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/
