[GRASS-dev] [GRASS GIS] #2740: v.in.ogr: wrong projection check when importing geometry column from table with multiple geometry columns

GRASS GIS GRASS GIS developer
,
1

#2740: v.in.ogr: wrong projection check when importing geometry column from table
with multiple geometry columns
---------------------------------------+-------------------------
Reporter: mlennert | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.0.2
Component: Vector | Version: svn-trunk
Keywords: v.in.ogr geometry columns | CPU: Unspecified
Platform: Unspecified |
---------------------------------------+-------------------------
r63604 introduced the possibility to import from data sources with
multiple geometry columns (gdal RFC41).

However, AFAIU, the projection check has not been changed and so the
projection is gotten from the layer, not the field (aka geometry column).

It seems to me that the gathering of information concerning the geometry
column in
[http://trac.osgeo.org/grass/browser/grass/trunk/vector/v.in.ogr/main.c#L870
lines 870ff of main.c] needs to be moved higher up to before getting the
projection
([http://trac.osgeo.org/grass/browser/grass/trunk/vector/v.in.ogr/main.c#L543
lines 543ff]), so that the projection can be gotten from the '''geometry
column''' and not from the layer.

But I'm not 100% sure of understanding the code correctly, so this needs
confirmation.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/2740&gt;
GRASS GIS <https://grass.osgeo.org>

2

#2740: v.in.ogr: wrong projection check when importing geometry column from table
with multiple geometry columns
--------------------------+---------------------------------------
  Reporter: mlennert | Owner: martinl
      Type: defect | Status: assigned
  Priority: normal | Milestone: 7.0.2
Component: Vector | Version: svn-trunk
Resolution: | Keywords: v.in.ogr geometry columns
       CPU: Unspecified | Platform: Unspecified
--------------------------+---------------------------------------
Changes (by martinl):

* owner: grass-dev@… => martinl
* cc: grass-dev@… (added)
* status: new => assigned

Comment:

Please try out r66184. After some testing could be backport to relbr70.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2740#comment:1&gt;
GRASS GIS <https://grass.osgeo.org>

3

#2740: v.in.ogr: wrong projection check when importing geometry column from table
with multiple geometry columns
--------------------------+---------------------------------------
  Reporter: mlennert | Owner: martinl
      Type: defect | Status: assigned
  Priority: normal | Milestone: 7.0.2
Component: Vector | Version: svn-trunk
Resolution: | Keywords: v.in.ogr geometry columns
       CPU: Unspecified | Platform: Unspecified
--------------------------+---------------------------------------

Comment (by mlennert):

Replying to [comment:1 martinl]:
> Please try out r66184. After some testing could be backport to relbr70.

Works nicely for my case.

Thanks !

Moritz

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2740#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

4

#2740: v.in.ogr: wrong projection check when importing geometry column from table
with multiple geometry columns
--------------------------+---------------------------------------
  Reporter: mlennert | Owner: martinl
      Type: defect | Status: closed
  Priority: normal | Milestone: 7.0.2
Component: Vector | Version: svn-trunk
Resolution: fixed | Keywords: v.in.ogr geometry columns
       CPU: Unspecified | Platform: Unspecified
--------------------------+---------------------------------------
Changes (by martinl):

* status: assigned => closed
* resolution: => fixed

Comment:

Replying to [comment:2 mlennert]:
> Replying to [comment:1 martinl]:
> > Please try out r66184. After some testing could be backport to
relbr70.
>
> Works nicely for my case.

perfect, backported in r66187. Closing.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2740#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>