Attachment 98794 Details for Bug 24649 – Bug 24649: Cloning item subfields misses a <li> tag

[patch] Bug 24649: Cloning item subfields misses a <li> tag

Bug-24649-Cloning-item-subfields-misses-a-li-tag.patch (text/plain), 1.65 KB, created by Marcel de Rooy on 2020-02-13 09:10:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2020-02-13 09:10:29 UTC

Size: 1.65 KB

patch

obsolete

>From 13973ac4a7bee331d0fbb225194dff0cff4c9185 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Thu, 13 Feb 2020 08:56:35 +0000
>Subject: [PATCH] Bug 24649: Cloning item subfields misses a <li> tag
>Content-Type: text/plain; charset=utf-8
>
>Instead of the parentNode (which is a div), we need the parent of the div
>(which is a li).
>So a simple fix.
>
>Test plan:
>Mark item subfield URI repeatable.
>Clone the subfield.
>Notice the difference between patch applied or not applied.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>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 641053149c..9149bb23e9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
>@@ -197,7 +197,7 @@
>                 <input type="hidden" name="subfield"  value="[% ite.subfield | html %]" />
>                 <input type="hidden" name="mandatory" value="[% ite.mandatory | html %]" />
>                 [% IF ( ite.repeatable ) %]
>-                    <a href="#" class="buttonPlus" onclick="CloneItemSubfield(this.parentNode); return false;">
>+                    <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 %]
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24649: 98794 | 98795 | 98797 | 98798 | 98829 | 98887 | 98888 | 98949 | 98958 | 98959