phpbb-extensions / ad-management

location/type/above_header.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
 	/**
 	* Construct an above_header template location object
 	*
-	* @param	\phpbb\user	$config	User object
 	*/
 	public function __construct(\phpbb\user $user)
 	{

Indentation +13 added lines, -13 removed lines

@@ -13,40 +13,40 @@
 class above_header extends base
 {
 	/**
-	* User object
-	* @var \phpbb\user
-	*/
+	 * User object
+	 * @var \phpbb\user
+	 */
 	protected $user;
 
 	/**
-	* Construct an above_header template location object
-	*
-	* @param	\phpbb\user	$config	User object
-	*/
+	 * Construct an above_header template location object
+	 *
+	 * @param	\phpbb\user	$config	User object
+	 */
 	public function __construct(\phpbb\user $user)
 	{
 		$this->user = $user;
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_id()
 	{
 		return 'above_header';
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_name()
 	{
 		return $this->user->lang('AD_ABOVE_HEADER');
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_desc()
 	{
 		return $this->user->lang('AD_ABOVE_HEADER_DESC');

location/type/below_header.php

Doc Comments -1 removed lines

@@ -21,7 +21,6 @@
 	/**
 	* Construct an above_header template location object
 	*
-	* @param	\phpbb\user	$config	User object
 	*/
 	public function __construct(\phpbb\user $user)
 	{

Indentation +13 added lines, -13 removed lines

@@ -13,40 +13,40 @@
 class below_header extends base
 {
 	/**
-	* User object
-	* @var \phpbb\user
-	*/
+	 * User object
+	 * @var \phpbb\user
+	 */
 	protected $user;
 
 	/**
-	* Construct an above_header template location object
-	*
-	* @param	\phpbb\user	$config	User object
-	*/
+	 * Construct an above_header template location object
+	 *
+	 * @param	\phpbb\user	$config	User object
+	 */
 	public function __construct(\phpbb\user $user)
 	{
 		$this->user = $user;
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_id()
 	{
 		return 'below_header';
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_name()
 	{
 		return $this->user->lang('AD_BELOW_HEADER');
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function get_desc()
 	{
 		return $this->user->lang('AD_BELOW_HEADER_DESC');

location/type/type_interface.php

Indentation +20 added lines, -20 removed lines

@@ -16,34 +16,34 @@
 interface type_interface
 {
 	/**
-	* Returns the unique ID of the location.
-	*
-	* @return string	ID of location.
-	*/
+	 * Returns the unique ID of the location.
+	 *
+	 * @return string	ID of location.
+	 */
 	public function get_id();
 
 	/**
-	* Returns the name of the location.
-	*
-	* @return string	Name of location.
-	*/
+	 * Returns the name of the location.
+	 *
+	 * @return string	Name of location.
+	 */
 	public function get_name();
 
 	/**
-	* Returns the description of the location.
-	*
-	* @return string	Description of location.
-	*/
+	 * Returns the description of the location.
+	 *
+	 * @return string	Description of location.
+	 */
 	public function get_desc();
 
 	/**
-	* Returns whether or not this location type will be displayed on a current page.
-	*
-	* Generally, you can always return true, but if you can narrow down the usage
-	* without adding extra load to server, this will further enhance the extension's
-	* performance.
-	*
-	* @return bool	True when location type will be displayed on a current page and false if not.
-	*/
+	 * Returns whether or not this location type will be displayed on a current page.
+	 *
+	 * Generally, you can always return true, but if you can narrow down the usage
+	 * without adding extra load to server, this will further enhance the extension's
+	 * performance.
+	 *
+	 * @return bool	True when location type will be displayed on a current page and false if not.
+	 */
 	public function will_display();
 }

location/type/base.php

Indentation +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
 abstract class base implements \phpbb\admanagement\location\type\type_interface
 {
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function will_display()
 	{
 		return true;

location/manager.php

Indentation +23 added lines, -23 removed lines

@@ -13,30 +13,30 @@ 
 class manager
 {
 	/**
-	* Array that contains all available template location types which are passed
-	* via the service container
-	* @var array
-	*/
+	 * Array that contains all available template location types which are passed
+	 * via the service container
+	 * @var array
+	 */
 	protected $template_locations;
 
 	/**
-	* Construct an template locations manager object
-	*
-	* @param	array	$template_locations	Template location types passed via the service container
-	*/
+	 * Construct an template locations manager object
+	 *
+	 * @param	array	$template_locations	Template location types passed via the service container
+	 */
 	public function __construct($template_locations)
 	{
 		$this->register_template_locations($template_locations);
 	}
 
 	/**
-	* Get a list of all template location types
-	*
-	* Returns an associated array where key is the location id
-	* and value is array of location name and location description.
-	*
-	* @return	array	Array containing a list of all avatar drivers
-	*/
+	 * Get a list of all template location types
+	 *
+	 * Returns an associated array where key is the location id
+	 * and value is array of location name and location description.
+	 *
+	 * @return	array	Array containing a list of all avatar drivers
+	 */
 	public function get_all_locations()
 	{
 		$location_types = array();
@@ -53,20 +53,20 @@ 
 	}
 
 	/**
-	* Get a list of all template location IDs
-	*
-	* @return	array	Array containing a list of all template location IDs
-	*/
+	 * Get a list of all template location IDs
+	 *
+	 * @return	array	Array containing a list of all template location IDs
+	 */
 	public function get_all_location_ids()
 	{
 		return array_keys($this->template_locations);
 	}
 
 	/**
-	* Register template locations
-	*
-	* @param	array	$template_locations	Template location types passed via the service container
-	*/
+	 * Register template locations
+	 *
+	 * @param	array	$template_locations	Template location types passed via the service container
+	 */
 	protected function register_template_locations($template_locations)
 	{
 		if (!empty($template_locations))

migrations/v10x/m3_template_locations_schema.php

Indentation +18 added lines, -18 removed lines

@@ -13,31 +13,31 @@ 
 class m3_template_locations_schema extends \phpbb\db\migration\migration
 {
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	public function effectively_installed()
 	{
 		return $this->db_tools->sql_table_exists($this->table_prefix . 'ad_locations');
 	}
 
 	/**
-	* {@inheritDoc}
-	*/
+	 * {@inheritDoc}
+	 */
 	static public function depends_on()
 	{
 		return array('\phpbb\admanagement\migrations\v10x\m2_acp_module');
 	}
 
 	/**
-	* Add the ad_locations table schema to the database:
-	*	ad_locations:
-	*		ad_location_id
-	*		ad_id
-	*		location_id
-	*
-	* @return array Array of table schema
-	* @access public
-	*/
+	 * Add the ad_locations table schema to the database:
+	 *	ad_locations:
+	 *		ad_location_id
+	 *		ad_id
+	 *		location_id
+	 *
+	 * @return array Array of table schema
+	 * @access public
+	 */
 	public function update_schema()
 	{
 		return array(
@@ -54,11 +54,11 @@ 
 	}
 
 	/**
-	* Drop the ad_locations table schema from the database
-	*
-	* @return array Array of table schema
-	* @access public
-	*/
+	 * Drop the ad_locations table schema from the database
+	 *
+	 * @return array Array of table schema
+	 * @access public
+	 */
 	public function revert_schema()
 	{
 		return array(

controller/admin_controller.php

Indentation +86 added lines, -86 removed lines

@@ -51,18 +51,18 @@ 
 	protected $errors = array();
 
 	/**
-	* Constructor
-	*
-	* @param \phpbb\db\driver\driver_interface		$db					DB driver interface
-	* @param \phpbb\template\template				$template			Template object
-	* @param \phpbb\user							$user				User object
-	* @param \phpbb\request\request					$request			Request object
-	* @param \phpbb\admanagement\location\manager	$location_manager	Template location manager object
-	* @param string									$ads_table			Ads table
-	* @param string									$ad_locations_table	Ad locations table
-	* @param string									$php_ext			PHP extension
-	* @param string									$ext_path			Path to this extension
-	*/
+	 * Constructor
+	 *
+	 * @param \phpbb\db\driver\driver_interface		$db					DB driver interface
+	 * @param \phpbb\template\template				$template			Template object
+	 * @param \phpbb\user							$user				User object
+	 * @param \phpbb\request\request					$request			Request object
+	 * @param \phpbb\admanagement\location\manager	$location_manager	Template location manager object
+	 * @param string									$ads_table			Ads table
+	 * @param string									$ad_locations_table	Ad locations table
+	 * @param string									$php_ext			PHP extension
+	 * @param string									$ext_path			Path to this extension
+	 */
 	public function __construct(\phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\user $user, \phpbb\request\request $request, \phpbb\admanagement\location\manager $location_manager, $ads_table, $ad_locations_table, $php_ext, $ext_path)
 	{
 		$this->db = $db;
@@ -77,10 +77,10 @@ 
 	}
 
 	/**
-	* Process user request
-	*
-	* @return void
-	*/
+	 * Process user request
+	 *
+	 * @return void
+	 */
 	public function main()
 	{
 		$this->user->add_lang_ext('phpbb/admanagement', 'acp');
@@ -97,31 +97,31 @@ 
 	}
 
 	/**
-	* Set page url
-	*
-	* @param string $u_action Custom form action
-	* @return void
-	*/
+	 * Set page url
+	 *
+	 * @param string $u_action Custom form action
+	 * @return void
+	 */
 	public function set_page_url($u_action)
 	{
 		$this->u_action = $u_action;
 	}
 
 	/**
-	* Get ACP page title for Ads module
-	*
-	* @return string	Language string for Ads ACP module
-	*/
+	 * Get ACP page title for Ads module
+	 *
+	 * @return string	Language string for Ads ACP module
+	 */
 	public function get_page_title()
 	{
 		return $this->user->lang('ACP_ADMANAGEMENT_TITLE');
 	}
 
 	/**
-	* Add an advertisement
-	*
-	* @return void
-	*/
+	 * Add an advertisement
+	 *
+	 * @return void
+	 */
 	public function action_add()
 	{
 		add_form_key('phpbb/admanagement/add');
@@ -173,10 +173,10 @@ 
 	}
 
 	/**
-	* Edit an advertisement
-	*
-	* @return void
-	*/
+	 * Edit an advertisement
+	 *
+	 * @return void
+	 */
 	public function action_edit()
 	{
 		$ad_id = $this->request->variable('id', 0);
@@ -265,30 +265,30 @@ 
 	}
 
 	/**
-	* Enable an advertisement
-	*
-	* @return void
-	*/
+	 * Enable an advertisement
+	 *
+	 * @return void
+	 */
 	public function action_enable()
 	{
 		$this->ad_enable(true);
 	}
 
 	/**
-	* Disable an advertisement
-	*
-	* @return void
-	*/
+	 * Disable an advertisement
+	 *
+	 * @return void
+	 */
 	public function action_disable()
 	{
 		$this->ad_enable(false);
 	}
 
 	/**
-	* Delete an advertisement
-	*
-	* @return void
-	*/
+	 * Delete an advertisement
+	 *
+	 * @return void
+	 */
 	public function action_delete()
 	{
 		$ad_id = $this->request->variable('id', 0);
@@ -329,10 +329,10 @@ 
 	}
 
 	/**
-	* Display the ads
-	*
-	* @return void
-	*/
+	 * Display the ads
+	 *
+	 * @return void
+	 */
 	public function list_ads()
 	{
 		$sql = 'SELECT ad_id, ad_name, ad_enabled
@@ -361,11 +361,11 @@ 
 	}
 
 	/**
-	* Enable/disable an advertisement
-	*
-	* @param	bool	$enable	Enable or disable the advertisement?
-	* @return void
-	*/
+	 * Enable/disable an advertisement
+	 *
+	 * @param	bool	$enable	Enable or disable the advertisement?
+	 * @return void
+	 */
 	protected function ad_enable($enable)
 	{
 		$ad_id = $this->request->variable('id', 0);
@@ -398,11 +398,11 @@ 
 	}
 
 	/**
-	* Check the form key.
-	*
-	* @param	string	$form_name	The name of the form.
-	* @return void
-	*/
+	 * Check the form key.
+	 *
+	 * @param	string	$form_name	The name of the form.
+	 * @return void
+	 */
 	protected function check_form_key($form_name)
 	{
 		if (!check_form_key($form_name))
@@ -412,10 +412,10 @@ 
 	}
 
 	/**
-	* Get admin form data.
-	*
-	* @return	array	Form data
-	*/
+	 * Get admin form data.
+	 *
+	 * @return	array	Form data
+	 */
 	protected function get_form_data()
 	{
 		return array(
@@ -428,11 +428,11 @@ 
 	}
 
 	/**
-	* Validate form data.
-	*
-	* @param	array	$data	The form data.
-	* @return void
-	*/
+	 * Validate form data.
+	 *
+	 * @param	array	$data	The form data.
+	 * @return void
+	 */
 	protected function validate($data)
 	{
 		if ($data['ad_name'] === '')
@@ -446,11 +446,11 @@ 
 	}
 
 	/**
-	* Assign form data to the template.
-	*
-	* @param	array	$data	The form data.
-	* @return void
-	*/
+	 * Assign form data to the template.
+	 *
+	 * @param	array	$data	The form data.
+	 * @return void
+	 */
 	protected function assign_form_data($data)
 	{
 		$this->template->assign_vars(array(
@@ -465,11 +465,11 @@ 
 	}
 
 	/**
-	* Assign template locations data to the template.
-	*
-	* @param	mixed	$data	The form data or nothing.
-	* @return	void
-	*/
+	 * Assign template locations data to the template.
+	 *
+	 * @param	mixed	$data	The form data or nothing.
+	 * @return	void
+	 */
 	protected function assign_locations($data = false)
 	{
 		foreach ($this->location_manager->get_all_locations() as $location_id => $location_data)
@@ -484,20 +484,20 @@ 
 	}
 
 	/**
-	* Print success message.
-	*
-	* It takes arguments in the form of a language key, followed by language substitution values.
-	*/
+	 * Print success message.
+	 *
+	 * It takes arguments in the form of a language key, followed by language substitution values.
+	 */
 	protected function success()
 	{
 		trigger_error(call_user_func_array(array($this->user, 'lang'), func_get_args()) . adm_back_link($this->u_action));
 	}
 
 	/**
-	* Print error message.
-	*
-	* It takes arguments in the form of a language key, followed by language substitution values.
-	*/
+	 * Print error message.
+	 *
+	 * It takes arguments in the form of a language key, followed by language substitution values.
+	 */
 	protected function error()
 	{
 		trigger_error(call_user_func_array(array($this->user, 'lang'), func_get_args()) . adm_back_link($this->u_action), E_USER_WARNING);

event/main_listener.php

Indentation +11 added lines, -11 removed lines

@@ -39,8 +39,8 @@ 
 	protected $ad_locations_table;
 
 	/**
-	* {@inheritdoc}
-	*/
+	 * {@inheritdoc}
+	 */
 	static public function getSubscribedEvents()
 	{
 		return array(
@@ -49,15 +49,15 @@ 
 	}
 
 	/**
-	* Constructor
-	*
-	* @param \phpbb\request\request					$request			Request object
-	* @param \phpbb\db\driver\driver_interface		$db					DB driver interface
-	* @param \phpbb\template\template				$template			Template object
-	* @param \phpbb\admanagement\location\manager	$location_manager	Template location manager object
-	* @param string									$ads_table			Ads table
-	* @param string									$ad_locations_table	Ad locations table
-	*/
+	 * Constructor
+	 *
+	 * @param \phpbb\request\request					$request			Request object
+	 * @param \phpbb\db\driver\driver_interface		$db					DB driver interface
+	 * @param \phpbb\template\template				$template			Template object
+	 * @param \phpbb\admanagement\location\manager	$location_manager	Template location manager object
+	 * @param string									$ads_table			Ads table
+	 * @param string									$ad_locations_table	Ad locations table
+	 */
 	public function __construct(\phpbb\request\request $request, \phpbb\db\driver\driver_interface $db, \phpbb\template\template $template, \phpbb\admanagement\location\manager $location_manager, $ads_table, $ad_locations_table)
 	{
 		$this->request = $request;