diff public/javascripts/select_list_move.js @ 1464:261b3d9a4903 redmine-2.4

Update to Redmine 2.4 branch rev 12663
author Chris Cannam
date Tue, 14 Jan 2014 14:37:42 +0000
parents 433d4f72a19b
children dffacf8a6908
line wrap: on
line diff
--- a/public/javascripts/select_list_move.js	Fri Jun 14 09:05:06 2013 +0100
+++ b/public/javascripts/select_list_move.js	Tue Jan 14 14:37:42 2014 +0000
@@ -1,14 +1,12 @@
 var NS4 = (navigator.appName == "Netscape" && parseInt(navigator.appVersion) < 5);
 
-function addOption(theSel, theText, theValue)
-{
+function addOption(theSel, theText, theValue) {
   var newOpt = new Option(theText, theValue);
   var selLength = theSel.length;
   theSel.options[selLength] = newOpt;
 }
 
-function swapOptions(theSel, index1, index2)
-{
+function swapOptions(theSel, index1, index2) {
   var text, value;
   text = theSel.options[index1].text;
   value = theSel.options[index1].value;
@@ -18,42 +16,31 @@
   theSel.options[index2].value = value;
 }
 
-function deleteOption(theSel, theIndex)
-{
+function deleteOption(theSel, theIndex) {
   var selLength = theSel.length;
-  if(selLength>0)
-  {
+  if (selLength > 0) {
     theSel.options[theIndex] = null;
   }
 }
 
-function moveOptions(theSelFrom, theSelTo)
-{
-
+function moveOptions(theSelFrom, theSelTo) {
   var selLength = theSelFrom.length;
   var selectedText = new Array();
   var selectedValues = new Array();
   var selectedCount = 0;
-
   var i;
-
-  for(i=selLength-1; i>=0; i--)
-  {
-    if(theSelFrom.options[i].selected)
-    {
+  for (i = selLength - 1; i >= 0; i--) {
+    if (theSelFrom.options[i].selected) {
       selectedText[selectedCount] = theSelFrom.options[i].text;
       selectedValues[selectedCount] = theSelFrom.options[i].value;
       deleteOption(theSelFrom, i);
       selectedCount++;
     }
   }
-
-  for(i=selectedCount-1; i>=0; i--)
-  {
+  for (i = selectedCount - 1; i >= 0; i--) {
     addOption(theSelTo, selectedText[i], selectedValues[i]);
   }
-
-  if(NS4) history.go(0);
+  if (NS4) history.go(0);
 }
 
 function moveOptionUp(theSel) {
@@ -73,11 +60,7 @@
 }
 
 // OK
-function selectAllOptions(id)
-{
-  var select = $('#'+id);/*
-  for (var i=0; i<select.options.length; i++) {
-    select.options[i].selected = true;
-  }*/
+function selectAllOptions(id) {
+  var select = $('#'+id);
   select.children('option').attr('selected', true);
 }