Attachment 39247 Details for Bug 13948 – [SIGNED-OFF] Bug 13948: Prevent explosion when Template::Plugin::Stash not installed

[patch] [SIGNED-OFF] Bug 13948: Prevent explosion when Template::Plugin::Stash not installed

SIGNED-OFF-Bug-13948-Prevent-explosion-when-Templa.patch (text/plain), 3.33 KB, created by Bernardo Gonzalez Kriegel on 2015-05-16 19:48:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bernardo Gonzalez Kriegel

Created: 2015-05-16 19:48:05 UTC

Size: 3.33 KB

patch

obsolete

>From f3fd59b8251800907da3e0967a6edd1a88447c0a Mon Sep 17 00:00:00 2001
>From: Mark Tompsett <mtompset@hotmail.com>
>Date: Fri, 15 May 2015 00:49:18 -0400
>Subject: [PATCH] [SIGNED-OFF] Bug 13948: Prevent explosion when
> Template::Plugin::Stash not installed
>
>TEST PLAN
>---------
>1) Install first two patches
>2) do not install, or uninstall Template::Plugin::Stash
>3) Upgrade to make sure system preference is added.
>4) Set the system preference to turn it on for Staff and OPAC
>5) Refresh staff -- kaboom
>6) Load OPAC -- kaboom
>7) Apply this patch
>8) Reload staff and OPAC
>   -- nice HTML comment about what is wrong.
>9) run koha qa test tools.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>
>Neat, runs well. Tested with/without sysprefs and Template::Plugin::Stash
>No koha-qa errors
>
>http://bugs.koha-community.org/show_bug.cgi?id=13848
>---
> .../prog/en/includes/doc-head-open.inc              | 21 ++++++++++++++-------
> .../bootstrap/en/includes/doc-head-open.inc         | 21 ++++++++++++++-------
> 2 files changed, 28 insertions(+), 14 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/doc-head-open.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/doc-head-open.inc
>index 0cafa59..59e6d53 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/doc-head-open.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/doc-head-open.inc
>@@ -3,15 +3,22 @@
> 
> [% USE Koha %]
> [% IF Koha.Preference('DumpTemplateVarsIntranet') %]
>-    [% USE Stash %]
>-    [% USE Dumper %]
>-    ', '- ->') %]
>-            [% Dumper.dump( Stash.stash() ) %]
>+    [% TRY %]
>+        [% USE Stash %]
>+        [% USE Dumper %]
>+        ', '- ->') %]
>+                [% Dumper.dump( Stash.stash() ) %]
>+            [% END %]
>         [% END %]
>+        -->
>+    [% CATCH %]
>+        
>     [% END %]
>-    -->
> [% END %]
> 
> [% IF ( bidi ) %]<html lang="[% lang %]" dir="[% bidi %]">[% ELSE %]<html lang="[% lang %]">[% END %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/doc-head-open.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/doc-head-open.inc
>index 4dc6cd5..62505aa 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/doc-head-open.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/doc-head-open.inc
>@@ -3,15 +3,22 @@
> 
> [% USE Koha %]
> [% IF Koha.Preference('DumpTemplateVarsOpac') %]
>-    [% USE Stash %]
>-    [% USE Dumper %]
>-    ', '- ->') %]
>-            [% Dumper.dump( Stash.stash() ) %]
>+    [% TRY %]
>+        [% USE Stash %]
>+        [% USE Dumper %]
>+        ', '- ->') %]
>+                [% Dumper.dump( Stash.stash() ) %]
>+            [% END %]
>         [% END %]
>+        -->
>+    [% CATCH %]
>+        
>     [% END %]
>-    -->
> [% END %]
> 
> [% IF ( bidi ) %]<html lang="[% lang %]" dir="[% bidi %]" class="no-js">[% ELSE %]<html lang="[% lang %]" class="no-js">[% END %]
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13948: 37444 | 37445 | 37981 | 39096 | 39097 | 39205 | 39245 | 39246 | 39247 | 39459 | 39460 | 39461 | 39994 | 39995 | 39996 | 39997