Merge "OOUI theme support"

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

index 0515566..d3ee9b9 100644 (file)
--- a/includes/exception/HttpError.php
+++ b/includes/exception/HttpError.php
@@ -18,6 +18,8 @@
   * @file
   */
  
+use MediaWiki\Logger\LoggerFactory;
+
  /**
   * Show an error that looks like an HTTP server error.
   * Replacement for wfHttpError().
@@ -33,7 +35,7 @@ class HttpError extends MWException {
          *
          * @param int $httpCode HTTP status code to send to the client
          * @param string|Message $content Content of the message
-        * @param string|Message $header Content of the header (\<title\> and \<h1\>)
+        * @param string|Message|null $header Content of the header (\<title\> and \<h1\>)
          */
         public function __construct( $httpCode, $content, $header = null ) {
                 parent::__construct( $content );
@@ -81,7 +83,7 @@ class HttpError extends MWException {
         }
  
         private function doLog() {
-               $logger = MWLoggerFactory::getInstance( 'HttpError' );
+               $logger = LoggerFactory::getInstance( 'HttpError' );
                 $content = $this->content;
  
                 if ( $content instanceof Message ) {
@@ -111,21 +113,21 @@ class HttpError extends MWException {
          */
         public function getHTML() {
                 if ( $this->header === null ) {
-                       $header = HttpStatus::getMessage( $this->httpCode );
+                       $titleHtml = htmlspecialchars( HttpStatus::getMessage( $this->httpCode ) );
                 } elseif ( $this->header instanceof Message ) {
-                       $header = $this->header->escaped();
+                       $titleHtml = $this->header->escaped();
                 } else {
-                       $header = htmlspecialchars( $this->header );
+                       $titleHtml = htmlspecialchars( $this->header );
                 }
  
                 if ( $this->content instanceof Message ) {
-                       $content = $this->content->escaped();
+                       $contentHtml = $this->content->escaped();
                 } else {
-                       $content = htmlspecialchars( $this->content );
+                       $contentHtml = htmlspecialchars( $this->content );
                 }
  
                 return "<!DOCTYPE html>\n" .
-               "<html><head><title>$header</title></head>\n" .
-               "<body><h1>$header</h1><p>$content</p></body></html>\n";
+               "<html><head><title>$titleHtml</title></head>\n" .
+               "<body><h1>$titleHtml</h1><p>$contentHtml</p></body></html>\n";
         }
  }