Xml: Fix Xml::fieldset() when $content is not given

A function in the Xml class could generate unbalanced tags. How
ironic. Why do we even have this instead of just the Html class?

Change-Id: Ie119c92aa4936e2f8982163d4b00d08949f49264

diff --git a/includes/Xml.php b/includes/Xml.php
index 0091513..eadc7d1 100644 (file)
--- a/includes/Xml.php
+++ b/includes/Xml.php
@@ -615,9 +615,9 @@ class Xml {
 
                if ( $content !== false ) {
                        $s .= $content . "\n";
-                       $s .= self::closeElement( 'fieldset' ) . "\n";
                }
 
+               $s .= self::closeElement( 'fieldset' ) . "\n";
                return $s;
        }