Migration to version 18.0

[OCA-git-bot]

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0

Modules to migrate

• account_einvoice_generate
• account_invoice_export - By @bizzappdev - [18.0] [MIG] account_invoice_export: Migration to 18.0 #1191
• account_invoice_export_job - By @TDu - [18.0][MIG] account_invoice_export_job #1204
• account_invoice_export_server_env - By @bizzappdev - [18.0] [MIG] account_invoice_export_server_env: Migration to 18.0 #1192
• account_invoice_facturx
• base_business_document_import - By @simahawk - [18.0][MIG] base_business_document_import #1167
• base_ebill_payment_contract - By @simahawk - [18.0][mig] base_ebill_payment_contract #1202
• base_edi - By @JordiMForgeFlow - [18.0][MIG] base_edi #1055
• base_facturx
• base_import_pdf_by_template
• base_ubl - By @simahawk - [18.0][MIG] base_ubl + split parse and generate + full test cov #1168
• partner_identification_import - By @simahawk - [18.0][MIG] partner_identification_import #1176
• pdf_helper - By @simahawk - [18.0][MIG] pdf_helper: Migration to 18.0  #1170 - (moved to [18.0][MIG] pdf_helper renamed to pdf_xml_attachment reporting-engine#1031)
• sale_order_customer_free_ref - By @rlimaeco - [18.0] [MIG] sale_order_customer_free_ref #1126
• sale_order_import - By @simahawk - [18.0][mig] sale_order_import #1171
• sale_order_import_packaging - By @henrybackman - [18.0][MIG] sale_order_import_packaging (renamed from sale_order_packaging_import) #1196
• sale_order_import_ubl - By @simahawk - [18.0][MIG] sale_order_import_ubl #1172
• sale_order_import_ubl_customer_free_ref - By @henrybackman - [18.0][MIG] sale_order_import_ubl_customer_free_ref: Migration to 18.0 #1189
• sale_order_import_ubl_line_customer_ref - By @simahawk - [18.0][MIG] sale_order_import_ubl_line_customer_ref #1175
• test_base_import_pdf_by_template

INFO: The Voxel modules have been moved to OCA/edi-voxel.

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

[alexis-via]

I'm facing a problem while migrating account_invoice_facturx from v17 to v18: in v18, a new field is_invoice_report was added to ir.actions.report and the method _is_invoice_report(report_ref) was updated to use that new field:
odoo/odoo@bb60952

The problem is that, by default, is_invoice_report is True on report "Invoices" (XMLID account.account_invoices) but it is False in report "Invoices without Payment" (XMLID account.account_invoices_without_payment)

In another commit, the report "Invoices" which has is_invoice_report=True was renamed to PDF and removed from the print menu, cf:
odoo/odoo@73c9fb4#diff-3b18ee604f9c7367c3d380bcc92f09c390e896f511bbe563f19116297a8ab653

So, if we continue to use the method _is_invoice_report(report_ref) to choose if we should attach the file factur-x.xml or not, the PDF retreived from the print menu will not have the XML attachment.

Here are the options we have:

1. in the post-install script of account_einvoice_generate (for example), we set is_invoice_report to True on report "PDF without Payment" (XMLID account.account_invoices_without_payment). Advantage: the user can always switch back to native behavior by changing the value of that field)
2. we stop using _is_invoice_report(report_ref) and switch back to an OCA method like we used to do in the past
3. we think there is a really good reason to consider that the invoice retreived from the print menu should not be a factur-x invoice (I don't see any... but you may have a different opinion)

For the moment, I plan to implement option 1. Please give your opinion on this before we take the final decision.