Merge pull request #472 from deeky666/fix-index-flags

Fix index flags
doctrine · Dec 28, 2013 · d473423 · d473423
2 parents 2196da9 + 1f72848
commit d473423
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/lib/Doctrine/DBAL/Schema/Index.php b/lib/Doctrine/DBAL/Schema/Index.php
@@ -259,7 +259,7 @@ public function getFlags()
      */
     public function addFlag($flag)
     {
-        $this->flags[strtolower($flag)] = true;
+        $this->_flags[strtolower($flag)] = true;
 
         return $this;
     }
@@ -273,7 +273,7 @@ public function addFlag($flag)
      */
     public function hasFlag($flag)
     {
-        return isset($this->flags[strtolower($flag)]);
+        return isset($this->_flags[strtolower($flag)]);
     }
 
     /**
@@ -285,6 +285,6 @@ public function hasFlag($flag)
      */
     public function removeFlag($flag)
     {
-        unset($this->flags[strtolower($flag)]);
+        unset($this->_flags[strtolower($flag)]);
     }
 }
diff --git a/tests/Doctrine/Tests/DBAL/Schema/IndexTest.php b/tests/Doctrine/Tests/DBAL/Schema/IndexTest.php
@@ -89,13 +89,16 @@ public function testFlags()
     {
         $idx1 = $this->createIndex();
         $this->assertFalse($idx1->hasFlag('clustered'));
+        $this->assertEmpty($idx1->getFlags());
 
         $idx1->addFlag('clustered');
         $this->assertTrue($idx1->hasFlag('clustered'));
         $this->assertTrue($idx1->hasFlag('CLUSTERED'));
+        $this->assertSame(array('clustered'), $idx1->getFlags());
 
         $idx1->removeFlag('clustered');
         $this->assertFalse($idx1->hasFlag('clustered'));
+        $this->assertEmpty($idx1->getFlags());
     }
 
     /**