[19.0][MIG] base_business_document_import: Migration to 19.0#1315
Merged
OCA-git-bot merged 99 commits intoApr 20, 2026
Conversation
…voice dict, cleaner organisation) Code refactoring: move code in base_business_document_import, factorise code for tax matching (it was duplicated in UBL and ZUGFeRD) Now support PDF with embedded UBL XML file Enable unittests on account_invoice_import_ubl More absolute xpath in account_invoice_import_ubl instead of relative xpath WARNING: these are big changes, I may have broken a few details
Adapt module account_invoice_import_ubl to use the new base_ubl module Small fixes
Better generation of address block in UBL (make it coherent with the datamodel of Odoo) Add generation of several UBL blocks: language, delivery, payment terms, customer party, spplier party Add parsing of zip in UBL party (will be used in the future for delivery partner match)
Use country code and state code to match partners UBL: Add delivery terms and line item UBL XML block generation UBL: add parsing of delivery block
Add unitests in base_business_document_import Small code enhancements/simplifications
Code cleanup/minor changes
Rename key 'quantity' to 'qty' in all parsing dicts['lines'] Add common methods compare_lines() and post_create_or_update() Make sure price_unit is always untaxed in UBL XML files
Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)
[FIX] LINT Use try/except when importing external libs Remove self.ensure_one() that has nothing to do in an api.model method
Rename __openerp__.py to __manifest__.py and set installable to False
Add match for accounts and journals
Improve match for journals and accounts
Also port all the modules that generate the XML documents: account_invoice_ubl, account_invoice_zugferd, purchase_order_ubl and sale_order_ubl
Note: it requires this fix on odoo 10: odoo/odoo#15649
… module Fix spelling mistake and other remarks on README by Tarteo
…o v10.0 base_business_document_import: Add support for the creation of res.bank
Add support for insertion of title in error messages
Add method to create SO in sale.order.import accessible via JSON-RPC
force_company ctx key is deprecated and fallsback to env.company.
The given seller it's a partner: use it properly.
When wrapping the error was still using the seller.name relation which does not exists anymore.
…andling Add raise_exception argument in several methods Avoid double entries in chatter
Currently translated at 100.0% (40 of 40 strings) Translation: edi-18.0/edi-18.0-base_business_document_import Translate-URL: https://translation.odoo-community.org/projects/edi-18-0/edi-18-0-base_business_document_import/it/
yankinmax
force-pushed
the
19.0-mig-base_business_document_import
branch
from
098e9ab to
6460ac2
Compare
yankinmax
force-pushed
the
19.0-mig-base_business_document_import
branch
from
cf26afa to
7e4b4ac
Compare
3 tasks
grindtildeath
approved these changes
Apr 7, 2026
yankinmax
force-pushed
the
19.0-mig-base_business_document_import
branch
from
7e4b4ac to
59428b0
Compare
Author
|
Hello @MohamedOsman7 can you pls take a look? |
MohamedOsman7
approved these changes
Apr 8, 2026
MohamedOsman7
left a comment
MohamedOsman7
left a comment
There was a problem hiding this comment.
Some commits can be sqashed (guide: https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate)
Otherwise: Code Review LGTM
Author
I've squashed administrative and translation commits where possible. |
Author
|
Hello @OCA/edi-maintainers can you pls take a look and possibly merge? |
simahawk
requested changes
Apr 14, 2026
simahawk
approved these changes
Apr 20, 2026
Contributor
|
/ocabot merge nobump |
Contributor
|
What a great day to merge this nice PR. Let's do it! |
Contributor
|
Congratulations, your PR was merged at 2f896c3. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends on: