WIP Adds bulk transfer ability during bulk checkout of Assets

app/Events/AssetsTransferredInBulk.php

@@ -0,0 +1,31 @@
+<?php
+
+
+namespace App\Events;
+
+use App\Models\Asset;
+use App\Models\User;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Foundation\Events\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+use Illuminate\Support\Collection;
+
+class AssetsTransferredInBulk
+{
+    use Dispatchable, SerializesModels;
+
+    public function __construct(
+        public Collection $transferable,
+        public Model      $transferredTo,
+        public Model      $transferredFrom,
+        public User       $admin,
+        public string     $transferred_at,
+        public string     $expected_checkin,
+        public string     $note,
+
+        public array     $originalValues,
+    )
+    {
+    }
+
+}

app/Http/Controllers/Assets/BulkAssetsController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Assets;
 
+use App\Events\AssetsTransferredInBulk;
 use App\Helpers\Helper;
 use App\Http\Controllers\CheckInOutRequest;
 use App\Http\Controllers\Controller;
@@ -12,6 +13,7 @@
 use App\View\Label;
 use Carbon\Carbon;
 use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Context;
 use Illuminate\Support\Facades\Crypt;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Gate;
@@ -620,17 +622,19 @@ public function destroy(Request $request) : RedirectResponse
     public function showCheckout() : View
     {
         $this->authorize('checkout', Asset::class);
-
         $alreadyAssigned = collect();
 
-        if (old('selected_assets') && is_array(old('selected_assets'))) {
-            $assets = Asset::findMany(old('selected_assets'));
+        if (!Setting::getSettings()->allow_bulk_asset_transfer) {
 
-            [$assignable, $alreadyAssigned] = $assets->partition(function (Asset $asset) {
-                return !$asset->assigned_to;
-            });
+            if (old('selected_assets') && is_array(old('selected_assets'))) {
+                $assets = Asset::findMany(old('selected_assets'));
 
-            session()->flashInput(['selected_assets' => $assignable->pluck('id')->values()->toArray()]);
+                [$assignable, $alreadyAssigned] = $assets->partition(function (Asset $asset) {
+                    return !$asset->assigned_to;
+                });
+
+                session()->flashInput(['selected_assets' => $assignable->pluck('id')->values()->toArray()]);
+            }
         }
 
         $do_not_change = ['' => trans('general.do_not_change')];
@@ -649,6 +653,7 @@ public function storeCheckout(AssetCheckoutRequest $request) : RedirectResponse
     {
         $this->authorize('checkout', Asset::class);
 
+
         try {
             $admin = auth()->user();
 
@@ -663,8 +668,9 @@ public function storeCheckout(AssetCheckoutRequest $request) : RedirectResponse
 
             $assets = Asset::findOrFail($asset_ids);
 
-            // Prevent checking out assets that are already checked out
-            if ($assets->pluck('assigned_to')->unique()->filter()->isNotEmpty()) {
+            [$alreadyAssigned, $unassigned] = $assets->collect()->partition(fn ($asset) => !is_null($asset->assigned_to));
+
+            if ($unassigned->pluck('assigned_to')->unique()->filter()->isNotEmpty()){
                 // re-add the asset ids so the assets select is re-populated
                 $request->session()->flashInput(['selected_assets' => $asset_ids]);
 
@@ -706,8 +712,8 @@ public function storeCheckout(AssetCheckoutRequest $request) : RedirectResponse
             }
 
             $errors = [];
-            DB::transaction(function () use ($target, $admin, $checkout_at, $expected_checkin, &$errors, $assets, $request) { //NOTE: $errors is passsed by reference!
-                foreach ($assets as $asset) {
+            DB::transaction(function () use ($target, $admin, $checkout_at, $expected_checkin, &$errors, $unassigned, $alreadyAssigned, $request) { //NOTE: $errors is passsed by reference!
+                foreach ($unassigned as $asset) {
                     $this->authorize('checkout', $asset);
 
                     // See if there is a status label passed
@@ -730,6 +736,32 @@ public function storeCheckout(AssetCheckoutRequest $request) : RedirectResponse
                         $errors = array_merge_recursive($errors, $asset->getErrors()->toArray());
                     }
                 }
+                if ($alreadyAssigned->isNotEmpty() && Setting::getSettings()->allow_bulk_asset_transfer) {
+                    Context::add('action', 'bulk_transfer');
+                    foreach ($alreadyAssigned as $asset) {
+                        $this->authorize('checkout', $asset);
+                        $transferredFrom = $asset->assignedTo;
+                        // See if there is a status label passed
+                        if ($request->filled('status_id')) {
+                            $asset->status_id = $request->get('status_id');
+                        }
+
+                        $checkout_success = $asset->transfer($alreadyAssigned, $target, $transferredFrom, $admin, $checkout_at, $expected_checkin, e($request->get('note')), $asset->name, null);
+
+                        //TODO - I think this logic is duplicated in the checkOut method?
+                        if ($target->location_id != '') {
+                            $asset->location_id = $target->location_id;
+                            // TODO - I don't know why this is being saved without events
+                            $asset::withoutEvents(function () use ($asset) {
+                                $asset->save();
+                            });
+                        }
+
+                        if (!$checkout_success) {
+                            $errors = array_merge_recursive($errors, $asset->getErrors()->toArray());
+                        }
+                    }
+                }
             });
 
             if (! $errors) {

app/Http/Controllers/SettingsController.php

@@ -352,6 +352,7 @@ public function postSettings(Request $request) : RedirectResponse
         $setting->dash_chart_type = $request->input('dash_chart_type');
         $setting->profile_edit = $request->input('profile_edit', 0);
         $setting->require_checkinout_notes = $request->input('require_checkinout_notes', 0);
+        $setting->allow_bulk_asset_transfer = $request->input('allow_bulk_asset_transfer', 0);
         $setting->manager_view_enabled = $request->input('manager_view_enabled', 0);
 
 

app/Listeners/CheckoutableListener.php

@@ -33,6 +33,7 @@
 use App\Notifications\CheckoutLicenseSeatNotification;
 use GuzzleHttp\Exception\ClientException;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Support\Facades\Context;
 use Illuminate\Support\Facades\Mail;
 use Illuminate\Support\Facades\Notification;
 use Exception;
@@ -434,6 +435,10 @@ private function shouldSendCheckoutEmailToUser(Model $checkoutable): bool
          * 3. The item should send an email at check-in/check-out
          */
 
+        if (Context::get('action') === 'transfer') {
+            return true;
+        }
+
         if ($checkoutable->requireAcceptance()) {
             return true;
         }

app/Listeners/LogListener.php

@@ -6,6 +6,7 @@
 use App\Events\AccessoryCheckedOut;
 use App\Events\AssetCheckedIn;
 use App\Events\AssetCheckedOut;
+use App\Events\AssetsTransferredInBulk;
 use App\Events\CheckoutableCheckedIn;
 use App\Events\CheckoutableCheckedOut;
 use App\Events\CheckoutAccepted;
@@ -75,6 +76,15 @@ public function onCheckoutAccepted(CheckoutAccepted $event)
         $logaction->save();
     }
 
+    /**
+     * @throws \Exception
+     */
+    public function onAssetsTransferredInBulk(AssetsTransferredInBulk $event): void
+    {
+        Log::debug('event passed to the listener:');
+        $event->transferable->logCheckout($event->note, $event->transferredTo, $event->transferable->last_checkout, $event->originalValues, true);
+
+    }
     public function onCheckoutDeclined(CheckoutDeclined $event)
     {
         $logaction = new Actionlog();
@@ -140,6 +150,7 @@ public function subscribe($events)
         $list = [
             'CheckoutableCheckedIn',
             'CheckoutableCheckedOut',
+            'AssetsTransferredInBulk',
             'CheckoutAccepted',
             'CheckoutDeclined',
             'UserMerged',