Modify ↓

Opened 5 years ago

Last modified 5 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 GerdP)

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?

Attachments (0)

Change History (1)

comment:1 by GerdP, 5 years ago

Description: modified (diff)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to GerdP.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.