ST_Shift_Longitude doesn't work with MULTIPOINT

[robe]

What steps will reproduce the problem?

1. SELECT ST_Shift_Longitude(ST_GeomFromText('MULTIPOINT(1 3, 4 5)'))

Gives error

ERROR: lwgeom.c:723: unknown geom type: 4

Error

ERROR: lwgeom.c:723: unknown geom type: 4 SQL state: XX000

—However this works fine - isn't MULTIPOLYGON more complex SELECT ST_Astext(ST_Shift_Longitude(ST_GeomFromText('MULTIPOLYGON(((1 3,4 5,3 4,3 4.5,1 3)))')))

Also doesn't work with Circular string - gives similarly cryptic message

SELECT ST_Shift_Longitude(ST_GeomFromText('CIRCULARSTRING(1 3, 4 5, 6 7)'))

ERROR: lwgeom.c:723: unknown geom type: 8

I just tried on newest 1.3.4SVN snapshot and 1.3.3. Gives same error on both.

[mcayland]

Should now be fixed in 1.3 branch/trunk. Note I've taken the line of returning a friendlier error message rather than looking at supporting CURVETYPE; perhaps this part is something to leave for 1.4?

ATB,

Mark.

[robe]

Yah you can leave for 1.4. I don't think its a commonly used function and MULTIPOINT is not a common geometry type anyway. I just noticed the error when I was coming up with examples in the docs. I had put in MULTIPOINT is not supported for this one (well for 1.4 docs). I'll just amend to say fixed in 1.4

[robe]

Ah Mark - sorry I misread your note. Yes even better. Now I can take out the comment about not working for MULTIPOINT in the docs (I'll just say pre 1.3.4 did not work).

[robe]

Thanks just confirmed it works with MULTIPOINT and GEOMETRYCOLLECTION containing multipoint in 1.3.4 SVN. Hmm I still need to confirm for 1.4 but don't have my system setup for that yet, but I'm going to close this out anyway.