update composer, adjust is_null to null ===

Author: dbu

composer.json

@@ -28,7 +28,7 @@
     ],
     "require": {
         "php": ">=5.3.0",
-        "phpcr/phpcr": "~2.1.0-beta10",
+        "phpcr/phpcr": "~2.1.0-RC1",
         "symfony/console": "~2.0"
     },
     "autoload": {

src/PHPCR/Util/CND/Scanner/AbstractScanner.php

@@ -41,7 +41,7 @@ public function applyFilters(Token $token)
 
             $token = $filter->filter($token);
 
-            if (is_null($token)) {
+            if (null === $token) {
                 break;
             }
         }

src/PHPCR/Util/QOM/QomToSql2QueryConverter.php

@@ -116,7 +116,7 @@ protected function convertSameNodeJoinCondition(QOM\SameNodeJoinConditionInterfa
         return $this->generator->evalSameNodeJoinCondition(
             $condition->getSelector1Name(),
             $condition->getSelector2Name(),
-            ! is_null($condition->getSelector2Path()) ? $this->convertPath($condition->getSelector2Path()) : null);
+            null !== $condition->getSelector2Path() ? $this->convertPath($condition->getSelector2Path()) : null);
     }
 
     /**

src/PHPCR/Util/QOM/Sql2Generator.php

@@ -17,16 +17,19 @@ class Sql2Generator extends BaseSqlGenerator
      * Selector ::= nodeTypeName ['AS' selectorName]
      * nodeTypeName ::= Name
      *
-     * @param string $nodeTypeName The node type of the selector. If it does not contain starting and ending brackets ([]) they will be added automatically
-     * @param string $selectorName
+     * @param string      $nodeTypeName The node type of the selector. If it
+     *      does not contain starting and ending brackets ([]) they will be
+     *      added automatically.
+     * @param string|null $selectorName The selector name. If it is different than
+     *      the nodeTypeName, the alias is declared.
      *
      * @return string
      */
     public function evalSelector($nodeTypeName, $selectorName = null)
     {
         $sql2 = $this->addBracketsIfNeeded($nodeTypeName);
 
-        if (! is_null($selectorName) && $nodeTypeName !== $selectorName) {
+        if (null !== $selectorName && $nodeTypeName !== $selectorName) {
             // if the selector name is the same as the type name, this is implicit for sql2
             $sql2 .= ' AS ' . $selectorName;
         }
@@ -112,7 +115,9 @@ public function evalSameNodeJoinCondition($sel1Name, $sel2Name, $sel2Path = null
             . $this->addBracketsIfNeeded($sel1Name) . ', '
             . $this->addBracketsIfNeeded($sel2Name)
         ;
-        $sql2 .= ! is_null($sel2Path) ? ', ' . $sel2Path : '';
+        if (null !== $sel2Path) {
+            $sql2 .= ', ' . $sel2Path;
+        }
         $sql2 .= ')';
 
         return $sql2;
@@ -165,7 +170,7 @@ public function evalDescendantNodeJoinCondition($descendantSelectorName, $ancest
     public function evalSameNode($path, $selectorName = null)
     {
         $sql2 = 'ISSAMENODE(';
-        $sql2 .= is_null($selectorName) ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
+        $sql2 .= null === $selectorName ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
         $sql2 .= ')';
 
         return $sql2;
@@ -180,7 +185,7 @@ public function evalSameNode($path, $selectorName = null)
     public function evalChildNode($path, $selectorName = null)
     {
         $sql2 = 'ISCHILDNODE(';
-        $sql2 .= is_null($selectorName) ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
+        $sql2 .= null === $selectorName ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
         $sql2 .= ')';
 
         return $sql2;
@@ -195,7 +200,7 @@ public function evalChildNode($path, $selectorName = null)
     public function evalDescendantNode($path, $selectorName = null)
     {
         $sql2 = 'ISDESCENDANTNODE(';
-        $sql2 .= is_null($selectorName) ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
+        $sql2 .= null === $selectorName ? $path : $this->addBracketsIfNeeded($selectorName) . ', ' . $path;
         $sql2 .= ')';
 
         return $sql2;
@@ -289,7 +294,7 @@ public function evalFullTextSearchScore($selectorValue = null)
      */
     public function evalPropertyValue($propertyName, $selectorName = null)
     {
-        $sql2 = ! is_null($selectorName) ? $this->addBracketsIfNeeded($selectorName) . '.' : '';
+        $sql2 = null !== $selectorName ? $this->addBracketsIfNeeded($selectorName) . '.' : '';
         if (false !== strpos($propertyName, ':')) {
             $propertyName = "[$propertyName]";
         }
@@ -340,11 +345,11 @@ public function evalColumns($columns)
     public function evalColumn($selectorName, $propertyName = null, $colname = null)
     {
         $sql2 = '';
-        if (! is_null($selectorName) && is_null($propertyName) && is_null($colname)) {
+        if (null !== $selectorName && null === $propertyName && null === $colname) {
             $sql2 .= $this->addBracketsIfNeeded($selectorName) . '.*';
         } else {
             $sql2 .= $this->evalPropertyValue($propertyName, $selectorName);
-            if (!is_null($colname) && $colname !== $propertyName) {
+            if (null !== $colname && $colname !== $propertyName) {
                 // if the column name is the same as the property name, this is implicit for sql2
                 $sql2 .=  ' AS ' . $colname;
             }

src/PHPCR/Util/QOM/Sql2ToQomQueryConverter.php

@@ -172,7 +172,6 @@ protected function parseSelector()
      */
     protected function parseName()
     {
-        // TODO: check it's the correct way to parse a JCR name
         return $this->scanner->fetchNextToken();
     }
 

tests/PHPCR/Tests/Util/ValueConverterTest.php

@@ -272,7 +272,7 @@ public function dataConversionMatrix()
      */
     public function testConvertType($value, $srctype, $expected, $targettype)
     {
-        if (is_null($expected)) {
+        if (null === $expected) {
             try {
                 $this->valueConverter->convertType($value, $targettype, $srctype);
                 $this->fail('Excpected that this conversion would throw an exception');