[gdal-dev] GML / NAS code redundancy

[Martin Landa]

Dear Even,

2014-04-05 23:50 GMT+02:00 Even Rouault <even.rouault at mines-paris.org>:

> Done. See http://trac.osgeo.org/gdal/changeset/27132

it's working perfectly!

I found out a small inconsistency when converting data to PostGIS. For
layers which have multiple geometries, column name refers to 'name'
tag - which is something what I would expect. Unfortunately for layers
with single geometry, column name refers to default 'wkb_geometry'.
It's cased by the fact that for these layers geometry column is added
here [1]. For layers with multiple geometry it's added afterwards [2].

For now I used  GEOMETRY_NAME variable in my scripts, but it would be
probably useful to fix that directly in PG driver, what do you think?
Thanks, Martin

[1] http://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/pg/ogrpgdatasource.cpp#L1492
[2] http://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/pg/ogrpgtablelayer.cpp#L2386