Only log checkin event if the assigned_to property is not empty

snipe · Oct 25, 2023 · ca1845e · ca1845e
1 parent 7ed58a1
commit ca1845e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/app/Importer/AssetImporter.php b/app/Importer/AssetImporter.php
@@ -145,8 +145,10 @@ public function createAssetIfNotExists(array $row)
             //-- user_id is a property of the abstract class Importer, which this class inherits from and it's setted by
             //-- the class that needs to use it (command importer or GUI importer inside the project).
             if (isset($target)) {
-                if ($asset->assigned_to != $target->id){
-                    event(new CheckoutableCheckedIn($asset, User::find($asset->assigned_to), Auth::user(), $asset->notes, date('Y-m-d H:i:s')));
+                if (!is_null($asset->assigned_to)){
+                    if ($asset->assigned_to != $target->id){
+                        event(new CheckoutableCheckedIn($asset, User::find($asset->assigned_to), Auth::user(), $asset->notes, date('Y-m-d H:i:s')));
+                    }
                 }
 
                 $asset->fresh()->checkOut($target, $this->user_id, date('Y-m-d H:i:s'), null, $asset->notes, $asset->name);