Use more verbose annotation for Auth::user if/else

snipe · Aug 14, 2020 · 29f3a5c · 29f3a5c
1 parent 134e8e6
commit 29f3a5c
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/app/Models/Loggable.php b/app/Models/Loggable.php
@@ -41,8 +41,9 @@ public function logCheckout($note, $target /* What are we checking out to? */)
         $settings = Setting::getSettings();
         $log = new Actionlog;
         $log = $this->determineLogItemType($log);
-        if(Auth::user())
+        if (Auth::user()) {
             $log->user_id = Auth::user()->id;
+        }
 
         if (!isset($target)) {
             throw new \Exception('All checkout logs require a target.');
@@ -144,9 +145,11 @@ public function logCheckin($target, $note)
 
         $log->location_id = null;
         $log->note = $note;
-        if(Auth::user())
-            $log->user_id = Auth::user()->id;
 
+        if (Auth::user()) {
+            $log->user_id = Auth::user()->id;
+        }
+
         $log->logaction('checkin from');
 
         $params = [