Move attribute classes from Mapping\Annotations to Mapping\Attribute namespace

Author: GromNaN

UPGRADE-2.16.md

@@ -1,5 +1,27 @@
 # UPGRADE FROM 2.15 to 2.16
 
+## Attribute namespaces
+
+Doctrine annotations are already deprecated in favor of PHP attributes. As of MongoDB ODM 2.16, the
+attribute namespaces have been changed from `Doctrine\ODM\MongoDB\Mapping\Annotations` to
+`Doctrine\ODM\MongoDB\Attribute`. The old annotation namespaces will continue to work for the time being, but
+they are deprecated and will be removed in 3.0.
+
+```diff
+- use Doctrine\ODM\MongoDB\Mapping\Annotations as ODM;
++ use Doctrine\ODM\MongoDB\Attribute as ODM;
+
+  #[ODM\Document]
+  class User
+  {
+      #[ODM\Id]
+      private string $id;
+
+      #[ODM\Field]
+      public string $name;
+  }
+```
+
 ## Lazy Proxy Directory
 
 Using proxy classes with PHP 8.4+ is deprecated, only native lazy objects will

phpstan-baseline.neon

@@ -379,10 +379,10 @@ parameters:
 			path: src/Iterator/IterableResult.php
 
 		-
-			message: '#^Method Doctrine\\ODM\\MongoDB\\Mapping\\Annotations\\SearchIndex\:\:__construct\(\) has parameter \$fields with no value type specified in iterable type array\.$#'
+			message: '#^Method Doctrine\\ODM\\MongoDB\\Mapping\\Attribute\\SearchIndex\:\:__construct\(\) has parameter \$fields with no value type specified in iterable type array\.$#'
 			identifier: missingType.iterableValue
 			count: 1
-			path: src/Mapping/Annotations/SearchIndex.php
+			path: src/Mapping/Attribute/SearchIndex.php
 
 		-
 			message: '#^Method Doctrine\\ODM\\MongoDB\\Mapping\\ClassMetadata\:\:addInheritedAssociationMapping\(\) has Doctrine\\ODM\\MongoDB\\Mapping\\MappingException in PHPDoc @throws tag but it''s not thrown\.$#'
@@ -469,7 +469,7 @@ parameters:
 			path: src/Mapping/Driver/XmlDriver.php
 
 		-
-			message: '#^Parameter \#2 \$mapping of method Doctrine\\ODM\\MongoDB\\Mapping\\Driver\\XmlDriver\:\:addFieldMapping\(\) expects array\{type\?\: string, fieldName\?\: string, name\?\: string, strategy\?\: string, association\?\: int, id\?\: bool, isOwningSide\?\: bool, collectionClass\?\: class\-string, \.\.\.\}, array\<string, array\<''contention''\|''max''\|''min''\|''precision''\|''queryType''\|''sparsity''\|''trimFactor''\|int\<0, max\>, Doctrine\\ODM\\MongoDB\\Mapping\\Annotations\\EncryptQuery\|float\|int\|MongoDB\\BSON\\Decimal128\|MongoDB\\BSON\\Int64\|MongoDB\\BSON\\UTCDateTime\|string\|null\>\|bool\|string\> given\.$#'
+			message: '#^Parameter \#2 \$mapping of method Doctrine\\ODM\\MongoDB\\Mapping\\Driver\\XmlDriver\:\:addFieldMapping\(\) expects array\{type\?\: string, fieldName\?\: string, name\?\: string, strategy\?\: string, association\?\: int, id\?\: bool, isOwningSide\?\: bool, collectionClass\?\: class\-string, \.\.\.\}, array\<string, array\<''contention''\|''max''\|''min''\|''precision''\|''queryType''\|''sparsity''\|''trimFactor''\|int\<0, max\>, Doctrine\\ODM\\MongoDB\\Mapping\\Attribute\\EncryptQuery\|float\|int\|MongoDB\\BSON\\Decimal128\|MongoDB\\BSON\\Int64\|MongoDB\\BSON\\UTCDateTime\|string\|null\>\|bool\|string\> given\.$#'
 			identifier: argument.type
 			count: 1
 			path: src/Mapping/Driver/XmlDriver.php
@@ -1057,13 +1057,13 @@ parameters:
 			path: tests/Tests/Functional/QueryableEncryptionTest.php
 
 		-
-			message: '#^Parameter \$discriminatorMap of attribute class Doctrine\\ODM\\MongoDB\\Mapping\\Annotations\\ReferenceOne constructor expects array\<string, class\-string\>\|null, array\<string, string\> given\.$#'
+			message: '#^Parameter \$discriminatorMap of attribute class Doctrine\\ODM\\MongoDB\\Mapping\\Attribute\\ReferenceOne constructor expects array\<string, class\-string\>\|null, array\<string, string\> given\.$#'
 			identifier: argument.type
 			count: 1
 			path: tests/Tests/Functional/TargetDocumentTest.php
 
 		-
-			message: '#^Parameter \$targetDocument of attribute class Doctrine\\ODM\\MongoDB\\Mapping\\Annotations\\ReferenceOne constructor expects class\-string\|null, string given\.$#'
+			message: '#^Parameter \$targetDocument of attribute class Doctrine\\ODM\\MongoDB\\Mapping\\Attribute\\ReferenceOne constructor expects class\-string\|null, string given\.$#'
 			identifier: argument.type
 			count: 1
 			path: tests/Tests/Functional/TargetDocumentTest.php

src/Mapping/Annotations/AbstractDocument.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
+
+use function class_alias;
+
+abstract class AbstractDocument implements Annotation
+{
+}
+
+// @phpstan-ignore class.notFound
+class_alias(AbstractDocument::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\AbstractDocument::class);

src/Mapping/Annotations/Annotation.php

@@ -2,7 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Doctrine\ODM\MongoDB\Mapping\Annotations;
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
+
+use function class_alias;
 
 abstract class AbstractField implements Annotation
 {
@@ -41,3 +43,6 @@ public function __construct(
         $this->notSaved = $notSaved;
     }
 }
+
+// @phpstan-ignore class.notFound
+class_alias(AbstractField::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\AbstractField::class);

src/Mapping/Attribute/AbstractDocument.php

@@ -2,7 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Doctrine\ODM\MongoDB\Mapping\Annotations;
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
+
+use function class_alias;
 
 abstract class AbstractIndex implements Annotation
 {
@@ -57,3 +59,6 @@ public function __construct(
         $this->partialFilterExpression = $partialFilterExpression;
     }
 }
+
+// @phpstan-ignore class.notFound
+class_alias(AbstractIndex::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\AbstractIndex::class);

src/Mapping/Annotations/AbstractField.php src/Mapping/Attribute/AbstractField.phpsrc/Mapping/Annotations/AbstractField.php renamed to src/Mapping/Attribute/AbstractField.php

@@ -2,11 +2,13 @@
 
 declare(strict_types=1);
 
-namespace Doctrine\ODM\MongoDB\Mapping\Annotations;
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
 
 use Attribute;
 use Doctrine\Common\Annotations\Annotation\NamedArgumentConstructor;
 
+use function class_alias;
+
 /**
  * Loads data from a different field if the original field is not set
  *
@@ -21,3 +23,6 @@ public function __construct(public $value, public ?string $name = null)
     {
     }
 }
+
+// @phpstan-ignore class.notFound
+class_alias(AlsoLoad::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\AlsoLoad::class);

src/Mapping/Annotations/AbstractIndex.php src/Mapping/Attribute/AbstractIndex.phpsrc/Mapping/Annotations/AbstractIndex.php renamed to src/Mapping/Attribute/AbstractIndex.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
+
+use function class_alias;
+
+interface Annotation
+{
+}
+
+// @phpstan-ignore class.notFound
+class_alias(Annotation::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\Annotation::class);

src/Mapping/Annotations/AlsoLoad.php src/Mapping/Attribute/AlsoLoad.phpsrc/Mapping/Annotations/AlsoLoad.php renamed to src/Mapping/Attribute/AlsoLoad.php

@@ -2,11 +2,13 @@
 
 declare(strict_types=1);
 
-namespace Doctrine\ODM\MongoDB\Mapping\Annotations;
+namespace Doctrine\ODM\MongoDB\Mapping\Attribute;
 
 use Attribute;
 use Doctrine\Common\Annotations\Annotation\NamedArgumentConstructor;
 
+use function class_alias;
+
 /**
  * Specifies the change tracking policy for a document
  *
@@ -24,3 +26,6 @@ public function __construct(string $value)
         $this->value = $value;
     }
 }
+
+// @phpstan-ignore class.notFound
+class_alias(ChangeTrackingPolicy::class, \Doctrine\ODM\MongoDB\Mapping\Annotations\ChangeTrackingPolicy::class);