yii\redis\ActiveRecord::deleteAll() refactoring - no need to write to socket 2…

yii\redis\ActiveRecord::deleteAll() refactoring - no need to write to socket 2 times when nothing found

yii\redis\ActiveRecord::deleteAll() refactoring - no need to write to socket 2…
a5506e64 · savvot · 4866ce3c · a5506e64
Commit a5506e64 authored Jun 05, 2014 by savvot
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

ActiveRecord.php extensions/redis/ActiveRecord.php +2 -2

No files found.
--- a/extensions/redis/ActiveRecord.php
+++ b/extensions/redis/ActiveRecord.php
@@ -233,11 +233,11 @@ class ActiveRecord extends BaseActiveRecord
     */
    public static function deleteAll($condition = null)
    {
-        $db = static::getDb();
-        $attributeKeys = [];
        $pks = self::fetchPks($condition);
        if(!$pks) return 0;

+        $db = static::getDb();
+        $attributeKeys = [];
        $db->executeCommand('MULTI');
        foreach ($pks as $pk) {
            $pk = static::buildKey($pk);