📦 Migrate to attributes - since support for Doctrine annotations dropped

Author: andrew-demb

src/Types/SymfonyUserInterfaceType.php

@@ -9,14 +9,10 @@
 use Symfony\Component\Security\Core\User\UserInterface;
 use TheCodingMachine\GraphQLite\FieldNotFoundException;
 
-/**
- * @Type(class=UserInterface::class)
- */
+#[Type(class: UserInterface::class)]
 class SymfonyUserInterfaceType
 {
-    /**
-     * @Field
-     */
+    #[Field]
     public function getUserName(UserInterface $user): string
     {
         // @phpstan-ignore-next-line Forward Compatibility for Symfony >=5.3
@@ -33,9 +29,9 @@ public function getUserName(UserInterface $user): string
     }
 
     /**
-     * @Field()
      * @return string[]
      */
+    #[Field]
     public function getRoles(UserInterface $user): array
     {
         $roles = [];

tests/Fixtures/Controller/TestGraphqlController.php

@@ -20,116 +20,89 @@
 
 class TestGraphqlController
 {
-
-    /**
-     * @Query()
-     */
+    #[Query]
     public function test(string $foo): string
     {
         return 'echo ' .$foo;
     }
 
     /**
-     * @Query()
      * @return Product[]
      */
+    #[Query]
     public function products(): array
     {
         return [
             new Product('Mouf', 9999)
         ];
     }
 
-    /**
-     * @Query()
-     */
+    #[Query]
     public function contact(): Contact
     {
         return new Contact('Mouf');
     }
 
-    /**
-     * @Mutation()
-     */
+    #[Mutation]
     public function saveProduct(Product $product): Product
     {
         return $product;
     }
 
     /**
-     * @Query()
      * @return Contact[]
      */
+    #[Query]
     public function contacts(): ArrayResult
     {
         return new ArrayResult([new Contact('Mouf')]);
     }
 
-    /**
-     * @Query()
-     * @return string
-     */
+    #[Query]
     public function triggerException(int $code = 0): string
     {
         throw new MyException('Boom', $code);
     }
 
-    /**
-     * @Query()
-     * @return string
-     */
+    #[Query]
     public function triggerAggregateException(): string
     {
         $exception1 = new GraphQLException('foo', 401);
         $exception2 = new GraphQLException('bar', 404, null, ['field' => 'baz', 'category' => 'MyCat']);
         throw new GraphQLAggregateException([$exception1, $exception2]);
     }
 
-    /**
-     * @Query()
-     * @Logged()
-     * @FailWith(null)
-     * @return string
-     */
+    #[Query]
+    #[Logged]
+    #[FailWith(null)]
     public function loggedQuery(): string
     {
         return 'foo';
     }
 
-    /**
-     * @Query()
-     * @Right("ROLE_ADMIN")
-     * @FailWith(null)
-     * @return string
-     */
+    #[Query]
+    #[Right('ROLE_ADMIN')]
+    #[FailWith(null)]
     public function withAdminRight(): string
     {
         return 'foo';
     }
 
-    /**
-     * @Query()
-     * @Right("ROLE_USER")
-     * @FailWith(null)
-     * @return string
-     */
+    #[Query]
+    #[Right('ROLE_USER')]
+    #[FailWith(null)]
     public function withUserRight(): string
     {
         return 'foo';
     }
 
-    /**
-     * @Query()
-     * @return string
-     */
+    #[Query]
     public function getUri(Request $request): string
     {
         return $request->getPathInfo();
     }
 
-    /**
-     * @Query
-     */
+    #[Query]
     #[Assertion(for: 'email', constraint: new Assert\Email())]
     public function findByMail(string $email = 'a@a.com'): string
     {

tests/Fixtures/Entities/BadClass.php

@@ -1,5 +1,9 @@
 <?php
 
+if (class_exists('\BadClass')) {
+    // workaround for https://github.com/alekitto/class-finder/issues/24
+    return;
+}
 
 // The namespace for this class is broken. It must not impact Symfony.
 class BadClass

tests/Fixtures/Entities/Contact.php

@@ -1,18 +1,14 @@
 <?php
 
-
 namespace TheCodingMachine\GraphQLite\Bundle\Tests\Fixtures\Entities;
 
-
 use stdClass;
 use TheCodingMachine\GraphQLite\Annotations\Field;
 use TheCodingMachine\GraphQLite\Annotations\Type;
 use TheCodingMachine\GraphQLite\Bundle\Tests\Fixtures\Controller\TestGraphqlController;
 use TheCodingMachine\GraphQLite\Annotations\Autowire;
 
-/**
- * @Type()
- */
+#[Type]
 class Contact
 {
     /**
@@ -25,51 +21,45 @@ public function __construct(string $name)
         $this->name = $name;
     }
 
-    /**
-     * @Field(name="name")
-     */
+    #[Field(name: 'name')]
     public function getName(): string
     {
         return $this->name;
     }
 
-    /**
-     * @Field()
-     * @Autowire(for="$testService")
-     * @Autowire(for="$someService", identifier="someService")
-     * @Autowire(for="$someAlias", identifier="someAlias")
-     * @return string
-     */
-    public function injectService(TestGraphqlController $testService = null, stdClass $someService = null, stdClass $someAlias = null): string
-    {
+    #[Field]
+    public function injectService(
+        #[Autowire]
+        TestGraphqlController $testService = null,
+        #[Autowire(identifier: 'someService')]
+        stdClass $someService = null,
+        #[Autowire(identifier: 'someAlias')]
+        stdClass $someAlias = null,
+    ): string {
         if (!$testService instanceof TestGraphqlController || $someService === null || $someAlias === null) {
             return 'KO';
         }
         return 'OK';
     }
 
-    /**
-     * @Field(prefetchMethod="prefetchData")
-     */
+    #[Field(prefetchMethod: 'prefetchData')]
     public function injectServicePrefetch($prefetchData): string
     {
         return $prefetchData;
     }
 
-    /**
-     * @Autowire(for="$someOtherService", identifier="someOtherService")
-     */
-    public function prefetchData(iterable $iterable, stdClass $someOtherService = null)
-    {
+    public function prefetchData(
+        iterable $iterable,
+        #[Autowire(identifier: 'someOtherService')]
+        stdClass $someOtherService = null,
+    ) {
         if ($someOtherService === null) {
             return 'KO';
         }
         return 'OK';
     }
 
-    /**
-     * @Field()
-     */
+    #[Field]
     public function getManager(): ?Contact
     {
         return null;

tests/Fixtures/Entities/Product.php

@@ -1,9 +1,7 @@
 <?php
 
-
 namespace TheCodingMachine\GraphQLite\Bundle\Tests\Fixtures\Entities;
 
-
 class Product
 {
     /**

tests/Fixtures/Types/ContactType.php

@@ -9,14 +9,10 @@
 use TheCodingMachine\GraphQLite\Bundle\Tests\Fixtures\Entities\Contact;
 
 
-/**
- * @ExtendType(class=Contact::class)
- */
+#[ExtendType(class: Contact::class)]
 class ContactType
 {
-    /**
-     * @Field()
-     */
+    #[Field]
     public function uppercaseName(Contact $contact): string
     {
         return strtoupper($contact->getName());

tests/Fixtures/Types/ProductFactory.php

@@ -9,10 +9,7 @@
 
 class ProductFactory
 {
-
-    /**
-     * @Factory()
-     */
+    #[Factory]
     public function buildProduct(string $name, float $price): Product
     {
         return new Product($name, $price);

tests/Fixtures/Types/ProductType.php

@@ -10,16 +10,12 @@
 use TheCodingMachine\GraphQLite\Bundle\Tests\Fixtures\Entities\Product;
 
 
-/**
- * @Type(class=Product::class)
- * @SourceField(name="name")
- * @SourceField(name="price")
- */
+#[Type(class: Product::class)]
+#[SourceField(name: 'name')]
+#[SourceField(name: 'price')]
 class ProductType
 {
-    /**
-     * @Field()
-     */
+    #[Field]
     public function getSeller(Product $product): ?Contact
     {
         return null;

tests/NoSecurityBundleFixtures/Controller/EchoController.php

@@ -1,16 +1,12 @@
 <?php
 
-
 namespace TheCodingMachine\GraphQLite\Bundle\Tests\NoSecurityBundleFixtures\Controller;
 
-
 use TheCodingMachine\GraphQLite\Annotations\Query;
 
 class EchoController
 {
-    /**
-     * @Query()
-     */
+    #[Query]
     public function echoMsg(string $message): string {
         return $message;
     }