Merge pull request #361 from UFOMelkor/hotfix/360-php56

Load php7 code in php7 only

Author: UFOMelkor

src/Hal/Component/Ast/NodeTraverser.php

@@ -30,7 +30,7 @@ public function __construct(Mother $traverser, $stopCondition = null)
     {
         if(null === $stopCondition) {
             $stopCondition = function($node) {
-                if($node instanceof Node\Stmt\Class_ || $node instanceof Node\Stmt\Interface_) {
+                if ($node instanceof Node\Stmt\Class_ || $node instanceof Node\Stmt\Interface_) {
                     return false;
                 }
 
@@ -96,47 +96,7 @@ public function traverseArray(array $nodes, array $visitors) {
 }
 
 if (PHP_VERSION_ID >= 70000) {
-    class NodeTraverser extends Mother
-    {
-        /** @var Traverser */
-        private $traverser;
-
-        public function __construct($cloneNodes = false, $stopCondition = null)
-        {
-            parent::__construct();
-            $this->traverser = new Traverser($this, $stopCondition);
-        }
-
-        public function traverseNode(Node $node): Node
-        {
-            return parent::traverseNode($node);
-        }
-
-        protected function traverseArray(array $nodes): array
-        {
-            return $this->traverser->traverseArray($nodes, $this->visitors);
-        }
-    }
+    class_alias(Php7NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
 } else {
-    class NodeTraverser extends Mother
-    {
-        /** @var Traverser */
-        private $traverser;
-
-        public function __construct($cloneNodes = false, $stopCondition = null)
-        {
-            parent::__construct();
-            $this->traverser = new Traverser($this, $stopCondition);
-        }
-
-        public function traverseNode(Node $node)
-        {
-            return parent::traverseNode($node);
-        }
-
-        protected function traverseArray(array $nodes)
-        {
-            return $this->traverser->traverseArray($nodes, $this->visitors);
-        }
-    }
-}
+    class_alias(Php5NodeTraverser::class, __NAMESPACE__ . '\\NodeTraverser');
+}

src/Hal/Component/Ast/Php5NodeTraverser.php

@@ -0,0 +1,27 @@
+<?php
+namespace Hal\Component\Ast;
+
+use PhpParser\Node;
+use PhpParser\NodeTraverser as Mother;
+
+class Php5NodeTraverser extends Mother
+{
+    /** @var Traverser */
+    private $traverser;
+
+    public function __construct($cloneNodes = false, $stopCondition = null)
+    {
+        parent::__construct();
+        $this->traverser = new Traverser($this, $stopCondition);
+    }
+
+    public function traverseNode(Node $node)
+    {
+        return parent::traverseNode($node);
+    }
+
+    protected function traverseArray(array $nodes)
+    {
+        return $this->traverser->traverseArray($nodes, $this->visitors);
+    }
+}

src/Hal/Component/Ast/Php7NodeTraverser.php

@@ -0,0 +1,27 @@
+<?php
+namespace Hal\Component\Ast;
+
+use PhpParser\Node;
+use PhpParser\NodeTraverser as Mother;
+
+class Php7NodeTraverser extends Mother
+{
+    /** @var Traverser */
+    private $traverser;
+
+    public function __construct($cloneNodes = false, $stopCondition = null)
+    {
+        parent::__construct();
+        $this->traverser = new Traverser($this, $stopCondition);
+    }
+
+    public function traverseNode(Node $node): Node
+    {
+        return parent::traverseNode($node);
+    }
+
+    protected function traverseArray(array $nodes): array
+    {
+        return $this->traverser->traverseArray($nodes, $this->visitors);
+    }
+}

tests/Component/Ast/NodeTraverserTest.php

@@ -0,0 +1,14 @@
+<?php
+namespace Test\Hal\Component\Ast;
+
+use Hal\Component\Ast\NodeTraverser;
+use PhpParser\NodeTraverser as BaseTraverser;
+use PHPUnit_Framework_TestCase;
+
+class NodeTraverserTest extends PHPUnit_Framework_TestCase
+{
+    public function testItCanBeInstantiated()
+    {
+        $this->assertInstanceOf(BaseTraverser::class, new NodeTraverser());
+    }
+}