@@ -68,7 +68,7 @@ discard block |
||
| 68 | 68 | * @param array $topic_data |
| 69 | 69 | * @param \phpbb\template\twig\twig $template |
| 70 | 70 | */ |
| 71 | - public function build(array $topic_data, \phpbb\template\twig\twig &$template) |
|
| 71 | + public function build(array $topic_data, \phpbb\template\twig\twig & $template) |
|
| 72 | 72 | { |
| 73 | 73 | $this->translator->add_lang('viewtopic'); |
| 74 | 74 | |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | 'S_POLL_ACTION' => $viewtopic_url, |
| 101 | 101 | 'S_FORM_TOKEN' => $this->sitemaker->get_form_key('posting'), |
| 102 | 102 | |
| 103 | - 'U_VIEW_RESULTS' => $viewtopic_url . '&view=viewpoll', |
|
| 103 | + 'U_VIEW_RESULTS' => $viewtopic_url.'&view=viewpoll', |
|
| 104 | 104 | )); |
| 105 | 105 | } |
| 106 | 106 | |
@@ -171,7 +171,7 @@ discard block |
||
| 171 | 171 | $post_id = (int) $topic_data['topic_first_post_id']; |
| 172 | 172 | |
| 173 | 173 | $sql = 'SELECT o.*, p.bbcode_bitfield, p.bbcode_uid |
| 174 | - FROM ' . POLL_OPTIONS_TABLE . ' o, ' . POSTS_TABLE . " p |
|
| 174 | + FROM ' . POLL_OPTIONS_TABLE.' o, '.POSTS_TABLE." p |
|
| 175 | 175 | WHERE o.topic_id = $topic_id |
| 176 | 176 | AND p.post_id = $post_id |
| 177 | 177 | AND p.topic_id = o.topic_id |
@@ -216,7 +216,7 @@ discard block |
||
| 216 | 216 | * @param int $poll_most |
| 217 | 217 | * @param \phpbb\template\twig\twig $template |
| 218 | 218 | */ |
| 219 | - private function build_poll_options(array $cur_voted_id, array $poll_info, $poll_total, $poll_most, \phpbb\template\twig\twig &$template) |
|
| 219 | + private function build_poll_options(array $cur_voted_id, array $poll_info, $poll_total, $poll_most, \phpbb\template\twig\twig & $template) |
|
| 220 | 220 | { |
| 221 | 221 | foreach ($poll_info as $poll_option) |
| 222 | 222 | { |
@@ -247,8 +247,8 @@ discard block |
||
| 247 | 247 | if ($this->user->data['is_registered']) |
| 248 | 248 | { |
| 249 | 249 | $sql = 'SELECT poll_option_id |
| 250 | - FROM ' . POLL_VOTES_TABLE . ' |
|
| 251 | - WHERE topic_id = ' . (int) $topic_id . ' |
|
| 250 | + FROM ' . POLL_VOTES_TABLE.' |
|
| 251 | + WHERE topic_id = ' . (int) $topic_id.' |
|
| 252 | 252 | AND vote_user_id = ' . $this->user->data['user_id']; |
| 253 | 253 | $result = $this->db->sql_query($sql); |
| 254 | 254 | |
@@ -263,9 +263,9 @@ discard block |
||
| 263 | 263 | // Cookie based guest tracking ... I don't like this but hum ho |
| 264 | 264 | // it's oft requested. This relies on "nice" users who don't feel |
| 265 | 265 | // the need to delete cookies to mess with results. |
| 266 | - if ($this->request->is_set($this->config['cookie_name'] . '_poll_' . $topic_id, \phpbb\request\request_interface::COOKIE)) |
|
| 266 | + if ($this->request->is_set($this->config['cookie_name'].'_poll_'.$topic_id, \phpbb\request\request_interface::COOKIE)) |
|
| 267 | 267 | { |
| 268 | - $cur_voted_id = explode(',', $this->request->variable($this->config['cookie_name'] . '_poll_' . $topic_id, '', true, \phpbb\request\request_interface::COOKIE)); |
|
| 268 | + $cur_voted_id = explode(',', $this->request->variable($this->config['cookie_name'].'_poll_'.$topic_id, '', true, \phpbb\request\request_interface::COOKIE)); |
|
| 269 | 269 | $cur_voted_id = array_map('intval', $cur_voted_id); |
| 270 | 270 | } |
| 271 | 271 | } |
@@ -303,8 +303,7 @@ |
||
| 303 | 303 | $cur_voted_id[] = $row['poll_option_id']; |
| 304 | 304 | } |
| 305 | 305 | $this->db->sql_freeresult($result); |
| 306 | - } |
|
| 307 | - else |
|
| 306 | + } else |
|
| 308 | 307 | { |
| 309 | 308 | // Cookie based guest tracking ... I don't like this but hum ho |
| 310 | 309 | // it's oft requested. This relies on "nice" users who don't feel |
@@ -127,7 +127,7 @@ |
||
| 127 | 127 | { |
| 128 | 128 | $this->entities = array_filter( |
| 129 | 129 | $this->entities, |
| 130 | - function ($v) use ($key) |
|
| 130 | + function($v) use ($key) |
|
| 131 | 131 | { |
| 132 | 132 | return $v !== $key; |
| 133 | 133 | } |
@@ -104,8 +104,7 @@ |
||
| 104 | 104 | if (!isset($key)) |
| 105 | 105 | { |
| 106 | 106 | $this->entities[] = $entity; |
| 107 | - } |
|
| 108 | - else |
|
| 107 | + } else |
|
| 109 | 108 | { |
| 110 | 109 | $this->entities[$key] = $entity; |
| 111 | 110 | } |
@@ -96,8 +96,7 @@ |
||
| 96 | 96 | if ($entity->get_item_id()) |
| 97 | 97 | { |
| 98 | 98 | $item = $this->tree->update_item($entity->get_item_id(), $sql_data); |
| 99 | - } |
|
| 100 | - else |
|
| 99 | + } else |
|
| 101 | 100 | { |
| 102 | 101 | $item = $this->tree->insert($sql_data); |
| 103 | 102 | } |
@@ -25,8 +25,8 @@ discard block |
||
| 25 | 25 | */ |
| 26 | 26 | protected function find_sql(array $sql_where) |
| 27 | 27 | { |
| 28 | - return 'SELECT * FROM ' . $this->entity_table . |
|
| 29 | - ((sizeof($sql_where)) ? ' WHERE ' . join(' AND ', $sql_where) : '') . ' |
|
| 28 | + return 'SELECT * FROM '.$this->entity_table. |
|
| 29 | + ((sizeof($sql_where)) ? ' WHERE '.join(' AND ', $sql_where) : '').' |
|
| 30 | 30 | ORDER BY position, weight ASC'; |
| 31 | 31 | } |
| 32 | 32 | |
@@ -40,12 +40,12 @@ discard block |
||
| 40 | 40 | // move blocks up for position |
| 41 | 41 | if ($condition instanceof $this->entity_class) |
| 42 | 42 | { |
| 43 | - $this->db->sql_query('UPDATE ' . $this->entity_table . ' |
|
| 43 | + $this->db->sql_query('UPDATE '.$this->entity_table.' |
|
| 44 | 44 | SET weight = weight - 1 |
| 45 | - WHERE weight > ' . (int) $condition->get_weight() . ' |
|
| 46 | - AND style = ' . (int) $condition->get_style() . ' |
|
| 47 | - AND route_id = ' . (int) $condition->get_route_id() . " |
|
| 48 | - AND position = '" . $this->db->sql_escape($condition->get_position()) . "'"); |
|
| 45 | + WHERE weight > ' . (int) $condition->get_weight().' |
|
| 46 | + AND style = ' . (int) $condition->get_style().' |
|
| 47 | + AND route_id = ' . (int) $condition->get_route_id()." |
|
| 48 | + AND position = '" . $this->db->sql_escape($condition->get_position())."'"); |
|
| 49 | 49 | } |
| 50 | 50 | } |
| 51 | 51 | |
@@ -65,10 +65,10 @@ discard block |
||
| 65 | 65 | protected function move_blocks_down(\blitze\sitemaker\model\entity_interface $entity) |
| 66 | 66 | { |
| 67 | 67 | /** @type \blitze\sitemaker\model\entity\block $entity */ |
| 68 | - $sql = 'UPDATE ' . $this->entity_table . ' |
|
| 68 | + $sql = 'UPDATE '.$this->entity_table.' |
|
| 69 | 69 | SET weight = weight + 1 |
| 70 | - WHERE weight >= ' . (int) $entity->get_weight() . ' |
|
| 71 | - AND route_id = ' . (int) $entity->get_route_id() . ' |
|
| 70 | + WHERE weight >= ' . (int) $entity->get_weight().' |
|
| 71 | + AND route_id = ' . (int) $entity->get_route_id().' |
|
| 72 | 72 | AND style = ' . (int) $entity->get_style(); |
| 73 | 73 | $this->db->sql_query($sql); |
| 74 | 74 | } |
@@ -135,7 +135,7 @@ discard block |
||
| 135 | 135 | $this->config->set('sm_show_forum_nav', $this->request->variable('show_forum_nav', 0)); |
| 136 | 136 | $this->config->set('sm_forum_icon', $this->request->variable('forum_icon', '')); |
| 137 | 137 | |
| 138 | - $this->trigger_error($this->translator->lang('SETTINGS_SAVED') . adm_back_link($this->u_action)); |
|
| 138 | + $this->trigger_error($this->translator->lang('SETTINGS_SAVED').adm_back_link($this->u_action)); |
|
| 139 | 139 | } |
| 140 | 140 | } |
| 141 | 141 | |
@@ -147,7 +147,7 @@ discard block |
||
| 147 | 147 | { |
| 148 | 148 | $style_prefs = (array) json_decode($this->config_text->get('sm_layout_prefs'), true); |
| 149 | 149 | |
| 150 | - $result = $this->db->sql_query('SELECT style_id, style_name FROM ' . STYLES_TABLE); |
|
| 150 | + $result = $this->db->sql_query('SELECT style_id, style_name FROM '.STYLES_TABLE); |
|
| 151 | 151 | |
| 152 | 152 | $styles = array(); |
| 153 | 153 | while ($row = $this->db->sql_fetchrow($result)) |
@@ -201,7 +201,7 @@ discard block |
||
| 201 | 201 | foreach ($layouts as $name => $path) |
| 202 | 202 | { |
| 203 | 203 | $selected = ($path == $pref) ? ' selected="selected"' : ''; |
| 204 | - $options .= '<option value="' . $path . '"' . $selected . '>' . $this->translator->lang('LAYOUT_' . strtoupper($name)) . '</option>'; |
|
| 204 | + $options .= '<option value="'.$path.'"'.$selected.'>'.$this->translator->lang('LAYOUT_'.strtoupper($name)).'</option>'; |
|
| 205 | 205 | } |
| 206 | 206 | |
| 207 | 207 | return $options; |
@@ -215,11 +215,11 @@ discard block |
||
| 215 | 215 | { |
| 216 | 216 | $views = array('basic', 'boxed', 'simple'); |
| 217 | 217 | |
| 218 | - $options = '<option value="">' . $this->translator->lang('BLOCK_VIEW_DEFAULT') . '</option>'; |
|
| 218 | + $options = '<option value="">'.$this->translator->lang('BLOCK_VIEW_DEFAULT').'</option>'; |
|
| 219 | 219 | foreach ($views as $view) |
| 220 | 220 | { |
| 221 | 221 | $selected = ($view == $pref) ? ' selected="selected"' : ''; |
| 222 | - $options .= '<option value="' . $view . '"' . $selected . '>' . $this->translator->lang('BLOCK_VIEW_' . strtoupper($view)) . '</option>'; |
|
| 222 | + $options .= '<option value="'.$view.'"'.$selected.'>'.$this->translator->lang('BLOCK_VIEW_'.strtoupper($view)).'</option>'; |
|
| 223 | 223 | } |
| 224 | 224 | |
| 225 | 225 | return $options; |
@@ -261,7 +261,7 @@ discard block |
||
| 261 | 261 | $path = dirname($path); |
| 262 | 262 | $name = basename($path); |
| 263 | 263 | |
| 264 | - $layouts[$name] = $this->phpbb_root_path . $path . '/'; |
|
| 264 | + $layouts[$name] = $this->phpbb_root_path.$path.'/'; |
|
| 265 | 265 | } |
| 266 | 266 | ksort($layouts); |
| 267 | 267 | |
@@ -88,8 +88,7 @@ |
||
| 88 | 88 | $return_data = $command->execute($style_id); |
| 89 | 89 | |
| 90 | 90 | $this->action_handler->clear_cache(); |
| 91 | - } |
|
| 92 | - catch (\blitze\sitemaker\exception\base $e) |
|
| 91 | + } catch (\blitze\sitemaker\exception\base $e) |
|
| 93 | 92 | { |
| 94 | 93 | $json_data['message'] = $e->get_message($this->translator); |
| 95 | 94 | } |
@@ -63,12 +63,10 @@ |
||
| 63 | 63 | $return_data = $command->execute(); |
| 64 | 64 | |
| 65 | 65 | $this->action_handler->clear_cache(); |
| 66 | - } |
|
| 67 | - catch (\blitze\sitemaker\exception\base $e) |
|
| 66 | + } catch (\blitze\sitemaker\exception\base $e) |
|
| 68 | 67 | { |
| 69 | 68 | $return_data['message'] = $e->get_message($this->translator); |
| 70 | - } |
|
| 71 | - catch (\Exception $e) |
|
| 69 | + } catch (\Exception $e) |
|
| 72 | 70 | { |
| 73 | 71 | $return_data['message'] = $this->translator->lang($e->getMessage()); |
| 74 | 72 | } |
@@ -108,7 +108,7 @@ discard block |
||
| 108 | 108 | { |
| 109 | 109 | $this->set_display_fields(); |
| 110 | 110 | |
| 111 | - $view = 'S_' . strtoupper($this->settings['template']); |
|
| 111 | + $view = 'S_'.strtoupper($this->settings['template']); |
|
| 112 | 112 | $post_data = $this->get_post_data($topic_data); |
| 113 | 113 | $topic_data = array_values($topic_data); |
| 114 | 114 | |
@@ -156,10 +156,10 @@ discard block |
||
| 156 | 156 | 'VIEWS' => (int) $row['topic_views'], |
| 157 | 157 | 'S_UNREAD_TOPIC' => $this->is_unread_topic($forum_id, $topic_id, $row['topic_last_post_time']), |
| 158 | 158 | |
| 159 | - 'U_VIEWTOPIC' => append_sid($this->phpbb_root_path . 'viewtopic.' . $this->php_ext, "f=$forum_id&t=$topic_id"), |
|
| 160 | - 'U_VIEWFORUM' => append_sid($this->phpbb_root_path . 'viewforum.' . $this->php_ext, "f=$forum_id"), |
|
| 161 | - 'U_NEW_POST' => append_sid($this->phpbb_root_path . 'viewtopic.' . $this->php_ext, "f=$forum_id&t=$topic_id&view=unread") . '#unread', |
|
| 162 | - 'U_LAST_POST' => append_sid($this->phpbb_root_path . 'viewtopic.' . $this->php_ext, "f=$forum_id&t=$topic_id&p=" . $row['topic_last_post_id']) . '#p' . $row['topic_last_post_id'], |
|
| 159 | + 'U_VIEWTOPIC' => append_sid($this->phpbb_root_path.'viewtopic.'.$this->php_ext, "f=$forum_id&t=$topic_id"), |
|
| 160 | + 'U_VIEWFORUM' => append_sid($this->phpbb_root_path.'viewforum.'.$this->php_ext, "f=$forum_id"), |
|
| 161 | + 'U_NEW_POST' => append_sid($this->phpbb_root_path.'viewtopic.'.$this->php_ext, "f=$forum_id&t=$topic_id&view=unread").'#unread', |
|
| 162 | + 'U_LAST_POST' => append_sid($this->phpbb_root_path.'viewtopic.'.$this->php_ext, "f=$forum_id&t=$topic_id&p=".$row['topic_last_post_id']).'#p'.$row['topic_last_post_id'], |
|
| 163 | 163 | )); |
| 164 | 164 | unset($topic_data[$i], $post_data[$topic_id]); |
| 165 | 165 | } |
@@ -257,8 +257,7 @@ discard block |
||
| 257 | 257 | if ($this->settings['context'] && $this->settings['preview_chars']) |
| 258 | 258 | { |
| 259 | 259 | $post_data = $this->forum_data->get_post_data($this->settings['context']); |
| 260 | - } |
|
| 261 | - else |
|
| 260 | + } else |
|
| 262 | 261 | { |
| 263 | 262 | $post_data = array_fill_keys(array_keys($topic_data), array(array('post_text' => '', 'bbcode_uid' => '', 'bbcode_bitfield' => ''))); |
| 264 | 263 | } |
@@ -277,8 +276,7 @@ discard block |
||
| 277 | 276 | $this->fields['user_id'] = 'topic_last_poster_id'; |
| 278 | 277 | |
| 279 | 278 | $this->ptemplate->assign_var('L_POST_BY_AUTHOR', $this->translator->lang('LAST_POST_BY_AUTHOR')); |
| 280 | - } |
|
| 281 | - else |
|
| 279 | + } else |
|
| 282 | 280 | { |
| 283 | 281 | $this->fields['time'] = 'topic_time'; |
| 284 | 282 | $this->fields['user_id'] = 'topic_poster'; |
@@ -72,8 +72,7 @@ discard block |
||
| 72 | 72 | 'title' => '', |
| 73 | 73 | 'content' => '', |
| 74 | 74 | ); |
| 75 | - } |
|
| 76 | - else |
|
| 75 | + } else |
|
| 77 | 76 | { |
| 78 | 77 | $this->fetch_new($bdata['settings']); |
| 79 | 78 | |
@@ -117,8 +116,7 @@ discard block |
||
| 117 | 116 | { |
| 118 | 117 | $sorting = 't.topic_last_post_time'; |
| 119 | 118 | $sql_array = $this->get_topics_sql(); |
| 120 | - } |
|
| 121 | - else |
|
| 119 | + } else |
|
| 122 | 120 | { |
| 123 | 121 | $sorting = 'p.post_time'; |
| 124 | 122 | $sql_array = $this->get_posts_sql(); |