Author: jleroux
Date: Mon Dec 27 21:36:33 2010
New Revision: 1053198

URL: http://svn.apache.org/viewvc?rev=1053198&view=rev
Log:
Use the power of Aptana to format correctly this file. I have adapted the 
default format setting to be more near our best practices. If you are 
interested I can share the formatting template. 

Anyway I think I will discuss that on dev ML latter. We could have it posted in 
the wiki, when agreed upon its content, and use it as part of our best 
practices...

Modified:
    ofbiz/trunk/framework/images/webapp/images/getDependentDropdownValues.js

Modified: 
ofbiz/trunk/framework/images/webapp/images/getDependentDropdownValues.js
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/getDependentDropdownValues.js?rev=1053198&r1=1053197&r2=1053198&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/getDependentDropdownValues.js 
(original)
+++ ofbiz/trunk/framework/images/webapp/images/getDependentDropdownValues.js 
Mon Dec 27 21:36:33 2010
@@ -16,7 +16,6 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-
 // *** getDependentDropdownValues allows to dynamically populate a dependent 
drop-down on change on its parent drop-down, doesn't require any fixed naming 
convention 
 // request      = request calling the service which retrieve the info from the 
DB, ex: getAssociatedStateList
 // paramKey     = parameter value used in the called service 
@@ -33,100 +32,103 @@
 //                               this is to handle a specific case where an 
input field is needed instead of a drop-down when no values are returned by the 
request
 //                               this will be maybe extended later to use an 
auto-completed drop-down or a lookup, instead of straight drop-down currently, 
when there are too much values to populate
 //                               this is e.g. currently used in the Product 
Price Rules screen
-function getDependentDropdownValues(request, paramKey, paramField, 
targetField, responseName, keyName, descName, selected, callback, hide, 
hideTitle, inputField) {
-       target = '#' + targetField;
-       input = '#' + inputField;
+function getDependentDropdownValues(request, paramKey, paramField, 
targetField, responseName, keyName, descName, selected, callback, hide, 
hideTitle, inputField){
+    target = '#' + targetField;
+    input = '#' + inputField;
     targetTitle = target + '_title'
     optionList = '';
     jQuery.ajax({
         url: request,
-        data: [ { name: paramKey, value: jQuery('#' + paramField).val()} ],  
// get requested value from parent drop-down field
-        dataType : 'json',
+        data: [{
+            name: paramKey,
+            value: jQuery('#' + paramField).val()
+        }], // get requested value from parent drop-down field
+        dataType: 'json',
         async: false,
         type: 'POST',
         success: function(result){
-               list = result[responseName];
-               // Create and show dependent select options
-               // Create and show dependent select options
-               if (list) {
-                       jQuery.each(list, function (key, value) {
-                           if (typeof value == 'string') {
-                               values = value.split(': ');
-                               if (values[1].indexOf(selected) >=0) {
-                                   optionList += "<option selected='selected' 
value = " + values[1] + " >" + values[0] + "</option>";
-                               } else {
-                                   optionList +=  "<option value = " + 
values[1] + " >" + values[0] + "</option>";
-                               }
-                           } else {
-                               if (value[keyName] == selected) {
-                                   optionList += "<option selected='selected' 
value = " + value[keyName] + " >" + value[descName] + "</option>";
-                               } else {
-                                   optionList += "<option value = " + 
value[keyName] + " >" + value[descName] + "</option>";
-                               }
-                           }
-                       })
-               };
+            list = result[responseName];
+            // Create and show dependent select options
+            // Create and show dependent select options
+            if (list) {
+                jQuery.each(list, function(key, value){
+                    if (typeof value == 'string') {
+                        values = value.split(': ');
+                        if (values[1].indexOf(selected) >= 0) {
+                            optionList += "<option selected='selected' value = 
" + values[1] + " >" + values[0] + "</option>";
+                        } else {
+                            optionList += "<option value = " + values[1] + " 
>" + values[0] + "</option>";
+                        }
+                    } else {
+                        if (value[keyName] == selected) {
+                            optionList += "<option selected='selected' value = 
" + value[keyName] + " >" + value[descName] + "</option>";
+                        } else {
+                            optionList += "<option value = " + value[keyName] 
+ " >" + value[descName] + "</option>";
+                        }
+                    }
+                })
+            };
             // Hide/show the dependent drop-down if hide=true else simply 
disable/enable
-            if ((!list) || (list.length < 1) || ((list.length == 1) && 
list[0].indexOf("_NA_") >=0)) {
-               jQuery(target).attr('disabled', 'disabled');
-               if (hide) {
-                       if (jQuery(target).is(':visible')) {
-                               jQuery(target).fadeOut(2500);
-                               if (hideTitle) 
jQuery(targetTitle).fadeOut(2500);
-                                       } else {
-                                               jQuery(target).fadeIn();
-                                               if (hideTitle) 
jQuery(targetTitle).fadeIn();                        
-                               jQuery(target).fadeOut(2500);
-                               if (hideTitle) 
jQuery(targetTitle).fadeOut(2500);
-                   }
-               }
+            if ((!list) || (list.length < 1) || ((list.length == 1) && 
list[0].indexOf("_NA_") >= 0)) {
+                jQuery(target).attr('disabled', 'disabled');
+                if (hide) {
+                    if (jQuery(target).is(':visible')) {
+                        jQuery(target).fadeOut(2500);
+                        if (hideTitle) jQuery(targetTitle).fadeOut(2500);
+                    } else {
+                        jQuery(target).fadeIn();
+                        if (hideTitle) jQuery(targetTitle).fadeIn();
+                        jQuery(target).fadeOut(2500);
+                        if (hideTitle) jQuery(targetTitle).fadeOut(2500);
+                    }
+                }
             } else {
-               jQuery(target).removeAttr('disabled');
-               if (hide) {
-                       if (!jQuery(target).is(':visible')) {
-                               jQuery(target).fadeIn();
-                               if (hideTitle) jQuery(targetTitle).fadeIn();
-                       }
+                jQuery(target).removeAttr('disabled');
+                if (hide) {
+                    if (!jQuery(target).is(':visible')) {
+                        jQuery(target).fadeIn();
+                        if (hideTitle) jQuery(targetTitle).fadeIn();
+                    }
                 }
             }
-       },
-           complete: function() {
+        },
+        complete: function(){
             // this is to handle a specific case where an input field is 
needed instead of a drop-down when no values are returned by the request (else 
if allow-empty="true" is used autoComplete handle the case)
             // this could be extended later to use an auto-completed drop-down 
or a lookup, instead of drop-down currently, when there are too much values to 
populate
             // Another option is to use an input field with Id instead of a 
drop-down, see setPriceRulesCondEventJs.ftl and top of 
getAssociatedPriceRulesConds service
             if (!list && inputField) {
                 jQuery(target).hide();
                 jQuery(input).show();
-            } else if (inputField) { 
+            } else if (inputField) {
                 jQuery(input).hide();
                 jQuery(target).show();
             }
             jQuery(target).html(optionList).click().change(); // .change() 
needed when using also asmselect on same field, .click() specifically for IE8
             if (callback != null) eval(callback);
-               }
+        }
     });
 }
-  
+
 //*** calls any service already mounted as an event
-function getServiceResult(request, params) {
+function getServiceResult(request, params){
     var data;
     jQuery.ajax({
-       type: 'POST',
-       url: request,
-       data: params,
-       async: false,
-       cache: false,
-       dataType: 'json',
-       success: function(result) {
-               data = result;
-       }
-    });    
+        type: 'POST',
+        url: request,
+        data: params,
+        async: false,
+        cache: false,
+        dataType: 'json',
+        success: function(result){
+            data = result;
+        }
+    });
     return data;
 }
 
 //*** checkUomConversion returns true if an UomConversion exists 
-function checkUomConversion(request, params) {
-    data = getServiceResult(request, params);     
+function checkUomConversion(request, params){
+    data = getServiceResult(request, params);
     return data['exist'];
 }
 


Reply via email to