@@ -142,8 +142,7 @@ discard block |
||
| 142 | 142 | try |
| 143 | 143 | { |
| 144 | 144 | $this->loader->load($album_id); |
| 145 | - } |
|
| 146 | - catch (\Exception $e) |
|
| 145 | + } catch (\Exception $e) |
|
| 147 | 146 | { |
| 148 | 147 | throw new \phpbb\exception\http_exception(404, 'ALBUM_NOT_EXIST'); |
| 149 | 148 | } |
@@ -203,8 +202,7 @@ discard block |
||
| 203 | 202 | 'phpbbgallery_core_album_upload', |
| 204 | 203 | array('album_id' => (int) $album_id) |
| 205 | 204 | )); |
| 206 | - } |
|
| 207 | - else |
|
| 205 | + } else |
|
| 208 | 206 | { |
| 209 | 207 | if ($album_data['contest_start'] + $album_data['contest_rating'] > time()) |
| 210 | 208 | { |
@@ -261,8 +259,7 @@ discard block |
||
| 261 | 259 | { |
| 262 | 260 | $image_status_check = ''; |
| 263 | 261 | $image_counter = $album_data['album_images_real']; |
| 264 | - } |
|
| 265 | - else |
|
| 262 | + } else |
|
| 266 | 263 | { |
| 267 | 264 | $image_status_check = " AND (image_status <> " . (int) \phpbbgallery\core\block::STATUS_UNAPPROVED . " OR image_user_id = $user_id)"; |
| 268 | 265 | |
@@ -279,8 +276,7 @@ discard block |
||
| 279 | 276 | if (in_array($sort_key, array('r', 'ra'))) |
| 280 | 277 | { |
| 281 | 278 | $sql_help_sort = ', image_id ' . (($sort_dir == 'd') ? 'ASC' : 'DESC'); |
| 282 | - } |
|
| 283 | - else |
|
| 279 | + } else |
|
| 284 | 280 | { |
| 285 | 281 | $sql_help_sort = ', image_id ' . (($sort_dir == 'd') ? 'DESC' : 'ASC'); |
| 286 | 282 | } |
@@ -471,8 +467,7 @@ discard block |
||
| 471 | 467 | )); |
| 472 | 468 | $this->url->meta_refresh(3, $back_link); |
| 473 | 469 | return $this->helper->render('gallery/message.html', $this->language->lang('GALLERY')); |
| 474 | - } |
|
| 475 | - else |
|
| 470 | + } else |
|
| 476 | 471 | { |
| 477 | 472 | $this->notifications_helper->add_albums($album_id); |
| 478 | 473 | $this->template->assign_vars(array( |
@@ -481,14 +476,12 @@ discard block |
||
| 481 | 476 | $this->url->meta_refresh(3, $back_link); |
| 482 | 477 | return $this->helper->render('gallery/message.html', $this->language->lang('GALLERY')); |
| 483 | 478 | } |
| 484 | - } |
|
| 485 | - else |
|
| 479 | + } else |
|
| 486 | 480 | { |
| 487 | 481 | if ($this->notifications_helper->get_watched_album($album_id) == 1) |
| 488 | 482 | { |
| 489 | 483 | $lang = $this->language->lang('UNWATCH_ALBUM'); |
| 490 | - } |
|
| 491 | - else |
|
| 484 | + } else |
|
| 492 | 485 | { |
| 493 | 486 | $lang = $this->language->lang('WATCH_ALBUM'); |
| 494 | 487 | } |
@@ -520,8 +513,7 @@ discard block |
||
| 520 | 513 | if (!$this->user->data['is_registered']) |
| 521 | 514 | { |
| 522 | 515 | login_box(); |
| 523 | - } |
|
| 524 | - else |
|
| 516 | + } else |
|
| 525 | 517 | { |
| 526 | 518 | //return $this->error('NOT_AUTHORISED', 403); |
| 527 | 519 | trigger_error($this->language->lang('NOT_AUTHORISED')); |
@@ -354,7 +354,7 @@ discard block |
||
| 354 | 354 | $album_user_id = $image_data['album_user_id']; |
| 355 | 355 | |
| 356 | 356 | //@todo: $rating = new phpbb_gallery_image_rating($image_data['image_id'], $image_data, $image_data); |
| 357 | - $image_data['rating'] = '0';//@todo: $rating->get_image_rating(false, false); |
|
| 357 | + $image_data['rating'] = '0'; //@todo: $rating->get_image_rating(false, false); |
|
| 358 | 358 | //@todo: unset($rating); |
| 359 | 359 | |
| 360 | 360 | $s_user_allowed = (($image_data['image_user_id'] == $this->user->data['user_id']) && ($album_status != (int) \phpbbgallery\core\block::ALBUM_LOCKED)); |
@@ -412,14 +412,14 @@ discard block |
||
| 412 | 412 | |
| 413 | 413 | 'U_USER_IP' => $show_ip && $this->auth->acl_check('m_status', $image_data['image_album_id'], $album_user_id) ? $image_data['image_user_ip'] : false, |
| 414 | 414 | 'S_IMAGE_REPORTED' => $image_data['image_reported'], |
| 415 | - 'U_IMAGE_REPORTED' => '',//($image_data['image_reported']) ? $phpbb_ext_gallery->url->append_sid('mcp', "mode=report_details&album_id={$image_data['image_album_id']}&option_id=" . $image_data['image_reported']) : '', |
|
| 415 | + 'U_IMAGE_REPORTED' => '', //($image_data['image_reported']) ? $phpbb_ext_gallery->url->append_sid('mcp', "mode=report_details&album_id={$image_data['image_album_id']}&option_id=" . $image_data['image_reported']) : '', |
|
| 416 | 416 | 'S_STATUS_APPROVED' => ($image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_APPROVED) ? true : false, |
| 417 | 417 | 'S_STATUS_UNAPPROVED' => ($image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_UNAPPROVED) ? true : false, |
| 418 | 418 | 'S_STATUS_UNAPPROVED_ACTION' => ($this->auth->acl_check('m_status', $image_data['image_album_id'], $album_user_id) && $image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_UNAPPROVED) ? $this->helper->route('phpbbgallery_core_moderate_image_approve', array('image_id' => $image_data['image_id'])) : '', |
| 419 | 419 | 'S_STATUS_LOCKED' => ($image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_LOCKED) ? true : false, |
| 420 | 420 | |
| 421 | 421 | 'U_REPORT' => ($this->auth->acl_check('m_report', $image_data['image_album_id'], $album_user_id) && $image_data['image_reported']) ? '123'/*$this->url->append_sid('mcp', "mode=report_details&album_id={$image_data['image_album_id']}&option_id=" . $image_data['image_reported'])*/ : '', |
| 422 | - 'U_STATUS' => '',//($this->auth->acl_check('m_status', $image_data['image_album_id'], $album_user_id)) ? $phpbb_ext_gallery->url->append_sid('mcp', "mode=queue_details&album_id={$image_data['image_album_id']}&option_id=" . $image_data['image_id']) : '', |
|
| 422 | + 'U_STATUS' => '', //($this->auth->acl_check('m_status', $image_data['image_album_id'], $album_user_id)) ? $phpbb_ext_gallery->url->append_sid('mcp', "mode=queue_details&album_id={$image_data['image_album_id']}&option_id=" . $image_data['image_id']) : '', |
|
| 423 | 423 | 'L_STATUS' => ($image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_UNAPPROVED) ? $this->language->lang('APPROVE_IMAGE') : (($image_data['image_status'] == (int) \phpbbgallery\core\block::STATUS_APPROVED) ? $this->language->lang('CHANGE_IMAGE_STATUS') : $this->language->lang('UNLOCK_IMAGE')), |
| 424 | 424 | |
| 425 | 425 | 'S_CONTEST_RANK' => $image_data['image_contest_rank'], |
@@ -221,8 +221,7 @@ discard block |
||
| 221 | 221 | { |
| 222 | 222 | $padding .= ' '; |
| 223 | 223 | $padding_store[$row['parent_id']] = $padding; |
| 224 | - } |
|
| 225 | - else if ($row['left_id'] > $right + 1) |
|
| 224 | + } else if ($row['left_id'] > $right + 1) |
|
| 226 | 225 | { |
| 227 | 226 | $padding = (isset($padding_store[$row['parent_id']])) ? $padding_store[$row['parent_id']] : ''; |
| 228 | 227 | } |
@@ -248,15 +247,13 @@ discard block |
||
| 248 | 247 | if (($select_id == $this->gallery_auth->get_setting_permissions()) && !$row['album_user_id']) |
| 249 | 248 | { |
| 250 | 249 | $list = true; |
| 251 | - } |
|
| 252 | - else if (!$row['album_user_id']) |
|
| 250 | + } else if (!$row['album_user_id']) |
|
| 253 | 251 | { |
| 254 | 252 | if ($this->gallery_auth->acl_check('a_list', $row['album_id'], $row['album_user_id']) || defined('IN_ADMIN')) |
| 255 | 253 | { |
| 256 | 254 | $list = true; |
| 257 | 255 | } |
| 258 | - } |
|
| 259 | - else if (!$ignore_personals) |
|
| 256 | + } else if (!$ignore_personals) |
|
| 260 | 257 | { |
| 261 | 258 | if ($row['album_user_id'] == $this->user->data['user_id']) |
| 262 | 259 | { |
@@ -267,16 +264,14 @@ discard block |
||
| 267 | 264 | if ($requested_permission) |
| 268 | 265 | { |
| 269 | 266 | $requested_own = !$this->gallery_auth->acl_check($requested_permission, $this->gallery_auth->get_own_album()); |
| 270 | - } |
|
| 271 | - else |
|
| 267 | + } else |
|
| 272 | 268 | { |
| 273 | 269 | $requested_own = false; // We need the negated version of true here |
| 274 | 270 | } |
| 275 | 271 | } |
| 276 | 272 | $list = (!$list) ? $access_own : $list; |
| 277 | 273 | $disabled = (!$disabled) ? $requested_own : $disabled; |
| 278 | - } |
|
| 279 | - else if ($row['album_user_id']) |
|
| 274 | + } else if ($row['album_user_id']) |
|
| 280 | 275 | { |
| 281 | 276 | if (!$c_access_personal) |
| 282 | 277 | { |
@@ -285,8 +280,7 @@ discard block |
||
| 285 | 280 | if ($requested_permission) |
| 286 | 281 | { |
| 287 | 282 | $requested_personal = !$this->gallery_auth->acl_check($requested_permission, $this->gallery_auth->get_personal_album()); |
| 288 | - } |
|
| 289 | - else |
|
| 283 | + } else |
|
| 290 | 284 | { |
| 291 | 285 | $requested_personal = false; // We need the negated version of true here |
| 292 | 286 | } |
@@ -298,8 +292,7 @@ discard block |
||
| 298 | 292 | if (($album_user_id != (int) \phpbbgallery\core\block::PUBLIC_ALBUM) && ($album_user_id != $row['album_user_id'])) |
| 299 | 293 | { |
| 300 | 294 | $list = false; |
| 301 | - } |
|
| 302 | - else if (($album_user_id != (int) \phpbbgallery\core\block::PUBLIC_ALBUM) && ($row['parent_id'] == 0)) |
|
| 295 | + } else if (($album_user_id != (int) \phpbbgallery\core\block::PUBLIC_ALBUM) && ($row['parent_id'] == 0)) |
|
| 303 | 296 | { |
| 304 | 297 | $disabled = true; |
| 305 | 298 | } |
@@ -317,8 +310,7 @@ discard block |
||
| 317 | 310 | $gallery_albumbox = "<select name='$select_name' id='$select_name'>"; |
| 318 | 311 | $gallery_albumbox .= $album_list; |
| 319 | 312 | $gallery_albumbox .= '</select>'; |
| 320 | - } |
|
| 321 | - else |
|
| 313 | + } else |
|
| 322 | 314 | { |
| 323 | 315 | $gallery_albumbox = $album_list; |
| 324 | 316 | } |
@@ -387,8 +379,7 @@ discard block |
||
| 387 | 379 | 'album_last_user_colour' => $row['image_user_colour'], |
| 388 | 380 | 'album_last_user_id' => $row['image_user_id'], |
| 389 | 381 | ); |
| 390 | - } |
|
| 391 | - else |
|
| 382 | + } else |
|
| 392 | 383 | { |
| 393 | 384 | // No approved image, so we clear the columns |
| 394 | 385 | $sql_ary = array( |
@@ -250,8 +250,7 @@ discard block |
||
| 250 | 250 | } |
| 251 | 251 | |
| 252 | 252 | $this->loader->load($this->data['image_album_id']); |
| 253 | - } |
|
| 254 | - catch (\Exception $e) |
|
| 253 | + } catch (\Exception $e) |
|
| 255 | 254 | { |
| 256 | 255 | throw new \phpbb\exception\http_exception(404, 'INVALID_IMAGE'); |
| 257 | 256 | } |
@@ -264,8 +263,7 @@ discard block |
||
| 264 | 263 | if ($this->gallery_auth->acl_check('m_status', $album_id, $album_data['album_user_id'])) |
| 265 | 264 | { |
| 266 | 265 | $image_status_check = ''; |
| 267 | - } |
|
| 268 | - else |
|
| 266 | + } else |
|
| 269 | 267 | { |
| 270 | 268 | $user_id = (int) $this->user->data['user_id']; |
| 271 | 269 | $image_status_check = ' AND (image_status = ' . (int) \phpbbgallery\core\block::STATUS_APPROVED . ' OR image_user_id = ' . $user_id . ')'; |
@@ -322,8 +320,7 @@ discard block |
||
| 322 | 320 | if (in_array($sort_key, array('r', 'ra'))) |
| 323 | 321 | { |
| 324 | 322 | $sql_help_sort = ', image_id ' . (($sort_dir == 'd') ? 'ASC' : 'DESC'); |
| 325 | - } |
|
| 326 | - else |
|
| 323 | + } else |
|
| 327 | 324 | { |
| 328 | 325 | $sql_help_sort = ', image_id ' . (($sort_dir == 'd') ? 'DESC' : 'ASC'); |
| 329 | 326 | } |
@@ -551,8 +548,7 @@ discard block |
||
| 551 | 548 | if (isset($album_data['contest_start'])) |
| 552 | 549 | { |
| 553 | 550 | $s_hide_comment_input = (time() < ($album_data['contest_start'] + $album_data['contest_end'])) ? true : false; |
| 554 | - } |
|
| 555 | - else |
|
| 551 | + } else |
|
| 556 | 552 | { |
| 557 | 553 | $s_hide_comment_input = false; |
| 558 | 554 | } |
@@ -598,8 +594,7 @@ discard block |
||
| 598 | 594 | //$this->template->assign_var('S_COMMENT_ACTION', append_sid($this->url->path('full') . 'comment/' . $image_id . '/add/0')); |
| 599 | 595 | $this->template->assign_var('S_COMMENT_ACTION', $this->helper->route('phpbbgallery_core_comment_add', array('image_id' => $image_id, 'comment_id' => 0))); |
| 600 | 596 | } |
| 601 | - } |
|
| 602 | - else if ($this->gallery_config->get('comment_user_control') && !$image_data['image_allow_comments']) |
|
| 597 | + } else if ($this->gallery_config->get('comment_user_control') && !$image_data['image_allow_comments']) |
|
| 603 | 598 | { |
| 604 | 599 | $this->template->assign_var('S_COMMENTS_DISABLED', true); |
| 605 | 600 | } |
@@ -790,8 +785,7 @@ discard block |
||
| 790 | 785 | 'NAME' => $field_data['PROFILE_FIELD_NAME'], |
| 791 | 786 | 'U_CONTACT' => $field_data['PROFILE_FIELD_CONTACT'], |
| 792 | 787 | )); |
| 793 | - } |
|
| 794 | - else |
|
| 788 | + } else |
|
| 795 | 789 | { |
| 796 | 790 | $this->template->assign_block_vars('commentrow.custom_fields', $field_data); |
| 797 | 791 | } |
@@ -917,8 +911,7 @@ discard block |
||
| 917 | 911 | { |
| 918 | 912 | $change_image_count = true; |
| 919 | 913 | } |
| 920 | - } |
|
| 921 | - else if ($this->request->variable('change_author', '', true)) |
|
| 914 | + } else if ($this->request->variable('change_author', '', true)) |
|
| 922 | 915 | { |
| 923 | 916 | $errors[] = $this->language->lang('INVALID_USERNAME'); |
| 924 | 917 | } |
@@ -939,8 +932,7 @@ discard block |
||
| 939 | 932 | { |
| 940 | 933 | $personal_album_id = $this->album->generate_personal_album($image_data['image_username'], $image_data['image_user_id'], $image_data['image_user_colour'], $image_user); |
| 941 | 934 | } |
| 942 | - } |
|
| 943 | - else |
|
| 935 | + } else |
|
| 944 | 936 | { |
| 945 | 937 | $personal_album_id = $this->gallery_user->get_data('personal_album_id'); |
| 946 | 938 | if (!$personal_album_id && $this->gallery_auth->acl_check('i_upload', $this->gallery_auth->get_own_album())) |
@@ -1108,8 +1100,7 @@ discard block |
||
| 1108 | 1100 | $this->notification_helper->read('approval', $album_id); |
| 1109 | 1101 | $this->url->meta_refresh(3, $album_backlink); |
| 1110 | 1102 | trigger_error($message); |
| 1111 | - } |
|
| 1112 | - else |
|
| 1103 | + } else |
|
| 1113 | 1104 | { |
| 1114 | 1105 | if (isset($_POST['cancel'])) |
| 1115 | 1106 | { |
@@ -1117,8 +1108,7 @@ discard block |
||
| 1117 | 1108 | $message .= '<br />' . sprintf($this->language->lang('CLICK_RETURN_IMAGE'), '<a href="' . $image_backlink . '">', '</a>'); |
| 1118 | 1109 | $this->url->meta_refresh(3, $image_backlink); |
| 1119 | 1110 | trigger_error($message); |
| 1120 | - } |
|
| 1121 | - else |
|
| 1111 | + } else |
|
| 1122 | 1112 | { |
| 1123 | 1113 | confirm_box(false, 'DELETE_IMAGE2', $s_hidden_fields); |
| 1124 | 1114 | } |
@@ -1223,8 +1213,7 @@ discard block |
||
| 1223 | 1213 | { |
| 1224 | 1214 | // @todo Add "redirect after login" url |
| 1225 | 1215 | login_box(); |
| 1226 | - } |
|
| 1227 | - else |
|
| 1216 | + } else |
|
| 1228 | 1217 | { |
| 1229 | 1218 | //return $this->error('NOT_AUTHORISED', 403); |
| 1230 | 1219 | redirect('gallery/album/' . $album_id); |