Attachment 85593 Details for Bug 22404 – Bug 22404: Fix parameter "for" of labels in subscription add form

[patch] Bug 22404: Fix parameter "for" of labels in subscription add form

Bug-22404-Fix-parameter-for-of-labels-in-subscript.patch (text/plain), 2.98 KB, created by Josef Moravec on 2019-02-25 08:49:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2019-02-25 08:49:26 UTC

Size: 2.98 KB

patch

obsolete

>From 64508cc2c92ed6f136cb2fa993a5da77821bdb3b Mon Sep 17 00:00:00 2001
>From: Josef Moravec <josef.moravec@gmail.com>
>Date: Mon, 25 Feb 2019 08:46:11 +0000
>Subject: [PATCH] Bug 22404: Fix parameter "for" of labels in subscription add
> form
>
>Attribute "for" of tag labelshould contain input id, not name.
>
>Test plan:
>1) Go to serials -> add subscription second step
>2) Click on labels of these three fields:
>    - First issue publication date
>    - Subscription start date
>    - Subscription end date
>
>--> without patch, nothing happen
>--> with path, the datepicker is shown
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt
>index 7fb822e187..714bd755b6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt
>@@ -224,7 +224,7 @@ fieldset.rows li.radio { width: 100%; } /* override staff-global.css */
>                                 <legend>Serials planning</legend>
>                                 <ol>
>                                     <li>
>-                                        <label for="firstacquidate" class="required">First issue publication date:</label>
>+                                        <label for="acqui_date" class="required">First issue publication date:</label>
>                                         [% UNLESS (more_than_one_serial) %]
>                                           <input type="text" size="10" id="acqui_date" name="firstacquidate" value="[% firstacquidate | $KohaDates %]" class="datepicker required" required="required" />
>                                         [% ELSE %]
>@@ -288,11 +288,11 @@ fieldset.rows li.radio { width: 100%; } /* override staff-global.css */
>                                         <input type="hidden" name="issuelengthcount">
>                                     </li>
>                                     <li>
>-                                        <label for="startdate" class="required"> Subscription start date:</label>
>+                                        <label for="from" class="required"> Subscription start date:</label>
>                                         <input type="text" size="10" id="from" name="startdate" value="[% startdate | $KohaDates %]" class="datepickerfrom required" required="required" />
>                                     </li>
>                                     <li>
>-                                        <label for="enddate">Subscription end date:</label>
>+                                        <label for="to">Subscription end date:</label>
>                                         <input type="text" size="10" id="to" name="enddate" value="[% enddate | $KohaDates %]" class="datepickerto" />
>                                     </li>
>                                     <li>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 22404: 85593 | 85721 | 85805