Attachment 27653 Details for Bug 11769 – [PASSED QA] Bug 11769 [Follow-up] Use validation plugin when creating new MARC framework

[patch] [PASSED QA] Bug 11769 [Follow-up] Use validation plugin when creating new MARC framework

PASSED-QA-Bug-11769-Follow-up-Use-validation-plugi.patch (text/plain), 2.66 KB, created by Katrin Fischer on 2014-04-26 13:04:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2014-04-26 13:04:21 UTC

Size: 2.66 KB

patch

obsolete

>From 1bf274e2c22b8c7ec33ab6324da20f0e3f0c2b20 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 17 Mar 2014 13:13:00 -0400
>Subject: [PATCH] [PASSED QA] Bug 11769 [Follow-up] Use validation plugin when
> creating new MARC framework
>
>This follow up adds the correct "required" style to the required form
>fields as well as the "required" text after each required field.
>
>To test, apply all patches and go to Administration -> MARC
>bibliographic framework -> New framework. The framework code and
>description fields should show that they are required.
>
>Signed-off-by: David Cook <dcook@prosentient.com.au>
>
>Works as described and feels a million times better.
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Works as described, template changes only.
>---
> .../prog/en/modules/admin/biblio_framework.tt           | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt
>index 4ef8103..bc66f60 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt
>@@ -116,11 +116,20 @@
>             <input type="hidden" name="modif" value="1" />
>         </li>
> 	[% ELSE %]
>-        <li><label for="frameworkcode">Framework code: </label><input type="text" id="frameworkcode" name="frameworkcode" size="4" maxlength="4" onblur="toUC(this)" required="required" class="required" /></li>
>+        <li>
>+            <label for="frameworkcode" class="required">Framework code: </label>
>+            <input type="text" id="frameworkcode" name="frameworkcode" size="4" maxlength="4" onblur="toUC(this)" required="required" class="required" />
>+            <span class="required">Required</span>
>+    </li>
> 	[% END %]
>-        <li><label for="description">Description: </label>
>-        <input type="text" name="frameworktext" id="description" size="40" maxlength="80" value="[% frameworktext |html %]" required="required" class="required" /></li></ol></fieldset>
>-        <fieldset class="action">	<input type="submit" value="Submit" class="submit" /></fieldset>
>+        <li>
>+            <label for="description" class="required">Description: </label>
>+            <input type="text" name="frameworktext" id="description" size="40" maxlength="80" value="[% frameworktext |html %]" required="required" class="required" />
>+            <span class="required">Required</span>
>+        </li>
>+    </ol>
>+    </fieldset>
>+    <fieldset class="action"><input type="submit" value="Submit" class="submit" /></fieldset>
>     </form>
> [% END %]
> 
>-- 
>1.8.3.2

Actions: View | Diff | Splinter Review

Attachments on bug 11769: 25295 | 25902 | 26417 | 27513 | 27514 | 27652 | 27653