[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
This commit is contained in:
@@ -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(
|
||||
|
||||
Reference in New Issue
Block a user