Add test for check empty placeholder container items.

Author: AndyDune

src/Helper/Placeholder/Container/AbstractContainer.php

@@ -117,14 +117,17 @@ public function toString($indent = null)
             ? $this->getWhitespace($indent)
             : $this->getIndent();
 
-        $items  = $this->getArrayCopy();
         // If we don't have items - do not show prefix and postfix
-        if (!count($items)) {
+        if (!$this->count()) {
             return '';
         }
+
+        $itemsToString = implode($this->getSeparator(), $this->getArrayCopy());
+        // todo check empty $itemsToString after trim() to return ''
+
         $return = $indent
             . $this->getPrefix()
-            . implode($this->getSeparator(), $items)
+            . $itemsToString
             . $this->getPostfix();
         $return = preg_replace("/(\r\n?|\n)/", '$1' . $indent, $return);
 

test/Helper/Placeholder/ContainerTest.php

@@ -432,4 +432,13 @@ public function testIndentationIsHonored()
         $this->assertContains("    <ul>\n", $string, $string);
         $this->assertContains("\n    </ul>", $string, $string);
     }
+
+    public function testNoUsePrefixPostfixIfNoItems()
+    {
+        $this->container->setPrefix("<h1>")
+                        ->setPostfix("</h1>");
+        $string = $this->container->toString();
+        $this->assertEquals('', $string);
+    }
+
 }