On Tue, 7 Jul 2026 11:10:23 GMT, Petr Štechmüller <[email protected]> wrote:

>> 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:
> 
>   More tests + copyright

modules/javafx.fxml/src/test/resources/test/javafx/fxml/class_with_default_list_property.fxml
 line 2:

> 1: <?xml version="1.0" encoding="UTF-8"?>
> 2: <!--

this should be above: `<?xml version="1.0" encoding="UTF-8"?>`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2167#discussion_r3536573925

Reply via email to