Doctrine DBAL

When creating foreign key constraints, the AbstractPlatform class does not quote the referenced table.

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.2.2
  • Fix Version/s: None
  • Component/s: Platforms
  • Security Level: All
  • Labels:
    None

Description

I have a table that is using a reserved MySQL keyword (right) and it's quoted in all the generated SQL but not in the foreign key constraint.

When I use doctrine.php orm:schema-tool:create I get this SQL line:
ALTER TABLE website ADD CONSTRAINT FK_476F5DE7B196EE6E FOREIGN KEY (right_id) REFERENCES right (id)

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabio B. Silva added a comment -

Hi,

Could you try it with doctrine 2.3 ?
There are lot of changes about quotes.
if it fails even with 2.3 changes please attach your entities.

Thanks..

Show
Fabio B. Silva added a comment - Hi, Could you try it with doctrine 2.3 ? There are lot of changes about quotes. if it fails even with 2.3 changes please attach your entities. Thanks..
Hide
Permalink
Padraig O'Sullivan added a comment - - edited

I verified this is fixed in trunk by creating a test. Worth creating pull request to add test case?

public function testQuotedForeignKeySQL()
{   
    $tableOld = new Table("website");
    $tableOld->addColumn('right_id', 'integer');
    $tableOld->addUnnamedForeignKeyConstraint('right', array('right_id'), array('id'));

    $sqls = array();
    foreach ($tableOld->getForeignKeys() AS $fk) {
        $sqls[] = $this->platform->getCreateForeignKeySQL($fk, $tableOld);
    }   

    $this->assertEquals(array("ALTER TABLE website ADD CONSTRAINT FK_476F5DE754976835 FOREIGN KEY (right_id) REFERENCES `right` (id)"), $sqls);
}
Show
Padraig O'Sullivan added a comment - - edited I verified this is fixed in trunk by creating a test. Worth creating pull request to add test case? public function testQuotedForeignKeySQL() { $tableOld = new Table( "website" ); $tableOld->addColumn('right_id', 'integer'); $tableOld->addUnnamedForeignKeyConstraint('right', array('right_id'), array('id')); $sqls = array(); foreach ($tableOld->getForeignKeys() AS $fk) { $sqls[] = $ this ->platform->getCreateForeignKeySQL($fk, $tableOld); } $ this ->assertEquals(array( "ALTER TABLE website ADD CONSTRAINT FK_476F5DE754976835 FOREIGN KEY (right_id) REFERENCES `right` (id)" ), $sqls); }
Hide
Permalink
Christophe Coevoet added a comment -

Can you paste your mapping for your right entity ?

Show
Christophe Coevoet added a comment - Can you paste your mapping for your right entity ?
Hide
Permalink
Alexander added a comment -

Can you provide us with more feedback?

Show
Alexander added a comment - Can you provide us with more feedback?
Hide
Permalink
Benjamin Eberlei added a comment -

Seems fixed as per comment of Patrick.

Show
Benjamin Eberlei added a comment - Seems fixed as per comment of Patrick.

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Marc-André Joyal
Vote (0)
Watch (6)

Dates

  • Created:
    Updated:
    Resolved: