Skip to content

fix(warnings): ASP0006 - #1158

Merged
sven-n merged 3 commits into
MUnique:masterfrom
eduardosmaniotto:fix/asp0006
Oct 9, 2026
Merged

sven-n merged 3 commits into
MUnique:masterfrom
eduardosmaniotto:fix/asp0006

Conversation

@eduardosmaniotto

Copy link
Copy Markdown
Contributor

Summary

Eliminates all 33 ASP0006 warnings by replacing manual RenderTreeBuilder
sequence counters with Razor markup (compiler-generated sequence numbers).
Rendered output is unchanged.

  • EnumSelect.cs → EnumSelect.razor + code-behind (same binder expression,
    splat-first attributes, conditional selected); plus bUnit coverage
  • New EditShell wrapper (Breadcrumb, <h1>, CascadingValue) + one
    .razor file per edit page (EditAccount, EditConnectionServer,
    EditConfig); AutoForm<> renders via DynamicComponent with identical
    parameters and callbacks. EditBase data logic untouched
  • Last 4 warnings suppressed with a documented #pragma at the single
    BaseComponentBuilder funnel: runtime-selected components can't use
    literals, and the threaded counter is unique and stable per render
    (reason in <remarks>)

Verification

  • dotnet build: 0 ASP0006, 0 errors, no new warnings
  • dotnet test MUnique.OpenMU.Web.Tests: 203/203 pass
  • Manual smoke: open the three edit pages, change a value, save;
    check enum dropdowns render and select correctly

@sven-n
sven-n merged commit 43ace6d into MUnique:master Oct 9, 2026
1 check passed
@eduardosmaniotto
eduardosmaniotto deleted the fix/asp0006 branch October 9, 2026 16:49
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.

2 participants