Browse Source

replaced FILTER_SANITIZE_STRING as it was being deprecated in php 8.1

CauseFX 4 years ago
parent
commit
8cacc07b8e
2 changed files with 3 additions and 3 deletions
  1. 2 2
      api/classes/organizr.class.php
  2. 1 1
      api/functions/log-functions.php

+ 2 - 2
api/classes/organizr.class.php

@@ -4873,7 +4873,7 @@ class Organizr
 		$array['type'] = ($array['type']) ?? 1;
 		$array['order'] = ($array['order']) ?? $this->getNextTabOrder() + 1;
 		if (array_key_exists('name', $array)) {
-			$array['name'] = filter_var($array['name'], FILTER_SANITIZE_STRING);
+			$array['name'] = htmlspecialchars($array['name']);
 			if ($this->isTabNameTaken($array['name'])) {
 				$this->setAPIResponse('error', 'Tab name: ' . $array['name'] . ' is already taken', 409);
 				return false;
@@ -4923,7 +4923,7 @@ class Organizr
 			return false;
 		}
 		if (array_key_exists('name', $array)) {
-			$array['name'] = filter_var($array['name'], FILTER_SANITIZE_STRING);
+			$array['name'] = htmlspecialchars($array['name']);
 			if ($this->isTabNameTaken($array['name'], $id)) {
 				$this->setAPIResponse('error', 'Tab name: ' . $array['name'] . ' is already taken', 409);
 				return false;

+ 1 - 1
api/functions/log-functions.php

@@ -212,7 +212,7 @@ trait LogFunctions
 		if ($this->hasDB()) {
 			$setLogger = false;
 			if ($username) {
-				$username = filter_var($username, FILTER_SANITIZE_STRING);
+				$username = htmlspecialchars($username);
 			}
 			if ($this->logger) {
 				if ($channel) {