[Closed] RFR: 8193286: IntegerSpinnerFactory does not wrap value correctly

[Ajit Ghaisas]

On Mon, 13 Apr 2020 06:59:08 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

> Issue : https://bugs.openjdk.java.net/browse/JDK-8193286
> 
> Root Cause :
> Incorrect implementation.
> Current implementation of int wrapValue(int,int,int) in Spinner.java works well if min is 0.
> Hence this implementation works with ListSpinnerValueFactory, but fails with IntegerSpinnerValueFactory.
> 
> Fix :
> Added a method for IntegerSpinnerValueFactory with separate implementation.
> 
> Testing :
> Added unit tests to test increment/decrement in multiple steps and with different amountToStepBy values.
> 
> Also, identified a related behavioral issue https://bugs.openjdk.java.net/browse/JDK-8242553, which will be addressed
> separately.

This pull request has been closed without being integrated.

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

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