opensource/filament
3
0
Fork 0
mirror of https://github.com/filamentphp/filament.git synced 2026-09-24 15:42:09 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: Merge action arguments without overwriting

Browse Source
This commit is contained in:
Dan Harrin
2025-09-16 16:50:03 +01:00
parent 34ed27f392
commit 67e29cfaec
2 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/actions/src/Concerns/HasArguments.php
+12 -5
View File
@@ -29,12 +29,19 @@ trait HasArguments
/**
* @param array<string, mixed> $arguments
*/
public function mergeArguments(array $arguments): static
public function mergeArguments(array $arguments, bool $shouldOverwriteExistingArguments = true): static
{
$this->arguments = [
...$this->arguments ?? [],
...$arguments,
];
if ($shouldOverwriteExistingArguments) {
$this->arguments = [
...$this->arguments ?? [],
...$arguments,
];
} else {
$this->arguments = [
...$arguments,
...$this->arguments ?? [],
];
}
return $this;
}
packages/actions/src/Concerns/InteractsWithActions.php
+1 -1
View File
@@ -495,7 +495,7 @@ trait InteractsWithActions
continue;
}
$resolvedAction->mergeArguments($action['arguments'] ?? []);
$resolvedAction->mergeArguments($action['arguments'] ?? [], shouldOverwriteExistingArguments: false);
$resolvedAction->nestingIndex($actionNestingIndex);
$resolvedAction->boot();