deblan/propel-bundle
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Clearly indicate that the TableDrop command is MySQL only

Browse source
This commit is contained in:
Kévin Gomez 2013-12-15 14:33:07 +00:00
parent b9ceee8abd
commit 10d0cb623f
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
Command/TableDropCommand.php
View file

@ -10,6 +10,7 @@
namespace Propel\PropelBundle\Command;
use Propel\Runtime\Adapter\Pdo\MysqlAdapter;
use Propel\Runtime\Propel;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
@ -44,10 +45,15 @@ class TableDropCommand extends ContainerAwareCommand
*/
protected function execute(InputInterface $input, OutputInterface $output)
{
if (!$input->getOption('force')) {
$output->writeln('<error>You have to use the "--force" option to drop some tables.</error>');
$connection = Propel::getConnection($input->getOption('connection'));
$adapter = Propel::getAdapter($connection->getName());
return;
if (!$adapter instanceof MysqlAdapter) {
return $output->writeln('<error>This command is MySQL only.</error>');
}
if (!$input->getOption('force')) {
return $output->writeln('<error>You have to use the "--force" option to drop some tables.</error>');
}
$tablesToDelete = $input->getArgument('table');
@ -71,9 +77,6 @@ class TableDropCommand extends ContainerAwareCommand
}
try {
$connection = Propel::getConnection($input->getOption('connection'));
$adapter = Propel::getAdapter($connection->getName());
$showStatement = $connection->prepare('SHOW TABLES;');
$showStatement->execute();