Bug 8787

Summary: If OpacMaintenance is on the requested page is loaded
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: AuthenticationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Paul Poulain <paul.poulain>
Severity: minor    
Priority: P3 CC: chris, dpavlin, liz, paul.poulain, veron
Version: 3.8   
Hardware: All   
OS: All   
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
[SIGNED-OFF] But 8787: don't load the page if OpacMaintenance is ON

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.