fix: keep the return from the interface instead

symfony · Jan 15, 2025 · 507afc1 · 507afc1
1 parent 0624f13
commit 507afc1
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 8 deletions.
diff --git a/src/Security/UserClassBuilder.php b/src/Security/UserClassBuilder.php
@@ -132,7 +132,7 @@ private function addGetRoles(ClassSourceManipulator $manipulator, UserClassConfi
             'getRoles',
             'array',
             false,
-            ['@see UserInterface', '@return list<string>']
+            ['@see UserInterface']
         );
 
         // $roles = $this->roles

diff --git a/tests/Security/fixtures/expected/UserEntityWithEmailAsIdentifier.php b/tests/Security/fixtures/expected/UserEntityWithEmailAsIdentifier.php
@@ -59,7 +59,6 @@ public function getUserIdentifier(): string
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserEntityWithPassword.php b/tests/Security/fixtures/expected/UserEntityWithPassword.php
@@ -54,7 +54,6 @@ public function setUserIdentifier(string $userIdentifier): static
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserEntityWithUser_IdentifierAsIdentifier.php b/tests/Security/fixtures/expected/UserEntityWithUser_IdentifierAsIdentifier.php
@@ -54,7 +54,6 @@ public function setUserIdentifier(string $user_identifier): static
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserEntityWithoutPassword.php b/tests/Security/fixtures/expected/UserEntityWithoutPassword.php
@@ -47,7 +47,6 @@ public function setUserIdentifier(string $userIdentifier): static
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserModelWithEmailAsIdentifier.php b/tests/Security/fixtures/expected/UserModelWithEmailAsIdentifier.php
@@ -43,7 +43,6 @@ public function getUserIdentifier(): string
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserModelWithPassword.php b/tests/Security/fixtures/expected/UserModelWithPassword.php
@@ -38,7 +38,6 @@ public function setUserIdentifier(string $userIdentifier): static
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {

diff --git a/tests/Security/fixtures/expected/UserModelWithoutPassword.php b/tests/Security/fixtures/expected/UserModelWithoutPassword.php
@@ -32,7 +32,6 @@ public function setUserIdentifier(string $userIdentifier): static
 
     /**
      * @see UserInterface
-     * @return list<string>
      */
     public function getRoles(): array
     {
-Original file line number
+Diff line change
@@ Expand Up / @@ -59,7 +59,6 @@ public function getUserIdentifier(): string @@
         /**
          * @see UserInterface
-         * @return list<string>
          */
         public function getRoles(): array
         {
@@ Expand Down @@