diff --git a/addons/website_forum/controllers/main.py b/addons/website_forum/controllers/main.py index 11426447e672f..8830cbc835119 100644 --- a/addons/website_forum/controllers/main.py +++ b/addons/website_forum/controllers/main.py @@ -18,6 +18,8 @@ from odoo.exceptions import UserError from odoo.http import request +from odoo.osv import expression + _logger = logging.getLogger(__name__) @@ -169,8 +171,14 @@ def forum_faq_karma(self, forum, **post): @http.route('/forum/get_tags', type='http', auth="public", methods=['GET'], website=True, sitemap=False) def tag_read(self, query='', limit=25, **post): + # TODO: In master always check the forum_id domain part and add forum_id + # as required method param, not in **post + forum_id = post.get('forum_id') + domain = [('name', '=ilike', (query or '') + "%")] + if forum_id: + domain = expression.AND([domain, [('forum_id', '=', int(forum_id))]]) data = request.env['forum.tag'].search_read( - domain=[('name', '=ilike', (query or '') + "%")], + domain=domain, fields=['id', 'name'], limit=int(limit), ) diff --git a/addons/website_forum/models/forum.py b/addons/website_forum/models/forum.py index 8739cb97a7a1a..45723132dabbe 100644 --- a/addons/website_forum/models/forum.py +++ b/addons/website_forum/models/forum.py @@ -226,7 +226,7 @@ def unlink(self): self._update_website_count() return super(Forum, self).unlink() - @api.model + @api.model # TODO: Remove me, this is not an `api.model` method def _tag_to_write_vals(self, tags=''): Tag = self.env['forum.tag'] post_tags = [] @@ -235,7 +235,7 @@ def _tag_to_write_vals(self, tags=''): 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 31cc27d77d3cc..d9554b1743f71 100644 --- a/addons/website_forum/static/src/js/website_forum.js +++ b/addons/website_forum/static/src/js/website_forum.js @@ -95,6 +95,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 b9d3bcdb94ab3..98f3781dd1a11 100644 --- a/addons/website_forum/tests/test_forum.py +++ b/addons/website_forum/tests/test_forum.py @@ -436,3 +436,25 @@ def test_forum_mode_discussions(self): 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.")