From 12f6a051ff547011841a79fa2e92543dcfb3f34c Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Wed, 16 Oct 2013 09:36:30 +0200 Subject: [PATCH] [FIX] website snippet: remove snippet and clean_for_save bzr revid: chm@openerp.com-20131016073630-y153xmzapqgm9dsm --- addons/website/static/src/js/website.snippets.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/addons/website/static/src/js/website.snippets.js b/addons/website/static/src/js/website.snippets.js index 9fd626cb8af..2181b8a650b 100644 --- a/addons/website/static/src/js/website.snippets.js +++ b/addons/website/static/src/js/website.snippets.js @@ -210,7 +210,10 @@ }, clean_for_save: function () { for (var k in this.snippets) { - $(this.snippets[k]).data("snippet-editor").clean_for_save(); + var editor = $(this.snippets[k]).data("snippet-editor"); + if (editor) { + editor.clean_for_save(); + } } }, make_active: function ($snipped_id) { @@ -728,8 +731,9 @@ start: function () { var self = this; this.$overlay.on('click', '.oe_snippet_remove', function () { - self.$target.detach(); self.onBlur(); + var index = _.indexOf(self.parent.snippets, self.$target.get(0)); + delete self.parent.snippets[index]; self.$target.remove(); return false; });