You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2010/12/29 18:15:01 UTC

svn commit: r1053677 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

Author: jaz
Date: Wed Dec 29 17:15:01 2010
New Revision: 1053677

URL: http://svn.apache.org/viewvc?rev=1053677&view=rev
Log:
changed placement of firing the changed event to the write_value function; since all setters call this function

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

Modified: ofbiz/trunk/framework/images/webapp/images/fieldlookup.js
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/fieldlookup.js?rev=1053677&r1=1053676&r2=1053677&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/fieldlookup.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/fieldlookup.js Wed Dec 29 17:15:01 2010
@@ -659,8 +659,7 @@ function set_value (value) {
     var target = obj_caller.target;
 
     write_value(value, target);
-    field = jQuery("#" + target.attr('id'));
-    field.trigger("lookup:changed");
+    field = jQuery("#" + target.attr('id'));    
     
     closeLookup();
 }
@@ -686,7 +685,8 @@ function write_value (value, target) {
     if (target && target != null) {
         setSourceColor(target);
         target.val(value);
-    }
+        target.trigger("lookup:changed");
+    }    
 }
 
 function set_multivalues(value) {