Re: 8134600 Can't pass ObservableList as argument using FXML

[Kevin Rushforth]

> Fix is applicable for both 7 and 8.

I presume you mean both *9* and 8?

-- Kevin


Andrey Rusakov wrote:

Hello, Everyone!
Here is my small fix for FXML, JDK-8134600. 

The reason was ProxyBuilder.getUserValue where check for Collection 
class was performed before check for ArrayListWrapper class, so check 
for ArrayListWrapper class could not be fulfilled at all.
Case is covered by 
FxmlTests/test/fxmltests/functional/staticPropertyLoadTest.java#coerceSlot 
test, so there is probably no need to create a new one.

Fix is applicable for both 7 and 8.
http://cr.openjdk.java.net/~arusakov/8134600/webrev.00/
 

Re: 8134600 Can't pass ObservableList as argument using FXML

[Andrey Rusakov]

Exactly

21.04.2016 18:08, Kevin Rushforth пишет:


> Fix is applicable for both 7 and 8.

I presume you mean both *9* and 8?

-- Kevin


Andrey Rusakov wrote:

Hello, Everyone!
Here is my small fix for FXML,JDK-8134600. 

The reason was ProxyBuilder.getUserValue where check for Collection 
class was performed before check for ArrayListWrapper class, so check 
for ArrayListWrapper class could not be fulfilled at all.
Case is covered by 
FxmlTests/test/fxmltests/functional/staticPropertyLoadTest.java#coerceSlot 
test, so there is probably no need to create a new one.

Fix is applicable for both 7 and 8.
http://cr.openjdk.java.net/~arusakov/8134600/webrev.00/
 

8134600 Can't pass ObservableList as argument using FXML

[Andrey Rusakov]

Hello, Everyone!
Here is my small fix for FXML,JDK-8134600. 

The reason was ProxyBuilder.getUserValue where check for Collection 
class was performed before check for ArrayListWrapper class, so check 
for ArrayListWrapper class could not be fulfilled at all.
Case is covered by 
FxmlTests/test/fxmltests/functional/staticPropertyLoadTest.java#coerceSlot 
test, so there is probably no need to create a new one.

Fix is applicable for both 7 and 8.
http://cr.openjdk.java.net/~arusakov/8134600/webrev.00/