Correctly fix the migration now that we have an iterator.

Can't use empty() on the iterator.
horde · May 13, 2015 · 516b2c3 · 516b2c3
1 parent 894d544
commit 516b2c3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/framework/History/migration/Horde/History/3_horde_history_add_modseq.php b/framework/History/migration/Horde/History/3_horde_history_add_modseq.php
@@ -13,12 +13,14 @@ public function up()
         $seq = 1;
 
         $this->beginDbTransaction();
+        $empty = true;
         foreach ($rows as $row) {
+            $empty = false;
             $this->update(
                 'UPDATE horde_histories SET history_modseq = ? WHERE history_id = ?',
                 array($seq++, $row['history_id']));
         }
-        if (!empty($rows)) {
+        if (!$empty) {
             $this->insert('INSERT INTO horde_histories_modseq (history_modseq) VALUES(?)', array($seq - 1));
         }
         $this->commitDbTransaction();