Merge pull request #133 from AndyDune/master

Don't show prefix and postfix if there is no placeholder items.

Author: weierophinney

src/Helper/Placeholder/Container/AbstractContainer.php

@@ -113,9 +113,14 @@ 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 (! count($this)) {
+            return '';
+        }
+
+        $indent = ($indent === null)
+            ? $this->getIndent()
+            : $this->getWhitespace($indent);
 
         $items  = $this->getArrayCopy();
         $return = $indent

test/Helper/Placeholder/ContainerTest.php

@@ -432,4 +432,17 @@ 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);
+
+        $this->container->set('');
+        $string = $this->container->toString();
+        $this->assertEquals('<h1></h1>', $string);
+    }
+
 }