makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-28 14:18:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/14402] Get rid of globals in tidy_plupload cron

Browse source 
PHPBB3-14402
This commit is contained in:
Marc Alexander 2016-01-10 10:10:41 +01:00
parent 3a82ace3eb
commit 3a555d94af
2 changed files with 16 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
phpBB/config/default/container/services_cron.yml
View file

@ -106,6 +106,8 @@ services:
arguments: arguments:
- '%core.root_path%' - '%core.root_path%'
- '@config' - '@config'
- '@log'
- '@user'
calls: calls:
- [set_name, [cron.task.core.tidy_plupload]] - [set_name, [cron.task.core.tidy_plupload]]
tags: tags:

20
phpBB/phpbb/cron/task/core/tidy_plupload.php
View file

@ -42,6 +42,12 @@ class tidy_plupload extends \phpbb\cron\task\base
*/ */
protected $config; protected $config;
/** @var \phpbb\log\log_interface */
protected $log;
/** @var \phpbb\user */
protected $user;
/** /**
* Directory where plupload stores temporary files. * Directory where plupload stores temporary files.
* @var string * @var string
@ -53,11 +59,15 @@ class tidy_plupload extends \phpbb\cron\task\base
* *
* @param string $phpbb_root_path The root path * @param string $phpbb_root_path The root path
* @param \phpbb\config\config $config The config * @param \phpbb\config\config $config The config
* @param \phpbb\log\log_interface $log Log
* @param \phpbb\user $user User object
*/ */
public function __construct($phpbb_root_path, \phpbb\config\config $config) public function __construct($phpbb_root_path, \phpbb\config\config $config, \phpbb\log\log_interface $log, \phpbb\user $user)
{ {
$this->phpbb_root_path = $phpbb_root_path; $this->phpbb_root_path = $phpbb_root_path;
$this->config = $config; $this->config = $config;
$this->log = $log;
$this->user = $user;
$this->plupload_upload_path = $this->phpbb_root_path . $this->config['upload_path'] . '/plupload'; $this->plupload_upload_path = $this->phpbb_root_path . $this->config['upload_path'] . '/plupload';
} }
@ -67,8 +77,6 @@ class tidy_plupload extends \phpbb\cron\task\base
*/ */
public function run() public function run()
{ {
global $user, $phpbb_log;
// Remove old temporary file (perhaps failed uploads?) // Remove old temporary file (perhaps failed uploads?)
$last_valid_timestamp = time() - $this->max_file_age; $last_valid_timestamp = time() - $this->max_file_age;
try try
@ -90,9 +98,9 @@ class tidy_plupload extends \phpbb\cron\task\base
} }
catch (\UnexpectedValueException $e) catch (\UnexpectedValueException $e)
{ {
$user_id = (empty($user->data)) ? ANONYMOUS : $user->data['user_id']; $user_id = (empty($this->user->data)) ? ANONYMOUS : $this->user->data['user_id'];
$user_ip = (empty($user->ip)) ? '' : $user->ip; $user_ip = (empty($this->user->ip)) ? '' : $this->user->ip;
$phpbb_log->add('critical', $user_id, $user_ip, 'LOG_PLUPLOAD_TIDY_FAILED', false, array( $this->log->add('critical', $user_id, $user_ip, 'LOG_PLUPLOAD_TIDY_FAILED', false, array(
$this->plupload_upload_path, $this->plupload_upload_path,
$e->getMessage(), $e->getMessage(),
$e->getTraceAsString() $e->getTraceAsString()