X-Git-Url: http://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FException.php;h=afa46ce2eacc65140598ed6f1a76505c6d18e7c2;hb=44eefff24c088ded89ad02dab4516b94e8c3aca5;hp=eb7159861c26222567d84b66fae0e09ad4910dda;hpb=5fe9c4363eed213142616baa906ec9323b231191;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/Exception.php b/includes/Exception.php index eb7159861c..afa46ce2ea 100644 --- a/includes/Exception.php +++ b/includes/Exception.php @@ -1,21 +1,28 @@ isArticle() ) ) && + return $this->useMessageCache() && + !empty( $GLOBALS['wgFullyInitialised'] ) && + ( !empty( $GLOBALS['wgArticle'] ) || ( !empty( $GLOBALS['wgOut'] ) && !$GLOBALS['wgOut']->isArticleRelated() ) ) && !empty( $GLOBALS['wgTitle'] ); } @@ -25,32 +32,45 @@ class MWException extends Exception { */ function useMessageCache() { global $wgLang; - return is_object( $wgLang ); + + foreach ( $this->getTrace() as $frame ) { + if ( isset( $frame['class'] ) && $frame['class'] === 'LocalisationCache' ) { + return false; + } + } + + return $wgLang instanceof Language; } /** * Run hook to allow extensions to modify the text of the exception * - * @param String $name class name of the exception - * @param Array $args arguments to pass to the callback functions - * @return mixed string to output or null if any hook has been called + * @param $name String: class name of the exception + * @param $args Array: arguments to pass to the callback functions + * @return Mixed: string to output or null if any hook has been called */ function runHooks( $name, $args = array() ) { global $wgExceptionHooks; - if( !isset( $wgExceptionHooks ) || !is_array( $wgExceptionHooks ) ) + + if ( !isset( $wgExceptionHooks ) || !is_array( $wgExceptionHooks ) ) { return; // Just silently ignore - if( !array_key_exists( $name, $wgExceptionHooks ) || !is_array( $wgExceptionHooks[ $name ] ) ) + } + + if ( !array_key_exists( $name, $wgExceptionHooks ) || !is_array( $wgExceptionHooks[ $name ] ) ) { return; + } + $hooks = $wgExceptionHooks[ $name ]; $callargs = array_merge( array( $this ), $args ); - foreach( $hooks as $hook ) { - if( is_string( $hook ) || ( is_array( $hook ) && count( $hook ) >= 2 && is_string( $hook[0] ) ) ) { //'function' or array( 'class', hook' ) + foreach ( $hooks as $hook ) { + if ( is_string( $hook ) || ( is_array( $hook ) && count( $hook ) >= 2 && is_string( $hook[0] ) ) ) { // 'function' or array( 'class', hook' ) $result = call_user_func_array( $hook, $callargs ); } else { $result = null; } - if( is_string( $result ) ) + + if ( is_string( $result ) ) return $result; } } @@ -58,14 +78,15 @@ class MWException extends Exception { /** * Get a message from i18n * - * @param String $key message name - * @param String $fallback default message if the message cache can't be - * called by the exception + * @param $key String: message name + * @param $fallback String: default message if the message cache can't be + * called by the exception * The function also has other parameters that are arguments for the message * @return String message with arguments replaced */ function msg( $key, $fallback /*[, params...] */ ) { $args = array_slice( func_get_args(), 2 ); + if ( $this->useMessageCache() ) { return wfMsgReal( $key, $args ); } else { @@ -82,7 +103,8 @@ class MWException extends Exception { */ function getHTML() { global $wgShowExceptionDetails; - if( $wgShowExceptionDetails ) { + + if ( $wgShowExceptionDetails ) { return '

' . nl2br( htmlspecialchars( $this->getMessage() ) ) . '

Backtrace:

' . nl2br( htmlspecialchars( $this->getTraceAsString() ) ) . "

\n"; @@ -99,7 +121,8 @@ class MWException extends Exception { */ function getText() { global $wgShowExceptionDetails; - if( $wgShowExceptionDetails ) { + + if ( $wgShowExceptionDetails ) { return $this->getMessage() . "\nBacktrace:\n" . $this->getTraceAsString() . "\n"; } else { @@ -114,6 +137,7 @@ class MWException extends Exception { return wfMsg( 'internalerror' ); } else { global $wgSitename; + return "$wgSitename error"; } } @@ -122,13 +146,15 @@ class MWException extends Exception { * Return the requested URL and point to file and line number from which the * exception occured * - * @return string + * @return String */ function getLogMessage() { global $wgRequest; + $file = $this->getFile(); $line = $this->getLine(); $message = $this->getMessage(); + if ( isset( $wgRequest ) ) { $url = $wgRequest->getRequestURL(); if ( !$url ) { @@ -144,6 +170,7 @@ class MWException extends Exception { /** Output the exception report using HTML */ function reportHTML() { global $wgOut; + if ( $this->useOutputPage() ) { $wgOut->setPageTitle( $this->getPageTitle() ); $wgOut->setRobotPolicy( "noindex,nofollow" ); @@ -151,33 +178,43 @@ class MWException extends Exception { $wgOut->enableClientCache( false ); $wgOut->redirect( '' ); $wgOut->clearHTML(); - if( $hookResult = $this->runHooks( get_class( $this ) ) ) { + + $hookResult = $this->runHooks( get_class( $this ) ); + if ( $hookResult ) { $wgOut->addHTML( $hookResult ); } else { $wgOut->addHTML( $this->getHTML() ); } + $wgOut->output(); } else { - if( $hookResult = $this->runHooks( get_class( $this ) . "Raw" ) ) { + $hookResult = $this->runHooks( get_class( $this ) . "Raw" ); + if ( $hookResult ) { die( $hookResult ); } - echo $this->htmlHeader(); - echo $this->getHTML(); - echo $this->htmlFooter(); + + if ( defined( 'MEDIAWIKI_INSTALL' ) || $this->htmlBodyOnly() ) { + echo $this->getHTML(); + } else { + echo $this->htmlHeader(); + echo $this->getHTML(); + echo $this->htmlFooter(); + } } } /** * Output a report about the exception and takes care of formatting. - * It will be either HTML or plain text based on $wgCommandLineMode. + * It will be either HTML or plain text based on isCommandLine(). */ function report() { - global $wgCommandLineMode; $log = $this->getLogMessage(); + if ( $log ) { wfDebugLog( 'exception', $log ); } - if ( $wgCommandLineMode ) { + + if ( self::isCommandLine() ) { wfPrintError( $this->getText() ); } else { $this->reportHTML(); @@ -189,30 +226,60 @@ class MWException extends Exception { * $wgOut to output the exception. */ function htmlHeader() { - global $wgLogo, $wgSitename, $wgOutputEncoding; + global $wgLogo, $wgOutputEncoding, $wgLang; if ( !headers_sent() ) { header( 'HTTP/1.0 500 Internal Server Error' ); - header( 'Content-type: text/html; charset='.$wgOutputEncoding ); + header( 'Content-type: text/html; charset=' . $wgOutputEncoding ); /* Don't cache error pages! They cause no end of trouble... */ header( 'Cache-control: none' ); header( 'Pragma: nocache' ); } - $title = $this->getPageTitle(); - echo " - - $title - - -

$title

- "; + + $title = Html::element( 'title', null, $this->getPageTitle() ); + + $left = 'left'; + $right = 'right'; + $dir = 'ltr'; + $code = 'en'; + + if ( $wgLang instanceof Language ) { + $left = $wgLang->alignStart(); + $right = $wgLang->alignEnd(); + $dir = $wgLang->getDir(); + $code = $wgLang->getCode(); + } + + $header = Html::element( 'img', array( + 'src' => $wgLogo, + 'style' => "float: $left; margin-$right: 1em;", + 'alt' => '' ), $this->getPageTitle() ); + + $attribs = array( 'dir' => $dir, 'lang' => $code ); + + return + Html::htmlHeader( $attribs ) . + Html::rawElement( 'head', null, $title ) . "\n". + Html::openElement( 'body' ) . "\n" . + Html::rawElement( 'h1', null, $header ) . "\n"; } /** * print the end of the html page if not using $wgOut. */ function htmlFooter() { - echo ""; + return Html::closeElement( 'body' ) . Html::closeElement( 'html' ); + } + + /** + * headers handled by subclass? + */ + function htmlBodyOnly() { + return false; + } + + static function isCommandLine() { + return !empty( $GLOBALS['wgCommandLineMode'] ) && !defined( 'MEDIAWIKI_INSTALL' ); } } @@ -248,6 +315,7 @@ class ErrorPageError extends MWException { function report() { global $wgOut; + $wgOut->showErrorPage( $this->title, $this->msg ); $wgOut->output(); } @@ -264,49 +332,62 @@ function wfInstallExceptionHandler() { * Report an exception to the user */ function wfReportException( Exception $e ) { - if ( $e instanceof MWException ) { - try { - $e->report(); - } catch ( Exception $e2 ) { - // Exception occurred from within exception handler - // Show a simpler error message for the original exception, - // don't try to invoke report() - $message = "MediaWiki internal error.\n\n" . - "Original exception: " . $e->__toString() . - "\n\nException caught inside exception handler: " . - $e2->__toString() . "\n"; - - if ( !empty( $GLOBALS['wgCommandLineMode'] ) ) { - wfPrintError( $message ); - } else { - echo nl2br( htmlspecialchars( $message ) ). "\n"; - } - } - } else { - $message = "Unexpected non-MediaWiki exception encountered, of type \"" . get_class( $e ) . "\"\n" . - $e->__toString() . "\n"; - if ( $GLOBALS['wgShowExceptionDetails'] ) { - $message .= "\n" . $e->getTraceAsString() ."\n"; - } - if ( !empty( $GLOBALS['wgCommandLineMode'] ) ) { - wfPrintError( $message ); - } else { - echo nl2br( htmlspecialchars( $message ) ). "\n"; - } - } + global $wgShowExceptionDetails; + + $cmdLine = MWException::isCommandLine(); + + if ( $e instanceof MWException ) { + try { + $e->report(); + } catch ( Exception $e2 ) { + // Exception occurred from within exception handler + // Show a simpler error message for the original exception, + // don't try to invoke report() + $message = "MediaWiki internal error.\n\n"; + + if ( $wgShowExceptionDetails ) { + $message .= 'Original exception: ' . $e->__toString() . "\n\n" . + 'Exception caught inside exception handler: ' . $e2->__toString(); + } else { + $message .= "Exception caught inside exception handler.\n\n" . + "Set \$wgShowExceptionDetails = true; at the bottom of LocalSettings.php " . + "to show detailed debugging information."; + } + + $message .= "\n"; + + if ( $cmdLine ) { + wfPrintError( $message ); + } else { + echo nl2br( htmlspecialchars( $message ) ) . "\n"; + } + } + } else { + $message = "Unexpected non-MediaWiki exception encountered, of type \"" . get_class( $e ) . "\"\n" . + $e->__toString() . "\n"; + + if ( $wgShowExceptionDetails ) { + $message .= "\n" . $e->getTraceAsString() . "\n"; + } + + if ( $cmdLine ) { + wfPrintError( $message ); + } else { + echo nl2br( htmlspecialchars( $message ) ) . "\n"; + } + } } /** * Print a message, if possible to STDERR. - * Use this in command line mode only (see wgCommandLineMode) + * Use this in command line mode only (see isCommandLine) */ function wfPrintError( $message ) { - #NOTE: STDERR may not be available, especially if php-cgi is used from the command line (bug #15602). + # NOTE: STDERR may not be available, especially if php-cgi is used from the command line (bug #15602). # Try to produce meaningful output anyway. Using echo may corrupt output to STDOUT though. if ( defined( 'STDERR' ) ) { fwrite( STDERR, $message ); - } - else { + } else { echo( $message ); } } @@ -324,9 +405,10 @@ function wfPrintError( $message ) { */ function wfExceptionHandler( $e ) { global $wgFullyInitialised; + wfReportException( $e ); - // Final cleanup, similar to wfErrorExit() + // Final cleanup if ( $wgFullyInitialised ) { try { wfLogProfilingData(); // uses $wgRequest, hence the $wgFullyInitialised condition