Merge pull request #241 from peter-gribanov/expr_to_string

MemberOfX::getFilter() and LogicX::getFilter() should return a string

src/Filter/MemberOfX.php

@@ -61,7 +61,7 @@ public function getFilter(QueryBuilder $qb, $dqlAlias)
         $field = ArgumentToOperandConverter::toField($this->field);
         $value = ArgumentToOperandConverter::toValue($this->value);
 
-        return $qb->expr()->isMemberOf(
+        return (string) $qb->expr()->isMemberOf(
             $value->transform($qb, $dqlAlias),
             $field->transform($qb, $dqlAlias)
         );

src/Logic/LogicX.php

@@ -78,7 +78,7 @@ public function getFilter(QueryBuilder $qb, $dqlAlias)
             );
         }
 
-        return call_user_func_array($expression, $children);
+        return (string) call_user_func_array($expression, $children);
     }
 
     /**

tests/Filter/MemberOfXSpec.php

@@ -41,21 +41,16 @@ public function it_is_an_expression()
         $this->shouldBeAnInstanceOf(Filter::class);
     }
 
-    public function it_returns_expression_func_object(
-        QueryBuilder $qb,
-        ArrayCollection $parameters,
-        Expr $exp,
-        Comparison $exp_comparison
-    ) {
+    public function it_returns_expression_func_object(QueryBuilder $qb, ArrayCollection $parameters, Expr $exp)
+    {
+        $exp_comparison = new Comparison(':comparison_10', 'MEMBER OF', 'a.age');
         $qb->expr()->willReturn($exp);
         $qb->getParameters()->willReturn($parameters);
         $parameters->count()->willReturn(10);
 
         $qb->setParameter('comparison_10', 18, null)->shouldBeCalled();
         $exp->isMemberOf(':comparison_10', 'a.age')->willReturn($exp_comparison);
 
-        $comparison = $this->getFilter($qb, null);
-
-        $comparison->shouldReturn($exp_comparison);
+        $this->getFilter($qb, null)->shouldReturn(':comparison_10 MEMBER OF a.age');
     }
 }