-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
[14.0][MIG] account_fiscal_year_closing #169
[14.0][MIG] account_fiscal_year_closing #169
Conversation
c1076a0
to
60166c4
Compare
"version": "14.0.1.0.0", | ||
"category": "Accounting & Finance", | ||
"website": "https://github.com/OCA/account-closing", | ||
"author": "Tecnativa, " "Odoo Community Association (OCA)", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this " "
admitted?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for having a look!
Yes it's admitted apparently (there are no complaints by linters) but it's ugly, just removed it
} | ||
|
||
# @api.onchange('closing_template_id') | ||
# Not working due to https://github.com/odoo/odoo/issues/20163 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently not: odoo/odoo#20163 (comment).
I enabled the onchange and removed both the method action_load_template
and is_new_template
that existed only to perform the onchange's job.
Tried the loading on onchange in my local environment and seems to be working fine
closing_type = fields.Selection( | ||
selection=_selection_closing_type, | ||
default="none", | ||
states={"posted": [("readonly", True)]}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A value here should always exist, see #165
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather wait for #165 to be merged and then eventually re-base to have the merged changes
I'm not sure if it's related to my data or something else I'm getting the following traceback while clicking on calculate button: As far I understand given a quick analysis I'm not sure if we should add the date on the model
|
8ca3714
to
074d8f8
Compare
074d8f8
to
77ac991
Compare
Hi, thanks for having a look!
The field I haven't managed to reproduce your error but seems reasonable to add the |
@sergiocorato can you update your review? Unfortunately #165 hasn't been merged yet so I think there is nothing to change here for #169 (comment). @petrus-v any news about #169 (comment)? The issue you found should be now fixed. |
I had a quick test regarding this module and sorry for this poor report without the way to reproduce that I can't even remember too... I can see your last commit should avoid this error it could be nice to add a test case to cover it but without the way to reproduce it's a bit annoying. To be honest I'll be pretty busy in next few weeks and not that much interest here to give a try right now :-/ |
Hello,
In method inverse_move_prepare of AccountFiscalyearClosingConfig, we have the case of move_ids stay False and can't return ids. Is it possible to have a look ? returning move_ids (w/o .ids) Thanks for your works. |
Hi, thanks for testing this. |
I'm not an accounting expert, we are testing a v14 coming from a v12. And during the 'Calculate' action, the error is raised. |
Hello, I come back after deep investigation. 1. Issue on saveMy templates are filled correctly (each config have the "accounts mapping", except the latest): How to reproduce:
I found this issue which talks about this kind of issue: odoo/odoo#11236 (comment)
And the save works again ! 2. Warning on deprecated force_company:just change by: tmpl = self.closing_template_id.with_company(self.company_id.id)
I don't know if you have theses issues. Regards. |
@SimoRubi hi, is there a plan/roadmap for completing porting of this module? |
Hi, this is in my TODO list but I don't know when I'll have time to spend for this. If you want to work on this you are welcome to supersede just leave a note here and create a new PR starting from this branch. |
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-closing-12.0/account-closing-12.0-account_fiscal_year_closing Translate-URL: https://translation.odoo-community.org/projects/account-closing-12-0/account-closing-12-0-account_fiscal_year_closing/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-closing-12.0/account-closing-12.0-account_fiscal_year_closing Translate-URL: https://translation.odoo-community.org/projects/account-closing-12-0/account-closing-12-0-account_fiscal_year_closing/
Currently translated at 66.6% (76 of 114 strings) Translation: account-closing-12.0/account-closing-12.0-account_fiscal_year_closing Translate-URL: https://translation.odoo-community.org/projects/account-closing-12-0/account-closing-12-0-account_fiscal_year_closing/it/
….fiscalyear.closing.template
Fixed warnings: Field account.fiscalyear.closing.closing_template_id: unknown parameter 'oldname', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in order to allow it Field account.move.fyc_id: unknown parameter 'delete', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in order to allow it The model account.fiscalyear.closing.config has no _description The model account.fiscalyear.closing.mapping has no _description The model account.fiscalyear.closing.type has no _description The model account.fiscalyear.closing.template has no _description The model account.fiscalyear.closing.config.template has no _description The model account.fiscalyear.closing.mapping.template has no _description The model account.fiscalyear.closing.type.template has no _description The model account.fiscalyear.closing.unbalanced.move has no access rules The model account.fiscalyear.closing.unbalanced.move.line has no access rules odoo.addons.base.models.ir_ui_view: Search tag requires at least one field element The method '_company_default_get' on res.company is deprecated and shouldn't be used anymore DeprecationWarning: RedirectWarning method 'post()' is a deprecated alias to 'action_post()' or _post()
…line. Otherwise the following might happen: File "/path/to/odoo/addons/account_fiscal_year_closing/models/account_fiscalyear_closing.py", line 328, in button_calculate res = self.calculate() File "/path/to/odoo/addons/account_fiscal_year_closing/models/account_fiscalyear_closing.py", line 315, in calculate return self._show_unbalanced_move_wizard(data) File "/path/to/odoo/addons/account_fiscal_year_closing/models/account_fiscalyear_closing.py", line 295, in _show_unbalanced_move_wizard wizard = self.env["account.fiscalyear.closing.unbalanced.move"].create(data) [...] ValueError: Invalid field 'date' on model 'account.fiscalyear.closing.unbalanced.move.line'
77ac991
to
7011fb0
Compare
hi @sergiocorato any chance for your review ? |
@OCA/accounting-maintainers any chance to review this porting? |
This PR has the |
@OCA/accounting-maintainers review? |
@matteoopenf there is a pending review for @sergiocorato |
#199 Superseed @SimoRubi @francesco-ooops @mde-spring @sergiocorato |
@SirTakobi la pr ho sistemato e fixato i test di la fammi sapere |
Migration from https://github.com/OCA/account-closing/tree/60de43819512b8b2df30dff2fb5cc4bda0b11fce