Attachment 35792 Details for Bug 11395 – Bug 11395: A control field is a field with tag < 10

[patch] Bug 11395: A control field is a field with tag < 10

Bug-11395-A-control-field-is-a-field-with-tag--10.patch (text/plain), 2.08 KB, created by Tomás Cohen Arazi (tcohen) on 2015-02-10 13:56:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2015-02-10 13:56:04 UTC

Size: 2.08 KB

patch

obsolete

>From b5d530fc4f452dee7a4f3caef7e089a70d0c639c Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Mon, 15 Dec 2014 10:29:36 +0100
>Subject: [PATCH] Bug 11395: A control field is a field with tag < 10
>
>This patch fixes the existing test about control fields.
>A control field is < 10, not <=10!
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
>---
> .../prog/en/modules/tools/marc_modification_templates.tt          | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>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 9f561db..bfef366 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
>@@ -23,13 +23,13 @@ $(document).ready(function() {
>                 alert(_("The destination should be filled."));
>                 return false;
>             }
>-            if ( ( $("#to_field").val()   <= 10 && $("#to_subfield").val().length   > 0 ) ||
>-                 ( $("#from_field").val() <= 10 && $("#from_subfield").val().length > 0 ) ) {
>+            if ( ( $("#to_field").val()   < 10 && $("#to_subfield").val().length   > 0 ) ||
>+                 ( $("#from_field").val() < 10 && $("#from_subfield").val().length > 0 ) ) {
>                  alert(_("If the field is a control field, the subfield should be empty"));
>                  return false;
>             }
>-            if ( ( $("#from_field").val() <= 10 && $("#to_subfield").val().length   == 0 ) ||
>-                 ( $("#to_field").val()   <= 10 && $("#from_subfield").val().length == 0 ) ) {
>+            if ( ( $("#from_field").val() < 10 && $("#to_subfield").val().length   == 0 ) ||
>+                 ( $("#to_field").val()   < 10 && $("#from_subfield").val().length == 0 ) ) {
>                 alert(_("A control field cannot be used with a regular field."));
>                 return false;
>              }
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 11395: 23502 | 23503 | 23504 | 23505 | 23506 | 23507 | 23508 | 23592 | 24634 | 26072 | 26981 | 28978 | 28999 | 29000 | 29001 | 29002 | 29003 | 29004 | 29056 | 29509 | 29569 | 31344 | 32181 | 32182 | 32184 | 32239 | 32240 | 32241 | 32242 | 32243 | 32244 | 32245 | 32246 | 32247 | 32248 | 32249 | 33145 | 33146 | 33147 | 33148 | 33149 | 33150 | 33151 | 33152 | 33153 | 33154 | 33155 | 34393 | 34394 | 34395 | 34396 | 34397 | 34398 | 34399 | 34400 | 34401 | 34402 | 34552 | 34553 | 34554 | 34555 | 34556 | 34557 | 34558 | 34559 | 34560 | 34561 | 34562 | 35783 | 35784 | 35785 | 35786 | 35787 | 35788 | 35789 | 35790 | 35791 | 35792 | 36295 | 36296 | 36297 | 36298 | 36299 | 36300 | 36301 | 36302 | 36303 | 36304