Merge branch 'DDC-1018'
This commit is contained in:
commit
60203af9b2
@ -718,6 +718,14 @@ class SqlWalker implements TreeWalker
|
||||
$join = $joinVarDecl->join;
|
||||
$joinType = $join->joinType;
|
||||
|
||||
if ($joinVarDecl->indexBy) {
|
||||
// For Many-To-One or One-To-One associations this obviously makes no sense, but is ignored silently.
|
||||
$this->_rsm->addIndexBy(
|
||||
$joinVarDecl->indexBy->simpleStateFieldPathExpression->identificationVariable,
|
||||
$joinVarDecl->indexBy->simpleStateFieldPathExpression->field
|
||||
);
|
||||
}
|
||||
|
||||
if ($joinType == AST\Join::JOIN_TYPE_LEFT || $joinType == AST\Join::JOIN_TYPE_LEFTOUTER) {
|
||||
$sql = ' LEFT JOIN ';
|
||||
} else {
|
||||
|
@ -6,6 +6,9 @@ use DateTime;
|
||||
|
||||
require_once __DIR__ . '/../../../TestInit.php';
|
||||
|
||||
/**
|
||||
* @group DDC-618
|
||||
*/
|
||||
class DDC618Test extends \Doctrine\Tests\OrmFunctionalTestCase
|
||||
{
|
||||
protected function setUp()
|
||||
@ -13,7 +16,8 @@ class DDC618Test extends \Doctrine\Tests\OrmFunctionalTestCase
|
||||
parent::setUp();
|
||||
try {
|
||||
$this->_schemaTool->createSchema(array(
|
||||
$this->_em->getClassMetadata(__NAMESPACE__ . '\DDC618Author')
|
||||
$this->_em->getClassMetadata(__NAMESPACE__ . '\DDC618Author'),
|
||||
$this->_em->getClassMetadata(__NAMESPACE__ . '\DDC618Book')
|
||||
));
|
||||
|
||||
// Create author 10/Joe with two books 22/JoeA and 20/JoeB
|
||||
@ -26,6 +30,10 @@ class DDC618Test extends \Doctrine\Tests\OrmFunctionalTestCase
|
||||
$author = new DDC618Author();
|
||||
$author->id = 11;
|
||||
$author->name = 'Alice';
|
||||
$author->addBook('In Wonderland');
|
||||
$author->addBook('Reloaded');
|
||||
$author->addBook('Test');
|
||||
|
||||
$this->_em->persist($author);
|
||||
|
||||
$this->_em->flush();
|
||||
@ -58,11 +66,71 @@ class DDC618Test extends \Doctrine\Tests\OrmFunctionalTestCase
|
||||
$this->assertArrayHasKey('Joe', $result, "INDEX BY A.name should return an index by the name of 'Joe'.");
|
||||
$this->assertArrayHasKey('Alice', $result, "INDEX BY A.name should return an index by the name of 'Alice'.");
|
||||
}
|
||||
|
||||
/**
|
||||
* @group DDC-1018
|
||||
*/
|
||||
public function testIndexByJoin()
|
||||
{
|
||||
$dql = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A '.
|
||||
'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
|
||||
$result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
|
||||
|
||||
$this->assertEquals(3, count($result[0]->books)); // Alice, Joe doesnt appear because he has no books.
|
||||
$this->assertEquals('Alice', $result[0]->name);
|
||||
$this->assertTrue( isset($result[0]->books["In Wonderland"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[0]->books["Reloaded"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[0]->books["Test"] ), "Indexing by title should have books by title.");
|
||||
|
||||
$result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_ARRAY);
|
||||
|
||||
$this->assertEquals(3, count($result[0]['books'])); // Alice, Joe doesnt appear because he has no books.
|
||||
$this->assertEquals('Alice', $result[0]['name']);
|
||||
$this->assertTrue( isset($result[0]['books']["In Wonderland"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[0]['books']["Reloaded"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[0]['books']["Test"] ), "Indexing by title should have books by title.");
|
||||
}
|
||||
|
||||
/**
|
||||
* @group DDC-1018
|
||||
*/
|
||||
public function testIndexByToOneJoinSilentlyIgnored()
|
||||
{
|
||||
$dql = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
|
||||
'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
|
||||
$result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
|
||||
|
||||
$this->assertInstanceOf('Doctrine\Tests\ORM\Functional\Ticket\DDC618Book', $result[0]);
|
||||
$this->assertInstanceOf('Doctrine\Tests\ORM\Functional\Ticket\DDC618Author', $result[0]->author);
|
||||
|
||||
$dql = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
|
||||
'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
|
||||
$result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_ARRAY);
|
||||
|
||||
$this->assertEquals("Alice", $result[0]['author']['name']);
|
||||
}
|
||||
|
||||
/**
|
||||
* @group DDC-1018
|
||||
*/
|
||||
public function testCombineIndexBy()
|
||||
{
|
||||
$dql = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A INDEX BY A.id '.
|
||||
'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
|
||||
$result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
|
||||
|
||||
$this->assertArrayHasKey(11, $result); // Alice
|
||||
|
||||
$this->assertEquals(3, count($result[11]->books)); // Alice, Joe doesnt appear because he has no books.
|
||||
$this->assertEquals('Alice', $result[11]->name);
|
||||
$this->assertTrue( isset($result[11]->books["In Wonderland"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[11]->books["Reloaded"] ), "Indexing by title should have books by title.");
|
||||
$this->assertTrue( isset($result[11]->books["Test"] ), "Indexing by title should have books by title.");
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @Entity
|
||||
* @Table (name="ddc618author", uniqueConstraints={ @Index (name="UQ_authorname", columns={ "name" }) })
|
||||
*/
|
||||
class DDC618Author
|
||||
{
|
||||
@ -75,8 +143,43 @@ class DDC618Author
|
||||
/** @Column(type="string") */
|
||||
public $name;
|
||||
|
||||
/**
|
||||
* @OneToMany(targetEntity="DDC618Book", mappedBy="author", cascade={"persist"})
|
||||
*/
|
||||
public $books;
|
||||
|
||||
public function __construct()
|
||||
{
|
||||
$this->books = new \Doctrine\Common\Collections\ArrayCollection;
|
||||
}
|
||||
|
||||
public function addBook($title)
|
||||
{
|
||||
$book = new DDC618Book($title, $this);
|
||||
$this->books[] = $book;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @Entity
|
||||
*/
|
||||
class DDC618Book
|
||||
{
|
||||
/**
|
||||
* @Id @GeneratedValue
|
||||
* @Column(type="integer")
|
||||
*/
|
||||
public $id;
|
||||
|
||||
/** @column(type="string") */
|
||||
public $title;
|
||||
|
||||
/** @ManyToOne(targetEntity="DDC618Author", inversedBy="books") */
|
||||
public $author;
|
||||
|
||||
function __construct($title, $author)
|
||||
{
|
||||
$this->title = $title;
|
||||
$this->author = $author;
|
||||
}
|
||||
}
|
Loading…
Reference in New Issue
Block a user