https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42663 --- Comment #12 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to David Cook from comment #8)
(In reply to David Cook from comment #7)
But the problem is Koha isn't an MVC web app like Catalyst, and Catlayst also only applies to web apps. It doesn't cover things like CLI tools or cronjobs. (That's why you end up with modules like https://metacpan.org/pod/Config::JFDI which provide Catlayst config outside of a web context.)
I mean I say it's a problem... I suppose we'd have those other components that can be used in the CLI tools and cronjobs so that's good... that should take care of those situations.
Maybe the only problem is that Koha::Context->current().
Koha::Context shouldn't depend on Koha::Middleware::Context. In fact, I don't think that we really need Koha::Middleware::Context. All Koha::Context->current() needs to do is return a singleton. So check for the $ENV{"koha.context"}. If it's set, return its contents. If it's not set, set it. That's it.
We cannot store structures in ENV, only strings. That's why I need to reuse the psgi_env var from the middleware. -- You are receiving this mail because: You are watching all bug changes.