Modify ↓
#23701 closed enhancement (fixed)
[Patch] Support for revert multiple changesets in Remote Control
| Reported by: | TrickyFoxy | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Plugin reverter | Version: | |
| Keywords: | remote control | Cc: |
Description
The plugin supports the ability to revert multiple changesets, but you can only revert one changeset using Remote Control.
The only question is: should this be a new parameter of the form ids=1,2,3 or can the allowed values of the current id parameter be expanded?
Related code:
https://josm.openstreetmap.de/browser/osm/applications/editors/josm/plugins/reverter/src/reverter/RevertChangesetHandler.java
Attachments (2)
Change History (7)
by , 9 months ago
| Attachment: | Multiple_revert.patch added |
|---|
comment:1 by , 9 months ago
| Keywords: | remote control added |
|---|---|
| Summary: | Support for revert multiple changesets in Remote Control → [Patch] Support for revert multiple changesets in Remote Control |
comment:2 by , 9 months ago
by , 9 months ago
| Attachment: | Support_for_revert_multiple_changesets_in_Remote_Control.patch added |
|---|
comment:3 by , 9 months ago
Thanks for your comment. The code actually looks better with this approach. The idea with ids is inspired by the OSM API :)
Note:
See TracTickets
for help on using tickets.
Splitting the argument in id and ids is a bad design. It's not logical why that should be used and it's not really compatible. Keep "id" and only extend the support by accepting more than one value.