Bug report #16301

OGR Buffer fails in Processing when datasource is PostGIS

Added by Paolo Cavallini over 3 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assignee:-
Category:Processing/OGR
Affected QGIS version:2.18.4 Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:invalid
Crashes QGIS or corrupts data:No Copied to github as #:24211

History

#1 Updated by Giovanni Manghi over 3 years ago

  • Status changed from Open to Closed
  • Resolution set to invalid

The user must specify the geometry column name.

#2 Updated by Paolo Cavallini over 3 years ago

The default should work, and the user should be warned what to type.
As it is, the tool is unusable by most.

#3 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

The default should work, and the user should be warned what to type.
As it is, the tool is unusable by most.

there is already a feature request for improvements. As it is it works, just select the proper geometry column name (that by the way is ok by default for most of formats, as shapefiles). If you want to geoprocess postgis layers within postgis with processing there is a proper plugin for that.

#4 Updated by Paolo Cavallini over 3 years ago

Probably adding a short help and a note to the param would help.
Thanks.

#5 Updated by Paolo Cavallini over 3 years ago

Anyway, IMHO this is still a bug: every command should work with the default values.

#6 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

Anyway, IMHO this is still a bug: every command should work with the default values.

feel free to submit the patch to automatically get the geom column name of inputs (not yet implemented in Processing afaik).

#7 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

Probably adding a short help and a note to the param would help.

sure

#8 Updated by Paolo Cavallini over 3 years ago

  • Status changed from Closed to Reopened
  • Resolution deleted (invalid)

#9 Updated by Giovanni Manghi over 3 years ago

  • Status changed from Reopened to Closed
  • Resolution set to invalid

#10 Updated by Paolo Cavallini over 3 years ago

As said, this requires a patch, as for the user it does not work: what is the purpose of closing it?

#11 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

As said, this requires a patch, as for the user it does not work: what is the purpose of closing it?

as it is it works just fine, what you propose is an improvement.

#12 Updated by Paolo Cavallini over 3 years ago

Sorry to insist: it works for many data sources, not for PostGIS, without user warnings.

#13 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

Sorry to insist: it works for many data sources, not for PostGIS, without user warnings.

the users just need to change "geometry" in "geom" or whataver is the geometry column name.
The parameter title is very clear

Geometry column name ("geometry" for shapefiles, maybe different for other formats)

is that not clear enough?!

#14 Updated by Paolo Cavallini over 3 years ago

You are right, thanks. Perhaps in the help we could add other common names for other formats (geom for PostGIS, other?).

#15 Updated by Giovanni Manghi over 3 years ago

Paolo Cavallini wrote:

You are right, thanks. Perhaps in the help we could add other common names for other formats (geom for PostGIS, other?).

there is a plan to tentatively add something to get automatically the geom column name.

Also available in: Atom PDF