Skip to content

[7.x.x] report the first wrong let variable type - #247

Open
Laurettta wants to merge 1 commit into
develop-7.x.xfrom
7.x.x/hotfix/let-type-check-before-return
Open

Laurettta wants to merge 1 commit into
develop-7.x.xfrom
7.x.x/hotfix/let-type-check-before-return

Conversation

@Laurettta

Copy link
Copy Markdown

The error for a wrong let type named a later variable instead of the first wrong one, because LetExpr.eval() ran the rest of the query before checking its own variable. I moved the type check to run first and added a test in XQueryTest. It failed before the change and passes now.

Closes #5315

@cla-bot cla-bot Bot added the cla-signed label Nov 3, 2026
@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Laurettta Laurettta changed the title [hotfix] report the first wrong let variable type [7.x.x] report the first wrong let variable type Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant