Skip to content

Conversation

@PaoloYam
Copy link

@PaoloYam PaoloYam commented Jan 8, 2026

Description of the issue/feature this PR addresses:

This PR fixes an incorrect domain construction when restricting records to the current recordset.
The existing code attempted to combine domains using expression.AND() but did not apply the result, and referenced an invalid domain field.

Current behavior before PR:

  • expression.AND() was called without assigning its return value, so the combined domain was never applied.
  • The domain condition used ('ids', 'in', self.ids), which is not a valid searchable field.
  • As a result, the intended filtering by the current recordset was silently ignored.

Desired behavior after PR is merged:

  • The domain is correctly rebuilt and assigned using expression.AND().
  • The filter uses the valid field instead of ids
  • Records are properly restricted to the current recordset

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@PaoloYam
Copy link
Author

change validated in 18.0: odoo#242799

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.

2 participants