diff options
| author | Peter Wu | 2014-07-20 23:23:17 +0200 |
|---|---|---|
| committer | Peter Wu | 2014-07-20 23:23:17 +0200 |
| commit | c7283e1cae6bbdc13464a77ae7f22bd9489fe089 (patch) | |
| tree | ffc767ddc0392b286d5cec0f053cf09796333077 /inc/collectd.inc.php | |
| parent | graph: don't overwrite config rrdtool_opts by rrdtool_opts from json plugin (diff) | |
| download | apt-panopticon_cgp-c7283e1cae6bbdc13464a77ae7f22bd9489fe089.zip apt-panopticon_cgp-c7283e1cae6bbdc13464a77ae7f22bd9489fe089.tar.gz apt-panopticon_cgp-c7283e1cae6bbdc13464a77ae7f22bd9489fe089.tar.bz2 apt-panopticon_cgp-c7283e1cae6bbdc13464a77ae7f22bd9489fe089.tar.xz | |
Defensive programming: more urlencode/htmlentities
Make build_url return an URL, not HTML. This separates presentation
from data.
plugin_header's return value is unused, remove the unnecessary return.
At places where `printf("<tag att='%s'>", $x);` is used, it is now
converted to `printf("<tag att=\"%s\">", htmlentities($x));` since
the single quote is not escaped by default by htmlentities.
In case the canvas style is used, JS should use `textContent` instead
of `innerHTML` to avoid reading `"` instead of `"`. Nobody (should)
use(s) IE6 anymore, so it is a safe change.
While at it, use the standard charset attribute of meta to specify
the character set (UTF-8).
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions