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] account_payment_term_extension: Migration to 18.0 #788

Open
wants to merge 57 commits into
base: 18.0
Choose a base branch
from

Conversation

TheMule71
Copy link

Replaces #776

@TheMule71 TheMule71 mentioned this pull request Feb 5, 2025
8 tasks
@TheMule71
Copy link
Author

TheMule71 commented Feb 5, 2025

I do have a question, according to the migration guidelines, you should

Remove references in CREDITS.rst of past financed migrations by other companies.

which I did, but I'm not totally comfortable with. The previous credits were for the 15.0->16.0 migration, and survived the 16.0 -> 17.0 migration. Should I keep them?

@TheMule71 TheMule71 force-pushed the 18.0-mig-account_payment_term_extension branch 2 times, most recently from 27aa7e0 to a28fe61 Compare February 5, 2025 11:49
Copy link

@gdemarco2esseti gdemarco2esseti left a comment

Choose a reason for hiding this comment

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

Functional review: KO

Whenever a change is made in the configuration, the following error appears when attempting to save:

Odoo Server Error

RPC_ERROR
Odoo Server Error

Occured on oca-account-payment-18-0-pr788-a28fe61a9e54.runboat.odoo-community.org on model account.payment.term and id 71 on 2025-02-07 09:25:03 GMT

Traceback (most recent call last):
File "/opt/odoo/odoo/http.py", line 1961, in _transactioning
return service_model.retrying(func, env=self.env)
File "/opt/odoo/odoo/service/model.py", line 137, in retrying
result = func()
File "/opt/odoo/odoo/http.py", line 1928, in _serve_ir_http
response = self.dispatcher.dispatch(rule.endpoint, args)
File "/opt/odoo/odoo/http.py", line 2176, in dispatch
result = self.request.registry['ir.http']._dispatch(endpoint)
File "/opt/odoo/odoo/addons/base/models/ir_http.py", line 329, in _dispatch
result = endpoint(**request.params)
File "/opt/odoo/odoo/http.py", line 727, in route_wrapper
result = endpoint(self, *args, **params_ok)
File "/opt/odoo/addons/web/controllers/dataset.py", line 35, in call_kw
return call_kw(request.env[model], method, args, kwargs)
File "/opt/odoo/odoo/api.py", line 517, in call_kw
result = getattr(recs, name)(*args, **kwargs)
File "/opt/odoo/addons/web/models/models.py", line 70, in web_save
self.write(vals)
File "/opt/odoo/odoo/models.py", line 4749, in write
field.write(self, value)
File "/opt/odoo/odoo/fields.py", line 4496, in write
self.write_batch([(records, value)])
File "/opt/odoo/odoo/fields.py", line 4517, in write_batch
self.write_real(records_commands_list, create)
File "/opt/odoo/odoo/fields.py", line 4686, in write_real
comodel.browse(command[1]).with_prefetch(prefetch_ids).write(command[2])
File "/opt/odoo/odoo/models.py", line 4771, in write
real_recs._validate_fields(vals, inverse_fields)
File "/opt/odoo/odoo/models.py", line 1599, in _validate_fields
check(self)
File "/mnt/data/odoo-addons-dir/account_payment_term_extension/models/account_payment_term_line.py", line 100, in _check_payment_days
raise exceptions.Warning(
AttributeError: module 'odoo.exceptions' has no attribute 'Warning'

The above server error caused the following client error:
RPC_ERROR: Odoo Server Error
RPC_ERROR
at makeErrorFromResponse (http://oca-account-payment-18-0-pr788-a28fe61a9e54.runboat.odoo-community.org/web/assets/50cbccd/web.assets_web.min.js:3134:163)
at XMLHttpRequest. (http://oca-account-payment-18-0-pr788-a28fe61a9e54.runboat.odoo-community.org/web/assets/50cbccd/web.assets_web.min.js:3139:13)

@TheMule71 TheMule71 force-pushed the 18.0-mig-account_payment_term_extension branch from a28fe61 to 8af13e9 Compare February 7, 2025 09:55
@TheMule71
Copy link
Author

TheMule71 commented Feb 7, 2025

Functional review: KO

Whenever a change is made in the configuration, the following error appears when attempting to save:

It happens only when a mistake is made when configuring the module. Eg. this is a screenshot from my tests on runboat:
image

Nicely spotted, it's there since 14.0, although the deprecation warning was visibile only in the logs.

Fixed.

@TheMule71
Copy link
Author

Test on runboat:
image

this is the exception to be displayed. Warning is no longer valid and has been deprecated since 14.0.

Alexis de Lattre and others added 22 commits February 7, 2025 15:40
…_term_extension

Add support for months and weeks in account.payment.term.line
Add unittest
PEP8 validation
Fix and add a feature for apply a chronological order on lines
Merge of the 'account_payment_term_multi_day' module
Add icon to module
[UPD] Update account_payment_term_extension.pot
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/es/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/pt_BR/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/fr/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/fr_CH/
Currently translated at 85.7% (24 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/hr/
andreampiovesana and others added 28 commits February 7, 2025 15:40
reformat
add test months
black
Currently translated at 100.0% (34 of 34 strings)

Translation: account-payment-14.0/account-payment-14.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-14-0/account-payment-14-0-account_payment_term_extension/es_AR/
Currently translated at 94.1% (32 of 34 strings)

Translation: account-payment-14.0/account-payment-14.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-14-0/account-payment-14-0-account_payment_term_extension/nl/
Currently translated at 100.0% (34 of 34 strings)

Translation: account-payment-14.0/account-payment-14.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-14-0/account-payment-14-0-account_payment_term_extension/it/
Currently translated at 100.0% (34 of 34 strings)

Translation: account-payment-14.0/account-payment-14.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-14-0/account-payment-14-0-account_payment_term_extension/it/
[UPD] Update account_payment_term_extension.pot
Currently translated at 100.0% (34 of 34 strings)

Translation: account-payment-15.0/account-payment-15.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-15-0/account-payment-15-0-account_payment_term_extension/ca/
Currently translated at 100.0% (34 of 34 strings)

Translation: account-payment-15.0/account-payment-15.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-15-0/account-payment-15-0-account_payment_term_extension/pt_BR/
Currently translated at 27.2% (9 of 33 strings)

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

Translation: account-payment-16.0/account-payment-16.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-16-0/account-payment-16-0-account_payment_term_extension/es/
Currently translated at 87.8% (29 of 33 strings)

Translation: account-payment-16.0/account-payment-16.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-16-0/account-payment-16-0-account_payment_term_extension/it/
Currently translated at 87.8% (29 of 33 strings)

Translation: account-payment-16.0/account-payment-16.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-16-0/account-payment-16-0-account_payment_term_extension/it/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (33 of 33 strings)

Translation: account-payment-16.0/account-payment-16.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-16-0/account-payment-16-0-account_payment_term_extension/sv/
Currently translated at 100.0% (33 of 33 strings)

Translation: account-payment-16.0/account-payment-16.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-16-0/account-payment-16-0-account_payment_term_extension/it/
Currently translated at 100.0% (44 of 44 strings)

Translation: account-payment-17.0/account-payment-17.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-17-0/account-payment-17-0-account_payment_term_extension/it/
@TheMule71 TheMule71 force-pushed the 18.0-mig-account_payment_term_extension branch from 8af13e9 to 2a2a6c3 Compare February 7, 2025 14:40
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.