Remove redundant savePath constructor parameter from FileAdapter.php

lucasnetau · Dec 20, 2023 · 73175df · 73175df
1 parent a203991
commit 73175df
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/SaveHandler/FileAdapter.php b/src/SaveHandler/FileAdapter.php
@@ -57,7 +57,7 @@ class FileAdapter implements SaveHandlerInterface {
 
     protected bool $asyncFaulty = false;
 
-    public function __construct( protected string $savePath, protected string $saveFileName, protected LoggerInterface $logger, protected ?LoopInterface $loop ) {
+    public function __construct( protected string $saveFileName, protected LoggerInterface $logger, protected ?LoopInterface $loop ) {
         $this->loop ??= Loop::get();
     }
 

diff --git a/src/Scheduler.php b/src/Scheduler.php
@@ -859,7 +859,7 @@ public function run() : void
         $this->initialiseManagementServer();
 
         /** Restore the state of the scheduler and engine */
-        $this->saveStateHandler = new FileAdapter(sys_get_temp_dir(), $this->saveFileName, $this->logger, $this->loop);
+        $this->saveStateHandler = new FileAdapter($this->saveFileName, $this->logger, $this->loop);
         $this->restoreState();
 
         /**
@@ -881,6 +881,7 @@ public function run() : void
 
         /** Handle request to run an action */
         $this->engine->on('action', function(Action $action) {
+            //@TODO Implement queue to rate limit execution of actions
             $actionName = $action->getCmd();
             if (isset($this->actionConfig[$actionName]))
             {