DakuTree / manga-tracker

application/models/Auth_Model.php

Braces +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
 			if($query->num_rows() > 0) {
 				//username exists, grab email
 				$email = $query->row('email');
-			}else{
+			} else {
 				//username doesn't exist, return FALSE
 				$email = NULL;
 			}

application/controllers/User/Auth/Signup.php

Braces +7 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
 		}
 
 		//login wasn't valid, failed, or this is a fresh login attempt
-		if(!$isValid){
+		if(!$isValid) {
 			//display the email validation form
 			$this->body_data['form_email'] = array(
 					'name'        => 'email',
@@ -86,7 +86,9 @@ 
 	//This continued signup occurs after the user clicks the verification link in their email.
 	private function _continue_signup($verificationCode) : void {
 		//check if validation is valid, if so return email, if not redirect to signup
-		if(!($email = $this->Auth->verificationCheck($verificationCode))) redirect('user/signup');
+		if(!($email = $this->Auth->verificationCheck($verificationCode))) {
+			redirect('user/signup');
+		}
 
 		//validation is valid, proceed as normal
 		$this->form_validation->set_rules('username',         'Username',           'required|min_length[4]|max_length[15]|valid_username|is_unique_username');
@@ -110,14 +112,15 @@ 
 				$this->Auth->verificationComplete($email);
 
 				redirect('help');
-			} else { //@codeCoverageIgnore
+			} else {
+//@codeCoverageIgnore
 				//Signup failed.
 				$isValid = FALSE;
 			}
 		}
 
 		//signup wasn't valid, failed, or this is a fresh signup attempt
-		if(!$isValid){
+		if(!$isValid) {
 			//display the create user form
 			$this->body_data['verificationCode'] = $verificationCode;
 

application/controllers/User/Auth/Forgot_Password.php

Braces +4 added lines, -2 removed lines

@@ -83,11 +83,13 @@
 				if ($change) {
 					//password changed successfully, redirect to login
 					redirect('user/login', 'refresh');
-				} else { //@codeCoverageIgnore
+				} else {
+//@codeCoverageIgnore
 					//password changed unsuccessfully, refresh page.
 					redirect('user/reset_password/'.$code, 'refresh');
 				}
-			} else { //@codeCoverageIgnore
+			} else {
+//@codeCoverageIgnore
 				//form is invalid OR first-time viewing page
 				$this->body_data['reset_code']    = $code;
 				$this->body_data['form_password'] = array(

application/libraries/MY_Form_validation.php

Braces +2 added lines, -3 removed lines

@@ -43,8 +43,7 @@ 
 	public function valid_password(string $password) : bool {
 		if(!($isValid = $this->min_length($password, $this->CI->config->item('min_password_length', 'ion_auth')))) {
 			$this->set_message('valid_password', 'The password is too short!');
-		}
-		elseif(!($isValid = $this->max_length($password, $this->CI->config->item('max_password_length', 'ion_auth')))) {
+		} elseif(!($isValid = $this->max_length($password, $this->CI->config->item('max_password_length', 'ion_auth')))) {
 			$this->set_message('valid_password', 'The password is too long!');
 		}
 		return $isValid;
@@ -108,7 +107,7 @@ 
 	 */
 	public function isRuleValid(string $ruleName) : bool {
 		$isValid = FALSE;
-		if(is_string($ruleName) && $this->has_rule($ruleName)){
+		if(is_string($ruleName) && $this->has_rule($ruleName)) {
 			$isValid = !array_key_exists($ruleName, $this->error_array());
 		}
 		return $isValid;

application/core/MY_Config.php

Braces +11 added lines, -24 removed lines

@@ -13,46 +13,36 @@ 
 	 * @param	bool	$fail_gracefully	Whether to just return FALSE or display an error message
 	 * @return	bool	TRUE if the file was loaded correctly or FALSE on failure
 	 */
-	public function load($file = '', $use_sections = FALSE, $fail_gracefully = FALSE)
-	{
+	public function load($file = '', $use_sections = FALSE, $fail_gracefully = FALSE) {
 		$file = ($file === '') ? 'config' : str_replace('.php', '', $file);
 		$loaded = FALSE;
 
-		foreach ($this->_config_paths as $path)
-		{
-			foreach (array($file, '_secure'.DIRECTORY_SEPARATOR.$file, ENVIRONMENT.DIRECTORY_SEPARATOR.$file) as $location)
-			{
+		foreach ($this->_config_paths as $path) {
+			foreach (array($file, '_secure'.DIRECTORY_SEPARATOR.$file, ENVIRONMENT.DIRECTORY_SEPARATOR.$file) as $location) {
 				$file_path = $path.'config/'.$location.'.php';
-				if (in_array($file_path, $this->is_loaded, TRUE))
-				{
+				if (in_array($file_path, $this->is_loaded, TRUE)) {
 					return TRUE;
 				}
 
-				if ( ! file_exists($file_path))
-				{
+				if ( ! file_exists($file_path)) {
 					continue;
 				}
 
 				include($file_path);
 
-				if ( ! isset($config) OR ! is_array($config))
-				{
-					if ($fail_gracefully === TRUE)
-					{
+				if ( ! isset($config) OR ! is_array($config)) {
+					if ($fail_gracefully === TRUE) {
 						return FALSE;
 					}
 
 					show_error('Your '.$file_path.' file does not appear to contain a valid configuration array.');
 				}
 
-				if ($use_sections === TRUE)
-				{
+				if ($use_sections === TRUE) {
 					$this->config[$file] = isset($this->config[$file])
 						? array_merge($this->config[$file], $config)
 						: $config;
-				}
-				else
-				{
+				} else {
 					$this->config = array_merge($this->config, $config);
 				}
 
@@ -63,12 +53,9 @@ 
 			}
 		}
 
-		if ($loaded === TRUE)
-		{
+		if ($loaded === TRUE) {
 			return TRUE;
-		}
-		elseif ($fail_gracefully === TRUE)
-		{
+		} elseif ($fail_gracefully === TRUE) {
 			return FALSE;
 		}
 

application/controllers/User/Auth/Login.php

Braces +2 added lines, -1 removed lines

@@ -46,7 +46,8 @@
 					$this->session->keep_flashdata('referred_from');
 					if($prevURL = $this->session->flashdata('referred_from')) {
 						redirect($prevURL);
-					} else { //@codeCoverageIgnore
+					} else {
+//@codeCoverageIgnore
 						redirect('/user/dashboard', 'refresh');
 					} //@codeCoverageIgnore
 				} else {

application/controllers/FrontPage.php

Braces +3 added lines, -1 removed lines

@@ -9,7 +9,9 @@
 		$this->header_data['title'] = 'Index';
 		$this->header_data['page']  = 'index';
 
-		if($this->User->logged_in()) redirect('user/dashboard');
+		if($this->User->logged_in()) {
+			redirect('user/dashboard');
+		}
 		$this->_render_page('FrontPage');
 	}
 }

_scripts/deploy.php

Braces +3 added lines, -1 removed lines

@@ -6,7 +6,9 @@
  */
 namespace Deployer;
 
-if(basename(getcwd()) !== 'manga-tracker') die('Bad CWD: Call from manga-tracker with dep -f=_scripts/deploy.php deploy');
+if(basename(getcwd()) !== 'manga-tracker') {
+	die('Bad CWD: Call from manga-tracker with dep -f=_scripts/deploy.php deploy');
+}
 
 require 'recipe/cachetool.php'; //requires deployer/recipes
 require 'recipe/codeigniter.php';

application/models/Tracker/Tracker_Admin_Model.php

Braces +7 added lines, -9 removed lines

@@ -85,7 +85,9 @@ 
 				}
 
 				$siteRateLimits[$row->site_class]++;
-				if($siteRateLimits[$row->site_class] > $hardRateLimit) continue;
+				if($siteRateLimits[$row->site_class] > $hardRateLimit) {
+					continue;
+				}
 
 				usleep(500000); // Delay requests by .5 seconds.
 				if($siteRateLimits[$row->site_class] > 25) {
@@ -175,8 +177,7 @@ 
 
 						print " - Something went wrong?\n";
 					}
-				}
-				else if(array_key_exists('status', $titleData)) {
+				} else if(array_key_exists('status', $titleData)) {
 					// Series has probably been deleted.
 
 					if($this->Tracker->title->updateTitleDataByID((int) $row->title_id, $titleData)) {
@@ -186,8 +187,7 @@ 
 
 						print " - Something went wrong?\n";
 					}
-				}
-				else if($site->canHaveNoChapters) {
+				} else if($site->canHaveNoChapters) {
 					// Previous statements failed, however site can have no chapters.
 					if($this->Tracker->title->updateTitleDataByID((int) $row->title_id, ['latest_chapter' => NULL])) {
 						print " - (No chapters found?)\n";
@@ -196,12 +196,10 @@ 
 
 						print " - Something went wrong?\n";
 					}
-				}
-				else {
+				} else {
 					log_message('error', 'handleUpdate failed due to invalid titleData info?');
 				}
-			}
-			else {
+			} else {
 				// If TitleData does not exist, either something has broken, or we've set up the site wrong.
 
 				//TODO: We should have some way to handle this in the site models.