diff --git a/addons/website_forum/controllers/main.py b/addons/website_forum/controllers/main.py index 2a614b965ed..d8f3339cd9c 100644 --- a/addons/website_forum/controllers/main.py +++ b/addons/website_forum/controllers/main.py @@ -20,6 +20,7 @@ from odoo.addons.portal.controllers.portal import _build_url_w_params from odoo.exceptions import UserError from odoo.http import request + _logger = logging.getLogger(__name__) @@ -179,9 +180,9 @@ class WebsiteForum(WebsiteProfile): return request.render("website_forum.faq_karma", values) @http.route('/forum/get_tags', type='http', auth="public", methods=['GET'], website=True, sitemap=False) - def tag_read(self, query='', limit=25, **post): + def tag_read(self, forum_id, query='', limit=25, **post): data = request.env['forum.tag'].search_read( - domain=[('name', '=ilike', (query or '') + "%")], + domain=[('forum_id', '=', int(forum_id)), ('name', '=ilike', (query or '') + "%")], fields=['id', 'name'], limit=int(limit), ) diff --git a/addons/website_forum/models/forum.py b/addons/website_forum/models/forum.py index 2ee1481f971..3068628130a 100644 --- a/addons/website_forum/models/forum.py +++ b/addons/website_forum/models/forum.py @@ -222,7 +222,6 @@ class Forum(models.Model): self._update_website_count() return super(Forum, self).unlink() - @api.model def _tag_to_write_vals(self, tags=''): Tag = self.env['forum.tag'] post_tags = [] @@ -231,7 +230,7 @@ class Forum(models.Model): for tag in (tag for tag in tags.split(',') if tag): if tag.startswith('_'): # it's a new tag # check that not already created meanwhile or maybe excluded by the limit on the search - tag_ids = Tag.search([('name', '=', tag[1:])]) + tag_ids = Tag.search([('name', '=', tag[1:]), ('forum_id', '=', self.id)]) if tag_ids: existing_keep.append(int(tag_ids[0])) else: diff --git a/addons/website_forum/static/src/js/website_forum.js b/addons/website_forum/static/src/js/website_forum.js index d607ac8fc92..a241d586c2c 100644 --- a/addons/website_forum/static/src/js/website_forum.js +++ b/addons/website_forum/static/src/js/website_forum.js @@ -94,6 +94,7 @@ publicWidget.registry.websiteForum = publicWidget.Widget.extend({ return { query: term, limit: 50, + forum_id: $('#wrapwrap').data('forum_id'), }; }, results: function (data) { diff --git a/addons/website_forum/tests/test_forum.py b/addons/website_forum/tests/test_forum.py index b9d3bcdb94a..98f3781dd1a 100644 --- a/addons/website_forum/tests/test_forum.py +++ b/addons/website_forum/tests/test_forum.py @@ -436,3 +436,25 @@ class TestForum(TestForumCommon): not discussions_post.uid_has_answered or discussions_post.forum_id.mode == 'discussions', True) self.assertEqual( discussions_post.uid_has_answered and discussions_post.forum_id.mode == 'questions', False) + + def test_tag_creation_multi_forum(self): + Post = self.env['forum.post'] + forum_1 = self.forum + forum_2 = forum_1.copy({ + 'name': 'Questions Forum' + }) + self.user_portal.karma = KARMA['tag_create'] + Post.with_user(self.user_portal).create({ + 'name': "Post Forum 1", + 'forum_id': forum_1.id, + 'tag_ids': forum_1._tag_to_write_vals('_Food'), + }) + Post.with_user(self.user_portal).create({ + 'name': "Post Forum 2", + 'forum_id': forum_2.id, + 'tag_ids': forum_2._tag_to_write_vals('_Food'), + }) + food_tags = self.env['forum.tag'].search([('name', '=', 'Food')]) + self.assertEqual(len(food_tags), 2, "One Food tag should have been created in each forum.") + self.assertIn(forum_1, food_tags.forum_id, "One Food tag should have been created for forum 1.") + self.assertIn(forum_2, food_tags.forum_id, "One Food tag should have been created for forum 2.")