refactor: Add cache invalidation method and tests for NestedSetsBehavior.

composer.json

@@ -17,6 +17,7 @@
         "ext-simplexml": "*",
         "infection/infection": "^0.27|^0.29",
         "maglnet/composer-require-checker": "^4.1",
+        "php-forge/support": "^0.1",
         "phpstan/extension-installer": "^1.4",
         "phpstan/phpstan-strict-rules": "^2.0.3",
         "phpunit/phpunit": "^10.2",

src/NestedSetsBehavior.php

@@ -103,20 +103,35 @@
      */
     public string $depthAttribute = 'depth';
 
+    /**
+     * Stores the depth value for the current operation.
+     */
+    protected int|null $depthValue = null;
+
     /**
      * Name of the attribute that stores the left boundary value of the node in the nested set tree.
      *
      * @phpstan-var 'lft' attribute name.
      */
     public string $leftAttribute = 'lft';
 
+    /**
+     * Stores the left value for the current operation.
+     */
+    protected int|null $leftValue = null;
+
     /**
      * Name of the attribute that stores the right boundary value of the node in the nested set tree.
      *
      * @phpstan-var 'rgt' attribute name.
      */
     public string $rightAttribute = 'rgt';
 
+    /**
+     * Stores the right value for the current operation.
+     */
+    protected int|null $rightValue = null;
+
     /**
      * Name of the attribute that stores the tree identifier for supporting multiple trees.
      */
@@ -130,21 +145,6 @@
      */
     private Connection|null $db = null;
 
-    /**
-     * Stores the depth value for the current operation.
-     */
-    private int|null $depthValue = null;
-
-    /**
-     * Stores the left value for the current operation.
-     */
-    private int|null $leftValue = null;
-
-    /**
-     * Stores the right value for the current operation.
-     */
-    private int|null $rightValue = null;
-
     /**
      * Handles post-deletion updates for the nested set structure.
      *
@@ -193,9 +193,7 @@
         }
 
         $this->shiftLeftRightAttribute($this->getRightValue(), $deltaValue);
-
-        $this->operation = null;
-        $this->node = null;
+        $this->invalidateCache();
     }
 
     /**
@@ -235,8 +233,7 @@
             );
         }
 
-        $this->operation = null;
-        $this->node = null;
+        $this->invalidateCache();
     }
 
     /**
@@ -268,17 +265,44 @@
 
         if ($this->operation === self::OPERATION_MAKE_ROOT) {
             $this->moveNodeAsRoot($currentOwnerTreeValue);
+            $this->invalidateCache();
 
             return;
         }
 
         if ($this->node === null) {
+            $this->invalidateCache();
+
             return;
         }
 
         $context = $this->createMoveContext($this->node, $this->operation);
-
         $this->moveNode($context);
+        $this->invalidateCache();
+    }
+
+    /**
+     * Invalidates cached attribute values and resets internal state.
+     *
+     * Clears the cached depth, left, and right attribute values, forcing them to be re-fetched from the owner model
+     * on next access.
+     *
+     * This method should be called after operations that modify the owner model's attributes to ensure that cached
+     * values remain consistent with the actual model state.
+     *
+     * Usage example:
+     * ```php
+     * // After modifying the model's attributes externally
+     * $behavior->invalidateCache();
+     * ```
+     */
+    public function invalidateCache(): void
+    {
+        $this->depthValue = null;
+        $this->leftValue = null;
+        $this->node = null;
+        $this->operation = null;
+        $this->rightValue = null;
     }
 
     /**
@@ -689,7 +713,7 @@
         $nodeLeft = $node->getAttribute($this->leftAttribute);
         $nodeRight = $node->getAttribute($this->rightAttribute);
 
         if ($this->getLeftValue() <= $nodeLeft || $this->getRightValue() >= $nodeRight) {
             return false;
         }
 
@@ -787,7 +811,7 @@
             $this->getRightValue(),
         );
 
         return $this->getOwner()::find()->andWhere($condition)->addOrderBy([$this->leftAttribute => SORT_ASC]);
     }
 
     /**
@@ -927,7 +951,7 @@
             $depth,
         );
 
         return $this->getOwner()::find()->andWhere($condition)->addOrderBy([$this->leftAttribute => SORT_ASC]);
     }
 
     /**
@@ -1106,7 +1130,7 @@
             $this->getTreeValue($this->getOwner()),
         );
         $result = $this->getOwner()::deleteAll($condition);
         $this->getOwner()->setOldAttributes(null);
         $this->getOwner()->afterDelete();
 
         return $result;
@@ -1134,7 +1158,7 @@
 
             $this->shiftLeftRightAttribute($context->targetPositionValue, $subtreeSize);
 
             if ($ownerLeftValue >= $context->targetPositionValue) {
                 $ownerLeftValue += $subtreeSize;
                 $ownerRightValue += $subtreeSize;
             }