DakuTree / manga-tracker

application/tests_codeception/_support/AcceptanceTester.php

Braces +1 added lines, -2 removed lines

@@ -16,8 +16,7 @@
  *
  * @SuppressWarnings(PHPMD)
 */
-class AcceptanceTester extends \Codeception\Actor
-{
+class AcceptanceTester extends \Codeception\Actor {
     use _generated\AcceptanceTesterActions;
 
    /**

application/tests_codeception/_support/FunctionalTester.php

Braces +1 added lines, -2 removed lines

@@ -16,8 +16,7 @@
  *
  * @SuppressWarnings(PHPMD)
 */
-class FunctionalTester extends \Codeception\Actor
-{
+class FunctionalTester extends \Codeception\Actor {
     use _generated\FunctionalTesterActions;
 
    /**

application/libraries/MY_Form_validation.php

Braces +2 added lines, -3 removed lines

@@ -30,8 +30,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;
@@ -91,7 +90,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 = !in_array($ruleName, array_keys($this->error_array()), TRUE);
 		}
 		return $isValid;

application/models/User_Options_Model.php

Braces +10 added lines, -3 removed lines

@@ -137,7 +137,9 @@ 
 				}
 
 				$success = $this->set_db($idData, $valueData);
-				if($success) $this->session->unset_tempdata("option_{$option}");
+				if($success) {
+					$this->session->unset_tempdata("option_{$option}");
+				}
 			} else {
 				$success = FALSE;
 			}
@@ -164,7 +166,9 @@ 
 			}
 
 			//Overall fallback method.
-			if(!isset($value)) $value = $this->options[$option]['default'];
+			if(!isset($value)) {
+				$value = $this->options[$option]['default'];
+			}
 		} else {
 			$value = FALSE;
 		}
@@ -215,7 +219,10 @@ 
 				//This should never happen.
 				break;
 		}
-		if(!isset($value)) $value = FALSE; //FIXME: This won't play nice with BOOL type false?
+		if(!isset($value)) {
+			$value = FALSE;
+		}
+		//FIXME: This won't play nice with BOOL type false?
 
 		return $value;
 	}

application/models/Tracker/Tracker_Title_Model.php

Braces +6 added lines, -2 removed lines

@@ -35,9 +35,13 @@
 			$titleID = $id;
 		} else {
 			//TODO: Check if title is valid URL!
-			if($create) $titleID = $this->addTitle($titleURL, $siteID);
+			if($create) {
+				$titleID = $this->addTitle($titleURL, $siteID);
+			}
+		}
+		if(!isset($titleID) || !$titleID) {
+			$titleID = 0;
 		}
-		if(!isset($titleID) || !$titleID) $titleID = 0;
 
 		return ($returnData && $titleID !== 0 ? $query->row_array() : $titleID);
 	}

application/core/MY_Controller.php

Braces +4 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 	protected $footer_data = array();
 	public    $global_data = array();
 
-	public function __construct(){
+	public function __construct() {
 		parent::__construct();
 
 		$this->load->driver('cache', array('adapter' => 'file', 'backup' => 'apc')); //Sadly we can't autoload this with params
@@ -88,7 +88,9 @@ 
 	public function __construct() {
 		parent::__construct();
 
-		if($this->ion_auth->logged_in()) redirect('/');
+		if($this->ion_auth->logged_in()) {
+			redirect('/');
+		}
 	}
 }
 

application/models/Tracker/Sites/WebToons.php

Braces +2 added lines, -1 removed lines

@@ -45,7 +45,8 @@
 
 		$content = $this->get_content($fullURL);
 		$data = $content['body'];
-		if($data !== 'Can\'t find the manga series.') { //FIXME: We should check for he proper error here.
+		if($data !== 'Can\'t find the manga series.') {
+//FIXME: We should check for he proper error here.
 			$xml = simplexml_load_string($data) or die("Error: Cannot create object");
 			if(isset($xml->{'channel'}->item[0])) {
 				$titleData['title'] = trim((string) $xml->{'channel'}->title);

application/controllers/TitleHistory.php

Braces +3 added lines, -1 removed lines

@@ -17,7 +17,9 @@
 		$this->body_data['totalPages']  = $historyData['totalPages'];
 		$this->body_data['titleID']     = (int) $titleID;
 
-		if($page > $this->body_data['totalPages'] && $page > 1) redirect("/history/{$titleID}/1");
+		if($page > $this->body_data['totalPages'] && $page > 1) {
+			redirect("/history/{$titleID}/1");
+		}
 
 		$this->_render_page("TitleHistory");
 	}

application/controllers/User/History.php

Braces +6 added lines, -2 removed lines

@@ -6,7 +6,9 @@ 
 	}
 
 	public function index(int $page = 1) : void {
-		if($page === 0) redirect('user/history/1');
+		if($page === 0) {
+			redirect('user/history/1');
+		}
 
 		$this->header_data['title'] = "History";
 		$this->header_data['page']  = "history";
@@ -16,7 +18,9 @@ 
 		$this->body_data['currentPage'] = (int) $page;
 		$this->body_data['totalPages']  = $historyData['totalPages'];
 
-		if($page > $this->body_data['totalPages'] && $page > 1) redirect('user/history/1');
+		if($page > $this->body_data['totalPages'] && $page > 1) {
+			redirect('user/history/1');
+		}
 
 		$this->_render_page('User/History');
 	}