https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41896
--- Comment #10 from Jonathan Druart <[email protected]> --- (In reply to David Cook from comment #5) > Comment on attachment 196218 [details] [review] > Bug 41896: Remove C4::Context from Koha::Cache > > Review of attachment 196218 [details] [review]: > ----------------------------------------------------------------- > > ::: Koha/Cache.pm > @@ +75,4 @@ > > # Should we continue to support MEMCACHED ENV vars? > > $self->{'namespace'} ||= $ENV{MEMCACHED_NAMESPACE}; > > my @servers = split /,/, $ENV{MEMCACHED_SERVERS} || ''; > > + $self->{namespace} = > > Koha::Config->get_instance->get('memcached_namespace') || 'koha'; > > By replacing ||= with = we're making the above checking of the > MEMCACHED_NAMESPACE environmental variable pointless. Are we sure we want to > do that? If so, then we should remove support for those ENV variables. No idea why I've done that! I've restored ||= -- 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/
