From 2bd53550aee7c1b7bceb0a085284e28aa6eb9573 Mon Sep 17 00:00:00 2001 From: Ben Smithurst Date: Sun, 5 Jan 2025 14:08:34 +0000 Subject: [PATCH] update config check to handle the default being true --- .../app/controllers/OPNsense/Vnstat/Api/ServiceController.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/vnstat/src/opnsense/mvc/app/controllers/OPNsense/Vnstat/Api/ServiceController.php b/net/vnstat/src/opnsense/mvc/app/controllers/OPNsense/Vnstat/Api/ServiceController.php index 41505caa4..bbf502baa 100644 --- a/net/vnstat/src/opnsense/mvc/app/controllers/OPNsense/Vnstat/Api/ServiceController.php +++ b/net/vnstat/src/opnsense/mvc/app/controllers/OPNsense/Vnstat/Api/ServiceController.php @@ -102,7 +102,7 @@ class ServiceController extends ApiMutableServiceControllerBase $config = Config::getInstance()->toArray(); $backend = new Backend(); - if (empty($config['OPNsense']['vnstat']['general']['separate_stats'])) { + if (isset($config['OPNsense']['vnstat']['general']['separate_stats']) && !$config['OPNsense']['vnstat']['general']['separate_stats']) { // Separate stats are not wanted, just get totals - that's the default in vnstat itself, // no need to specify anything here. $response = $backend->configdpRun("vnstat", [ $type ]);