[UPDATED] Simpler logging refactor

app/Actions/CheckoutRequests/CancelCheckoutRequestAction.php

@@ -2,6 +2,7 @@
 
 namespace App\Actions\CheckoutRequests;
 
+use App\Enums\ActionType;
 use App\Models\Actionlog;
 use App\Models\Asset;
 use App\Models\Company;
@@ -27,14 +28,10 @@ public static function run(Asset $asset, User $user)
         $data['item_quantity'] = 1;
         $settings = Setting::getSettings();
 
-        $logaction = new Actionlog();
-        $logaction->item_id = $data['asset_id'] = $asset->id;
-        $logaction->item_type = $data['item_type'] = Asset::class;
-        $logaction->created_at = $data['requested_date'] = date('Y-m-d H:i:s');
-        $logaction->target_id = $data['user_id'] = auth()->id();
-        $logaction->target_type = User::class;
-        $logaction->location_id = $user->location_id ?? null;
-        $logaction->logaction('request canceled');
+        $asset->setLogTarget(auth()->user());
+        $asset->setLogLocationOverride($user->location_id);
+        $asset->setLogAction(ActionType::RequestCanceled);
+        $asset->save();
 
         try {
             $settings->notify(new RequestAssetCancelation($data));

app/Actions/CheckoutRequests/CreateCheckoutRequestAction.php

@@ -2,6 +2,7 @@
 
 namespace App\Actions\CheckoutRequests;
 
+use App\Enums\ActionType;
 use App\Exceptions\AssetNotRequestable;
 use App\Models\Actionlog;
 use App\Models\Asset;
@@ -28,18 +29,15 @@ public static function run(Asset $asset, User $user): string
         }
 
         $data['item'] = $asset;
+        $data['item_type'] = Asset::class; // TODO - generalize?
         $data['target'] = $user;
         $data['item_quantity'] = 1;
         $settings = Setting::getSettings();
 
-        $logaction = new Actionlog();
-        $logaction->item_id = $data['asset_id'] = $asset->id;
-        $logaction->item_type = $data['item_type'] = Asset::class;
-        $logaction->created_at = $data['requested_date'] = date('Y-m-d H:i:s');
-        $logaction->target_id = $data['user_id'] = auth()->id();
-        $logaction->target_type = User::class;
-        $logaction->location_id = $user->location_id ?? null;
-        $logaction->logaction('requested');
+        $asset->setLogTarget(auth()->user());
+        $asset->setLogLocationOverride($user->location_id);
+        $asset->setLogAction(ActionType::Requested);
+        $asset->save();
 
         $asset->request();
         $asset->increment('requests_counter', 1);

app/Console/Commands/CheckinLicensesFromAllUsers.php

@@ -2,6 +2,7 @@
 
 namespace App\Console\Commands;
 
+use App\Enums\ActionType;
 use App\Models\License;
 use App\Models\LicenseSeat;
 use App\Models\User;
@@ -81,7 +82,11 @@ public function handle()
                 }
 
                 // Log the checkin
-                $seat->logCheckin($seat->user, 'Checked in via cli tool');
+                $seat->setLogTarget($seat->user);
+                $seat->setLogNote('Checked in via cli tool');
+                $seat->setLogAction(ActionType::CheckinFrom);
+                $seat->save(); //this is going to be a dual-save, do we want that?!
+//                $seat->logCheckin($seat->user, 'Checked in via cli tool');
             }
         }
     }

app/Enums/ActionType.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace App\Enums;
+
+enum ActionType: string {
+    case Restore = 'restore';
+    case TwoFactorReset = '2FA reset';
+    case CheckinFrom = 'checkin from';
+    case RequestCanceled = 'request canceled';
+    case Requested = 'requested';
+    case DeleteSeats = 'delete seats';
+    case AddSeats = 'add seats';
+    case Update = 'update';
+    case Create = 'create';
+    case Delete = 'delete';
+    case Uploaded = 'uploaded';
+    case NoteAdded = 'note added';
+    case Audit = 'audit';
+    case Checkout = 'checkout';
+}

app/Http/Controllers/Accessories/AccessoriesFilesController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Accessories;
 
+use App\Enums\ActionType;
 use App\Helpers\StorageHelper;
 use App\Http\Controllers\Controller;
 use App\Http\Requests\UploadFileRequest;
@@ -47,7 +48,10 @@ public function store(UploadFileRequest $request, $accessoryId = null) : Redirec
 
                     $file_name = $request->handleFile('private_uploads/accessories/', 'accessory-'.$accessory->id, $file);
                     //Log the upload to the log
-                    $accessory->logUpload($file_name, e($request->input('notes')));
+                    $accessory->setLogFilename($file_name);
+                    $accessory->setLogNote(e($request->input('notes')));
+                    $accessory->setLogAction(ActionType::Uploaded);
+                    $accessory->save();
                 }
 
 

app/Http/Controllers/Accessories/AccessoryCheckoutController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Accessories;
 
+use App\Enums\ActionType;
 use App\Events\CheckoutableCheckedOut;
 use App\Helpers\Helper;
 use App\Http\Controllers\CheckInOutRequest;
@@ -71,10 +72,10 @@ public function store(AccessoryCheckoutRequest $request, Accessory $accessory) :
         $this->authorize('checkout', $accessory);
 
         $target = $this->determineCheckoutTarget();
-        
-        $accessory->checkout_qty = $request->input('checkout_qty', 1);
-        
-        for ($i = 0; $i < $accessory->checkout_qty; $i++) {
+
+        $checkout_qty = $request->input('checkout_qty', 1);
+
+        for ($i = 0; $i < $checkout_qty; $i++) {
 
             $accessory_checkout = new AccessoryCheckout([
                 'accessory_id' => $accessory->id,
@@ -88,6 +89,11 @@ public function store(AccessoryCheckoutRequest $request, Accessory $accessory) :
             $accessory_checkout->save();
         }
 
+        $accessory->setLogTarget($target);
+        $accessory->setLogNote($request->input('note'));
+        $accessory->setLogQuantity($checkout_qty);
+        $accessory->setLogAction(ActionType::Checkout);
+        $accessory->save();
         event(new CheckoutableCheckedOut($accessory,  $target, auth()->user(), $request->input('note')));
 
         $request->request->add(['checkout_to_type' => request('checkout_to_type')]);

app/Http/Controllers/Api/AccessoriesController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Events\CheckoutableCheckedOut;
 use App\Helpers\Helper;
 use App\Http\Controllers\CheckInOutRequest;
@@ -276,9 +277,9 @@ public function checkout(AccessoryCheckoutRequest $request, Accessory $accessory
     {
         $this->authorize('checkout', $accessory);
         $target = $this->determineCheckoutTarget();
-        $accessory->checkout_qty = $request->input('checkout_qty', 1);
+        $checkout_qty = $request->input('checkout_qty', 1);
 
-        for ($i = 0; $i < $accessory->checkout_qty; $i++) {
+        for ($i = 0; $i < $checkout_qty; $i++) {
 
             $accessory_checkout = new AccessoryCheckout([
                 'accessory_id' => $accessory->id,
@@ -292,6 +293,12 @@ public function checkout(AccessoryCheckoutRequest $request, Accessory $accessory
             $accessory_checkout->save();
         }
 
+        $accessory->setLogTarget($target);
+        $accessory->setLogNote($request->input('note'));
+        $accessory->setLogQuantity($checkout_qty);
+        $accessory->setLogAction(ActionType::Checkout);
+        $accessory->save();
+
         // Set this value to be able to pass the qty through to the event
         event(new CheckoutableCheckedOut($accessory, $target, auth()->user(), $request->input('note')));
 
@@ -319,7 +326,10 @@ public function checkin(Request $request, $accessoryUserId = null)
         $accessory = Accessory::find($accessory_checkout->accessory_id);
         $this->authorize('checkin', $accessory);
 
-        $accessory->logCheckin(User::find($accessory_checkout->assigned_to), $request->input('note'));
+        $accessory->setLogTarget(User::find($accessory_checkout->assigned_to));
+        $accessory->setLogNote($request->input('note'));
+        $accessory->setLogAction(ActionType::CheckinFrom);
+        $accessory->save();
 
         // Was the accessory updated?
         if ($accessory_checkout->delete()) {

app/Http/Controllers/Api/AssetFilesController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Helpers\StorageHelper;
 use App\Http\Transformers\UploadedFilesTransformer;
 use Illuminate\Support\Facades\Storage;
@@ -55,8 +56,11 @@ public function store(UploadFileRequest $request, $assetId = null) : JsonRespons
             // Loop over the attached files and add them to the asset
             foreach ($request->file('file') as $file) {
                 $file_name = $request->handleFile('private_uploads/assets/','hardware-'.$asset->id, $file);
-
-                $asset->logUpload($file_name, e($request->get('notes')));
+
+                $asset->setLogFilename($file_name);
+                $asset->setLogNote(e($request->input('notes')));
+                $asset->setLogAction(ActionType::Uploaded);
+                $asset->save();
             }
 
             // All done - report success

app/Http/Controllers/Api/AssetModelFilesController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Helpers\StorageHelper;
 use Illuminate\Support\Facades\Storage;
 use App\Helpers\Helper;
@@ -54,8 +55,11 @@ public function store(UploadFileRequest $request, $assetModelId = null) : JsonRe
             // Loop over the attached files and add them to the asset
             foreach ($request->file('file') as $file) {
                 $file_name = $request->handleFile('private_uploads/assetmodels/','model-'.$assetModel->id, $file);
-
-                $assetModel->logUpload($file_name, e($request->get('notes')));
+
+                $assetModel->setLogFilename($file_name);
+                $assetModel->setLogNote(e($request->input('notes')) ?? ''); //weird, but we need this for tests to pass
+                $assetModel->setLogAction(ActionType::Uploaded);
+                $assetModel->save();
             }
 
             // All done - report success

app/Http/Controllers/Api/AssetsController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Events\CheckoutableCheckedIn;
 use App\Http\Requests\StoreAssetRequest;
 use App\Http\Requests\UpdateAssetRequest;
@@ -1149,33 +1150,11 @@ public function audit(Request $request, Asset $asset): JsonResponse
                 return response()->json(Helper::formatStandardApiResponse('error', null,  $asset->getErrors()));
             }
 
-
-            /**
-             * Even though we do a save() further down, we don't want to log this as a "normal" asset update,
-             * which would trigger the Asset Observer and would log an asset *update* log entry (because the
-             * de-normed fields like next_audit_date on the asset itself will change on save()) *in addition* to
-             * the audit log entry we're creating through this controller.
-             *
-             * To prevent this double-logging (one for update and one for audit), we skip the observer and bypass
-             * that de-normed update log entry by using unsetEventDispatcher(), BUT invoking unsetEventDispatcher()
-             * will bypass normal model-level validation that's usually handled at the observer)
-             *
-             * We handle validation on the save() by checking if the asset is valid via the ->isValid() method,
-             * which manually invokes Watson Validating to make sure the asset's model is valid.
-             *
-             * @see \App\Observers\AssetObserver::updating()
-             * @see \App\Models\Asset::save()
-             */
-
-             $asset->unsetEventDispatcher();
-
-
-            /**
-             * Invoke Watson Validating to check the asset itself and check to make sure it saved correctly.
-             * We have to invoke this manually because of the unsetEventDispatcher() above.)
-             */
-            if ($asset->isValid() && $asset->save()) {
-                $asset->logAudit(request('note'), request('location_id'), null, $originalValues);
+            // this 'new' audit system logs the changes via log_meta
+            $asset->setLogNote(request('note'));
+            $asset->setLogLocationOverride(request('location_id'));
+            $asset->setLogAction(ActionType::Audit);
+            if ($asset->save()) {
                 return response()->json(Helper::formatStandardApiResponse('success', $payload, trans('admin/hardware/message.audit.success')));
             }
 

app/Http/Controllers/Api/ComponentsController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Http\Transformers\ComponentsTransformer;
@@ -285,7 +286,7 @@ public function checkout(Request $request, $componentId) : JsonResponse
         if ($component->numRemaining() >= $request->get('assigned_qty')) {
 
             $asset = Asset::find($request->input('assigned_to'));
-            $component->assigned_to = $request->input('assigned_to');
+            $component->setLogTarget($asset);
 
             $component->assets()->attach($component->id, [
                 'component_id' => $component->id,
@@ -296,7 +297,11 @@ public function checkout(Request $request, $componentId) : JsonResponse
                 'note' => $request->get('note'),
             ]);
 
-            $component->logCheckout($request->input('note'), $asset);
+            $component->setLogLocationOverride($asset->location);
+            $component->setLogNote($request->input('note'));
+            $component->setLogQuantity($request->get('assigned_qty'));
+            $component->setLogAction(ActionType::Checkout);
+            $component->save();
 
             return response()->json(Helper::formatStandardApiResponse('success', null,  trans('admin/components/message.checkout.success')));
         }
@@ -350,6 +355,12 @@ public function checkin(Request $request, $component_asset_id) : JsonResponse
 
             $asset = Asset::find($component_assets->asset_id);
 
+            // the 'event()' below no longer does the logging; that needs to be done here.
+            $component->setLogTarget($asset);
+            $component->setLogNote($request->input('note'));
+            $component->setLogAction(ActionType::CheckinFrom);
+            $component->save();
+
             event(new CheckoutableCheckedIn($component, $asset, auth()->user(), $request->input('note'), Carbon::now()));
 
             return response()->json(Helper::formatStandardApiResponse('success', null,  trans('admin/components/message.checkin.success')));

app/Http/Controllers/Api/ConsumablesController.php

@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers\Api;
 
+use App\Enums\ActionType;
 use App\Events\CheckoutableCheckedOut;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
@@ -258,7 +259,7 @@ public function checkout(Request $request, $id) : JsonResponse
 
         $this->authorize('checkout', $consumable);
 
-        $consumable->checkout_qty = $request->input('checkout_qty', 1);
+        $checkout_qty = $request->input('checkout_qty', 1);
 
         // Make sure there is at least one available to checkout
         if ($consumable->numRemaining() <= 0) {
@@ -271,8 +272,8 @@ public function checkout(Request $request, $id) : JsonResponse
         }
 
         // Make sure there is at least one available to checkout
-        if ($consumable->numRemaining() <= 0 || $consumable->checkout_qty > $consumable->numRemaining()) {
-            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/consumables/message.checkout.unavailable', ['requested' => $consumable->checkout_qty, 'remaining' => $consumable->numRemaining() ])));
+        if ($consumable->numRemaining() <= 0 || $checkout_qty > $consumable->numRemaining()) {
+            return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/consumables/message.checkout.unavailable', ['requested' => $checkout_qty, 'remaining' => $consumable->numRemaining()])));
         }
 
 
@@ -284,9 +285,9 @@ public function checkout(Request $request, $id) : JsonResponse
         }
 
         // Update the consumable data
-        $consumable->assigned_to = $request->input('assigned_to');
+        $consumable->setLogTarget($user);
 
-        for ($i = 0; $i < $consumable->checkout_qty; $i++) {
+        for ($i = 0; $i < $checkout_qty; $i++) {
             $consumable->users()->attach($consumable->id,
                 [
                     'consumable_id' => $consumable->id,
@@ -296,7 +297,10 @@ public function checkout(Request $request, $id) : JsonResponse
                 ]
             );
         }
-
+        $consumable->setLogQuantity($checkout_qty);
+        $consumable->setLogNote($request->input('note'));
+        $consumable->setLogAction(ActionType::Checkout);
+        $consumable->save();
 
         event(new CheckoutableCheckedOut($consumable, $user, auth()->user(), $request->input('note')));