diff options
author | Pim van den Berg | 2013-10-09 22:46:47 +0200 |
---|---|---|
committer | Pim van den Berg | 2013-10-09 22:46:47 +0200 |
commit | 1aab1e38b61242d4d19e65719a9682827d92853c (patch) | |
tree | bbdec43e60cf3118f5c2154c1acc421fc5521b3b /type/Uptime.class.php | |
parent | type: fix undefined index in ds_names[$source] before empty check (diff) | |
download | apt-panopticon_cgp-1aab1e38b61242d4d19e65719a9682827d92853c.zip apt-panopticon_cgp-1aab1e38b61242d4d19e65719a9682827d92853c.tar.gz apt-panopticon_cgp-1aab1e38b61242d4d19e65719a9682827d92853c.tar.bz2 apt-panopticon_cgp-1aab1e38b61242d4d19e65719a9682827d92853c.tar.xz |
Revert "type: fix undefined index in ds_names[$source] before empty check"
From the docs:
"No warning is generated if the variable does not exist. That means empty() is
essentially the concise equivalent to !isset($var) || $var == false."
This reverts commit f2a4593558b3a3ec696e9cc1de93fa898b9521bb.
Reported-By: Benoit Monin (https://github.com/pommi/CGP/commit/f2a4593)
Diffstat (limited to '')
-rw-r--r-- | type/Uptime.class.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/type/Uptime.class.php b/type/Uptime.class.php index 6e08b0b..4ada117 100644 --- a/type/Uptime.class.php +++ b/type/Uptime.class.php | |||
@@ -41,7 +41,7 @@ class Type_Uptime extends Type_Default { | |||
41 | 41 | ||
42 | $c = 0; | 42 | $c = 0; |
43 | foreach ($sources as $source) { | 43 | foreach ($sources as $source) { |
44 | $dsname = (isset($this->ds_names[$source]) && !empty($this->ds_names[$source])) ? $this->ds_names[$source] : $source; | 44 | $dsname = empty($this->ds_names[$source]) ? $source : $this->ds_names[$source]; |
45 | $color = is_array($this->colors) ? (isset($this->colors[$source])?$this->colors[$source]:$this->colors[$c++]) : $this->colors; | 45 | $color = is_array($this->colors) ? (isset($this->colors[$source])?$this->colors[$source]:$this->colors[$c++]) : $this->colors; |
46 | 46 | ||
47 | //current value | 47 | //current value |