Opened 6 years ago

Closed 6 years ago

#3308 closed defect (fixed)

v.in.ogr GUI generates invalid layer parameter

Reported by: marisn Owned by: martinl
Priority: major Milestone: 7.2.2
Component: wxGUI Version: 7.2.0
Keywords: v.in.ogr Cc:
CPU: Unspecified Platform: Unspecified

Description

When importing a single layer from SQLite database, v.in.ogr GUI generates layer parameter "layer_name|Geometry". Still such value is not a valid layer name for v.in.ogr and thus it is failing with an error: "Layer <layer_name|Geometry> not available". A workaround is to copy actual command and remove the "|Geometry" part and then execute it from a CLI.

The error seems to come from implementing GDAL RFC41 in r63610 thus assigning to the author of the commit.

Change History (7)

comment:1 by marisn, 6 years ago

This problem also affects v.import as it shares the same GUI parts with v.in.ogr.

Looking at r63604 hints that v.import (and its wrapper dialog) should have geometry_column option and pass its value to v.in.ogr geometry parameter. This, of course, is already an enhancement.

comment:2 by marisn, 6 years ago

Version: svn-trunk7.2.0

See also #3333. Changed affected version to 7.2.0 as per #3333

comment:3 by martinl, 6 years ago

In 70892:

v.in.ogr GUI generates invalid layer parameter (see #3308)

comment:4 by martinl, 6 years ago

Should be fixed in r70892. (*) Currently geometry column is returned by gui_core.widget.LayersList.GetLayers() as a part of layer name (which is not ideal). Moreover both GUI wrapper (Import vector data) and G72:v.in.ogr discard this information, what can be problem when OGR layer contains more geometry columns. Moreover G72:v.import lacks geometry option at all. This should be improved in 7.4.0.

(*) After some testing could backported. Please let me know.

comment:5 by martinl, 6 years ago

Milestone: 7.4.07.2.1

comment:6 by martinl, 6 years ago

Milestone: 7.2.17.2.2

comment:7 by martinl, 6 years ago

Resolution: fixed
Status: assignedclosed

In 71280:

v.in.ogr GUI generates invalid layer parameter (merge r70892 from trunk; fix #3308)

Note: See TracTickets for help on using tickets.