Skip to content

forms: setValidationScope side-effect#1084

Closed
Tezar wants to merge 219 commits into
nette:masterfrom
Tezar:doc-2.4
Closed

forms: setValidationScope side-effect#1084
Tezar wants to merge 219 commits into
nette:masterfrom
Tezar:doc-2.4

Conversation

@Tezar

@Tezar Tezar commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

Clarify impact of setValidationScope on getValues()

@dg dg force-pushed the master branch 11 times, most recently from cbb39d0 to ac8e22c Compare June 14, 2026 16:49
@dg dg force-pushed the master branch 4 times, most recently from 2a9bd2e to 5b94fa4 Compare June 28, 2026 16:53
@dg dg force-pushed the master branch 13 times, most recently from 29d832d to 36025b4 Compare July 5, 2026 14:51
@dg

dg commented Jul 5, 2026

Copy link
Copy Markdown
Member

Thanks! Verified against the source (Container::getUntrustedValues() filters by the submitter's validation scope), the behavior is correct. Incorporated in e7a6d9b9e as a separate, more explicit paragraph explaining that getValues() returns only the values of controls within the validation scope. Closing since the branch predates several force-pushes and can't be merged directly.

@dg dg closed this Jul 5, 2026
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.