[Bug 7721] New: Global xslt parsing in XSLT.pm under mod_perl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 Priority: P5 - low Change sponsored?: --- Bug ID: 7721 Assignee: oleonard@myacpl.org Summary: Global xslt parsing in XSLT.pm under mod_perl QA Contact: koha.sekjal@gmail.com Severity: enhancement Classification: Unclassified OS: All Reporter: pelletiermaxime@gmail.com Hardware: All Status: NEW Version: rel_3_6 Component: OPAC Product: Koha When running ModPerl::PerlRunPrefork, the wrong xls file is used in opac-search and opac-detail. There is a global variable $stylesheet above the function XSLTParse4Display that keeps the cached xsl files in memory. The problem is that with mod_perl it says in memory, so it sometimes use detail xls in search and vice-versa. Patch to follow that fixes it by transforming $stylesheet into a hash to have a cache for each xls file. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 --- Comment #1 from Maxime Pelletier <pelletiermaxime@gmail.com> --- Created attachment 8218 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8218&action=edit patch to fix the problem A possible enhancement and better way to do it would be to use memcached to have a cache for everyone not using mod_perl. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff CC| |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #2 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch does not apply. Also, it would be very helpful to have a test plan for this. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 --- Comment #3 from Maxime Pelletier <pelletiermaxime@gmail.com> --- I probably won't ever have the time to redo this patch for 3.8, and anyway I doubt many (or anyone ?) besides us is running koha in mod_perl, so i'm leaving it as won't apply. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |pelletiermaxime@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7721 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |RESOLVED CC| |chris@bigballofwax.co.nz Resolution|--- |FIXED --- Comment #4 from Chris Cormack <chris@bigballofwax.co.nz> --- With the changes to the xslt files preferences this is no longer needed. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org