Fixes the use of DataGrid Template Contexts for ExtraProperties#25466
Conversation
|
Hi @carlosv14, thanks for the fix! Could you change the base branch of this PR from You can change the base branch via the "Edit" button next to the PR title on GitHub. Thanks! |
|
Hi @maliming changed the base branch as requested. Thanks! |
d84c563 to
3c000cf
Compare
There was a problem hiding this comment.
Pull request overview
Updates AbpExtensibleDataGrid to work with Blazorise 2.0+ DataGrid template contexts, where the template context is a context object (with an Item) rather than the row item itself. This restores rendering of ExtraProperties values in grids (fixing the regression reported in #25465).
Changes:
- Use
context.Item(instead ofcontext) when accessing the row entity forExtraPropertiesrendering.
Description
Resolves #25465
TODO: As per Blazorise migration docs DataGrid Template Contexts now receive context objects instead of the raw row item and any code that assumes context is TItem should be updated. So the line
var entity = context as IHasExtraProperties;should bevar entity = context.Item as IHasExtraProperties;otherwise entity is null and propertyValue is also null.Checklist