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

[Mohammed Shekha(Open ERP)]

Hello,

Fixed the issue of button click which is inside one2many list, here when workflow buttons are clicked record is not reloaded from the database, hence buttons are not shown properly based on workflow state.

Demo -: Manufacturing -> Manufacturing Order -> Edit any record which is in "ready to produce" state, activate workorder tab, click on start, pause or finish button record is not reloaded.

Reason :- The is faced only in one2many list view because when record is reloaded it will read the record from the cache, hence it will load the same record, instead the record should be read from the database, hence removed the cache record object so that the id in read_ids can set into to_get list and goes to read from the database.

Thanks.

[lp:~openerp-dev/openerp-web/6.1-opw-577195ReloadRecordIssue-msh]

2427. By Mohammed Shekha(Open ERP) on 2012-08-09

[FIX]Refixed the issue of record reload in one2many when button is clicked.

[Mohammed Shekha(Open ERP) (msh-openerp)]

Hello,

The same has been fixed and superseded by http://bazaar.launchpad.net/~openerp/openerp-web/6.1/revision/2451.

Revision No.: 2451.

Hence I am rejecting this one.

Thanks.

review: Disapprove

Unmerged revisions

2427. By Mohammed Shekha(Open ERP) on 2012-08-09

[FIX]Refixed the issue of record reload in one2many when button is clicked.

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-07-20 12:48:22 +0000
+++ addons/web/static/src/js/view_form.js	2012-08-09 05:43:18 +0000
@@ -2805,10 +2805,14 @@
         });
     },
     do_button_action: function (name, id, callback) {
+        var self = this;
         var _super = _.bind(this._super, this);
 
         this.o2m.view.do_save().then(function () {
             _super(name, id, callback);
+            var cached = _.detect(self.dataset.cache, function(x) {return x.id === id; });
+            var idx = _.indexOf(self.dataset.cache, cached);
+            self.dataset.cache.splice(idx, 1)
         });
     }
 });