From fcb96f53d7ffd286b1c8f4b577638f5ec34cfb3e Mon Sep 17 00:00:00 2001
From: =?utf8?q?Uwe=20Kleine-K=C3=B6nig?= <uwe@kleine-koenig.org>
Date: Fri, 16 Dec 2022 23:22:04 +0100
Subject: [PATCH] luci-app-statistics: Generate graphs for humidity sensors
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

This fixes graph generation for sensors that provide humidity data.

Suggested-by: Jo-Philipp Wich <jo@mein.io>
Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
(cherry picked from commit 9f321b831f41b73554819ac54e308dfa3677ab45)
---
 .../statistics/rrdtool/definitions/sensors.js | 50 ++++++++++++++-----
 .../view/statistics/plugins/sensors.js        |  1 +
 2 files changed, 38 insertions(+), 13 deletions(-)

diff --git a/applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/sensors.js b/applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/sensors.js
index b3361e1bf4..3da1bcfd82 100644
--- a/applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/sensors.js
+++ b/applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/sensors.js
@@ -7,20 +7,44 @@ return baseclass.extend({
 	title: _('Sensors'),
 
 	rrdargs: function(graph, host, plugin, plugin_instance, dtype) {
-		return {
-			per_instance: true,
-			title: "%H: %pi - %di",
-			vlabel: "\xb0C",
-			number_format: "%4.1lf\xb0C",
-			data: {
-				types: [ "temperature" ],
-				options: {
-					temperature__value: {
-						color: "ff0000",
-						title: "Temperature"
+		var rv = [];
+		var types = graph.dataTypes(host, plugin, plugin_instance);
+
+		if (types.indexOf('temperature') > -1) {
+			rv.push({
+				per_instance: true,
+				title: "%H: %pi - %di",
+				vlabel: "\xb0C",
+				number_format: "%4.1lf\xb0C",
+				data: {
+					types: [ "temperature" ],
+					options: {
+						temperature__value: {
+							color: "ff0000",
+							title: "Temperature"
+						}
+					}
+				}
+			});
+		}
+		if (types.indexOf('humidity') > -1) {
+			rv.push({
+				per_instance: true,
+				title: "%H: %pi - %di",
+				vlabel: "%RH",
+				number_format: "%4.1lf %%RH",
+				data: {
+					types: [ "humidity" ],
+					options: {
+						humidity__value: {
+							color: "0000ff",
+							title: "Humidity"
+						}
 					}
 				}
-			}
-		};
+			});
+		}
+
+		return rv;
 	}
 });
diff --git a/applications/luci-app-statistics/htdocs/luci-static/resources/view/statistics/plugins/sensors.js b/applications/luci-app-statistics/htdocs/luci-static/resources/view/statistics/plugins/sensors.js
index 8424966d2d..a1a2472654 100644
--- a/applications/luci-app-statistics/htdocs/luci-static/resources/view/statistics/plugins/sensors.js
+++ b/applications/luci-app-statistics/htdocs/luci-static/resources/view/statistics/plugins/sensors.js
@@ -8,6 +8,7 @@ var sensorTypes = [
 	/^(?:ain|in|vccp|vdd|vid|vin|volt|voltbatt|vrm)[0-9]*$/,	'voltage',
 	/^(?:cpu_temp|remote_temp|temp)[0-9]*$/,					'temperature',
 	/^(?:fan)[0-9]*$/,											'fanspeed',
+	/^(?:humidity)[0-9]*$/,										'humidity',
 	/^(?:power)[0-9]*$/,										'power'
 ];
 
-- 
2.30.2