skip commit for empty action config

chriskapp · chriskapp · commit 948580f092c6 · 2026-02-04T21:11:47.000+01:00
diff --git a/src/Service/Action/Committer.php b/src/Service/Action/Committer.php
@@ -38,8 +38,12 @@ public function __construct(
     ) {
     }
 
-    public function commit(int $actionId, string $config, UserContext $context): void
+    public function commit(int $actionId, ?string $config, UserContext $context): void
     {
+        if (empty($config)) {
+            return;
+        }
+
         $previousHash = $this->actionCommitTable->findPreviousHash($actionId);
 
         $now = LocalDateTime::now();

Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,12 @@ public function __construct(`
`38`	`38`	`) {`
`39`	`39`	`}`
`40`	`40`
`41`		`- public function commit(int $actionId, string $config, UserContext $context): void`
	`41`	`+ public function commit(int $actionId, ?string $config, UserContext $context): void`
`42`	`42`	`{`
	`43`	`+ if (empty($config)) {`
	`44`	`+ return;`
	`45`	`+ }`
	`46`	`+`
`43`	`47`	`$previousHash = $this->actionCommitTable->findPreviousHash($actionId);`
`44`	`48`
`45`	`49`	`$now = LocalDateTime::now();`