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