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

[FIX] sustainability_purchase: Fix traceback with partner merging wizard #222

Draft
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

BurkhalterY
Copy link
Member

Description

The native partner merging wizard doesn't support fields of type reference because it is never used by Odoo on model res.partner. Because we add a reference field on res.partner, we got a traceback using this wizard on a record with this field filled. This PR fix it.

Related Issues

https://opennet.odooapps.ch/web#id=115597&cids=2&menu_id=1803&model=project.task&view_type=form (private)

Self proofreading checklist

  • Code Review: Have you reviewed your code to ensure it follows best practices
    and meets the project's coding standards?
  • Testing: Have you tested your changes locally to ensure they work as expected?
  • Documentation: Have you updated any relevant documentation or comments to
    reflect your changes?
  • Pre-commit Checks: Have you ensured that pre-commit checks have automatically
    run upon committing to catch any potential issues?

Test Instructions

Choose two partners with a parent_id defined and try to merge them. Without this commit, traceback 💥

Additional Notes

N/A

The native partner merging wizard doesn't support fields of type `reference` because it is never used by Odoo on model `res.partner`.
Because we add a `reference` field on `res.partner`, we got a traceback using this wizard on a record with this field filled. This commit fix it.

C.f. ONS(project.task,115597)
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.

1 participant