The getOp method renamed to getOperator

ihortymoshenko · May 4, 2013 · d3aa934 · d3aa934
1 parent eb91524
commit d3aa934
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/src/IMT/DataGrid/Filter/Reflection/Doctrine/ORM/FilterReflection.php b/src/IMT/DataGrid/Filter/Reflection/Doctrine/ORM/FilterReflection.php
@@ -59,7 +59,7 @@ public function reflect(FilterInterface $filter = null)
             $ruleReflectionClass = sprintf(
                 '%s\Rule\%sRuleReflection',
                 __NAMESPACE__,
-                ucfirst($rule->getOp())
+                ucfirst($rule->getOperator())
             );
 
             if (!class_exists($ruleReflectionClass)) {

diff --git a/src/IMT/DataGrid/Filter/Rule.php b/src/IMT/DataGrid/Filter/Rule.php
@@ -70,7 +70,7 @@ public function getField()
     /**
      * {@inheritDoc}
      */
-    public function getOp()
+    public function getOperator()
     {
         return $this->options['op'];
     }

diff --git a/src/IMT/DataGrid/Filter/RuleInterface.php b/src/IMT/DataGrid/Filter/RuleInterface.php
@@ -38,5 +38,5 @@ public function getField();
      *
      * @return string
      */
-    public function getOp();
+    public function getOperator();
 }
diff --git a/tests/IMT/DataGrid/Tests/Filter/Reflection/Doctrine/ORM/AbstractTestCase.php b/tests/IMT/DataGrid/Tests/Filter/Reflection/Doctrine/ORM/AbstractTestCase.php
@@ -77,7 +77,7 @@ protected function getRuleMock($op = 'bn')
             ->will($this->returnValue('field'));
         $rule
             ->expects($this->any())
-            ->method('getOp')
+            ->method('getOperator')
             ->will($this->returnValue($op));
 
         return $rule;

diff --git a/tests/IMT/DataGrid/Tests/Filter/RuleTest.php b/tests/IMT/DataGrid/Tests/Filter/RuleTest.php
@@ -82,10 +82,10 @@ public function testGetField()
     }
 
     /**
-     * @covers IMT\DataGrid\Filter\Rule::getOp
+     * @covers IMT\DataGrid\Filter\Rule::getOperator
      */
-    public function testGetOp()
+    public function testGetOperator()
     {
-        $this->assertEquals('bn', $this->rule->getOp());
+        $this->assertEquals('bn', $this->rule->getOperator());
     }
 }