From b90c663702bd4abe867e01c225677dc1af9ab1f0 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Fri, 21 Aug 2015 09:17:27 +0200 Subject: [PATCH] [FIX] website: two sliders or galleries can have the same id (because the user can edit separate part like footer or using versioning) Backport of 394a0c8bb40ffa6eb080dd9e3bc9bce16db3d6f1. --- .../website/static/src/js/website.snippets.editor.js | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/addons/website/static/src/js/website.snippets.editor.js b/addons/website/static/src/js/website.snippets.editor.js index 6a5a334a0a0..d638146021e 100644 --- a/addons/website/static/src/js/website.snippets.editor.js +++ b/addons/website/static/src/js/website.snippets.editor.js @@ -944,23 +944,15 @@ }); website.snippet.options.slider = website.snippet.Option.extend({ - unique_id: function () { - var id = 0; - $(".carousel").each(function () { - var cid = 1 + parseInt($(this).attr("id").replace(/[^0123456789]/g, ''),10); - if (id < cid) id = cid; - }); - return "myCarousel" + id; - }, drop_and_build_snippet: function() { - this.id = this.unique_id(); + this.id = "myCarousel_" + new Date().getTime(); this.$target.attr("id", this.id); this.$target.find("[data-slide]").attr("data-cke-saved-href", "#" + this.id); this.$target.find("[data-target]").attr("data-target", "#" + this.id); this.rebind_event(); }, on_clone: function ($clone) { - var id = this.unique_id(); + var id = "myCarousel_" + new Date().getTime(); $clone.attr("id", id); $clone.find("[data-slide]").attr("href", "#" + id); $clone.find("[data-slide-to]").attr("data-target", "#" + id);