Merge lp:~openerp-dev/openerp-web/6.1-opw-591727-msh into lp:openerp-web/6.1

[Mohammed Shekha(Open ERP)]

Hello,

Allow whole one2many reload on button action, there are lot of scenario where button inside one2many creates new record for one2many which requires whole one2many to reloaad so that new record get retrieved.

Demo:- To generate this issue there are lot of scenario, go to Warehouse -> Delivery Order -> Open any existing one and click on "Split in Production Lots" button of stock move, and create one lot, it creates new record in stock move but it doesn't show up untill we reload whole view or re-open that record again.

So to fix this issue add reload_on_button option same as 7.0.

I also set options in addons wherever I faced this issue and created a branch lp:~openerp-dev/openobject-addons/6.1-opw-591727-msh.

Thanks.

Unmerged revisions

2513. By Mohammed Shekha<email address hidden> on 2013-04-24

[IMP]Allow whole one2many reload on button action, there are lot of scenario where button inside one2many creates new record for one2many which requires whole one2many to reloaad so that new record get retrieved.

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2013-03-13 09:26:29 +0000
+++ addons/web/static/src/js/view_form.js	2013-04-24 06:25:33 +0000
@@ -2462,6 +2462,7 @@
         this.form_last_update = $.Deferred();
         this.init_form_last_update = this.form_last_update;
         this.disable_utility_classes = true;
+        this.options = this.get_definition_options();
     },
     start: function() {
         this._super.apply(this, arguments);
@@ -2839,11 +2840,21 @@
         });
     },
     do_button_action: function (name, id, callback) {
+        var self = this;
         var _super = _.bind(this._super, this);
+        //Reload whole record as it needed when record is created by button action, based on option reload_on_button
+        if (!self.o2m.options.reload_on_button) {
+        	this.o2m.view.do_save().then(function () {
+                _super(name, id, callback);
+            });
+        }else {
+        	this.o2m.view.do_save().then(function () {
+	        	_super(name, id, function() {
+	        		self.o2m.view.reload();
+	        	});
+        	});
+        }
 
-        this.o2m.view.do_save().then(function () {
-            _super(name, id, callback);
-        });
     },
     reload_record: function (record) {
         // Evict record.id from cache to ensure it will be reloaded correctly