Completed
Pull Request — master (#26)
by Daniel
16:40 queued 13:36
created
services/blocks/cfg_handler.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 
63 63
 		if (!function_exists('build_cfg_template'))
64 64
 		{
65
-			include($this->phpbb_root_path . 'includes/functions_acp.' . $this->php_ext); // @codeCoverageIgnore
65
+			include($this->phpbb_root_path.'includes/functions_acp.'.$this->php_ext); // @codeCoverageIgnore
66 66
 		}
67 67
 
68 68
 		// We fake this class as it is needed by the build_cfg_template function
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 		// @codeCoverageIgnoreStart
84 84
 		if (!function_exists('validate_config_vars'))
85 85
 		{
86
-			include($this->phpbb_root_path . 'includes/functions_acp.' . $this->php_ext);
86
+			include($this->phpbb_root_path.'includes/functions_acp.'.$this->php_ext);
87 87
 		}
88 88
 		// @codeCoverageIgnoreEnd
89 89
 
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 		$vars['append'] = $this->append_field($vars);
172 172
 
173 173
 		$type = explode(':', $vars['type']);
174
-		$method = 'prep_' . $type[0] . '_field_for_display';
174
+		$method = 'prep_'.$type[0].'_field_for_display';
175 175
 
176 176
 		if (is_callable(array($this, $method)))
177 177
 		{
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
 		$l_explain = '';
216 216
 		if (!empty($vars['explain']))
217 217
 		{
218
-			$l_explain = (isset($vars['lang_explain'])) ? $this->translator->lang($vars['lang_explain']) : $this->translator->lang($vars['lang'] . '_EXPLAIN');
218
+			$l_explain = (isset($vars['lang_explain'])) ? $this->translator->lang($vars['lang_explain']) : $this->translator->lang($vars['lang'].'_EXPLAIN');
219 219
 		}
220 220
 
221 221
 		return $l_explain;
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
 	{
316 316
 		$this->prep_checkbox_field_for_display($vars, $type, $field);
317 317
 
318
-		$vars['method'] ='build_multi_select';
318
+		$vars['method'] = 'build_multi_select';
319 319
 	}
320 320
 
321 321
 	/**
Please login to merge, or discard this patch.
services/tree/builder.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
 	 */
56 56
 	public function get_item_info($item_id = 0)
57 57
 	{
58
-		$sql = 'SELECT * FROM ' . $this->table_name . ' ' . $this->get_sql_where('WHERE') .
59
-			(($item_id) ? " AND {$this->column_item_id} = " . (int) $item_id : '');
58
+		$sql = 'SELECT * FROM '.$this->table_name.' '.$this->get_sql_where('WHERE').
59
+			(($item_id) ? " AND {$this->column_item_id} = ".(int) $item_id : '');
60 60
 
61 61
 		$result = $this->db->sql_query($sql);
62 62
 		$row = $this->db->sql_fetchrow($result);
@@ -75,8 +75,8 @@  discard block
 block discarded – undo
75 75
 	public function update_item($item_id, array $sql_data)
76 76
 	{
77 77
 		$sql = "UPDATE {$this->table_name}
78
-			SET " . $this->db->sql_build_array('UPDATE', $sql_data) . "
79
-			WHERE $this->column_item_id = " . (int) $item_id;
78
+			SET ".$this->db->sql_build_array('UPDATE', $sql_data)."
79
+			WHERE $this->column_item_id = ".(int) $item_id;
80 80
 		$this->db->sql_query($sql);
81 81
 
82 82
 		return $sql_data;
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 
110 110
 		// Rather than updating each item individually, we will just delete all items
111 111
 		// then add them all over again with new parent_id|left_id|right_id
112
-		$this->db->sql_query("DELETE FROM {$this->table_name} " . $this->get_sql_where('WHERE'));
112
+		$this->db->sql_query("DELETE FROM {$this->table_name} ".$this->get_sql_where('WHERE'));
113 113
 
114 114
 		// Now we add it back
115 115
 		$sql_data = $this->add_sub_tree($tree, 0);
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
 
159 159
 			if ($depth && !$parent_id)
160 160
 			{
161
-				throw new \RuntimeException($this->message_prefix . 'MALFORMED_TREE');
161
+				throw new \RuntimeException($this->message_prefix.'MALFORMED_TREE');
162 162
 			}
163 163
 
164 164
 			$key = $i + $max_id + 1;
@@ -230,23 +230,23 @@  discard block
 block discarded – undo
230 230
 		foreach ($branch as $i => $row)
231 231
 		{
232 232
 			$left_id	= $right_id + 1;
233
-			$right_id   = $right_id + 2;
233
+			$right_id = $right_id + 2;
234 234
 
235 235
 			$sql_data[$i] = $row;
236
-			$sql_data[$i]['parent_id']	= $parent_id;
237
-			$sql_data[$i]['left_id']	= $left_id;
238
-			$sql_data[$i]['right_id']	= $right_id;
239
-			$sql_data[$i]['depth']		= $depth;
236
+			$sql_data[$i]['parent_id'] = $parent_id;
237
+			$sql_data[$i]['left_id'] = $left_id;
238
+			$sql_data[$i]['right_id'] = $right_id;
239
+			$sql_data[$i]['depth'] = $depth;
240 240
 
241 241
 			if ($row['parent_id'])
242 242
 			{
243
-				$left_id	= $sql_data[$row['parent_id']]['right_id'];
244
-				$right_id   = $left_id + 1;
243
+				$left_id = $sql_data[$row['parent_id']]['right_id'];
244
+				$right_id = $left_id + 1;
245 245
 
246
-				$sql_data[$i]['parent_id']	= $row['parent_id'];
247
-				$sql_data[$i]['depth']		= $sql_data[$row['parent_id']]['depth'] + 1;
248
-				$sql_data[$i]['left_id']	= $left_id;
249
-				$sql_data[$i]['right_id']	= $right_id;
246
+				$sql_data[$i]['parent_id'] = $row['parent_id'];
247
+				$sql_data[$i]['depth'] = $sql_data[$row['parent_id']]['depth'] + 1;
248
+				$sql_data[$i]['left_id'] = $left_id;
249
+				$sql_data[$i]['right_id'] = $right_id;
250 250
 
251 251
 				$this->update_right_side($sql_data, $right_id, $row['parent_id'], $branch);
252 252
 			}
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 				$this->db->sql_transaction('rollback');
273 273
 				$this->lock->release();
274 274
 
275
-				throw new \RuntimeException($this->message_prefix . 'INVALID_PARENT');
275
+				throw new \RuntimeException($this->message_prefix.'INVALID_PARENT');
276 276
 			}
277 277
 
278 278
 			// adjust items in affected branch
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 	protected function get_max_id($column, $use_sql_where = true)
301 301
 	{
302 302
 		$sql = "SELECT MAX($column) AS $column
303
-			FROM {$this->table_name} " .
303
+			FROM {$this->table_name} ".
304 304
 			(($use_sql_where) ? $this->get_sql_where('WHERE') : '');
305 305
 		$result = $this->db->sql_query($sql);
306 306
 		$max_id = $this->db->sql_fetchfield($column);
Please login to merge, or discard this patch.
services/tree/display.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 	{
79 79
 		$sql = "SELECT *
80 80
 			FROM $this->items_table
81
-			WHERE $this->pk = " . (int) $node_id ;
81
+			WHERE $this->pk = ".(int) $node_id;
82 82
 		$result = $this->db->sql_query($sql);
83 83
 		$row = $this->db->sql_fetchrow($result);
84 84
 		$this->db->sql_freeresult($result);
@@ -107,8 +107,8 @@  discard block
 block discarded – undo
107 107
 					$this->items_table => ' t'
108 108
 				),
109 109
 				'WHERE'		=> array(
110
-					't.depth ' . (($level) ? " BETWEEN $start AND " . ($start + $level) : ' >= ' . $start),
111
-					(($this->sql_where) ? 't.' . $this->sql_where : ''),
110
+					't.depth '.(($level) ? " BETWEEN $start AND ".($start + $level) : ' >= '.$start),
111
+					(($this->sql_where) ? 't.'.$this->sql_where : ''),
112 112
 				),
113 113
 				'ORDER_BY'	=> 't.left_id ASC',
114 114
 			),
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 	 * @param \phpbb\template\twig\twig $template
148 148
 	 * @param string $handle
149 149
 	 */
150
-	public function display_list(array $data, \phpbb\template\twig\twig &$template, $handle = 'tree')
150
+	public function display_list(array $data, \phpbb\template\twig\twig & $template, $handle = 'tree')
151 151
 	{
152 152
 		$prev_depth = 0;
153 153
 		$parental_depth = array(0 => -1);
@@ -156,10 +156,10 @@  discard block
 block discarded – undo
156 156
 		for ($i = 0, $size = sizeof($data); $i < $size; $i++)
157 157
 		{
158 158
 			$row 		= $data[$i];
159
-			$this_depth	= $parental_depth[$row['parent_id']] + 1;
160
-			$repeat		= abs($prev_depth - $this_depth);
159
+			$this_depth = $parental_depth[$row['parent_id']] + 1;
160
+			$repeat = abs($prev_depth - $this_depth);
161 161
 
162
-			$tpl_data	= array(
162
+			$tpl_data = array(
163 163
 				'PREV_DEPTH'	=> $prev_depth,
164 164
 				'THIS_DEPTH'	=> $this_depth,
165 165
 				'NUM_KIDS'		=> $this->count_descendants($row),
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 
170 170
 			for ($j = 0; $j < $repeat; $j++)
171 171
 			{
172
-				$template->assign_block_vars($handle . '.close', array());
172
+				$template->assign_block_vars($handle.'.close', array());
173 173
 			}
174 174
 
175 175
 			$prev_depth = $this_depth;
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
 
179 179
 		for ($i = 0; $i < $prev_depth; $i++)
180 180
 		{
181
-			$template->assign_block_vars('close_' . $handle, array());
181
+			$template->assign_block_vars('close_'.$handle, array());
182 182
 		}
183 183
 	}
184 184
 
@@ -217,10 +217,10 @@  discard block
 block discarded – undo
217 217
 			}
218 218
 
219 219
 			$right = $row['right_id'];
220
-			$title = $padding . '&#x251c;&#x2500; ' . $row[$title_column];
220
+			$title = $padding.'&#x251c;&#x2500; '.$row[$title_column];
221 221
 			$selected = (in_array($row[$this->pk], $selected_ids)) ? ' selected="selected' : '';
222 222
 
223
-			$return_options .= '<option value="' . $row[$this->pk] . '"' . $selected . '>' . $title . '</option>';
223
+			$return_options .= '<option value="'.$row[$this->pk].'"'.$selected.'>'.$title.'</option>';
224 224
 			$return_data[$row[$this->pk]] = $title;
225 225
 		}
226 226
 
Please login to merge, or discard this patch.
services/date_range.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
 		$time = $this->user->create_datetime($this->time);
39 39
 		$now = phpbb_gmgetdate($time->getTimestamp() + $time->getOffset());
40 40
 
41
-		$method = 'get_' . $range;
41
+		$method = 'get_'.$range;
42 42
 		$data = array(
43 43
 			'start'	=> 0,
44 44
 			'stop'	=> 0,
Please login to merge, or discard this patch.
services/auto_lang.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -64,8 +64,8 @@  discard block
 block discarded – undo
64 64
 			$default_lang_files = $finder
65 65
 				->prefix($lang_file)
66 66
 				->suffix(".$this->php_ext")
67
-				->extension_directory('/language/' . basename($this->config['default_lang']))
68
-				->core_path('language/' . basename($this->config['default_lang']) . '/')
67
+				->extension_directory('/language/'.basename($this->config['default_lang']))
68
+				->core_path('language/'.basename($this->config['default_lang']).'/')
69 69
 				->find();
70 70
 		}
71 71
 
@@ -84,8 +84,8 @@  discard block
 block discarded – undo
84 84
 		$user_lang_files = $finder
85 85
 			->prefix($lang_file)
86 86
 			->suffix(".$this->php_ext")
87
-			->extension_directory('/language/' . $this->user->lang_name)
88
-			->core_path('language/' . $this->user->lang_name . '/')
87
+			->extension_directory('/language/'.$this->user->lang_name)
88
+			->core_path('language/'.$this->user->lang_name.'/')
89 89
 			->find();
90 90
 
91 91
 		$lang_files = array_unique(array_merge($user_lang_files, $english_lang_files, $default_lang_files));
Please login to merge, or discard this patch.
services/menus/action/add_menu.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
 
21 21
 		/** @type \blitze\sitemaker\model\menus\entity\menu $entity */
22 22
 		$entity = $menu_mapper->create_entity(array(
23
-			'menu_name' => $this->translator->lang('MENU') . '-' . mt_rand(1000, 9999),
23
+			'menu_name' => $this->translator->lang('MENU').'-'.mt_rand(1000, 9999),
24 24
 		));
25 25
 
26 26
 		$entity = $menu_mapper->save($entity);
Please login to merge, or discard this patch.
services/menus/action/update_item.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@
 block discarded – undo
34 34
 
35 35
 		if (isset($allowed_fields[$field]))
36 36
 		{
37
-			$mutator = 'set_' . $field;
37
+			$mutator = 'set_'.$field;
38 38
 			$entity->$mutator($allowed_fields[$field]);
39 39
 			$item_mapper->save($entity);
40 40
 		}
Please login to merge, or discard this patch.
services/menus/action_handler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
 	 */
47 47
 	public function create($action)
48 48
 	{
49
-		$action_class = 'blitze\\sitemaker\\services\\menus\\action\\' . $action;
49
+		$action_class = 'blitze\\sitemaker\\services\\menus\\action\\'.$action;
50 50
 
51 51
 		if (!class_exists($action_class))
52 52
 		{
Please login to merge, or discard this patch.
services/menus/display.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	 * @param string $handle
66 66
 	 * @return void
67 67
 	 */
68
-	public function display_navlist(array $data, \phpbb\template\twig\twig &$template, $handle = 'tree')
68
+	public function display_navlist(array $data, \phpbb\template\twig\twig & $template, $handle = 'tree')
69 69
 	{
70 70
 		$this->prepare_items($data);
71 71
 
@@ -79,10 +79,10 @@  discard block
 block discarded – undo
79 79
 				$row['num_kids'] = $this->count_descendants($row);
80 80
 
81 81
 				$template->assign_block_vars($handle, array_change_key_case($row, CASE_UPPER));
82
-				$this->close_open_tags($template, $handle . '.close', abs($prev_depth - $this_depth));
82
+				$this->close_open_tags($template, $handle.'.close', abs($prev_depth - $this_depth));
83 83
 			}
84 84
 
85
-			$this->close_open_tags($template, 'close_' . $handle, ($this_depth - $this->min_depth));
85
+			$this->close_open_tags($template, 'close_'.$handle, ($this_depth - $this->min_depth));
86 86
 		}
87 87
 	}
88 88
 
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 			}
122 122
 
123 123
 			$is_current_item = $this->is_current_item($row);
124
-			$this_depth	= $this->parental_depth[$row['parent_id']] + 1;
124
+			$this_depth = $this->parental_depth[$row['parent_id']] + 1;
125 125
 
126 126
 			$this->set_parental_depth($row, $this_depth, $leaf, $is_current_item);
127 127
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
 	 * @param int $repeat
235 235
 	 * @return void
236 236
 	 */
237
-	protected function close_open_tags(\phpbb\template\twig\twig &$template, $handle, $repeat)
237
+	protected function close_open_tags(\phpbb\template\twig\twig & $template, $handle, $repeat)
238 238
 	{
239 239
 		for ($i = 0; $i < $repeat; $i++)
240 240
 		{
Please login to merge, or discard this patch.