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/13981] Add events to capture avatar deletion or overwriting

Browse source 
PHPBB3-13981
This commit is contained in:
javiexin 2015-07-03 21:39:36 +02:00
parent 558d604d83
commit e0efd5ee57
1 changed files with 53 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

60
phpBB/phpbb/avatar/driver/upload.php
View file

@ -80,6 +80,8 @@ class upload extends \phpbb\avatar\driver\driver
*/ */
public function process_form($request, $template, $user, $row, &$error) public function process_form($request, $template, $user, $row, &$error)
{ {
global $phpbb_dispatcher;
if (!$this->can_upload()) if (!$this->can_upload())
{ {
return false; return false;
@ -151,13 +153,34 @@ class upload extends \phpbb\avatar\driver\driver
$destination = ''; $destination = '';
} }
// Move file and overwrite any existing image /**
$file->move_file($destination, true); * Before overwriting an existing avatar with a newly uploaded avatar
*
* @event core.avatar_driver_upload_overwrite_before
* @var string destination Destination directory where the file is going to be moved
* @var string prefix Prefix for the avatar filename
* @var array row Array with avatar row data
* @var array error Array of errors, if filled in by this event file will not be moved
* @since 3.1.6-RC1
*/
$vars = array(
'destination',
'prefix',
'row',
'error',
);
extract($phpbb_dispatcher->trigger_event('core.avatar_driver_upload_overwrite_before', compact($vars)));
if (sizeof($file->error)) if (!sizeof($error))
{
// Move file and overwrite any existing image
$file->move_file($destination, true);
}
$error = array_merge($error, $file->error);
if (sizeof($error))
{ {
$file->remove(); $file->remove();
$error = array_merge($error, $file->error);
return false; return false;
} }
@ -185,10 +208,33 @@ class upload extends \phpbb\avatar\driver\driver
*/ */
public function delete($row) public function delete($row)
{ {
$ext = substr(strrchr($row['avatar'], '.'), 1); global $phpbb_dispatcher;
$filename = $this->phpbb_root_path . $this->config['avatar_path'] . '/' . $this->config['avatar_salt'] . '_' . $row['id'] . '.' . $ext;
if (file_exists($filename)) $error = array();
$destination = $this->config['avatar_path'];
$prefix = $this->config['avatar_salt'] . '_';
$ext = substr(strrchr($row['avatar'], '.'), 1);
$filename = $this->phpbb_root_path . $destination . '/' . $prefix . $row['id'] . '.' . $ext;
/**
* Before deleting an existing avatar
*
* @event core.avatar_driver_upload_delete_before
* @var string destination Destination directory where the file is going to be deleted
* @var string prefix Prefix for the avatar filename
* @var array row Array with avatar row data
* @var array error Array of errors, if filled in by this event file will not be deleted
* @since 3.1.6-RC1
*/
$vars = array(
'destination',
'prefix',
'row',
'error',
);
extract($phpbb_dispatcher->trigger_event('core.avatar_driver_upload_delete_before', compact($vars)));
if (!sizeof($error) && file_exists($filename))
{ {
@unlink($filename); @unlink($filename);
} }