Skip to content

[Backport queued_ltr_backports] return 0 for count-based attributes in Join attributes by location (summary) Processing algorithm (fix #40108)#66448

Open
alexbruy wants to merge 3 commits into
qgis:queued_ltr_backportsfrom
alexbruy:backport-66322-to-queued_ltr_backports
Open

[Backport queued_ltr_backports] return 0 for count-based attributes in Join attributes by location (summary) Processing algorithm (fix #40108)#66448
alexbruy wants to merge 3 commits into
qgis:queued_ltr_backportsfrom
alexbruy:backport-66322-to-queued_ltr_backports

Conversation

@alexbruy

Copy link
Copy Markdown
Contributor

Description

Manual backport of #66322.

In the Join attributes by location (summary) Processing algorithm non-matching features (features that can not be joined) have NULL for their summaries. While this makes sense for aggregates that depend on field values like min, max, sum, this is wrong for count-based values like count, unique, filled which should use 0.

This happens because for non-matching features the code just resizes the attributes filling all additional attributes with NULLs. Instead we should fill them manually using 0 where this make sense.

Fixes #40108.

AI tool usage

  • AI tool(s) (Copilot, Claude, or something similar) supported my development of this PR. See our policy about AI tool use. Use of AI tools must be indicated. Failure to be honest might result in banning.

@github-actions github-actions Bot added the Processing Relating to QGIS Processing framework or individual Processing algorithms label Jun 11, 2026
@github-actions github-actions Bot added this to the 3.44.13 milestone Jun 11, 2026
@alexbruy alexbruy changed the title Backport 66322 to queued ltr backports [Backport queued_ltr_backports] return 0 for count-based attributes in Join attributes by location (summary) Processing algorithm (fix #40108) Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Processing Relating to QGIS Processing framework or individual Processing algorithms

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant