Attachment 40246 Details for Bug 12616 – Bug 12616: Locale in subscriptions not preselecting correctly

[patch] Bug 12616: Locale in subscriptions not preselecting correctly

Bug-12616-Locale-in-subscriptions-not-preselecting.patch (text/plain), 1.53 KB, created by Bernardo Gonzalez Kriegel on 2015-06-17 16:32:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bernardo Gonzalez Kriegel

Created: 2015-06-17 16:32:52 UTC

Size: 1.53 KB

patch

obsolete

>From 47d91e84855d740fd0d57198160b3e8340c5207b Mon Sep 17 00:00:00 2001
>From: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>Date: Wed, 17 Jun 2015 13:22:49 -0300
>Subject: [PATCH] Bug 12616: Locale in subscriptions not preselecting
> correctly
>
>There is a problem if a language is present but
>don't have ISO639-2 code. Locale pulldown on serial
>suscription is malformed.
>
>To reproduce on master:
>a) remove some entries on language_rfc4646_to_iso639
>b) go to Serials > New suscription
>c) Put any value on Vendor and record, press Next>>
>d) Look at locale pulldown, it must default to last
>removed lang from a), also other langs has no value
>and are also 'selected' on html
>
>To test:
>1) Reproduce the bug
>2) Apply the patch
>3) Add New suscription, pulldown must be fixed
>---
> C4/Languages.pm |    3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/C4/Languages.pm b/C4/Languages.pm
>index be907ee..384fd7a 100644
>--- a/C4/Languages.pm
>+++ b/C4/Languages.pm
>@@ -249,7 +249,8 @@ sub getLanguages {
>                 $language_subtag_registry->{language_description} = $language_descriptions->{description};
>             }
>         }
>-        if ( !$language_list || index (  $language_list, $language_subtag_registry->{ iso639_2_code } ) >= 0) {
>+        # Do not push unless valid iso639-2 code
>+        if ( $language_subtag_registry->{ iso639_2_code } and ( !$language_list || index (  $language_list, $language_subtag_registry->{ iso639_2_code } ) >= 0) ) {
>             push @languages_loop, $language_subtag_registry;
>         }
>     }
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 12616: 40246 | 40250 | 40254