Adamw has submitted this change and it was merged.

Change subject: Throw MWException instead of Exception
......................................................................


Throw MWException instead of Exception

Change-Id: I3c7bc32b1b1a5d74be70502e4bd21c4a8df62d6b
---
M Timeline.php
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Adamw: Verified; Looks good to me, approved
  jenkins-bot: Checked



diff --git a/Timeline.php b/Timeline.php
index 1cab1ff..abc2033 100644
--- a/Timeline.php
+++ b/Timeline.php
@@ -66,7 +66,7 @@
 /**
  * @param $timelinesrc string
  * @param $args array
- * @throws Exception
+ * @throws MWException
  * @return string HTML
  */
 function wfRenderTimeline( $timelinesrc, array $args ) {
@@ -141,7 +141,7 @@
                                $svgHandle = fopen( $svgFilename, "r" );
                                wfRestoreWarnings();
                                if ( !$svgHandle ) {
-                                       throw new Exception( "Unable to open 
file $svgFilename for reading the timeline size" );
+                                       throw new MWException( "Unable to open 
file $svgFilename for reading the timeline size" );
                                }
                                while ( !feof( $svgHandle ) ) {
                                        $line = fgets( $svgHandle );

-- 
To view, visit https://gerrit.wikimedia.org/r/78955
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3c7bc32b1b1a5d74be70502e4bd21c4a8df62d6b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/timeline
Gerrit-Branch: master
Gerrit-Owner: Matmarex <[email protected]>
Gerrit-Reviewer: Adamw <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to