| @@ -146,8 +146,7 @@ | ||
| 146 | 146 | |
| 147 | 147 | $event['additional_sql_data'] = $additional_sql_data; | 
| 148 | 148 | unset($exif); | 
| 149 | - } | |
| 150 | - else | |
| 149 | + } else | |
| 151 | 150 |  		{ | 
| 152 | 151 | $additional_sql_data = $event['additional_sql_data']; | 
| 153 | 152 | $additional_sql_data['image_exif_data'] = ''; | 
| @@ -54,16 +54,16 @@ | ||
| 54 | 54 | } | 
| 55 | 55 | |
| 56 | 56 | /** | 
| 57 | - * Constructor | |
| 58 | - * NOTE: The parameters of this method must match in order and type with | |
| 59 | - * the dependencies defined in the services.yml file for this service. | |
| 60 | - * | |
| 61 | - * @param \phpbb\user $user User object | |
| 62 | - * @param \phpbbgallery\core\config $gallery_config Core gallery config object | |
| 63 | - * @param \phpbbgallery\core\auth\auth $gallery_auth Core gallery auth object | |
| 64 | - * @param \phpbbgallery\core\url $gallery_url Core gallery url object | |
| 65 | - * @param \phpbbgallery\core\user $gallery_user Core gallery user wrapper | |
| 66 | - */ | |
| 57 | + * Constructor | |
| 58 | + * NOTE: The parameters of this method must match in order and type with | |
| 59 | + * the dependencies defined in the services.yml file for this service. | |
| 60 | + * | |
| 61 | + * @param \phpbb\user $user User object | |
| 62 | + * @param \phpbbgallery\core\config $gallery_config Core gallery config object | |
| 63 | + * @param \phpbbgallery\core\auth\auth $gallery_auth Core gallery auth object | |
| 64 | + * @param \phpbbgallery\core\url $gallery_url Core gallery url object | |
| 65 | + * @param \phpbbgallery\core\user $gallery_user Core gallery user wrapper | |
| 66 | + */ | |
| 67 | 67 | |
| 68 | 68 | public function __construct(\phpbb\user $user, \phpbbgallery\core\config $gallery_config, \phpbbgallery\core\auth\auth $gallery_auth, \phpbbgallery\core\url $gallery_url, \phpbbgallery\core\user $gallery_user) | 
| 69 | 69 |  	{ | 
| @@ -70,7 +70,7 @@ discard block | ||
| 70 | 70 | $this->user = $user; | 
| 71 | 71 | $this->gallery_config = $gallery_config; | 
| 72 | 72 | $this->gallery_auth = $gallery_auth; | 
| 73 | - $this->gallery_url = $gallery_url; | |
| 73 | + $this->gallery_url = $gallery_url; | |
| 74 | 74 | $this->gallery_user = $gallery_user; | 
| 75 | 75 | } | 
| 76 | 76 | |
| @@ -83,7 +83,7 @@ discard block | ||
| 83 | 83 |  			{ | 
| 84 | 84 |  				$this->user->add_lang_ext('phpbbgallery/exif', 'info_exif'); | 
| 85 | 85 | |
| 86 | -				$return_ary['vars']['IMAGE_SETTINGS']['disp_exifdata'] = array('lang' => 'DISP_EXIF_DATA',		'validate' => 'bool',	'type' => 'radio:yes_no'); | |
| 86 | +				$return_ary['vars']['IMAGE_SETTINGS']['disp_exifdata'] = array('lang' => 'DISP_EXIF_DATA', 'validate' => 'bool', 'type' => 'radio:yes_no'); | |
| 87 | 87 | $event['return_ary'] = $return_ary; | 
| 88 | 88 | } | 
| 89 | 89 | } | 
| @@ -140,10 +140,10 @@ discard block | ||
| 140 | 140 | } | 
| 141 | 141 | |
| 142 | 142 | /** | 
| 143 | - * Delete reports for given report_ids | |
| 144 | - * | |
| 145 | - * @param mixed $report_ids Array or integer with report_id. | |
| 146 | - */ | |
| 143 | + * Delete reports for given report_ids | |
| 144 | + * | |
| 145 | + * @param mixed $report_ids Array or integer with report_id. | |
| 146 | + */ | |
| 147 | 147 | public function delete($report_ids) | 
| 148 | 148 |  	{ | 
| 149 | 149 | $report_ids = self::cast_mixed_int2array($report_ids); | 
| @@ -163,10 +163,10 @@ discard block | ||
| 163 | 163 | |
| 164 | 164 | |
| 165 | 165 | /** | 
| 166 | - * Delete reports for given image_ids | |
| 167 | - * | |
| 168 | - * @param mixed $image_ids Array or integer with image_id. | |
| 169 | - */ | |
| 166 | + * Delete reports for given image_ids | |
| 167 | + * | |
| 168 | + * @param mixed $image_ids Array or integer with image_id. | |
| 169 | + */ | |
| 170 | 170 | public function delete_images($image_ids) | 
| 171 | 171 |  	{ | 
| 172 | 172 | $image_ids = self::cast_mixed_int2array($image_ids); | 
| @@ -194,10 +194,10 @@ discard block | ||
| 194 | 194 | |
| 195 | 195 | |
| 196 | 196 | /** | 
| 197 | - * Delete reports for given album_ids | |
| 198 | - * | |
| 199 | - * @param mixed $album_ids Array or integer with album_id. | |
| 200 | - */ | |
| 197 | + * Delete reports for given album_ids | |
| 198 | + * | |
| 199 | + * @param mixed $album_ids Array or integer with album_id. | |
| 200 | + */ | |
| 201 | 201 | public function delete_albums($album_ids) | 
| 202 | 202 |  	{ | 
| 203 | 203 | $album_ids = self::cast_mixed_int2array($album_ids); | 
| @@ -249,8 +249,7 @@ discard block | ||
| 249 | 249 |  			{ | 
| 250 | 250 | $mod_array[] = 0; | 
| 251 | 251 | } | 
| 252 | - } | |
| 253 | - else | |
| 252 | + } else | |
| 254 | 253 |  		{ | 
| 255 | 254 | $mod_array = array($album); | 
| 256 | 255 | } | 
| @@ -340,8 +339,7 @@ discard block | ||
| 340 | 339 | $this->template->assign_vars(array( | 
| 341 | 340 |  				'TOTAL_PAGES'				=> $this->language->lang('PAGE_TITLE_NUMBER', $page + 1), | 
| 342 | 341 | )); | 
| 343 | - } | |
| 344 | - else | |
| 342 | + } else | |
| 345 | 343 |  		{ | 
| 346 | 344 | $this->pagination->generate_template_pagination(array( | 
| 347 | 345 | 'routes' => array( | 
| @@ -396,8 +394,7 @@ discard block | ||
| 396 | 394 | if (is_array($ids)) | 
| 397 | 395 |  		{ | 
| 398 | 396 |  			return array_map('intval', $ids); | 
| 399 | - } | |
| 400 | - else | |
| 397 | + } else | |
| 401 | 398 |  		{ | 
| 402 | 399 | return array((int) $ids); | 
| 403 | 400 | } | 
| @@ -64,7 +64,7 @@ discard block | ||
| 64 | 64 | protected $reports_table; | 
| 65 | 65 | |
| 66 | 66 | public function __construct(\phpbbgallery\core\log $gallery_log, \phpbbgallery\core\auth\auth $gallery_auth, \phpbb\user $user, | 
| 67 | - \phpbb\language\language $language, \phpbb\db\driver\driver_interface $db, \phpbb\user_loader $user_loader, | |
| 67 | + \phpbb\language\language $language, \phpbb\db\driver\driver_interface $db, \phpbb\user_loader $user_loader, | |
| 68 | 68 | \phpbbgallery\core\album\album $album, \phpbb\template\template $template, \phpbb\controller\helper $helper, | 
| 69 | 69 | \phpbbgallery\core\config $gallery_config, \phpbb\pagination $pagination, \phpbbgallery\core\notification\helper $notification_helper, | 
| 70 | 70 | $images_table, $reports_table) | 
| @@ -363,7 +363,7 @@ discard block | ||
| 363 | 363 | 'REPORT_TIME' => $this->user->format_date($VAR['report_time']), | 
| 364 | 364 | )); | 
| 365 | 365 | unset($album_tmp); | 
| 366 | - $reported_images_count ++; | |
| 366 | + $reported_images_count++; | |
| 367 | 367 | } | 
| 368 | 368 | $this->template->assign_vars(array( | 
| 369 | 369 |  			'TOTAL_IMAGES_REPORTED' => $status == 1 ? $this->language->lang('WAITING_REPORTED_IMAGE', (int) $count) : $this->language->lang('WAITING_REPORTED_DONE', (int) $count), | 
| @@ -279,7 +279,7 @@ discard block | ||
| 279 | 279 | include_once($this->root_path . 'includes/message_parser.' . $this->php_ext); | 
| 280 | 280 | } | 
| 281 | 281 | $message_parser = new \parse_message(); | 
| 282 | - $message_parser->message = utf8_normalize_nfc($this->get_description()); | |
| 282 | + $message_parser->message = utf8_normalize_nfc($this->get_description()); | |
| 283 | 283 | if ($message_parser->message) | 
| 284 | 284 |  		{ | 
| 285 | 285 | $message_parser->parse(true, true, true, true, false, true, true, true); | 
| @@ -334,7 +334,7 @@ discard block | ||
| 334 | 334 | |
| 335 | 335 | $sql_ary = array_merge($sql_ary, $additional_sql_data); | 
| 336 | 336 | |
| 337 | - $sql = 'UPDATE ' . $this->images_table. ' | |
| 337 | + $sql = 'UPDATE ' . $this->images_table . ' | |
| 338 | 338 |  			SET ' . $this->db->sql_build_array('UPDATE', $sql_ary) . ' | 
| 339 | 339 | WHERE image_id = ' . (int) $image_id; | 
| 340 | 340 | $this->db->sql_query($sql); | 
| @@ -452,7 +452,7 @@ discard block | ||
| 452 | 452 | // Rotate the image | 
| 453 | 453 |  		if ($this->gallery_config->get('allow_rotate') && $this->get_rotating()) | 
| 454 | 454 |  		{ | 
| 455 | -			$this->tools->rotate_image($this->get_rotating(),$this->gallery_config->get('allow_resize')); | |
| 455 | +			$this->tools->rotate_image($this->get_rotating(), $this->gallery_config->get('allow_resize')); | |
| 456 | 456 | if ($this->tools->rotated) | 
| 457 | 457 |  			{ | 
| 458 | 458 |  				$this->tools->write_image($this->tools->image_source, $this->gallery_config->get('jpg_quality'), true); | 
| @@ -144,8 +144,7 @@ discard block | ||
| 144 | 144 |  			{ | 
| 145 | 145 | $this->zip_file = $this->file; | 
| 146 | 146 | $this->upload_zip(); | 
| 147 | - } | |
| 148 | - else | |
| 147 | + } else | |
| 149 | 148 |  			{ | 
| 150 | 149 | $image_id = $this->prepare_file(); | 
| 151 | 150 | if ($image_id) | 
| @@ -210,8 +209,7 @@ discard block | ||
| 210 | 209 | if (is_dir($current_dir . $file)) | 
| 211 | 210 |  			{ | 
| 212 | 211 | $this->read_zip_folder($current_dir . $file . '/'); | 
| 213 | - } | |
| 214 | - else if (in_array(utf8_substr(strtolower($file), utf8_strrpos($file, '.') + 1), self::get_allowed_types(false, true))) | |
| 212 | + } else if (in_array(utf8_substr(strtolower($file), utf8_strrpos($file, '.') + 1), self::get_allowed_types(false, true))) | |
| 215 | 213 |  			{ | 
| 216 | 214 | if (!$this->file_limit || ($this->uploaded_files < $this->file_limit)) | 
| 217 | 215 |  				{ | 
| @@ -231,23 +229,20 @@ discard block | ||
| 231 | 229 |  					{ | 
| 232 | 230 | $this->uploaded_files++; | 
| 233 | 231 | $this->images[] = (int) $image_id; | 
| 234 | - } | |
| 235 | - else | |
| 232 | + } else | |
| 236 | 233 |  					{ | 
| 237 | 234 | if ($this->file->error) | 
| 238 | 235 |  						{ | 
| 239 | 236 |  							$this->new_error($this->language->lang('UPLOAD_ERROR', $this->file->get('uploadname'), implode('<br />» ', $this->file->error))); | 
| 240 | 237 | } | 
| 241 | 238 | } | 
| 242 | - } | |
| 243 | - else | |
| 239 | + } else | |
| 244 | 240 |  				{ | 
| 245 | 241 | $this->quota_error(); | 
| 246 | 242 | @unlink($current_dir . $file); | 
| 247 | 243 | } | 
| 248 | 244 | |
| 249 | - } | |
| 250 | - else | |
| 245 | + } else | |
| 251 | 246 |  			{ | 
| 252 | 247 | @unlink($current_dir . $file); | 
| 253 | 248 | } | 
| @@ -356,8 +351,7 @@ discard block | ||
| 356 | 351 |  		{ | 
| 357 | 352 |  			$this->file->clean_filename('unique_ext'); | 
| 358 | 353 |  			$this->file->move_file(substr($this->gallery_url->path('upload'), 0, -1), false, false, CHMOD_ALL); | 
| 359 | - } | |
| 360 | - else | |
| 354 | + } else | |
| 361 | 355 |  		{ | 
| 362 | 356 | // Okay, this looks hacky, but what we do here is, we store the directory name in the filename. | 
| 363 | 357 | // However phpBB strips directories form the filename, when moving, so we need to specify that again. | 
| @@ -411,8 +405,7 @@ discard block | ||
| 411 | 405 | //$this->file->height = $this->tools->image_size['height']; | 
| 412 | 406 | //$this->file->width = $this->tools->image_size['width']; | 
| 413 | 407 | } | 
| 414 | - } | |
| 415 | - else | |
| 408 | + } else | |
| 416 | 409 |  			{ | 
| 417 | 410 | $this->file->remove(); | 
| 418 | 411 |  				$this->new_error($this->language->lang('UPLOAD_ERROR', $this->file->get('uploadname'), $this->language->lang('UPLOAD_IMAGE_SIZE_TOO_BIG'))); | 
| @@ -15,94 +15,94 @@ discard block | ||
| 15 | 15 | class upload | 
| 16 | 16 |  { | 
| 17 | 17 | /** | 
| 18 | - * @var \phpbb\user | |
| 19 | - */ | |
| 18 | + * @var \phpbb\user | |
| 19 | + */ | |
| 20 | 20 | protected $user; | 
| 21 | 21 | |
| 22 | 22 | /** | 
| 23 | - * @var \phpbb\language\language | |
| 24 | - */ | |
| 23 | + * @var \phpbb\language\language | |
| 24 | + */ | |
| 25 | 25 | protected $language; | 
| 26 | 26 | |
| 27 | 27 | /** | 
| 28 | - * @var \phpbb\db\driver\driver_interface | |
| 29 | - */ | |
| 28 | + * @var \phpbb\db\driver\driver_interface | |
| 29 | + */ | |
| 30 | 30 | protected $db; | 
| 31 | 31 | |
| 32 | 32 | /** | 
| 33 | - * @var \phpbb\event\dispatcher_interface | |
| 34 | - */ | |
| 33 | + * @var \phpbb\event\dispatcher_interface | |
| 34 | + */ | |
| 35 | 35 | protected $phpbb_dispatcher; | 
| 36 | 36 | |
| 37 | 37 | /** | 
| 38 | - * @var \phpbb\request\request | |
| 39 | - */ | |
| 38 | + * @var \phpbb\request\request | |
| 39 | + */ | |
| 40 | 40 | protected $request; | 
| 41 | 41 | |
| 42 | 42 | /** | 
| 43 | - * @var \phpbb\files\upload | |
| 44 | - */ | |
| 43 | + * @var \phpbb\files\upload | |
| 44 | + */ | |
| 45 | 45 | protected $file_upload; | 
| 46 | 46 | |
| 47 | 47 | /** | 
| 48 | - * @var \phpbbgallery\core\image\image | |
| 49 | - */ | |
| 48 | + * @var \phpbbgallery\core\image\image | |
| 49 | + */ | |
| 50 | 50 | protected $gallery_image; | 
| 51 | 51 | |
| 52 | 52 | /** | 
| 53 | - * @var \phpbbgallery\core\config | |
| 54 | - */ | |
| 53 | + * @var \phpbbgallery\core\config | |
| 54 | + */ | |
| 55 | 55 | protected $gallery_config; | 
| 56 | 56 | |
| 57 | 57 | /** | 
| 58 | - * @var \phpbbgallery\core\url | |
| 59 | - */ | |
| 58 | + * @var \phpbbgallery\core\url | |
| 59 | + */ | |
| 60 | 60 | protected $gallery_url; | 
| 61 | 61 | |
| 62 | 62 | /** | 
| 63 | - * @var \phpbbgallery\core\block | |
| 64 | - */ | |
| 63 | + * @var \phpbbgallery\core\block | |
| 64 | + */ | |
| 65 | 65 | protected $block; | 
| 66 | 66 | |
| 67 | 67 | /** | 
| 68 | - * @var \phpbbgallery\core\file\file | |
| 69 | - */ | |
| 68 | + * @var \phpbbgallery\core\file\file | |
| 69 | + */ | |
| 70 | 70 | protected $tools; | 
| 71 | 71 | |
| 72 | 72 | /** | 
| 73 | - * @var string | |
| 74 | - */ | |
| 73 | + * @var string | |
| 74 | + */ | |
| 75 | 75 | protected $images_table; | 
| 76 | 76 | |
| 77 | 77 | /** | 
| 78 | - * @var string | |
| 79 | - */ | |
| 78 | + * @var string | |
| 79 | + */ | |
| 80 | 80 | protected $root_path; | 
| 81 | 81 | |
| 82 | 82 | /** | 
| 83 | - * @var string | |
| 84 | - */ | |
| 83 | + * @var string | |
| 84 | + */ | |
| 85 | 85 | protected $php_ext; | 
| 86 | 86 | |
| 87 | 87 | /** | 
| 88 | - * Number of Files per Directory | |
| 89 | - * | |
| 90 | - * If this constant is set to a value >0 the gallery will create a new directory, | |
| 91 | - * when the current directory has more files in it than set here. | |
| 92 | - */ | |
| 88 | + * Number of Files per Directory | |
| 89 | + * | |
| 90 | + * If this constant is set to a value >0 the gallery will create a new directory, | |
| 91 | + * when the current directory has more files in it than set here. | |
| 92 | + */ | |
| 93 | 93 | const NUM_FILES_PER_DIR = 0; | 
| 94 | 94 | |
| 95 | 95 | /** | 
| 96 | - * Objects: phpBB Upload, 2 Files and Image-Functions | |
| 97 | - */ | |
| 96 | + * Objects: phpBB Upload, 2 Files and Image-Functions | |
| 97 | + */ | |
| 98 | 98 | private $upload = null; | 
| 99 | 99 | private $file = null; | 
| 100 | 100 | private $zip_file = null; | 
| 101 | 101 | //private $tools = null; | 
| 102 | 102 | |
| 103 | 103 | /** | 
| 104 | - * Basic variables... | |
| 105 | - */ | |
| 104 | + * Basic variables... | |
| 105 | + */ | |
| 106 | 106 | public $loaded_files = 0; | 
| 107 | 107 | public $uploaded_files = 0; | 
| 108 | 108 | public $errors = array(); | 
| @@ -230,8 +230,8 @@ discard block | ||
| 230 | 230 | } | 
| 231 | 231 | |
| 232 | 232 | /** | 
| 233 | - * Upload a zip file and save the images into the import/ directory. | |
| 234 | - */ | |
| 233 | + * Upload a zip file and save the images into the import/ directory. | |
| 234 | + */ | |
| 235 | 235 | public function upload_zip() | 
| 236 | 236 |  	{ | 
| 237 | 237 |  		if (!class_exists('compress_zip')) | 
| @@ -379,14 +379,14 @@ discard block | ||
| 379 | 379 |  		$file_link = $this->gallery_url->path('upload') . $this->image_data[$image_id]['image_filename']; | 
| 380 | 380 | |
| 381 | 381 | /** | 
| 382 | - * Event upload image before | |
| 383 | - * | |
| 384 | - * @event phpbbgallery.core.upload.update_image_before | |
| 385 | - * @var array additional_sql_data array of additional settings | |
| 386 | - * @var array image_data array of image_data | |
| 387 | - * @var string file_link link to file | |
| 388 | - * @since 1.2.0 | |
| 389 | - */ | |
| 382 | + * Event upload image before | |
| 383 | + * | |
| 384 | + * @event phpbbgallery.core.upload.update_image_before | |
| 385 | + * @var array additional_sql_data array of additional settings | |
| 386 | + * @var array image_data array of image_data | |
| 387 | + * @var string file_link link to file | |
| 388 | + * @since 1.2.0 | |
| 389 | + */ | |
| 390 | 390 |  		$vars = array('additional_sql_data', 'image_data', 'file_link'); | 
| 391 | 391 |  		extract($this->phpbb_dispatcher->trigger_event('phpbbgallery.core.upload.update_image_before', compact($vars))); | 
| 392 | 392 | |
| @@ -394,12 +394,12 @@ discard block | ||
| 394 | 394 | if (!$this->prepare_file_update($image_id)) | 
| 395 | 395 |  		{ | 
| 396 | 396 | /** | 
| 397 | - * Event upload image update | |
| 398 | - * | |
| 399 | - * @event phpbbgallery.core.upload.update_image_nofilechange | |
| 400 | - * @var array additional_sql_data array of additional settings | |
| 401 | - * @since 1.2.0 | |
| 402 | - */ | |
| 397 | + * Event upload image update | |
| 398 | + * | |
| 399 | + * @event phpbbgallery.core.upload.update_image_nofilechange | |
| 400 | + * @var array additional_sql_data array of additional settings | |
| 401 | + * @since 1.2.0 | |
| 402 | + */ | |
| 403 | 403 |  			$vars = array('additional_sql_data'); | 
| 404 | 404 |  			extract($this->phpbb_dispatcher->trigger_event('phpbbgallery.core.upload.update_image_nofilechange', compact($vars))); | 
| 405 | 405 | } | 
| @@ -415,8 +415,8 @@ discard block | ||
| 415 | 415 | } | 
| 416 | 416 | |
| 417 | 417 | /** | 
| 418 | - * Prepare file on upload: rotate and resize | |
| 419 | - */ | |
| 418 | + * Prepare file on upload: rotate and resize | |
| 419 | + */ | |
| 420 | 420 | public function prepare_file() | 
| 421 | 421 |  	{ | 
| 422 | 422 | $upload_dir = $this->get_current_upload_dir(); | 
| @@ -446,13 +446,13 @@ discard block | ||
| 446 | 446 | $file = $this->file; | 
| 447 | 447 | |
| 448 | 448 | /** | 
| 449 | - * Event upload image update | |
| 450 | - * | |
| 451 | - * @event phpbbgallery.core.upload.prepare_file_before | |
| 452 | - * @var array additional_sql_data array of additional settings | |
| 453 | - * @var array file File object | |
| 454 | - * @since 1.2.0 | |
| 455 | - */ | |
| 449 | + * Event upload image update | |
| 450 | + * | |
| 451 | + * @event phpbbgallery.core.upload.prepare_file_before | |
| 452 | + * @var array additional_sql_data array of additional settings | |
| 453 | + * @var array file File object | |
| 454 | + * @since 1.2.0 | |
| 455 | + */ | |
| 456 | 456 |  		$vars = array('additional_sql_data', 'file'); | 
| 457 | 457 |  		extract($this->phpbb_dispatcher->trigger_event('phpbbgallery.core.upload.prepare_file_before', compact($vars))); | 
| 458 | 458 | |
| @@ -804,8 +804,8 @@ discard block | ||
| 804 | 804 | } | 
| 805 | 805 | |
| 806 | 806 | /** | 
| 807 | - * Generate some kind of check so users only complete the upload for their images | |
| 808 | - */ | |
| 807 | + * Generate some kind of check so users only complete the upload for their images | |
| 808 | + */ | |
| 809 | 809 | public function generate_hidden_fields() | 
| 810 | 810 |  	{ | 
| 811 | 811 | $checks = array(); | 
| @@ -34,18 +34,18 @@ discard block | ||
| 34 | 34 | const NUM_IMAGES = 3; | 
| 35 | 35 | |
| 36 | 36 | /** | 
| 37 | - * There are different modes to calculate who won the contest. | |
| 38 | - * This value should be one of the constant-names below. | |
| 39 | - */ | |
| 37 | + * There are different modes to calculate who won the contest. | |
| 38 | + * This value should be one of the constant-names below. | |
| 39 | + */ | |
| 40 | 40 | static public $mode = self::MODE_AVERAGE; | 
| 41 | 41 | |
| 42 | 42 | /** | 
| 43 | - * The image with the highest average wins. | |
| 44 | - */ | |
| 43 | + * The image with the highest average wins. | |
| 44 | + */ | |
| 45 | 45 | const MODE_AVERAGE = 1; | 
| 46 | 46 | /** | 
| 47 | - * The image with the highest number of total points wins. | |
| 48 | - */ | |
| 47 | + * The image with the highest number of total points wins. | |
| 48 | + */ | |
| 49 | 49 | const MODE_SUM = 2; | 
| 50 | 50 | |
| 51 | 51 | public function __construct(\phpbb\db\driver\driver_interface $db, | 
| @@ -59,14 +59,14 @@ discard block | ||
| 59 | 59 | } | 
| 60 | 60 | |
| 61 | 61 | /** | 
| 62 | - * Get the contest row from the table | |
| 63 | - * | |
| 64 | - * @param int $id ID of the contest or album, depending on second parameter | |
| 65 | - * @param string $mode contest or album ID to get the contest. | |
| 66 | - * @param bool $throw_error Shall we throw an error if the contest was not found? | |
| 67 | - * | |
| 68 | - * @return mixed Either the array or boolean false if contest does not exist | |
| 69 | - */ | |
| 62 | + * Get the contest row from the table | |
| 63 | + * | |
| 64 | + * @param int $id ID of the contest or album, depending on second parameter | |
| 65 | + * @param string $mode contest or album ID to get the contest. | |
| 66 | + * @param bool $throw_error Shall we throw an error if the contest was not found? | |
| 67 | + * | |
| 68 | + * @return mixed Either the array or boolean false if contest does not exist | |
| 69 | + */ | |
| 70 | 70 | public function get_contest($id, $mode = 'contest', $throw_error = true) | 
| 71 | 71 |  	{ | 
| 72 | 72 | $sql = 'SELECT * | 
| @@ -190,9 +190,9 @@ | ||
| 190 | 190 | $this->db->sql_freeresult($result); | 
| 191 | 191 | |
| 192 | 192 | $sql = 'UPDATE ' . $this->contest_table . ' | 
| 193 | - SET contest_first = ' . (int) $first .', | |
| 194 | - contest_second = ' . (int) $second .', | |
| 195 | - contest_third = ' . (int) $third .' | |
| 193 | + SET contest_first = ' . (int) $first . ', | |
| 194 | + contest_second = ' . (int) $second . ', | |
| 195 | + contest_third = ' . (int) $third . ' | |
| 196 | 196 | WHERE contest_album_id = ' . (int) $album_id; | 
| 197 | 197 | $this->db->sql_query($sql); | 
| 198 | 198 | |
| @@ -165,8 +165,7 @@ | ||
| 165 | 165 |  			{ | 
| 166 | 166 | $this->resync($album_id); | 
| 167 | 167 | } | 
| 168 | - } | |
| 169 | - else | |
| 168 | + } else | |
| 170 | 169 |  		{ | 
| 171 | 170 | $this->resync((int) $album_ids); | 
| 172 | 171 | } | 
| @@ -158,7 +158,7 @@ discard block | ||
| 158 | 158 | */ | 
| 159 | 159 | public function display_box() | 
| 160 | 160 |  	{ | 
| 161 | -		$this->template->assign_var('GALLERY_RATING', self::MODE_SELECT);//@todo: phpbb_ext_gallery_core_config::get('rating_mode')); | |
| 161 | +		$this->template->assign_var('GALLERY_RATING', self::MODE_SELECT); //@todo: phpbb_ext_gallery_core_config::get('rating_mode')); | |
| 162 | 162 | |
| 163 | 163 |  		switch (self::MODE_SELECT)//@todo: phpbb_ext_gallery_core_config::get('rating_mode')) | 
| 164 | 164 |  		{ | 
| @@ -201,7 +201,7 @@ discard block | ||
| 201 | 201 | */ | 
| 202 | 202 | public function get_image_rating($user_rating = false, $display_contest_end = true) | 
| 203 | 203 |  	{ | 
| 204 | -		$this->template->assign_var('GALLERY_RATING', self::MODE_SELECT);//@todo: phpbb_ext_gallery_core_config::get('rating_mode')); | |
| 204 | +		$this->template->assign_var('GALLERY_RATING', self::MODE_SELECT); //@todo: phpbb_ext_gallery_core_config::get('rating_mode')); | |
| 205 | 205 | |
| 206 | 206 |  		switch (self::MODE_SELECT)//@todo: phpbb_ext_gallery_core_config::get('rating_mode')) | 
| 207 | 207 |  		{ | 
| @@ -216,8 +216,7 @@ discard block | ||
| 216 | 216 |  						return $this->language->lang('CONTEST_RATING_HIDDEN'); | 
| 217 | 217 | } | 
| 218 | 218 |  					return $this->language->lang('CONTEST_RESULT_HIDDEN', $this->user->format_date(($this->album_data('contest_start') + $this->album_data('contest_end')), false, true)); | 
| 219 | - } | |
| 220 | - else | |
| 219 | + } else | |
| 221 | 220 |  				{ | 
| 222 | 221 | if ($user_rating) | 
| 223 | 222 |  					{ | 
| @@ -364,8 +363,7 @@ discard block | ||
| 364 | 363 | if (is_array($image_ids)) | 
| 365 | 364 |  		{ | 
| 366 | 365 |  			$image_ids = array_map('intval', $image_ids); | 
| 367 | - } | |
| 368 | - else | |
| 366 | + } else | |
| 369 | 367 |  		{ | 
| 370 | 368 | $image_ids = (int) $image_ids; | 
| 371 | 369 | } | 
| @@ -399,8 +397,7 @@ discard block | ||
| 399 | 397 | if (is_array($image_ids)) | 
| 400 | 398 |  		{ | 
| 401 | 399 |  			$image_ids = array_map('intval', $image_ids); | 
| 402 | - } | |
| 403 | - else | |
| 400 | + } else | |
| 404 | 401 |  		{ | 
| 405 | 402 | $image_ids = (int) $image_ids; | 
| 406 | 403 | } | 
| @@ -13,91 +13,91 @@ discard block | ||
| 13 | 13 | class rating | 
| 14 | 14 |  { | 
| 15 | 15 | /** | 
| 16 | - * @var \phpbb\db\driver\driver_interface | |
| 17 | - */ | |
| 16 | + * @var \phpbb\db\driver\driver_interface | |
| 17 | + */ | |
| 18 | 18 | protected $db; | 
| 19 | 19 | |
| 20 | 20 | /** | 
| 21 | - * @var \phpbb\template\template | |
| 22 | - */ | |
| 21 | + * @var \phpbb\template\template | |
| 22 | + */ | |
| 23 | 23 | protected $template; | 
| 24 | 24 | |
| 25 | 25 | /** | 
| 26 | - * @var \phpbb\user | |
| 27 | - */ | |
| 26 | + * @var \phpbb\user | |
| 27 | + */ | |
| 28 | 28 | protected $user; | 
| 29 | 29 | |
| 30 | 30 | /** | 
| 31 | - * @var \phpbb\language\language | |
| 32 | - */ | |
| 31 | + * @var \phpbb\language\language | |
| 32 | + */ | |
| 33 | 33 | protected $language; | 
| 34 | 34 | |
| 35 | 35 | /** | 
| 36 | - * @var \phpbb\request\request | |
| 37 | - */ | |
| 36 | + * @var \phpbb\request\request | |
| 37 | + */ | |
| 38 | 38 | protected $request; | 
| 39 | 39 | |
| 40 | 40 | /** | 
| 41 | - * @var \phpbbgallery\core\config | |
| 42 | - */ | |
| 41 | + * @var \phpbbgallery\core\config | |
| 42 | + */ | |
| 43 | 43 | protected $gallery_config; | 
| 44 | 44 | |
| 45 | 45 | /** | 
| 46 | - * @var \phpbbgallery\core\auth\auth | |
| 47 | - */ | |
| 46 | + * @var \phpbbgallery\core\auth\auth | |
| 47 | + */ | |
| 48 | 48 | protected $gallery_auth; | 
| 49 | 49 | |
| 50 | 50 | /** | 
| 51 | - * @var string | |
| 52 | - */ | |
| 51 | + * @var string | |
| 52 | + */ | |
| 53 | 53 | protected $images_table; | 
| 54 | 54 | |
| 55 | 55 | /** | 
| 56 | - * @var string | |
| 57 | - */ | |
| 56 | + * @var string | |
| 57 | + */ | |
| 58 | 58 | protected $albums_table; | 
| 59 | 59 | |
| 60 | 60 | /** | 
| 61 | - * @var string | |
| 62 | - */ | |
| 61 | + * @var string | |
| 62 | + */ | |
| 63 | 63 | protected $rates_table; | 
| 64 | 64 | |
| 65 | 65 | /** | 
| 66 | - * The image ID we want to rate | |
| 67 | - */ | |
| 66 | + * The image ID we want to rate | |
| 67 | + */ | |
| 68 | 68 | public $image_id = 0; | 
| 69 | 69 | |
| 70 | 70 | /** | 
| 71 | - * Private objects with the values for the image/album from the database | |
| 72 | - */ | |
| 71 | + * Private objects with the values for the image/album from the database | |
| 72 | + */ | |
| 73 | 73 | private $image_data = null; | 
| 74 | 74 | private $album_data = null; | 
| 75 | 75 | |
| 76 | 76 | /** | 
| 77 | - * Rating the user gave the image. | |
| 78 | - */ | |
| 77 | + * Rating the user gave the image. | |
| 78 | + */ | |
| 79 | 79 | public $user_rating = null; | 
| 80 | 80 | |
| 81 | 81 | /** | 
| 82 | - * Is rating currently possible? | |
| 83 | - * Might be blocked because of contest-settings. | |
| 84 | - */ | |
| 82 | + * Is rating currently possible? | |
| 83 | + * Might be blocked because of contest-settings. | |
| 84 | + */ | |
| 85 | 85 | public $rating_enabled = false; | 
| 86 | 86 | |
| 87 | 87 | /** | 
| 88 | - * Classic-rating box with a dropdown. | |
| 89 | - */ | |
| 88 | + * Classic-rating box with a dropdown. | |
| 89 | + */ | |
| 90 | 90 | const MODE_SELECT = 1; | 
| 91 | 91 | |
| 92 | 92 | /** | 
| 93 | - * Rating with stars, like the old-system from youtube. | |
| 93 | + * Rating with stars, like the old-system from youtube. | |
| 94 | 94 | //@todo: const MODE_STARS = 2; | 
| 95 | - */ | |
| 95 | + */ | |
| 96 | 96 | |
| 97 | 97 | /** | 
| 98 | - * Simple thumbs up or down. | |
| 98 | + * Simple thumbs up or down. | |
| 99 | 99 | //@todo: const MODE_THUMB = 3; | 
| 100 | - */ | |
| 100 | + */ | |
| 101 | 101 | |
| 102 | 102 | /** | 
| 103 | 103 | * Constructor | 
| @@ -204,8 +204,8 @@ discard block | ||
| 204 | 204 | } | 
| 205 | 205 | |
| 206 | 206 | /** | 
| 207 | - * Displays the box where the user can rate the image. | |
| 208 | - */ | |
| 207 | + * Displays the box where the user can rate the image. | |
| 208 | + */ | |
| 209 | 209 | public function display_box() | 
| 210 | 210 |  	{ | 
| 211 | 211 |  		$this->template->assign_var('GALLERY_RATING', self::MODE_SELECT);//@todo: phpbb_ext_gallery_core_config::get('rating_mode')); | 
| @@ -280,8 +280,8 @@ discard block | ||
| 280 | 280 | } | 
| 281 | 281 | |
| 282 | 282 | /** | 
| 283 | - * Get rated value for a image | |
| 284 | - */ | |
| 283 | + * Get rated value for a image | |
| 284 | + */ | |
| 285 | 285 | private function get_image_rating_value() | 
| 286 | 286 |  	{ | 
| 287 | 287 | /*if (phpbb_ext_gallery_core_contest::$mode == phpbb_ext_gallery_core_contest::MODE_SUM) | 
| @@ -295,14 +295,14 @@ discard block | ||
| 295 | 295 | } | 
| 296 | 296 | |
| 297 | 297 | /** | 
| 298 | - * Is the user allowed to rate? | |
| 299 | - * Following statements must be true: | |
| 300 | - * - User must have permissions. | |
| 301 | - * - User is neither owner of the image nor guest. | |
| 302 | - * - Album and image are not locked. | |
| 303 | - * | |
| 304 | - * @return bool | |
| 305 | - */ | |
| 298 | + * Is the user allowed to rate? | |
| 299 | + * Following statements must be true: | |
| 300 | + * - User must have permissions. | |
| 301 | + * - User is neither owner of the image nor guest. | |
| 302 | + * - Album and image are not locked. | |
| 303 | + * | |
| 304 | + * @return bool | |
| 305 | + */ | |
| 306 | 306 | public function is_allowed() | 
| 307 | 307 |  	{ | 
| 308 | 308 |  		return $this->gallery_auth->acl_check('i_rate', $this->album_data('album_id'), $this->album_data('album_user_id')) && | 
| @@ -311,25 +311,25 @@ discard block | ||
| 311 | 311 | } | 
| 312 | 312 | |
| 313 | 313 | /** | 
| 314 | - * Is the user able to rate? | |
| 315 | - * Following statements must be true: | |
| 316 | - * - User must be allowed to rate | |
| 317 | - * - If the image is in a contest, it must be in the rating timespan | |
| 318 | - * | |
| 319 | - * @return bool | |
| 320 | - */ | |
| 314 | + * Is the user able to rate? | |
| 315 | + * Following statements must be true: | |
| 316 | + * - User must be allowed to rate | |
| 317 | + * - If the image is in a contest, it must be in the rating timespan | |
| 318 | + * | |
| 319 | + * @return bool | |
| 320 | + */ | |
| 321 | 321 | public function is_able() | 
| 322 | 322 |  	{ | 
| 323 | 323 |  		return $this->is_allowed(); //&& phpbb_ext_gallery_core_contest::is_step('rate', $this->album_data(true)); | 
| 324 | 324 | } | 
| 325 | 325 | |
| 326 | 326 | /** | 
| 327 | - * Get rating from a user for a given image | |
| 328 | - * | |
| 329 | - * @param int $user_id | |
| 330 | - * | |
| 331 | - * @return mixed False if the user did not rate or is guest, otherwise int the points. | |
| 332 | - */ | |
| 327 | + * Get rating from a user for a given image | |
| 328 | + * | |
| 329 | + * @param int $user_id | |
| 330 | + * | |
| 331 | + * @return mixed False if the user did not rate or is guest, otherwise int the points. | |
| 332 | + */ | |
| 333 | 333 | public function get_user_rating($user_id) | 
| 334 | 334 |  	{ | 
| 335 | 335 | if (isset($this->user_rating[$user_id])) | 
| @@ -405,10 +405,10 @@ discard block | ||
| 405 | 405 | } | 
| 406 | 406 | |
| 407 | 407 | /** | 
| 408 | - * Recalculate the average image-rating and such stuff. | |
| 409 | - * | |
| 410 | - * @param mixed $image_ids Array or integer with image_id where we recalculate the rating. | |
| 411 | - */ | |
| 408 | + * Recalculate the average image-rating and such stuff. | |
| 409 | + * | |
| 410 | + * @param mixed $image_ids Array or integer with image_id where we recalculate the rating. | |
| 411 | + */ | |
| 412 | 412 | public function recalc_image_rating($image_ids) | 
| 413 | 413 |  	{ | 
| 414 | 414 | if (is_array($image_ids)) | 
| @@ -439,11 +439,11 @@ discard block | ||
| 439 | 439 | } | 
| 440 | 440 | |
| 441 | 441 | /** | 
| 442 | - * Delete all ratings for given image_ids | |
| 443 | - * | |
| 444 | - * @param mixed $image_ids Array or integer with image_id where we delete the rating. | |
| 445 | - * @param bool $reset_average Shall we also reset the average? We can save that query, when the images are deleted anyway. | |
| 446 | - */ | |
| 442 | + * Delete all ratings for given image_ids | |
| 443 | + * | |
| 444 | + * @param mixed $image_ids Array or integer with image_id where we delete the rating. | |
| 445 | + * @param bool $reset_average Shall we also reset the average? We can save that query, when the images are deleted anyway. | |
| 446 | + */ | |
| 447 | 447 | public function delete_ratings($image_ids, $reset_average = false) | 
| 448 | 448 |  	{ | 
| 449 | 449 | if (is_array($image_ids)) | 
| @@ -23,13 +23,13 @@ discard block | ||
| 23 | 23 | * | 
| 24 | 24 | * Constants defining some album properties | 
| 25 | 25 | */ | 
| 26 | - const PUBLIC_ALBUM = 0; | |
| 26 | + const PUBLIC_ALBUM = 0; | |
| 27 | 27 | |
| 28 | - const TYPE_CAT = 0; | |
| 29 | - const TYPE_UPLOAD = 1; | |
| 28 | + const TYPE_CAT = 0; | |
| 29 | + const TYPE_UPLOAD = 1; | |
| 30 | 30 | const TYPE_CONTEST = 2; | 
| 31 | 31 | |
| 32 | - const ALBUM_OPEN = 0; | |
| 32 | + const ALBUM_OPEN = 0; | |
| 33 | 33 | const ALBUM_LOCKED = 1; | 
| 34 | 34 | |
| 35 | 35 | /** | 
| @@ -58,22 +58,22 @@ discard block | ||
| 58 | 58 | /** | 
| 59 | 59 | * Only visible for moderators. | 
| 60 | 60 | */ | 
| 61 | - const STATUS_UNAPPROVED = 0; | |
| 61 | + const STATUS_UNAPPROVED = 0; | |
| 62 | 62 | |
| 63 | 63 | /** | 
| 64 | 64 | * Visible for everyone with the i_view-permissions | 
| 65 | 65 | */ | 
| 66 | - const STATUS_APPROVED = 1; | |
| 66 | + const STATUS_APPROVED = 1; | |
| 67 | 67 | |
| 68 | 68 | /** | 
| 69 | 69 | * Visible for everyone with the i_view-permissions, but only moderators can comment. | 
| 70 | 70 | */ | 
| 71 | - const STATUS_LOCKED = 2; | |
| 71 | + const STATUS_LOCKED = 2; | |
| 72 | 72 | |
| 73 | 73 | /** | 
| 74 | 74 | * Orphan files are only visible for their author, because they're not yet ready uploaded. | 
| 75 | 75 | */ | 
| 76 | - const STATUS_ORPHAN = 3; | |
| 76 | + const STATUS_ORPHAN = 3; | |
| 77 | 77 | |
| 78 | 78 | /** | 
| 79 | 79 | * Constants regarding the image contest relation | 
| @@ -316,7 +316,7 @@ | ||
| 316 | 316 | |
| 317 | 317 | public function generate_image_src() | 
| 318 | 318 |  	{ | 
| 319 | - $this->image_src = $this->path . $this->data['image_filename']; | |
| 319 | + $this->image_src = $this->path . $this->data['image_filename']; | |
| 320 | 320 | |
| 321 | 321 | if ($this->data['image_filemissing'] || !file_exists($this->path_source . $this->data['image_filename'])) | 
| 322 | 322 |  		{ | 
| @@ -104,12 +104,12 @@ discard block | ||
| 104 | 104 | } | 
| 105 | 105 | |
| 106 | 106 | /** | 
| 107 | - * Image File Controller | |
| 108 | -	*	Route: gallery/image/{image_id}/source | |
| 109 | - * | |
| 110 | - * @param int $image_id | |
| 111 | - * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 112 | - */ | |
| 107 | + * Image File Controller | |
| 108 | +	 *	Route: gallery/image/{image_id}/source | |
| 109 | + * | |
| 110 | + * @param int $image_id | |
| 111 | + * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 112 | + */ | |
| 113 | 113 | public function source($image_id) | 
| 114 | 114 |  	{ | 
| 115 | 115 | $this->auth->load_user_permissions($this->user->data['user_id']); | 
| @@ -160,12 +160,12 @@ discard block | ||
| 160 | 160 | } | 
| 161 | 161 | |
| 162 | 162 | /** | 
| 163 | - * Image File Controller | |
| 164 | -	*	Route: gallery/image/{image_id}/medium | |
| 165 | - * | |
| 166 | - * @param int $image_id | |
| 167 | - * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 168 | - */ | |
| 163 | + * Image File Controller | |
| 164 | +	 *	Route: gallery/image/{image_id}/medium | |
| 165 | + * | |
| 166 | + * @param int $image_id | |
| 167 | + * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 168 | + */ | |
| 169 | 169 | public function medium($image_id) | 
| 170 | 170 |  	{ | 
| 171 | 171 | |
| @@ -195,12 +195,12 @@ discard block | ||
| 195 | 195 | } | 
| 196 | 196 | |
| 197 | 197 | /** | 
| 198 | - * Image File Controller | |
| 199 | -	*	Route: gallery/image/{image_id}/mini | |
| 200 | - * | |
| 201 | - * @param int $image_id | |
| 202 | - * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 203 | - */ | |
| 198 | + * Image File Controller | |
| 199 | +	 *	Route: gallery/image/{image_id}/mini | |
| 200 | + * | |
| 201 | + * @param int $image_id | |
| 202 | + * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 203 | + */ | |
| 204 | 204 | public function mini($image_id) | 
| 205 | 205 |  	{ | 
| 206 | 206 | $this->path = $this->path_mini; | 
| @@ -356,11 +356,11 @@ discard block | ||
| 356 | 356 | } | 
| 357 | 357 | |
| 358 | 358 | /** | 
| 359 | - * Image File Controller | |
| 360 | -	*	Route: gallery/image/{image_id}/x | |
| 361 | - * | |
| 362 | - * @return \Symfony\Component\HttpFoundation\BinaryFileResponseResponse A Symfony Response object | |
| 363 | - */ | |
| 359 | + * Image File Controller | |
| 360 | +	 *	Route: gallery/image/{image_id}/x | |
| 361 | + * | |
| 362 | + * @return \Symfony\Component\HttpFoundation\BinaryFileResponseResponse A Symfony Response object | |
| 363 | + */ | |
| 364 | 364 | public function display() | 
| 365 | 365 |  	{ | 
| 366 | 366 |  		$this->tool->set_last_modified($this->gallery_user->get_data('user_permissions_changed')); | 
| @@ -238,8 +238,7 @@ discard block | ||
| 238 | 238 | $this->data['album_user_id'] = 1; | 
| 239 | 239 | $this->data['image_filemissing'] = 0; | 
| 240 | 240 | $this->data['album_watermark'] = 0; | 
| 241 | - } | |
| 242 | - else | |
| 241 | + } else | |
| 243 | 242 |  		{ | 
| 244 | 243 | $sql = 'SELECT * | 
| 245 | 244 | FROM ' . $this->table_images . ' i | 
| @@ -409,8 +408,7 @@ discard block | ||
| 409 | 408 |  			{ | 
| 410 | 409 |  				$response->headers->set('expires', '-1'); | 
| 411 | 410 | } | 
| 412 | - } | |
| 413 | - else | |
| 411 | + } else | |
| 414 | 412 |  		{ | 
| 415 | 413 |  			$response->headers->set('Content-Disposition', 'inline; ' . $this->tool->header_filename(htmlspecialchars_decode($this->tool->image_name) . '.' . $this->tool->image_type)); | 
| 416 | 414 | if ($this->tool->is_ie_greater7($this->user->browser)) | 
| @@ -249,8 +249,7 @@ discard block | ||
| 249 | 249 | 'last_image' => end($process->images), | 
| 250 | 250 | ); | 
| 251 | 251 | $this->notification_helper->new_image($data); | 
| 252 | - } | |
| 253 | - else | |
| 252 | + } else | |
| 254 | 253 |  			{ | 
| 255 | 254 | $target = array( | 
| 256 | 255 | 'album_id' => $album_id, | 
| @@ -355,8 +354,7 @@ discard block | ||
| 355 | 354 |  					{ | 
| 356 | 355 |  						$this->language->add_lang('ucp'); | 
| 357 | 356 | $error_array[] = $this->language->lang($result . '_USERNAME'); | 
| 358 | - } | |
| 359 | - else | |
| 357 | + } else | |
| 360 | 358 |  					{ | 
| 361 | 359 | $process->set_username($username); | 
| 362 | 360 | } | 
| @@ -375,8 +373,7 @@ discard block | ||
| 375 | 373 | if (!$process->uploaded_files) | 
| 376 | 374 |  				{ | 
| 377 | 375 |  					$process->new_error($this->language->lang('UPLOAD_NO_FILE')); | 
| 378 | - } | |
| 379 | - else | |
| 376 | + } else | |
| 380 | 377 |  				{ | 
| 381 | 378 | $mode = 'upload_edit'; | 
| 382 | 379 | // Remove submit, so we get the first screen of step 2. | 
| @@ -422,8 +419,7 @@ discard block | ||
| 422 | 419 | if ($VAR == 'zip') | 
| 423 | 420 |  						{ | 
| 424 | 421 | continue; | 
| 425 | - } | |
| 426 | - else | |
| 422 | + } else | |
| 427 | 423 |  						{ | 
| 428 | 424 | $filetypes[] = $VAR; | 
| 429 | 425 | } | 
| @@ -523,8 +519,7 @@ discard block | ||
| 523 | 519 | 'last_image' => end($process->images), | 
| 524 | 520 | ); | 
| 525 | 521 | $this->notification_helper->new_image($data); | 
| 526 | - } | |
| 527 | - else | |
| 522 | + } else | |
| 528 | 523 |  				{ | 
| 529 | 524 | $target = array( | 
| 530 | 525 | 'album_id' => (int) $album_id, | 
| @@ -591,8 +586,7 @@ discard block | ||
| 591 | 586 | if (file_exists($phpbbgallery_core_file) && is_writable($phpbbgallery_core_file) && file_exists($phpbbgallery_core_file_source) && is_writable($phpbbgallery_core_file_source) && file_exists($phpbbgallery_core_file_medium) && is_writable($phpbbgallery_core_file_medium) && file_exists($phpbbgallery_core_file_mini) && is_writable($phpbbgallery_core_file_mini)) | 
| 592 | 587 |  		{ | 
| 593 | 588 | return true; | 
| 594 | - } | |
| 595 | - else | |
| 589 | + } else | |
| 596 | 590 |  		{ | 
| 597 | 591 | return false; | 
| 598 | 592 | } | 
| @@ -165,8 +165,8 @@ discard block | ||
| 165 | 165 |  		$this->language->add_lang(array('gallery'), 'phpbbgallery/core'); | 
| 166 | 166 |  		$this->language->add_lang('search'); | 
| 167 | 167 | /** | 
| 168 | - * Build the sort options | |
| 169 | - */ | |
| 168 | + * Build the sort options | |
| 169 | + */ | |
| 170 | 170 |  		$limit_days = array(0 => $this->language->lang('ALL_IMAGES'), 1 => $this->language->lang('1_DAY'), 7 => $this->language->lang('7_DAYS'), 14 => $this->language->lang('2_WEEKS'), 30 => $this->language->lang('1_MONTH'), 90 => $this->language->lang('3_MONTHS'), 180 => $this->language->lang('6_MONTHS'), 365 => $this->language->lang('1_YEAR')); | 
| 171 | 171 |  		$sort_by_text = array('t' => $this->language->lang('TIME'), 'n' => $this->language->lang('IMAGE_NAME'), 'u' => $this->language->lang('SORT_USERNAME'), 'vc' => $this->language->lang('GALLERY_VIEWS')); | 
| 172 | 172 |  		$sort_by_sql = array('t' => 'image_time', 'n' => 'image_name_clean', 'u' => 'image_username_clean', 'vc' => 'image_view_count'); | 
| @@ -369,11 +369,11 @@ discard block | ||
| 369 | 369 | } | 
| 370 | 370 | |
| 371 | 371 | /** | 
| 372 | - * Index Controller | |
| 373 | - * Route: gallery/search/random | |
| 374 | - * | |
| 375 | - * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 376 | - */ | |
| 372 | + * Index Controller | |
| 373 | + * Route: gallery/search/random | |
| 374 | + * | |
| 375 | + * @return \Symfony\Component\HttpFoundation\Response A Symfony Response object | |
| 376 | + */ | |
| 377 | 377 | public function random() | 
| 378 | 378 |  	{ | 
| 379 | 379 |  		$this->language->add_lang(array('gallery'), 'phpbbgallery/core'); | 
| @@ -232,8 +232,7 @@ discard block | ||
| 232 | 232 | if ($search_terms == 'all') | 
| 233 | 233 |  				{ | 
| 234 | 234 | $keywords .= ' ' . $add_keywords; | 
| 235 | - } | |
| 236 | - else | |
| 235 | + } else | |
| 237 | 236 |  				{ | 
| 238 | 237 | $search_terms = 'all'; | 
| 239 | 238 |  					$keywords = preg_replace('#\s+#u', ' |', $keywords) . ' ' .$add_keywords; | 
| @@ -271,8 +270,7 @@ discard block | ||
| 271 | 270 | if (empty($search_album)) | 
| 272 | 271 |  			{ | 
| 273 | 272 |  				$sql_where[] = $this->db->sql_in_set('i.image_album_id', $this->gallery_auth->acl_album_ids('i_view')); | 
| 274 | - } | |
| 275 | - else | |
| 273 | + } else | |
| 276 | 274 |  			{ | 
| 277 | 275 |  				$sql_where[] = $this->db->sql_in_set('i.image_album_id', $search_album); | 
| 278 | 276 | } | 
| @@ -141,10 +141,10 @@ discard block | ||
| 141 | 141 | |
| 142 | 142 | public function base($page = 1) | 
| 143 | 143 |  	{ | 
| 144 | -		$search_id		= $this->request->variable('search_id', ''); | |
| 144 | +		$search_id = $this->request->variable('search_id', ''); | |
| 145 | 145 |  		$image_id		= $this->request->variable('image_id', 0); | 
| 146 | 146 | |
| 147 | -		$submit			= $this->request->variable('submit', false); | |
| 147 | +		$submit = $this->request->variable('submit', false); | |
| 148 | 148 |  		$keywords		= utf8_normalize_nfc($this->request->variable('keywords', '', true)); | 
| 149 | 149 |  		$add_keywords	= utf8_normalize_nfc($this->request->variable('add_keywords', '', true)); | 
| 150 | 150 |  		$username		= $this->request->variable('username', '', true); | 
| @@ -152,13 +152,13 @@ discard block | ||
| 152 | 152 |  		$search_terms	= $this->request->variable('terms', 'all'); | 
| 153 | 153 |  		$search_album	= $this->request->variable('aid', array(0)); | 
| 154 | 154 |  		$search_child	= $this->request->variable('sc', true); | 
| 155 | -		$search_fields	= $this->request->variable('sf', 'all'); | |
| 156 | -		$sort_days		= $this->request->variable('st', 0); | |
| 155 | +		$search_fields = $this->request->variable('sf', 'all'); | |
| 156 | +		$sort_days = $this->request->variable('st', 0); | |
| 157 | 157 |  		$sort_key		= $this->request->variable('sk', 't'); | 
| 158 | 158 |  		$sort_dir		= $this->request->variable('sd', 'd'); | 
| 159 | 159 |  		$filtered		= $this->request->variable('filtered', false); | 
| 160 | 160 | |
| 161 | -		$start 			= ($page - 1) * $this->gallery_config->get('items_per_page'); | |
| 161 | +		$start = ($page - 1) * $this->gallery_config->get('items_per_page'); | |
| 162 | 162 | if ($filtered) | 
| 163 | 163 |  		{ | 
| 164 | 164 | $submit = true; | 
| @@ -175,7 +175,7 @@ discard block | ||
| 175 | 175 |  		if ($this->gallery_config->get('allow_rates')) | 
| 176 | 176 |  		{ | 
| 177 | 177 |  			$sort_by_text['ra'] = $this->language->lang('RATING'); | 
| 178 | - $sort_by_sql['ra'] = 'image_rate_points';//@todo: (phpbb_gallery_contest::$mode == phpbb_gallery_contest::MODE_SUM) ? 'image_rate_points' : 'image_rate_avg'; | |
| 178 | + $sort_by_sql['ra'] = 'image_rate_points'; //@todo: (phpbb_gallery_contest::$mode == phpbb_gallery_contest::MODE_SUM) ? 'image_rate_points' : 'image_rate_avg'; | |
| 179 | 179 |  			$sort_by_text['r'] = $this->language->lang('RATES_COUNT'); | 
| 180 | 180 | $sort_by_sql['r'] = 'image_rates'; | 
| 181 | 181 | } | 
| @@ -206,7 +206,7 @@ discard block | ||
| 206 | 206 |  				{ | 
| 207 | 207 |  					trigger_error(sprintf($this->language->lang('TOO_FEW_AUTHOR_CHARS'), $this->config['min_search_author_chars'])); | 
| 208 | 208 | } | 
| 209 | -				$username_parsed = (strpos($username, '*') !== false) ? ' username_clean ' . $this->db->sql_like_expression(str_replace('*', $this->db->get_any_char(), utf8_clean_string($this->db->sql_escape($username)))) : ' username_clean = \'' . $this->db->sql_escape(utf8_clean_string($username)) .'\''; | |
| 209 | +				$username_parsed = (strpos($username, '*') !== false) ? ' username_clean ' . $this->db->sql_like_expression(str_replace('*', $this->db->get_any_char(), utf8_clean_string($this->db->sql_escape($username)))) : ' username_clean = \'' . $this->db->sql_escape(utf8_clean_string($username)) . '\''; | |
| 210 | 210 | $sql = 'SELECT user_id | 
| 211 | 211 | FROM ' . USERS_TABLE . ' | 
| 212 | 212 | WHERE ' . $username_parsed . ' | 
| @@ -224,7 +224,7 @@ discard block | ||
| 224 | 224 | |
| 225 | 225 | if (!empty($user_id)) | 
| 226 | 226 |  			{ | 
| 227 | -				$sql_where[] =  $this->db->sql_in_set('i.image_user_id', $user_id); | |
| 227 | +				$sql_where[] = $this->db->sql_in_set('i.image_user_id', $user_id); | |
| 228 | 228 | } | 
| 229 | 229 | // if we search in an existing search result just add the additional keywords. But we need to use "all search terms"-mode | 
| 230 | 230 | // so we can keep the old keywords in their old mode, but add the new ones as required words | 
| @@ -237,7 +237,7 @@ discard block | ||
| 237 | 237 | else | 
| 238 | 238 |  				{ | 
| 239 | 239 | $search_terms = 'all'; | 
| 240 | -					$keywords = preg_replace('#\s+#u', ' |', $keywords) . ' ' .$add_keywords; | |
| 240 | +					$keywords = preg_replace('#\s+#u', ' |', $keywords) . ' ' . $add_keywords; | |
| 241 | 241 | } | 
| 242 | 242 | } | 
| 243 | 243 |  			$keywords_ary = ($keywords) ? explode(' ', $keywords) : array(); | 
| @@ -262,7 +262,7 @@ discard block | ||
| 262 | 262 | $match_search_query = ''; | 
| 263 | 263 | foreach ($matches as $match) | 
| 264 | 264 |  				{ | 
| 265 | -					$match_search_query .= (($match_search_query) ? ' OR ' : '') . 'LOWER('. $match . ') '; | |
| 265 | +					$match_search_query .= (($match_search_query) ? ' OR ' : '') . 'LOWER(' . $match . ') '; | |
| 266 | 266 |  					$match_search_query .= $this->db->sql_like_expression(str_replace('*', $this->db->get_any_char(), $this->db->get_any_char() . mb_strtolower($word) . $this->db->get_any_char())); | 
| 267 | 267 | } | 
| 268 | 268 |  				$search_query .= ((!$search_query) ? '' : (($search_terms == 'all') ? ' AND ' : ' OR ')) . '(' . $match_search_query . ')'; | 
| @@ -291,7 +291,7 @@ discard block | ||
| 291 | 291 |  				trigger_error('NO_SEARCH_RESULTS'); | 
| 292 | 292 | } | 
| 293 | 293 | $sql_array['SELECT'] = '*, a.album_name, a.album_status, a.album_user_id, a.album_id'; | 
| 294 | - $sql_array['LEFT_JOIN'] = array( | |
| 294 | + $sql_array['LEFT_JOIN'] = array( | |
| 295 | 295 | array( | 
| 296 | 296 | 'FROM' => array($this->albums_table => 'a'), | 
| 297 | 297 | 'ON' => 'a.album_id = i.image_album_id', | 
| @@ -319,7 +319,7 @@ discard block | ||
| 319 | 319 |  			{ | 
| 320 | 320 |  				$this->image->assign_block('imageblock.image', $row, $show_options, $thumbnail_link, $imagename_link); | 
| 321 | 321 | } | 
| 322 | -			$current_page = $page*$this->gallery_config->get('items_per_page'); | |
| 322 | +			$current_page = $page * $this->gallery_config->get('items_per_page'); | |
| 323 | 323 | $this->pagination->generate_template_pagination(array( | 
| 324 | 324 | 'routes' => array( | 
| 325 | 325 | 'phpbbgallery_core_search', |