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

[persistence] Handle null value for unit field of filters #3716

Merged
merged 1 commit into from
Jul 20, 2023
Merged

[persistence] Handle null value for unit field of filters #3716

merged 1 commit into from
Jul 20, 2023

Commits on Jul 20, 2023

  1. [persistence] Handle null value for unit field of filters 

    Setting up a treshold filter with the UI, it did not work because the unit field was blank.
I got an NPE from PersistenceTresholdFilter, and the PersistenceIncludeFilter would also throw a NPE in that case.
For PersistenceTimeFilter, defaulting to "s" is just cosmetic.

Picks-up PR #3681 and should be merged for the 4.0 release, because the UI does not prevent the unit field from being null.

Signed-off-by: Florian Hotze <[email protected]>
    @florian-h05
    florian-h05 committed Jul 20, 2023
    Configuration menu
    Copy the full SHA
    d7141d5 View commit details
    Browse the repository at this point in the history