Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[18.0][MIG] connector: Migration to 18.0 #494

Merged
merged 809 commits into from
Oct 17, 2024

Conversation

maxime-c2c and others added 30 commits October 8, 2024 14:27
Commit c97ebbd was a frontport from 7.0.
However, openerp.sql_db.dsn was changed in 8.0 in that it now returns a
tuple. Extract the db_name from the returned tuple.
The first item contains the name of the DB such as 'odoo_db' and the
second item contains 'user=gbaconnier dbname=odoo_db' which is what
expects psycopg2.connect.
This is a temporary fix. In version 4.0,
dbfilter will be completely ignored by connector.
When PG is localized, error messages are not ascii and jobs are not postponed. Instead they are failed with a 'unicode decode error'.

Note: even if PG ```lc_messages``` is set to en_us.utf8 on a localized system, the error message prefixes (such as ```DETAILS:```) are still localized.
This is a safeguard to prevent someone to write arbitrary code in jobs.
Builtin types and datetime/timedelta are allowed in job arguments, and a
new function 'whitelist_unpickle_global' allows to register new objects
if needed.
* In openerp.addons.connector.session.ConnectorSession:
  * Remove method 'search'
  * Remove method 'write'
  * Remove method 'browse'
  * Remove method 'read'
  * Remove method 'create'
  * Remove method 'unlink'
* In openerp.addons.connector.connector.ConnectorUnit:
  * Remove method 'get_connector_unit_for_model'
  * Remove method 'get_binder_for_model'
  * Remove property 'environment'
* In openerp.addons.connector.connector.ConnectorEnvironment:
  * Remove method 'set_lang'
* In openerp.addons.connector.connector.connector.MetaConnectorUnit
  * Remove property 'model_name'
* Remove class openerp.addons.connector.connector.Environment
* Remove method openerp.addons.connector.connector.intall_in_connector
* In openerp.addons.connector.unit.synchronizer:
  * Remove class 'ExportSynchronizer'
  * Remove class 'ImportSynchronizer'
  * Remove class 'DeleteSynchronizer'
* Remove module openerp.addons.connector.deprecate.
*
OCA-git-bot and others added 13 commits October 8, 2024 15:13
Currently translated at 43.7% (7 of 16 strings)

Translation: connector-16.0/connector-16.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-16-0/connector-16-0-test_connector/it/
Currently translated at 43.7% (7 of 16 strings)

Translation: connector-16.0/connector-16.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-16-0/connector-16-0-test_connector/sl/
Currently translated at 100.0% (16 of 16 strings)

Translation: connector-16.0/connector-16.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-16-0/connector-16-0-test_connector/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: connector-17.0/connector-17.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-17-0/connector-17-0-test_connector/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: connector-17.0/connector-17.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-17-0/connector-17-0-test_connector/
Currently translated at 100.0% (15 of 15 strings)

Translation: connector-17.0/connector-17.0-test_connector
Translate-URL: https://translation.odoo-community.org/projects/connector-17-0/connector-17-0-test_connector/it/
@simahawk
Copy link
Contributor

simahawk commented Oct 9, 2024

/ocabot migration connector

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Oct 9, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 9, 2024
6 tasks
Copy link
Member

@mmequignon mmequignon left a comment

Choose a reason for hiding this comment

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

LGTM
You can drop the test-requirements.txt file, as those modules have been merged.
Thank you !

@simahawk
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-494-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3b6e7b9 into OCA:18.0 Oct 17, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e0b5ccb. Thanks a lot for contributing to OCA. ❤️

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.