Thermal sensors broke

q54e3w

From last nights update resolving some menu issues, this appears new.

2.3-ALPHA (amd64) 
built on Thu Dec 17 02:00:21 CST 2015 
FreeBSD 10.2-STABLE

Fatal error: Cannot redeclare saveThresholdSettings() (previously declared in /usr/local/www/widgets/widgets/thermal_sensors.widget.php:96) in /usr/local/www/widgets/widgets/thermal_sensors.widget.php on line 96 Call Stack: 0.0002 226952 1\. {main}() /usr/local/www/index.php:0 PHP ERROR: Type: 1, File: /usr/local/www/widgets/widgets/thermal_sensors.widget.php, Line: 96, Message: Cannot redeclare saveThresholdSettings() (previously declared in /usr/local/www/widgets/widgets/thermal_sensors.widget.php:96)

phil.davis

It sounds like you still have the index.php version that uses $i as a variable name.
Do you also have Dyn DNS Status widget enabled on the dashboard? That causes a problem because it also uses $i.
I suggest gitsync to the latest code. Otherwise get the latest index.php from https://github.com/pfsense/pfsense/blob/master/src/usr/local/www/index.php

The problem is caused because each widget PHP code gets included twice, so any function in a widget will give this "cannot redeclare" fatal error.

q54e3w

yes I do, thanks Phil for the feedback.

Steve_B

New snapshot is available now with correction for this.

q54e3w

great timing, with my skillz I would likely have broken Git trying to pull down the commit :o

jdillard

you don't break git, git breaks you ;)

if you go to the developer shell (option 12) and run "playback gitsync master" it will sync with that latest code in github. You'll have to install git first (it prompts you), but it is fairly straightforward. Feel free to play around with it in a vm and workout your git skillz.

jwt

Note that it doesn't install new compiled code, so you'll probably want to update as well