https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16088
--- Comment #22 from Jacek Ablewicz <[email protected]> --- Looks good, tests good and works really good - now there is only one CGI->new() call! Aparently get_template_and_user(), which usually triggers the 1st getlanguage() call is doing it properly with $cgi parameter included. Not sure if cache clean in setlanguagecookie() is strictly necessary, but it's not harming performance in any way, and this sub is used also in installer, better to be on the safe side. Koha::Cache::set_in_L1_cache_only() may need some kind of unit test, if it's intended to stay there. In t/Cache.t skip "Cache not enabled", 33 unless ( $cache->is_cache_active() && defined $cache ); number of tests to skip is out of sync, but it was out of sync before this patch as well. -- 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/
