Skip to content

[fork-CI] [19.0][MIG] base_import_match: Migration to 19.0#3

Draft
dnplkndll wants to merge 31 commits into
19.0from
19.0-mig-base_import_match
Draft

[fork-CI] [19.0][MIG] base_import_match: Migration to 19.0#3
dnplkndll wants to merge 31 commits into
19.0from
19.0-mig-base_import_match

Conversation

@dnplkndll
Copy link
Copy Markdown

Migration of base_import_match from 18.0 to 19.0.

Fork-CI validation PR. Will be closed once upstream PR against OCA/server-backend is
opened from the same branch.

Jairo Llopis and others added 30 commits May 29, 2026 23:00
By default, when importing data (like CSV import) with the ``base_import``
module, Odoo follows this rule:

1. If you import the XMLID of a record, make an **update**.
2. If you do not, **create** a new record.

This module allows you to set additional rules to match if a given import is an
update or a new record.

This is useful when you need to sync heterogeneous databases, and the field you
use to match records in those databases with Odoo's is not the XMLID but the
name, VAT, email, etc.

After installing this module, the import logic will be changed to:

1. If you import the XMLID of a record, make an **update**.
2. If you do not:
   1. If there are import match rules for the model you are importing:
       1. Discard the rules that require fields you are not importing.
       2. Traverse the remaining rules one by one in order to find a match in
          the database.
          1. If one match is found:
             1. Stop traversing the rest of valid rules.
             2. **Update** that record.
          2. If zero or multiple matches are found:
             1. Continue with the next rule.
          3. If all rules are exhausted and no single match is found:
             1. **Create** a new record.
   2. If there are no match rules for your model:
      1. **Create** a new record.
This patch allows to import res.partner records by VAT when is_company==True by default.
* Fix wrong README format.

* [FIX][base_import_match] Avoid UnicodeEncodeError.

When the model or field you chose was translated and had some non-ascii
character, you got an error like this: `UnicodeEncodeError: 'ascii' codec can't
encode character u'\xed' in position 15: ordinal not in range(128)`.

Now, using unicode strings, that won't happen again.

* Do not require a hidden field.

* Further unicode protection, add ondelete clause.
Includes:

- Normal migration steps.
- Usage of brand new `_inherit = "base"` in Odoo 10, which implies
  removing a lot of monkey-patching code.
- Log a warning when multiple matches are found.
[UPD] README.rst

[UPD] Update base_import_match.pot
Currently translated at 100.0% (27 of 27 strings)

Translation: server-backend-12.0/server-backend-12.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-12-0/server-backend-12-0-base_import_match/pt_BR/
[UPD] Update base_import_match.pot
Currently translated at 100.0% (27 of 27 strings)

Translation: server-backend-14.0/server-backend-14.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_import_match/vi_VN/
Cached methods should never return recordsets, which are tied to
specific context, env, cursor, uid. Instead, they should return IDs
which, later, can be browsed in the current context.

With this change _usable_rules() cached method is now returing ids instead
of a recordset, and also the _match_find method is properly browsing the
results ids of the cached method.
Currently translated at 51.8% (14 of 27 strings)

Translation: server-backend-16.0/server-backend-16.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_import_match/it/
Currently translated at 51.8% (14 of 27 strings)

Translation: server-backend-16.0/server-backend-16.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_import_match/it/
Currently translated at 100.0% (27 of 27 strings)

Translation: server-backend-16.0/server-backend-16.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_import_match/es/
Currently translated at 51.8% (14 of 27 strings)

Translation: server-backend-16.0/server-backend-16.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_import_match/it/
Currently translated at 100.0% (27 of 27 strings)

Translation: server-backend-16.0/server-backend-16.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_import_match/it/
Includes:
- POT file update
- Post-merge bot changes
Currently translated at 85.1% (23 of 27 strings)

Translation: server-backend-18.0/server-backend-18.0-base_import_match
Translate-URL: https://translation.odoo-community.org/projects/server-backend-18-0/server-backend-18-0-base_import_match/sl/
Signed-off-by: Don Kendall <dkendall@ledoweb.com>
- Rebuild the test self-contained for no-demo CI: the test matched against
  demo partners (by xmlid / db-id / vat / email / name) AND relied on the
  email/name/parent match rules that live in demo/. setUpClass now creates the
  partners + those match rules and the imports are inlined (static CSVs dropped).
- Search view: dropped expand + string="Group By" from <group> (M15).

Signed-off-by: Don Kendall <dkendall@ledoweb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.