Modify

Opened 16 months ago

Closed 16 months ago

Last modified 14 months ago

#19924 closed defect (fixed)

Extruder: Problem with common node extruding to the inside.

Reported by: skyper Owned by: GerdP
Priority: normal Milestone: 20.11
Component: Core Version: latest
Keywords: template_report regression extruder Cc:

Description

What steps will reproduce the problem?

  1. Create a new way with two nodes
  2. Change to extruder mode and create an rectangle area.
  3. Create another connected rectangle area pressing Alt in extruder mode
  4. Try to separate the two areas by extruding the common segment to the inner side of one area

What is the expected result?

Two separate ways without a common node

What happens instead?

One former corner node is not released and still part of both ways.

Please provide any additional information below. Attach a screenshot if possible.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-10-11 23:10:16 +0200 (Sun, 11 Oct 2020)
Revision:17170
Build-Date:2020-10-12 01:30:50
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (1)

extrude.PNG (5.4 KB) - added by GerdP 16 months ago.

Download all attachments as: .zip

Change History (8)

Changed 16 months ago by GerdP

Attachment: extrude.PNG added

comment:1 Changed 16 months ago by GerdP

Milestone: 20.10
Owner: changed from team to GerdP
Status: newassigned

Regression of r17141.

comment:2 Changed 16 months ago by GerdP

Resolution: fixed
Status: assignedclosed

In 17183/josm:

fix #19924: Extruder: Problem with common node extruding to the inside.

  • revert r17141
  • use wNew.getNodes() to create ChangeNodesCommand

Stupid me! Methods Way.addNode() and Way.removeNode() cannot simply be replaced by List.add() or List.remove(). Results can differ.

Last edited 16 months ago by GerdP (previous) (diff)

comment:3 Changed 16 months ago by GerdP

Lesson learned: Found the same error in DataSet.unlinkNodeFromWays(Node node) and wondered why nobody noticed it. Turned out that it is one possible reason for tickets with DataIntegrityProblemException: Deleted node referenced
Yippi!

comment:4 Changed 16 months ago by stoecker

So making errors is not always bad. Congratulations!

comment:5 in reply to:  3 Changed 16 months ago by simon04

Replying to GerdP:

Turned out that it is one possible reason for tickets with DataIntegrityProblemException: Deleted node referenced

Yay!

comment:6 in reply to:  4 Changed 16 months ago by skyper

Replying to stoecker:

So making errors is not always bad. Congratulations!

+1

Nice, thanks for the fast fix.

comment:7 Changed 14 months ago by Don-vip

Milestone: 20.1020.11

Milestone renamed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain GerdP.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.