Skip to content

[10.0][MIG][base_import_match] Migration and update - #871

Merged
pedrobaeza merged 7 commits into
OCA:10.0from
Tecnativa:10.0-base_import_match
Aug 16, 2017
Merged

[10.0][MIG][base_import_match] Migration and update#871
pedrobaeza merged 7 commits into
OCA:10.0from
Tecnativa:10.0-base_import_match

Conversation

@yajo

@yajo yajo commented Jun 22, 2017

Copy link
Copy Markdown
Member

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.

@Tecnativa

@yajo yajo added this to the 10.0 milestone Jun 22, 2017
@yajo yajo self-assigned this Jun 22, 2017
@pedrobaeza pedrobaeza mentioned this pull request Jun 22, 2017
63 tasks

@chienandalu chienandalu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality tested on runbot 👍

Comment thread base_import_match/README.rst Outdated
#. Go to *Settings > Technical > Database Structure > Import Match*.
#. *Create*.
#. Choose a *Model*.
#. Choose the *Fields* that conform an unique key in that model.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/an/a

Jairo Llopis and others added 7 commits August 16, 2017 19:08
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.
@pedrobaeza
pedrobaeza force-pushed the 10.0-base_import_match branch from bcec962 to ab0ceeb Compare August 16, 2017 17:08
@pedrobaeza
pedrobaeza merged commit 13032f3 into OCA:10.0 Aug 16, 2017
@pedrobaeza
pedrobaeza deleted the 10.0-base_import_match branch August 16, 2017 17:09
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
…it_bugfix

Pre-commit bugfixing for auditlog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants