Attachment 25301 Details for Bug 11707 – [SIGNED-OFF] Bug 11707 - Additional attributes broken for category code with space

[patch] [SIGNED-OFF] Bug 11707 - Additional attributes broken for category code with space

SIGNED-OFF-Bug-11707---Additional-attributes-broke.patch (text/plain), 2.15 KB, created by Marc Véron on 2014-02-15 10:03:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2014-02-15 10:03:03 UTC

Size: 2.15 KB

patch

obsolete

>From ac25ed035ed25eafc1a23d8ffb3b04a2eb069383 Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Fri, 7 Feb 2014 10:13:39 +0100
>Subject: [PATCH] [SIGNED-OFF] Bug 11707 - Additional attributes broken for
> category code with space
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>When editing additional attributes of a patron of a category with a code with space (ie "CAT 3"), the Javascript is broken and Additional attributes are not displayed.
>
>This patch corrects by adding simple quotes aroud category code in JS code.
>
>Test plan :
>- Create an patron additional attribute
>- Create a new patron category with a space in code. ie "CAT 3"
>- Create a new patron of this category : /cgi-bin/koha/members/memberentry.pl?op=add&categorycode=CAT 3
>=> Without patch the JS is broken (Syntax error, unrecognized expression: [data-category_code=CAT 3]) and additional attributes are not displayed.
>=> With patch the JS is not broken and additional attributes are displayed and can be edited.
>
>In current Koha version it is not possible to create categroy codes with blanks.
>To test, I changed an existing category code directly in the database.
>Then I followed the Test plan.
>Without patch, attribute types were not displayed.
>With patch, they were displayed as expected.
>
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>index 05b5b4e..1ddb001 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>@@ -99,7 +99,7 @@
>         }
>         var mytables = $(".attributes_table");
>         $(mytables).find("li").hide();
>-        $(mytables).find(" li[data-category_code="+category_code+"]").show();
>+        $(mytables).find(" li[data-category_code='"+category_code+"']").show();
>         $(mytables).find(" li[data-category_code='']").show();
>     }
> 
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 11707: 25113 | 25301 | 25567