Integrated: 8187309: TreeCell must not change tree's data

Jeanette Winzenburg fastegal at openjdk.java.net
Thu Feb 10 10:26:20 UTC 2022


On Wed, 2 Feb 2022 14:18:18 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

> Issue was TreeView commit editing implementation violated the spec'ed mechanism:
> 
> - no default commit handler on TreeView
> - TreeCell modifying the data directly
> 
> Fix is to move the saving of the edited value from cell into a default commit handler in tree and set that handler in the constructor.
> 
> Added tests that failed/passed before/after the fix (along with a sanity test for default commit that passed also before). Also fixed a test bug (incorrect registration of custom commit handler, see [JDK-8280951)](https://bugs.openjdk.java.net/browse/JDK-8280951) in TreeViewTest.test_rt_29650 to keep it passing.

This pull request has now been integrated.

Changeset: 4cf66d9f
Author:    Jeanette Winzenburg <fastegal at openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/4cf66d9f6e96aab29c4c5fc72fa3e81a38a8d783
Stats:     95 lines in 4 files changed: 89 ins; 4 del; 2 mod

8187309: TreeCell must not change tree's data

Reviewed-by: mstrauss, mhanl, aghaisas

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

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


More information about the openjfx-dev mailing list