Bug 10595: don't display OpacTopissue page when system preference is turned off
authorMirko Tietgen <mirko@abunchofthings.net>
Tue, 16 Jul 2013 19:51:01 +0000 (21:51 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 14 Aug 2013 13:47:21 +0000 (10:47 -0300)
commit094bfd0d17689c92a4e9366af63ffb7fcbe99399
tree8eb503a16c4296499a86c6c1634c75b2cf2ff34b
parent1ebaaa8c4b3c27afa9fd2ef67a9430fafb82b544
Bug 10595: don't display OpacTopissue page when system preference is turned off

When the syspref is turned off, the link is hidden but the page is still
accessible. This patch checks the syspref and redirects to 404 if it is disabled.

Test plan:

Before applying the patch

Enable OpacTopissue
  Observe there is a link to opac-topissues.pl next to Advanced search etc.
  Follow that link, observe that opac-topissues.pl is loading
Disable OpacTopissue
  Observe the link is gone
  Manually open /koha/opac-topissues.pl and check that it is loading even though the feature is disabled

Apply patch

(OpacTopissue still diabled)
Manually open /koha/opac-topissues.pl and observe that you are being redirected to the 404 page.
Enable OpacTopissue
  Follow the link or manuall open opac-topissues.pl and check that it is working

Signoff the patch.

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 04ca81bc358f1bc6425e1cef431d9e4abfc744d8)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Works as expected
opac/opac-topissues.pl