Opened 9 years ago

Closed 3 years ago

Last modified 3 years ago

#3239 closed enhancement (fixed)

ST_{Mod,New}EdgeHeal does not check for TopoGeometry defined by removed node

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 3.2.0
Component: topology Version: 2.1.x
Keywords: Cc:

Description (last modified by strk)

Reading the code I don't think we're checking for the node being removed by edge healing operations to be used in the definition of a TopoGeometry. We should !

See also #3248 and #3231

Change History (16)

comment:1 by robe, 9 years ago

Why do we need a 2.2.1? We aren't done with 2.2.0 yet. This sounds like a bug to me that should be fixed in 2.2.0 before release no?

comment:2 by pramsey, 7 years ago

Milestone: PostGIS 2.1.9PostGIS 2.2.6

comment:3 by pramsey, 7 years ago

Milestone: PostGIS 2.2.6PostGIS 2.2.7

comment:4 by robe, 6 years ago

Milestone: PostGIS 2.2.7PostGIS 2.2.8

comment:5 by Algunenano, 5 years ago

Milestone: PostGIS 2.2.8PostGIS 2.3.10

Moving to a non EOL release.

comment:6 by robe, 5 years ago

Milestone: PostGIS 2.3.10PostGIS 2.5.4

comment:7 by pramsey, 4 years ago

Milestone: PostGIS 2.5.4PostGIS 2.5.5

comment:8 by robe, 4 years ago

Milestone: PostGIS 2.5.5PostGIS 3.0.3

comment:9 by robe, 3 years ago

Milestone: PostGIS 3.0.3PostGIS 3.1.0

comment:10 by strk, 3 years ago

See also #3231 (isolated nodes are also not protected from removal)

comment:11 by robe, 3 years ago

Milestone: PostGIS 3.1.0PostGIS 3.0.4

comment:12 by strk, 3 years ago

Description: modified (diff)

comment:14 by strk, 3 years ago

Milestone: PostGIS 3.0.4PostGIS 3.2.0
Type: defectenhancement

comment:15 by Sandro Santilli <strk@…>, 3 years ago

Resolution: fixed
Status: newclosed

In 1ea1cb6/git:

Prevent healing edges if connecting node is used in a TopoGeometry

Closes #3239 in main branch (3.2.0dev)

comment:16 by Sandro Santilli <strk@…>, 3 years ago

In 168f3f4/git:

Prevent removing isolated node used in TopoGeometry definition

Closes #3239 in main branch (3.2.0dev)

Note: See TracTickets for help on using tickets.