Project

General

Profile

Actions

Bug #12470

closed

Thermal Sensors Dashboard widget filter for negative values refers to invalid variable

Added by Daniel Cameron about 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Dashboard
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
22.01
Release Notes:
Affected Version:
All
Affected Architecture:
All

Description

The Thermal Sensors widget has code to filter out any sysctl line that contains ' -', but is checking a $negsign variable that doesn't exist instead of the $v that contains the correct string to be checking

$dfout_filtered = array_filter($dfout, function($v) {
    return strpos($negsign, ' -') === false;
});

This is fixed by pull request https://github.com/pfsense/pfsense/pull/4540 and has been tested to work on my install by manually changing this line (I have an ACPI thermal zone that reports a constant -273.1C that no longer shows up in the widget since the change)

Actions #1

Updated by Jim Pingle about 3 years ago

  • Subject changed from thermal_sensors.widget.php: filter for negative values refers to invalid variable to Thermal Sensors Dashboard widget filter for negative values refers to invalid variable
  • Category changed from Web Interface to Dashboard
  • Status changed from New to Pull Request Review
  • Target version set to 2.6.0
  • Plus Target Version set to 22.01
  • Release Notes deleted (Default)
Actions #2

Updated by Viktor Gurov almost 3 years ago

  • Status changed from Pull Request Review to Feedback

Merged

Actions #3

Updated by Jim Pingle almost 3 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF