Merge "Workaround image magick issue with greyscale xcf files"

[lhc/web/wiklou.git] / includes / HtmlFormatter.php
diff --git a/includes/HtmlFormatter.php b/includes/HtmlFormatter.php

index 83d0530..38aa392 100644 (file)
--- a/includes/HtmlFormatter.php
+++ b/includes/HtmlFormatter.php
@@ -136,7 +136,13 @@ class HtmlFormatter {
                 wfProfileIn( __METHOD__ );
                 $removals = $this->parseItemsToRemove();
  
-               if ( !$removals ) {
+               // Bail out early if nothing to do
+               if ( array_reduce( $removals,
+                       function( $carry, $item ) {
+                               return $carry && !$item;
+                       },
+                       true
+               ) ) {
                         wfProfileOut( __METHOD__ );
                         return array();
                 }
@@ -253,7 +259,8 @@ class HtmlFormatter {
          * specify the $element in the method it'll change the underlying dom and you won't be able to get
          * it back.
          *
-        * @param DOMElement|string|null $element ID of element to get HTML from or false to get it from the whole tree
+        * @param DOMElement|string|null $element ID of element to get HTML from or
+        *   false to get it from the whole tree
          * @return string Processed HTML
          */
         public function getText( $element = null ) {
@@ -287,11 +294,12 @@ class HtmlFormatter {
                                 // XML code paths if possible and fix there.
                                 $html = str_replace( '&#13;', '', $html );
                         }
-                       $html = preg_replace( '/<!--.*?-->|^.*?<body>|<\/body>.*$/s', '', $html );
                         wfProfileOut( __METHOD__ . '-fixes' );
                 } else {
                         $html = $this->html;
                 }
+               // Remove stuff added by wrapHTML()
+               $html = preg_replace( '/<!--.*?-->|^.*?<body>|<\/body>.*$/s', '', $html );
                 $html = $this->onHtmlReady( $html );
  
                 wfProfileIn( __METHOD__ . '-flatten' );
@@ -306,9 +314,13 @@ class HtmlFormatter {
         }
  
         /**
+        * Helper function for parseItemsToRemove(). This function extracts the selector type
+        * and the raw name of a selector from a CSS-style selector string and assigns those
+        * values to parameters passed by reference. For example, if given '#toc' as the
+        * $selector parameter, it will assign 'ID' as the $type and 'toc' as the $rawName.
          * @param string $selector CSS selector to parse
-        * @param string $type
-        * @param string $rawName
+        * @param string $type The type of selector (ID, CLASS, TAG_CLASS, or TAG)
+        * @param string $rawName The raw name of the selector
          * @return bool Whether the selector was successfully recognised
          */
         protected function parseSelector( $selector, &$type, &$rawName ) {
@@ -332,7 +344,8 @@ class HtmlFormatter {
         }
  
         /**
-        * Transforms CSS selectors into an internal representation suitable for processing
+        * Transforms CSS-style selectors into an internal representation suitable for
+        * processing by filterContent()
          * @return array
          */
         protected function parseItemsToRemove() {