Removed ping + performance graphs and tab if skip ping check #2175

Merged
merged 4 commits into from Oct 20, 2015

Projects

None yet

3 participants

@laf
Member
laf commented Oct 17, 2015

Fix #2174

This will now remove the icmp graph, performance tab and won't insert new perf data or rrd data for the same.

@laf laf Removed ping + performance graphs and tab if skip ping check
bbae6e9
@laf laf added Polling WebUI labels Oct 17, 2015
@laf laf Stop adding device_perf data if no ping
96a0abb
@laf laf Scrut fixes
94c5afb
@SaaldjorMike
Member

@laf If you are referring to the initialisation at line 148, then this is not really the same variable since this is named $poll_update_array where the use scrut is complaining about is about $update_array.

@laf laf Scrut fix
ff1fc6f
@laf
Member
laf commented Oct 17, 2015

Thanks @SaaldjorMike Sorted now :)

@laf
Member
laf commented Oct 20, 2015

bump

@SaaldjorMike SaaldjorMike self-assigned this Oct 20, 2015
@SaaldjorMike SaaldjorMike merged commit 8b27e04 into librenms:master Oct 20, 2015

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
@laf laf deleted the laf:issue-2174 branch Oct 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment