Display as form for actions

fracsi · javiereguiluz · commit bb11a7576b81 · 2025-02-20T20:44:39.000+01:00
diff --git a/assets/css/easyadmin-theme/base.css b/assets/css/easyadmin-theme/base.css
@@ -575,7 +575,7 @@ a.user-menu-wrapper .user-details:hover {
 }
 .content-header .page-actions:empty { display: none; }
 
-.content-header .page-actions .btn + .btn {
+.content-header .page-actions :is(.btn,form:has(.btn)) + :is(.btn,form:has(.btn)) {
     margin-inline-start: 10px;
 }
 
diff --git a/src/Config/Action.php b/src/Config/Action.php
@@ -158,6 +158,13 @@ public function displayAsButton(): self
         return $this;
     }
 
+    public function displayAsForm(): self
+    {
+        $this->dto->setHtmlElement('form');
+
+        return $this;
+    }
+
     public function setHtmlAttributes(array $attributes): self
     {
         $this->dto->setHtmlAttributes($attributes);
diff --git a/src/Dto/ActionDto.php b/src/Dto/ActionDto.php
@@ -293,6 +293,8 @@ public function getAsConfigObject(): Action
 
         if ('a' === $this->htmlElement) {
             $action->displayAsLink();
+        } elseif ('form' === $this->htmlElement) {
+            $action->displayAsForm();
         } else {
             $action->displayAsButton();
         }
diff --git a/templates/crud/action.html.twig b/templates/crud/action.html.twig
@@ -15,4 +15,15 @@
             {%- if action.label is not empty -%}<span class="action-label">{{ action.label|trans|raw }}</span>{%- endif -%}
         </span>
     </button>
+{% elseif 'form' == action.htmlElement %}
+    {%- set form_id = 'form-' ~ action.name ~ '-' ~ random() -%}
+    {%- do action.setHtmlAttribute('form', form_id) -%}
+    <form action="{{ action.linkUrl }}" method="POST" id="{{ form_id }}">
+        <button class="{{ action.cssClass }}" {% for name, value in action.htmlAttributes %}{{ name }}="{{ (value.trans is defined ? value|trans : value)|e('html') }}" {% endfor %}>
+            <span class="btn-label">
+                {%- if action.icon %}<twig:ea:Icon name="{{ action.icon }}"/> {% endif -%}
+                {%- if action.label is not empty -%}<span class="action-label">{{ action.label|trans|raw }}</span>{%- endif -%}
+            </span>
+        </button>
+    </form>
 {% endif %}
diff --git a/tests/Controller/ActionsCrudControllerTest.php b/tests/Controller/ActionsCrudControllerTest.php
@@ -52,4 +52,13 @@ public function testDynamicLabels()
         static::assertSame('Action 7: Category 0', $crawler->filter('a.dropdown-item[data-action-name="action7"]')->text());
         static::assertSame('Reset', $crawler->filter('a.dropdown-item[data-action-name="action8"]')->text());
     }
+
+    public function testFormAction()
+    {
+        $crawler = $this->client->request('GET', $this->generateIndexUrl());
+
+        static::assertCount(1, $crawler->filter('form[id^="form-action9-"]'));
+        static::assertCount(1, $crawler->filter('form[id^="form-action9-"] > .btn'));
+        static::assertSame('POST', $crawler->filter('form[id^="form-action9-"]')->attr('method'));
+    }
 }
diff --git a/tests/TestApplication/src/Controller/ActionsCrudController.php b/tests/TestApplication/src/Controller/ActionsCrudController.php
@@ -43,6 +43,8 @@ public function configureActions(Actions $actions): Actions
         // this tests that the 'Reset' label is interpreted as a string and not as a callable to the PHP reset() function
         $action8 = Action::new('action8')->linkToCrudAction('')->setLabel('Reset');
 
+        $action9 = Action::new('action9')->linkToCrudAction('')->createAsGlobalAction()->displayAsForm();
+
         return $actions
             ->add(Crud::PAGE_INDEX, $action1)
             ->add(Crud::PAGE_INDEX, $action2)
@@ -52,6 +54,7 @@ public function configureActions(Actions $actions): Actions
             ->add(Crud::PAGE_INDEX, $action6)
             ->add(Crud::PAGE_INDEX, $action7)
             ->add(Crud::PAGE_INDEX, $action8)
+            ->add(Crud::PAGE_INDEX, $action9)
             ->update(Crud::PAGE_INDEX, Action::NEW, function (Action $action) {
                 return $action->setIcon('fa fa-fw fa-plus')->setLabel(false);
             })

Original file line number	Diff line number	Diff line change
`@@ -575,7 +575,7 @@ a.user-menu-wrapper .user-details:hover {`
`575`	`575`	`}`
`576`	`576`	`.content-header .page-actions:empty { display: none; }`
`577`	`577`
`578`		`-.content-header .page-actions .btn + .btn {`
	`578`	`+.content-header .page-actions :is(.btn,form:has(.btn)) + :is(.btn,form:has(.btn)) {`
`579`	`579`	`margin-inline-start: 10px;`
`580`	`580`	`}`
`581`	`581`
Original file line number	Diff line number	Diff line change
`@@ -158,6 +158,13 @@ public function displayAsButton(): self`
`158`	`158`	`return $this;`
`159`	`159`	`}`
`160`	`160`
	`161`	`+ public function displayAsForm(): self`
	`162`	`+ {`
	`163`	`+ $this->dto->setHtmlElement('form');`
	`164`	`+`
	`165`	`+ return $this;`
	`166`	`+ }`
	`167`	`+`
`161`	`168`	`public function setHtmlAttributes(array $attributes): self`
`162`	`169`	`{`
`163`	`170`	`$this->dto->setHtmlAttributes($attributes);`
Original file line number	Diff line number	Diff line change
`@@ -293,6 +293,8 @@ public function getAsConfigObject(): Action`
`293`	`293`
`294`	`294`	`if ('a' === $this->htmlElement) {`
`295`	`295`	`$action->displayAsLink();`
	`296`	`+ } elseif ('form' === $this->htmlElement) {`
	`297`	`+ $action->displayAsForm();`
`296`	`298`	`} else {`
`297`	`299`	`$action->displayAsButton();`
`298`	`300`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,4 +52,13 @@ public function testDynamicLabels()`
`52`	`52`	`static::assertSame('Action 7: Category 0', $crawler->filter('a.dropdown-item[data-action-name="action7"]')->text());`
`53`	`53`	`static::assertSame('Reset', $crawler->filter('a.dropdown-item[data-action-name="action8"]')->text());`
`54`	`54`	`}`
	`55`	`+`
	`56`	`+ public function testFormAction()`
	`57`	`+ {`
	`58`	`+ $crawler = $this->client->request('GET', $this->generateIndexUrl());`
	`59`	`+`
	`60`	`+ static::assertCount(1, $crawler->filter('form[id^="form-action9-"]'));`
	`61`	`+ static::assertCount(1, $crawler->filter('form[id^="form-action9-"] > .btn'));`
	`62`	`+ static::assertSame('POST', $crawler->filter('form[id^="form-action9-"]')->attr('method'));`
	`63`	`+ }`
`55`	`64`	`}`