Opened 11 years ago

Closed 7 years ago

Last modified 7 years ago

#2411 closed defect (fixed)

ST_Reverse returns curves unmodified

Reported by: strk Owned by: strk
Priority: blocker Milestone: PostGIS 2.4.0
Component: postgis Version: 1.5.X
Keywords: Cc:

Description

Passing a CIRCULARSTRING to ST_Reverse returns the input untouched. It should either raise an exception or do the reversal, IMHO.

Affected versions from 1.5.9 to 2.2.0dev

Change History (9)

comment:1 by robe, 11 years ago

Milestone: PostGIS 2.0.4 → PostGIS 2.2.0

comment:2 by strk, 10 years ago

Milestone: PostGIS 2.2.0 → PostGIS Future

comment:3 by strk, 7 years ago

Milestone: PostGIS Future → PostGIS 2.4.0

Up to 2.3 affected still..

comment:4 by dbaston, 7 years ago

Priority: medium → blocker

comment:5 by dbaston, 7 years ago

Owner: changed from strk to pramsey

comment:6 by strk, 7 years ago

Owner: changed from pramsey to strk
Status: new → assigned

I've patch ready, with tests. Regina do you think it should go in manual that behavior changed/was enhanced in 2.4.0 ? Or should I rather backport to all supported branches when it makes sense ?

comment:7 by strk, 7 years ago

I opted for documenting it as an enhancement in 2.4.0

comment:8 by strk, 7 years ago

Resolution: → fixed
Status: assigned → closed

In 15566:

Add support for curves in lwgeom_reverse/ST_Reverse

Includes tests
Closes #2411

comment:9 by robe, 7 years ago

Looks like you already did it, but I agree that this should go in 2.4.0 and not backported to prior branches.

Because it's annoying as an app developer to have to worry about micros. e.g. if anyone uses curves they'd have to say supported in 2.3.4+, 2.2.whataver, 2.4.0

easier for them to just say 2.4 and be done with it. No having to micromanage :)

Note: See TracTickets for help on using tickets.