jenkins-bot has submitted this change and it was merged.

Change subject: Fix undefined variable warning
......................................................................


Fix undefined variable warning

Variable was only used to cleanup the error message (rm path info),
so it didn't really make a difference it was undefined. Not like
the temp directory = /tmp is a secret.

Change-Id: Iefcdd433272c87791b3fb2bfa562f4ee002fc97d
---
M Score.body.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Reedy: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/Score.body.php b/Score.body.php
index 3201c4b..56b294f 100644
--- a/Score.body.php
+++ b/Score.body.php
@@ -592,7 +592,7 @@
                }
                if ( $rc2 != 0 ) {
                        $output .= "\nexited with status: " . $rc2;
-                       self::throwCallException( wfMessage( 
'score-compilererr' ), $output, $options['factory_dir'] );
+                       self::throwCallException( wfMessage( 
'score-compilererr' ), $output, $options['factory_directory'] );
                }
                if ( !file_exists( $factoryMidi ) ) {
                        throw new ScoreException( wfMessage( 'score-nomidi' ) );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iefcdd433272c87791b3fb2bfa562f4ee002fc97d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Score
Gerrit-Branch: master
Gerrit-Owner: Brian Wolff <[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