Skip to content

DataGrid: fix validation data object with repaintChangesOnly (T1323690)#32867

Merged
anna-shakhova merged 1 commit intoDevExpress:26_1from
anna-shakhova:26_1__T1323690_validation_data
Mar 13, 2026
Merged

DataGrid: fix validation data object with repaintChangesOnly (T1323690)#32867
anna-shakhova merged 1 commit intoDevExpress:26_1from
anna-shakhova:26_1__T1323690_validation_data

Conversation

@anna-shakhova
Copy link
Contributor

No description provided.

@anna-shakhova anna-shakhova self-assigned this Mar 11, 2026
@anna-shakhova anna-shakhova requested a review from a team as a code owner March 11, 2026 12:07
Copilot AI review requested due to automatic review settings March 11, 2026 12:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes DataGrid validation callback data payload when repaintChangesOnly is enabled, ensuring validators receive the correct row data after edits are reverted.

Changes:

  • Adjusted ValidatingController’s dataGetter to provide a non-empty data object when there is no active change for the row.
  • Added integration tests to verify validationCallback receives real row data after reverting changes and merged (edited) row data during active editing.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/devextreme/js/__internal/grids/grid_core/validating/m_validating.ts Updates validator dataGetter logic to supply appropriate row data when changes are absent/present.
packages/devextreme/js/__internal/grids/grid_core/validating/tests/validating.integration.test.ts Adds regression tests for correct validationCallback data with repaintChangesOnly + revert/edit flows.

@anna-shakhova anna-shakhova merged commit ec1d4b5 into DevExpress:26_1 Mar 13, 2026
106 checks passed
@anna-shakhova anna-shakhova deleted the 26_1__T1323690_validation_data branch March 13, 2026 14:15
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.

3 participants