[Koha-bugs] [Bug 15562] Make Koha more suitable for shared hosting with plack

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Feb 1 02:49:45 CET 2016


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

--- Comment #6 from Srdjan Jankovic <srdjan at catalyst.net.nz> ---
I have no specific plan for this. Existing code change should be transparent
and is mainly in two core files
C4:Context:
* in the way it is used (initialised  and imported)
* the way sysprefs are cached
* default cache is now stored in context rather than being a singleton
Koha::Database:
* new_schema() now takes optional context param
There's a number of files which have as a consequence all
Koha::Cache->get_instance calls replaced with C4::Context->cache

Arguably I should have replaced Koha::Database->[new()]->schema calls with
C4::Context->schema, to get it in line with the cache change, (and maybe I
will), but that would mean changing every file. Besides
Koha::Database->new()->schema hurts your eyes and soul much more than
C4::Context->schema so we may feel some pressure to change the way we are using
the database handle.

I assume all relevant functionality is covered in the tests, and tests passing
with MEMCACHED_SERVERS set is a good starting point.

Otherwise what I tried was
* going to (many) pages, checking that context is loaded properly, and the
database connection is ok
* checking that sysprefs caching works

I can only suggest checking other pages which use cache.

The change is not deployed on our test/demo sites yet, but will be shortly.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list