From ca4ddd547bcd57a5b7f0b763fcdbcc1ee622f4e3 Mon Sep 17 00:00:00 2001
From: Joe <104938042+lrljoe@users.noreply.github.com>
Date: Wed, 25 Oct 2023 17:00:51 +0100
Subject: [PATCH] V3 Develop to V3 Master (#1439)

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>

* v3 - Adding CollapseAlways Option for Columns (#1403)

* Add CollapseAlways & Fix ReorderColumn

* Remove superfluous md:hidden from the icon

* Tweaks for Column Collapsing For BS4/BS5

* Fix Collapsed on Bootstrap

* Tweak Bootstrap Theme

---------

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>

* Minor tweaks to blades for reorder cols (#1411)

* V3 - Fix Reorder For Bootstrap (#1412)

* Minor tweaks to blades for reorder cols

* Bootstrap Reorder Toggle

* Add DiscordBot (#1413)

* Docs livewire namespace fix (#1420)

* Fixed Livewire namespace in docs

* Updated livewire url in docs

* V3  - Fix Localisation (#1424)

* Fix Localisations

* Fix styling

* Load Original and Publish Localisation Paths

* Fix styling

---------

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>

* v3 - Add Loading Placeholder (#1421)

* Initial Loading Placeholder Code, Use Hourglass Spinner

* Add wrappers, Class Merging, Colouring

* Renaming Traits - Adding LoadingPlaceholder Tests, Add Placeholder Test Table

* Update Docs for Placeholder

---------

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>

* v3 - Column - Allow Enum in ReturnType for Column getContent/renderContent (#1436)

* Fix Enum Column Return TypeHints

* v3 - Add "Confirmation" Option to Bulk Actions (#1437)

* Add BulkActionConfirms

* Add wire:confirm capability to bulk actions

* Update Changelog

* Fix error with typehinting

* Add Tests

* Adding default confirmation message customisation

* Update Default Message

* Update Tests

---------

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>

---------

Co-authored-by: lrljoe <lrljoe@users.noreply.github.com>
Co-authored-by: Anthony Rappa <rappa819@gmail.com>
---
 CHANGELOG.md                                  |  9 ++-
 docs/bulk-actions/available-methods.md        | 51 ++++++++++++++++
 resources/lang/en.json                        |  1 +
 .../tools/toolbar/bootstrap.blade.php         |  3 +
 .../tools/toolbar/tailwind.blade.php          |  3 +
 .../BulkActionsConfiguration.php              | 34 +++++++++++
 src/Traits/Helpers/BulkActionsHelpers.php     | 22 ++++++-
 src/Traits/WithBulkActions.php                | 10 ++--
 src/Views/Traits/Helpers/ColumnHelpers.php    |  4 +-
 .../BulkActionsConfigurationTest.php          | 59 +++++++++++++++++++
 .../Traits/Helpers/BulkActionsHelpersTest.php | 24 ++++++++
 11 files changed, 210 insertions(+), 10 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ddd8715db..abfd018ba 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,7 +2,14 @@
 
 All notable changes to `laravel-livewire-tables` will be documented in this file
 
-## [3.0.0-beta.4] - 2023-10-17
+## [3.0.0-beta.4] - 2023-10-25
+- Fix Return Type hinting for Column Rendering to allow Enum columns
+- Add Bulk Action Confirmations, using wire:confirm
+  - setBulkActionConfirms
+  - setBulkActionConfirmMessage
+  - setBulkActionConfirmMessages 
+  - setBulkActionDefaultConfirmationMessage
+  - Localisation for confirmation message
 - Introduction of Loading Placeholder
 - Docs livewire namespace fix [Here](https://github.com/rappasoft/laravel-livewire-tables/pull/1420)
 - Add CollapseAlways capability for Columns
diff --git a/docs/bulk-actions/available-methods.md b/docs/bulk-actions/available-methods.md
index 245e99a2b..41507dc24 100644
--- a/docs/bulk-actions/available-methods.md
+++ b/docs/bulk-actions/available-methods.md
@@ -133,3 +133,54 @@ public function configure(): void
     $this->setHideBulkActionsWhenEmptyDisabled();
 }
 ```
+
+## setBulkActionConfirms
+
+When a bulk action is included in the array passed to setBulkActionConfirms, the default wire:confirm pop-up will appear prior to executing the bulk action.  The default message is: "Are you sure?".  This should only be used if you wish to use the default message.
+
+```php
+public function configure(): void
+{
+    $this->setBulkActionConfirms([
+        'delete',
+        'reset'
+    ]);
+}
+```
+## setBulkActionDefaultConfirmationMessage
+
+You may use this method to over-ride the default message.  To override the confirmation message for an individual Bulk Action, see the below setBulkActionConfirmMessage and setBulkActionConfirmMessages.  You may also use the language files to do this.
+
+```php
+public function configure(): void
+{
+    $this->setBulkActionDefaultConfirmationMessage('Are you certain?');
+}
+```
+
+## setBulkActionConfirmMessage
+
+You may use this method to specify a message other than the default message.
+
+```php
+public function configure(): void
+{
+    $this->setBulkActionConfirmMessage('delete', 'Do you want to delete these items?');
+}
+```
+
+## setBulkActionConfirmMessages
+
+You may pass an array to this method, to more effectively update the confirmation message for a larger quantity of bulk actions.  This expects an array keyed by the bulk action name, with the value being the message that will be displayed to the user.
+
+```php
+public function configure(): void
+{
+    $this->setBulkActionConfirmMessages([
+        'delete' => 'Are you sure you want to delete these items?',
+        'purge' => 'Are you sure you want to purge these items?',
+        'reassign' => 'This will reassign selected items, are you sure?',
+    ]);
+}
+```
+
diff --git a/resources/lang/en.json b/resources/lang/en.json
index 0456f57e2..87b34925e 100644
--- a/resources/lang/en.json
+++ b/resources/lang/en.json
@@ -4,6 +4,7 @@
     "Applied Filters": "Applied Filters",
     "Applied Sorting": "Applied Sorting",
     "Bulk Actions": "Bulk Actions",
+    "Bulk Actions Confirm": "Are you sure?",
     "Clear": "Clear",
     "Columns": "Columns",
     "Debugging Values": "Debugging Values",
diff --git a/resources/views/components/tools/toolbar/bootstrap.blade.php b/resources/views/components/tools/toolbar/bootstrap.blade.php
index 2cb4e0b98..ca8e516c8 100644
--- a/resources/views/components/tools/toolbar/bootstrap.blade.php
+++ b/resources/views/components/tools/toolbar/bootstrap.blade.php
@@ -220,6 +220,9 @@
                             <a
                                 href="#"
                                 wire:click="{{ $action }}"
+                                @if($component->hasConfirmationMessage($action))
+                                    wire:confirm="{{ $component->getBulkActionConfirmMessage($action) }}"
+                                @endif
                                 wire:key="{{ $tableName }}-bulk-action-{{ $action }}"
                                 @class([
                                     'dropdown-item' => $component->isBootstrap(),
diff --git a/resources/views/components/tools/toolbar/tailwind.blade.php b/resources/views/components/tools/toolbar/tailwind.blade.php
index 51a3e4ab0..7daa59c67 100644
--- a/resources/views/components/tools/toolbar/tailwind.blade.php
+++ b/resources/views/components/tools/toolbar/tailwind.blade.php
@@ -186,6 +186,9 @@ class="origin-top-right absolute right-0 mt-2 w-full md:w-48 rounded-md shadow-l
                                 @foreach ($component->getBulkActions() as $action => $title)
                                     <button
                                         wire:click="{{ $action }}"
+                                        @if($component->hasConfirmationMessage($action))
+                                            wire:confirm="{{ $component->getBulkActionConfirmMessage($action) }}"
+                                        @endif
                                         wire:key="{{ $tableName }}-bulk-action-{{ $action }}"
                                         type="button"
                                         class="block w-full px-4 py-2 text-sm leading-5 text-gray-700 hover:bg-gray-100 hover:text-gray-900 focus:outline-none focus:bg-gray-100 focus:text-gray-900 flex items-center space-x-2 dark:text-white dark:hover:bg-gray-600"
diff --git a/src/Traits/Configuration/BulkActionsConfiguration.php b/src/Traits/Configuration/BulkActionsConfiguration.php
index eedfd93c7..0e4375045 100644
--- a/src/Traits/Configuration/BulkActionsConfiguration.php
+++ b/src/Traits/Configuration/BulkActionsConfiguration.php
@@ -76,4 +76,38 @@ public function setHideBulkActionsWhenEmptyDisabled(): self
 
         return $this;
     }
+
+    public function setBulkActionConfirms(array $bulkActionConfirms): self
+    {
+        foreach ($bulkActionConfirms as $bulkAction) {
+            if (! $this->hasConfirmationMessage($bulkAction)) {
+                $this->setBulkActionConfirmMessage($bulkAction, $this->getBulkActionDefaultConfirmationMessage());
+            }
+        }
+
+        return $this;
+    }
+
+    public function setBulkActionConfirmMessage(string $action, string $confirmationMessage): self
+    {
+        $this->bulkActionConfirms[$action] = $confirmationMessage;
+
+        return $this;
+    }
+
+    public function setBulkActionConfirmMessages(array $bulkActionMessages): self
+    {
+        foreach ($bulkActionMessages as $bulkAction => $confirmationMessage) {
+            $this->setBulkActionConfirmMessage($bulkAction, $confirmationMessage);
+        }
+
+        return $this;
+    }
+
+    public function setBulkActionDefaultConfirmationMessage(string $defaultConfirmationMessage): self
+    {
+        $this->bulkActionConfirmDefaultMessage = $defaultConfirmationMessage;
+
+        return $this;
+    }
 }
diff --git a/src/Traits/Helpers/BulkActionsHelpers.php b/src/Traits/Helpers/BulkActionsHelpers.php
index 629265a04..b488146a8 100644
--- a/src/Traits/Helpers/BulkActionsHelpers.php
+++ b/src/Traits/Helpers/BulkActionsHelpers.php
@@ -146,12 +146,30 @@ public function setAllSelected(): void
     {
         $this->setSelectAllEnabled();
         $this->setSelected((clone $this->baseQuery())->pluck($this->getBuilder()->getModel()->getTable().'.'.$this->getPrimaryKey())->map(fn ($item) => (string) $item)->toArray());
-
-        //$this->setSelected((clone $this->baseQuery())->pluck($this->getPrimaryKey())->map(fn ($item) => (string) $item)->toArray());
     }
 
     public function showBulkActionsDropdownAlpine(): bool
     {
         return $this->bulkActionsAreEnabled() && $this->hasBulkActions();
     }
+
+    public function getBulkActionConfirms(): array
+    {
+        return array_keys($this->bulkActionConfirms);
+    }
+
+    public function hasConfirmationMessage(string $bulkAction): bool
+    {
+        return isset($this->bulkActionConfirms[$bulkAction]);
+    }
+
+    public function getBulkActionConfirmMessage(string $bulkAction): string
+    {
+        return $this->bulkActionConfirms[$bulkAction] ?? $this->getBulkActionDefaultConfirmationMessage();
+    }
+
+    public function getBulkActionDefaultConfirmationMessage(): string
+    {
+        return isset($this->bulkActionConfirmDefaultMessage) ? $this->bulkActionConfirmDefaultMessage : __('Bulk Actions Confirm');
+    }
 }
diff --git a/src/Traits/WithBulkActions.php b/src/Traits/WithBulkActions.php
index 391f8d3f6..177e225f5 100644
--- a/src/Traits/WithBulkActions.php
+++ b/src/Traits/WithBulkActions.php
@@ -16,16 +16,16 @@ trait WithBulkActions
 
     public array $bulkActions = [];
 
+    public array $bulkActionConfirms = [];
+
     public array $selected = [];
 
     public bool $hideBulkActionsWhenEmpty = false;
 
+    public ?string $bulkActionConfirmDefaultMessage;
+
     public function bulkActions(): array
     {
-        if (property_exists($this, 'bulkActions')) {
-            return $this->bulkActions;
-        }
-
-        return [];
+        return property_exists($this, 'bulkActions') ? $this->bulkActions : [];
     }
 }
diff --git a/src/Views/Traits/Helpers/ColumnHelpers.php b/src/Views/Traits/Helpers/ColumnHelpers.php
index 2f5069431..2aeefb6a5 100644
--- a/src/Views/Traits/Helpers/ColumnHelpers.php
+++ b/src/Views/Traits/Helpers/ColumnHelpers.php
@@ -93,7 +93,7 @@ public function getColumnSelectName(): ?string
     }
 
     // TODO: Test
-    public function renderContents(Model $row): null|string|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
+    public function renderContents(Model $row): null|string|\BackedEnum|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     {
         if ($this->shouldCollapseOnMobile() && $this->shouldCollapseOnTablet()) {
             throw new DataTableConfigurationException('You should only specify a columns should collapse on mobile OR tablet, not both.');
@@ -103,7 +103,7 @@ public function renderContents(Model $row): null|string|HtmlString|DataTableConf
     }
 
     // TODO: Test
-    public function getContents(Model $row): null|string|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
+    public function getContents(Model $row): null|string|\BackedEnum|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     {
         if ($this->isLabel()) {
             $value = call_user_func($this->getLabelCallback(), $row, $this);
diff --git a/tests/Traits/Configuration/BulkActionsConfigurationTest.php b/tests/Traits/Configuration/BulkActionsConfigurationTest.php
index eb4091855..834c5932a 100644
--- a/tests/Traits/Configuration/BulkActionsConfigurationTest.php
+++ b/tests/Traits/Configuration/BulkActionsConfigurationTest.php
@@ -91,4 +91,63 @@ public function can_set_bulk_actions(): void
 
         $this->assertTrue($this->basicTable->hasBulkActions());
     }
+
+    /** @test */
+    public function can_set_bulk_action_confirms(): void
+    {
+        $this->assertSame([], $this->basicTable->getBulkActionConfirms());
+
+        $this->basicTable->setBulkActionConfirms(['deactivate', 'delete']);
+
+        $this->assertSame(['deactivate', 'delete'], $this->basicTable->getBulkActionConfirms());
+
+    }
+
+    /** @test */
+    public function can_set_bulk_action_custom_message(): void
+    {
+        $this->basicTable->setBulkActionConfirms(['deactivate', 'delete']);
+
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('deactivate'));
+
+        $this->basicTable->setBulkActionConfirmMessage('deactivate', 'do you want to deactivate?');
+
+        $this->assertSame('do you want to deactivate?', $this->basicTable->getBulkActionConfirmMessage('deactivate'));
+
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('delete'));
+
+    }
+
+    /** @test */
+    public function can_set_bulk_action_custom_messages(): void
+    {
+        $this->basicTable->setBulkActionConfirms(['purge', 'delete', 'reassign', 'deactivate']);
+
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('deactivate'));
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('reassign'));
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('delete'));
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('purge'));
+
+        $this->basicTable->setBulkActionConfirmMessages([
+            'delete' => 'Are you sure you want to delete these items?',
+            'purge' => 'Are you sure you want to purge these items?',
+            'reassign' => 'This will reassign selected items, are you sure?',
+        ]);
+
+        $this->assertSame('Are you sure you want to delete these items?', $this->basicTable->getBulkActionConfirmMessage('delete'));
+        $this->assertSame('This will reassign selected items, are you sure?', $this->basicTable->getBulkActionConfirmMessage('reassign'));
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('deactivate'));
+        $this->assertSame('Are you sure you want to purge these items?', $this->basicTable->getBulkActionConfirmMessage('purge'));
+    }
+
+    /** @test */
+    public function can_set_bulk_action_default_confirmation_message(): void
+    {
+        $this->assertSame('Are you sure?', $this->basicTable->getBulkActionDefaultConfirmationMessage());
+
+        $this->basicTable->setBulkActionDefaultConfirmationMessage('Test Default Message');
+
+        $this->assertSame('Test Default Message', $this->basicTable->getBulkActionDefaultConfirmationMessage());
+
+    }
 }
diff --git a/tests/Traits/Helpers/BulkActionsHelpersTest.php b/tests/Traits/Helpers/BulkActionsHelpersTest.php
index dd00bd591..15d284950 100644
--- a/tests/Traits/Helpers/BulkActionsHelpersTest.php
+++ b/tests/Traits/Helpers/BulkActionsHelpersTest.php
@@ -180,4 +180,28 @@ public function set_select_all_selects_all(): void
 
         $this->assertSame(['1', '2', '3', '4', '5'], $this->basicTable->getSelected());
     }
+
+    /** @test */
+    public function can_get_bulk_action_confirms(): void
+    {
+        $this->assertSame([], $this->basicTable->getBulkActionConfirms());
+    }
+
+    /** @test */
+    public function can_find_if_bulk_action_has_confirm_message(): void
+    {
+        $this->assertFalse($this->basicTable->hasConfirmationMessage('test123'));
+    }
+
+    /** @test */
+    public function bulk_action_confirm_returns_default_message_if_not_set(): void
+    {
+        $this->assertSame($this->basicTable->getBulkActionDefaultConfirmationMessage(), $this->basicTable->getBulkActionConfirmMessage('test'));
+    }
+
+    /** @test */
+    public function can_get_bulk_action_default_confirmation_message(): void
+    {
+        $this->assertSame('Are you sure?', $this->basicTable->getBulkActionDefaultConfirmationMessage());
+    }
 }