[18.0][MIG] account_financial_report_multi_company: Migration to 18.0#703
[18.0][MIG] account_financial_report_multi_company: Migration to 18.0#703
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the account_financial_report_multi_company module from a previous version to Odoo 18.0, enabling multi-company financial report generation functionality.
Key changes:
- Complete module structure creation with all necessary files for the migration
- Implementation of multi-company support for trial balance reports
- Addition of company selection wizard and report templates
Reviewed Changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
__manifest__.py |
Module manifest defining dependencies and data files |
__init__.py |
Module initialization importing wizards and reports |
wizards/trial_balance_wizard.py |
Wizard extending trial balance with multi-company field |
wizards/trial_balance_wizard_views.xml |
UI view modifications for company selection |
report/trial_balance.py |
Core report logic for multi-company trial balance |
report/abstract_report.py |
Base report functionality with company data |
report/trial_balance_xlsx.py |
Excel export with company column |
report/templates/trial_balance.xml |
Report template with company display |
| Documentation and translation files | Supporting files for module description and localization |
| relation="trial_balance_report_wizard_company_rel", | ||
| column1="trial_balance_report_wizard_id", | ||
| column2="company_id", | ||
| default=lambda self: self.env.company.ids, |
There was a problem hiding this comment.
The default value uses self.env.company.ids but self.env.company is a single record, so .ids returns a list with one ID. This should be self.env.company (without .ids) since the field expects a recordset for Many2many default values.
| default=lambda self: self.env.company.ids, | |
| default=lambda self: self.env.company, |
1eca225 to
6891ca6
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
6891ca6 to
874e201
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 18.0 #703 +/- ##
=======================================
Coverage ? 48.33%
=======================================
Files ? 7
Lines ? 60
Branches ? 5
=======================================
Hits ? 29
Misses ? 31
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.