refacto: use Weakmap for UnitOfWork internal storage instead of uid i…

…ndexed arrays

CHANGELOG

@@ -1,7 +1,9 @@
 3.10.0 (unreleased):
+    * Drop PHP 7
     * Remove hard dependency to pimple, add it in suggest
     * Collection is now JsonSerializable
     * Add DateTimeImmutable serialization
+    * Refacto: use, when possible, Weakmap for UnitOfWork internal storage instead of uuid indexed array
     * Fix: Query Generator now handle null values
 
 3.9.1 (2024-07-19):

composer.json

@@ -21,7 +21,7 @@
     "type": "library",
     "license": "Apache-2.0",
     "require": {
-        "php": ">=7.1",
+        "php": ">=8.0",
         "aura/sqlquery": "^2.6",
         "doctrine/cache": "^1.6"
     },

src/Ting/Repository/Hydrator.php

@@ -31,6 +31,7 @@
 use CCMBenchmark\Ting\Serializer\UnserializeInterface;
 use CCMBenchmark\Ting\UnitOfWork;
 use Generator;
+use WeakMap;
 
 /**
  * @template T
@@ -45,7 +46,7 @@ class Hydrator implements HydratorInterface
     protected $objectDatabase     = [];
     protected $objectSchema       = [];
     protected $unserializeAliases = [];
-    protected $alreadyManaged     = [];
+    protected WeakMap $alreadyManaged;
     protected $references         = [];
     protected $metadataList       = [];
 
@@ -69,6 +70,11 @@ class Hydrator implements HydratorInterface
      */
     protected $identityMap = false;
 
+    public function __construct()
+    {
+        $this->alreadyManaged = new WeakMap();
+    }
+
     /**
      * @param bool $enable
      * @throws Exception
@@ -243,7 +249,7 @@ private function hasVirtualObject(array $result)
      */
     private function unserializeVirtualObjectProperty(\stdClass $virtualObject)
     {
-        foreach ($this->unserializeAliases as $aliasName => list($unserialize, $options)) {
+        foreach ($this->unserializeAliases as $aliasName => [$unserialize, $options]) {
             if (isset($virtualObject->$aliasName) === true) {
                 $virtualObject->$aliasName = $unserialize->unserialize($virtualObject->$aliasName, $options);
             }
@@ -326,7 +332,6 @@ function (Metadata $metadata) use ($column, &$result) {
                         if ($this->identityMap === false) {
                             // If identityMap is disabled, it clones the object and reset UUID
                             $result[$column['table']] = clone $this->references[$ref];
-                            unset($result[$column['table']]->tingUUID);
                         } else {
                             // If identityMap is enabled, it uses the same object
                             $result[$column['table']] = $this->references[$ref];
@@ -440,13 +445,9 @@ function (Metadata $metadata) use ($column, &$result) {
      */
     private function manageIfYouCan($entity)
     {
-        if (isset($entity->tingUUID) === true && isset($this->alreadyManaged[$entity->tingUUID]) === true) {
-            return;
-        }
-
-        if (\is_object($entity) === true && ($entity instanceof NotifyPropertyInterface) === true) {
+        if (\is_object($entity) === true && ($entity instanceof NotifyPropertyInterface) === true && $this->alreadyManaged->offsetExists($entity) === false) {
             $this->unitOfWork->manage($entity);
-            $this->alreadyManaged[$entity->tingUUID] = true;
+            $this->alreadyManaged[$entity] = true;
         }
     }
 }

src/Ting/Repository/HydratorRelational.php

@@ -72,6 +72,7 @@ final class HydratorRelational extends Hydrator
 
     public function __construct()
     {
+        parent::__construct();
         $this->config = new \SplDoublyLinkedList();
     }
 

src/Ting/Repository/HydratorSingleObject.php

@@ -33,7 +33,6 @@
  */
 class HydratorSingleObject extends Hydrator
 {
-
     /**
      * @return \Generator<int, T>
      */