Merge pull request #128 from phpcr/not_fix

Fixed parenthesis for NOT constraint

Author: dbu

src/PHPCR/Util/QOM/BaseSqlGenerator.php

@@ -83,7 +83,7 @@ public function evalOr($constraint1, $constraint2)
      */
     public function evalNot($constraint)
     {
-        return "NOT $constraint";
+        return "(NOT $constraint)";
     }
 
     /**

tests/PHPCR/Tests/Util/QOM/BaseSqlGeneratorTest.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace PHPCR\Tests\Util\QOM;
+
+use PHPCR\Util\QOM\Sql1Generator;
+use PHPCR\Util\ValueConverter;
+
+abstract class BaseSqlGeneratorTest extends \PHPUnit_Framework_TestCase
+{
+    public function testNot()
+    {
+        $string = $this->generator->evalNot("foo = bar");
+        $this->assertSame("(NOT foo = bar)", $string);
+    }
+}

tests/PHPCR/Tests/Util/QOM/Sql2GeneratorTest.php

@@ -5,7 +5,7 @@
 use PHPCR\Util\QOM\Sql2Generator;
 use PHPCR\Util\ValueConverter;
 
-class Sql2GeneratorTest extends \PHPUnit_Framework_TestCase
+class Sql2GeneratorTest extends BaseSqlGeneratorTest
 {
     protected $generator;