fix: move sanitize directly to sanitizeLayout

Signed-off-by: mykh-hailo <kristianderonta0205@gmail.com>

Author: mykh-hailo

apps/dashboard/lib/Service/DashboardService.php

@@ -31,7 +31,7 @@ public function __construct(
 	 */
 	public function getLayout(): array {
 		$systemDefault = $this->appConfig->getAppValueString('layout', 'recommendations,spreed,mail,calendar');
-		return $this->sanitizeStringList(
+		return $this->sanitizeLayout(
 			explode(',', $this->userConfig->getValueString($this->userId, 'dashboard', 'layout', $systemDefault)),
 		);
 	}
@@ -41,7 +41,18 @@ public function getLayout(): array {
 	 * @return list<string>
 	 */
 	public function sanitizeLayout(array $layout): array {
-		return $this->sanitizeStringList($layout);
+		$seen = [];
+		$result = [];
+		foreach ($layout as $value) {
+			if ($value === '' || isset($seen[$value])) {
+				continue;
+			}
+
+			$seen[$value] = true;
+			$result[] = $value;
+		}
+
+		return $result;
 	}
 
 	/**
@@ -80,25 +91,4 @@ public function getBirthdate(): string {
 
 		return $birthdate->getValue();
 	}
-
-	/**
-	 * Keep insertion order while removing empty and duplicate values.
-	 *
-	 * @param list<string> $values
-	 * @return list<string>
-	 */
-	private function sanitizeStringList(array $values): array {
-		$seen = [];
-		$result = [];
-		foreach ($values as $value) {
-			if ($value === '' || isset($seen[$value])) {
-				continue;
-			}
-
-			$seen[$value] = true;
-			$result[] = $value;
-		}
-
-		return $result;
-	}
 }