Bug report #15097: Issues when running "Import into PostGIS" in batch mode - QGIS Application - QGIS Issue Tracking

Bug report #15097

Issues when running "Import into PostGIS" in batch mode

Added by Giovanni Manghi over 8 years ago. Updated about 8 years ago.

Status:	Closed
Priority:	Severe/Regression
Assignee:	Victor Olaya
Category:	Processing/QGIS
Affected QGIS version:	master	Regression?:	No
Operating System:		Easy fix?:	No
Pull Request or Patch supplied:	No	Resolution:
Crashes QGIS or corrupts data:	No	Copied to github as #:	23039

Description

when selecting layers from filesystem and the table name is left blank, to leave the tool use the input vector as name, then the absolute path of the input vector is used as table name
"table to import parameter" is filled with "none" when the tool is opened, and there is the suggestion to leave it blank to use the layer name. But then if the user leave "none" this word is used as table name.
"none" is used also as default for the "primary key field", if the user leave it then "none" is used as name for the PK column
compared to the tool in DB Manager it seems to miss the source/destination CRS parameters

15097.mp4 - screencast (2.6 MB) R. R., 2016-08-23 11:10 AM

Associated revisions

Revision e58177cf
Added by Victor Olaya about 8 years ago

[processing] fixes for 'import into PostGIS' alg

fixes #15097

Revision e55d1923
Added by Victor Olaya about 8 years ago

[processing] fixes for 'import into PostGIS' alg

fixes #15097

History

#1 Updated by Giovanni Manghi over 8 years ago

Priority changed from Normal to Severe/Regression
Target version changed from Future Release - High Priority to Version 2.16

the "none" issues are a regression since 2.8.*

#2 Updated by R. R. about 8 years ago

File 15097.mp4 added

To me, this is a really important issue. I've uploaded a screencast for documentation purposes.

#3 Updated by Victor Olaya about 8 years ago

These should be fixed already.

The only thing not fixed is your last suggestion about the dest crs, since that is a feature request. Also it would change the syntax of the algorithm, so it cannot be added to this next release. We should wait until 3.0

#4 Updated by Giovanni Manghi about 8 years ago

Hi Victor, thanks for the fixes. I see still two issues:

1) when the table name is left blank, once imported the table in Postgis has also the ".shp" in its name. I think is necessary to strip the extensions from inputs names.

2) if the field for primary key is left blank the tool gives an error

Error importing to PostGIS Creation of data source "public"."aaa" failed: ERROR: syntax error at or near "," LINE 1: CREATE TABLE "public"."aaa"(, PRIMARY KEY () ) ^

I would suggest to fill this field out of the box with a default like "gid" or just "id".

#5 Updated by Victor Olaya about 8 years ago

Status changed from Open to Closed

Fixed in changeset e58177cf9e360021618d6029821cbc8416496cff.

#6 Updated by Giovanni Manghi about 8 years ago

Status changed from Closed to Reopened

the latest build of master on osgeo4w is 979d619 that does not contain the latest commits, among the others the latest patch for this ticket. So I applied the patch manually but at that point the tool returns "Error importing to PostGIS Creation of data source "schema_name"."" failed: ERROR: zero-length delimited identifier at or near """" LINE 1: CREATE TABLE "schema_name".""(gid serial, PRIMARY KEY (gid) ) ^" but I soon realized that this error is also shown without the patch. At this point from "master_2" I downloaded the whole Processing folder and placed in plugins folder in .qgis2: no more error but also "missing parameter value: Table to import..." when leaving the table name blank. The "primary key field" seems fixed.