Merge pull request #41 from yurenzhen/master

修复当 user_id 为 string 类型(如uuid)时判断错误而导致的 updated_by 和 created_by 不更新问题.

src/Someline/Base/Models/BaseModel.php

@@ -135,6 +135,21 @@ protected function updateIps()
         }
     }
 
+    /**
+     * check the $user_id is valid.
+     *
+     * @param  int|string $user_id
+     * @return bool
+     */
+    protected function validUserId($user_id): bool
+    {
+        if ('int' === $this->keyType)
+            return $user_id > 0;
+
+        if ('string' === $this->keyType)
+            return strlen($user_id) > 0;
+    }
+
     /**
      * Update the creation and update by users.
      *
@@ -143,7 +158,7 @@ protected function updateIps()
     protected function updateUsers()
     {
         $user_id = $this->getAuthUserId();
-        if (!($user_id > 0)) {
+        if (!$this->validUserId($user_id)) {
             return;
         }
 
@@ -159,7 +174,7 @@ protected function updateUsers()
     /**
      * @return bool
      */
-    public function isAuthUserOwner()
+    public function isAuthUserOwner(): bool
     {
         return $this->getAuthUserId() == $this->getUserId();
     }