From 6fe22a03a3b2386fe118116a5f7d9219f5920235 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 2 Sep 2026 13:19:04 +0000 Subject: [PATCH] Match the softened frozen-view-builder wording The rule table described the old builder as something that "matters more than it looks". The linter no longer says it that way: `z2ui5_cl_xml_view` keeps working and is headed for an addon of its own, so the row now says what the finding is really about - none of the rules above it can read that API, so none of them ran - and recommends the switch for what it buys. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_011qkUxU6fypKKwy1WE8kmDA --- docs/advanced/linter.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/advanced/linter.md b/docs/advanced/linter.md index f89eb9bc..ae98cea3 100644 --- a/docs/advanced/linter.md +++ b/docs/advanced/linter.md @@ -152,7 +152,7 @@ them: | Rule | What it catches | | --- | --- | | `unknown-binding-path` | a hand-written `{/TYPO}` the derived model has no path for — the field just stays empty. Inside a bound aggregation a relative `{TYPO}` is resolved against the **row**, so a misspelled column is caught too | -| `frozen-view-builder` | the class still builds its view with `z2ui5_cl_xml_view`, the frozen predecessor. It matters more than it looks: that API is what nearly all public abap2UI5 material shows, and therefore what a language model writes when asked for an app | +| `frozen-view-builder` | the class builds its view with `z2ui5_cl_xml_view`, the predecessor. That is fine to keep — it works, and is headed for an addon of its own — but no rule above can read it, so none of them ran: switching to `z2ui5_cl_ui5_view_builder` is what buys the whole gate | | `binding-to-nonpublic` | a `PROTECTED`/`PRIVATE` attribute bound — only `PUBLIC` attributes are serialized into the model, so the first roundtrip fails with `BINDING_ERROR` | | `binding-to-local` | a local variable bound — the instance is serialized across the roundtrip, the method stack is not, so the value is lost | | `unconverted-abap-boolean` | an ABAP boolean written straight into the view: it arrives as `'X'`/`' '` and UI5 reads any non-empty string as true, so `visible = abap_false` makes the control **visible** |