[PATCH] orthogonalizing tool do not work proper

[andre68@…]

If a closed way has some extra nodes and the way is orthogonalized then undoing fails in at least the attached case. This is the only situation where I've seen this so I can't say if there are other cases.

Simply try the attached case and you will see what I mean.

Stable (r1981) does not have this problem since it refuses to orthogonalize such ways.

[vsandre]

The problem occurs when the start/end-node of the rectangle is not in an edge but in the middle of two egdes.

I add an other case of a strange behaviour of this function. The left node is the start node of the rectangle.

• open the file an select the way, now push q and you get a something what looks like a house. why? undo works ok.
• now move the start/end node in the middle of the two other nodes, push q and the way will be deleted.

[jttt]

bastiK, are you working on this track? I looked into the code and it looks like in r2014 (#3205) the check for bad angles was removed but the edges with bad angles are still there and sometimes produce a way with nonsense values. At least for bug reported by vsandre, not sure about andre68.

Please let me know if you're working on a patch or if I should do that.

[bastiK]

I'm currently at a rework of the orthogonalizing tool - it's almost done, so no need to fix this.

[bastiK]

I suppose the rework won't make it into the next tested.

So if you think it's an easy fix - go ahead!

[anonymous]

Ticket #3657 has been marked as a duplicate of this ticket.

[bastiK]

Remake of the orthogonalize tool.

New:

• No longer required, that the way is closed.

• Multiple ways sharing mulitiple nodes are handled better.

• The two (optional) reference nodes stay fixed.

As a side effect it is now possible to extend a way keeping the exact same direction. (However, it's not very straightforward.)

What it not does: Ignore angles of 45 degrees as requested by the user in #3657. There is a workaround, though: Add an auxilary way on top of the old, covering only the orthogonal parts. Orthogonalize and remove it.

[stoecker]

Why do we need a specific menu point for undoing Ortho?

[bastiK]

What menu point? It shouldn't show up in the menu.

Sometimes the shape contains nodes that are already perfectly well placed.

This could be some imported data or something else. Then it is convenient to take back the orthonalization for certain nodes.

However, I'm not entirely happy with the way this feature is exposed to the user.

The only way to find out about it, is to read the usage dialog (requires, that the user has tried the tool on unsuited selection) or to read the (nonexistent) help in the wiki.

[stoecker]

Well, ok I misread the patch. What do the changes in MainMenu.java do? You changed the type of the action variable and add another variable which is unused. Why?

Regarding the help --> you can add some info at ​http://josm.openstreetmap.de/wiki/Help/Action/Orthogonalize?action=edit

[bastiK]

It's a new JosmAction with it's own shortcut. (Shift-Q, hope this doesn't conflict with plugins.)

I thought this is the place, where all Actions are initialized, so I put it there.

It can be placed in the constructor of OrthogonalizeAction as well, but then you have a "two in one" action.
Don't know, what's less confusing...

I see this would have needed some commenting. :)

[stoecker]

Hmm, does not apply. Please update and comment the lines in MainMenu.

[bastiK]

Ok.

[stoecker]

(In [2268]) fix #3571 - patch by bastiK - improve orthogonalization action