Optimize check count placeholder container items count.

Author: AndyDune

src/Helper/Placeholder/Container/AbstractContainer.php

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

test/Helper/Placeholder/ContainerTest.php

@@ -441,4 +441,13 @@ public function testNoUsePrefixPostfixIfNoItems()
         $this->assertEquals('', $string);
     }
 
+    public function testUsePrefixPostfixIfEmptyItems()
+    {
+        $this->container->setPrefix("<h1>")
+            ->setPostfix("</h1>");
+        $this->container->set('');
+        $string = $this->container->toString();
+        $this->assertEquals('<h1></h1>', $string);
+    }
+
 }