Opened 9 years ago

Closed 3 years ago

#3232 closed enhancement (fixed)

ST_MoveIsoNode allows corrupting topology by not checking containing face

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

Description

There's no check in ST_MoveIsoNode for maintaining the "containing_face" field of the node. I think it should just forbid to move the isolated node from one face to another, to be consistent with other editing functions requiring that the underlying topology does not change.

Change History (7)

comment:1 by strk, 9 years ago

Milestone: PostGIS 2.0.8 → PostGIS Future
Version: trunk → 2.0.x

comment:2 by robe, 7 years ago

Milestone: PostGIS Future → PostGIS Fund Me

Milestone renamed

comment:3 by strk, 7 years ago

Resolution: → fixed
Status: new → closed

In 16043:

Do not allow empty points as topology isolated nodes

Includes testcase
Fixes #3232

comment:4 by strk, 7 years ago

Sorry, wrong reference: r16043 was a fix for #3234, not this ticket.

comment:5 by strk, 7 years ago

Resolution: fixed
Status: closed → reopened

comment:6 by strk, 3 years ago

Milestone: PostGIS Fund Me → PostGIS 3.2.0
Type: defect → enhancement

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

Resolution: → fixed
Status: reopened → closed

In ea18ecdd/git:

Prevent moving an isolated node to different face

Closes #3232 in main branch (3.2.0dev)

Includes regression test, moving existing
tests from sqlmm to ad-hoc file

Note: See TracTickets for help on using tickets.