[ticket/15699] Use template variables

PHPBB3-15699
This commit is contained in:
rubencm 2021-03-22 13:37:16 +01:00 committed by Ruben Calvo
parent e80e70ad2c
commit 30e6673cce
No known key found for this signature in database
3 changed files with 24 additions and 18 deletions

View file

@ -126,9 +126,9 @@
<dl> <dl>
<dt><label for="update_type">{{ lang('STORAGE_UPDATE_TYPE') }}{{ lang('COLON') }}</label></dt> <dt><label for="update_type">{{ lang('STORAGE_UPDATE_TYPE') }}{{ lang('COLON') }}</label></dt>
<dd> <dd>
<label><input id="update_type" class="radio" name="update_type" value="{{ constant('STORAGE_UPDATE_TYPE_CONFIG') }}" checked="checked" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_CONFIG') }}</label> <label><input id="update_type" class="radio" name="update_type" value="{{ STORAGE_UPDATE_TYPE_CONFIG }}" checked="checked" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_CONFIG') }}</label>
<label><input class="radio" name="update_type" value="{{ constant('STORAGE_UPDATE_TYPE_COPY') }}" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_COPY') }}</label> <label><input class="radio" name="update_type" value="{{ STORAGE_UPDATE_TYPE_COPY }}" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_COPY') }}</label>
<label><input class="radio" name="update_type" value="{{ constant('STORAGE_UPDATE_TYPE_MOVE') }}" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_MOVE') }}</label> <label><input class="radio" name="update_type" value="{{ STORAGE_UPDATE_TYPE_MOVE }}" type="radio"> {{ lang('STORAGE_UPDATE_TYPE_MOVE') }}</label>
</dd> </dd>
</dl> </dl>
</fieldset> </fieldset>

View file

@ -86,6 +86,13 @@ class acp_storage
/** @var mixed */ /** @var mixed */
protected $state; protected $state;
/**
* Update type constants
*/
public const STORAGE_UPDATE_TYPE_CONFIG = 0;
public const STORAGE_UPDATE_TYPE_COPY = 1;
public const STORAGE_UPDATE_TYPE_MOVE = 2;
/** /**
* @param string $id * @param string $id
* @param string $mode * @param string $mode
@ -181,7 +188,7 @@ class acp_storage
} }
// If update_type is copy or move, copy files from the old to the new storage // If update_type is copy or move, copy files from the old to the new storage
if (in_array($this->state['update_type'], [STORAGE_UPDATE_TYPE_COPY, STORAGE_UPDATE_TYPE_MOVE], true)) if (in_array($this->state['update_type'], [self::STORAGE_UPDATE_TYPE_COPY, self::STORAGE_UPDATE_TYPE_MOVE], true))
{ {
$i = 0; $i = 0;
foreach ($this->state['storages'] as $storage_name => $storage_options) foreach ($this->state['storages'] as $storage_name => $storage_options)
@ -208,7 +215,7 @@ class acp_storage
{ {
$this->save_state(); $this->save_state();
meta_refresh(1, append_sid($this->u_action . '&amp;action=update&amp;hash=' . generate_link_hash('acp_storage'))); meta_refresh(1, append_sid($this->u_action . '&amp;action=update&amp;hash=' . generate_link_hash('acp_storage')));
trigger_error($this->lang->lang('STORAGE_UPDATE_REDIRECT', $this->lang->lang('STORAGE_' . strtoupper($storage_name) . '_TITLE'), $i + 1, count($this->state['storages']))); trigger_error($this->lang->lang('self::STORAGE_UPDATE_REDIRECT', $this->lang->lang('STORAGE_' . strtoupper($storage_name) . '_TITLE'), $i + 1, count($this->state['storages'])));
} }
$stream = $current_adapter->read_stream($row['file_path']); $stream = $current_adapter->read_stream($row['file_path']);
@ -228,7 +235,7 @@ class acp_storage
} }
// If update_type is move files, remove the old files // If update_type is move files, remove the old files
if ($this->state['update_type'] === STORAGE_UPDATE_TYPE_MOVE) if ($this->state['update_type'] === self::STORAGE_UPDATE_TYPE_MOVE)
{ {
$i = 0; $i = 0;
foreach ($this->state['storages'] as $storage_name => $storage_options) foreach ($this->state['storages'] as $storage_name => $storage_options)
@ -355,7 +362,7 @@ class acp_storage
$this->state = [ $this->state = [
// Save the value of the checkbox, to remove all files from the // Save the value of the checkbox, to remove all files from the
// old storage once they have been successfully moved // old storage once they have been successfully moved
'update_type' => (int) $this->request->variable('update_type', STORAGE_UPDATE_TYPE_CONFIG), 'update_type' => (int) $this->request->variable('update_type', self::STORAGE_UPDATE_TYPE_CONFIG),
'storage_index' => 0, 'storage_index' => 0,
'file_index' => 0, 'file_index' => 0,
'remove_storage_index' => 0, 'remove_storage_index' => 0,
@ -432,6 +439,10 @@ class acp_storage
'S_ERROR' => !empty($messages), 'S_ERROR' => !empty($messages),
'U_ACTION' => $this->u_action . '&amp;hash=' . generate_link_hash('acp_storage'), 'U_ACTION' => $this->u_action . '&amp;hash=' . generate_link_hash('acp_storage'),
'STORAGE_UPDATE_TYPE_CONFIG' => self::STORAGE_UPDATE_TYPE_CONFIG,
'STORAGE_UPDATE_TYPE_COPY' => self::STORAGE_UPDATE_TYPE_COPY,
'STORAGE_UPDATE_TYPE_MOVE' => self::STORAGE_UPDATE_TYPE_MOVE,
]); ]);
} }

View file

@ -228,11 +228,6 @@ define('CAPTCHA_MAX_CHARS', 7);
// Additional constants // Additional constants
define('VOTE_CONVERTED', 127); define('VOTE_CONVERTED', 127);
// Storage update methods
define('STORAGE_UPDATE_TYPE_CONFIG', 0);
define('STORAGE_UPDATE_TYPE_COPY', 1);
define('STORAGE_UPDATE_TYPE_MOVE', 2);
// BC global FTW // BC global FTW
global $table_prefix; global $table_prefix;