feat: Add shouldPersist property to MachineDefinition

tarfin-labs · Jan 22, 2024 · aa807d4 · aa807d4
1 parent a38e519
commit aa807d4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Actor/Machine.php b/src/Actor/Machine.php
@@ -162,7 +162,6 @@ public function start(State|string|null $state = null): self
      */
     public function send(
         EventBehavior|array|string $event,
-        bool $shouldPersist = true,
     ): State {
         if ($this->state !== null) {
             $lock = Cache::lock('mre:'.$this->state->history->first()->root_event_id, 60);
@@ -187,7 +186,7 @@ public function send(
                 default                          => $this->definition->transition($event, $this->state)
             };
 
-            if ($shouldPersist === true) {
+            if ($this->definition->shouldPersist) {
                 $this->persist();
             }
 

diff --git a/src/Definition/MachineDefinition.php b/src/Definition/MachineDefinition.php
@@ -60,6 +60,9 @@ class MachineDefinition
     /** Indicates whether the scenario is enabled. */
     public bool $scenariosEnabled = false;
 
+    /** machine-based variable that determines whether to persist the state change. */
+    public bool $shouldPersist = true;
+
     // endregion
 
     // region Constructor
@@ -83,6 +86,8 @@ private function __construct(
     ) {
         $this->scenariosEnabled = isset($this->config['scenarios_enabled']) && $this->config['scenarios_enabled'] === true;
 
+        $this->shouldPersist = $this->config['should_persist'] ?? $this->shouldPersist;
+
         $this->root = $this->createRootStateDefinition($config);
 
         // Checks if the scenario is enabled, and if true, creates scenario state definitions.