#11694 - update to 3.3.x

doctrine · Oct 23, 2024 · 69ef7f8 · 69ef7f8
1 parent 61385f6
commit 69ef7f8
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 51 deletions.
diff --git a/src/Persisters/Entity/BasicEntityPersister.php b/src/Persisters/Entity/BasicEntityPersister.php
@@ -173,8 +173,7 @@ class BasicEntityPersister implements EntityPersister
     private readonly CachedPersisterContext $limitsHandlingContext;
     private readonly CachedPersisterContext $noLimitsContext;
 
-    /** @var ?string */
-    private $filterHash = null;
+    private string|null $filterHash = null;
 
     /**
      * Initializes a new <tt>BasicEntityPersister</tt> that uses the given EntityManager

diff --git a/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/ChangeFiltersTest.php b/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/ChangeFiltersTest.php
@@ -24,9 +24,7 @@ public function setUp(): void
         ]);
     }
 
-    /**
-     * @return non-empty-array<"companyA"|"companyB", array{orderId: int, userId: int}>
-     */
+    /** @return non-empty-array<"companyA"|"companyB", array{orderId: int, userId: int}> */
     private function prepareData(): array
     {
         $user1  = new User(self::COMPANY_A);
@@ -97,7 +95,7 @@ public function testUseQueryBuilder(): void
 
         ['companyA' => $companyA, 'companyB' => $companyB] = $this->prepareData();
 
-        $getOrderByIdCache = function (int $orderId): ?Order {
+        $getOrderByIdCache = function (int $orderId): Order|null {
             return $this->_em->createQueryBuilder()
                 ->select('orderMaster, user')
                 ->from(Order::class, 'orderMaster')

diff --git a/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/CompanySQLFilter.php b/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/CompanySQLFilter.php
@@ -11,14 +11,14 @@
 
 class CompanySQLFilter extends SQLFilter
 {
-    public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias): string
+    public function addFilterConstraint(ClassMetadata $targetEntity, string $targetTableAlias): string
     {
         if ($targetEntity->getName() === User::class) {
-            return sprintf('%s.%s = %s', $targetTableAlias, $targetEntity->fieldMappings['company']['fieldName'], $this->getParameter('company'));
+            return sprintf('%s.%s = %s', $targetTableAlias, $targetEntity->fieldMappings['company']->fieldName, $this->getParameter('company'));
         }
 
         if ($targetEntity->getName() === Order::class) {
-            return sprintf('%s.%s = %s', $targetTableAlias, $targetEntity->fieldMappings['company']['fieldName'], $this->getParameter('company'));
+            return sprintf('%s.%s = %s', $targetTableAlias, $targetEntity->fieldMappings['company']->fieldName, $this->getParameter('company'));
         }
 
         return '';

diff --git a/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/Order.php b/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/Order.php
@@ -6,34 +6,20 @@
 
 use Doctrine\ORM\Mapping as ORM;
 
-/**
- * @ORM\Entity
- * @ORM\Table(name="Order_Master")
- */
+#[ORM\Entity]
+#[ORM\Table('Order_Master')]
 class Order
 {
-    /**
-     * @ORM\Id
-     * @ORM\Column(type="integer")
-     * @ORM\GeneratedValue(strategy="AUTO")
-     *
-     * @var int
-     */
-    public $id;
+    #[ORM\Id]
+    #[ORM\Column(type: 'integer')]
+    #[ORM\GeneratedValue(strategy: 'AUTO')]
+    public int $id;
 
-    /**
-     * @ORM\Column(type="string")
-     *
-     * @var string
-     */
-    public $company;
+    #[ORM\Column(type: 'string')]
+    public string $company;
 
-    /**
-     * @ORM\ManyToOne(targetEntity="User", fetch="EAGER")
-     *
-     * @var User
-     */
-    public $user;
+    #[ORM\ManyToOne(targetEntity: User::class, fetch: 'EAGER')]
+    public User $user;
 
     public function __construct(User $user)
     {

diff --git a/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/User.php b/tests/Tests/ORM/Functional/Ticket/SwitchContextWithFilter/User.php
@@ -6,27 +6,17 @@
 
 use Doctrine\ORM\Mapping as ORM;
 
-/**
- * @ORM\Entity
- * @ORM\Table(name="User_Master")
- */
+#[ORM\Entity]
+#[ORM\Table('User_Master')]
 class User
 {
-    /**
-     * @ORM\Id
-     * @ORM\Column(type="integer")
-     * @ORM\GeneratedValue(strategy="AUTO")
-     *
-     * @var int
-     */
-    public $id;
+    #[ORM\Id]
+    #[ORM\Column(type: 'integer')]
+    #[ORM\GeneratedValue(strategy: 'AUTO')]
+    public int $id;
 
-    /**
-     * @ORM\Column(type="string")
-     *
-     * @var string
-     */
-    public $company;
+    #[ORM\Column(type: 'string')]
+    public string $company;
 
     public function __construct(string $company)
     {