Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[backport] don't try to dissolve buffer results if there are no features (fix #46396) #47384

Merged
merged 2 commits into from
Feb 17, 2022
Merged

[backport] don't try to dissolve buffer results if there are no features (fix #46396) #47384

merged 2 commits into from
Feb 17, 2022

Conversation

alexbruy
Copy link
Contributor

Description

Buffering an empty layer with native "Buffer" algorithm produces empty output layer, while the same operation with enabled "Dissolve result" option ends with error message

Feature could not be written to Buffered_a03cb901_262a_4ed5_b22c_23aeb10000b1: Could not add feature with geometry type GeometryCollection to layer of type MultiPolygon
Could not write feature into OUTPUT
Execution failed after 0.01 seconds

This creates problems when algorithm is used as part of a model. For consistence it is better to produce empty layer in both cases. Fixes #46396.

Manual backport of #47348.

@alexbruy alexbruy added Backport Is a backport of another pull request Processing Relating to QGIS Processing framework or individual Processing algorithms labels Feb 16, 2022
@github-actions github-actions bot added this to the 3.24.0 milestone Feb 16, 2022
@nyalldawson nyalldawson merged commit 0e4bd99 into qgis:queued_ltr_backports Feb 17, 2022
@alexbruy alexbruy deleted the buffer-dissolve-backport branch February 18, 2022 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport Is a backport of another pull request Processing Relating to QGIS Processing framework or individual Processing algorithms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants