> 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:

  Make sure instantiation works also when ProxyBuilder is not used

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

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

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

  Stats: 49 lines in 6 files changed: 25 ins; 13 del; 11 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