@@ -40,7 +40,7 @@ discard block |
||
| 40 | 40 | public function get_users_groups() |
| 41 | 41 | { |
| 42 | 42 | $sql = 'SELECT group_id |
| 43 | - FROM ' . USER_GROUP_TABLE . ' |
|
| 43 | + FROM ' . USER_GROUP_TABLE.' |
|
| 44 | 44 | WHERE user_id = ' . (int) $this->user->data['user_id']; |
| 45 | 45 | $result = $this->db->sql_query($sql); |
| 46 | 46 | |
@@ -83,13 +83,13 @@ discard block |
||
| 83 | 83 | $sql = $this->_get_group_sql($mode); |
| 84 | 84 | $result = $this->db->sql_query($sql); |
| 85 | 85 | |
| 86 | - $options = '<option value="0">' . $this->translator->lang('ALL_GROUPS') . '</option>'; |
|
| 86 | + $options = '<option value="0">'.$this->translator->lang('ALL_GROUPS').'</option>'; |
|
| 87 | 87 | while ($row = $this->db->sql_fetchrow($result)) |
| 88 | 88 | { |
| 89 | 89 | $group_name = $this->_get_group_name($row); |
| 90 | 90 | $group_class = $this->_get_group_class($row['group_type']); |
| 91 | 91 | $selected_option = $this->_get_selected_option($row['group_id'], $selected); |
| 92 | - $options .= '<option' . $group_class . ' value="' . $row['group_id'] . '"' . $selected_option . '>' . $group_name . '</option>'; |
|
| 92 | + $options .= '<option'.$group_class.' value="'.$row['group_id'].'"'.$selected_option.'>'.$group_name.'</option>'; |
|
| 93 | 93 | } |
| 94 | 94 | $this->db->sql_freeresult($result); |
| 95 | 95 | |
@@ -103,8 +103,8 @@ discard block |
||
| 103 | 103 | private function _get_group_sql($mode) |
| 104 | 104 | { |
| 105 | 105 | return 'SELECT group_id, group_name, group_type |
| 106 | - FROM ' . GROUPS_TABLE . |
|
| 107 | - (($mode === 'special') ? ' WHERE group_type = ' . GROUP_SPECIAL : '') . ' |
|
| 106 | + FROM ' . GROUPS_TABLE. |
|
| 107 | + (($mode === 'special') ? ' WHERE group_type = '.GROUP_SPECIAL : '').' |
|
| 108 | 108 | ORDER BY group_type DESC, group_name ASC'; |
| 109 | 109 | } |
| 110 | 110 | |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | */ |
| 115 | 115 | private function _get_group_name(array $row) |
| 116 | 116 | { |
| 117 | - return ($row['group_type'] == GROUP_SPECIAL) ? $this->translator->lang('G_' . $row['group_name']) : ucfirst($row['group_name']); |
|
| 117 | + return ($row['group_type'] == GROUP_SPECIAL) ? $this->translator->lang('G_'.$row['group_name']) : ucfirst($row['group_name']); |
|
| 118 | 118 | } |
| 119 | 119 | |
| 120 | 120 | /** |
@@ -63,7 +63,7 @@ discard block |
||
| 63 | 63 | // @codeCoverageIgnoreStart |
| 64 | 64 | if (!function_exists('phpbb_get_user_avatar')) |
| 65 | 65 | { |
| 66 | - include($phpbb_root_path . 'includes/functions_display.' . $php_ext); |
|
| 66 | + include($phpbb_root_path.'includes/functions_display.'.$php_ext); |
|
| 67 | 67 | } |
| 68 | 68 | // @codeCoverageIgnoreEnd |
| 69 | 69 | } |
@@ -101,8 +101,8 @@ discard block |
||
| 101 | 101 | */ |
| 102 | 102 | protected function member_posts(array $row) |
| 103 | 103 | { |
| 104 | - $u_posts = append_sid($this->phpbb_root_path . 'search.' . $this->php_ext, "author_id={$row['user_id']}&sr=posts" . $this->explain_range); |
|
| 105 | - $user_posts = '<a href="' . $u_posts . '">' . $row['user_posts'] . '</a>'; |
|
| 104 | + $u_posts = append_sid($this->phpbb_root_path.'search.'.$this->php_ext, "author_id={$row['user_id']}&sr=posts".$this->explain_range); |
|
| 105 | + $user_posts = '<a href="'.$u_posts.'">'.$row['user_posts'].'</a>'; |
|
| 106 | 106 | |
| 107 | 107 | return array( |
| 108 | 108 | 'USERNAME' => get_username_string('full', $row['user_id'], $row['username'], $row['user_colour']), |
@@ -170,7 +170,7 @@ discard block |
||
| 170 | 170 | 'WHERE' => $this->db->sql_in_set('u.user_type', array(USER_NORMAL, USER_FOUNDER)), |
| 171 | 171 | ); |
| 172 | 172 | |
| 173 | - $sql_method = 'set_' . $this->settings['query_type'] . '_sql'; |
|
| 173 | + $sql_method = 'set_'.$this->settings['query_type'].'_sql'; |
|
| 174 | 174 | call_user_func_array(array($this, $sql_method), array(&$sql_ary)); |
| 175 | 175 | |
| 176 | 176 | $this->set_range_sql($sql_ary); |
@@ -184,7 +184,7 @@ discard block |
||
| 184 | 184 | protected function set_visits_sql(array &$sql_ary) |
| 185 | 185 | { |
| 186 | 186 | $sql_ary['SELECT'] .= ', u.user_lastvisit as member_date'; |
| 187 | - $sql_ary['WHERE'] .= ' AND u.user_lastvisit <> 0 AND u.user_id <> ' . (int) $this->user->data['user_id']; |
|
| 187 | + $sql_ary['WHERE'] .= ' AND u.user_lastvisit <> 0 AND u.user_id <> '.(int) $this->user->data['user_id']; |
|
| 188 | 188 | $sql_ary['ORDER_BY'] = 'u.user_lastvisit DESC'; |
| 189 | 189 | |
| 190 | 190 | $this->sql_date_field = 'user_lastvisit'; |
@@ -200,7 +200,7 @@ discard block |
||
| 200 | 200 | $this->info_header = ''; |
| 201 | 201 | $this->view_mode = 'member_bots'; |
| 202 | 202 | |
| 203 | - $sql_ary['WHERE'] = 'u.user_type = ' . USER_IGNORE; |
|
| 203 | + $sql_ary['WHERE'] = 'u.user_type = '.USER_IGNORE; |
|
| 204 | 204 | } |
| 205 | 205 | |
| 206 | 206 | /** |
@@ -209,7 +209,7 @@ discard block |
||
| 209 | 209 | protected function set_tenured_sql(array &$sql_ary) |
| 210 | 210 | { |
| 211 | 211 | $sql_ary['SELECT'] .= ', u.user_regdate as member_date'; |
| 212 | - $sql_ary['ORDER_BY'] = 'u.user_regdate ' . (($this->settings['query_type'] == 'tenured') ? 'ASC' : 'DESC'); |
|
| 212 | + $sql_ary['ORDER_BY'] = 'u.user_regdate '.(($this->settings['query_type'] == 'tenured') ? 'ASC' : 'DESC'); |
|
| 213 | 213 | |
| 214 | 214 | $this->sql_date_field = 'u.user_regdate'; |
| 215 | 215 | $this->settings['date_range'] = ''; |
@@ -232,7 +232,7 @@ discard block |
||
| 232 | 232 | $sql_ary['SELECT'] .= ', COUNT(p.post_id) as user_posts'; |
| 233 | 233 | $sql_ary['FROM'] += array(TOPICS_TABLE => 't'); |
| 234 | 234 | $sql_ary['FROM'] += array(POSTS_TABLE => 'p'); |
| 235 | - $sql_ary['WHERE'] .= ' AND ' . time() . ' > t.topic_time AND t.topic_id = p.topic_id AND p.post_visibility = ' . ITEM_APPROVED . ' AND p.poster_id = u.user_id'; |
|
| 235 | + $sql_ary['WHERE'] .= ' AND '.time().' > t.topic_time AND t.topic_id = p.topic_id AND p.post_visibility = '.ITEM_APPROVED.' AND p.poster_id = u.user_id'; |
|
| 236 | 236 | $sql_ary['GROUP_BY'] = 'u.user_id'; |
| 237 | 237 | $sql_ary['ORDER_BY'] = 'user_posts DESC, u.username ASC'; |
| 238 | 238 | |
@@ -249,7 +249,7 @@ discard block |
||
| 249 | 249 | if ($this->settings['date_range'] && $this->sql_date_field) |
| 250 | 250 | { |
| 251 | 251 | $range = $this->date_range->get($this->settings['date_range']); |
| 252 | - $this->explain_range = '&date=' . $range['date']; |
|
| 252 | + $this->explain_range = '&date='.$range['date']; |
|
| 253 | 253 | |
| 254 | 254 | $sql_ary['WHERE'] .= " AND {$this->sql_date_field} BETWEEN {$range['start']} AND {$range['stop']}"; |
| 255 | 255 | } |
@@ -96,13 +96,13 @@ discard block |
||
| 96 | 96 | public function set_javascript_data($route, $style_id) |
| 97 | 97 | { |
| 98 | 98 | $board_url = generate_board_url(); |
| 99 | - $ajax_url = $board_url . ((!$this->config['enable_mod_rewrite']) ? '/app.' . $this->php_ext : ''); |
|
| 99 | + $ajax_url = $board_url.((!$this->config['enable_mod_rewrite']) ? '/app.'.$this->php_ext : ''); |
|
| 100 | 100 | |
| 101 | 101 | $is_default_route = $u_default_route = false; |
| 102 | 102 | if ($this->config['sitemaker_default_layout']) |
| 103 | 103 | { |
| 104 | 104 | $is_default_route = ($this->config['sitemaker_default_layout'] === $route) ? true : false; |
| 105 | - $u_default_route .= $board_url . '/' . $this->config['sitemaker_default_layout']; |
|
| 105 | + $u_default_route .= $board_url.'/'.$this->config['sitemaker_default_layout']; |
|
| 106 | 106 | $u_default_route = reapply_sid($u_default_route); |
| 107 | 107 | } |
| 108 | 108 | |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | list($controller_service, $controller_method) = explode(':', $controller); |
| 151 | 151 | $controller_params = $symfony_request->attributes->get('_route_params'); |
| 152 | 152 | $controller_object = $this->phpbb_container->get($controller_service); |
| 153 | - $controller_class = get_class($controller_object); |
|
| 153 | + $controller_class = get_class($controller_object); |
|
| 154 | 154 | |
| 155 | 155 | $r = new \ReflectionMethod($controller_class, $controller_method); |
| 156 | 156 | $class_params = $r->getParameters(); |
@@ -163,7 +163,7 @@ discard block |
||
| 163 | 163 | 'CONTROLLER_METHOD' => $controller_method, |
| 164 | 164 | 'CONTROLLER_PARAMS' => $controller_arguments, |
| 165 | 165 | 'S_IS_STARTPAGE' => $this->is_startpage($controller_service, $controller_arguments), |
| 166 | - 'UA_EXTENSION' => $namespace . '/' . $extension, |
|
| 166 | + 'UA_EXTENSION' => $namespace.'/'.$extension, |
|
| 167 | 167 | )); |
| 168 | 168 | } |
| 169 | 169 | } |
@@ -198,11 +198,11 @@ discard block |
||
| 198 | 198 | { |
| 199 | 199 | $routes_ary = $this->get_routes(); |
| 200 | 200 | |
| 201 | - $options = '<option value="">' . $this->translator->lang('SELECT') . '</option>'; |
|
| 201 | + $options = '<option value="">'.$this->translator->lang('SELECT').'</option>'; |
|
| 202 | 202 | foreach ($routes_ary as $route) |
| 203 | 203 | { |
| 204 | 204 | $selected = ($route == $current_route) ? ' selected="selected"' : ''; |
| 205 | - $options .= '<option value="' . $route . '"' . $selected . '>' . $route . '</option>'; |
|
| 205 | + $options .= '<option value="'.$route.'"'.$selected.'>'.$route.'</option>'; |
|
| 206 | 206 | } |
| 207 | 207 | |
| 208 | 208 | return $options; |
@@ -219,7 +219,7 @@ discard block |
||
| 219 | 219 | $options = ''; |
| 220 | 220 | foreach ($ex_positions as $position) |
| 221 | 221 | { |
| 222 | - $options .= '<option value="' . $position . '" selected="selected">' . $position . '</option>'; |
|
| 222 | + $options .= '<option value="'.$position.'" selected="selected">'.$position.'</option>'; |
|
| 223 | 223 | } |
| 224 | 224 | |
| 225 | 225 | return $options; |
@@ -22,7 +22,7 @@ discard block |
||
| 22 | 22 | // @codeCoverageIgnoreStart |
| 23 | 23 | if (!function_exists('make_forum_select')) |
| 24 | 24 | { |
| 25 | - include($phpbb_root_path . 'includes/functions_admin.' . $php_ext); |
|
| 25 | + include($phpbb_root_path.'includes/functions_admin.'.$php_ext); |
|
| 26 | 26 | } |
| 27 | 27 | // @codeCoverageIgnoreEnd |
| 28 | 28 | } |
@@ -43,7 +43,7 @@ discard block |
||
| 43 | 43 | $forum_options = array('' => 'ALL_FORUMS'); |
| 44 | 44 | foreach ($forumlist as $row) |
| 45 | 45 | { |
| 46 | - $forum_options[$row['forum_id']] = $row['padding'] . $row['forum_name']; |
|
| 46 | + $forum_options[$row['forum_id']] = $row['padding'].$row['forum_name']; |
|
| 47 | 47 | } |
| 48 | 48 | |
| 49 | 49 | return $forum_options; |