RFR: 8089398: [ChoiceBox, ComboBox] throws NPE on setting value on null selectionModel [v6]

Ajit Ghaisas aghaisas at openjdk.java.net
Wed Sep 8 06:35:09 UTC 2021


On Tue, 27 Jul 2021 12:41:00 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

>> This PR fixes multiple NPEs in Choice-and ComboBox, when the selection model is null.
>> 
>> ChoiceBox: 
>> - Null check in **valueProperty()** listener
>> 
>> ComboBox:
>> - Null check in **editableProperty()** listener
>> - Null check in **valueProperty()** listener
>> - Null check in **ComboBoxListViewSkin#updateValue()**
>> - Null check in **ComboBoxListViewSkin#layoutChildren()**
>> - 2x Null check in **ComboBoxListViewSkin#createListView()**
>> 
>> ~~The tests checks, that no NPE is printed to the console.
>> Some checks, that the set value is still displayed (either in the ComboBox button cell or the ChoiceBox display label)~~
>
> Marius Hanl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fixed another NPE on skin creation when the cbx selection model is null

The latest patch does not apply cleanly due to the change in copyright headers.
It was fixed in - [JDK-8270960](https://bugs.openjdk.java.net/browse/JDK-8270960)

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

PR: https://git.openjdk.java.net/jfx/pull/557


More information about the openjfx-dev mailing list