Prevent to visit node that got dropped later in loop

Author: kylekatarnls

composer.json

@@ -14,8 +14,7 @@
     "ext-mbstring": "*"
   },
   "require-dev": {
-    "phpunit/phpunit": "^6.0 || ^7.0",
-    "codeclimate/php-test-reporter": "dev-master"
+    "phpunit/phpunit": "^6.5.14 || ^7.5.20 || ^8.5.31 || ^9.5.27"
   },
   "minimum-stability": "stable",
   "autoload": {

src/JsPhpize/Compiler/Compiler.php

@@ -191,14 +191,23 @@ protected function visitDyiade(Dyiade $dyiade, $indent)
                 return $this->compileLazyDyiade($this->engine->getHelperName('and'), $leftHand, $rightHand);
             case '+':
                 $arguments = [$leftHand, $rightHand];
-                while (
-                    ($dyiade = $dyiade->rightHand) instanceof Dyiade &&
-                    $dyiade->operator === '+'
-                ) {
-                    /* @var Dyiade $dyiade */
+                $innerRightHand = $dyiade->rightHand;
+
+                if ($innerRightHand instanceof Dyiade && $innerRightHand->operator === '+') {
+                    $last = $dyiade;
                     array_pop($arguments);
-                    $arguments[] = $this->visitNode($dyiade->leftHand, $indent);
-                    $arguments[] = $this->visitNode($dyiade->rightHand, $indent);
+
+                    while (
+                        ($dyiade = $dyiade->rightHand) instanceof Dyiade &&
+                        $dyiade->operator === '+'
+                    ) {
+                        $last = $dyiade;
+                        $arguments[] = $this->visitNode($dyiade->leftHand, $indent);
+                    }
+
+                    if ($last instanceof Dyiade && $last->operator === '+') {
+                        $arguments[] = $this->visitNode($last->rightHand, $indent);
+                    }
                 }
 
                 return $this->helperWrap($this->engine->getHelperName('plus'), $arguments);

src/JsPhpize/Compiler/Helpers/Dot.h

@@ -73,7 +73,7 @@ function ($base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/Dot.ref.h

@@ -73,7 +73,7 @@ function (&$base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/DotObject.h

@@ -73,7 +73,7 @@ function ($base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/DotObject.ref.h

@@ -73,7 +73,7 @@ function (&$base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/DotWithArrayPrototype.h

@@ -153,7 +153,7 @@ function ($base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/DotWithArrayPrototype.ref.h

@@ -153,7 +153,7 @@ function (&$base) {
                 {
                     $value = $this->getValue();
 
-                    if (method_exists($value, '__toBoolean')) {
+                    if (is_object($value) && method_exists($value, '__toBoolean')) {
                         return $value->__toBoolean();
                     }
 

src/JsPhpize/Compiler/Helpers/Set.h

@@ -5,7 +5,7 @@ function ($base, $key, $operator, $value) {
                 $base[$key] = $value;
                 break;
             }
-            if (method_exists($base, $method = "set" . ucfirst($key))) {
+            if (is_object($base) && method_exists($base, $method = "set" . ucfirst($key))) {
                 $base->$method($value);
                 break;
             }

src/JsPhpize/Stream/ExpressionStream.php

@@ -2,11 +2,26 @@
 
 namespace JsPhpize\Stream;
 
+use Attribute;
+
+// @codeCoverageIgnoreStart
+if (\PHP_VERSION >= 8 && \PHP_VERSION < 8.2 && !class_exists('AllowDynamicProperties')) {
+    #[Attribute(Attribute::TARGET_CLASS)]
+    final class AllowDynamicProperties
+    {
+        public function __construct()
+        {
+        }
+    }
+}
+// @codeCoverageIgnoreEnd
+
 /**
  * Creates a wrapper in order to allow the Zend PhpRenderer
  * to include the compiled file.
  * Class JsPhpize\Stream\ExrpressionStream.
  */
+#[\AllowDynamicProperties]
 class ExpressionStream
 {
     /**