Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-575643-cpa into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-575643-cpa/+merge/110811

Hello,

In any shared view exported with the share module, the onchange doesn't work.
For example, if you export the sale order view, changing the partner does not 
change the addresses.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-575643-cpa/+merge/110811
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-575643-cpa.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-06-14 08:40:09 +0000
+++ addons/web/static/src/js/view_form.js	2012-06-18 13:04:39 +0000
@@ -325,7 +325,7 @@
         var self = this;
         return this.on_change_mutex.exec(function() {
             try {
-                var response = {}, can_process_onchange = $.Deferred();
+                var response = {}, can_process_onchange = $.Deferred(), can_process_get_default = $.Deferred();
                 processed = processed || [];
                 processed.push(widget.name);
                 var on_change = widget.node.attrs.on_change;
@@ -365,7 +365,7 @@
                     var condition = fieldname + '=' + value;
 
                     if (value) {
-                        can_process_onchange = self.rpc({
+                        can_process_get_default = self.rpc({
                             url: '/web/dataset/call',
                             async: false
                         }, {
@@ -385,11 +385,12 @@
                         });
                     }
                 }
-                if (can_process_onchange.isRejected()) {
-                    return can_process_onchange;
+                if (can_process_get_default.isRejected()) {
+                    return can_process_get_default;
                 }
-
-                return self.on_processed_onchange(response, processed);
+                $.when(can_process_onchange, can_process_get_default).then(function(){
+                    return self.on_processed_onchange(response, processed);
+                })
             } catch(e) {
                 console.error(e);
                 return $.Deferred().reject();

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to