Modify

Opened 12 years ago

Closed 5 years ago

#7265 closed defect (fixed)

building tool (w) deletes POIs and takes tags

Reported by: dieterdreist Owned by: Upliner
Priority: major Milestone:
Component: Plugin buildings_tools Version: latest
Keywords: building tool poi node polygon, building_tools_replacement_mode Cc:

Description

The building tool (w) started recently to delete POIs (if you draw a new building above it) and incorporates the tags into the building polygon. This is a nice idea and an interesting advanced feature, but it is not at all suited to be the default behaviour (should be with a modifier). Please don't delete points automatically. In my case there was a long building along the street for several housenumbers (with lots of stores in), but currently only one shop (a restaurant) was mapped, so the result was plain wrong.

Attachments (0)

Change History (5)

comment:1 by simon04, 12 years ago

Component: CorePlugin building_tools
Owner: changed from team to Upliner

in reply to:  1 comment:2 by openstreetmap.org-user-d1g, 8 years ago

Replying to simon04:

Yes, this should be optional. I don't think Upliner has time for JOSM/OSM, but if he makes it optional it would be better.

But I don't think this is "major" because everyone was to told to draw "areas" in empty space:

Help/Action/DrawBuilding

And not to draw buildings over "shops" and other "buildings"...

building_tools was used by many users, if users are able to use without damage (I haven't seen reports about "building_tools centric" vandalism).

It is not so mandatory as it seemed at first, what do you think?

PS. You can create much more havoc using "delete" function, right? That's not a reason to remove it or to block inclusion of "building_tools" in core.

But I support that JOSM tools SHOULD BE history preserving (very very please, not mandatory).

Similarly Help/Action/ImproveWayAccuracy: "Please try to use as much existing nodes as you can, because thus you keep their history" - it is not worth our time to implement it in code, we will likely make mistakes in code and have to review them later for little automation functionality.

comment:3 by openstreetmap.org-user-d1g, 8 years ago

Keywords: building_tools_replacement_mode added

comment:4 by oleksii.zagorskyi@…, 5 years ago

As I see, plugin's setting "Use address nodes under buildings" allows to switch the behavior.

comment:5 by anonymous, 5 years ago

Resolution: fixed
Status: newclosed

let me mark it as fixed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Upliner.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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