Merge pull request #73 from wgevaert/5.0.0-typing-and-consistency

Check if map is empty before adding to node

Author: marijnvanwezel

src/Expressions/Literals/List_.php

@@ -71,6 +71,16 @@ public function getExpressions(): array
         return $this->expressions;
     }
 
+    /**
+     * Returns whether this list is empty.
+     *
+     * @return bool
+     */
+    public function isEmpty(): bool
+    {
+        return empty($this->expressions);
+    }
+
     /**
      * @inheritDoc
      */

src/Expressions/Literals/Map.php

@@ -88,6 +88,16 @@ public function getElements(): array
         return $this->elements;
     }
 
+    /**
+     * Checks if this map is empty
+     *
+     * @return bool
+     */
+    public function isEmpty(): bool
+    {
+        return empty($this->elements);
+    }
+
     /**
      * @inheritDoc
      */

src/Patterns/Node.php

@@ -150,7 +150,7 @@ private function nodeInnerToString(): string
             }
         }
 
-        if (isset($this->properties)) {
+        if (isset($this->properties) && !$this->properties->isEmpty()) {
             if ($nodeInner !== "") {
                 $nodeInner .= " ";
             }