Skip to content
New issue

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

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

Already on GitHub? # to your account

[18.0][MIG] account_reconcile_oca #808

Merged
merged 211 commits into from
Mar 2, 2025

Conversation

JordiBForgeFlow
Copy link
Member

@JordiBForgeFlow JordiBForgeFlow commented Feb 25, 2025

etobella and others added 28 commits February 25, 2025 18:30
* In "reconcile" tab, don't propose the move of the currently processed statement line

* In accounting, only select/search parent partners
* refactoring JS in order to add logic
* Improve views
* Minor fixes in Odoo code in order to fix all possible options
…of account move number (name) fot the counterpart account move line
Currently translated at 1.0% (1 of 92 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_reconcile_oca/es/
Currently translated at 25.0% (23 of 92 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_reconcile_oca/hr/
Currently translated at 81.5% (75 of 92 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_reconcile_oca/tr/
By default, account_account name is a char field. But if
l10n_multilang is installed, it is translatable, hence
the DB column is now a jsonb.
We must handle both cases in the select query.
Currently translated at 76.0% (70 of 92 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_reconcile_oca/pt_BR/
Currently translated at 100.0% (92 of 92 strings)

Translation: account-reconcile-16.0/account-reconcile-16.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-16-0/account-reconcile-16-0-account_reconcile_oca/fr/
OCA-git-bot and others added 11 commits February 25, 2025 18:30
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-reconcile-17.0/account-reconcile-17.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-17-0/account-reconcile-17-0-account_reconcile_oca/
Currently translated at 100.0% (122 of 122 strings)

Translation: account-reconcile-17.0/account-reconcile-17.0-account_reconcile_oca
Translate-URL: https://translation.odoo-community.org/projects/account-reconcile-17-0/account-reconcile-17-0-account_reconcile_oca/it/
… log

WARNING prod py.warnings: /opt/odoo/auto/addons/account_reconcile_oca/models/account_bank_statement_line.py:616: DeprecationWarning: Since 17.0, deprecated method, read display_name instead
… to avoid warning log

WARNING prod py.warnings: /opt/odoo/auto/addons/account/models/account_move.py:4651: UserWarning: unsupported operand type(s) for "==": 'res.partner()' == '403'
@JordiBForgeFlow JordiBForgeFlow force-pushed the 18.0-mig-account_reconcile_oca branch from 43f87b4 to 203719b Compare February 25, 2025 17:31
@JordiBForgeFlow JordiBForgeFlow force-pushed the 18.0-mig-account_reconcile_oca branch from 203719b to d4b9d8f Compare February 25, 2025 17:33
@JordiBForgeFlow
Copy link
Member Author

@etobella All is green here

@victoralmau
Copy link
Member

Please, cherry-pick #809 to commit history

…_get_reconcile_line()

Example use case:
 File "/opt/odoo/auto/addons/account_reconcile_oca/models/account_bank_statement_line.py", line 541, in _compute_reconcile_data_info
    record.reconcile_data_info = record._default_reconcile_data(
  File "/opt/odoo/auto/addons/account_reconcile_oca/models/account_bank_statement_line.py", line 698, in _default_reconcile_data
    reconcile_auxiliary_id, lines = self._get_reconcile_line(
  File "/opt/odoo/auto/addons/account_reconcile_oca/models/account_bank_statement_line.py", line 1164, in _get_reconcile_line
    new_vals = super()._get_reconcile_line(
  File "/opt/odoo/auto/addons/account_reconcile_oca/models/account_reconcile_abstract.py", line 49, in _get_reconcile_line
    original_amount = amount = net_amount = line.debit - line.credit
  File "/opt/odoo/custom/src/odoo/odoo/fields.py", line 1154, in __get__
    record.ensure_one()
  File "/opt/odoo/custom/src/odoo/odoo/models.py", line 5204, in ensure_one
    raise ValueError("Expected singleton: %s" % self)
ValueError: Expected singleton: account.move.line(208, 211, 212)

TT55221
@JordiBForgeFlow
Copy link
Member Author

Please, cherry-pick #809 to commit history

@victoralmau Done!

Copy link

@houssine78 houssine78 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Some commit squashing in the Jordi's work may be good, but let's not be picky.

/ocabot migration account_reconcile_oca
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 2, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 2, 2025
4 tasks
@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-808-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 093beb0 into OCA:18.0 Mar 2, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.