If user is instance of Authenticatable call getAuthIdentifier() defined by the interface rather than assuming it has a public id field (#799)

Author: roberthunt

src/DataCollector/GateCollector.php

@@ -6,6 +6,7 @@
 use DebugBar\DataCollector\MessagesCollector;
 use Illuminate\Contracts\Auth\Access\Gate;
 use Illuminate\Contracts\Auth\Access\Authorizable;
+use Illuminate\Contracts\Auth\Authenticatable;
 use Symfony\Component\VarDumper\Cloner\VarCloner;
 
 /**
@@ -25,12 +26,20 @@ public function __construct(Gate $gate)
 
     public function addCheck(Authorizable $user = null, $ability, $result, $arguments = [])
     {
+        $userKey = 'user';
+        $userId = null;
+
+        if ($user) {
+            $userKey = snake_case(class_basename($user));
+            $userId = $user instanceof Authenticatable ? $user->getAuthIdentifier() : $user->id;
+        }
+
         $label = $result ? 'success' : 'error';
 
         $this->addMessage([
             'ability' => $ability,
             'result' => $result,
-            ($user ? snake_case(class_basename($user)) : 'user') => ($user ? $user->id : null),
+            $userKey => $userId,
             'arguments' => $this->getDataFormatter()->formatVar($arguments),
         ], $label, false);
     }