Skip to content

Conversation

@DiegoCardoso
Copy link
Contributor

Description

Add the same behavior as when createComponenet is called and the component renderer returns null to the updateComponent method. That prevents a NPE in the AbstractComponentDataGenerator#refreshData method when it tries to call getElement in the updated component.

Fixes #6825

Add the same behavior as when `createComponenet` is called and the
component renderer returns `null` to the `updateComponent` method. That
prevents a NPE in the `AbstractComponentDataGenerator#refreshData`
method when it tries to call `getElement` in the updated component.

Fixes #6825
@DiegoCardoso DiegoCardoso force-pushed the fix/renderer/update-null-check branch from 4b928dd to 82839d4 Compare October 17, 2025 09:12
@sonarqubecloud
Copy link

@DiegoCardoso DiegoCardoso merged commit d56325a into main Oct 20, 2025
5 checks passed
@DiegoCardoso DiegoCardoso deleted the fix/renderer/update-null-check branch October 20, 2025 12:59
vaadin-bot pushed a commit that referenced this pull request Oct 20, 2025
…#8168)

Add the same behavior as when `createComponenet` is called and the
component renderer returns `null` to the `updateComponent` method. That
prevents a NPE in the `AbstractComponentDataGenerator#refreshData`
method when it tries to call `getElement` in the updated component.

Fixes #6825
vaadin-bot pushed a commit that referenced this pull request Oct 20, 2025
…#8168)

Add the same behavior as when `createComponenet` is called and the
component renderer returns `null` to the `updateComponent` method. That
prevents a NPE in the `AbstractComponentDataGenerator#refreshData`
method when it tries to call `getElement` in the updated component.

Fixes #6825
DiegoCardoso added a commit that referenced this pull request Oct 20, 2025
…#8168) (#8173)

Add the same behavior as when `createComponenet` is called and the
component renderer returns `null` to the `updateComponent` method. That
prevents a NPE in the `AbstractComponentDataGenerator#refreshData`
method when it tries to call `getElement` in the updated component.

Fixes #6825

Co-authored-by: Diego Cardoso <[email protected]>
DiegoCardoso added a commit that referenced this pull request Oct 20, 2025
…#8168) (#8172)

Add the same behavior as when `createComponenet` is called and the
component renderer returns `null` to the `updateComponent` method. That
prevents a NPE in the `AbstractComponentDataGenerator#refreshData`
method when it tries to call `getElement` in the updated component.

Fixes #6825

Co-authored-by: Diego Cardoso <[email protected]>
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.

Inconsistent null handling in ComponentDataGenerator

4 participants