From 6ccadfb55ea85bae60c4ffadf03f14111fbfdaf6 Mon Sep 17 00:00:00 2001
From: Franco Fichtner <franco@opnsense.org>
Date: Thu, 4 Jan 2018 13:36:19 +0100
Subject: [PATCH] mvc: make enabled checks consistent

Spotted by: @dharrigan
---
 .../OPNsense/Base/ApiMutableServiceControllerBase.php         | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/opnsense/mvc/app/controllers/OPNsense/Base/ApiMutableServiceControllerBase.php b/src/opnsense/mvc/app/controllers/OPNsense/Base/ApiMutableServiceControllerBase.php
index 678d27229..d19f59b08 100644
--- a/src/opnsense/mvc/app/controllers/OPNsense/Base/ApiMutableServiceControllerBase.php
+++ b/src/opnsense/mvc/app/controllers/OPNsense/Base/ApiMutableServiceControllerBase.php
@@ -199,14 +199,14 @@ abstract class ApiMutableServiceControllerBase extends ApiControllerBase
         $response = $backend->configdRun(escapeshellarg(static::$internalServiceName) . ' status');
 
         if (strpos($response, 'not running') > 0) {
-            if ((string)$model->getNodeByReference(static::$internalServiceEnabled) == 1) {
+            if ((string)$model->getNodeByReference(static::$internalServiceEnabled) == '1') {
                 $status = 'stopped';
             } else {
                 $status = 'disabled';
             }
         } elseif (strpos($response, 'is running') > 0) {
             $status = 'running';
-        } elseif ((string)$model->getNodeByReference(static::$internalServiceEnabled) == 0) {
+        } elseif ((string)$model->getNodeByReference(static::$internalServiceEnabled) == '0') {
             $status = 'disabled';
         } else {
             $status = 'unknown';