Bug report #19808
[Processing] Description not in line with "input layer" parameter capabilities
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 6 years ago
The Dissolve algorithm also allows point layers in input
#2 Updated by Giovanni Manghi about 6 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 6 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 6 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 6 years ago
- Resolution set to fixed/implemented
- Status changed from Open to Closed
Seems already fixed in master