Anomie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/93087
Change subject: MWException: Cleanup exception message output
......................................................................
MWException: Cleanup exception message output
Change I0a9e9244 lost the message when handling a non-MWException
exception, and for a long time MWException's getHTML and getText have
been missing the actual file and line where the exception was thrown.
We may as well use MWExceptionHandler::getLogMessage to fix all of
these, resulting in a more standardized exception output.
Change-Id: I8a9b4831c9c586bafe0a54516ff779cdfb008984
---
M includes/Exception.php
1 file changed, 3 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/87/93087/1
diff --git a/includes/Exception.php b/includes/Exception.php
index 9623a84..ac98564 100644
--- a/includes/Exception.php
+++ b/includes/Exception.php
@@ -140,7 +140,7 @@
global $wgShowExceptionDetails;
if ( $wgShowExceptionDetails ) {
- return '<p>' . nl2br( htmlspecialchars(
$this->getMessage() ) ) .
+ return '<p>' . nl2br( htmlspecialchars(
MWExceptionHandler::getLogMessage( $this ) ) ) .
'</p><p>Backtrace:</p><p>' . nl2br(
htmlspecialchars( MWExceptionHandler::getRedactedTraceAsString( $this ) ) ) .
"</p>\n";
} else {
@@ -165,7 +165,7 @@
global $wgShowExceptionDetails;
if ( $wgShowExceptionDetails ) {
- return $this->getMessage() .
+ return MWExceptionHandler::getLogMessage( $this ) .
"\nBacktrace:\n" .
MWExceptionHandler::getRedactedTraceAsString( $this ) . "\n";
} else {
return "Set \$wgShowExceptionDetails = true; " .
@@ -641,8 +641,7 @@
get_class( $e ) . "\"";
if ( $wgShowExceptionDetails ) {
- $message .= "\nexception '" . get_class( $e ) .
"' in " .
- $e->getFile() . ":" . $e->getLine() .
"\nStack trace:\n" .
+ $message .= "\n" .
MWExceptionHandler::getLogMessage( $e ) . "\nBacktrace:\n" .
self::getRedactedTraceAsString( $e ) .
"\n";
}
--
To view, visit https://gerrit.wikimedia.org/r/93087
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a9b4831c9c586bafe0a54516ff779cdfb008984
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
