1
0
mirror of https://github.com/phpbb/phpbb.git synced 2025-03-14 04:30:29 +01:00

[ticket/16671] Fix coding style

PHPBB3-16671
This commit is contained in:
Máté Bartus 2020-12-31 14:40:14 +01:00
parent cb894cee42
commit 8830354555
40 changed files with 73 additions and 76 deletions

View File

@ -26,11 +26,12 @@ use Symfony\Component\DependencyInjection\ContainerInterface;
class check extends \phpbb\console\command\command
{
/** @var \phpbb\config\config */
/** @var config */
protected $config;
/** @var \Symfony\Component\DependencyInjection\ContainerBuilder */
protected $phpbb_container;
/**
* @var language
*/

View File

@ -64,7 +64,7 @@ class resolver implements ControllerResolverInterface
/**
* Load a controller callable
*
* @param \Symfony\Component\HttpFoundation\Request $request Symfony Request object
* @param Request $request Symfony Request object
* @return bool|Callable Callable or false
* @throws \phpbb\controller\exception
*/
@ -119,7 +119,7 @@ class resolver implements ControllerResolverInterface
* and should match the parameters of the method you are using as your
* controller.
*
* @param \Symfony\Component\HttpFoundation\Request $request Symfony Request object
* @param Request $request Symfony Request object
* @param mixed $controller A callable (controller class, method)
* @return array An array of arguments to pass to the controller
* @throws \phpbb\controller\exception

View File

@ -26,17 +26,17 @@ use Symfony\Component\HttpKernel\Event\PostResponseEvent;
class cron_runner_listener implements EventSubscriberInterface
{
/**
* @var \phpbb\lock\db
* @var db
*/
private $cron_lock;
/**
* @var \phpbb\cron\manager
* @var manager
*/
private $cron_manager;
/**
* @var \phpbb\request\request_interface
* @var request_interface
*/
private $request;

View File

@ -80,8 +80,6 @@ class manager
* and puts them into $this->tasks.
*
* @param array|\Traversable $tasks Array of instances of \phpbb\cron\task\task
*
* @return null
*/
public function load_tasks($tasks)
{
@ -94,8 +92,6 @@ class manager
/**
* Loads registered tasks from the container, wraps them
* and puts them into $this->tasks.
*
* @return null
*/
public function load_tasks_from_container()
{
@ -116,7 +112,7 @@ class manager
*
* If no tasks are ready, null is returned.
*
* @return \phpbb\cron\task\wrapper|null
* @return wrapper|null
*/
public function find_one_ready_task()
{
@ -161,7 +157,7 @@ class manager
* Web runner uses this method to resolve names to tasks.
*
* @param string $name Name of the task to look up.
* @return \phpbb\cron\task\wrapper A wrapped task corresponding to the given name, or null.
* @return wrapper A wrapped task corresponding to the given name, or null.
*/
public function find_task($name)
{
@ -193,7 +189,7 @@ class manager
* Wraps a task inside an instance of \phpbb\cron\task\wrapper.
*
* @param \phpbb\cron\task\task $task The task.
* @return \phpbb\cron\task\wrapper The wrapped task.
* @return wrapper The wrapped task.
*/
public function wrap_task(\phpbb\cron\task\task $task)
{

View File

@ -21,7 +21,7 @@ class mssql_extractor extends base_extractor
* Writes closing line(s) to database backup
*
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
public function write_end()
{
@ -195,7 +195,7 @@ class mssql_extractor extends base_extractor
*
* @param string $table_name name of the database table
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
protected function write_data_mssqlnative($table_name)
{
@ -311,7 +311,7 @@ class mssql_extractor extends base_extractor
*
* @param string $table_name name of the database table
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
protected function write_data_odbc($table_name)
{

View File

@ -87,7 +87,7 @@ class mysql_extractor extends base_extractor
*
* @param string $table_name name of the database table
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
protected function write_data_mysqli($table_name)
{
@ -177,7 +177,7 @@ class mysql_extractor extends base_extractor
*
* @param string $table_name name of the database table
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
protected function new_write_table($table_name)
{
@ -202,7 +202,7 @@ class mysql_extractor extends base_extractor
*
* @param string $table_name name of the database table
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
protected function old_write_table($table_name)
{

View File

@ -324,7 +324,7 @@ class postgres_extractor extends base_extractor
* Writes closing line(s) to database backup
*
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
public function write_end()
{

View File

@ -136,7 +136,7 @@ class sqlite3_extractor extends base_extractor
* Writes closing line(s) to database backup
*
* @return null
* @throws \phpbb\db\extractor\exception\extractor_not_initialized_exception when calling this function before init_extractor()
* @throws extractor_not_initialized_exception when calling this function before init_extractor()
*/
public function write_end()
{

View File

@ -22,7 +22,7 @@ class container_configuration implements ConfigurationInterface
/**
* Generates the configuration tree builder.
*
* @return \Symfony\Component\Config\Definition\Builder\TreeBuilder The tree builder
* @return TreeBuilder The tree builder
*/
public function getConfigTreeBuilder()
{

View File

@ -21,7 +21,7 @@ use Symfony\Component\DependencyInjection\ContainerInterface;
class service_collection extends \ArrayObject
{
/**
* @var \Symfony\Component\DependencyInjection\ContainerInterface
* @var ContainerInterface
*/
protected $container;

View File

@ -59,7 +59,7 @@ class upload
/** @var \bantu\IniGetWrapper\IniGetWrapper ini_get() wrapper */
protected $php_ini;
/** @var \phpbb\language\language Language class */
/** @var language Language class */
protected $language;
/** @var request_interface Request class */

View File

@ -98,7 +98,7 @@ class install extends \phpbb\console\command\command
{
$this->iohandler_factory->set_environment('cli');
/** @var \phpbb\install\helper\iohandler\cli_iohandler $iohandler */
/** @var cli_iohandler $iohandler */
$iohandler = $this->iohandler_factory->get();
$style = new SymfonyStyle($input, $output);
$iohandler->set_style($style, $output);

View File

@ -98,7 +98,7 @@ class update extends \phpbb\console\command\command
{
$this->iohandler_factory->set_environment('cli');
/** @var \phpbb\install\helper\iohandler\cli_iohandler $iohandler */
/** @var cli_iohandler $iohandler */
$iohandler = $this->iohandler_factory->get();
$style = new SymfonyStyle($input, $output);
$iohandler->set_style($style, $output);

View File

@ -40,7 +40,7 @@ class helper
protected $installer_config;
/**
* @var \phpbb\language\language
* @var language
*/
protected $language;
@ -50,37 +50,37 @@ class helper
protected $language_cookie;
/**
* @var \phpbb\language\language_file_helper
* @var language_file_helper
*/
protected $lang_helper;
/**
* @var \phpbb\install\helper\navigation\navigation_provider
* @var navigation_provider
*/
protected $navigation_provider;
/**
* @var \phpbb\template\template
* @var template
*/
protected $template;
/**
* @var \phpbb\path_helper
* @var path_helper
*/
protected $path_helper;
/**
* @var \phpbb\request\request
* @var request
*/
protected $phpbb_request;
/**
* @var \phpbb\symfony_request
* @var symfony_request
*/
protected $request;
/**
* @var \phpbb\routing\router
* @var router
*/
protected $router;

View File

@ -35,7 +35,7 @@ class container_factory
protected $php_ext;
/**
* @var \phpbb\request\request
* @var request
*/
protected $request;
@ -78,7 +78,7 @@ class container_factory
* @return \Symfony\Component\DependencyInjection\ContainerInterface|Object phpBB's dependency injection container
* or the service specified in $service_name
*
* @throws \phpbb\install\exception\cannot_build_container_exception When container cannot be built
* @throws cannot_build_container_exception When container cannot be built
* @throws \Symfony\Component\DependencyInjection\Exception\InvalidArgumentException If the service is not defined
* @throws \Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException When a circular reference is detected
* @throws \Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException When the service is not defined
@ -101,7 +101,7 @@ class container_factory
*
* @return mixed
*
* @throws \phpbb\install\exception\cannot_build_container_exception When container cannot be built
* @throws cannot_build_container_exception When container cannot be built
*/
public function get_parameter($param_name)
{
@ -117,7 +117,7 @@ class container_factory
/**
* Build dependency injection container
*
* @throws \phpbb\install\exception\cannot_build_container_exception When container cannot be built
* @throws cannot_build_container_exception When container cannot be built
*/
protected function build_container()
{

View File

@ -230,7 +230,7 @@ class database
*
* @return bool|array true if table prefix is valid, array of errors otherwise
*
* @throws \phpbb\install\exception\invalid_dbms_exception When $dbms is not a valid
* @throws invalid_dbms_exception When $dbms is not a valid
*/
public function validate_table_prefix($dbms, $table_prefix)
{

View File

@ -54,7 +54,7 @@ class factory
*
* @return \phpbb\install\helper\iohandler\iohandler_interface
*
* @throws \phpbb\install\helper\iohandler\exception\iohandler_not_implemented_exception
* @throws iohandler_not_implemented_exception
* When the specified iohandler_interface does not exists
*/
public function get()

View File

@ -137,7 +137,7 @@ class installer
if (!$this->install_config->get('cache_purged_before', false) && $this->purge_cache_before)
{
/** @var \phpbb\cache\driver\driver_interface $cache */
/** @var driver_interface $cache */
$cache = $this->container_factory->get('cache.driver');
$cache->purge();
$this->install_config->set('cache_purged_before', true);
@ -311,7 +311,7 @@ class installer
try
{
/** @var \phpbb\cache\driver\driver_interface $cache */
/** @var driver_interface $cache */
$cache = $this->container_factory->get('cache.driver');
$cache->purge();
}

View File

@ -22,7 +22,7 @@ class installer_configuration implements ConfigurationInterface
/**
* Generates the configuration tree builder.
*
* @return \Symfony\Component\Config\Definition\Builder\TreeBuilder The tree builder
* @return TreeBuilder The tree builder
*/
public function getConfigTreeBuilder()
{

View File

@ -36,7 +36,7 @@ class add_modules extends \phpbb\install\task_base
protected $extension_manager;
/**
* @var \phpbb\install\helper\iohandler\iohandler_interface
* @var iohandler_interface
*/
protected $iohandler;

View File

@ -36,7 +36,7 @@ class notify_user extends \phpbb\install\task_base
protected $auth;
/**
* @var \phpbb\config\db
* @var db
*/
protected $config;

View File

@ -91,7 +91,7 @@ class obtain_admin_data extends \phpbb\install\task_base implements \phpbb\insta
*
* @param bool $use_request_data Whether to use submited data
*
* @throws \phpbb\install\exception\user_interaction_required_exception When the user is required to provide data
* @throws user_interaction_required_exception When the user is required to provide data
*/
protected function request_form_data($use_request_data = false)
{

View File

@ -110,7 +110,7 @@ class obtain_board_data extends \phpbb\install\task_base implements \phpbb\insta
*
* @param bool $use_request_data Whether to use submited data
*
* @throws \phpbb\install\exception\user_interaction_required_exception When the user is required to provide data
* @throws user_interaction_required_exception When the user is required to provide data
*/
protected function request_form_data($use_request_data = false)
{

View File

@ -108,7 +108,7 @@ class obtain_database_data extends \phpbb\install\task_base implements \phpbb\in
*
* @param bool $use_request_data Whether to use submited data
*
* @throws \phpbb\install\exception\user_interaction_required_exception When the user is required to provide data
* @throws user_interaction_required_exception When the user is required to provide data
*/
protected function request_form_data($use_request_data = false)
{

View File

@ -28,12 +28,12 @@ class obtain_file_updater_method extends task_base
protected $available_methods;
/**
* @var \phpbb\install\helper\config
* @var config
*/
protected $installer_config;
/**
* @var \phpbb\install\helper\iohandler\iohandler_interface
* @var iohandler_interface
*/
protected $iohandler;

View File

@ -22,12 +22,12 @@ use phpbb\install\task_base;
class obtain_update_ftp_data extends task_base
{
/**
* @var \phpbb\install\helper\config
* @var config
*/
protected $installer_config;
/**
* @var \phpbb\install\helper\iohandler\iohandler_interface
* @var iohandler_interface
*/
protected $iohandler;

View File

@ -21,12 +21,12 @@ use phpbb\install\task_base;
class obtain_update_settings extends task_base
{
/**
* @var \phpbb\install\helper\config
* @var config
*/
protected $installer_config;
/**
* @var \phpbb\install\helper\iohandler\iohandler_interface
* @var iohandler_interface
*/
protected $iohandler;

View File

@ -22,7 +22,7 @@ class updater_configuration implements ConfigurationInterface
/**
* Generates the configuration tree builder.
*
* @return \Symfony\Component\Config\Definition\Builder\TreeBuilder The tree builder
* @return TreeBuilder The tree builder
*/
public function getConfigTreeBuilder()
{

View File

@ -399,7 +399,7 @@ class language
*
* @return int The plural-case we need to use for the number plural-rule combination
*
* @throws \phpbb\language\exception\invalid_plural_rule_exception When $force_rule has an invalid value
* @throws invalid_plural_rule_exception When $force_rule has an invalid value
*/
public function get_plural_form($number, $force_rule = false)
{

View File

@ -76,7 +76,7 @@ class module_manager
*
* @return array Array of data fetched from the database
*
* @throws \phpbb\module\exception\module_not_found_exception When there is no module with $module_id
* @throws module_not_found_exception When there is no module with $module_id
*/
public function get_module_row($module_id, $module_class)
{
@ -243,7 +243,7 @@ class module_manager
*
* @param array &$module_data The module data
*
* @throws \phpbb\module\exception\module_not_found_exception When parent module or the category is not exist
* @throws module_not_found_exception When parent module or the category is not exist
*/
public function update_module_data(&$module_data)
{
@ -340,8 +340,8 @@ class module_manager
* @param int $to_parent_id ID of the target parent module
* @param string $module_class Class of the module (acp, ucp, mcp etc...)
*
* @throws \phpbb\module\exception\module_not_found_exception If the module specified to move modules from does not
* have any children.
* @throws module_not_found_exception If the module specified to move modules from does not
* have any children.
*/
public function move_module($from_module_id, $to_parent_id, $module_class)
{
@ -433,7 +433,7 @@ class module_manager
* @param int $module_id ID of the module to delete
* @param string $module_class Class of the module (acp, ucp, mcp etc...)
*
* @throws \phpbb\module\exception\module_exception When the specified module cannot be removed
* @throws module_exception When the specified module cannot be removed
*/
public function delete_module($module_id, $module_class)
{
@ -482,7 +482,7 @@ class module_manager
*
* @return string Returns the language name of the module
*
* @throws \phpbb\module\exception\module_not_found_exception When the specified module does not exists
* @throws module_not_found_exception When the specified module does not exists
*/
public function move_module_by($module_row, $module_class, $action = 'move_up', $steps = 1)
{

View File

@ -108,7 +108,7 @@ class email extends \phpbb\notification\method\messenger_base
{
$insert_buffer = new \phpbb\db\sql_insert_buffer($this->db, $this->notification_emails_table);
/** @var \phpbb\notification\type\type_interface $notification */
/** @var type_interface $notification */
foreach ($this->queue as $notification)
{
$data = self::clean_data($notification->get_insert_array());

View File

@ -97,7 +97,7 @@ abstract class messenger_base extends \phpbb\notification\method\base
$messenger = new \messenger();
// Time to go through the queue and send emails
/** @var \phpbb\notification\type\type_interface $notification */
/** @var type_interface $notification */
foreach ($this->queue as $notification)
{
if ($notification->get_email_template() === false)

View File

@ -92,7 +92,7 @@ class report
* @param int $id ID of the entity to report
* @param string $mode
* @return \Symfony\Component\HttpFoundation\Response a Symfony response object
* @throws \phpbb\exception\http_exception when $mode or $id is invalid for some reason
* @throws http_exception when $mode or $id is invalid for some reason
*/
public function handle($id, $mode)
{

View File

@ -37,7 +37,7 @@ class handler_factory
*
* @param string $type
* @return \phpbb\report\report_handler_interface
* @throws \phpbb\report\exception\factory_invalid_argument_exception if $type is not valid
* @throws factory_invalid_argument_exception if $type is not valid
*/
public function get_instance($type)
{

View File

@ -55,7 +55,7 @@ abstract class report_handler implements report_handler_interface
*
* @param \phpbb\db\driver\driver_interface $db
* @param \phpbb\event\dispatcher_interface $dispatcher
* @param \phpbb\config\db $config
* @param \phpbb\config\config $config
* @param \phpbb\auth\auth $auth
* @param \phpbb\user $user
* @param \phpbb\notification\manager $notification

View File

@ -22,7 +22,7 @@ class report_handler_pm extends report_handler
{
/**
* {@inheritdoc}
* @throws \phpbb\report\exception\pm_reporting_disabled_exception when PM reporting is disabled on the board
* @throws pm_reporting_disabled_exception when PM reporting is disabled on the board
*/
public function add_report($id, $reason_id, $report_text, $user_notify)
{
@ -92,7 +92,7 @@ class report_handler_pm extends report_handler
/**
* {@inheritdoc}
* @throws \phpbb\report\exception\pm_reporting_disabled_exception when PM reporting is disabled on the board
* @throws pm_reporting_disabled_exception when PM reporting is disabled on the board
*/
public function validate_report_request($id)
{

View File

@ -28,7 +28,7 @@ class report_handler_post extends report_handler
/**
* {@inheritdoc}
* @throws \phpbb\report\exception\report_permission_denied_exception when the user does not have permission to report the post
* @throws report_permission_denied_exception when the user does not have permission to report the post
*/
public function add_report($id, $reason_id, $report_text, $user_notify)
{
@ -89,7 +89,7 @@ class report_handler_post extends report_handler
/**
* {@inheritdoc}
* @throws \phpbb\report\exception\report_permission_denied_exception when the user does not have permission to report the post
* @throws report_permission_denied_exception when the user does not have permission to report the post
*/
public function validate_report_request($id)
{

View File

@ -129,7 +129,7 @@ class twig extends \phpbb\template\base
*
* @return array Style tree, most specific first
*
* @throws \phpbb\template\exception\user_object_not_available When user service was not set
* @throws user_object_not_available When user service was not set
*/
public function get_user_style()
{

View File

@ -25,7 +25,7 @@ class bbcode_merger
protected $configurator;
/**
* @param \phpbb\textformatter\s9e\factory $factory
* @param factory $factory
*/
public function __construct(factory $factory)
{

View File

@ -207,7 +207,7 @@ class factory implements \phpbb\textformatter\cache_interface
* Modify the s9e\TextFormatter configurator before the default settings are set
*
* @event core.text_formatter_s9e_configure_before
* @var \s9e\TextFormatter\Configurator configurator Configurator instance
* @var Configurator configurator Configurator instance
* @since 3.2.0-a1
*/
$vars = array('configurator');
@ -368,7 +368,7 @@ class factory implements \phpbb\textformatter\cache_interface
* Modify the s9e\TextFormatter configurator after the default settings are set
*
* @event core.text_formatter_s9e_configure_after
* @var \s9e\TextFormatter\Configurator configurator Configurator instance
* @var Configurator configurator Configurator instance
* @since 3.2.0-a1
*/
$vars = array('configurator');
@ -446,7 +446,7 @@ class factory implements \phpbb\textformatter\cache_interface
/**
* Configure the Autolink / Autoemail plugins used to linkify text
*
* @param \s9e\TextFormatter\Configurator $configurator
* @param Configurator $configurator
* @return void
*/
protected function configure_autolink(Configurator $configurator)