use $INPUT instead of $_REQUEST

ConX · Nov 14, 2020 · 874317c · 874317c
1 parent afb157b
commit 874317c
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/admin.php b/admin.php
@@ -56,34 +56,35 @@ protected function reloadBar() {
      * Execute the requested action
      */
     public function handle() {
+        global $INPUT;
 
-        if (isset($_REQUEST['add'])) {
+        if ($INPUT->has('add')) {
             if (!checkSecurityToken()) return;
 
             $conf = $this->loadCBData();
-            if(!$conf) {
+            if (!$conf) {
                 $conf = array();
             }
             $type = 0;
-            if ($_REQUEST["pretag"] != "" && $_REQUEST["posttag"] != "") {
+            if ($INPUT->str('pretag') != '' && $INPUT->str('posttag') != '') {
                 $type = 1;
             }
             array_push($conf, array(
-                'label' => $_REQUEST["label"],
-                'code'  => $_REQUEST["code"],
+                'label' => $INPUT->str('label'),
+                'code'  => $INPUT->str('code'),
                 'type'  => $type,
-                'pretag'  => $_REQUEST["pretag"],
-                'posttag' => $_REQUEST["posttag"],
-                'icon' => $_REQUEST["icon"],
+                'pretag'  => $INPUT->str('pretag'),
+                'posttag' => $INPUT->str('posttag'),
+                'icon'  => $INPUT->str('icon'),
             ));
 
             $this->saveCBData($conf);
             $this->reloadBar();
-        } elseif (isset($_REQUEST['delete'])) {
+        } elseif ($INPUT->has('delete')) {
             if (!checkSecurityToken()) return;
 
             $conf = $this->loadCBData();
-            unset($conf[$_REQUEST["delete"]]);
+            unset($conf[$INPUT->int('delete')]);
             $this->saveCBData($conf);
             $this->reloadBar();
         }