From 8f83d773ada85e3c092f68291d1ec4468f84135d Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 21 May 2021 17:11:09 +0000
Subject: [PATCH] Bug 28423: JavaScript error on MARC modifications page

This patch makes a minor change to the MARC modifications template so
that the "mmtas" variable isn't defined if there is no JSON to be
assigned as its value.

To test, apply the patch and go to Administration -> MARC modification
templates.

 - If necessary, add a template with at least one action.
 - Check the browser console, there should be no errors.
 - Click the "Edit" button corresponding to one of the template actions.
 - The details of the action should load correctly in the edit form and
   there should be no errors in the console.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../prog/en/modules/tools/marc_modification_templates.tt      | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/marc_modification_templates.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/marc_modification_templates.tt
index f8698b28a0..6459266260 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/marc_modification_templates.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/marc_modification_templates.tt
@@ -354,7 +354,9 @@
     [% Asset.js("js/tools-menu.js") | $raw %]
     [% Asset.js("js/marc_modification_templates.js") | $raw %]
     <script>
-        var mmtas = [% ActionsLoop.json | $raw %]
+        [% IF ActionsLoop %]
+            var mmtas = [% ActionsLoop.json | $raw %]
+        [% END %]
     </script>
 [% END %]
 
-- 
2.39.5