Bug 8787 - If OpacMaintenance is on the requested page is loaded
Summary: If OpacMaintenance is on the requested page is loaded
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Authentication (show other bugs)
Version: 3.8
Hardware: All All
: P3 minor (vote)
Assignee: Jonathan Druart
QA Contact: Paul Poulain
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-19 07:34 UTC by Jonathan Druart
Modified: 2013-12-05 20:01 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
But 8787: don't load the page if OpacMaintenance is ON (1.04 KB, patch)
2012-09-19 08:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
[SIGNED-OFF] But 8787: don't load the page if OpacMaintenance is ON (1.21 KB, patch)
2012-09-19 20:03 UTC, Marc Véron
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2012-09-19 07:34:29 UTC
If the system preference OpacMaintenance is ON, the user is redirected to the maintenance.pl page but the requested page is executed.
Comment 1 Jonathan Druart 2012-09-19 08:01:19 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2012-09-19 20:03:41 UTC
Created attachment 12367 [details] [review]
[SIGNED-OFF] But 8787: don't load the page if OpacMaintenance is ON

If the syspref OpacMaintenance is ON, it is useless to loaded the
requested page.

To test:
- switch on the syspref OpacMaintenace
- check in your apache access log, zebra log, etc. the requested page is
  not loaded (i.e. on the opac-search.pl page)

Signed-off-by: Marc Veron <veron@veron.ch>
Checked by watching  other_vhosts_access.log
Works as expected
Comment 3 Jared Camins-Esakov 2012-10-06 04:23:14 UTC
Upping the severity.
Comment 4 Paul Poulain 2012-10-24 16:14:49 UTC
QA comment: one line patch

passed QA (lowered severity)
Comment 5 Paul Poulain 2012-10-24 16:15:39 UTC
Patch pushed to master
Comment 6 Chris Cormack 2012-10-24 20:21:27 UTC
Pushed to 3.8.x will be in 3.8.7
Comment 7 Liz Rea 2012-11-18 23:50:49 UTC
pushed to 3.6.x, will be in 3.6.11.