r53737 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r53736‎ | r53737 | r53738 >
Date:00:19, 25 July 2009
Author:mrzman
Status:ok
Tags:
Comment:
Fix a couple notices/strict warnings
Modified paths:
  • /trunk/phase3/includes/HttpFunctions.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/HttpFunctions.php
@@ -93,7 +93,7 @@
9494 * should write to a file location and give updates
9595 *
9696 */
97 - private function initBackgroundDownload( $url, $target_file_path, $content_length = null ){
 97+ private static function initBackgroundDownload( $url, $target_file_path, $content_length = null ){
9898 global $wgMaxUploadSize, $IP, $wgPhpCli, $wgServer;
9999 $status = Status::newGood();
100100
@@ -115,7 +115,7 @@
116116
117117 // run the background download request:
118118 $cmd = $wgPhpCli . ' ' . $IP . "/maintenance/http_session_download.php --sid {$session_id} --usk {$upload_session_key}";
119 - $pid = wfShellBackgroundExec( $cmd, $retval );
 119+ $pid = wfShellBackgroundExec( $cmd );
120120 // the pid is not of much use since we won't be visiting this same apache any-time soon.
121121 if( !$pid )
122122 return Status::newFatal( 'could not run background shell exec' );
@@ -127,7 +127,7 @@
128128 return $status;
129129 }
130130
131 - function getUploadSessionKey(){
 131+ static function getUploadSessionKey(){
132132 $key = mt_rand( 0, 0x7fffffff );
133133 $_SESSION['wsUploadData'][$key] = array();
134134 return $key;
@@ -566,4 +566,4 @@
567567 }
568568 }
569569
570 -}
\ No newline at end of file
 570+}

Status & tagging log