From 448f4e1c8c4c0e96b983145163bd7200f3de9705 Mon Sep 17 00:00:00 2001
From: Franco Fichtner <franco@opnsense.org>
Date: Mon, 7 May 2018 00:10:38 +0200
Subject: [PATCH] sysutils/nut: proper service logic

PR: https://forum.opnsense.org/index.php?topic=7759.0
---
 .../nut/src/etc/inc/plugins.inc.d/nut.inc     | 49 ++++++++++---------
 1 file changed, 27 insertions(+), 22 deletions(-)

diff --git a/sysutils/nut/src/etc/inc/plugins.inc.d/nut.inc b/sysutils/nut/src/etc/inc/plugins.inc.d/nut.inc
index cac1ddff4..32c21c22d 100644
--- a/sysutils/nut/src/etc/inc/plugins.inc.d/nut.inc
+++ b/sysutils/nut/src/etc/inc/plugins.inc.d/nut.inc
@@ -44,30 +44,35 @@ function nut_services()
 {
     $services = array();
 
-    if (nut_enabled()) {
-        $services[] = array(
-            'description' => gettext('Network UPS Tools'),
-            'configd' => array(
-                'restart' => array('nut restart'),
-                'start' => array('nut start'),
-                'stop' => array('nut stop'),
-            ),
-            'name' => 'nut',
-            'pidfile' => '/var/db/nut/upsd.pid'
-        );
+    if (!nut_enabled()) {
+        return $services;
     }
-    if (nut_enabled() && nut_netclient()) {
-        $services[] = array(
-            'description' => gettext('Network UPS Monitor'),
-            'configd' => array(
-                'restart' => array('nut restart'),
-                'start' => array('nut start'),
-                'stop' => array('nut stop'),
-            ),
-            'name' => 'upsmon',
-            'pidfile' => '/var/db/nut/upsmon.pid'
-        );
+
+    $services[] = array(
+        'description' => gettext('Network UPS Monitor'),
+        'configd' => array(
+            'restart' => array('nut restart'),
+            'start' => array('nut start'),
+            'stop' => array('nut stop'),
+        ),
+        'pidfile' => '/var/db/nut/upsmon.pid',
+        'name' => 'upsmon',
+    );
+
+    if (nut_netclient()) {
+        return $services;
     }
 
+    $services[] = array(
+        'description' => gettext('Network UPS Tools'),
+        'configd' => array(
+            'restart' => array('nut restart'),
+            'start' => array('nut start'),
+            'stop' => array('nut stop'),
+        ),
+        'pidfile' => '/var/db/nut/upsd.pid',
+        'name' => 'nut',
+    );
+
     return $services;
 }