Opened 12 years ago

Last modified 6 years ago

#7468 new defect

Do not delete address nodes but use them as corner node

Reported by: skyper Owned by: Upliner
Priority: normal Milestone:
Component: Plugin buildings_tools Version:
Keywords: address node, building_tools_replacement_mode Cc: jhfi, brogo


I already reported this on #7328.


Replying to skyper:

By the way: BT should not delete address nodes but use the node as one corner node with its properties moved to the way.

I agree with you there. As I said, there is room for improvement.

I mark this as defect as you loose the history of the address tags right now.

Attachments (0)

Change History (9)

comment:1 by joshdoe, 12 years ago

The code for this is implemented in Utilsplugin2, in ReplaceGeometryUtils.upgradeNode(Node subject, OsmPrimitive reference). Perhaps it should depend on utilsplugin2, or if building_tools is to be integrated in core, we can consider moving this class from the plugin to core as well.

comment:2 by skyper, 12 years ago

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

comment:3 by stephan75, 9 years ago

Is this an alternative?

Have utils_plugin2 and building_tools plugin installed,

have a single node from OSM database with any addr:xxx data

then draw a building outline via building_tool around this "old" addr: node, but be sure via buildingtools setting that this node is NOT deleted,

Then select that node and also select the new building outline (that consists of four new nodes,

choose via "moore tools" menu the feature of "Replace Geometry" and transfer all tags from the node to the outline,

after that click each of the four nodes of that outline, and see
that one of them is now the OLD one node with the node ID number from OSM database.

Thus we can see that transferring that old node to the building outline is possible!!!!

So we need a transfer from the feature "Replace Geometry" to building_tool, I think.

Is this possible with little effort?


comment:4 by mdk, 9 years ago

First of all there are three possible candidates for the old history:

  1. The building outline. But as far as I know we have no operations to “merge“ the history of two objects or replace the history of the way with the one of a node.
  2. Add existing address node to the building outline (as an entrance). But where to add the node (automatically)?
  3. Replace an existing outline node with old address node, as suggested in this issue.

When I start thinking about the third solution, I found much more arguments against this approach than supporting ones:

  1. We have to choose an arbitrary victim node to be replaced. This node will loose his history instead. What is the criteria for choosing the node to be deleted?
  2. Assuming we have a rectangular building, I would never expect the address history at a corner node. Not finding the history is in my opinion the same as loosing the history.
  3. Assuming some simple terraced buildings. The inner buildings has only nodes which are also part of other buildings. To which building belongs the history, if the replaced node is part of two three or even four buildings?
  4. If the replaced node is member of more than one way (other buildings, fences, parking places, …), the node must be replaced in all ways. This cause (unnecessary) modifications on all the involved ways.
  5. Again think about terraced buildings. How do we want to prevent replacing a node which belongs to more than one building more than once? In this case we will loose an old address history anyway.

Edit: This is more suitable for #6285. For this issue (creating a new building), 1.) is not relevent.

Last edited 9 years ago by mdk (previous) (diff)

comment:5 by, 8 years ago

Keywords: building_tools_replacement_mode added

comment:6 by Klumbumbus, 6 years ago

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

comment:7 by Klumbumbus, 6 years ago

Cc: jhfi added

comment:8 by Klumbumbus, 6 years ago

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

comment:9 by Klumbumbus, 6 years ago

Cc: brogo added

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain Upliner.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from Upliner to the specified user.
Next status will be 'needinfo'. The owner will be changed from Upliner to skyper.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from Upliner to anonymous. Next status will be 'assigned'.

Add Comment

E-mail address and name can be saved in the Preferences .
Note: See TracTickets for help on using tickets.