> Setting a null selection model in TableView and TreeTableView produce NPE on > sorting (and probably in some other situations) because the check for null is > missing in several places. > > Setting a null selection model is a valid way to disable selection in a > (tree)table. > > There is also a similar issue with ListView > [JDK-8279640](https://bugs.openjdk.org/browse/JDK-8279640). > > changes: > - added check for null selection model where it was missing > - clear focused row index on setting null selection model in TreeTableView > - clear selected cells on setting a new selection model
Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision: 8187145: added tests ------------- Changes: - all: https://git.openjdk.org/jfx/pull/876/files - new: https://git.openjdk.org/jfx/pull/876/files/5f37fe6d..bf8a0b37 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=876&range=03 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=876&range=02-03 Stats: 236 lines in 1 file changed: 236 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jfx/pull/876.diff Fetch: git fetch https://git.openjdk.org/jfx pull/876/head:pull/876 PR: https://git.openjdk.org/jfx/pull/876
