[ADD] viin_brand_base, viin_brand_crm: add new module by 0xhieula · Pull Request #1 · Viindoo/branding

0xhieula · 2020-09-09T08:00:07Z

Mô tả vấn đề

Tạm gọi module cần update là A, module update brand của mình là brand_A.
Trong trường hợp sử dụng init hook để update icon module. Module brand của mình chỉ chạy 1 lần duy nhất khi được install. Vì vậy, trường hợp A update hoặc các module mà A depends được update (sự kiện này sẽ kéo theo việc module brand_A bị update một các thụ động do brand_A depends A) thì các thay đổi về icon sẽ bị đè (do A set icon bằng file *.xml).

Gỉải pháp và các xử lý

Không sử dụng init_hook nữa, thay vào đó sẽ gọi trực tiếp hàm modify từ module viin_brand_base từ file *.xml. Việc này đảm bảo module luôn chạy update icon cho module A ngay cả khi update chủ động hay bị động.

[BUG] Vấn đề phát sinh

Mudule CRM của odoo sẽ gặp lỗi ở lần uninstall thứ 2 (Xác nhận lỗi của Odoo, không liên quan tới module phía mình).

royle-vietnam · 2020-09-10T01:03:59Z

+    'name': "Viindoo Branding TVTMA HR Payroll (Community Edition)",
+    'name_vi_VN': "Ung dụng TVTMA HR Payroll (Community Edition) với thương hiệu Viindoo",


Repo này chỉ thay đổi icon cho bản CE của Odoo thôi, đối với các module của công ty thì thay đổi icon trực tiếp, anh thấy em đã làm rồi, nên em xoá module này đi nhé

leotranviindoo · 2020-09-10T07:13:37Z

+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+ 	<!--  viin_brand_ -->
+	<menuitem name="Invoicing"


Chỉ thay đổi lại thông tin web_icon, không ghi đè lại name

Suggested change

<menuitem name="Invoicing"

<menuitem

leotranviindoo · 2020-09-10T07:15:01Z

+    module_icon.write({'icon' : '/viin_brand_contacts/static/description/icon.png'})
+    module_icon._get_icon_image()
+
+def uninstall_brand_icon(cr, registry):


2 phương thức này có thể phối hợp với @manhtu5796 để làm 1 module viin_brand_base và nhét mấy phương thức chung này vào đấy để tái sử dụng. CC @thongthele

vâng ạ, bọn em sẽ xử lý phần viin_brand_base bên #3

0xhieula · 2020-09-12T03:05:45Z

@leotranviindoo anh ơi, em push thay đổi ở nhánh này.

leotranviindoo · 2020-09-12T03:28:39Z

+    _description = 'Viindoo Brand Base model'
+
+
+    def modify_module_icon(self, module_name, module_brand = '', icon_path = 'static/description/icon.png'):


Suggested change

def modify_module_icon(self, module_name, module_brand = '', icon_path = 'static/description/icon.png'):

def update_module_icon(self, origin_module_name, brand_module_name='', icon_path = 'static/description/icon.png'):

leotranviindoo · 2020-09-12T03:28:58Z

+
+    def modify_module_icon(self, module_name, module_brand = '', icon_path = 'static/description/icon.png'):
+        '''
+        Change module icon


Suggested change

Change module icon

This method used to set origin module's icon by brand module's icon and restore after brand module uninstalled

leotranviindoo · 2020-09-12T03:29:11Z

+        '''
+        Change module icon
+
+        @param module_name: module name want to restore icon


Suggested change

@param module_name: module name want to restore icon

@param origin_module_name: module name needed to replace icon

leotranviindoo · 2020-09-12T03:29:28Z

+        Change module icon
+
+        @param module_name: module name want to restore icon
+        @param module_brand: module name contains icon file and handle,


Suggested change

@param module_brand: module name contains icon file and handle,

@param brand_module_name: origin module name replaced by brand module name.

If it is not passed then it means the method is used to restore module icon to origin

leotranviindoo · 2020-09-12T03:29:38Z

+
+        @param module_name: module name want to restore icon
+        @param module_brand: module name contains icon file and handle,
+        @param icon_path: file icon default, default: static/description/icon.png


Suggested change

@param icon_path: file icon default, default: static/description/icon.png

@param icon_path: default: static/description/icon.png

leotranviindoo · 2020-09-12T03:33:46Z

@Cinereouss anh có comment cho method modify_module_icon chú làm tương tự với modify_menu_icon nhé. Về cơ bản thì anh đồng ý với ý tưởng và cách đóng gói phương thức của chú. Có điều sao module base không gọi function từ xml giống module CRM mà vẫn viết post_init_hook? Ngoài ra file update_icon.xml nên đưa vào thư mục data thay vì thư mục views.

CC @manhtu5796 @thongthele

0xhieula · 2020-09-12T03:37:33Z

@Cinereouss anh có comment cho method modify_module_icon chú làm tương tự với modify_menu_icon nhé. Về cơ bản thì anh đồng ý với ý tưởng và cách đóng gói phương thức của chú. Có điều sao module base không gọi function từ xml giống module CRM mà vẫn viết post_init_hook? Ngoài ra file update_icon.xml nên đưa vào thư mục data thay vì thư mục views.

CC @manhtu5796 @thongthele

Vâng anh, em rõ rồi.

leotranviindoo · 2020-09-12T04:11:38Z

+def uninstall_brand_icon(cr, registry):
+    env = api.Environment(cr, SUPERUSER_ID, {})
+    env['viin.brand.base'].modify_menu_icon('base', 'base.menu_administration', icon_path='static/description/settings.png')
+    env['viin.brand.base'].modify_menu_icon('base', 'base.menu_management', icon_path='static/description/modules.png')


Chưa đổi lại tên hàm ở đây à? Cả bên CRM nữa

leotranviindoo · 2020-09-12T04:17:05Z

+
+    def update_menu_icon(self, origin_module_name, menu_xml_id, icon_path = 'static/description/icon.png'):
+        '''
+        This method used to set origin menu module's icon by brand menu module's icon and restore after brand module uninstalled


Suggested change

This method used to set origin menu module's icon by brand menu module's icon and restore after brand module uninstalled

This method used to set origin menu app icon by brand menu icon and restore after brand module uninstalled

leotranviindoo · 2020-09-12T04:17:22Z

+        '''
+        This method used to set origin menu module's icon by brand menu module's icon and restore after brand module uninstalled
+
+        @param origin_module_name: module name needed to replace icon


Suggested change

@param origin_module_name: module name needed to replace icon

@param origin_module_name: module name needed to replace menu app icon

update_menu_icon

leotranviindoo · 2020-09-13T03:03:11Z

@Cinereouss @manhtu5796 anh merge PR này vào 13.0 rồi nhé. 2 chú rebase và làm tiếp nhé. Ngoài ra nhớ lưu ý đến technical convention, anh có sửa lại tiêu đề của PR cho đúng quy ước.

…om promo The eCommerce "Powered by" footer stated "The Viindoo#1 Open Source eCommerce Software" (vi_VN: "... mã nguồn mở số 1"), an unverifiable superiority ranking. Vietnam's advertising law forbids "Viindoo#1"/"best" style claims without supporting evidence and penalises them heavily, so remove the ranking wording from both the template source and the Vietnamese translation. - views/templates.xml: promo label "The Viindoo#1 Open Source eCommerce Software" -> "Open Source eCommerce Software"; add a comment at the label noting the VN advertising-law constraint. - i18n/vi_VN.po and .pot: realign msgid to the new source string; msgstr "... mã nguồn mở số 1" -> "... mã nguồn mở". - bump module version 0.1 -> 0.2 so existing databases pick up the updated view and translation on upgrade.

The eCommerce "Powered by" footer stated "The #1 Open Source eCommerce Software" (vi_VN: "... mã nguồn mở số 1"), an unverifiable superiority ranking. Vietnam's advertising law forbids "#1"/"best" style claims without supporting evidence and penalises them heavily, so remove the ranking wording from both the template source and the Vietnamese translation. - views/templates.xml: promo label "The #1 Open Source eCommerce Software" -> "Open Source eCommerce Software"; add a comment at the label noting the VN advertising-law constraint. - i18n/vi_VN.po and .pot: realign msgid to the new source string; msgstr "... mã nguồn mở số 1" -> "... mã nguồn mở". - bump module version 0.1 -> 0.2 so existing databases pick up the updated view and translation on upgrade.

[IMP] Viin Brand icon

dbc1264

royle-vietnam reviewed Sep 10, 2020

View reviewed changes

[IMP] Re-commit delete module duplicate

835cf55

leotranviindoo reviewed Sep 10, 2020

View reviewed changes

[IMP][FIX] Update module viin base

88abe41

leotranviindoo reviewed Sep 12, 2020

View reviewed changes

[IMP] Update Module Viin_brand base

65281df

leotranviindoo reviewed Sep 12, 2020

View reviewed changes

0xhieula and others added 3 commits September 12, 2020 11:12

[IMP] Re-update module Viin_brand_base

621bcd7

Update viin_brand_base/models/viin_brand_base.py

d3bfdf9

Update viin_brand_base/models/viin_brand_base.py

7484f1e

leotranviindoo reviewed Sep 12, 2020

View reviewed changes

0xhieula and others added 3 commits September 12, 2020 11:27

[IMP] Re-commit change comment in func update_module_icon and

cfd3381

update_menu_icon

[IMP] Add file hooks

c23b1bf

[ADD]viin_brand_base, viin_brand_crm: add new module

fc0c813

leotranviindoo changed the title ~~[IMP] Update Viin Brand icon~~ [ADD] viin_brand_base, viin_brand_crm: add new module Sep 13, 2020

leotranviindoo merged commit a21f116 into Viindoo:13.0 Sep 13, 2020

SonCrits mentioned this pull request Jul 24, 2026

[IMP][17.0] viin_brand_website_sale: drop unverifiable "#1" claim from promo #659

Merged

SonCrits mentioned this pull request Jul 24, 2026

[IMP][13.0] viin_brand_website_sale: drop unverifiable "#1" claim from promo #660

Open

		'name': "Viindoo Branding TVTMA HR Payroll (Community Edition)",
		'name_vi_VN': "Ung dụng TVTMA HR Payroll (Community Edition) với thương hiệu Viindoo",

		_description = 'Viindoo Brand Base model'


		def modify_module_icon(self, module_name, module_brand = '', icon_path = 'static/description/icon.png'):

	def modify_module_icon(self, module_name, module_brand = '', icon_path = 'static/description/icon.png'):
	def update_module_icon(self, origin_module_name, brand_module_name='', icon_path = 'static/description/icon.png'):

	Change module icon
	This method used to set origin module's icon by brand module's icon and restore after brand module uninstalled

	@param module_name: module name want to restore icon
	@param origin_module_name: module name needed to replace icon

	@param module_brand: module name contains icon file and handle,
	@param brand_module_name: origin module name replaced by brand module name.
	If it is not passed then it means the method is used to restore module icon to origin

	@param icon_path: file icon default, default: static/description/icon.png
	@param icon_path: default: static/description/icon.png

	This method used to set origin menu module's icon by brand menu module's icon and restore after brand module uninstalled
	This method used to set origin menu app icon by brand menu icon and restore after brand module uninstalled

Uh oh!

Conversation

0xhieula commented Sep 9, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Mô tả vấn đề

Gỉải pháp và các xử lý

[BUG] Vấn đề phát sinh

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

0xhieula commented Sep 12, 2020

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

leotranviindoo commented Sep 12, 2020

Uh oh!

0xhieula commented Sep 12, 2020

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leotranviindoo commented Sep 13, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

0xhieula commented Sep 9, 2020 •

edited

Loading