IndexedPath timestamp fall back in case of an exception does not work well

Oliver-Loeffler / FXFileChooser

Custom JavaFX file chooser which allows quick manual filtering, which allows to add Path predicates as filter and which is testable using TestFX.

Apache License 2.0

44 stars 8 forks source link

IndexedPath timestamp fall back in case of an exception does not work well #35

Closed Oliver-Loeffler closed 4 years ago

Oliver-Loeffler commented 4 years ago

Sorting of paths by using IndexedPath timestamp does not work as expected when the fallback timestamp (instead of last modified timestamp) is used.

Behavior is unexpected as "old" elements sometimes appear inbetween "new" path elements.

TODO: Review fallback and compare strategy.

Oliver-Loeffler commented 4 years ago

Closed with 46d9a8e5b9b894d9fff8df19a5bde665aee3019c. There was also a bug in FileChooserController where the wrong comparator was used, when descending ordering was requested. See 86fae71dfc81ce2b9a3eac852dfd1dbad0ae391f.