Attachment 37960 Details for Bug 10299 – Bug 10299: authority plugin - authority field are reversed

[patch] Bug 10299: authority plugin - authority field are reversed

Bug-10299-authority-plugin---authority-field-are-r.patch (text/plain), 2.13 KB, created by Jonathan Druart on 2015-04-16 13:57:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-04-16 13:57:08 UTC

Size: 2.13 KB

patch

obsolete

>From 73e4dca987c33da0f144fec92641eb4b273eee7b Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Thu, 16 Apr 2015 15:52:33 +0200
>Subject: [PATCH] Bug 10299: authority plugin - authority field are reversed
>
>If you click the plugin next to an authority field that is already
>populated it takes the values from the field and puts them in the search
>in the reverse order.
>
>Test plan:
>1/ Edit a biblio
>2/ Fill the subfield for an authority field (for instance 650$v, 650$x,
>650$y, etc.)
>3/ Click on the authority plugin
>Before this patch, the values were concatenated in the reverse order.
>With this patch, it should not.
>---
> koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js b/koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js
>index 70ae6f9..e1fa828 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js
>+++ b/koha-tmpl/intranet-tmpl/prog/en/js/cataloging.js
>@@ -40,17 +40,18 @@ function openAuth(tagsubfieldid,authtype,source) {
>     var tagsubfield=getTagInputnameFilter(tagsubfieldid);
>     var elementsubfcode=getSubfieldCode(element.name);
>     var mainmainstring=element.value;
>-    var mainstring="";
>+    var mainstring = new Array();
>     var inputs = element.parentNode.parentNode.getElementsByTagName("input");
> 
>     for (var myindex =0; myindex<inputs.length;myindex++){
>         if (inputs[myindex].name && inputs[myindex].name.match(tagsubfield)){
>             var subfieldcode=getSubfieldCode(inputs[myindex].name);
>             if (isNaN(parseInt(subfieldcode)) && inputs[myindex].value != "" && subfieldcode!=elementsubfcode){
>-                mainstring=inputs[myindex].value+" "+mainstring;
>+                mainstring.push(inputs[myindex].value);
>             }
>         }
>     }
>+    mainstring = mainstring.join(' ');
>     newin=window.open("../authorities/auth_finder.pl?source="+source+"&authtypecode="+authtype+"&index="+tagsubfieldid+"&value_mainstr="+encodeURI(mainmainstring)+"&value_main="+encodeURI(mainstring), "_blank",'width=700,height=550,toolbar=false,scrollbars=yes');
> }
> 
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 10299: 18269 | 37960 | 39199 | 39293