DakuTree / manga-tracker

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/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');
 	}

application/controllers/User/PublicList.php

Braces +3 added lines, -1 removed lines

@@ -60,7 +60,9 @@
 			$show_404 = TRUE;
 		}
 
-		if($show_404) show_404();
+		if($show_404) {
+			show_404();
+		}
 	}
 
 	private function _walk_recursive_remove (array $array, callable $callback) : array {

application/controllers/User/Favourites.php

Braces +6 added lines, -2 removed lines

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

application/helpers/generic_helper.php

Braces +6 added lines, -5 removed lines

@@ -32,18 +32,19 @@
 	return $time_string;
 }
 
-if (!function_exists('http_parse_headers')) { #http://www.php.net/manual/en/function.http-parse-headers.php#112917
+if (!function_exists('http_parse_headers')) {
+#http://www.php.net/manual/en/function.http-parse-headers.php#112917
 	function http_parse_headers (string $raw_headers) : array {
 		$headers = array(); // $headers = [];
 		foreach (explode("\n", $raw_headers) as $i => $h) {
 			$h = explode(':', $h, 2);
-			if (isset($h[1])){
-				if(!isset($headers[$h[0]])){
+			if (isset($h[1])) {
+				if(!isset($headers[$h[0]])) {
 					$headers[$h[0]] = trim($h[1]);
-				}else if(is_array($headers[$h[0]])){
+				} else if(is_array($headers[$h[0]])) {
 					$tmp = array_merge($headers[$h[0]],array(trim($h[1])));
 					$headers[$h[0]] = $tmp;
-				}else{
+				} else {
 					$tmp = array_merge(array($headers[$h[0]]),array(trim($h[1])));
 					$headers[$h[0]] = $tmp;
 				}

application/models/Tracker/Tracker_List_Model.php

Braces +6 added lines, -2 removed lines

@@ -114,11 +114,15 @@
 
 							if($sortOrder == 'asc') {
 								$unreadSort = ($a_text <=> $b_text);
-								if($unreadSort) return $unreadSort;
+								if($unreadSort) {
+									return $unreadSort;
+								}
 								return $a_text2 <=> $b_text2;
 							} else {
 								$unreadSort = ($a_text <=> $b_text);
-								if($unreadSort) return $unreadSort;
+								if($unreadSort) {
+									return $unreadSort;
+								}
 								return $b_text2 <=> $a_text2;
 							}
 						});

application/controllers/TitleHistory.php

Braces +3 added lines, -1 removed lines

@@ -23,7 +23,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");
 

_scripts/generate_foolslide.php

Braces +10 added lines, -4 removed lines

@@ -1,6 +1,8 @@ 
 <?php declare(strict_types=1);
 
-if(!extension_loaded('gd')) die('GD ext is required to run this!');
+if(!extension_loaded('gd')) {
+	die('GD ext is required to run this!');
+}
 
 chdir(dirname(__FILE__).'/../'); //Just to make things easier, change dir to project root.
 
@@ -9,7 +11,7 @@ 
 	private $className;
 
 	public function __construct() {
-		if(isset($_SERVER['argv']) && count($_SERVER['argv']) === 3){
+		if(isset($_SERVER['argv']) && count($_SERVER['argv']) === 3) {
 			$this->baseURL   = rtrim($_SERVER['argv'][1], '/');
 			$this->className = $_SERVER['argv'][2];
 
@@ -98,7 +100,9 @@ 
 		$baseFile = file_get_contents($baseFileName);
 
 		$parse = parse_url($this->baseURL);
-		if(strpos($baseFile, $parse['host']) !== false) die("Domain already exists in userscript?");
+		if(strpos($baseFile, $parse['host']) !== false) {
+			die("Domain already exists in userscript?");
+		}
 
 		preg_match('/\@updated      ([0-9\-]+)[\r\n]+.*?\@version      ([0-9\.]+)/s', $baseFile, $matches);
 
@@ -188,7 +192,9 @@ 
 			}
 		}
 
-		if(empty($titleArr)) die("API isn't returning any titles?");
+		if(empty($titleArr)) {
+			die("API isn't returning any titles?");
+		}
 		return $titleArr;
 	}
 }

application/core/MY_Log.php

Braces +10 added lines, -20 removed lines

@@ -67,8 +67,7 @@ 
 		// detect and register all PHP errors in this log hence forth
 		ErrorHandler::register($this->log);
 
-		if ($this->config['introspection_processor'])
-		{
+		if ($this->config['introspection_processor']) {
 			// add controller and line number info to each log message
 			// 2 = depth in the stacktrace to ignore. This gives us the file
 			// making the call to log_message();
@@ -76,10 +75,8 @@ 
 		}
 
 		// decide which handler(s) to use
-		foreach ($this->config['handlers'] as $value)
-		{
-			switch ($value)
-			{
+		foreach ($this->config['handlers'] as $value) {
+			switch ($value) {
 				case 'file':
 					$handler = new RotatingFileHandler($this->config['file_logfile']);
 					$formatter = new LineFormatter(null, null, $config['file_multiline']);
@@ -140,35 +137,28 @@ 
 	 * @param $msg
 	 * @return bool
 	 */
-	public function write_log($level = 'error', $msg)
-	{
+	public function write_log($level = 'error', $msg) {
 		$level = strtoupper($level);
 
 		// verify error level
-		if (!isset($this->_levels[$level]))
-		{
+		if (!isset($this->_levels[$level])) {
 			$this->log->addError('unknown error level: ' . $level);
 			$level = 'ALL';
 		}
 
 		// filter out anything in $this->config['exclusion_list']
-		if (!empty($this->config['exclusion_list']))
-		{
-			foreach ($this->config['exclusion_list'] as $findme)
-			{
+		if (!empty($this->config['exclusion_list'])) {
+			foreach ($this->config['exclusion_list'] as $findme) {
 				$pos = strpos($msg, $findme);
-				if ($pos !== false)
-				{
+				if ($pos !== false) {
 					// just exit now - we don't want to log this error
 					return true;
 				}
 			}
 		}
 
-		if ($this->_levels[$level] <= $this->config['threshold'])
-		{
-			switch ($level)
-			{
+		if ($this->_levels[$level] <= $this->config['threshold']) {
+			switch ($level) {
 				case 'ERROR':
 					$this->log->addError($msg);
 					break;