The current comparison for configured threshold values is not type-safe and thus ignores the not so un-common threshold of 0. Attached patch fixes the issue.

CommentFileSizeAuthor
#6 inerdiff-2684191-2-6-do-not-test.diff900 bytesa.milkovsky
#6 d7_monitoring-0-values-6.patch4.2 KBa.milkovsky
#2 d7_monitoring-0-values.patch4.2 KBfago

Comments

fago created an issue. See original summary.

fago’s picture

fago
Primary language German
Location Vienna
commented
StatusFileSize
new d7_monitoring-0-values.patch4.2 KB
miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

Oopsie... Interestingly we never needed that "not-so-uncommon" threshold value ;-) - but sure!

Status: Needs review » Needs work

The last submitted patch, 2: d7_monitoring-0-values.patch, failed testing.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

Does this threshold problem also apply for 8.x?

a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented
Status: Needs work » Needs review
StatusFileSize
new d7_monitoring-0-values-6.patch4.2 KB
new inerdiff-2684191-2-6-do-not-test.diff900 bytes

There was a typo. Fixed the patch in #2

a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented

The same issue in Drupal 8 #2704061: Sensors ignore thresholds of 0. Drupal 8.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Reviewed & tested by the community

ops, too many parentheses. Good to have test coverage! Thanks, a.milkovsky - I think this is ready now.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

This looks good, but tests would definitely be helpful here, we added some to the D8 port, should be possible to backport them to 7.x

  • Berdir committed 0de4ed0 on 7.x-1.x authored by a.milkovsky 
    Issue #2684191 by a.milkovsky, fago: Sensors ignore thresholds of 0
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Title: Sensors ignore thresholds of 0 » [Needs tests] Sensors ignore thresholds of 0
Status: Reviewed & tested by the community » Needs work
Issue tags: +Novice

Decided to commit and keeping open for tests. Maybe someone will pick this up. Tagging as novice, it is not trivial but the tests have been written, they just need to be applied from the other issue.