Modify

Opened 3 years ago

Closed 3 years ago

#5577 closed enhancement (fixed)

[PATCH] Add nodes on intersections action

Reported by: Upliner Owned by: team
Priority: major 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 3 years ago.
add to /trunk/images
intersect_way.patch (24.8 KB) - added by Upliner 3 years ago.

Download all attachments as: .zip

Change History (7)

Changed 3 years ago by Upliner

add to /trunk/images

Changed 3 years ago by Upliner

comment:1 Changed 3 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 3 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 3 years ago by bastiK

In [3636/josm]:

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

comment:4 Changed 3 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 3 years ago by bastiK

  • Resolution set to fixed
  • Status changed from new to closed

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

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.