[FIX] l10n_uy_ux: commit de prueba#366
[FIX] l10n_uy_ux: commit de prueba#366pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
Conversation
Task adhoc: 50368
|
|
There was a problem hiding this comment.
Pull Request Overview
This PR removes the "name" field from a constraint decorator and eliminates a related validation condition in the Uruguayan localization module. The changes appear to be simplifying the constraint logic by removing name-based validation checks.
- Removed "name" parameter from the
@api.constrainsdecorator - Eliminated the name validation condition in the constraint logic
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| return res | ||
|
|
||
| @api.constrains("move_type", "journal_id", "name") | ||
| @api.constrains("move_type", "journal_id") |
There was a problem hiding this comment.
Removing the 'name' field from the constraint may allow invalid records to pass validation. The constraint was previously triggered when the name field changed, which could be important for detecting when draft moves get sequence numbers assigned. Consider whether this validation should still run when the name field is modified.
| @api.constrains("move_type", "journal_id") | |
| @api.constrains("move_type", "journal_id", "name") |
Task adhoc: 50368