From 5ab21e13a36902c13905ff92bb1cc0d728dbd467 Mon Sep 17 00:00:00 2001
From: ggh-openerp <ggh@tinyerp.com>
Date: Tue, 9 Oct 2012 11:46:18 +0530
Subject: [IMP] Remove add_last in on_unlink method

bzr revid: ggh@tinyerp.com-20121009061618-ohsjlzf14jpqws2q
---
 addons/web/static/src/js/data.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'addons/web/static/src/js/data.js')

diff --git a/addons/web/static/src/js/data.js b/addons/web/static/src/js/data.js
index 220d42737da..79f909c0908 100644
--- a/addons/web/static/src/js/data.js
+++ b/addons/web/static/src/js/data.js
@@ -690,9 +690,11 @@ instance.web.DataSet =  instance.web.CallbackEnabled.extend({
 
 instance.web.DataSetStatic =  instance.web.DataSet.extend({
     init: function(parent, model, context, ids) {
+        var self = this;
         this._super(parent, model, context);
         // all local records
         this.ids = ids || [];
+        self.on("on_rec_unlink", self, self.on_unlink);
     },
     read_slice: function (fields, options) {
         options = options || {};
@@ -711,7 +713,7 @@ instance.web.DataSetStatic =  instance.web.DataSet.extend({
         }
     },
     unlink: function(ids) {
-        this.on_unlink(ids);
+        this.trigger('on_rec_unlink', ids);
         return $.Deferred().resolve({result: true});
     },
     on_unlink: function(ids) {
-- 
cgit v1.2.3


From 6b509ab8eae0808a272a8d5f145854e7e60b2c5c Mon Sep 17 00:00:00 2001
From: "vta vta@openerp.com" <>
Date: Wed, 10 Oct 2012 16:36:07 +0200
Subject: [FIX] on_unlink.

bzr revid: vta@openerp.com-20121010143607-nv3lhmy8r27m1ls9
---
 addons/web/static/src/js/data.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

(limited to 'addons/web/static/src/js/data.js')

diff --git a/addons/web/static/src/js/data.js b/addons/web/static/src/js/data.js
index 79f909c0908..ce85e0ac43f 100644
--- a/addons/web/static/src/js/data.js
+++ b/addons/web/static/src/js/data.js
@@ -575,6 +575,7 @@ instance.web.DataSet =  instance.web.CallbackEnabled.extend({
      * @param {Number|String} ids identifier of the record to delete
      */
     unlink: function(ids) {
+        this.trigger('unlink', ids);
         return this._model.call('unlink', [ids], {context: this._model.context()});
     },
     /**
@@ -694,7 +695,6 @@ instance.web.DataSetStatic =  instance.web.DataSet.extend({
         this._super(parent, model, context);
         // all local records
         this.ids = ids || [];
-        self.on("on_rec_unlink", self, self.on_unlink);
     },
     read_slice: function (fields, options) {
         options = options || {};
@@ -713,12 +713,10 @@ instance.web.DataSetStatic =  instance.web.DataSet.extend({
         }
     },
     unlink: function(ids) {
-        this.trigger('on_rec_unlink', ids);
+        this.set_ids(_.without.apply(null, [this.ids].concat(ids)));
+        this.trigger('unlink', ids);
         return $.Deferred().resolve({result: true});
     },
-    on_unlink: function(ids) {
-        this.set_ids(_.without.apply(null, [this.ids].concat(ids)));
-    }
 });
 
 instance.web.DataSetSearch =  instance.web.DataSet.extend({
-- 
cgit v1.2.3