Modify

Opened 5 years ago

Closed 5 years ago

#18521 closed enhancement (fixed)

Update shortcuts parsing script

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

Description

I've simplified the MoveAction logic in r15630. As a side effect the cron script is unable to parse the shortcut creation and raises a warning for this new statement:

Shortcut.registerShortcut("core:move" + getId(), tr("Move objects {0}", getLocalizedName()), getShortcutKey(), Shortcut.SHIFT);}

What needs to be done? Can I do something with javadoc comments, or does the script must be updated? (in this case, can you please do it?)

Attachments (0)

Change History (2)

comment:1 by stoecker, 5 years ago

I added a comment feature like here (trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java#L175), but it seems I was not wise enough to allow providing of all necessary information. I have to update the parsing code to be more flexible in special cases. Hardcoding the specialities seems not a good approach (should have known that from the very beginning, but seems I ignored my own rules once again :-)

comment:2 by stoecker, 5 years ago

Resolution: fixed
Status: newclosed

In 15666/josm:

fix #18521 - allow exceptions for shortcut parser

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 .
 
Note: See TracTickets for help on using tickets.