Avoid double evaluation on Blade components

filamentphp · Feb 1, 2025 · 18fe308 · 18fe308
1 parent ac809c2
commit 18fe308
Show file tree

Hide file tree

Showing 3 changed files with 156 additions and 61 deletions.
diff --git a/packages/actions/resources/views/action-modal.blade.php b/packages/actions/resources/views/action-modal.blade.php
@@ -1,23 +1,44 @@
 <form wire:submit.prevent="callMountedAction">
+    @php
+        $actionModalAlignment = $action->getModalAlignment();
+        $actionIsModalAutofocused = $action->isModalAutofocused();
+        $actionHasModalCloseButton = $action->hasModalCloseButton();
+        $actionIsModalClosedByClickingAway = $action->isModalClosedByClickingAway();
+        $actionIsModalClosedByEscaping = $action->isModalClosedByEscaping();
+        $actionModalDescription = $action->getModalDescription();
+        $actionExtraModalWindowAttributeBag = $action->getExtraModalWindowAttributeBag();
+        $actionModalFooterActions = $action->getVisibleModalFooterActions();
+        $actionModalFooterActionsAlignment = $action->getModalFooterActionsAlignment();
+        $actionModalHeading = $action->getModalHeading();
+        $actionModalIcon = $action->getModalIcon();
+        $actionModalIconColor = $action->getModalIconColor();
+        $actionModalId = "fi-{$this->getId()}-action-{$action->getNestingIndex()}";
+        $actionIsModalSlideOver = $action->isModalSlideOver();
+        $actionIsModalFooterSticky = $action->isModalFooterSticky();
+        $actionIsModalHeaderSticky = $action->isModalHeaderSticky();
+        $actionModalWidth = $action->getModalWidth();
+        $actionModalWireKey = "{$this->getId()}.actions.{$action->getName()}.modal";
+    @endphp
+
     <x-filament::modal
-        :alignment="$action->getModalAlignment()"
-        :autofocus="$action?->isModalAutofocused()"
-        :close-button="$action?->hasModalCloseButton()"
-        :close-by-clicking-away="$action?->isModalClosedByClickingAway()"
-        :close-by-escaping="$action?->isModalClosedByEscaping()"
-        :description="$action->getModalDescription()"
-        :extra-modal-window-attribute-bag="$action->getExtraModalWindowAttributeBag()"
-        :footer-actions="$action->getVisibleModalFooterActions()"
-        :footer-actions-alignment="$action->getModalFooterActionsAlignment()"
-        :heading="$action->getModalHeading()"
-        :icon="$action->getModalIcon()"
-        :icon-color="$action->getModalIconColor()"
-        :id="'fi-' . $this->getId() . '-action-' . $action->getNestingIndex()"
-        :slide-over="$action->isModalSlideOver()"
-        :sticky-footer="$action->isModalFooterSticky()"
-        :sticky-header="$action->isModalHeaderSticky()"
-        :width="$action->getModalWidth()"
-        :wire:key="$this->getId() . '.actions.' . $action->getName() . '.modal'"
+        :alignment="$actionModalAlignment"
+        :autofocus="$actionIsModalAutofocused"
+        :close-button="$actionHasModalCloseButton"
+        :close-by-clicking-away="$actionIsModalClosedByClickingAway"
+        :close-by-escaping="$actionIsModalClosedByEscaping"
+        :description="$actionModalDescription"
+        :extra-modal-window-attribute-bag="$actionExtraModalWindowAttributeBag"
+        :footer-actions="$actionModalFooterActions"
+        :footer-actions-alignment="$actionModalFooterActionsAlignment"
+        :heading="$actionModalHeading"
+        :icon="$actionModalIcon"
+        :icon-color="$actionModalIconColor"
+        :id="$actionModalId"
+        :slide-over="$actionIsModalSlideOver"
+        :sticky-footer="$actionIsModalFooterSticky"
+        :sticky-header="$actionIsModalHeaderSticky"
+        :width="$actionModalWidth"
+        :wire:key="$actionModalWireKey"
         x-on:modal-closed.stop="$wire.unmountAction(false)"
     >
         {{ $action->getModalContent() }}