Bug 10052: QA Followup
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 13 May 2013 07:05:55 +0000 (09:05 +0200)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 23 May 2013 15:55:12 +0000 (08:55 -0700)
As Katrin pointed out, the intranetstylesheet setting was not respected in the
help pages. The script help.pl does not use get_template_and_user which sets
these variables via Auth.pm but calls gettemplate.
This makes it necessary to 'manually' set them. This patch does that.
Evidently, this could be the case for some other templates vars too..

Test plan:
Apply all patches for 10052.
Change intranetstylesheet. Check a help page.
Change intranetcolorstylesheet. Check a help page.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Looks all good to me now.
Passes tests and still no string changes.
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

help.pl

diff --git a/help.pl b/help.pl
index f641fe6..fde99d9 100755 (executable)
--- a/help.pl
+++ b/help.pl
@@ -55,6 +55,10 @@ unless ( -e "$htdocs/$theme/$lang/modules/$from" ) {
 }
 
 my $template = C4::Templates::gettemplate($from, 'intranet', $query);
-$template->param( referer => $refer );
+$template->param(
+    referer => $refer,
+    intranetstylesheet => C4::Context->preference("intranetstylesheet"),
+    intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
+);
 
 output_html_with_http_headers $query, "", $template->output;