Bug 12886: shelfbrowser script should write headers
authorJonathan Druart <jonathan.druart@biblibre.com>
Mon, 8 Sep 2014 14:09:22 +0000 (16:09 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Fri, 21 Nov 2014 23:54:38 +0000 (20:54 -0300)
opac/svc/shelfbrowser.pl does not write http headers.

On a production server, this patch resolves a server internal error
(500):
  Bad header=    <div id="shelfbrowser">

I did not reproduce on my local instance, but this should be fixed.

Test plan:
Verify there is no regression on browsing a shelf (with "en" and translated
templates)

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes tests and QA script, no regressions found.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
opac/svc/shelfbrowser.pl

index 33c9cde..611c503 100755 (executable)
@@ -38,4 +38,4 @@ if (C4::Context->preference("OPACShelfBrowser")) {
     }
 }
 
-print $template->output;
+output_html_with_http_headers $cgi, $cookie, $template->output;