From 916c0c0394ebbe1c518ee4b774d79d4b4e6e836e Mon Sep 17 00:00:00 2001
From: spencerrlongg <spencer@spencerlong.com>
Date: Tue, 12 Nov 2024 14:46:56 -0600
Subject: [PATCH] good progress on update action

---
 app/Actions/Assets/UpdateAssetAction.php      | 151 +++++++++++++++++-
 .../Controllers/Assets/AssetsController.php   |  92 +++++++----
 routes/web/hardware.php                       |   4 +-
 3 files changed, 209 insertions(+), 38 deletions(-)

diff --git a/app/Actions/Assets/UpdateAssetAction.php b/app/Actions/Assets/UpdateAssetAction.php
index 117c8f0ae6..21ba576153 100644
--- a/app/Actions/Assets/UpdateAssetAction.php
+++ b/app/Actions/Assets/UpdateAssetAction.php
@@ -2,10 +2,157 @@
 
 namespace App\Actions\Assets;
 
+use App\Events\CheckoutableCheckedIn;
+use App\Helpers\Helper;
+use App\Models\Asset;
+use App\Models\AssetModel;
+use App\Models\Company;
+use App\Models\Statuslabel;
+use Carbon\Carbon;
+use Illuminate\Support\Facades\Crypt;
+use Illuminate\Support\Facades\Gate;
+use Illuminate\Support\Facades\Log;
+
 class UpdateAssetAction
 {
-    public static function run()
+    public static function run(
+        Asset $asset,
+        $status_id = null,
+        $warranty_months = null,
+        $purchase_cost = null,
+        $purchase_date = null,
+        $next_audit_date = null,
+        $asset_eol_date = null,
+        $supplier_id = null,
+        $expected_checkin = null,
+        $requestable = null,
+        $rtd_location_id = null,
+        $byod = null
+    )
     {
-        //    stuff
+        // Check if the asset exists
+        //if (!$asset = Asset::find($assetId)) {
+        //    // Redirect to the asset management page with error
+        //    return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist'));
+        //}
+        //$this->authorize($asset);
+
+        $asset->status_id = $status_id;
+        $asset->warranty_months = $warranty_months;
+        $asset->purchase_cost = $purchase_cost;
+        $asset->purchase_date = $purchase_date;
+        $asset->next_audit_date = $next_audit_date;
+        if ($request->filled('purchase_date') && !$request->filled('asset_eol_date') && ($asset->model->eol > 0)) {
+            $asset->purchase_date = $request->input('purchase_date', null);
+            $asset->asset_eol_date = Carbon::parse($request->input('purchase_date'))->addMonths($asset->model->eol)->format('Y-m-d');
+            $asset->eol_explicit = false;
+        } elseif ($request->filled('asset_eol_date')) {
+            $asset->asset_eol_date = $request->input('asset_eol_date', null);
+            $months = Carbon::parse($asset->asset_eol_date)->diffInMonths($asset->purchase_date);
+            if ($asset->model->eol) {
+                if ($months != $asset->model->eol > 0) {
+                    $asset->eol_explicit = true;
+                } else {
+                    $asset->eol_explicit = false;
+                }
+            } else {
+                $asset->eol_explicit = true;
+            }
+        } elseif (!$request->filled('asset_eol_date') && (($asset->model->eol) == 0)) {
+            $asset->asset_eol_date = null;
+            $asset->eol_explicit = false;
+        }
+        $asset->supplier_id = $request->input('supplier_id', null);
+        $asset->expected_checkin = $request->input('expected_checkin', null);
+        $asset->requestable = $request->input('requestable', 0);
+        $asset->rtd_location_id = $request->input('rtd_location_id', null);
+        $asset->byod = $request->input('byod', 0);
+
+        $status = Statuslabel::find($status_id);
+
+        // This is a non-deployable status label - we should check the asset back in.
+        if (($status && $status->getStatuslabelType() != 'deployable') && ($target = $asset->assignedTo)) {
+
+            $originalValues = $asset->getRawOriginal();
+            $asset->assigned_to = null;
+            $asset->assigned_type = null;
+            $asset->accepted = null;
+
+            event(new CheckoutableCheckedIn($asset, $target, auth()->user(), 'Checkin on asset update', date('Y-m-d H:i:s'), $originalValues));
+        }
+
+        if ($asset->assigned_to == '') {
+            $asset->location_id = $request->input('rtd_location_id', null);
+        }
+
+
+        if ($request->filled('image_delete')) {
+            try {
+                unlink(public_path().'/uploads/assets/'.$asset->image);
+                $asset->image = '';
+            } catch (\Exception $e) {
+                Log::info($e);
+            }
+        }
+
+        // Update the asset data
+
+        $serial = $request->input('serials');
+        $asset->serial = $request->input('serials');
+
+        if (is_array($request->input('serials'))) {
+            $asset->serial = $serial[1];
+        }
+
+        $asset->name = $request->input('name');
+        $asset->company_id = Company::getIdForCurrentUser($request->input('company_id'));
+        $asset->model_id = $request->input('model_id');
+        $asset->order_number = $request->input('order_number');
+
+        $asset_tags = $request->input('asset_tags');
+        $asset->asset_tag = $request->input('asset_tags');
+
+        if (is_array($request->input('asset_tags'))) {
+            $asset->asset_tag = $asset_tags[1];
+        }
+
+        $asset->notes = $request->input('notes');
+
+        $asset = $request->handleImages($asset);
+
+        // Update custom fields in the database.
+        // Validation for these fields is handlded through the AssetRequest form request
+        // FIXME: No idea why this is returning a Builder error on db_column_name.
+        // Need to investigate and fix. Using static method for now.
+        $model = AssetModel::find($request->get('model_id'));
+        if (($model) && ($model->fieldset)) {
+            foreach ($model->fieldset->fields as $field) {
+
+                if ($field->field_encrypted == '1') {
+                    if (Gate::allows('assets.view.encrypted_custom_fields')) {
+                        if (is_array($request->input($field->db_column))) {
+                            $asset->{$field->db_column} = Crypt::encrypt(implode(', ', $request->input($field->db_column)));
+                        } else {
+                            $asset->{$field->db_column} = Crypt::encrypt($request->input($field->db_column));
+                        }
+                    }
+                } else {
+                    if (is_array($request->input($field->db_column))) {
+                        $asset->{$field->db_column} = implode(', ', $request->input($field->db_column));
+                    } else {
+                        $asset->{$field->db_column} = $request->input($field->db_column);
+                    }
+                }
+            }
+        }
+
+        session()->put(['redirect_option' => $request->get('redirect_option'), 'checkout_to_type' => $request->get('checkout_to_type')]);
+
+        if ($asset->save()) {
+            return redirect()->to(Helper::getRedirectOption($request, $assetId, 'Assets'))
+                ->with('success', trans('admin/hardware/message.update.success'));
+        }
+
+        return redirect()->back()->withInput()->withErrors($asset->getErrors());
     }
 }
\ No newline at end of file
diff --git a/app/Http/Controllers/Assets/AssetsController.php b/app/Http/Controllers/Assets/AssetsController.php
index c650d9f699..b5c1bf8643 100755
--- a/app/Http/Controllers/Assets/AssetsController.php
+++ b/app/Http/Controllers/Assets/AssetsController.php
@@ -4,11 +4,13 @@ namespace App\Http\Controllers\Assets;
 
 use App\Actions\Assets\DestroyAssetAction;
 use App\Actions\Assets\StoreAssetAction;
+use App\Actions\Assets\UpdateAssetAction;
 use App\Events\CheckoutableCheckedIn;
 use App\Exceptions\CheckoutNotAllowed;
 use App\Helpers\Helper;
 use App\Http\Controllers\Controller;
 use App\Http\Requests\Assets\DestroyAssetRequest;
+use App\Http\Requests\Assets\UpdateAssetRequest;
 use App\Http\Requests\ImageUploadRequest;
 use App\Http\Requests\Assets\StoreAssetRequest;
 use App\Models\Actionlog;
@@ -228,46 +230,66 @@ class AssetsController extends Controller
      * @since [v1.0]
      * @author [A. Gianotto] [<snipe@snipe.net>]
      */
-    public function update(ImageUploadRequest $request, $assetId = null) : RedirectResponse
+    public function update(UpdateAssetRequest $request, Asset $asset): RedirectResponse
     {
 
         // Check if the asset exists
-        if (! $asset = Asset::find($assetId)) {
-            // Redirect to the asset management page with error
-            return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist'));
+        //if (! $asset = Asset::find($assetId)) {
+        //    // Redirect to the asset management page with error
+        //    return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist'));
+        //}
+        //$this->authorize($asset);
+        try {
+            $asset = UpdateAssetAction::run(
+                $asset,
+                status_id: $request->validated('status_id'),
+                warranty_months: $request->validated('warranty_months'),
+                purchase_cost: $request->validated('purchase_cost'),
+                purchase_date: $request->validated('purchase_date'),
+                next_audit_date: $request->validated('next_audit_date'),
+                asset_eol_date: $request->validated('asset_eol_date'),
+                supplier_id: $request->validated('supplier_id'),
+                expected_checkin: $request->validated('expected_checkin'),
+                requestable: $request->validated('requestable'),
+                rtd_location_id: $request->validated('rtd_location_id'),
+                byod: $request->validated('byod')
+            );
+            return redirect()->back()->with('success', trans('admin/hardware/message.update.success'));
+        } catch (\Exception $e) {
+            report($e);
+            return redirect()->back()->with('error', trans('admin/hardware/message.update.error'));
         }
-        $this->authorize($asset);
 
-        $asset->status_id = $request->input('status_id', null);
-        $asset->warranty_months = $request->input('warranty_months', null);
-        $asset->purchase_cost = $request->input('purchase_cost', null);
-        $asset->purchase_date = $request->input('purchase_date', null);
-        $asset->next_audit_date = $request->input('next_audit_date', null);
-        if ($request->filled('purchase_date') && !$request->filled('asset_eol_date') && ($asset->model->eol > 0)) {
-            $asset->purchase_date = $request->input('purchase_date', null); 
-            $asset->asset_eol_date = Carbon::parse($request->input('purchase_date'))->addMonths($asset->model->eol)->format('Y-m-d');
-            $asset->eol_explicit = false;
-        } elseif ($request->filled('asset_eol_date')) {
-           $asset->asset_eol_date = $request->input('asset_eol_date', null);
-           $months = Carbon::parse($asset->asset_eol_date)->diffInMonths($asset->purchase_date);
-           if($asset->model->eol) {
-               if($months != $asset->model->eol > 0) {
-                   $asset->eol_explicit = true;
-               } else {
-                   $asset->eol_explicit = false;
-               }
-           } else {
-               $asset->eol_explicit = true;
-           }
-        } elseif (!$request->filled('asset_eol_date') && (($asset->model->eol) == 0)) {
-           $asset->asset_eol_date = null;
-		   $asset->eol_explicit = false;
-        }
-        $asset->supplier_id = $request->input('supplier_id', null);
-        $asset->expected_checkin = $request->input('expected_checkin', null);
-        $asset->requestable = $request->input('requestable', 0);
-        $asset->rtd_location_id = $request->input('rtd_location_id', null);
-        $asset->byod = $request->input('byod', 0);
+        //$asset->status_id = $request->input('status_id', null);
+        //$asset->warranty_months = $request->input('warranty_months', null);
+        //$asset->purchase_cost = $request->input('purchase_cost', null);
+        //$asset->purchase_date = $request->input('purchase_date', null);
+        //$asset->next_audit_date = $request->input('next_audit_date', null);
+        //if ($request->filled('purchase_date') && !$request->filled('asset_eol_date') && ($asset->model->eol > 0)) {
+        //    $asset->purchase_date = $request->input('purchase_date', null);
+        //    $asset->asset_eol_date = Carbon::parse($request->input('purchase_date'))->addMonths($asset->model->eol)->format('Y-m-d');
+        //    $asset->eol_explicit = false;
+        //} elseif ($request->filled('asset_eol_date')) {
+        //   $asset->asset_eol_date = $request->input('asset_eol_date', null);
+        //   $months = Carbon::parse($asset->asset_eol_date)->diffInMonths($asset->purchase_date);
+        //   if($asset->model->eol) {
+        //       if($months != $asset->model->eol > 0) {
+        //           $asset->eol_explicit = true;
+        //       } else {
+        //           $asset->eol_explicit = false;
+        //       }
+        //   } else {
+        //       $asset->eol_explicit = true;
+        //   }
+        //} elseif (!$request->filled('asset_eol_date') && (($asset->model->eol) == 0)) {
+        //   $asset->asset_eol_date = null;
+        //   $asset->eol_explicit = false;
+        //}
+        //$asset->supplier_id = $request->input('supplier_id', null);
+        //$asset->expected_checkin = $request->input('expected_checkin', null);
+        //$asset->requestable = $request->input('requestable', 0);
+        //$asset->rtd_location_id = $request->input('rtd_location_id', null);
+        //$asset->byod = $request->input('byod', 0);
 
         $status = Statuslabel::find($request->input('status_id'));
 
diff --git a/routes/web/hardware.php b/routes/web/hardware.php
index aa8b0c087b..f12bc60169 100644
--- a/routes/web/hardware.php
+++ b/routes/web/hardware.php
@@ -165,6 +165,8 @@ Route::group(
 
 Route::delete('/hardware/{asset}', [AssetsController::class, 'destroy'])->name('hardware.destroy');
 
+Route::match(['put', 'patch'], '/hardware/{asset}', [AssetsController::class, 'update'])->name('hardware.update');
+
 Route::resource('hardware', 
         AssetsController::class, 
         [
@@ -174,7 +176,7 @@ Route::resource('hardware',
                     'show' => 'view',
                 ],
         ],
-            'except' => ['destroy'],
+            'except' => ['destroy', 'update'],
 ]);
 
 Route::get('ht/{any?}',