Extract method to get event scheduler status

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
phpmyadmin · Apr 17, 2020 · fc968a2 · fc968a2
1 parent 9b7c493
commit fc968a2
Showing 1 changed file with 15 additions and 8 deletions.
diff --git a/libraries/classes/Database/Events.php b/libraries/classes/Database/Events.php
@@ -462,26 +462,33 @@ public function getQueryFromRequest()
         return $query;
     }
 
-    public function getFooterToggleButton(): string
+    private function getEventSchedulerStatus(): bool
     {
-        global $db, $table;
-
-        $es_state = $this->dbi->fetchValue(
-            "SHOW GLOBAL VARIABLES LIKE 'event_scheduler'",
+        $state = $this->dbi->fetchValue(
+            'SHOW GLOBAL VARIABLES LIKE \'event_scheduler\'',
             0,
             1
         );
-        $es_state = mb_strtolower($es_state);
+        $state = strtoupper($state);
+
+        return $state === 'ON' || $state === '1';
+    }
+
+    public function getFooterToggleButton(): string
+    {
+        global $db, $table;
+
+        $state = $this->getEventSchedulerStatus();
         $options = [
             0 => [
                 'label' => __('OFF'),
                 'value' => 'SET GLOBAL event_scheduler="OFF"',
-                'selected' => $es_state != 'on',
+                'selected' => ! $state,
             ],
             1 => [
                 'label' => __('ON'),
                 'value' => 'SET GLOBAL event_scheduler="ON"',
-                'selected' => $es_state == 'on',
+                'selected' => $state,
             ],
         ];