Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1928 closed defect (fixed)

st_geomfromgml multipoint issue

Reported by: fordfrog Owned by: strk
Priority: medium Milestone: PostGIS 2.0.2
Component: postgis Version: 2.0.x
Keywords: history Cc: petr@…

Description

using postgresql 9.1.4 and postgis 2.0.1 on linux amd64

this query should return gml string but instead returns empty result:

test=# select st_asgml(st_geomfromgml('<gml:MultiPoint xmlns:gml="http://www.opengis.net/gml/3.2" gml:id="DOB.545058.X" srsName="urn:ogc:def:crs:EPSG::2065" srsDimension="2"><gml:pointMembers><gml:Point gml:id="DOB.545058.1"><gml:pos>496547.00 1139895.00</gml:pos></gml:Point></gml:pointMembers></gml:MultiPoint>'));

st_asgml


(1 row)

there is probably general issue with reading the result of st_geomfromgml created from gml:MultiPoint element containing gml:pointMembers. tests with elements with gml:pointMember (singular) within gml:MultiPoint worked fine for me. unfurtunately the data i am processing use gml:pointMembers.

Change History (13)

comment:1 by fordfrog, 12 years ago

i just noticed i am using postgis 2.0.0, gonna try that with 2.0.1 and will report back

comment:2 by fordfrog, 12 years ago

so postgis 2.0.1 is affected too

comment:3 by fordfrog, 12 years ago

when using st_astext(), it returns "MULTIPOINT Z EMPTY". also, looking at the code, there is no string pointMembers, just pointMember, so it seems support for this element is missing.

comment:4 by kashif, 12 years ago

The issue will also occur with MultiCurve and MultiPolygon, where in particular the parser only matches curveMember and polygonMember etc. I can try to fix all these issues if no one is working on it

comment:5 by strk, 12 years ago

I think nobody is working on this. Please don't forget test cases

comment:6 by kashif, 12 years ago

great will send you a patch by the weekend with tests of course :-)

comment:7 by kashif, 12 years ago

Here is the initial patch with all the tests passing:

https://github.com/postgis/postgis/pull/2.diff

Kashif Rasul and Shoaib Burq / SpacialDB

comment:8 by xificurk, 12 years ago

Cc: petr@… added

comment:9 by kashif, 12 years ago

Could you kindly let me know if there is any issue with this fix not getting merged so I can fix it?

Thanks!

comment:10 by strk, 12 years ago

Owner: changed from pramsey to strk
Status: newassigned

Looks fine to me. Will pull right now.

comment:11 by kashif, 12 years ago

great and thank you once again for all your help!

comment:12 by strk, 12 years ago

Resolution: fixed
Status: assignedclosed

So it is in 2.0 branch as r10353 and in trunk as r10354. Thank you for your contribution !

comment:13 by robe, 12 years ago

Keywords: history added
Note: See TracTickets for help on using tickets.