[IMP] website snippet new option, gallery

website.snippet: The methods described in javascript will be called automatically from the data-method_name = value on li snippets options. The methods are called in the order relative to children of xml (eg: <li data-method1=value1> ... <li data-method2=value2> ... </ li>, the method1 is then called method2). Methods receive two arguments: type (over, click, reset) and value. data-snippet-id-option becomes data-snippet-id. Several xml options can use the same data-snippet-id.

add a new gallery snippet

convert website_sale to the new option system

1 files changed, 3 insertions, 7 deletions

diff --git a/addons/mass_mailing/static/src/js/website_mass_mailing.editor.js b/addons/mass_mailing/static/src/js/website_mass_mailing.editor.js
index 4caba1d29b5..0a460caaa81 100644
--- a/addons/mass_mailing/static/src/js/website_mass_mailing.editor.js
+++ b/addons/mass_mailing/static/src/js/website_mass_mailing.editor.js
@@ -5,8 +5,9 @@
     var _t = openerp._t;
 
     website.snippet.options.mailing_list_subscribe = website.snippet.Option.extend({
-        on_prompt: function () {
+        choose_mailing_list: function (type, value) {
             var self = this;
+            if (type !== "click") return;
             return website.prompt({
                 id: "editor_new_mailing_list_subscribe_button",
                 window_title: _t("Add a Newsletter Subscribe Button"),
@@ -22,15 +23,10 @@
         drop_and_build_snippet: function() {
             var self = this;
             this._super();
-            this.on_prompt().fail(function () {
+            this.choose_mailing_list('click').fail(function () {
                 self.editor.on_remove();
             });
         },
-        start : function () {
-            var self = this;
-            this.$el.find(".js_mailing_list").on("click", _.bind(this.on_prompt, this));
-            this._super();
-        },
         clean_for_save: function () {
             this.$target.addClass("hidden");
         },