From f1bc81d09f35bcec1e0d090aa8a41698b44b2233 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Mon, 26 Jul 2021 18:25:19 +0200
Subject: [PATCH] Bug 28445: Isolate add item form to an include file

To ease reusability
---
 .../prog/en/includes/html_helpers.inc         | 121 ++++++++++++++++++
 .../prog/en/modules/cataloguing/additem.tt    |  90 +------------
 2 files changed, 122 insertions(+), 89 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
index 86eab2547f4..6aac6ae28ec 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers.inc
@@ -58,3 +58,124 @@
         </option>
     [% END %]
 [% END %]
+
+[% BLOCK subfields_for_item %]
+    <ol>
+        [% FOREACH subfield IN subfields %]
+            [% IF subfield.kohafield == 'items.more_subfields_xml' %]
+                [% SET kohafield = 'items.more_subfields_xml_' _ subfield.subfield %]
+            [% ELSE %]
+                [% SET kohafield = subfield.kohafield %]
+            [% END %]
+
+            <li>
+                <div class="subfield_line" style="[% subfield.visibility | html %]" id="subfield[% subfield.tag | html %][% subfield.subfield | html %][% subfield.random | html %]">
+                    [% IF ( subfield.mandatory ) %]
+                        <label class="required">[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label>
+                    [% ELSE %]
+                       <label>[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label>
+                    [% END %]
+
+                    [% SET mv = subfield.marc_value %]
+                    [% IF ( mv.type == 'hidden' ) %]
+                        <input type="hidden" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]">
+                    [% ELSIF ( mv.type == 'select' ) %]
+                        [% IF ( mv.readonly || subfield.IS_RETURN_CLAIM ) %]
+                            <select name="[% kohafield | html %]" id="[%- mv.id | html -%]" class="input_marceditor" readonly="readonly" disabled="disabled" data-width="50%">
+                        [% ELSE %]
+                            <select name="[% kohafield | html %]" id="[%- mv.id | html -%]" class="input_marceditor select2" data-category="[% mv.category | html %]" data-width="50%">
+                        [% END %]
+
+                        [% FOREACH aval IN mv.values %]
+                            [% IF aval == mv.default %]
+                                <option value="[%- aval | html -%]" selected="selected">[%- mv.labels.$aval | html -%]</option>
+                            [% ELSE %]
+                                [% IF subfield.IS_LOST_AV && Koha.Preference("ClaimReturnedLostValue") && aval == Koha.Preference("ClaimReturnedLostValue") %]
+                                    <option disabled="disabled" value="[%- aval | html -%]" title="Return claims must be processed from the patron details page">[%- mv.labels.$aval | html -%]</option>
+                                [%  ELSE %]
+                                    <option value="[%- aval | html -%]">[%- mv.labels.$aval | html -%]</option>
+                                [% END %]
+                            [% END %]
+                        [% END %]
+                        </select>
+                    [% ELSIF ( mv.type == 'text_auth' ) %]
+                        [% IF mv.readonly %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
+                        [% ELSE %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
+                            [% SET dopop = "Dopop('/cgi-bin/koha/authorities/auth_finder.pl?authtypecode=\"${mv.authtypecode}\"&index=${mv.id}','${mv.id}')" %]
+                            <a href="#" class="buttonDot"  onclick="[%- dopop | html -%]; return false;" title="Tag editor">...</a>
+                        [% END %]
+                    [% ELSIF ( mv.type == 'text_plugin' ) %]
+                        [% IF mv.readonly %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
+                        [% ELSE %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
+                            [% IF ( mv.nopopup ) %]
+                                <a href="#" id="buttonDot_[%- mv.id | html -%]" class="[%- mv.class | html -%]" title="No popup">...</a>
+                            [% ELSE  %]
+                                <a href="#" id="buttonDot_[%- mv.id | html -%]" class="[%- mv.class | html -%]" title="Tag editor">...</a>
+                            [% END %]
+                            [%- mv.javascript | $raw -%]
+                        [% END %]
+                    [% ELSIF ( mv.type == 'text' ) %]
+                        [% IF mv.readonly %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
+                        [% ELSE %]
+                            <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
+                        [% END %]
+                    [% ELSIF ( mv.type == 'textarea' ) %]
+                        [% IF mv.readonly %]
+                            <textarea id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" rows="5" cols="64" readonly="readonly" >[% mv.value | html %]</textarea>
+                        [% ELSE %]
+                            <textarea id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" rows="5" cols="64" >[% mv.value | html %]</textarea>
+                        [% END %]
+                    [% END %]
+
+                    [% IF subfield.kohafield == 'items.more_subfields_xml' %]
+                        <input type="hidden" name="items.more_subfields_xml" value="[% subfield.subfield | html %]" />
+                    [% END %]
+                    <input type="hidden" name="tag"       value="[% subfield.tag | html %]" />
+                    <input type="hidden" name="subfield"  value="[% subfield.subfield | html %]" />
+                    <input type="hidden" name="mandatory" value="[% subfield.mandatory | html %]" />
+                    <input type="hidden" name="important" value="[% subfield.important | html %]" />
+
+                    [% IF add_regex %]
+                        <span name="regex_fields" style="display: none;">
+                            s/<input type="text" id="[% mv.id | html %]" name="regex_search" placeholder="regex pattern" />/
+                            <input type="text" id="[% mv.id | html %]" name="regex_replace" placeholder="regex replacement" />/
+                            <input type="text" id="[% mv.id | html %]" name="regex_modifiers" placeholder="ig" size="3" />
+                        </span>
+                    [% END %]
+
+                    [% IF add_delete_checkbox %]
+                        [% UNLESS ( subfield.mandatory ) %]
+                            <input type="checkbox" id="row[% subfield.tag | html %][% subfield.subfield | html %][% subfield.random | html %]" title="Check to delete subfield [% subfield.subfield | html %]" name="disable_input" value="[% subfield.subfield | html %]" />
+                        [% ELSE %]
+                            <span class="required">Required</span>
+                        [% END %]
+                    [% ELSE %]
+                        [% IF ( subfield.mandatory ) %] <span class="required">Required</span>
+                        [% ELSIF ( subfield.important ) %] <span class="important">Important</span>
+                        [% END %]
+                    [% END %]
+
+                    [% IF add_regex %]
+                        [% IF (mv.type == 'text' || mv.type == 'text2' || mv.type == 'textarea' ) %]
+                            <a href="#" name="field_regex" id="[% subfield.id | html %]" >RegEx</a>
+                        [% END %]
+                    [% END %]
+
+                    [% IF ( subfield.repeatable ) %]
+                        <a href="#" class="buttonPlus" onclick="CloneItemSubfield(this.parentNode.parentNode); return false;">
+                            <img src="[% interface | html %]/[% theme | html %]/img/clone-subfield.png" alt="Clone" title="Clone this subfield" />
+                        </a>
+                    [% END %]
+
+                    <span class="hint" id="hint[% subfield.tag | html %][% subfield.subfield | html %][% subfield.random | html %]"></span>
+
+                </div>
+            </li>
+        [% END %]
+    </ol>
+[% END %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
index 345329ecc7a..0988065fb6b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
@@ -153,95 +153,7 @@
         <h2 id="edititem">Edit Item #[% itemnumber | html %][% IF ( barcode ) %] / Barcode [% barcode | html %][% END %]</h2>
     [% END %]
 	<fieldset class="rows">
-	<ol>
-        [% FOREACH subfield IN subfields %]
-
-                [% IF subfield.kohafield == 'items.more_subfields_xml' %]
-                    [% SET kohafield = 'items.more_subfields_xml_' _ subfield.subfield %]
-                [% ELSE %]
-                    [% SET kohafield = subfield.kohafield %]
-                [% END %]
-
-               <li><div class="subfield_line" style="[% subfield.visibility | html %]" id="subfield[% subfield.tag | html %][% subfield.subfield | html %][% subfield.random | html %]">
-                [% IF ( subfield.mandatory ) %]
-               <label class="required">[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label>
-               [% ELSE %]
-               <label>[% subfield.subfield | html %] - [% subfield.marc_lib | $raw %]</label>
-               [% END %]
-
-                [% SET mv = subfield.marc_value %]
-                [% IF ( mv.type == 'hidden' ) %]
-                    <input type="hidden" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]">
-                [% ELSIF ( mv.type == 'select' ) %]
-                    [% IF ( mv.readonly || subfield.IS_RETURN_CLAIM ) %]
-                        <select name="[% kohafield | html %]" id="[%- mv.id | html -%]" class="input_marceditor" readonly="readonly" disabled="disabled" data-width="50%">
-                    [% ELSE %]
-                        <select name="[% kohafield | html %]" id="[%- mv.id | html -%]" class="input_marceditor" data-category="[% mv.category | html %]" data-width="50%">
-                    [% END %]
-                    [% FOREACH aval IN mv.values %]
-                        [% IF aval == mv.default %]
-                        <option value="[%- aval | html -%]" selected="selected">[%- mv.labels.$aval | html -%]</option>
-                        [% ELSE %]
-                            [% IF subfield.IS_LOST_AV && Koha.Preference("ClaimReturnedLostValue") && aval == Koha.Preference("ClaimReturnedLostValue") %]
-                                <option disabled="disabled" value="[%- aval | html -%]" title="Return claims must be processed from the patron details page">[%- mv.labels.$aval | html -%]</option>
-                            [%  ELSE %]
-                                <option value="[%- aval | html -%]">[%- mv.labels.$aval | html -%]</option>
-                            [% END %]
-                        [% END %]
-                    [% END %]
-                    </select>
-                [% ELSIF ( mv.type == 'text_auth' ) %]
-                    [% IF mv.readonly %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
-                    [% ELSE %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
-                        [% SET dopop = "Dopop('/cgi-bin/koha/authorities/auth_finder.pl?authtypecode=\"${mv.authtypecode}\"&index=${mv.id}','${mv.id}')" %]
-                        <a href="#" class="buttonDot"  onclick="[%- dopop | html -%]; return false;" title="Tag editor">...</a>
-                    [% END %]
-                [% ELSIF ( mv.type == 'text_plugin' ) %]
-                    [% IF mv.readonly %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
-                    [% ELSE %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
-                        [% IF ( mv.nopopup ) %]
-                            <a href="#" id="buttonDot_[%- mv.id | html -%]" class="[%- mv.class | html -%]" title="No popup">...</a>
-                        [% ELSE  %]
-                            <a href="#" id="buttonDot_[%- mv.id | html -%]" class="[%- mv.class | html -%]" title="Tag editor">...</a>
-                        [% END %]
-                        [%- mv.javascript | $raw -%]
-                    [% END %]
-                [% ELSIF ( mv.type == 'text' ) %]
-                    [% IF mv.readonly %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" readonly="readonly" />
-                    [% ELSE %]
-                        <input type="text" id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" maxlength="[%- mv.maxlength | html -%]" value="[%- mv.value | html -%]" />
-                    [% END %]
-                [% ELSIF ( mv.type == 'textarea' ) %]
-                    [% IF mv.readonly %]
-                        <textarea id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" rows="5" cols="64" readonly="readonly" >[% mv.value | html %]</textarea>
-                    [% ELSE %]
-                        <textarea id="[%- mv.id | html -%]" name="[% kohafield | html %]" class="input_marceditor" rows="5" cols="64" >[% mv.value | html %]</textarea>
-                    [% END %]
-                [% END %]
-
-                [% IF subfield.kohafield == 'items.more_subfields_xml' %]
-                    <input type="hidden" name="items.more_subfields_xml" value="[% subfield.subfield | html %]" />
-                [% END %]
-                <input type="hidden" name="tag"       value="[% subfield.tag | html %]" />
-                <input type="hidden" name="subfield"  value="[% subfield.subfield | html %]" />
-                <input type="hidden" name="mandatory" value="[% subfield.mandatory | html %]" />
-                <input type="hidden" name="important" value="[% subfield.important | html %]" />
-                [% IF ( subfield.repeatable ) %]
-                    <a href="#" class="buttonPlus" onclick="CloneItemSubfield(this.parentNode.parentNode); return false;">
-                        <img src="[% interface | html %]/[% theme | html %]/img/clone-subfield.png" alt="Clone" title="Clone this subfield" />
-                    </a>
-                [% END %]
-                [% IF ( subfield.mandatory ) %] <span class="required">Required</span>
-                [% ELSIF ( subfield.important ) %] <span class="important">Important</span>
-                [% END %]
-            </div></li>
-        [% END %]
-    </ol>
+        [% PROCESS subfields_for_item subfields => subfields %]
     </fieldset>
     <input type="hidden" name="indicator" value=" " />
     <input type="hidden" name="indicator" value=" " />
-- 
2.20.1