Opened 5 years ago

Last modified 5 years ago

#20225 new defect

Method Preferences.getAllPossiblePreferenceDirs() is rather useless — at Initial Version

Reported by: GerdP Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: Cc: Don-vip, stoecker

Description

This is a follow up of ticket:19906#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?

Change History (0)

Note: See TracTickets for help on using tickets.