diff options
| author | Marc Alexander <admin@m-a-styles.de> | 2014-11-21 23:16:22 +0100 |
|---|---|---|
| committer | Marc Alexander <admin@m-a-styles.de> | 2014-11-21 23:16:22 +0100 |
| commit | 171837eefe2f8e17597629b108e2aa30c0f4055f (patch) | |
| tree | 4646873e5041bf5b88e1ebf533d78f358547d1e1 /phpBB/phpbb/file_downloader.php | |
| parent | 352648f173e7b132544bf3eaa494184bec6d5aa2 (diff) | |
| download | forums-171837eefe2f8e17597629b108e2aa30c0f4055f.tar forums-171837eefe2f8e17597629b108e2aa30c0f4055f.tar.gz forums-171837eefe2f8e17597629b108e2aa30c0f4055f.tar.bz2 forums-171837eefe2f8e17597629b108e2aa30c0f4055f.tar.xz forums-171837eefe2f8e17597629b108e2aa30c0f4055f.zip | |
[ticket/13358] Do not pass variables by reference
PHPBB3-13358
Diffstat (limited to 'phpBB/phpbb/file_downloader.php')
| -rw-r--r-- | phpBB/phpbb/file_downloader.php | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/phpBB/phpbb/file_downloader.php b/phpBB/phpbb/file_downloader.php index 2d5d9a7516..2a2b8dbab5 100644 --- a/phpBB/phpbb/file_downloader.php +++ b/phpBB/phpbb/file_downloader.php @@ -38,6 +38,10 @@ class file_downloader */ function get($host, $directory, $filename, $port = 80, $timeout = 6) { + // Set default values for error variables + $this->error_number = 0; + $this->error_string = ''; + if ($socket = @fsockopen($host, $port, $this->error_number, $this->error_string, $timeout)) { @fputs($socket, "GET $directory/$filename HTTP/1.0\r\n"); @@ -95,30 +99,22 @@ class file_downloader } /** - * Set error string - * - * @param string $error_string Error string + * Get error string * - * @return self + * @return string Error string */ - public function set_error_string(&$error_string) + public function get_error_string() { - $this->error_string = &$error_string; - - return $this; + return $this->error_string; } /** - * Set error number + * Get error number * - * @param int $error_number Error number - * - * @return self + * @return int Error number */ - public function set_error_number(&$error_number) + public function get_error_number() { - $this->error_number = &$error_number; - - return $this; + return $this->error_number; } } |