Skip to content

[FIX] l10n_uy_ux: commit de prueba#366

Open
pablohmontenegro wants to merge 1 commit into
ingadhoc:18.0from
adhoc-dev:18.0-t-50368-pam
Open

[FIX] l10n_uy_ux: commit de prueba#366
pablohmontenegro wants to merge 1 commit into
ingadhoc:18.0from
adhoc-dev:18.0-t-50368-pam

Conversation

@pablohmontenegro

Copy link
Copy Markdown
Contributor

Task adhoc: 50368

Copilot AI review requested due to automatic review settings October 20, 2025 13:10
@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.constrains decorator
  • 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")

Copilot AI Oct 20, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
@api.constrains("move_type", "journal_id")
@api.constrains("move_type", "journal_id", "name")

Copilot uses AI. Check for mistakes.
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.

3 participants