diff --git a/addons/website/controllers/main.py b/addons/website/controllers/main.py index 88c8b85f321..427381ac781 100644 --- a/addons/website/controllers/main.py +++ b/addons/website/controllers/main.py @@ -258,17 +258,16 @@ class Website(openerp.addons.web.controllers.main.Home): pass return request.make_response(image_data, headers) - @website.route(['/website/publish/'], type='http', auth="public") - def publish(self, **post): - _id = int(post['id']) - _object = request.registry[post['object']] + @website.route(['/website/publish'], type='json', auth="public") + def publish(self, id, object): + _id = int(id) + _object = request.registry[object] obj = _object.browse(request.cr, request.uid, _id) _object.write(request.cr, request.uid, [_id], {'website_published': not obj.website_published}, context=request.context) obj = _object.browse(request.cr, request.uid, _id) - return obj.website_published and "1" or "0" @website.route(['/website/kanban/'], type='http', auth="public") diff --git a/addons/website/static/src/css/website.css b/addons/website/static/src/css/website.css index 5a064a40a9e..d3d275aac71 100644 --- a/addons/website/static/src/css/website.css +++ b/addons/website/static/src/css/website.css @@ -352,27 +352,12 @@ ul.nav-stacked > li > a { } /* ---- PUBLISH ---- */ -a[data-publish] { - text-decoration: none !important; - z-index: 2; -} -a[data-publish] .label { - padding: 5px 8px; -} -a[data-publish] .css_unpublish, a[data-publish] .css_publish, a[data-publish] .css_unpublished, a[data-publish] .css_published { +.dropdown .css_unpublish, .dropdown .css_publish { display: none; } -a[data-publish][data-publish='off'] .css_unpublished, a[data-publish][data-publish='off']:hover .css_publish { - display: inline; -} -a[data-publish][data-publish='off']:hover .css_unpublished { - display: none; -} -a[data-publish][data-publish='on'] .css_published, a[data-publish][data-publish='on']:hover .css_unpublish { - display: inline; -} -a[data-publish][data-publish='on']:hover .css_published { - display: none; + +.dropdown.css_publish .css_unpublish, .dropdown.css_unpublish .css_publish { + display: block; } .unpublish { diff --git a/addons/website/static/src/css/website.sass b/addons/website/static/src/css/website.sass index cb0e93d9170..5e3468d9f33 100644 --- a/addons/website/static/src/css/website.sass +++ b/addons/website/static/src/css/website.sass @@ -267,23 +267,11 @@ ul.nav-stacked > li > a text-transform: uppercase /* ---- PUBLISH ---- */ -a[data-publish] - text-decoration: none !important - z-index: 2 - .label - padding: 5px 8px - .css_unpublish, .css_publish, .css_unpublished, .css_published - display: none - &[data-publish='off'] - .css_unpublished, &:hover .css_publish - display: inline - &:hover .css_unpublished - display: none - &[data-publish='on'] - .css_published, &:hover .css_unpublish - display: inline - &:hover .css_published - display: none + +.dropdown .css_unpublish, .dropdown .css_publish + display: none +.dropdown.css_publish .css_unpublish, .dropdown.css_unpublish .css_publish + display: block .unpublish opacity: 0.5 diff --git a/addons/website/static/src/js/website.js b/addons/website/static/src/js/website.js index b26a0a7e76c..c652ed1c459 100644 --- a/addons/website/static/src/js/website.js +++ b/addons/website/static/src/js/website.js @@ -116,19 +116,24 @@ dom_ready.then(function () { /* ----- PUBLISHING STUFF ---- */ - $('[data-publish]:has(.js_publish)').each(function () { - $(this).attr("data-publish", $(".js_publish li.active", this).size() ? "on" : 'off'); + $('[data-publish]:has(.js_publish_management)').each(function () { + $(this).attr("data-publish", $(".js_publish_management .btn-success", this).size() ? "on" : 'off'); }); - $(document).on('click', '.js_publish a.js_publish_btn', function (e) { - var $li = $(this).parent("li"); - var $data = $li.parents(".js_publish:first"); - var publish = $li.hasClass("active"); - $li.toggleClass("active"); - $.post('/website/publish', {'id': $data.data('id'), 'object': $data.data('object')}, function (result) { - $li.toggleClass("active", !!+result); - $li.parents("[data-publish]").attr("data-publish", +result ? 'on' : 'off'); - }); + $(document).on('click', '.js_publish_management .js_publish_btn', function (e) { + var $data = $(this).parents(".js_publish_management:first"); + var $btn = $data.find('.btn:first'); + var publish = $btn.hasClass("btn-success"); + + $data.toggleClass("css_unpublish css_publish"); + $btn.removeClass("btn-default btn-success"); + + openerp.jsonRpc('/website/publish', 'call', {'id': +$data.data('id'), 'object': $data.data('object')}) + .then(function (result) { + $btn.toggleClass("btn-default", !+result).toggleClass("btn-success", !!+result); + $data.toggleClass("css_unpublish", !+result).toggleClass("css_publish", !!+result); + $data.parents("[data-publish]").attr("data-publish", +result ? 'on' : 'off'); + }); }); /* ----- KANBAN WEBSITE ---- */ diff --git a/addons/website/views/views.xml b/addons/website/views/views.xml index c57b69e5b58..f550234949e 100644 --- a/addons/website/views/views.xml +++ b/addons/website/views/views.xml @@ -338,13 +338,17 @@ -