mirror of
https://github.com/phpbb/phpbb.git
synced 2025-07-31 22:10:45 +02:00
[ticket/15910] Pass object arguments by reference implicitly
PHPBB3-15910
This commit is contained in:
@@ -51,7 +51,7 @@ class gd extends captcha_abstract
|
||||
return 'CAPTCHA_GD';
|
||||
}
|
||||
|
||||
function acp_page($id, &$module)
|
||||
function acp_page($id, $module)
|
||||
{
|
||||
global $user, $template, $phpbb_log, $request;
|
||||
global $config;
|
||||
|
@@ -33,7 +33,7 @@ class gd_wave extends captcha_abstract
|
||||
return '\\phpbb\\captcha\\gd_wave';
|
||||
}
|
||||
|
||||
function acp_page($id, &$module)
|
||||
function acp_page($id, $module)
|
||||
{
|
||||
global $user;
|
||||
|
||||
|
@@ -33,7 +33,7 @@ class nogd extends captcha_abstract
|
||||
return '\\phpbb\\captcha\\non_gd';
|
||||
}
|
||||
|
||||
function acp_page($id, &$module)
|
||||
function acp_page($id, $module)
|
||||
{
|
||||
global $user;
|
||||
|
||||
|
@@ -638,7 +638,7 @@ class qa
|
||||
/**
|
||||
* API function - The ACP backend, this marks the end of the easy methods
|
||||
*/
|
||||
function acp_page($id, &$module)
|
||||
function acp_page($id, $module)
|
||||
{
|
||||
global $config, $request, $phpbb_log, $template, $user;
|
||||
|
||||
@@ -776,7 +776,7 @@ class qa
|
||||
/**
|
||||
* This handles the list overview
|
||||
*/
|
||||
function acp_question_list(&$module)
|
||||
function acp_question_list($module)
|
||||
{
|
||||
global $db, $template;
|
||||
|
||||
|
@@ -66,7 +66,7 @@ class recaptcha extends captcha_abstract
|
||||
throw new \Exception('No generator class given.');
|
||||
}
|
||||
|
||||
function acp_page($id, &$module)
|
||||
function acp_page($id, $module)
|
||||
{
|
||||
global $config, $template, $user, $phpbb_log, $request;
|
||||
|
||||
|
@@ -261,7 +261,7 @@ class upload
|
||||
*
|
||||
* @param filespec $file Instance of filespec class
|
||||
*/
|
||||
public function common_checks(&$file)
|
||||
public function common_checks($file)
|
||||
{
|
||||
// Filesize is too big or it's 0 if it was larger than the maxsize in the upload form
|
||||
if ($this->max_filesize && ($file->get('filesize') > $this->max_filesize || $file->get('filesize') == 0))
|
||||
@@ -297,7 +297,7 @@ class upload
|
||||
*
|
||||
* @return bool True if extension is allowed, false if not
|
||||
*/
|
||||
public function valid_extension(&$file)
|
||||
public function valid_extension($file)
|
||||
{
|
||||
return (in_array($file->get('extension'), $this->allowed_extensions)) ? true : false;
|
||||
}
|
||||
@@ -310,7 +310,7 @@ class upload
|
||||
* @return bool True if dimensions are valid or no constraints set, false
|
||||
* if not
|
||||
*/
|
||||
public function valid_dimensions(&$file)
|
||||
public function valid_dimensions($file)
|
||||
{
|
||||
if (!$this->max_width && !$this->max_height && !$this->min_width && !$this->min_height)
|
||||
{
|
||||
@@ -350,7 +350,7 @@ class upload
|
||||
*
|
||||
* @return bool True if content is valid, false if not
|
||||
*/
|
||||
public function valid_content(&$file)
|
||||
public function valid_content($file)
|
||||
{
|
||||
return ($file->check_content($this->disallowed_content));
|
||||
}
|
||||
|
Reference in New Issue
Block a user