Fix some code style issues (#386)

sulu · Jun 6, 2024 · 572776c · 572776c
1 parent 3f341b7
commit 572776c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 19 deletions.
diff --git a/Controller/FormTokenController.php b/Controller/FormTokenController.php
@@ -30,13 +30,14 @@ public function __construct(CsrfTokenManagerInterface $csrfTokenManager)
     public function tokenAction(Request $request): Response
     {
         $formName = $request->get('form');
+        $formName = \is_string($formName) ? $formName : '';
         $csrfToken = $this->csrfTokenManager->getToken($formName)->getValue();
 
         $content = $csrfToken;
 
         if ($request->get('html')) {
-            $formName = htmlspecialchars($formName, ENT_QUOTES, 'UTF-8');
-            $csrfToken = htmlspecialchars($csrfToken, ENT_QUOTES, 'UTF-8');
+            $formName = \htmlspecialchars($formName, \ENT_QUOTES, 'UTF-8');
+            $csrfToken = \htmlspecialchars($csrfToken, \ENT_QUOTES, 'UTF-8');
 
             $content = \sprintf(
                 '<input type="hidden" id="%s__token" name="%s[_token]" value="%s" />',

diff --git a/Tests/Functional/Controller/FormTokenControllerTest.php b/Tests/Functional/Controller/FormTokenControllerTest.php
@@ -22,9 +22,9 @@
 class FormTokenControllerTest extends SuluTestCase
 {
     /**
-     * @var FormTokenController
+     * @var FormTokenControllerTest
      */
-    private formTokenController $formTokenController;
+    private $formTokenController;
 
     protected function setUp(): void
     {

diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon
@@ -240,21 +240,6 @@ parameters:
 			count: 1
 			path: Controller/FormController.php
 
-		-
-			message: "#^Parameter \\#1 \\$tokenId of method Symfony\\\\Component\\\\Security\\\\Csrf\\\\CsrfTokenManagerInterface\\:\\:getToken\\(\\) expects string, mixed given\\.$#"
-			count: 1
-			path: Controller/FormTokenController.php
-
-		-
-			message: "#^Parameter \\#2 \\.\\.\\.\\$values of function sprintf expects bool\\|float\\|int\\|string\\|null, mixed given\\.$#"
-			count: 1
-			path: Controller/FormTokenController.php
-
-		-
-			message: "#^Parameter \\#3 \\.\\.\\.\\$values of function sprintf expects bool\\|float\\|int\\|string\\|null, mixed given\\.$#"
-			count: 1
-			path: Controller/FormTokenController.php
-
 		-
 			message: "#^Cannot call method all\\(\\) on Symfony\\\\Component\\\\Form\\\\FormInterface\\|null\\.$#"
 			count: 1