Permalink
Browse files

Merge pull request #153 from rozwell/sfPropelMigrateTask

sfPropelMigrateTask: returns invalid value when there is nothing to migrate
  • Loading branch information...
willdurand committed Aug 2, 2012
2 parents 104d6ff + 2592a64 commit 67a39369c625d32c2223a5d2e742040423bc1066
@@ -64,7 +64,7 @@ protected function execute($arguments = array(), $options = array())
if (!$nextMigrationTimestamp = array_pop($previousTimestamps))
{
$this->logSection('propel', 'No migration were ever executed on this database - nothing to reverse.');
- return false;
+ return true;
}
$this->logSection('propel', sprintf(
'Executing migration %s down',
@@ -150,6 +150,8 @@ protected function execute($arguments = array(), $options = array())
{
$this->logSection('propel', 'Reverse migration complete. No more migration available for reverse');
}
+
+ return true;
}
}
@@ -63,7 +63,7 @@ protected function execute($arguments = array(), $options = array())
if (!$nextMigrationTimestamp = $manager->getFirstUpMigrationTimestamp())
{
$this->logSection('propel', 'All migrations were already executed - nothing to migrate.');
- return false;
+ return true;
}
$timestamps = $manager->getValidMigrationTimestamps();
@@ -142,6 +142,8 @@ protected function execute($arguments = array(), $options = array())
}
$this->logSection('propel', 'Migration complete. No further migration to execute.');
+
+ return true;
}
}
@@ -63,7 +63,7 @@ protected function execute($arguments = array(), $options = array())
if (!$nextMigrationTimestamp = $manager->getFirstUpMigrationTimestamp())
{
$this->logSection('propel', 'All migrations were already executed - nothing to migrate.');
- return false;
+ return true;
}
$this->logSection('propel', sprintf(
'Executing migration %s up',
@@ -142,6 +142,8 @@ protected function execute($arguments = array(), $options = array())
{
$this->logSection('propel', 'Migration complete. No further migration to execute.');
}
+
+ return true;
}
}

0 comments on commit 67a3936

Please sign in to comment.