blitze / phpBB-ext-sitemaker_content

services/actions/type/pre_delete.php

Indentation +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 	 * @param \phpbb\language\language					$language			Language Object
 	 * @param \phpbb\template\template					$template			Template object
 	 * @param \blitze\content\services\types			$content_types		Content types object
-	*/
+	 */
 	public function __construct(\phpbb\language\language $language, \phpbb\template\template $template, \blitze\content\services\types $content_types)
 	{
 		$this->language = $language;

services/actions/type/toggle_status.php

Indentation +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	 * @param \blitze\content\services\types			$content_types			Content types object
 	 * @param \blitze\content\model\mapper_factory		$mapper_factory			Mapper factory object
 	 * @param boolean									$redirect				Used for testing
-	*/
+	 */
 	public function __construct(\phpbb\cache\driver\driver_interface $cache, \blitze\content\services\types $content_types, \blitze\content\model\mapper_factory $mapper_factory, $redirect = true)
 	{
 		$this->cache = $cache;

services/actions/type/index.php

Indentation +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 	 * @param string								$phpbb_root_path		Path to the phpbb includes directory.
 	 * @param string								$relative_admin_path	Relative admin root path
 	 * @param string								$php_ext				php file extension
-	*/
+	 */
 	public function __construct(\phpbb\controller\helper $controller_helper, \phpbb\language\language $language, \phpbb\template\template $template, \blitze\content\services\types $content_types, $phpbb_root_path, $relative_admin_path, $php_ext)
 	{
 		$this->controller_helper = $controller_helper;

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
 		$this->template = $template;
 		$this->content_types = $content_types;
 		$this->phpbb_root_path = $phpbb_root_path;
-		$this->phpbb_admin_path = $this->phpbb_root_path . $relative_admin_path;
+		$this->phpbb_admin_path = $this->phpbb_root_path.$relative_admin_path;
 		$this->php_ext = $php_ext;
 	}
 
@@ -76,19 +76,19 @@ 
 
 				'S_ENABLED'		=> $entity->get_content_enabled(),
 
-				'U_DELETE'		=> $u_action . '&do=pre_delete&type=' . $type,
-				'U_EDIT'		=> $u_action . '&do=edit&type=' . $type,
-				'U_STATUS'		=> $u_action . '&do=toggle_status&type=' . $type,
+				'U_DELETE'		=> $u_action.'&do=pre_delete&type='.$type,
+				'U_EDIT'		=> $u_action.'&do=edit&type='.$type,
+				'U_STATUS'		=> $u_action.'&do=toggle_status&type='.$type,
 				'U_VIEW'		=> $this->controller_helper->route('blitze_content_index', array('type' => $type)),
-				'U_POST'		=> append_sid("{$this->phpbb_root_path}ucp." . $this->php_ext, "i=-blitze-content-ucp-content_module&mode=content&action=post&type={$type}"),
-				'U_GROUP_PERMS'	=> append_sid("{$this->phpbb_admin_path}index." . $this->php_ext, "i=acp_permissions&mode=setting_group_global"),
-				'U_FORUM_PERMS'	=> append_sid("{$this->phpbb_admin_path}index." . $this->php_ext, "i=acp_permissions&mode=setting_forum_local&forum_id[]=$forum_id"),
+				'U_POST'		=> append_sid("{$this->phpbb_root_path}ucp.".$this->php_ext, "i=-blitze-content-ucp-content_module&mode=content&action=post&type={$type}"),
+				'U_GROUP_PERMS'	=> append_sid("{$this->phpbb_admin_path}index.".$this->php_ext, "i=acp_permissions&mode=setting_group_global"),
+				'U_FORUM_PERMS'	=> append_sid("{$this->phpbb_admin_path}index.".$this->php_ext, "i=acp_permissions&mode=setting_forum_local&forum_id[]=$forum_id"),
 			);
 		}
 
 		$this->template->assign_vars(array(
 			'TYPES'			=> $types_list,
-			'U_ADD_TYPE'	=> $u_action . "&do=add",
+			'U_ADD_TYPE'	=> $u_action."&do=add",
 		));
 	}
 }

services/actions/type/save.php

Indentation +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 	 * @param string									$phpbb_admin_path       Relative admin root path
 	 * @param string									$php_ext				php file extension
 	 * @param boolean									$auto_refresh			Used for testing
-	*/
+	 */
 	public function __construct(\phpbb\auth\auth $auth, \phpbb\cache\driver\driver_interface $cache, \phpbb\config\config $config, \phpbb\db\driver\driver_interface $db, \phpbb\language\language $language, \phpbb\request\request_interface $request, \blitze\content\services\types $content_types, \blitze\sitemaker\services\forum\manager $forum_manager, \blitze\content\model\mapper_factory $mapper_factory, $phpbb_admin_path, $php_ext, $auto_refresh = true)
 	{
 		$this->auth = $auth;

Spacing +4 added lines, -4 removed lines

@@ -108,7 +108,7 @@ 
 	 * @param \blitze\content\model\entity\type $unsaved_entity
 	 * @return void
 	 */
-	protected function handle_content_type($type, \blitze\content\model\entity\type &$unsaved_entity)
+	protected function handle_content_type($type, \blitze\content\model\entity\type & $unsaved_entity)
 	{
 		$forum_perm_from = $this->request->variable('copy_forum_perm', 0);
 
@@ -187,8 +187,8 @@ 
 	{
 		if (!$type)
 		{
-			$u_set_permission = append_sid("{$this->phpbb_admin_path}index.$this->php_ext", 'i=permissions&mode=setting_forum_local&forum_id[]=' . $forum_id, true);
-			$message = $this->language->lang('CONTENT_TYPE_CREATED', '<a href="' . $u_set_permission . '">', '</a>');
+			$u_set_permission = append_sid("{$this->phpbb_admin_path}index.$this->php_ext", 'i=permissions&mode=setting_forum_local&forum_id[]='.$forum_id, true);
+			$message = $this->language->lang('CONTENT_TYPE_CREATED', '<a href="'.$u_set_permission.'">', '</a>');
 		}
 		else
 		{
@@ -237,7 +237,7 @@ 
 		if ($old_langname !== $new_langname)
 		{
 			$forum_name = $this->language->lang($new_langname);
-			$sql = 'UPDATE ' . FORUMS_TABLE . " SET forum_name = '" . $this->db->sql_escape($forum_name) . "' WHERE forum_id = " . (int) $forum_id;
+			$sql = 'UPDATE '.FORUMS_TABLE." SET forum_name = '".$this->db->sql_escape($forum_name)."' WHERE forum_id = ".(int) $forum_id;
 			$this->db->sql_query($sql);
 		}
 	}

Braces +2 added lines, -4 removed lines

@@ -121,8 +121,7 @@ 
 			$unsaved_entity->set_content_id($entity->get_content_id());
 			$this->handle_langname_change($forum_id, $entity->get_content_langname(), $unsaved_entity->get_content_langname());
 			$this->copy_forum_permissions($forum_id, $forum_perm_from);
-		}
-		else
+		} else
 		{
 			$forum_id = $this->create_content_forum($unsaved_entity->get_content_langname(), $forum_perm_from);
 			$unsaved_entity->set_forum_id($forum_id);
@@ -189,8 +188,7 @@ 
 		{
 			$u_set_permission = append_sid("{$this->phpbb_admin_path}index.$this->php_ext", 'i=permissions&mode=setting_forum_local&forum_id[]=' . $forum_id, true);
 			$message = $this->language->lang('CONTENT_TYPE_CREATED', '<a href="' . $u_set_permission . '">', '</a>');
-		}
-		else
+		} else
 		{
 			$this->meta_refresh(3, $u_action);
 			$message = $this->language->lang('CONTENT_TYPE_UPDATED');

services/actions/type/load_field.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 	 * @param \phpbb\request\request_interface				$request			Request object
 	 * @param \phpbb\template\template						$template			Template object
 	 * @param \blitze\content\services\form\fields_factory	$fields_factory		Fields factory  object
-	 * @param string										$admin_path			Admin path
+	 * @param \blitze\content\services\form\fields_factory										$admin_path			Admin path
 	 */
 	public function __construct(\phpbb\request\request_interface $request, \phpbb\template\template $template, \blitze\content\services\form\fields_factory $fields_factory, $admin_path)
 	{

services/form/field/range.php

Doc Comments +1 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
 	}
 
 	/**
-	 * @param string $values
+	 * @param string $value
 	 * @return array
 	 */
 	protected function get_range($value)
@@ -122,7 +122,6 @@ 
 	}
 
 	/**
-	 * @param array $data
 	 * @return array
 	 */
 	protected function set_prepost(&$item, $key, array $props)

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
 			),
 			'css'   => array(
 				'@blitze_content/vendor/ion.rangeSlider/css/ion.rangeSlider.min.css',
-				'@blitze_content/vendor/ion.rangeSlider/css/ion.rangeSlider.' . $data['field_props']['theme'] . '.min.css',
+				'@blitze_content/vendor/ion.rangeSlider/css/ion.rangeSlider.'.$data['field_props']['theme'].'.min.css',
 			)
 		));
 
@@ -127,6 +127,6 @@ 
 	 */
 	protected function set_prepost(&$item, $key, array $props)
 	{
-		$item = $props['prefix'] . $item . $props['postfix'];
+		$item = $props['prefix'].$item.$props['postfix'];
 	}
 }

services/form/field/textarea.php

Doc Comments +4 added lines, -5 removed lines

@@ -96,8 +96,6 @@ 
 	/**
 	 * Display content field
 	 *
-	 * @param string $field_value
-	 * @param array $field_data
 	 * @param string $view_mode
 	 * @param array $topic_data
 	 * @return mixed
@@ -172,7 +170,7 @@ 
 	 * @param string $value
 	 * @param string $topic_url
 	 * @param string $view_mode
-	 * @return void
+	 * @return string
 	 */
 	protected function generate_field_pages($field_name, $value, $topic_url, $view_mode)
 	{
@@ -202,8 +200,8 @@ 
 	/**
 	 * Generate pagination for topic subpages
 	 *
-	 * @param array $pages
-	 * @param array $page_titles
+	 * @param string[] $pages
+	 * @param string[] $page_titles
 	 * @param int $start
 	 * @param string $topic_url
 	 * @param string $view_mode
@@ -225,6 +223,7 @@ 
 	 *
 	 * @param int $start
 	 * @param $page_titles
+	 * @param string $topic_url
 	 * @return void
 	 */
 	protected function generate_toc($start, $page_titles, $topic_url)

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
 			$asset_path = $this->util->get_web_path();
 			$this->util->add_assets(array(
 				'js'   => array(
-					$asset_path . 'assets/javascript/editor.js',
+					$asset_path.'assets/javascript/editor.js',
 					'@blitze_content/assets/form/textarea.min.js'
 				)
 			));
@@ -145,12 +145,12 @@ 
 		// Assigning custom bbcodes
 		if (!function_exists('display_custom_bbcodes'))
 		{
-			include($this->phpbb_root_path . 'includes/functions_display.' . $this->php_ext);
+			include($this->phpbb_root_path.'includes/functions_display.'.$this->php_ext);
 		}
 
 		display_custom_bbcodes();
 
-		$bbcode_status	= ($this->config['allow_bbcode'] && $this->auth->acl_get('f_bbcode', $forum_id)) ? true : false;
+		$bbcode_status = ($this->config['allow_bbcode'] && $this->auth->acl_get('f_bbcode', $forum_id)) ? true : false;
 
 		$dataref = $this->template_context->get_data_ref();
 		$this->ptemplate->assign_block_vars_array('custom_tags', (isset($dataref['custom_tags'])) ? $dataref['custom_tags'] : array());
@@ -235,7 +235,7 @@ 
 			$this->template->assign_block_vars('toc', array(
 				'TITLE'		=> $title,
 				'S_PAGE'	=> ($page === $start),
-				'U_VIEW'	=> append_sid($topic_url, 'page=' . $page),
+				'U_VIEW'	=> append_sid($topic_url, 'page='.$page),
 			));
 		}
 	}

controller/field_controller.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 	 */
 	protected function set_prop($prop, array &$data)
 	{
-		$field_prop = $this->request->variable('field_' . $prop, array('' => array(0 => '')), true);
+		$field_prop = $this->request->variable('field_'.$prop, array('' => array(0 => '')), true);
 
 		if (null !== ($array = array_pop($field_prop)))
 		{

model/entity/field.php

Braces +2 added lines, -4 removed lines

@@ -168,8 +168,7 @@ 
 		{
 			$data = generate_text_for_edit($this->field_explain, $this->field_exp_uid, $this->field_exp_options);
 			return $data['text'];
-		}
-		else
+		} else
 		{
 			$parse_flags = ($this->field_exp_bitfield ? OPTION_FLAG_BBCODE : 0) | OPTION_FLAG_SMILIES;
 			return generate_text_for_display($this->field_explain, $this->field_exp_uid, $this->field_exp_bitfield, $parse_flags);
@@ -186,8 +185,7 @@ 
 		if (!is_array($props))
 		{
 			$this->field_props = $props;
-		}
-		else if (sizeof($props))
+		} else if (sizeof($props))
 		{
 			$this->field_props = json_encode($props);
 		}