Modify

Opened 12 years ago

Closed 12 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 12 years ago.
add to /trunk/images
intersect_way.patch (24.8 KB) - added by Upliner 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by Upliner

Attachment: addintersect.png added

add to /trunk/images

Changed 12 years ago by Upliner

Attachment: intersect_way.patch added

comment:1 Changed 12 years ago by bastiK

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 Changed 12 years ago by Upliner

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 Changed 12 years ago by bastiK

In [3636/josm]:

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

comment:4 Changed 12 years ago by bastiK

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 Changed 12 years ago by bastiK

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.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.