Re-enable batch delete if delete is re-enabled and re-enable delete i…

…f batch delete is re-enabled

src/Config/Actions.php

@@ -134,6 +134,13 @@ public function disable(string ...$disabledActionNames): self
 
     public function enable(string ...$enabledActionNames): self
     {
+        // if 'delete' or 'batch delete' is enabled, both are enabled automatically.
+        // This is the most common case, but user can re-disable the action if needed manually
+        if (\in_array(Action::DELETE, $enabledActionNames, true) || \in_array(Action::BATCH_DELETE, $enabledActionNames, true)) {
+            $enabledActionNames[] = Action::DELETE;
+            $enabledActionNames[] = Action::BATCH_DELETE;
+        }
+
         $this->dto->enableActions($enabledActionNames);
 
         return $this;

tests/Config/ActionsTest.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace EasyCorp\Bundle\EasyAdminBundle\Tests\Config;
+
+use EasyCorp\Bundle\EasyAdminBundle\Config\Action;
+use EasyCorp\Bundle\EasyAdminBundle\Config\Actions;
+use PHPUnit\Framework\TestCase;
+
+class ActionsTest extends TestCase
+{
+    public function testEnableDeleteAction(): void
+    {
+        $actions = Actions::new();
+        $actions->disable(Action::DELETE);
+
+        $this->assertSame(['delete', 'batchDelete'], $actions->getAsDto(null)->getDisabledActions());
+
+        $actions->enable(Action::DELETE);
+        $this->assertSame([], $actions->getAsDto(null)->getDisabledActions());
+    }
+
+    public function testEnableBatchDeleteAction(): void
+    {
+        $actions = Actions::new();
+        $actions->disable(Action::BATCH_DELETE);
+
+        $this->assertSame(['batchDelete'], $actions->getAsDto(null)->getDisabledActions());
+
+        $actions->enable(Action::BATCH_DELETE);
+        $this->assertSame([], $actions->getAsDto(null)->getDisabledActions());
+    }
+
+    public function testEnableBatchDeleteActionWillEnableDeleteAsWell(): void
+    {
+        $actions = Actions::new();
+        $actions->disable(Action::DELETE, Action::BATCH_DELETE);
+
+        $this->assertSame(['delete', 'batchDelete'], $actions->getAsDto(null)->getDisabledActions());
+
+        $actions->enable(Action::BATCH_DELETE);
+        $this->assertSame([], $actions->getAsDto(null)->getDisabledActions());
+    }
+}