Merge pull request #174 from phpcr/php-72

build with php 7.2

Author: dbu

.travis.yml

@@ -4,6 +4,7 @@ php:
   - 5.6
   - 7.0
   - 7.1
+  - 7.2
 
 env:
   - PACKAGE_VERSION=high

composer.json

@@ -27,7 +27,7 @@
         }
     ],
     "require": {
-        "php": "^5.6|7.0.x|7.1.x",
+        "php": "^5.6|^7.0",
         "phpcr/phpcr": "~2.1.0",
         "symfony/console": "~2.3|~3.0"
     },

src/PHPCR/Util/Console/Helper/PhpcrHelper.php

@@ -115,7 +115,9 @@ public function processNode(OutputInterface $output, NodeInterface $node, array
                 );
             } else {
                 $closureString = $closure;
-                $closure = create_function('$session, $node', $closure);
+                $closure = function (SessionInterface $session, NodeInterface $node) use ($closureString) {
+                    eval($closureString);
+                };
                 $output->writeln(sprintf(
                     '<comment> > Applying closure: %s</comment>',
                     strlen($closureString) > 75 ? substr($closureString, 0, 72).'...' : $closureString

src/PHPCR/Util/QOM/Sql1Generator.php

@@ -120,7 +120,9 @@ public function evalFullTextSearch($selectorName, $searchExpression, $propertyNa
      */
     public function evalColumns($columns)
     {
-        if (count($columns) === 0) {
+        if ((!is_array($columns) && !$columns instanceof \Countable)
+            || count($columns) === 0
+        ) {
             return 's';
         }
 

src/PHPCR/Util/QOM/Sql2Generator.php

@@ -325,7 +325,9 @@ public function evalPropertyValue($propertyName, $selectorName = null)
      */
     public function evalColumns($columns)
     {
-        if (count($columns) === 0) {
+        if ((!is_array($columns) && !$columns instanceof \Countable)
+            || count($columns) === 0
+        ) {
             return '*';
         }