r57447 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r57446‎ | r57447 | r57448 >
Date:23:46, 6 October 2009
Author:dale
Status:deferred
Tags:
Comment:
dont check
Modified paths:
  • /branches/wmf-deployment/extensions/OggHandler/OggHandler_body.php (modified) (history)

Diff [purge]

Index: branches/wmf-deployment/extensions/OggHandler/OggHandler_body.php
@@ -242,7 +242,7 @@
243243 ' -n ' . wfEscapeShellArg( $dstPath ) . ' ' .
244244 ' ' . wfEscapeShellArg( $file->getPath() ) . ' 2>&1';
245245 $returnText = wfShellExec( $cmd, $retval );
246 - if ( !$this->removeBadFile( $dstPath, $retval ) || $retval ) {
 246+ if ( ! $this->removeBadFile( $dstPath, $retval ) ) {
247247 return new OggVideoDisplay( $file, $file->getURL(), $dstUrl, $width, $height, $length, $dstPath );
248248 }
249249 }

Follow-up revisions

RevisionCommit summaryAuthorDate
r57448Merging core changes from trunk r56150 through r57447 ... there may be some m...brion00:35, 7 October 2009

Status & tagging log