@@ -42,7 +42,7 @@ discard block |
||
42 | 42 | $options = $this->get_select_options($option_ary, $selected_item, $key, $data_toggle_key); |
43 | 43 | $data_toggle = ($data_toggle_key) ? ' data-togglable-settings="true"' : ''; |
44 | 44 | |
45 | - return '<select id="' . $key . '" name="config[' . $key . ']' . (($multi_select) ? '[]" multiple="multiple"' : '"') . (($size > 1) ? ' size="' . $size . '"' : '') . $data_toggle . '>' . $options . '</select>'; |
|
45 | + return '<select id="'.$key.'" name="config['.$key.']'.(($multi_select) ? '[]" multiple="multiple"' : '"').(($size > 1) ? ' size="'.$size.'"' : '').$data_toggle.'>'.$options.'</select>'; |
|
46 | 46 | } |
47 | 47 | |
48 | 48 | /** |
@@ -57,11 +57,11 @@ discard block |
||
57 | 57 | { |
58 | 58 | $selected_items = $this->ensure_array($selected_items); |
59 | 59 | |
60 | - $html = '<select id="' . $field . '" name="config[' . $field . '][]" multiple="multiple">'; |
|
60 | + $html = '<select id="'.$field.'" name="config['.$field.'][]" multiple="multiple">'; |
|
61 | 61 | foreach ($option_ary as $value => $title) |
62 | 62 | { |
63 | 63 | $selected = $this->get_selected_option($value, $selected_items); |
64 | - $html .= '<option value="' . $value . '"' . $selected . '>' . $this->translator->lang($title) . '</option>'; |
|
64 | + $html .= '<option value="'.$value.'"'.$selected.'>'.$this->translator->lang($title).'</option>'; |
|
65 | 65 | } |
66 | 66 | $html .= '</select>'; |
67 | 67 | |
@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | foreach ($option_ary as $value => $title) |
121 | 121 | { |
122 | 122 | $selected = $this->get_selected_option($value, $selected_item, 'checked'); |
123 | - $html .= '<label><input type="radio" name="config[' . $key . ']" value="' . $value . '"' . $selected . ' class="radio" /> ' . $this->translator->lang($title) . '</label><br />'; |
|
123 | + $html .= '<label><input type="radio" name="config['.$key.']" value="'.$value.'"'.$selected.' class="radio" /> '.$this->translator->lang($title).'</label><br />'; |
|
124 | 124 | } |
125 | 125 | |
126 | 126 | return $html; |
@@ -164,7 +164,7 @@ discard block |
||
164 | 164 | */ |
165 | 165 | protected function get_selected_option($needle, array $haystack, $type = 'selected') |
166 | 166 | { |
167 | - return (in_array($needle, $haystack)) ? ' ' . $type . '="' . $type . '"' : ''; |
|
167 | + return (in_array($needle, $haystack)) ? ' '.$type.'="'.$type.'"' : ''; |
|
168 | 168 | } |
169 | 169 | |
170 | 170 | /** |
@@ -178,12 +178,12 @@ discard block |
||
178 | 178 | */ |
179 | 179 | protected function get_checkbox_column(array $row, array $selected_items, $field, $column_class, $column_count, &$index) |
180 | 180 | { |
181 | - $column = '<div class="' . $column_class . $field . '-checkbox" id="' . $field . '-col-' . $column_count . '">'; |
|
181 | + $column = '<div class="'.$column_class.$field.'-checkbox" id="'.$field.'-col-'.$column_count.'">'; |
|
182 | 182 | foreach ($row as $value => $title) |
183 | 183 | { |
184 | 184 | $title = $this->translator->lang($title); |
185 | 185 | $selected = $this->get_selected_option($value, $selected_items, 'checked'); |
186 | - $column .= '<label><input type="checkbox" name="config[' . $field . '][' . $index . ']" value="' . $value . '"' . $selected . ' class="checkbox" /> ' . $title . '</label><br />'; |
|
186 | + $column .= '<label><input type="checkbox" name="config['.$field.']['.$index.']" value="'.$value.'"'.$selected.' class="checkbox" /> '.$title.'</label><br />'; |
|
187 | 187 | $index++; |
188 | 188 | } |
189 | 189 | $column .= '</div>'; |
@@ -204,8 +204,8 @@ discard block |
||
204 | 204 | foreach ($option_ary as $value => $title) |
205 | 205 | { |
206 | 206 | $selected = $this->get_selected_option($value, $selected_items); |
207 | - $togglable_option = ($togglable_key) ? ' data-toggle-setting="#' . $togglable_key . '-' . $value . '"' : ''; |
|
208 | - $options .= '<option value="' . $value . '"' . $selected . $togglable_option . '>' . $this->translator->lang($title) . '</option>'; |
|
207 | + $togglable_option = ($togglable_key) ? ' data-toggle-setting="#'.$togglable_key.'-'.$value.'"' : ''; |
|
208 | + $options .= '<option value="'.$value.'"'.$selected.$togglable_option.'>'.$this->translator->lang($title).'</option>'; |
|
209 | 209 | } |
210 | 210 | return $options; |
211 | 211 | } |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | // @codeCoverageIgnoreStart |
62 | 62 | if (!function_exists('build_cfg_template')) |
63 | 63 | { |
64 | - include($this->phpbb_root_path . 'includes/functions_acp.' . $this->php_ext); |
|
64 | + include($this->phpbb_root_path.'includes/functions_acp.'.$this->php_ext); |
|
65 | 65 | } |
66 | 66 | // @codeCoverageIgnoreEnd |
67 | 67 | |
@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | // @codeCoverageIgnoreStart |
101 | 101 | if (!function_exists('validate_config_vars')) |
102 | 102 | { |
103 | - include($this->phpbb_root_path . 'includes/functions_acp.' . $this->php_ext); |
|
103 | + include($this->phpbb_root_path.'includes/functions_acp.'.$this->php_ext); |
|
104 | 104 | } |
105 | 105 | // @codeCoverageIgnoreEnd |
106 | 106 | |
@@ -218,7 +218,7 @@ discard block |
||
218 | 218 | if (empty($vars['object'])) |
219 | 219 | { |
220 | 220 | $object = $this; |
221 | - $method = 'prep_' . $type[0] . '_field_for_display'; |
|
221 | + $method = 'prep_'.$type[0].'_field_for_display'; |
|
222 | 222 | |
223 | 223 | if (is_callable(array($this, $method))) |
224 | 224 | { |
@@ -280,7 +280,7 @@ discard block |
||
280 | 280 | $l_explain = ''; |
281 | 281 | if (!empty($vars['explain'])) |
282 | 282 | { |
283 | - $l_explain = (isset($vars['lang_explain'])) ? $this->translator->lang($vars['lang_explain']) : $this->translator->lang($vars['lang'] . '_EXPLAIN'); |
|
283 | + $l_explain = (isset($vars['lang_explain'])) ? $this->translator->lang($vars['lang_explain']) : $this->translator->lang($vars['lang'].'_EXPLAIN'); |
|
284 | 284 | } |
285 | 285 | |
286 | 286 | return $l_explain; |
@@ -341,9 +341,9 @@ discard block |
||
341 | 341 | { |
342 | 342 | $vars['method'] = 'build_select'; |
343 | 343 | $vars['params'][] = $field; |
344 | - $vars['params'][] = $type[1] ?: 1; // size |
|
345 | - $vars['params'][] = (bool) $type[2] ?: false; // multi select |
|
346 | - $vars['params'][] = $type[3] ?: ''; // togggle key |
|
344 | + $vars['params'][] = $type[1] ?: 1; // size |
|
345 | + $vars['params'][] = (bool) $type[2] ?: false; // multi select |
|
346 | + $vars['params'][] = $type[3] ?: ''; // togggle key |
|
347 | 347 | $type[0] = 'custom'; |
348 | 348 | } |
349 | 349 | |
@@ -383,7 +383,7 @@ discard block |
||
383 | 383 | { |
384 | 384 | $this->prep_checkbox_field_for_display($vars, $type, $field); |
385 | 385 | |
386 | - $vars['method'] ='build_multi_select'; |
|
386 | + $vars['method'] = 'build_multi_select'; |
|
387 | 387 | } |
388 | 388 | |
389 | 389 | /** |