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.

Sign up for GitHub

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

Jump to bottom

Fix aggregate function returns incorrect results (LTR backport) #59085

Merged
merged 3 commits into from
Oct 16, 2024
Merged

Fix aggregate function returns incorrect results (LTR backport) #59085

merged 3 commits into from
Oct 16, 2024

Commits on Oct 15, 2024

  1. Fix aggregate function returns incorrect results 

    When the aggregate uses variables, we need to ensure that the
cache key correctly considers the current value of ALL those
variables. Otherwise we'll return incorrect results when
an expression is re-evaluated after changing the variable
value for the context.

Fixes use of aggregate function with @symbol_label in legends.

Fixes qgis#58221

(cherry picked from commit 7b88103)
    @nyalldawson
    nyalldawson committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    61b7970 View commit details
    Browse the repository at this point in the history

  2. Include variable name in result 

    (cherry picked from commit f618993)
    @nyalldawson
    nyalldawson committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    0f5796d View commit details
    Browse the repository at this point in the history

  3. Sipify

    @nyalldawson
    nyalldawson committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    863d033 View commit details
    Browse the repository at this point in the history