[Koha-bugs] [Bug 28519] Add a 2nd directory for Perl modules

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #32 from David Cook <dcook at prosentient.com.au> ---
(In reply to Jonathan Druart from comment #31)
> Not directly related to the discussion, but see also bug 17427 comment 29.
> When we will move (finally) to Data::Session we will hit a problem with the
> package not in /usr/share/perl5/CGI/Session/Serialize/ (not along with
> Serialize.pm at least).

That's a good point (although wouldn't it be
"/usr/share/perl5/Data/Session/Serialize"?). That said, I feel like this is an
argument against moving to Data::Session. That get_my_serializers function is a
dreadful bit of code. 

I'm not seeing any packages for Data::Session in Debian or Ubuntu and it seems
like it has a lot of dependencies. Also, Ron Savage took over CGI::Session and
self-declared that his Data::Session was the successor to CGI::Session. Do we
have any evidence that anyone actually uses it?

I'd be tempted to either use Plack::Middleware::Session and finally put the
nail in CGI, or write our own Koha::Session module. It doesn't need to be very
complicated.

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