MediaWiki 1.16.0

[autoinstallsdev/mediawiki.git] / includes / OutputHandler.php
diff --git a/includes/OutputHandler.php b/includes/OutputHandler.php

index 2b3e9fae55f97bd74f1d44833b01631ad71a9d58..91819cc76a69d7b14d326a310de8ae114b3df0c1 100644 (file)
--- a/includes/OutputHandler.php
+++ b/includes/OutputHandler.php
@@ -5,12 +5,12 @@
   */
  function wfOutputHandler( $s ) {
         global $wgDisableOutputCompression, $wgValidateAllHtml;
-    $s = wfMangleFlashPolicy( $s );
-    if ( $wgValidateAllHtml ) {
+       $s = wfMangleFlashPolicy( $s );
+       if ( $wgValidateAllHtml ) {
                 $headers = apache_response_headers();
                 $isHTML = true;
                 foreach ( $headers as $name => $value ) {
-                       if ( strtolower( $name ) == 'content-type' && strpos( $value, 'text/html' ) === false ) {
+                       if ( strtolower( $name ) == 'content-type' && strpos( $value, 'text/html' ) === false && strpos( $value, 'application/xhtml+xml' ) === false ) {
                                 $isHTML = false;
                                 break;
                         }
@@ -37,7 +37,7 @@ function wfOutputHandler( $s ) {
   * @private
   */
  function wfRequestExtension() {
-       /// @fixme -- this sort of dupes some code in WebRequest::getRequestUrl()
+       /// @todo Fixme: this sort of dupes some code in WebRequest::getRequestUrl()
         if( isset( $_SERVER['REQUEST_URI'] ) ) {
                 // Strip the query string...
                 list( $path ) = explode( '?', $_SERVER['REQUEST_URI'], 2 );
@@ -74,12 +74,9 @@ function wfGzipHandler( $s ) {
                 return $s;
         }
  
-       if( isset( $_SERVER['HTTP_ACCEPT_ENCODING'] ) ) {
-               $tokens = preg_split( '/[,; ]/', $_SERVER['HTTP_ACCEPT_ENCODING'] );
-               if ( in_array( 'gzip', $tokens ) ) {
-                       header( 'Content-Encoding: gzip' );
-                       $s = gzencode( $s, 3 );
-               }
+       if( wfClientAcceptsGzip() ) {
+               header( 'Content-Encoding: gzip' );
+               $s = gzencode( $s, 6 );
         }
  
         // Set vary header if it hasn't been set already
@@ -93,7 +90,10 @@ function wfGzipHandler( $s ) {
         }
         if ( !$foundVary ) {
                 header( 'Vary: Accept-Encoding' );
-               header( 'X-Vary-Options: Accept-Encoding;list-contains=gzip' );
+               global $wgUseXVO;
+               if ( $wgUseXVO ) {
+                       header( 'X-Vary-Options: Accept-Encoding;list-contains=gzip' );
+               }
         }
         return $s;
  }
@@ -123,10 +123,9 @@ function wfDoContentLength( $length ) {
   * Replace the output with an error if the HTML is not valid
   */
  function wfHtmlValidationHandler( $s ) {
-       global $IP;
-       $tidy = new tidy;
-       $tidy->parseString( $s, "$IP/includes/tidy.conf", 'utf8' );
-       if ( $tidy->getStatus() == 0 ) {
+
+       $errors = '';
+       if ( MWTidy::checkErrors( $s, $errors ) ) {
                 return $s;
         }
  
@@ -134,7 +133,7 @@ function wfHtmlValidationHandler( $s ) {
  
         $out = <<<EOT
  <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
-<html>
+<html xmlns="http://www.w3.org/1999/xhtml" lang="en" dir="ltr">
  <head>
  <title>HTML validation error</title>
  <style>
@@ -147,7 +146,7 @@ li { white-space: pre }
  <ul>
  EOT;
  
-       $error = strtok( $tidy->errorBuffer, "\n" );
+       $error = strtok( $errors, "\n" );
         $badLines = array();
         while ( $error !== false ) {
                 if ( preg_match( '/^line (\d+)/', $error, $m ) ) {
@@ -158,8 +157,9 @@ EOT;
                 $error = strtok( "\n" );
         }
  
-       $out .= '<pre>' . htmlspecialchars( $tidy->errorBuffer ) . '</pre>';
-       $out .= '<ol>';
+       $out .= '</ul>';
+       $out .= '<pre>' . htmlspecialchars( $errors ) . '</pre>';
+       $out .= "<ol>\n";
         $line = strtok( $s, "\n" );
         $i = 1;
         while ( $line !== false ) {
@@ -168,7 +168,7 @@ EOT;
                 } else {
                         $out .= '<li>';
                 }
-               $out .= htmlspecialchars( $line ) . '</li>';
+               $out .= htmlspecialchars( $line ) . "</li>\n";
                 $line = strtok( "\n" );
                 $i++;
         }