diff --git a/addons/website_mail_group/controllers/main.py b/addons/website_mail_group/controllers/main.py index e4387ba547b..dc0acce7644 100644 --- a/addons/website_mail_group/controllers/main.py +++ b/addons/website_mail_group/controllers/main.py @@ -45,6 +45,9 @@ class MailGroup(http.Controller): def subscription(self, group_id=0, action=False ,**post): cr, uid, context = request.cr, request.uid, request.context group_obj = request.registry.get('mail.group') + public_uid = request.registry['website'].get_public_user(cr , uid, context) + if uid == public_uid: + return ['is_public'] if action: group_obj.message_subscribe_users(cr, uid, [group_id], context=context) else: diff --git a/addons/website_mail_group/static/src/js/website_mail_group.js b/addons/website_mail_group/static/src/js/website_mail_group.js index 3b856d12684..651ed7fa6c3 100644 --- a/addons/website_mail_group/static/src/js/website_mail_group.js +++ b/addons/website_mail_group/static/src/js/website_mail_group.js @@ -4,12 +4,14 @@ $(document).ready(function () { var $link = $(ev.currentTarget); var href = $link.attr("href"); var group_id = href.match(/subscription\/([0-9]+)/)[1]; - var action = href.match(/action=(.*)/)[1] == 'follow' ? true : false; + var action = href.match(/action=(.*)/)[1] == 'subscribe' ? true : false; openerp.jsonRpc("/groups/subscription/", 'call', { 'group_id': parseInt(group_id), 'action' : action, }) .then(function (data) { + if(data[0]) + return window.location ='/web/login?redirect=/groups/'; if (action){ $('li#'+ group_id).toggleClass('hidden visible'); $('.unfollow_' + group_id).toggleClass('visible hidden'); diff --git a/addons/website_mail_group/views/website_mail_group.xml b/addons/website_mail_group/views/website_mail_group.xml index 6c6b23416ed..5000ca85a1d 100644 --- a/addons/website_mail_group/views/website_mail_group.xml +++ b/addons/website_mail_group/views/website_mail_group.xml @@ -1,6 +1,11 @@ +