Anaƫl Closson (openerp) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-581137-acl into lp:openerp-web/6.1.

Requested reviews:
  OpenERP R&D Team (openerp-dev)
Related bugs:
  Bug #944854 in OpenERP Web: "[m2o] context menu icon causes "member not 
found" error in IE8 first time it is clicked"
  https://bugs.launchpad.net/openerp-web/+bug/944854

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-581137-acl/+merge/146602

This bug only affects IE 8

Steps to reproduce : 

* Create a new a sale order
* Click on the customer m2o box to open the popup menu

On IE 8 : an exception is raised

On other browsers : the popup menu is opened

That's the same behavior for all m2o boxes.


-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-581137-acl/+merge/146602
Your team OpenERP R&D Team is requested to review the proposed merge of 
lp:~openerp-dev/openerp-web/6.1-opw-581137-acl into lp:openerp-web/6.1.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-12-04 11:02:12 +0000
+++ addons/web/static/src/js/view_form.js	2013-02-05 10:21:25 +0000
@@ -357,6 +357,7 @@
     do_onchange: function(widget, processed) {
         var self = this;
         return this.on_change_mutex.exec(function() {
+
             try {
                 var response = {}, can_process_onchange = $.Deferred();
                 processed = processed || [];
@@ -1469,7 +1470,6 @@
         return this.get_value();
     },
     update_dom: function(show_invalid) {
-        console.log('update_dom: show_invalid', show_invalid);
         this._super.apply(this, arguments);
         if (this.field.translate) {
             this.$element.find('.oe_field_translate').toggle(!!this.view.datarecord.id);
@@ -2117,7 +2117,7 @@
                         return true;
                     }, menuStyle: {width: "200px"}
                 });
-                $.async_when().then(function() {self.$menu_btn.trigger(e);});
+                $.when().then(function() {self.$menu_btn.trigger(e);});
             });
         });
         this.$menu_btn.bind({

_______________________________________________
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