Merge pull request #93 from keboola/adamvyborny-allow-to-reset-legacy-primary-keys

AdamVyborny · web-flow · commit 174723f7c426 · 2024-09-27T10:42:17.000+02:00
fix: allow to reset legacyPrimaryKeys by adding null type to setter
diff --git a/src/Manifest/ManifestManager/Options/OutTable/ManifestOptions.php b/src/Manifest/ManifestManager/Options/OutTable/ManifestOptions.php
@@ -387,7 +387,7 @@ public function setDeleteWhereOperator(?string $deleteWhereOperator): ManifestOp
         return $this;
     }
 
-    public function setLegacyPrimaryKeys(array $primaryKey): ManifestOptions
+    public function setLegacyPrimaryKeys(?array $primaryKey): ManifestOptions
     {
         $this->legacyPrimaryKeys = $primaryKey;
         return $this;

Original file line number	Diff line number	Diff line change
`@@ -387,7 +387,7 @@ public function setDeleteWhereOperator(?string $deleteWhereOperator): ManifestOp`
`387`	`387`	`return $this;`
`388`	`388`	`}`
`389`	`389`
`390`		`- public function setLegacyPrimaryKeys(array $primaryKey): ManifestOptions`
	`390`	`+ public function setLegacyPrimaryKeys(?array $primaryKey): ManifestOptions`
`391`	`391`	`{`
`392`	`392`	`$this->legacyPrimaryKeys = $primaryKey;`
`393`	`393`	`return $this;`