From 57f7fbaa0e83dffc9c6ee3416d145e4c5a1d8702 Mon Sep 17 00:00:00 2001 From: Fabio Barbero Date: Fri, 20 May 2022 07:48:48 +0000 Subject: [PATCH] [IMP] knowledge: show article icon for member based on Purpose ======= Currently, the icon of the "based on" article in the share panel is only shown for the internal permission and not for the member. Ensure consistency by adding it to the member section as well. Task-2852908 Part-of: odoo/odoo#91430 --- addons/knowledge/controllers/main.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/addons/knowledge/controllers/main.py b/addons/knowledge/controllers/main.py index ab50889df9c..9d91eadc5e7 100644 --- a/addons/knowledge/controllers/main.py +++ b/addons/knowledge/controllers/main.py @@ -194,9 +194,11 @@ class KnowledgeController(http.Controller): ('email', 'partner_email'), ('partner_share', 'partner_share'), ], - 'knowledge.article': [('name', 'based_on_name')], + 'knowledge.article': [ + ('icon', 'based_on_icon'), + ('name', 'based_on_name'), + ], })[article.id] - for partner_id, member in members_permission.items(): # empty member added by '_get_article_member_permissions', don't show it in the panel if not member['member_id']: @@ -216,7 +218,7 @@ class KnowledgeController(http.Controller): 'partner_name': member['partner_name'], 'partner_email': member['partner_email'], 'permission': member['permission'], - 'based_on': member['based_on_name'], + 'based_on': f'{member["based_on_icon"] or "📄"} {member["based_on_name"]}' if member['based_on_name'] else False, 'based_on_id': member['based_on'], 'partner_share': member['partner_share'], 'is_unique_writer': member['permission'] == "write" and article.inherited_permission != "write" and not any(