Make Skin::formatDebugHTML()'s formatting work when memory usage is greather that 10M
[lhc/web/wiklou.git] / includes / Xml.php
index f965516..a0722f7 100644 (file)
@@ -191,7 +191,7 @@ class Xml {
                }
                if( $year ) {
                        $encYear = intval( $year );
-               } else if( $encMonth ) {
+               } elseif( $encMonth ) {
                        $thisMonth = intval( gmdate( 'n' ) );
                        $thisYear = intval( gmdate( 'Y' ) );
                        if( intval($encMonth) > $thisMonth ) {
@@ -334,10 +334,10 @@ class Xml {
 
        /**
         * Convenience function to build an HTML radio button
-        * @param $name value of the name attribute
-        * @param $value value of the value attribute
-        * @param $checked Whether the checkbox is checked or not
-        * @param $attribs other attributes
+        * @param $name String value of the name attribute
+        * @param $value String value of the value attribute
+        * @param $checked Bool Whether the checkbox is checked or not
+        * @param $attribs Array other attributes
         * @return string HTML
         */
        public static function radio( $name, $value, $checked = false, $attribs = array() ) {
@@ -351,17 +351,23 @@ class Xml {
         * Convenience function to build an HTML form label
         * @param $label String text of the label
         * @param $id
-        * @param $attribs Array an attribute array.  This will usuall be 
+        * @param $attribs Array an attribute array.  This will usuall be
         *     the same array as is passed to the corresponding input element,
-        *     so this function will cherry-pick appropriate attributes to 
-        *     apply to the label as well; currently only class is applied.
+        *     so this function will cherry-pick appropriate attributes to
+        *     apply to the label as well; only class and title are applied.
         * @return string HTML
         */
        public static function label( $label, $id, $attribs = array() ) {
                $a = array( 'for' => $id );
+
+               # FIXME avoid copy pasting below:
                if( isset( $attribs['class'] ) ){
                                $a['class'] = $attribs['class'];
                }
+               if( isset( $attribs['title'] ) ){
+                               $a['title'] = $attribs['title'];
+               }
+
                return self::element( 'label', $a, $label );
        }
 
@@ -370,8 +376,8 @@ class Xml {
         * @param $label String text of the label
         * @param $name String value of the name attribute
         * @param $id String id of the input
-        * @param $size int value of the size attribute
-        * @param $value value of the value attribute
+        * @param $size Int|Bool value of the size attribute
+        * @param $value String|Bool value of the value attribute
         * @param $attribs array other attributes
         * @return string HTML
         */
@@ -383,11 +389,11 @@ class Xml {
        /**
         * Same as Xml::inputLabel() but return input and label in an array
         *
-        * @param $label
-        * @param $name
-        * @param $id
-        * @param $size
-        * @param $value
+        * @param $label String
+        * @param $name String
+        * @param $id String
+        * @param $size Int|Bool
+        * @param $value String|Bool
         * @param $attribs array
         *
         * @return array
@@ -460,7 +466,7 @@ class Xml {
                if( $selected ) {
                        $attribs['selected'] = 'selected';
                }
-               return self::element( 'option', $attribs, $text );
+               return Html::element( 'option', $attribs, $text );
        }
 
        /**
@@ -500,19 +506,24 @@ class Xml {
                                        $optgroup = false;
                                }
                        }
+
                        if( $optgroup ) $options .= self::closeElement('optgroup');
 
                $attribs = array();
+
                if( $name ) {
                        $attribs['id'] = $name;
                        $attribs['name'] = $name;
                }
+
                if( $class ) {
                        $attribs['class'] = $class;
                }
+
                if( $tabindex ) {
                        $attribs['tabindex'] = $tabindex;
                }
+
                return Xml::openElement( 'select', $attribs )
                        . "\n"
                        . $options
@@ -531,9 +542,11 @@ class Xml {
         */
        public static function fieldset( $legend = false, $content = false, $attribs = array() ) {
                $s = Xml::openElement( 'fieldset', $attribs ) . "\n";
+
                if ( $legend ) {
                        $s .= Xml::element( 'legend', null, $legend ) . "\n";
                }
+
                if ( $content !== false ) {
                        $s .= $content . "\n";
                        $s .= Xml::closeElement( 'fieldset' ) . "\n";
@@ -594,6 +607,7 @@ class Xml {
                        "\xe2\x80\x8c" => "\\u200c", // ZERO WIDTH NON-JOINER
                        "\xe2\x80\x8d" => "\\u200d", // ZERO WIDTH JOINER
                );
+
                return strtr( $string, $pairs );
        }
 
@@ -635,6 +649,7 @@ class Xml {
                                if ( $s != '{' ) {
                                        $s .= ', ';
                                }
+
                                $s .= '"' . self::escapeJsString( $name ) . '": ' .
                                        self::encodeJsVar( $elt );
                        }
@@ -646,8 +661,8 @@ class Xml {
        }
 
        /**
-        * Create a call to a JavaScript function. The supplied arguments will be 
-        * encoded using Xml::encodeJsVar(). 
+        * Create a call to a JavaScript function. The supplied arguments will be
+        * encoded using Xml::encodeJsVar().
         *
         * @param $name String The name of the function to call, or a JavaScript expression
         *    which evaluates to a function object which is called.
@@ -660,19 +675,22 @@ class Xml {
        public static function encodeJsCall( $name, $args ) {
                $s = "$name(";
                $first = true;
+
                foreach ( $args as $arg ) {
                        if ( $first ) {
                                $first = false;
                        } else {
                                $s .= ', ';
                        }
+
                        $s .= Xml::encodeJsVar( $arg );
                }
+
                $s .= ");\n";
+
                return $s;
        }
 
-
        /**
         * Check if a string is well-formed XML.
         * Must include the surrounding tag.
@@ -696,7 +714,9 @@ class Xml {
                        xml_parser_free( $parser );
                        return false;
                }
+
                xml_parser_free( $parser );
+
                return true;
        }
 
@@ -714,6 +734,7 @@ class Xml {
                        '<html>' .
                        $text .
                        '</html>';
+
                return Xml::isWellFormed( $html );
        }
 
@@ -759,7 +780,6 @@ class Xml {
 
                $form .= "</tbody></table>";
 
-
                return $form;
        }
 
@@ -772,78 +792,59 @@ class Xml {
         */
        public static function buildTable( $rows, $attribs = array(), $headers = null ) {
                $s = Xml::openElement( 'table', $attribs );
+
                if ( is_array( $headers ) ) {
+                       $s .= Xml::openElement( 'thead', $attribs );
+
                        foreach( $headers as $id => $header ) {
                                $attribs = array();
-                               if ( is_string( $id ) ) $attribs['id'] = $id;
+
+                               if ( is_string( $id ) ) {
+                                       $attribs['id'] = $id;
+                               }
+
                                $s .= Xml::element( 'th', $attribs, $header );
                        }
+                       $s .= Xml::closeElement( 'thead' );
                }
+
                foreach( $rows as $id => $row ) {
                        $attribs = array();
-                       if ( is_string( $id ) ) $attribs['id'] = $id;
+
+                       if ( is_string( $id ) ) {
+                               $attribs['id'] = $id;
+                       }
+
                        $s .= Xml::buildTableRow( $attribs, $row );
                }
+
                $s .= Xml::closeElement( 'table' );
+
                return $s;
        }
 
        /**
         * Build a row for a table
-        * @param $attribs An array of attributes to apply to the tr tag
-        * @param $cells An array of strings to put in <td>
+        * @param $attribs array An array of attributes to apply to the tr tag
+        * @param $cells array An array of strings to put in <td>
         * @return string
         */
        public static function buildTableRow( $attribs, $cells ) {
                $s = Xml::openElement( 'tr', $attribs );
+
                foreach( $cells as $id => $cell ) {
+
                        $attribs = array();
-                       if ( is_string( $id ) ) $attribs['id'] = $id;
+
+                       if ( is_string( $id ) ) {
+                               $attribs['id'] = $id;
+                       }
+
                        $s .= Xml::element( 'td', $attribs, $cell );
                }
+
                $s .= Xml::closeElement( 'tr' );
-               return $s;
-       }
 
-       /**
-        * Get HTML for an info box with an icon.
-        *
-        * @param $text String: wikitext, get this with wfMsgNoTrans()
-        * @param $icon String: icon name, file in skins/common/images
-        * @param $alt String: alternate text for the icon
-        * @param $class String: additional class name to add to the wrapper div
-        * @param $useStylePath
-        *
-        * @return string
-        */
-       static function infoBox( $text, $icon, $alt, $class = false, $useStylePath = true ) {
-               global $wgStylePath;
-               
-               if ( $useStylePath ) {
-                       $icon = $wgStylePath.'/common/images/'.$icon;
-               }
-               
-               
-               $s  = Xml::openElement( 'div', array( 'class' => "mw-infobox $class") );
-               
-               $s .= Xml::openElement( 'div', array( 'class' => 'mw-infobox-left' ) ).
-                               Html::element( 'img',
-                                       array(
-                                               'src' => $icon,
-                                               'alt' => $alt,
-                                       )
-                               ).
-                               Xml::closeElement( 'div' );
-
-               $s .= Xml::openElement( 'div', array( 'class' => 'mw-infobox-right' ) ).
-                               $text.
-                               Xml::closeElement( 'div' );
-               $s .= Xml::element( 'div', array( 'style' => 'clear: left;' ), ' ' );
-
-               $s .= Xml::closeElement( 'div' );
-
-               $s .= Xml::element( 'div', array( 'style' => 'clear: left;' ), ' ' );
-               
                return $s;
        }
 }
@@ -857,9 +858,11 @@ class XmlSelect {
                if ( $name ) {
                        $this->setAttribute( 'name', $name );
                }
+
                if ( $id ) {
                        $this->setAttribute( 'id', $id );
                }
+
                if ( $default !== false ) {
                        $this->default = $default;
                }
@@ -885,7 +888,7 @@ class XmlSelect {
         * @return array|null
         */
        public function getAttribute( $name ) {
-               if ( isset($this->attributes[$name]) ) {
+               if ( isset( $this->attributes[$name] ) ) {
                        return $this->attributes[$name];
                } else {
                        return null;
@@ -899,7 +902,8 @@ class XmlSelect {
        public function addOption( $name, $value = false ) {
                // Stab stab stab
                $value = ($value !== false) ? $value : $name;
-               $this->options[] = Xml::option( $name, $value, $value === $this->default );
+
+               $this->options[] = array( $name => $value );
        }
 
        /**
@@ -910,7 +914,7 @@ class XmlSelect {
         * @param  $options
         */
        public function addOptions( $options ) {
-               $this->options[] = trim( self::formatOptions( $options, $this->default ) );
+               $this->options[] = $options;
        }
 
        /**
@@ -924,10 +928,11 @@ class XmlSelect {
         */
        static function formatOptions( $options, $default = false ) {
                $data = '';
+
                foreach( $options as $label => $value ) {
                        if ( is_array( $value ) ) {
                                $contents = self::formatOptions( $value, $default );
-                               $data .= Xml::tags( 'optgroup', array( 'label' => $label ), $contents ) . "\n";
+                               $data .= Html::rawElement( 'optgroup', array( 'label' => $label ), $contents ) . "\n";
                        } else {
                                $data .= Xml::option( $label, $value, $value === $default ) . "\n";
                        }
@@ -940,14 +945,19 @@ class XmlSelect {
         * @return string
         */
        public function getHTML() {
-               return Xml::tags( 'select', $this->attributes, implode( "\n", $this->options ) );
-       }
+               $contents = '';
 
+               foreach ( $this->options as $options ) {
+                       $contents .= self::formatOptions( $options, $this->default );
+               }
+
+               return Html::rawElement( 'select', $this->attributes, rtrim( $contents ) );
+       }
 }
 
 /**
- * A wrapper class which causes Xml::encodeJsVar() and Xml::encodeJsCall() to 
- * interpret a given string as being a JavaScript expression, instead of string 
+ * A wrapper class which causes Xml::encodeJsVar() and Xml::encodeJsCall() to
+ * interpret a given string as being a JavaScript expression, instead of string
  * data.
  *
  * Example: