https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41896

--- Comment #5 from David Cook <[email protected]> ---
Comment on attachment 196218
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196218
Bug 41896: Remove C4::Context from Koha::Cache

Review of attachment 196218:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=41896&attachment=196218)
-----------------------------------------------------------------

::: 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.

-- 
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/

Reply via email to