Merge branch 'fix/#2286-foreign-keys-sql-database-parameter-evaluation' into 2.5

Close #2286

lib/Doctrine/DBAL/Platforms/MySqlPlatform.php

@@ -194,10 +194,7 @@ class MySqlPlatform extends AbstractPlatform
                "  c.constraint_name = k.constraint_name AND ".
                "  c.table_name = $table */ WHERE k.table_name = $table";
 
-        // @TODO: This needs fixing. The condition has to be inverted.
-        // When fixed, AbstractMySQLPlatformTestCase::testQuotesDatabaseNameInListTableForeignKeysSQL test
-        // has to be completed.
-        $databaseNameSql = null === $database ? $database : 'DATABASE()';
+        $databaseNameSql = null === $database ? 'DATABASE()' : $database;
 
         $sql .= " AND k.table_schema = $databaseNameSql /*!50116 AND c.constraint_schema = $databaseNameSql */";
         $sql .= " AND k.`REFERENCED_COLUMN_NAME` is not NULL";

tests/Doctrine/Tests/DBAL/Platforms/AbstractMySQLPlatformTestCase.php

@@ -799,8 +799,6 @@ abstract class AbstractMySQLPlatformTestCase extends AbstractPlatformTestCase
      */
     public function testQuotesDatabaseNameInListTableForeignKeysSQL()
     {
-        $this->markTestIncomplete('Test does not work due to a bug in MySqlplatform::getListTableForeignKeysSQL');
-
         $this->assertContains("'Foo''Bar\\\\'", $this->_platform->getListTableForeignKeysSQL('foo_table', "Foo'Bar\\"), '', true);
     }
 
@@ -819,4 +817,16 @@ abstract class AbstractMySQLPlatformTestCase extends AbstractPlatformTestCase
     {
         $this->assertContains("'Foo''Bar\\\\'", $this->_platform->getListTableColumnsSQL('foo_table', "Foo'Bar\\"), '', true);
     }
+
+    public function testListTableForeignKeysSQLEvaluatesDatabase()
+    {
+        $sql = $this->_platform->getListTableForeignKeysSQL('foo');
+
+        $this->assertContains('DATABASE()', $sql);
+
+        $sql = $this->_platform->getListTableForeignKeysSQL('foo', 'bar');
+
+        $this->assertContains('bar', $sql);
+        $this->assertNotContains('DATABASE()', $sql);
+    }
 }