> ProxyBuilder is used by the FXML loader to instantiate classes whose 
> constructors are annotated with `@NamedArg`. When such a class also exposes a 
> read-only `Map` property (e.g. getProperties() — the same pattern used by 
> `javafx.scene.Node`), setting child elements under that property in FXML 
> caused incorrect behaviour or a runtime error.
> 
> **Root cause:** _getReadOnlyProperty()_ always returned an `ArrayListWrapper` 
> regardless of the actual **getter** return type. When the getter returns a 
> `Map`, an `ArrayListWrapper` is the wrong container and the entries are never 
> transferred to the real map on the object.
> 
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Petr Štechmüller has updated the pull request incrementally with one additional 
commit since the last revision:

  Add ObservableSet check + tests

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/2167/files
  - new: https://git.openjdk.org/jfx/pull/2167/files/b855f7e0..be105453

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2167&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2167&range=00-01

  Stats: 271 lines in 8 files changed: 251 ins; 5 del; 15 mod
  Patch: https://git.openjdk.org/jfx/pull/2167.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2167/head:pull/2167

PR: https://git.openjdk.org/jfx/pull/2167

Reply via email to