[Patch] Annoying warning about deleting nodes with id:0 outside BBox

[skyper]

I get warnings about deleting nodes with id:0 outside BBox.

There should be no warning as you can not delete any incomplete object by deleting a node with id:0

r5033

[simon04]

Please attach a test file, I cannot reproduce (1. download some area, 2. add a node outside the area, 3. delete this node).

[skyper]

Replying to simon04:

Please attach a test file, I cannot reproduce (1. download some area, 2. add a node outside the area, 3. delete this node).

Sorry not much time right now:

Please, try it with a way id:0.

Thanks

[skyper]

Replying to skyper:

Replying to simon04:

Please attach a test file, I cannot reproduce (1. download some area, 2. add a node outside the area, 3. delete this node).

Sorry not much time right now:

I had five minutes !

Sorry my description is incomplete.

1. download an closed way with at least one node outside BBox
2. use terracer to terrace keeping the original way
3. delete the new created closed way with the node outside BBox

It has to due with ways. I gonna attach an example.

[skyper]

sample

[simon04]

The attached patch should take care of this issue. Patch explained in words: As the delete operation already takes care of including/excluding way-nodes from deletion, there is no need to check ways in the outlying check.

To be on the safe side, I'd await the end of stabilization phase although I do not expect any problems with the patch.

[simon04]

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

[simon04]

In 5060/josm:

fix #7470 - tune warning of nodes out of downloaded bbox