Opened 4 years ago
Last modified 4 years ago
#20225 new defect
Method Preferences.getAllPossiblePreferenceDirs() is rather useless
Reported by: | GerdP | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | |
Keywords: | Cc: | Don-vip, stoecker |
Description (last modified by )
This is a follow up of ticket:19006#commen:14
I think the changes in r7834 introduced an unintended change in method Preferences.getAllPossiblePreferenceDirs()
Before, this method returned a list with a special order, now the collection is an unordered HashSet. The method is used by ImageProvider
as a "last resort" to find images and I think this requires an order to prevent unpredictable behaviour.
Another problem is that some of the directory names returned by this method have a separator charactor at the end, others haven't.
I am not sure why this method still exists. It seems to implement some backward compatibility for very early JOSM versions?