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

Change subject: pass text not status object to setTranscodeError
......................................................................


pass text not status object to setTranscodeError

Change-Id: I477a383bbf064ac5a77b2baa7f6fb0e979b01fee
---
M WebVideoTranscode/WebVideoTranscodeJob.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/WebVideoTranscode/WebVideoTranscodeJob.php 
b/WebVideoTranscode/WebVideoTranscodeJob.php
index c5dcd14..094c8d9 100644
--- a/WebVideoTranscode/WebVideoTranscodeJob.php
+++ b/WebVideoTranscode/WebVideoTranscodeJob.php
@@ -243,7 +243,7 @@
                                // no need to invalidate all pages with video. 
Because all pages remain valid ( no $transcodeKey derivative )
                                // just clear the file page ( so that the 
transcode table shows the error )
                                $this->title->invalidateCache();
-                               $this->setTranscodeError( $transcodeKey, 
$result );
+                               $this->setTranscodeError( $transcodeKey, 
$result->getWikiText() );
                                $status = false;
                        } else {
                                $bitrate = round( intval( filesize( 
$this->getTargetEncodePath() ) /  $file->getLength() ) * 8 );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I477a383bbf064ac5a77b2baa7f6fb0e979b01fee
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TimedMediaHandler
Gerrit-Branch: master
Gerrit-Owner: J <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to