https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41896 --- Comment #10 from Jonathan Druart <jonathan.druart@gmail.com> --- (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.