Skip to content

Improve error when FieldIdentifier is invalid in EditContext#65869

Open
Bellambharath wants to merge 4 commits intodotnet:mainfrom
Bellambharath:fix/45096-fieldidentifier-null-message
Open

Improve error when FieldIdentifier is invalid in EditContext#65869
Bellambharath wants to merge 4 commits intodotnet:mainfrom
Bellambharath:fix/45096-fieldidentifier-null-message

Conversation

@Bellambharath
Copy link
Copy Markdown

Summary:

  1. Validate FieldIdentifier in EditContext and throw a clearer error for invalid input.
  2. Add test for invalid FieldIdentifier passed to NotifyFieldChanged.

Testing:

  1. dotnet test src/Components/Forms/test/Microsoft.AspNetCore.Components.Forms.Tests.csproj

Fixes #45096

@Bellambharath Bellambharath requested a review from a team as a code owner March 19, 2026 08:18
@github-actions github-actions Bot added the area-blazor Includes: Blazor, Razor Components label Mar 19, 2026
@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Mar 19, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Thanks for your PR, @@Bellambharath. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@Bellambharath
Copy link
Copy Markdown
Author

CI failed in Helix:
Microsoft.AspNetCore.Mvc.FunctionalTests--net11.0
Looks unrelated to my change (no MVC functional test changes). Could you please re-run CI?

@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Looks like this PR hasn't been active for some time and the codebase could have been changed in the meantime.
To make sure no conflicting changes have occurred, please rerun validation before merging. You can do this by leaving an /azp run comment here (requires commit rights), or by simply closing and reopening.

@dotnet-policy-service dotnet-policy-service Bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Mar 26, 2026
@Bellambharath
Copy link
Copy Markdown
Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 65869 in repo dotnet/aspnetcore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components community-contribution Indicates that the PR has been added by a community member pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blazor Value cannot be null. (Parameter 'obj')

1 participant