Summary: | Manage C4::VirtualShelves in C4::Auth for performance | ||
---|---|---|---|
Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
Component: | Architecture, internals, and plumbing | Assignee: | Fridolin Somers <fridolin.somers> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | jonathan.druart, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 7923, 13998 | ||
Attachments: |
Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance
Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance [PASSED QA] Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance |
Description
Fridolin Somers
2015-03-17 13:49:08 UTC
Created attachment 36950 [details] [review] Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance In C4::Auth there is a use C4::VirtualShelves. Virtualshelves are displayed in all OPAC pages, but not in intranet. For performance, we should move this into a require only for opac pages. This patch adds a condition to fetch virtualshelves only if opac and moves the dependancy on C4::VirtualShelves into require calls. On my desktop, I have those compilation times for C4/Auth.pm : - Without patch : 0,41 seconds - With patch : 0,22 seconds This performance improvement is very usefull for pages that only use a few as dependancy, like errors/404.pl Test plan : - Be sure there are some public lists - Apply patch - Go to opac (not logged-in) - Click on "Lists" - Check you see the public lists - Login with a user - Be sure this user has some private lists - Click on "Lists" - Check you see the public and private lists - Logout - Go to /cgi-bin/koha/opac-reserve.pl - You see the loggin page - Click on "Lists" - Check you see the public lists - Go to intranet - Check you can loggin Created attachment 37002 [details] [review] Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance In C4::Auth there is a use C4::VirtualShelves. Virtualshelves are displayed in all OPAC pages, but not in intranet. For performance, we should move this into a require only for opac pages. This patch adds a condition to fetch virtualshelves only if opac and moves the dependancy on C4::VirtualShelves into require calls. On my desktop, I have those compilation times for C4/Auth.pm : - Without patch : 0,41 seconds - With patch : 0,22 seconds This performance improvement is very usefull for pages that only use a few as dependancy, like errors/404.pl Test plan : - Be sure there are some public lists - Apply patch - Go to opac (not logged-in) - Click on "Lists" - Check you see the public lists - Login with a user - Be sure this user has some private lists - Click on "Lists" - Check you see the public and private lists - Logout - Go to /cgi-bin/koha/opac-reserve.pl - You see the loggin page - Click on "Lists" - Check you see the public lists - Go to intranet - Check you can loggin Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 37315 [details] [review] [PASSED QA] Bug 13852 - Manage C4::VirtualShelves in C4::Auth for performance In C4::Auth there is a use C4::VirtualShelves. Virtualshelves are displayed in all OPAC pages, but not in intranet. For performance, we should move this into a require only for opac pages. This patch adds a condition to fetch virtualshelves only if opac and moves the dependancy on C4::VirtualShelves into require calls. On my desktop, I have those compilation times for C4/Auth.pm : - Without patch : 0,41 seconds - With patch : 0,22 seconds This performance improvement is very usefull for pages that only use a few as dependancy, like errors/404.pl Test plan : - Be sure there are some public lists - Apply patch - Go to opac (not logged-in) - Click on "Lists" - Check you see the public lists - Login with a user - Be sure this user has some private lists - Click on "Lists" - Check you see the public and private lists - Logout - Go to /cgi-bin/koha/opac-reserve.pl - You see the loggin page - Click on "Lists" - Check you see the public lists - Go to intranet - Check you can loggin Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Patch pushed to master. Thanks (for the Pastis) Frido! |