fixed several scrutinizer bugs

src/ActiveQuery.php

@@ -31,7 +31,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
     const EVENT_INIT = 'init';
 
     /**
-     * @var array a list of relations that this query should be joined with
+     * @var array|null a list of relations that this query should be joined with
      */
     public $joinWith = [];
 
@@ -43,7 +43,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
     /**
      * Constructor.
      *
-     * @param array $modelClass the model class associated with this query
+     * @param string $modelClass the model class associated with this query
      * @param array $config configurations to be applied to the newly created query object
      */
     public function __construct($modelClass, $config = [])

src/ActiveRecord.php

@@ -166,7 +166,6 @@ public function getPrimaryValue()
     {
         $primaryValue = $this->primaryValue();
 
-        $result = null;
         if ($primaryValue instanceof \Closure) {
             return call_user_func($primaryValue, [$this]);
         } elseif (is_array($primaryValue)) {
@@ -281,6 +280,9 @@ public function insert($runValidation = true, $attributes = null, $options = [])
         return true;
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function delete($options = [])
     {
         if (!$this->beforeDelete()) {
@@ -295,11 +297,7 @@ public function delete($options = [])
         $this->setOldAttributes(null);
         $this->afterDelete();
 
-        if ($result === false) {
-            return 0;
-        } else {
-            return 1;
-        }
+        return $result === false ? false : true;
     }
 
     public function update($runValidation = true, $attributeNames = null, $options = [])
@@ -338,7 +336,7 @@ protected function updateInternal($attributes = null, $options = [])
 
         $this->afterSave(false, $changedAttributes);
 
-        return $result;
+        return $result === false ? false : true;
     }
 
     /**