Modify ↓

Opened 5 years ago

Closed 5 years ago

#18283 closed enhancement (fixed)

[PATCH] JOSM Remote Control handlers made by third parties should be able to register a preference

Reported by: taylor.smock Owned by: team
Priority: normal Milestone: 19.11
Component: Core remotecontrol Version:
Keywords: plugin Cc:

Description

It would be nice for plugins that add a remote control action to be able to show a preference in Remote Control Preferences (to enable/disable it).

Attachments (1)

18283.patch​ (5.8 KB ) - added by taylor.smock 5 years ago.
Don't hardcode PermissionPrefWithDefault values for getPermissionPrefs

Download all attachments as: .zip

Change History (6)

by taylor.smock, 5 years ago

Attachment: 18283.patch​ added

Don't hardcode PermissionPrefWithDefault values for getPermissionPrefs

comment:1 by Don-vip, 5 years ago

Milestone: → 19.11

comment:2 by Don-vip, 5 years ago

Why did you change the order of static initializations? Also, why are the PermissionPrefWithDefault.addPermissionPref in the middle of addRequestHandlerClass calls, and not before or after?

comment:3 by taylor.smock, 5 years ago

I changed the order in order to keep the pre-existing order in the Permitted actions: section of the remote control preferences.

The keeping the pre-existing order of actions doesn't matter, then I can change the order to be more readable.

comment:4 by Don-vip, 5 years ago

OK I get it. Let's keep the order then.

comment:5 by Don-vip, 5 years ago

Resolution: → fixed
Status: new → closed

In 15500/josm:

fix #18283 - Remote controle: don't hardcode PermissionPrefWithDefault values for getPermissionPrefs (patch by taylor.smock, modified)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

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