Merge pull request doctrine#10964 from soltmar/2.16.x-update-order-of-mapping-attributes

Updated order of mapping attribute parameters

Author: greg0ire

UPGRADE.md

@@ -1,5 +1,10 @@
 # Upgrade to 3.0
 
+## Minor BC BREAK: Changed order of arguments passed to `OneToOne`, `ManyToOne` and `Index` mapping PHP attributes
+
+To keep PHP mapping attributes consistent, order of arguments passed to above attributes has been changed
+so `$targetEntity` is a first argument now. This change affects only non-named arguments usage.
+
 ## BC BREAK: AUTO keyword for identity generation defaults to IDENTITY for PostgreSQL now
 
 When using the AUTO strategy to let Doctrine determine the identity generation mecehanism for

lib/Doctrine/ORM/Mapping/Index.php

@@ -16,9 +16,9 @@ final class Index implements MappingAttribute
      * @param array<string,mixed>|null $options
      */
     public function __construct(
+        public readonly string|null $name = null,
         public readonly array|null $columns = null,
         public readonly array|null $fields = null,
-        public readonly string|null $name = null,
         public readonly array|null $flags = null,
         public readonly array|null $options = null,
     ) {

lib/Doctrine/ORM/Mapping/OneToMany.php

@@ -15,8 +15,8 @@ final class OneToMany implements MappingAttribute
      * @psalm-param 'LAZY'|'EAGER'|'EXTRA_LAZY' $fetch
      */
     public function __construct(
-        public readonly string|null $mappedBy = null,
         public readonly string|null $targetEntity = null,
+        public readonly string|null $mappedBy = null,
         public readonly array|null $cascade = null,
         public readonly string $fetch = 'LAZY',
         public readonly bool $orphanRemoval = false,

lib/Doctrine/ORM/Mapping/OneToOne.php

@@ -15,9 +15,9 @@ final class OneToOne implements MappingAttribute
      * @psalm-param 'LAZY'|'EAGER'|'EXTRA_LAZY' $fetch
      */
     public function __construct(
+        public readonly string|null $targetEntity = null,
         public readonly string|null $mappedBy = null,
         public readonly string|null $inversedBy = null,
-        public readonly string|null $targetEntity = null,
         public readonly array|null $cascade = null,
         public readonly string $fetch = 'LAZY',
         public readonly bool $orphanRemoval = false,