Skip to content

Commit

Permalink
Rename alter table events
Browse files Browse the repository at this point in the history
  • Loading branch information
jsor committed Dec 9, 2011
1 parent b51762c commit 6abba72
Show file tree
Hide file tree
Showing 7 changed files with 36 additions and 36 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
* @version $Revision$
* @author Jan Sorgalla <jsorgalla@googlemail.com>
*/
class SchemaAlterTableAddedColumnEventArgs extends SchemaEventArgs
class SchemaAlterTableAddColumnEventArgs extends SchemaEventArgs
{
/**
* @var Column
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
* @version $Revision$
* @author Jan Sorgalla <jsorgalla@googlemail.com>
*/
class SchemaAlterTableChangedColumnEventArgs extends SchemaEventArgs
class SchemaAlterTableChangeColumnEventArgs extends SchemaEventArgs
{
/**
* @var ColumnDiff
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
* @version $Revision$
* @author Jan Sorgalla <jsorgalla@googlemail.com>
*/
class SchemaAlterTableRemovedColumnEventArgs extends SchemaEventArgs
class SchemaAlterTableRemoveColumnEventArgs extends SchemaEventArgs
{
/**
* @var Column
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
* @version $Revision$
* @author Jan Sorgalla <jsorgalla@googlemail.com>
*/
class SchemaAlterTableRenamedColumnEventArgs extends SchemaEventArgs
class SchemaAlterTableRenameColumnEventArgs extends SchemaEventArgs
{
/**
* @var string
Expand Down
8 changes: 4 additions & 4 deletions lib/Doctrine/DBAL/Events.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ private function __construct() {}
const onSchemaCreateTable = 'onSchemaCreateTable';
const onSchemaCreateTableColumn = 'onSchemaCreateTableColumn';
const onSchemaDropTable = 'onSchemaDropTable';
const onSchemaAlterTableAddedColumn = 'onSchemaAlterTableAddedColumn';
const onSchemaAlterTableRemovedColumn = 'onSchemaAlterTableRemovedColumn';
const onSchemaAlterTableChangedColumn = 'onSchemaAlterTableChangedColumn';
const onSchemaAlterTableRenamedColumn = 'onSchemaAlterTableRenamedColumn';
const onSchemaAlterTableAddColumn = 'onSchemaAlterTableAddColumn';
const onSchemaAlterTableRemoveColumn = 'onSchemaAlterTableRemoveColumn';
const onSchemaAlterTableChangeColumn = 'onSchemaAlterTableChangeColumn';
const onSchemaAlterTableRenameColumn = 'onSchemaAlterTableRenameColumn';
const onSchemaColumnDefinition = 'onSchemaColumnDefinition';
}
32 changes: 16 additions & 16 deletions lib/Doctrine/DBAL/Platforms/PostgreSqlPlatform.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@
use Doctrine\DBAL\Schema\TableDiff,
Doctrine\DBAL\Schema\Table,
Doctrine\DBAL\Events,
Doctrine\DBAL\Event\SchemaAlterTableAddedColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableRemovedColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableChangedColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableRenamedColumnEventArgs;
Doctrine\DBAL\Event\SchemaAlterTableAddColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableRemoveColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableChangeColumnEventArgs,
Doctrine\DBAL\Event\SchemaAlterTableRenameColumnEventArgs;

/**
* PostgreSqlPlatform.
Expand Down Expand Up @@ -388,9 +388,9 @@ public function getAlterTableSQL(TableDiff $diff)
$columnSql = array();

foreach ($diff->addedColumns as $column) {
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableAddedColumn)) {
$eventArgs = new SchemaAlterTableAddedColumnEventArgs($column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableAddedColumn, $eventArgs);
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableAddColumn)) {
$eventArgs = new SchemaAlterTableAddColumnEventArgs($column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableAddColumn, $eventArgs);

$columnSql = array_merge($columnSql, $eventArgs->getSql());

Expand All @@ -407,9 +407,9 @@ public function getAlterTableSQL(TableDiff $diff)
}

foreach ($diff->removedColumns as $column) {
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableRemovedColumn)) {
$eventArgs = new SchemaAlterTableRemovedColumnEventArgs($column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableRemovedColumn, $eventArgs);
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableRemoveColumn)) {
$eventArgs = new SchemaAlterTableRemoveColumnEventArgs($column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableRemoveColumn, $eventArgs);

$columnSql = array_merge($columnSql, $eventArgs->getSql());

Expand All @@ -423,9 +423,9 @@ public function getAlterTableSQL(TableDiff $diff)
}

foreach ($diff->changedColumns AS $columnDiff) {
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableChangedColumn)) {
$eventArgs = new SchemaAlterTableChangedColumnEventArgs($columnDiff, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableChangedColumn, $eventArgs);
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableChangeColumn)) {
$eventArgs = new SchemaAlterTableChangeColumnEventArgs($columnDiff, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableChangeColumn, $eventArgs);

$columnSql = array_merge($columnSql, $eventArgs->getSql());

Expand Down Expand Up @@ -473,9 +473,9 @@ public function getAlterTableSQL(TableDiff $diff)
}

foreach ($diff->renamedColumns as $oldColumnName => $column) {
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableRenamedColumn)) {
$eventArgs = new SchemaAlterTableRenamedColumnEventArgs($oldColumnName, $column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableRenamedColumn, $eventArgs);
if (null !== $this->_eventManager && $this->_eventManager->hasListeners(Events::onSchemaAlterTableRenameColumn)) {
$eventArgs = new SchemaAlterTableRenameColumnEventArgs($oldColumnName, $column, $diff, $this);
$this->_eventManager->dispatchEvent(Events::onSchemaAlterTableRenameColumn, $eventArgs);

$columnSql = array_merge($columnSql, $eventArgs->getSql());

Expand Down
24 changes: 12 additions & 12 deletions tests/Doctrine/Tests/DBAL/Platforms/AbstractPlatformTestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -219,32 +219,32 @@ public function testGetDropTableSqlDispatchEvent()
public function testGetAlterTableSqlDispatchEvent()
{
$events = array(
'onSchemaAlterTableAddedColumn',
'onSchemaAlterTableRemovedColumn',
'onSchemaAlterTableChangedColumn',
'onSchemaAlterTableRenamedColumn'
'onSchemaAlterTableAddColumn',
'onSchemaAlterTableRemoveColumn',
'onSchemaAlterTableChangeColumn',
'onSchemaAlterTableRenameColumn'
);

$listenerMock = $this->getMock('GetAlterTableSqlDispatchEvenListener', $events);
$listenerMock
->expects($this->once())
->method('onSchemaAlterTableAddedColumn');
->method('onSchemaAlterTableAddColumn');
$listenerMock
->expects($this->once())
->method('onSchemaAlterTableRemovedColumn');
->method('onSchemaAlterTableRemoveColumn');
$listenerMock
->expects($this->once())
->method('onSchemaAlterTableChangedColumn');
->method('onSchemaAlterTableChangeColumn');
$listenerMock
->expects($this->once())
->method('onSchemaAlterTableRenamedColumn');
->method('onSchemaAlterTableRenameColumn');

$eventManager = new EventManager();
$events = array(
Events::onSchemaAlterTableAddedColumn,
Events::onSchemaAlterTableRemovedColumn,
Events::onSchemaAlterTableChangedColumn,
Events::onSchemaAlterTableRenamedColumn
Events::onSchemaAlterTableAddColumn,
Events::onSchemaAlterTableRemoveColumn,
Events::onSchemaAlterTableChangeColumn,
Events::onSchemaAlterTableRenameColumn
);
$eventManager->addEventListener($events, $listenerMock);

Expand Down

0 comments on commit 6abba72

Please sign in to comment.