[feature/avatars] Shorten avatar driver's class name

The addition of "core_" that was used until now would require additional
avatar drivers (e.g. in extensions) to also include that in their class
name which would be incorrect.

PHPBB3-10018
This commit is contained in:
Marc Alexander 2012-11-16 17:19:04 +01:00
parent cc86bd9267
commit 0a5d54cc9b
8 changed files with 18 additions and 18 deletions

View file

@ -1,6 +1,6 @@
services: services:
avatar.driver.core.gravatar: avatar.driver.gravatar:
class: phpbb_avatar_driver_core_gravatar class: phpbb_avatar_driver_gravatar
arguments: arguments:
- @config - @config
- @request - @request
@ -10,8 +10,8 @@ services:
tags: tags:
- { name: avatar.driver } - { name: avatar.driver }
avatar.driver.core.local: avatar.driver.local:
class: phpbb_avatar_driver_core_local class: phpbb_avatar_driver_local
arguments: arguments:
- @config - @config
- @request - @request
@ -21,8 +21,8 @@ services:
tags: tags:
- { name: avatar.driver } - { name: avatar.driver }
avatar.driver.core.remote: avatar.driver.remote:
class: phpbb_avatar_driver_core_remote class: phpbb_avatar_driver_remote
arguments: arguments:
- @config - @config
- @request - @request
@ -32,8 +32,8 @@ services:
tags: tags:
- { name: avatar.driver } - { name: avatar.driver }
avatar.driver.core.upload: avatar.driver.upload:
class: phpbb_avatar_driver_core_upload class: phpbb_avatar_driver_upload
arguments: arguments:
- @config - @config
- @request - @request

View file

@ -147,7 +147,7 @@ abstract class phpbb_avatar_driver implements phpbb_avatar_driver_interface
**/ **/
public function is_enabled() public function is_enabled()
{ {
$driver = preg_replace('#^phpbb_avatar_driver_core_#', '', get_class($this)); $driver = preg_replace('#^phpbb_avatar_driver_#', '', get_class($this));
return $this->config["allow_avatar_$driver"]; return $this->config["allow_avatar_$driver"];
} }
@ -157,7 +157,7 @@ abstract class phpbb_avatar_driver implements phpbb_avatar_driver_interface
**/ **/
public function get_template_name() public function get_template_name()
{ {
$driver = preg_replace('#^phpbb_avatar_driver_core_#', '', get_class($this)); $driver = preg_replace('#^phpbb_avatar_driver_#', '', get_class($this));
$template = "ucp_avatar_options_$driver.html"; $template = "ucp_avatar_options_$driver.html";
return $template; return $template;

View file

@ -20,7 +20,7 @@ if (!defined('IN_PHPBB'))
* @package avatars * @package avatars
*/ */
// @todo: rename classes to phpbb_ext_foo_avatar_driver_foo and similar // @todo: rename classes to phpbb_ext_foo_avatar_driver_foo and similar
class phpbb_avatar_driver_core_gravatar extends phpbb_avatar_driver class phpbb_avatar_driver_gravatar extends phpbb_avatar_driver
{ {
/** /**
* We'll need to create a different type of avatar for gravatar * We'll need to create a different type of avatar for gravatar

View file

@ -19,7 +19,7 @@ if (!defined('IN_PHPBB'))
* Handles avatars selected from the board gallery * Handles avatars selected from the board gallery
* @package avatars * @package avatars
*/ */
class phpbb_avatar_driver_core_local extends phpbb_avatar_driver class phpbb_avatar_driver_local extends phpbb_avatar_driver
{ {
/** /**
* @inheritdoc * @inheritdoc

View file

@ -19,7 +19,7 @@ if (!defined('IN_PHPBB'))
* Handles avatars hosted remotely * Handles avatars hosted remotely
* @package avatars * @package avatars
*/ */
class phpbb_avatar_driver_core_remote extends phpbb_avatar_driver class phpbb_avatar_driver_remote extends phpbb_avatar_driver
{ {
/** /**
* @inheritdoc * @inheritdoc

View file

@ -19,7 +19,7 @@ if (!defined('IN_PHPBB'))
* Handles avatars uploaded to the board * Handles avatars uploaded to the board
* @package avatars * @package avatars
*/ */
class phpbb_avatar_driver_core_upload extends phpbb_avatar_driver class phpbb_avatar_driver_upload extends phpbb_avatar_driver
{ {
/** /**
* @inheritdoc * @inheritdoc

View file

@ -57,13 +57,13 @@ class phpbb_avatar_manager
switch ($avatar_type) switch ($avatar_type)
{ {
case AVATAR_GALLERY: case AVATAR_GALLERY:
$avatar_type = 'avatar.driver.core.local'; $avatar_type = 'avatar.driver.local';
break; break;
case AVATAR_UPLOAD: case AVATAR_UPLOAD:
$avatar_type = 'avatar.driver.core.upload'; $avatar_type = 'avatar.driver.upload';
break; break;
case AVATAR_REMOTE: case AVATAR_REMOTE:
$avatar_type = 'avatar.driver.core.remote'; $avatar_type = 'avatar.driver.remote';
break; break;
} }

View file

@ -564,7 +564,7 @@ class ucp_profile
if (check_form_key('ucp_avatar')) if (check_form_key('ucp_avatar'))
{ {
$driver = str_replace('_', '.', request_var('avatar_driver', '')); $driver = str_replace('_', '.', request_var('avatar_driver', ''));
$config_name = preg_replace('#^avatar.driver.core.#', '', $driver); $config_name = preg_replace('#^avatar.driver.#', '', $driver);
if (in_array($driver, $avatar_drivers) && $config["allow_avatar_$config_name"]) if (in_array($driver, $avatar_drivers) && $config["allow_avatar_$config_name"])
{ {
$avatar = $phpbb_avatar_manager->get_driver($driver); $avatar = $phpbb_avatar_manager->get_driver($driver);