From b1c69ebab9c09ab498485ea7595e82631fe25b19 Mon Sep 17 00:00:00 2001 From: Stefano Rodriguez Date: Sat, 8 Sep 2012 14:59:16 +0200 Subject: [PATCH 1/4] adedd failing test for PR #440 --- .../SingleTableInheritanceType/Structure.php | 22 ++++ .../SingleTableInheritanceType/User.php | 123 ++++++++++++++++++ .../UserFollowedObject.php | 32 +++++ .../UserFollowedStructure.php | 54 ++++++++ .../UserFollowedUser.php | 55 ++++++++ .../Tests/ORM/Tools/SchemaToolTest.php | 32 +++++ 6 files changed, 318 insertions(+) create mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php create mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php create mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php create mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php create mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php new file mode 100755 index 000000000..6be981aeb --- /dev/null +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php @@ -0,0 +1,22 @@ +followedUsers = new ArrayCollection(); + $this->followedStructures = new ArrayCollection(); + } + + /* + * Remove followers + * + * @param UserFollowedUser $followers + */ + private function removeFollower(UserFollowedUser $followers) + { + $this->followers->removeElement($followers); + } + + /** + * Add followedUsers + * + * @param UserFollowedUser $followedUsers + * @return User + */ + public function addFollowedUser(UserFollowedUser $followedUsers) + { + $this->followedUsers[] = $followedUsers; + + return $this; + } + + /** + * Remove followedUsers + * + * @param UserFollowedUser $followedUsers + * @return User + */ + public function removeFollowedUser(UserFollowedUser $followedUsers) + { + $this->followedUsers->removeElement($followedUsers); + + return $this; + } + + /** + * Get followedUsers + * + * @return Doctrine\Common\Collections\Collection + */ + public function getFollowedUsers() + { + return $this->followedUsers; + } + + /** + * Add followedStructures + * + * @param UserFollowedStructure $followedStructures + * @return User + */ + public function addFollowedStructure(UserFollowedStructure $followedStructures) + { + $this->followedStructures[] = $followedStructures; + + return $this; + } + + /** + * Remove followedStructures + * + * @param UserFollowedStructure $followedStructures + * @return User + */ + public function removeFollowedStructure(UserFollowedStructure $followedStructures) + { + $this->followedStructures->removeElement($followedStructures); + + return $this; + } + + /** + * Get followedStructures + * + * @return Doctrine\Common\Collections\Collection + */ + public function getFollowedStructures() + { + return $this->followedStructures; + } +} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php new file mode 100755 index 000000000..9996660c0 --- /dev/null +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php @@ -0,0 +1,32 @@ +id; + } +} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php new file mode 100755 index 000000000..8ee0888cf --- /dev/null +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php @@ -0,0 +1,54 @@ +user = $user; + $this->followedStructure = $followedStructure; + } + + /** + * + * @return User + */ + public function getUser() + { + return $this->user; + } + + /** + * Gets followed structure + * + * @return Structure + */ + public function getFollowedStructure() + { + return $this->followedStructure; + } +} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php new file mode 100755 index 000000000..0f90a2416 --- /dev/null +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php @@ -0,0 +1,55 @@ +user = $user; + $this->followedUser = $followedUser; + } + + /** + * {@inheritdoc} + */ + public function getUser() + { + return $this->user; + } + + /** + * Gets followed user + * + * @return User + */ + public function getFollowedUser() + { + return $this->followedUser; + } + +} diff --git a/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php b/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php index 66093a38d..c215cf6f0 100644 --- a/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php +++ b/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php @@ -32,6 +32,38 @@ class SchemaToolTest extends \Doctrine\Tests\OrmTestCase $this->assertTrue($schema->getTable('cms_users')->columnsAreIndexed(array('username')), "username column should be indexed."); } + public function testForeignKeyOnSTIWithMultipleMapping() + { + $em = $this->_getTestEntityManager(); + $schemaTool = new SchemaTool($em); + + $classes = array( + $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\User'), + $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\Structure'), + $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedObject'), + $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedStructure'), + $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedUser') + ); + + $schema = $schemaTool->getSchemaFromMetadata($classes); + $this->assertTrue($schema->hasTable('users_followed_objects'), "Table users_followed_objects should exist."); + + /* @var $table \Doctrine\DBAL\Schema\Table */ + $table = ($schema->getTable('users_followed_objects')); + $this->assertTrue($table->columnsAreIndexed(array('object_id'))); + $this->assertTrue($table->columnsAreIndexed(array('user_id'))); + $foreignKeys = $table->getForeignKeys(); + $this->assertCount(1, $foreignKeys, 'user_id column has to have FK, but not object_id'); + + /* @var $fk \Doctrine\DBAL\Schema\ForeignKeyConstraint */ + $fk = reset($foreignKeys); + $this->assertEquals('users', $fk->getForeignTableName()); + + $localColumns = $fk->getLocalColumns(); + $this->assertContains('user_id', $localColumns); + $this->assertCount(1, $localColumns); + } + public function testAnnotationOptionsAttribute() { $em = $this->_getTestEntityManager(); From 482da95352e88af508c742e4cc871e6cb867e79e Mon Sep 17 00:00:00 2001 From: Stefano Rodriguez Date: Sat, 8 Sep 2012 05:00:31 +0200 Subject: [PATCH 2/4] The schema tool now doesn't add a foreign constraint when subclassess of a STI use the same field to map relations with entities of different classes --- lib/Doctrine/ORM/Tools/SchemaTool.php | 52 ++++++++++++++----- .../SingleTableInheritanceType/Structure.php | 20 +++---- .../SingleTableInheritanceType/User.php | 31 +++++------ .../UserFollowedStructure.php | 2 +- 4 files changed, 66 insertions(+), 39 deletions(-) diff --git a/lib/Doctrine/ORM/Tools/SchemaTool.php b/lib/Doctrine/ORM/Tools/SchemaTool.php index ec7023075..3eb60e87c 100644 --- a/lib/Doctrine/ORM/Tools/SchemaTool.php +++ b/lib/Doctrine/ORM/Tools/SchemaTool.php @@ -42,6 +42,7 @@ use Doctrine\ORM\Tools\Event\GenerateSchemaEventArgs; * @author Jonathan Wage * @author Roman Borschel * @author Benjamin Eberlei + * @author Stefano Rodriguez */ class SchemaTool { @@ -142,6 +143,9 @@ class SchemaTool $metadataSchemaConfig->setExplicitForeignKeyIndexes(false); $schema = new Schema(array(), array(), $metadataSchemaConfig); + $addedFks = array(); + $blacklistedFks = array(); + foreach ($classes as $class) { if ($this->processingNotRequired($class, $processedClasses)) { continue; @@ -150,8 +154,8 @@ class SchemaTool $table = $schema->createTable($this->quoteStrategy->getTableName($class, $this->platform)); if ($class->isInheritanceTypeSingleTable()) { - $this->gatherColumns($class, $table); - $this->gatherRelationsSql($class, $table, $schema); + $columns = $this->gatherColumns($class, $table); + $this->gatherRelationsSql($class, $table, $schema, $addedFks, $blacklistedFks); // Add the discriminator column $this->addDiscriminatorColumnDefinition($class, $table); @@ -165,7 +169,7 @@ class SchemaTool foreach ($class->subClasses as $subClassName) { $subClass = $this->em->getClassMetadata($subClassName); $this->gatherColumns($subClass, $table); - $this->gatherRelationsSql($subClass, $table, $schema); + $this->gatherRelationsSql($subClass, $table, $schema, $addedFks, $blacklistedFks); $processedClasses[$subClassName] = true; } } elseif ($class->isInheritanceTypeJoined()) { @@ -182,7 +186,7 @@ class SchemaTool } } - $this->gatherRelationsSql($class, $table, $schema); + $this->gatherRelationsSql($class, $table, $schema, $addedFks, $blacklistedFks); // Add the discriminator column only to the root table if ($class->name == $class->rootEntityName) { @@ -211,7 +215,7 @@ class SchemaTool throw ORMException::notSupported(); } else { $this->gatherColumns($class, $table); - $this->gatherRelationsSql($class, $table, $schema); + $this->gatherRelationsSql($class, $table, $schema, $addedFks, $blacklistedFks); } $pkColumns = array(); @@ -432,9 +436,11 @@ class SchemaTool * @param ClassMetadata $class * @param \Doctrine\DBAL\Schema\Table $table * @param \Doctrine\DBAL\Schema\Schema $schema + * @param array $addedFks + * @param array $blacklistedFks * @return void */ - private function gatherRelationsSql($class, Table $table, $schema) + private function gatherRelationsSql($class, $table, $schema, &$addedFks, &$blacklistedFks) { foreach ($class->associationMappings as $mapping) { if (isset($mapping['inherited'])) { @@ -446,7 +452,7 @@ class SchemaTool if ($mapping['type'] & ClassMetadata::TO_ONE && $mapping['isOwningSide']) { $primaryKeyColumns = $uniqueConstraints = array(); // PK is unnecessary for this relation-type - $this->_gatherRelationJoinColumns($mapping['joinColumns'], $table, $foreignClass, $mapping, $primaryKeyColumns, $uniqueConstraints); + $this->_gatherRelationJoinColumns($mapping['joinColumns'], $table, $foreignClass, $mapping, $primaryKeyColumns, $uniqueConstraints, $addedFks, $blacklistedFks); foreach($uniqueConstraints as $indexName => $unique) { $table->addUniqueIndex($unique['columns'], is_numeric($indexName) ? null : $indexName); @@ -463,10 +469,10 @@ class SchemaTool $primaryKeyColumns = $uniqueConstraints = array(); // Build first FK constraint (relation table => source table) - $this->_gatherRelationJoinColumns($joinTable['joinColumns'], $theJoinTable, $class, $mapping, $primaryKeyColumns, $uniqueConstraints); + $this->_gatherRelationJoinColumns($joinTable['joinColumns'], $theJoinTable, $class, $mapping, $primaryKeyColumns, $uniqueConstraints, $addedFks, $blacklistedFks); // Build second FK constraint (relation table => target table) - $this->_gatherRelationJoinColumns($joinTable['inverseJoinColumns'], $theJoinTable, $foreignClass, $mapping, $primaryKeyColumns, $uniqueConstraints); + $this->_gatherRelationJoinColumns($joinTable['inverseJoinColumns'], $theJoinTable, $foreignClass, $mapping, $primaryKeyColumns, $uniqueConstraints, $addedFks, $blacklistedFks); $theJoinTable->setPrimaryKey($primaryKeyColumns); @@ -522,8 +528,10 @@ class SchemaTool * @param array $mapping * @param array $primaryKeyColumns * @param array $uniqueConstraints + * @param array $addedFks + * @param array $blacklistedFks */ - private function _gatherRelationJoinColumns($joinColumns, $theJoinTable, $class, $mapping, &$primaryKeyColumns, &$uniqueConstraints) + private function _gatherRelationJoinColumns($joinColumns, $theJoinTable, $class, $mapping, &$primaryKeyColumns, &$uniqueConstraints, &$addedFks, &$blacklistedFks) { $localColumns = array(); $foreignColumns = array(); @@ -587,9 +595,27 @@ class SchemaTool } } - $theJoinTable->addForeignKeyConstraint( - $foreignTableName, $localColumns, $foreignColumns, $fkOptions - ); + $compositeName = $theJoinTable->getName().'.'.implode('', $localColumns); + if (isset($addedFks[$compositeName]) + && ($foreignTableName != $addedFks[$compositeName]['foreignTableName'] + || 0 < count(array_diff($foreignColumns, $addedFks[$compositeName]['foreignColumns']))) + ) { + foreach ($theJoinTable->getForeignKeys() as $fkName => $key) { + if (0 === count(array_diff($key->getLocalColumns(), $localColumns)) + && (($key->getForeignTableName() != $foreignTableName) + || 0 < count(array_diff($key->getForeignColumns(), $foreignColumns))) + ) { + $theJoinTable->removeForeignKey($fkName); + break; + } + } + $blacklistedFks[$compositeName] = true; + } elseif (!isset($blacklistedFks[$compositeName])) { + $addedFks[$compositeName] = array('foreignTableName' => $foreignTableName, 'foreignColumns' => $foreignColumns); + $theJoinTable->addUnnamedForeignKeyConstraint( + $foreignTableName, $localColumns, $foreignColumns, $fkOptions + ); + } } /** diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php index 6be981aeb..26c577c51 100755 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php @@ -8,15 +8,15 @@ namespace Doctrine\Tests\Models\SingleTableInheritanceType; */ class Structure { - /** - * @Id - * @Column(type="integer") - * @GeneratedValue(strategy="AUTO") - */ - protected $id; + /** + * @Id + * @Column(type="integer") + * @GeneratedValue(strategy="AUTO") + */ + protected $id; - /** - * @Column(type="string", length=32, nullable=true) - */ - protected $name; + /** + * @Column(type="string", length=32, nullable=true) + */ + protected $name; } diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php index 26bf14688..8af0170cb 100755 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php @@ -10,17 +10,17 @@ use Doctrine\Common\Collections\ArrayCollection; */ class User { - /** - * @Id - * @Column(type="integer") - * @GeneratedValue(strategy="AUTO") - */ - protected $id; + /** + * @Id + * @Column(type="integer") + * @GeneratedValue(strategy="AUTO") + */ + protected $id; - /** - * @Column(type="string", length=32, nullable=true) - */ - protected $name; + /** + * @Column(type="string", length=32, nullable=true) + */ + protected $name; /** * @var ArrayCollection $followedUsers @@ -34,7 +34,8 @@ class User */ protected $followedStructures; - public function __construct() { + public function __construct() + { $this->followedUsers = new ArrayCollection(); $this->followedStructures = new ArrayCollection(); } @@ -52,7 +53,7 @@ class User /** * Add followedUsers * - * @param UserFollowedUser $followedUsers + * @param UserFollowedUser $followedUsers * @return User */ public function addFollowedUser(UserFollowedUser $followedUsers) @@ -65,7 +66,7 @@ class User /** * Remove followedUsers * - * @param UserFollowedUser $followedUsers + * @param UserFollowedUser $followedUsers * @return User */ public function removeFollowedUser(UserFollowedUser $followedUsers) @@ -88,7 +89,7 @@ class User /** * Add followedStructures * - * @param UserFollowedStructure $followedStructures + * @param UserFollowedStructure $followedStructures * @return User */ public function addFollowedStructure(UserFollowedStructure $followedStructures) @@ -101,7 +102,7 @@ class User /** * Remove followedStructures * - * @param UserFollowedStructure $followedStructures + * @param UserFollowedStructure $followedStructures * @return User */ public function removeFollowedStructure(UserFollowedStructure $followedStructures) diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php index 8ee0888cf..8648407da 100755 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php +++ b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php @@ -24,7 +24,7 @@ class UserFollowedStructure extends UserFollowedObject /** * Construct a UserFollowedStructure entity * - * @param User $user + * @param User $user * @param Structure $followedStructure */ public function __construct(User $user, Structure $followedStructure) From 5e2a433828c3197327de7c4700aec2abbebe3491 Mon Sep 17 00:00:00 2001 From: Benjamin Eberlei Date: Mon, 12 Nov 2012 14:05:24 +0100 Subject: [PATCH 3/4] Inlined Test and Entities into DDC2138Test --- .../ORM/Functional/Ticket/DDC2138Test.php | 317 ++++++++++++++++++ .../ORM/Functional/Ticket/DDC258Test.php | 4 +- .../Tests/ORM/Tools/SchemaToolTest.php | 2 +- 3 files changed, 319 insertions(+), 4 deletions(-) create mode 100644 tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2138Test.php diff --git a/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2138Test.php b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2138Test.php new file mode 100644 index 000000000..87fb16d84 --- /dev/null +++ b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2138Test.php @@ -0,0 +1,317 @@ +_em; + $schemaTool = new SchemaTool($em); + + $classes = array( + $em->getClassMetadata(__NAMESPACE__ . '\DDC2138User'), + $em->getClassMetadata(__NAMESPACE__ . '\DDC2138Structure'), + $em->getClassMetadata(__NAMESPACE__ . '\DDC2138UserFollowedObject'), + $em->getClassMetadata(__NAMESPACE__ . '\DDC2138UserFollowedStructure'), + $em->getClassMetadata(__NAMESPACE__ . '\DDC2138UserFollowedUser') + ); + + $schema = $schemaTool->getSchemaFromMetadata($classes); + $this->assertTrue($schema->hasTable('users_followed_objects'), "Table users_followed_objects should exist."); + + /* @var $table \Doctrine\DBAL\Schema\Table */ + $table = ($schema->getTable('users_followed_objects')); + $this->assertTrue($table->columnsAreIndexed(array('object_id'))); + $this->assertTrue($table->columnsAreIndexed(array('user_id'))); + $foreignKeys = $table->getForeignKeys(); + $this->assertCount(1, $foreignKeys, 'user_id column has to have FK, but not object_id'); + + /* @var $fk \Doctrine\DBAL\Schema\ForeignKeyConstraint */ + $fk = reset($foreignKeys); + $this->assertEquals('users', $fk->getForeignTableName()); + + $localColumns = $fk->getLocalColumns(); + $this->assertContains('user_id', $localColumns); + $this->assertCount(1, $localColumns); + } +} + + + +/** + * @Table(name="structures") + * @Entity + */ +class DDC2138Structure +{ + /** + * @Id + * @Column(type="integer") + * @GeneratedValue(strategy="AUTO") + */ + protected $id; + + /** + * @Column(type="string", length=32, nullable=true) + */ + protected $name; +} + +/** + * @Entity + * @Table(name="users_followed_objects") + * @InheritanceType("SINGLE_TABLE") + * @DiscriminatorColumn(name="object_type", type="smallint") + * @DiscriminatorMap({4 = "DDC2138UserFollowedUser", 3 = "DDC2138UserFollowedStructure"}) + */ +abstract class DDC2138UserFollowedObject +{ + /** + * @var integer $id + * + * @Column(name="id", type="integer") + * @Id + * @GeneratedValue(strategy="AUTO") + */ + protected $id; + + /** + * Get id + * + * @return integer + */ + public function getId() + { + return $this->id; + } +} + +/** + * @Entity + */ +class DDC2138UserFollowedStructure extends DDC2138UserFollowedObject +{ + /** + * @ManyToOne(targetEntity="DDC2138User", inversedBy="followedStructures") + * @JoinColumn(name="user_id", referencedColumnName="id", nullable=false) + * @var User $user + */ + protected $user; + + /** + * @ManyToOne(targetEntity="DDC2138Structure") + * @JoinColumn(name="object_id", referencedColumnName="id", nullable=false) + * @var Structure $followedStructure + */ + private $followedStructure; + + /** + * Construct a UserFollowedStructure entity + * + * @param User $user + * @param Structure $followedStructure + */ + public function __construct(User $user, Structure $followedStructure) + { + $this->user = $user; + $this->followedStructure = $followedStructure; + } + + /** + * + * @return User + */ + public function getUser() + { + return $this->user; + } + + /** + * Gets followed structure + * + * @return Structure + */ + public function getFollowedStructure() + { + return $this->followedStructure; + } +} + +/** + * @Entity + */ +class DDC2138UserFollowedUser extends DDC2138UserFollowedObject +{ + /** + * @ManyToOne(targetEntity="DDC2138User", inversedBy="followedUsers") + * @JoinColumn(name="user_id", referencedColumnName="id", nullable=false) + * @var User $user + */ + protected $user; + + /** + * @ManyToOne(targetEntity="DDC2138User") + * @JoinColumn(name="object_id", referencedColumnName="id", nullable=false) + * @var User $user + */ + private $followedUser; + + /** + * Construct a UserFollowedUser entity + * + * @param User $user + * @param User $followedUser + * @param bool $giveAgency + */ + public function __construct(User $user, User $followedUser) + { + $this->user = $user; + $this->followedUser = $followedUser; + } + + /** + * {@inheritdoc} + */ + public function getUser() + { + return $this->user; + } + + /** + * Gets followed user + * + * @return User + */ + public function getFollowedUser() + { + return $this->followedUser; + } + +} + +/** + * @Table(name="users") + * @Entity + */ +class DDC2138User +{ + /** + * @Id + * @Column(type="integer") + * @GeneratedValue(strategy="AUTO") + */ + protected $id; + + /** + * @Column(type="string", length=32, nullable=true) + */ + protected $name; + + /** + * @var ArrayCollection $followedUsers + * @OneToMany(targetEntity="DDC2138UserFollowedUser", mappedBy="user", cascade={"persist"}, orphanRemoval=true) + */ + protected $followedUsers; + + /** + * @var ArrayCollection $followedStructures + * @OneToMany(targetEntity="DDC2138UserFollowedStructure", mappedBy="user", cascade={"persist"}, orphanRemoval=true) + */ + protected $followedStructures; + + public function __construct() + { + $this->followedUsers = new ArrayCollection(); + $this->followedStructures = new ArrayCollection(); + } + + /* + * Remove followers + * + * @param UserFollowedUser $followers + */ + private function removeFollower(UserFollowedUser $followers) + { + $this->followers->removeElement($followers); + } + + /** + * Add followedUsers + * + * @param UserFollowedUser $followedUsers + * @return User + */ + public function addFollowedUser(UserFollowedUser $followedUsers) + { + $this->followedUsers[] = $followedUsers; + + return $this; + } + + /** + * Remove followedUsers + * + * @param UserFollowedUser $followedUsers + * @return User + */ + public function removeFollowedUser(UserFollowedUser $followedUsers) + { + $this->followedUsers->removeElement($followedUsers); + + return $this; + } + + /** + * Get followedUsers + * + * @return Doctrine\Common\Collections\Collection + */ + public function getFollowedUsers() + { + return $this->followedUsers; + } + + /** + * Add followedStructures + * + * @param UserFollowedStructure $followedStructures + * @return User + */ + public function addFollowedStructure(UserFollowedStructure $followedStructures) + { + $this->followedStructures[] = $followedStructures; + + return $this; + } + + /** + * Remove followedStructures + * + * @param UserFollowedStructure $followedStructures + * @return User + */ + public function removeFollowedStructure(UserFollowedStructure $followedStructures) + { + $this->followedStructures->removeElement($followedStructures); + + return $this; + } + + /** + * Get followedStructures + * + * @return Doctrine\Common\Collections\Collection + */ + public function getFollowedStructures() + { + return $this->followedStructures; + } +} diff --git a/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC258Test.php b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC258Test.php index 0d766eaa0..39b918d26 100644 --- a/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC258Test.php +++ b/tests/Doctrine/Tests/ORM/Functional/Ticket/DDC258Test.php @@ -1,8 +1,6 @@ schemaCalled = true; } -} \ No newline at end of file +} From 624ef309f05eb0cb9cee1cbbfaec6802952a59bd Mon Sep 17 00:00:00 2001 From: Benjamin Eberlei Date: Mon, 12 Nov 2012 15:01:20 +0100 Subject: [PATCH 4/4] Remove unnecssary code --- .../SingleTableInheritanceType/Structure.php | 22 ---- .../SingleTableInheritanceType/User.php | 124 ------------------ .../UserFollowedObject.php | 32 ----- .../UserFollowedStructure.php | 54 -------- .../UserFollowedUser.php | 55 -------- .../Tests/ORM/Tools/SchemaToolTest.php | 32 ----- 6 files changed, 319 deletions(-) delete mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php delete mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/User.php delete mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php delete mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php delete mode 100755 tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php deleted file mode 100755 index 26c577c51..000000000 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/Structure.php +++ /dev/null @@ -1,22 +0,0 @@ -followedUsers = new ArrayCollection(); - $this->followedStructures = new ArrayCollection(); - } - - /* - * Remove followers - * - * @param UserFollowedUser $followers - */ - private function removeFollower(UserFollowedUser $followers) - { - $this->followers->removeElement($followers); - } - - /** - * Add followedUsers - * - * @param UserFollowedUser $followedUsers - * @return User - */ - public function addFollowedUser(UserFollowedUser $followedUsers) - { - $this->followedUsers[] = $followedUsers; - - return $this; - } - - /** - * Remove followedUsers - * - * @param UserFollowedUser $followedUsers - * @return User - */ - public function removeFollowedUser(UserFollowedUser $followedUsers) - { - $this->followedUsers->removeElement($followedUsers); - - return $this; - } - - /** - * Get followedUsers - * - * @return Doctrine\Common\Collections\Collection - */ - public function getFollowedUsers() - { - return $this->followedUsers; - } - - /** - * Add followedStructures - * - * @param UserFollowedStructure $followedStructures - * @return User - */ - public function addFollowedStructure(UserFollowedStructure $followedStructures) - { - $this->followedStructures[] = $followedStructures; - - return $this; - } - - /** - * Remove followedStructures - * - * @param UserFollowedStructure $followedStructures - * @return User - */ - public function removeFollowedStructure(UserFollowedStructure $followedStructures) - { - $this->followedStructures->removeElement($followedStructures); - - return $this; - } - - /** - * Get followedStructures - * - * @return Doctrine\Common\Collections\Collection - */ - public function getFollowedStructures() - { - return $this->followedStructures; - } -} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php deleted file mode 100755 index 9996660c0..000000000 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedObject.php +++ /dev/null @@ -1,32 +0,0 @@ -id; - } -} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php deleted file mode 100755 index 8648407da..000000000 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedStructure.php +++ /dev/null @@ -1,54 +0,0 @@ -user = $user; - $this->followedStructure = $followedStructure; - } - - /** - * - * @return User - */ - public function getUser() - { - return $this->user; - } - - /** - * Gets followed structure - * - * @return Structure - */ - public function getFollowedStructure() - { - return $this->followedStructure; - } -} diff --git a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php b/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php deleted file mode 100755 index 0f90a2416..000000000 --- a/tests/Doctrine/Tests/Models/SingleTableInheritanceType/UserFollowedUser.php +++ /dev/null @@ -1,55 +0,0 @@ -user = $user; - $this->followedUser = $followedUser; - } - - /** - * {@inheritdoc} - */ - public function getUser() - { - return $this->user; - } - - /** - * Gets followed user - * - * @return User - */ - public function getFollowedUser() - { - return $this->followedUser; - } - -} diff --git a/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php b/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php index 4152b0554..0ca1bf4cc 100644 --- a/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php +++ b/tests/Doctrine/Tests/ORM/Tools/SchemaToolTest.php @@ -32,38 +32,6 @@ class SchemaToolTest extends \Doctrine\Tests\OrmTestCase $this->assertTrue($schema->getTable('cms_users')->columnsAreIndexed(array('username')), "username column should be indexed."); } - public function testForeignKeyOnSTIWithMultipleMapping() - { - $em = $this->_getTestEntityManager(); - $schemaTool = new SchemaTool($em); - - $classes = array( - $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\User'), - $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\Structure'), - $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedObject'), - $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedStructure'), - $em->getClassMetadata('Doctrine\Tests\Models\SingleTableInheritanceType\UserFollowedUser') - ); - - $schema = $schemaTool->getSchemaFromMetadata($classes); - $this->assertTrue($schema->hasTable('users_followed_objects'), "Table users_followed_objects should exist."); - - /* @var $table \Doctrine\DBAL\Schema\Table */ - $table = ($schema->getTable('users_followed_objects')); - $this->assertTrue($table->columnsAreIndexed(array('object_id'))); - $this->assertTrue($table->columnsAreIndexed(array('user_id'))); - $foreignKeys = $table->getForeignKeys(); - $this->assertCount(1, $foreignKeys, 'user_id column has to have FK, but not object_id'); - - /* @var $fk \Doctrine\DBAL\Schema\ForeignKeyConstraint */ - $fk = reset($foreignKeys); - $this->assertEquals('users', $fk->getForeignTableName()); - - $localColumns = $fk->getLocalColumns(); - $this->assertContains('user_id', $localColumns); - $this->assertCount(1, $localColumns); - } - public function testAnnotationOptionsAttribute() { $em = $this->_getTestEntityManager();