-
Notifications
You must be signed in to change notification settings - Fork 25
[MIG][13.0]l10n_vn: map account_tag with account_tax_report_line #337
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 13.0
Are you sure you want to change the base?
Conversation
4d150a7 to
27113bc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Del đống này đi có vấn đề gì không nhỉ? Các báo cáo tài chính đang sử dụng các tag này ở 12, sau đó lên 13 mà không còn tag này nữa thì liệu đoạn map bên dưới của chú có care được không nhỉ @lathuat1997
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chỗ này là del trong file XML thôi anh ạ, khi migrate lên thì DB vẫn còn dữ liệu anh ạ.
PR này em đang định map, nhưng em đang cân nhắc lại có nên map hay xử lý theo hướng khác. Vì:
- Trên v13 sinh ra các
tagmới, và như nhà mình hiện tại cũng đang sử dụng cáctagmới này chứ không sử dụng cáctagnhư trên v12. - Khi migration, trên phần Thuế, ở các dòng "Phân bổ cho Hóa đơn" và các dòng "Phân bổ cho giấy báo có", hiện tại OCA đang đưa các
tagmà gắn trực tiếp với Thuế ở v12 để gắn vào các dòng này rồi cập nhật lại các dữ liệu liên quan(các trường dữ liệu có gắn đếntagtrong account.move.line) - Tuy nhiên trên v13, nếu không phải là migration, khi cài module
l10n_vnlên thì các dòng "Phân bổ cho Hóa đơn" và "Phân bổ cho giấy báo có" sẽ tự động được gắn cáctagmới được sinh ra khi cài module.
=> Nên em nghĩ có thể xử lại theo cách tác động đến chỗ migration của OCA nếu như các dữ liệu không bị ảnh hưởng bởi các tag cũ (nếu cập nhật được lại các dữ liệu cũ theo các tag mới).
cc @vietduc1989
27113bc to
c56600e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 dòng này có tác dụng map các thẻ tài khoản mà đang gắn trực tiếp với thuế sang các dòng trên thuế, sau đó map các thẻ với các phát sinh kế toán tương ứng.
Cách map này hiện tại không phù hợp với nghiệp vụ và các thẻ tài khoản đang có trên dữ liệu của khách hiện tại nên em comment lại đoạn này.
Người dùng có thể tự tạo ra các thẻ mới và gắn theo ý muốn, chưa rõ trong trường hợp đó thì cách map hiện tại của OCA có hợp lý không nên em chưa xóa 2 hàm này.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hàm này lấy từ file post-migration của module account, có tác dụng gắn các thẻ từ các dòng trên Thuế vào các phát sinh kế toán tương ứng.
c56600e to
39fb05d
Compare
Liên kết các account_tax_report_line với các account_tag tương ứng
Ticket: https://viindoo.com/web#id=8632&menu_id=89&cids=1&model=helpdesk.ticket&view_type=form