fixup! fix(Table): support for index declared with single field as st…

…ring
FriendsOfSymfony1 · Apr 12, 2024 · a7cfab2 · a7cfab2
1 parent df3108e
commit a7cfab2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/lib/Doctrine/Table.php b/lib/Doctrine/Table.php
@@ -880,7 +880,9 @@ public function addCheckConstraint($definition, $name)
     public function addIndex($index, array $definition)
     {
         if (isset($definition['fields'])) {
-            foreach ((array) $definition['fields'] as $key => $field) {
+            $definition['fields'] = (array) $definition['fields'];
+
+            foreach ($definition['fields'] as $key => $field) {
                 if (is_numeric($key)) {
                     $definition['fields'][$key] = $this->getColumnName($field);
                 } else {

diff --git a/tests/Table/IndexesUsingSingleFieldStringTestCase.php b/tests/Table/IndexesUsingSingleFieldStringTestCase.php
@@ -2,12 +2,7 @@
 
 class Doctrine_Table_IndexesUsingSingleFieldString_TestCase extends Doctrine_UnitTestCase
 {
-    public function prepareTables()
-    {
-        $this->tables = array('IndexDeclaredWithSingleFieldStringRecord');
-
-        parent::prepareTables();
-    }
+    protected $tables = array('IndexDeclaredWithSingleFieldStringRecord');
 
     public function testSupportIndexesUsingSingleFieldString()
     {