@@ -13,8 +13,8 @@ |
||
| 13 | 13 | { |
| 14 | 14 | public function execute($style_id) |
| 15 | 15 | { |
| 16 | - $name = $this->request->variable('block', ''); |
|
| 17 | - $route = $this->request->variable('route', ''); |
|
| 16 | + $name = $this->request->variable('block', ''); |
|
| 17 | + $route = $this->request->variable('route', ''); |
|
| 18 | 18 | |
| 19 | 19 | if (($block_instance = $this->block_factory->get_block($name)) === null) |
| 20 | 20 | { |
@@ -29,7 +29,7 @@ |
||
| 29 | 29 | $headers = $headers[0]; |
| 30 | 30 | } |
| 31 | 31 | |
| 32 | - preg_match('/HTTP\/.* ([0-9]+) .*/', $headers , $status); |
|
| 32 | + preg_match('/HTTP\/.* ([0-9]+) .*/', $headers, $status); |
|
| 33 | 33 | |
| 34 | 34 | return ($status[1] == 200) ? true : false; |
| 35 | 35 | } |
@@ -339,7 +339,7 @@ |
||
| 339 | 339 | $this->store['topic'][$row['topic_id']] = $row; |
| 340 | 340 | |
| 341 | 341 | $this->store['tracking'][$row['forum_id']]['topic_list'][] = $row['topic_id']; |
| 342 | - $this->store['tracking'][$row['forum_id']]['mark_time'] =& $row['forum_mark_time']; |
|
| 342 | + $this->store['tracking'][$row['forum_id']]['mark_time'] = & $row['forum_mark_time']; |
|
| 343 | 343 | $this->store['post_ids']['first'][] = $row['topic_first_post_id']; |
| 344 | 344 | $this->store['post_ids']['last'][] = $row['topic_last_post_id']; |
| 345 | 345 | } |
@@ -121,7 +121,7 @@ |
||
| 121 | 121 | { |
| 122 | 122 | $this->_entities = array_filter( |
| 123 | 123 | $this->_entities, |
| 124 | - function ($v) use ($key) |
|
| 124 | + function($v) use ($key) |
|
| 125 | 125 | { |
| 126 | 126 | return $v !== $key; |
| 127 | 127 | } |
@@ -33,13 +33,13 @@ |
||
| 33 | 33 | protected $data = array(); |
| 34 | 34 | |
| 35 | 35 | /** |
| 36 | - * Construct |
|
| 37 | - * |
|
| 38 | - * @param \phpbb\db\driver\driver_interface $db Database connection |
|
| 39 | - * @param string $items_table Table name |
|
| 40 | - * @param string $pk Primary key |
|
| 41 | - * @param string $sql_where Column restriction |
|
| 42 | - */ |
|
| 36 | + * Construct |
|
| 37 | + * |
|
| 38 | + * @param \phpbb\db\driver\driver_interface $db Database connection |
|
| 39 | + * @param string $items_table Table name |
|
| 40 | + * @param string $pk Primary key |
|
| 41 | + * @param string $sql_where Column restriction |
|
| 42 | + */ |
|
| 43 | 43 | public function __construct(\phpbb\db\driver\driver_interface $db, $items_table, $pk, $sql_where = '') |
| 44 | 44 | { |
| 45 | 45 | $this->db = $db; |
@@ -79,7 +79,7 @@ discard block |
||
| 79 | 79 | { |
| 80 | 80 | $sql = "SELECT * |
| 81 | 81 | FROM $this->items_table |
| 82 | - WHERE $this->pk = " . (int) $node_id ; |
|
| 82 | + WHERE $this->pk = ".(int) $node_id; |
|
| 83 | 83 | $result = $this->db->sql_query($sql); |
| 84 | 84 | $row = $this->db->sql_fetchrow($result); |
| 85 | 85 | $this->db->sql_freeresult($result); |
@@ -108,8 +108,8 @@ discard block |
||
| 108 | 108 | $this->items_table => 'i' |
| 109 | 109 | ), |
| 110 | 110 | 'WHERE' => array( |
| 111 | - 'i.depth ' . (($max_depth) ? " BETWEEN $start AND " . ($start + $max_depth) : ' >= ' . $start), |
|
| 112 | - (($this->sql_where) ? 'i.' . $this->sql_where : ''), |
|
| 111 | + 'i.depth '.(($max_depth) ? " BETWEEN $start AND ".($start + $max_depth) : ' >= '.$start), |
|
| 112 | + (($this->sql_where) ? 'i.'.$this->sql_where : ''), |
|
| 113 | 113 | ), |
| 114 | 114 | 'ORDER_BY' => 'i.left_id ASC', |
| 115 | 115 | ), |
@@ -155,7 +155,7 @@ discard block |
||
| 155 | 155 | * @param \phpbb\template\twig\twig $template |
| 156 | 156 | * @param string $handle |
| 157 | 157 | */ |
| 158 | - public function display_list(array $data, \phpbb\template\twig\twig &$template, $handle = 'tree') |
|
| 158 | + public function display_list(array $data, \phpbb\template\twig\twig & $template, $handle = 'tree') |
|
| 159 | 159 | { |
| 160 | 160 | $prev_depth = 0; |
| 161 | 161 | $parental_depth = array(0 => -1); |
@@ -164,10 +164,10 @@ discard block |
||
| 164 | 164 | for ($i = 0, $size = sizeof($data); $i < $size; $i++) |
| 165 | 165 | { |
| 166 | 166 | $row = $data[$i]; |
| 167 | - $this_depth = $parental_depth[$row['parent_id']] + 1; |
|
| 168 | - $repeat = abs($prev_depth - $this_depth); |
|
| 167 | + $this_depth = $parental_depth[$row['parent_id']] + 1; |
|
| 168 | + $repeat = abs($prev_depth - $this_depth); |
|
| 169 | 169 | |
| 170 | - $tpl_data = array( |
|
| 170 | + $tpl_data = array( |
|
| 171 | 171 | 'PREV_DEPTH' => $prev_depth, |
| 172 | 172 | 'THIS_DEPTH' => $this_depth, |
| 173 | 173 | 'NUM_KIDS' => $this->count_descendants($row), |
@@ -177,7 +177,7 @@ discard block |
||
| 177 | 177 | |
| 178 | 178 | for ($j = 0; $j < $repeat; $j++) |
| 179 | 179 | { |
| 180 | - $template->assign_block_vars($handle . '.close', array()); |
|
| 180 | + $template->assign_block_vars($handle.'.close', array()); |
|
| 181 | 181 | } |
| 182 | 182 | |
| 183 | 183 | $prev_depth = $this_depth; |
@@ -186,7 +186,7 @@ discard block |
||
| 186 | 186 | |
| 187 | 187 | for ($i = 0; $i < $prev_depth; $i++) |
| 188 | 188 | { |
| 189 | - $template->assign_block_vars('close_' . $handle, array()); |
|
| 189 | + $template->assign_block_vars('close_'.$handle, array()); |
|
| 190 | 190 | } |
| 191 | 191 | } |
| 192 | 192 | |
@@ -225,10 +225,10 @@ discard block |
||
| 225 | 225 | } |
| 226 | 226 | |
| 227 | 227 | $right = $row['right_id']; |
| 228 | - $title = $padding . '├─ ' . $row[$title_column]; |
|
| 228 | + $title = $padding.'├─ '.$row[$title_column]; |
|
| 229 | 229 | $selected = (in_array($row[$this->pk], $selected_ids)) ? ' selected="selected' : ''; |
| 230 | 230 | |
| 231 | - $return_options .= '<option value="' . $row[$this->pk] . '"' . $selected . '>' . $title . '</option>'; |
|
| 231 | + $return_options .= '<option value="'.$row[$this->pk].'"'.$selected.'>'.$title.'</option>'; |
|
| 232 | 232 | $return_data[$row[$this->pk]] = $title; |
| 233 | 233 | } |
| 234 | 234 | |
@@ -91,8 +91,8 @@ discard block |
||
| 91 | 91 | */ |
| 92 | 92 | private function clean_styles() |
| 93 | 93 | { |
| 94 | - $routes_ary = $this->manager->get_routes_per_style(); |
|
| 95 | - $style_ids = $this->get_style_ids(); |
|
| 94 | + $routes_ary = $this->manager->get_routes_per_style(); |
|
| 95 | + $style_ids = $this->get_style_ids(); |
|
| 96 | 96 | |
| 97 | 97 | $routes = array(); |
| 98 | 98 | foreach ($routes_ary as $style_id => $style_routes) |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | |
| 123 | 123 | foreach ($routes as $route => $row) |
| 124 | 124 | { |
| 125 | - $url = $board_url . '/' . (($row['ext_name']) ? 'app.php' : '') . $row['route']; |
|
| 125 | + $url = $board_url.'/'.(($row['ext_name']) ? 'app.php' : '').$row['route']; |
|
| 126 | 126 | |
| 127 | 127 | // Route no longer exists => remove all blocks for route |
| 128 | 128 | if ($this->url_checker->exists($url) !== true) |
@@ -185,7 +185,7 @@ discard block |
||
| 185 | 185 | |
| 186 | 186 | if (sizeof($block_ids)) |
| 187 | 187 | { |
| 188 | - $this->db->sql_query('DELETE FROM ' . $this->cblocks_table . ' WHERE ' . $this->db->sql_in_set('block_id', $block_ids)); |
|
| 188 | + $this->db->sql_query('DELETE FROM '.$this->cblocks_table.' WHERE '.$this->db->sql_in_set('block_id', $block_ids)); |
|
| 189 | 189 | } |
| 190 | 190 | } |
| 191 | 191 | |
@@ -56,8 +56,8 @@ discard block |
||
| 56 | 56 | } |
| 57 | 57 | |
| 58 | 58 | /** |
| 59 | - * {@inheritdoc} |
|
| 60 | - */ |
|
| 59 | + * {@inheritdoc} |
|
| 60 | + */ |
|
| 61 | 61 | public function to_array() |
| 62 | 62 | { |
| 63 | 63 | $attributes = $this->get_attributes(); |
@@ -74,8 +74,8 @@ discard block |
||
| 74 | 74 | } |
| 75 | 75 | |
| 76 | 76 | /** |
| 77 | - * {@inheritdoc} |
|
| 78 | - */ |
|
| 77 | + * {@inheritdoc} |
|
| 78 | + */ |
|
| 79 | 79 | public function to_db() |
| 80 | 80 | { |
| 81 | 81 | $this->check_required(); |
@@ -25,7 +25,7 @@ discard block |
||
| 25 | 25 | { |
| 26 | 26 | foreach ($data as $name => $value) |
| 27 | 27 | { |
| 28 | - $mutator = 'set_' . $name; |
|
| 28 | + $mutator = 'set_'.$name; |
|
| 29 | 29 | $this->$mutator($value); |
| 30 | 30 | } |
| 31 | 31 | } |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | $data = array(); |
| 66 | 66 | foreach ($attributes as $attribute) |
| 67 | 67 | { |
| 68 | - $accessor = 'get_' . $attribute; |
|
| 68 | + $accessor = 'get_'.$attribute; |
|
| 69 | 69 | $data[$attribute] = $this->$accessor(); |
| 70 | 70 | } |
| 71 | 71 | unset($data['db_fields'], $data['required_fields']); |
@@ -85,7 +85,7 @@ discard block |
||
| 85 | 85 | */ |
| 86 | 86 | public function save(\blitze\sitemaker\model\entity_interface $entity) |
| 87 | 87 | { |
| 88 | - $accessor = 'get_' . $this->entity_pkey; |
|
| 88 | + $accessor = 'get_'.$this->entity_pkey; |
|
| 89 | 89 | if (is_null($entity->$accessor())) |
| 90 | 90 | { |
| 91 | 91 | $entity = $this->insert($entity); |
@@ -107,7 +107,7 @@ discard block |
||
| 107 | 107 | { |
| 108 | 108 | if ($condition instanceof $this->entity_class) |
| 109 | 109 | { |
| 110 | - $accessor = 'get_' . $this->entity_pkey; |
|
| 110 | + $accessor = 'get_'.$this->entity_pkey; |
|
| 111 | 111 | $condition = array($this->entity_pkey, '=', $condition->$accessor()); |
| 112 | 112 | } |
| 113 | 113 | else |
@@ -117,7 +117,7 @@ discard block |
||
| 117 | 117 | } |
| 118 | 118 | |
| 119 | 119 | $sql_where = $this->get_sql_condition($condition); |
| 120 | - $this->db->sql_query('DELETE FROM ' . $this->entity_table . (sizeof($sql_where) ? ' WHERE ' . join(' AND ', $sql_where) : '')); |
|
| 120 | + $this->db->sql_query('DELETE FROM '.$this->entity_table.(sizeof($sql_where) ? ' WHERE '.join(' AND ', $sql_where) : '')); |
|
| 121 | 121 | } |
| 122 | 122 | |
| 123 | 123 | /** |
@@ -136,9 +136,9 @@ discard block |
||
| 136 | 136 | */ |
| 137 | 137 | protected function insert(\blitze\sitemaker\model\entity_interface $entity) |
| 138 | 138 | { |
| 139 | - $this->db->sql_query('INSERT INTO ' . $this->entity_table . ' ' . $this->db->sql_build_array('INSERT', $entity->to_db())); |
|
| 139 | + $this->db->sql_query('INSERT INTO '.$this->entity_table.' '.$this->db->sql_build_array('INSERT', $entity->to_db())); |
|
| 140 | 140 | |
| 141 | - $mutator = 'set_' . $this->entity_pkey; |
|
| 141 | + $mutator = 'set_'.$this->entity_pkey; |
|
| 142 | 142 | $entity->$mutator((int) $this->db->sql_nextid()); |
| 143 | 143 | |
| 144 | 144 | return $entity; |
@@ -152,11 +152,11 @@ discard block |
||
| 152 | 152 | */ |
| 153 | 153 | protected function update(\blitze\sitemaker\model\entity_interface $entity) |
| 154 | 154 | { |
| 155 | - $accessor = 'get_' . $this->entity_pkey; |
|
| 155 | + $accessor = 'get_'.$this->entity_pkey; |
|
| 156 | 156 | |
| 157 | - return $this->db->sql_query('UPDATE ' . $this->entity_table . ' |
|
| 158 | - SET ' . $this->db->sql_build_array('UPDATE', $entity->to_db()) . ' |
|
| 159 | - WHERE ' . $this->entity_pkey . ' = ' . (int) $entity->$accessor()); |
|
| 157 | + return $this->db->sql_query('UPDATE '.$this->entity_table.' |
|
| 158 | + SET ' . $this->db->sql_build_array('UPDATE', $entity->to_db()).' |
|
| 159 | + WHERE ' . $this->entity_pkey.' = '.(int) $entity->$accessor()); |
|
| 160 | 160 | } |
| 161 | 161 | |
| 162 | 162 | /** |
@@ -165,8 +165,8 @@ discard block |
||
| 165 | 165 | */ |
| 166 | 166 | protected function find_sql(array $sql_where) |
| 167 | 167 | { |
| 168 | - return 'SELECT * FROM ' . $this->entity_table . |
|
| 169 | - (sizeof($sql_where) ? ' WHERE ' . join(' AND ', $sql_where) : ''); |
|
| 168 | + return 'SELECT * FROM '.$this->entity_table. |
|
| 169 | + (sizeof($sql_where) ? ' WHERE '.join(' AND ', $sql_where) : ''); |
|
| 170 | 170 | } |
| 171 | 171 | |
| 172 | 172 | /** |
@@ -182,7 +182,7 @@ discard block |
||
| 182 | 182 | { |
| 183 | 183 | list($field, $operator, $value) = $info; |
| 184 | 184 | |
| 185 | - $callable = 'get_sql_where_' . gettype($value); |
|
| 185 | + $callable = 'get_sql_where_'.gettype($value); |
|
| 186 | 186 | if (is_callable(array($this, $callable))) |
| 187 | 187 | { |
| 188 | 188 | $sql_where[] = call_user_func_array(array($this, $callable), array($field, $value, $operator)); |
@@ -220,7 +220,7 @@ discard block |
||
| 220 | 220 | */ |
| 221 | 221 | protected function get_sql_where_string($field, $value, $operator) |
| 222 | 222 | { |
| 223 | - return $field . " $operator '" . $this->db->sql_escape($value) . "'"; |
|
| 223 | + return $field." $operator '".$this->db->sql_escape($value)."'"; |
|
| 224 | 224 | } |
| 225 | 225 | |
| 226 | 226 | /** |
@@ -231,7 +231,7 @@ discard block |
||
| 231 | 231 | */ |
| 232 | 232 | protected function get_sql_where_integer($field, $value, $operator) |
| 233 | 233 | { |
| 234 | - return $field . " $operator " . (int) $value; |
|
| 234 | + return $field." $operator ".(int) $value; |
|
| 235 | 235 | } |
| 236 | 236 | |
| 237 | 237 | /** |
@@ -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 | |
@@ -132,7 +132,7 @@ discard block |
||
| 132 | 132 | $post_id = (int) $topic_data['topic_first_post_id']; |
| 133 | 133 | |
| 134 | 134 | $sql = 'SELECT o.*, p.bbcode_bitfield, p.bbcode_uid |
| 135 | - FROM ' . POLL_OPTIONS_TABLE . ' o, ' . POSTS_TABLE . " p |
|
| 135 | + FROM ' . POLL_OPTIONS_TABLE.' o, '.POSTS_TABLE." p |
|
| 136 | 136 | WHERE o.topic_id = $topic_id |
| 137 | 137 | AND p.post_id = $post_id |
| 138 | 138 | AND p.topic_id = o.topic_id |
@@ -177,7 +177,7 @@ discard block |
||
| 177 | 177 | * @param int $poll_most |
| 178 | 178 | * @param \phpbb\template\twig\twig $template |
| 179 | 179 | */ |
| 180 | - private function _build_poll_options(array $cur_voted_id, array $poll_info, $poll_total, $poll_most, \phpbb\template\twig\twig &$template) |
|
| 180 | + private function _build_poll_options(array $cur_voted_id, array $poll_info, $poll_total, $poll_most, \phpbb\template\twig\twig & $template) |
|
| 181 | 181 | { |
| 182 | 182 | foreach ($poll_info as $poll_option) |
| 183 | 183 | { |
@@ -208,8 +208,8 @@ discard block |
||
| 208 | 208 | if ($this->user->data['is_registered']) |
| 209 | 209 | { |
| 210 | 210 | $sql = 'SELECT poll_option_id |
| 211 | - FROM ' . POLL_VOTES_TABLE . ' |
|
| 212 | - WHERE topic_id = ' . (int) $topic_id . ' |
|
| 211 | + FROM ' . POLL_VOTES_TABLE.' |
|
| 212 | + WHERE topic_id = ' . (int) $topic_id.' |
|
| 213 | 213 | AND vote_user_id = ' . $this->user->data['user_id']; |
| 214 | 214 | $result = $this->db->sql_query($sql); |
| 215 | 215 | |
@@ -224,9 +224,9 @@ discard block |
||
| 224 | 224 | // Cookie based guest tracking ... I don't like this but hum ho |
| 225 | 225 | // it's oft requested. This relies on "nice" users who don't feel |
| 226 | 226 | // the need to delete cookies to mess with results. |
| 227 | - if ($this->request->is_set($this->config['cookie_name'] . '_poll_' . $topic_id, \phpbb\request\request_interface::COOKIE)) |
|
| 227 | + if ($this->request->is_set($this->config['cookie_name'].'_poll_'.$topic_id, \phpbb\request\request_interface::COOKIE)) |
|
| 228 | 228 | { |
| 229 | - $cur_voted_id = explode(',', $this->request->variable($this->config['cookie_name'] . '_poll_' . $topic_id, '', true, \phpbb\request\request_interface::COOKIE)); |
|
| 229 | + $cur_voted_id = explode(',', $this->request->variable($this->config['cookie_name'].'_poll_'.$topic_id, '', true, \phpbb\request\request_interface::COOKIE)); |
|
| 230 | 230 | $cur_voted_id = array_map('intval', $cur_voted_id); |
| 231 | 231 | } |
| 232 | 232 | } |