Bug report #19808

[Processing] Description not in line with "input layer" parameter capabilities

Added by Harrissou Santanna about 2 years ago. Updated almost 2 years ago.

Status:Closed
Priority:Normal
Assignee:-
Category:Processing/Core
Affected QGIS version:3.3(master) Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:27632

Description

Some algorithms are described as accepting line or polygon features but the input layer drop-down still lists point layer (I don't know if point layers should be excluded or the description may need some review):
Extract vertices / extract specific vertices

Moreover, there's some kind of conflict between "vertices" and "points" so I don't know how desirable/expected is the point geometry presence in "Remove duplicate vertices".

History

#1 Updated by Harrissou Santanna about 2 years ago

The Dissolve algorithm also allows point layers in input

#2 Updated by Giovanni Manghi about 2 years ago

Harrissou Santanna wrote:

The Dissolve algorithm also allows point layers in input

dissolving points is ok as dissolving lines or polygons (to get multi-point features).

#3 Updated by Harrissou Santanna about 2 years ago

I meant conflict between description and input: I don't think anyone will expect it works with point given the following.

This algorithm takes a polygon or line vector layer and combines their geometries into new geometries.One or more attributes can be specified to dissolve only geometries belonging to the same class (having the same value for the specified attributes), alternatively all geometries can be dissolved.
All output geometries will be converted to multi geometries. In case the input is a polygon layer, common boundaries of adjacent polygons being dissolved will get erased.

#4 Updated by Giovanni Manghi about 2 years ago

Harrissou Santanna wrote:

I meant conflict between description and input: I don't think anyone will expect it works with point given the following.

ahhhh got it.

#5 Updated by Alexander Bruy almost 2 years ago

  • Resolution set to fixed/implemented
  • Status changed from Open to Closed

Seems already fixed in master

Also available in: Atom PDF