Modify

Opened 14 years ago

Closed 14 years ago

#5577 closed enhancement (fixed)

[PATCH] Add nodes on intersections action

Reported by: Upliner Owned by: team
Priority: major Milestone:
Component: Core Version:
Keywords: Cc:

Description

There was the Intersect_way plugin that allowed to add nodes on intersections of ways. Unfortunately, author haven't published the source code and the plugin became unmaintatined and unusable. Since then similar functionality was implemented in "Join areas" action. I propose to extract addIntersections function to new action and share it with "Join areas". It's better than copy-pasting this function to new plugin.

Attachments (2)

addintersect.png (420 bytes ) - added by Upliner 14 years ago.
add to /trunk/images
intersect_way.patch (24.8 KB ) - added by Upliner 14 years ago.

Download all attachments as: .zip

Change History (7)

by Upliner, 14 years ago

Attachment: addintersect.png added

add to /trunk/images

by Upliner, 14 years ago

Attachment: intersect_way.patch added

comment:1 by bastiK, 14 years ago

Isn't it easier to double click on the intersection point? Maybe it's not so well known, but it places the node at the exact point of intersection (because of snap effect).

comment:2 by Upliner, 14 years ago

Oh, I didn't know about this feature.
However, when drawing a grid of streets, the new streets creates a lot of intersections with existing streets and with each other. It's harder to point at them all. If you don't want to integrate this function in the core can you at least make addIntersections method public and static? It can also be used in the validator plugin making able to automatically fix the "crossing ways" error.

comment:3 by bastiK, 14 years ago

In [3636/josm]:

see #5577 (patch by extropy) - Move some methods and make them public static

comment:4 by bastiK, 14 years ago

Sure, i'm not saying it is totally useless, the method I described is not really documented after all...

In case you add a plugin, please call it 'toolsplugin' or something like that, so we can add other more specialized tools and helpers.

comment:5 by bastiK, 14 years ago

Resolution: fixed
Status: newclosed

Added it to the new UtilPlugin2. (In [o24236])

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.