Friday, May 24, 2024
2 changes
Code cleanup and technical improvements
A function that displays carryover lines has been moved from the core accounting module to the account reports module, where it actually belongs. This fixes a dependency issue where the function was calling other functions that only exist in account reports, making it unusable without that module installed.
Original PR description
`action_view_carryover_lines` calls account.report._get_date_bounds_info which is only defined in `account_reports`, and is itself only called from `account_reports`. It wouldn't have worked anyway for anyone who would have tried to call it without account_reports installed. So we take the liberty of moving it to account_reports in stable. See https://github.com/odoo/odoo/pull/155614#discussion_r1609626035 for context Forward-Port-Of: odoo/enterprise#63114 Forward-Port-Of: odoo/enterprise#63092
A function that displays carryover lines has been moved from the core accounting module to the accounting reports module, where it actually belongs. This change improves code organization since the function only works when the reports module is installed and is only used there.
Original PR description
`action_view_carryover_lines` calls account.report._get_date_bounds_info which is only defined in `account_reports`, and is itself only called from `account_reports`. It wouldn't have worked anyway for anyone who would have tried to call it without account_reports installed. So we take the liberty of moving it to account_reports in stable. See https://github.com/odoo/odoo/pull/155614#discussion_r1609626035 for context Forward-Port-Of: odoo/odoo#166586 Forward-Port-Of: odoo/odoo#166545