DakuTree / manga-tracker

application/migrations/008_tracker_add_complete.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 class Migration_Tracker_add_complete extends CI_Migration {
 	public function __construct() {

application/migrations/010_tracker_add_category.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 class Migration_Tracker_add_category extends CI_Migration {
 	public function __construct() {

application/core/MY_Controller.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 	protected $footer_data = array();
 	public    $global_data = array();
 
-	public function __construct(){
+	public function __construct() {
 		parent::__construct();
 
 		//FIXME: This is pretty much a phpUnit hack. Without it phpUnit fails here. We need a proper way to fake user/admin testing.

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();
 
 		//FIXME: This is pretty much a phpUnit hack. Without it phpUnit fails here. We need a proper way to fake user/admin testing.
@@ -80,7 +80,9 @@ 
 	public function __construct() {
 		parent::__construct();
 
-		if($this->ion_auth->logged_in()) redirect('/');
+		if($this->ion_auth->logged_in()) {
+			redirect('/');
+		}
 	}
 }
 

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 class MY_Controller extends CI_Controller {
 	protected $header_data = array();

application/core/MY_Output.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 class MY_Output extends CI_Output {
 	protected $current_header = [

application/models/Site_Model.php

Indentation +2 added lines, -2 removed lines

@@ -228,8 +228,8 @@
 		//Bato.to is annoying and locks stuff behind auth. See: https://github.com/DakuTree/manga-tracker/issues/14#issuecomment-233830855
 		$cookies = [
 			"lang_option={$title_lang}",
-		    "member_id=" . $this->config->item('batoto_cookie_member_id'),
-		    "pass_hash=" . $this->config->item('batoto_cookie_pass_hash')
+			"member_id=" . $this->config->item('batoto_cookie_member_id'),
+			"pass_hash=" . $this->config->item('batoto_cookie_pass_hash')
 		];
 		$data = $this->get_content($title_url, implode("; ", $cookies));
 		if(!$data) {

Spacing +25 added lines, -25 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');
+<?php declare(strict_types = 1); defined('BASEPATH') OR exit('No direct script access allowed');
 
 class Site_Model extends CI_Model {
 	public function __construct() {
@@ -16,10 +16,10 @@ 
 	public function isValidTitleURL(string $title_url) : bool {}
 	public function isValidChapter(string $chapter): bool {}
 
-	protected function get_content(string $url, string $cookie_string = "", string $cookiejar_path = ""){
+	protected function get_content(string $url, string $cookie_string = "", string $cookiejar_path = "") {
 		$ch = curl_init();
 		curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
-		curl_setopt($ch, CURLOPT_ENCODING , "gzip");
+		curl_setopt($ch, CURLOPT_ENCODING, "gzip");
 
 		if(!empty($cookie_string))  curl_setopt($ch, CURLOPT_COOKIE, $cookie_string);
 		if(!empty($cookiejar_path)) curl_setopt($ch, CURLOPT_COOKIEFILE, $cookiejar_path);
@@ -111,8 +111,8 @@ 
 
 				$link = preg_replace('/^(.*\/)(?:[0-9]+\.html)?$/', '$1', (string) $nodes_chapter[0]->getAttribute('href'));
 				$chapterURLSegments = explode('/', $link);
-				$titleData['latest_chapter'] = $chapterURLSegments[5] . (isset($chapterURLSegments[6]) && !empty($chapterURLSegments[6]) ? "/{$chapterURLSegments[6]}" : "");
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
+				$titleData['latest_chapter'] = $chapterURLSegments[5].(isset($chapterURLSegments[6]) && !empty($chapterURLSegments[6]) ? "/{$chapterURLSegments[6]}" : "");
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
 			}
 		} else {
 			//TODO: Throw ERRORS;
@@ -171,8 +171,8 @@ 
 
 				$link = preg_replace('/^(.*\/)(?:[0-9]+\.html)?$/', '$1', (string) $nodes_chapter[0]->getAttribute('href'));
 				$chapterURLSegments = explode('/', $link);
-				$titleData['latest_chapter'] = $chapterURLSegments[5] . (isset($chapterURLSegments[6]) && !empty($chapterURLSegments[6]) ? "/{$chapterURLSegments[6]}" : "");
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
+				$titleData['latest_chapter'] = $chapterURLSegments[5].(isset($chapterURLSegments[6]) && !empty($chapterURLSegments[6]) ? "/{$chapterURLSegments[6]}" : "");
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
 			}
 		} else {
 			//TODO: Throw ERRORS;
@@ -212,7 +212,7 @@ 
 
 		$chapter_parts = explode(':--:', $chapter);
 		return [
-			'url'    => "http://bato.to/reader#" . $chapter_parts[0],
+			'url'    => "http://bato.to/reader#".$chapter_parts[0],
 			'number' => $chapter_parts[1]
 		];
 	}
@@ -228,8 +228,8 @@ 
 		//Bato.to is annoying and locks stuff behind auth. See: https://github.com/DakuTree/manga-tracker/issues/14#issuecomment-233830855
 		$cookies = [
 			"lang_option={$title_lang}",
-		    "member_id=" . $this->config->item('batoto_cookie_member_id'),
-		    "pass_hash=" . $this->config->item('batoto_cookie_pass_hash')
+		    "member_id=".$this->config->item('batoto_cookie_member_id'),
+		    "pass_hash=".$this->config->item('batoto_cookie_pass_hash')
 		];
 		$data = $this->get_content($title_url, implode("; ", $cookies));
 		if(!$data) {
@@ -261,13 +261,13 @@ 
 					preg_match('/^(?:Vol\.(?<volume>\S+) )?(?:Ch.(?<chapter>[^\s:]+)):?.*/', trim($chapter_element->nodeValue), $text);
 
 					$titleData['title']          = html_entity_decode(trim($xpath->query('//h1[@class="ipsType_pagetitle"]')->item(0)->nodeValue));
-					$titleData['latest_chapter'] = substr($chapter_element->getAttribute('href'), 22) . ':--:' . ((!empty($text['volume']) ? 'v'.$text['volume'].'/' : '') . 'c'.$text['chapter']);
+					$titleData['latest_chapter'] = substr($chapter_element->getAttribute('href'), 22).':--:'.((!empty($text['volume']) ? 'v'.$text['volume'].'/' : '').'c'.$text['chapter']);
 
 					$dateString = $updated_element->nodeValue;
 					if($dateString == 'An hour ago') {
 						$dateString = '1 hour ago';
 					}
-					$titleData['last_updated']   = date("Y-m-d H:i:s", strtotime(preg_replace('/ (-|\[A\]).*$/', '', $dateString)));
+					$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime(preg_replace('/ (-|\[A\]).*$/', '', $dateString)));
 				} else {
 					log_message('error', "Batoto: Regex missing <td> ({$title_url})");
 					return NULL;
@@ -315,7 +315,7 @@ 
 		*/
 
 		$chapterData = [
-			'url'    => 'http://dynasty-scans.com/chapters/' . $title_parts[0].'_'.$chapter,
+			'url'    => 'http://dynasty-scans.com/chapters/'.$title_parts[0].'_'.$chapter,
 			'number' => ''
 		];
 
@@ -355,14 +355,14 @@ 
 
 			preg_match('/<b>.*<\/b>/', $data, $matchesT);
 			preg_match('/\/doujins\/[^"]+">(.+)?(?=<\/a>)<\/a>/', $data, $matchesD);
-			$titleData['title'] = (!empty($matchesD) ? (substr($matchesD[1], 0, -7) !== 'Original' ? substr($matchesD[1], 0, -7).' - ' : '') : '') . substr($matchesT[0], 3, -4);
+			$titleData['title'] = (!empty($matchesD) ? (substr($matchesD[1], 0, -7) !== 'Original' ? substr($matchesD[1], 0, -7).' - ' : '') : '').substr($matchesT[0], 3, -4);
 
 			$data = preg_replace('/^[\S\s]*(<dl class=\'chapter-list\'>[\S\s]*<\/dl>)[\S\s]*$/', '$1', $data);
 			preg_match_all('/<dd>[\s\S]+?(?=<\/dd>)<\/dd>/', $data, $matches);
 			$latest_chapter_html = array_pop($matches[0]);
 
 			preg_match('/\/chapters\/([^"]+)/', $latest_chapter_html, $matches);
-			$titleData['latest_chapter'] = substr($matches[1], strlen($title_url)+1);
+			$titleData['latest_chapter'] = substr($matches[1], strlen($title_url) + 1);
 			//FIXME: THIS IS A TEMP FIX, SEE https://github.com/DakuTree/manga-tracker/issues/58
 			if(!$titleData['latest_chapter']) {
 				log_message('error', 'DynastyScans::getTitleData cannot parse title properly as it contains oneshot. || URL: '.$title_url);
@@ -370,13 +370,13 @@ 
 			}
 
 			preg_match('/<small>released (.*)<\/small>/', $latest_chapter_html, $matches);
-			$titleData['last_updated']   = date("Y-m-d H:i:s", strtotime(str_replace('\'', '', $matches[1])));
+			$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime(str_replace('\'', '', $matches[1])));
 		} elseif($title_parts[1] == '1') {
 			$data = $this->get_content('http://dynasty-scans.com/chapters/'.$title_url);
 
 			preg_match('/<b>.*<\/b>/', $data, $matchesT);
 			preg_match('/\/doujins\/[^"]+">(.+)?(?=<\/a>)<\/a>/', $data, $matchesD);
-			$titleData['title'] = (!empty($matchesD) ? ($matchesD[1] !== 'Original' ? $matchesD[1].' - ' : '') : '') . substr($matchesT[0], 3, -4);
+			$titleData['title'] = (!empty($matchesD) ? ($matchesD[1] !== 'Original' ? $matchesD[1].' - ' : '') : '').substr($matchesT[0], 3, -4);
 
 			$titleData['latest_chapter'] = 'oneshot'; //This will never change
 
@@ -442,7 +442,7 @@ 
 				$nodes_chapter = $xpath->query("td[1]/a", $nodes_row[0]);
 
 				$titleData['latest_chapter'] = preg_replace('/^.*\/([0-9]+)$/', '$1', (string) $nodes_chapter[0]->getAttribute('href'));
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
 			}
 		} else {
 			//TODO: Throw ERRORS;
@@ -500,7 +500,7 @@ 
 				$nodes_chapter = $xpath->query("td[1]/a", $nodes_row[0]);
 
 				$titleData['latest_chapter'] = preg_replace('/^.*\/(.*?\/[0-9]+)\/[0-9]+$/', '$1', (string) $nodes_chapter[0]->getAttribute('href'));
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->nodeValue));
 			}
 		} else {
 			//TODO: Throw ERRORS;
@@ -565,8 +565,8 @@ 
 				$titleData['title'] = trim((string) $xml->channel->title);
 
 				$chapterURLSegments = explode('/', ((string) $xml->channel->item[0]->link));
-				$titleData['latest_chapter'] = preg_replace('/^.*?([0-9]+)$/', '$1', $chapterURLSegments[7]) . ':--:' . $chapterURLSegments[6];
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $xml->channel->item[0]->pubDate));
+				$titleData['latest_chapter'] = preg_replace('/^.*?([0-9]+)$/', '$1', $chapterURLSegments[7]).':--:'.$chapterURLSegments[6];
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $xml->channel->item[0]->pubDate));
 			}
 		} else {
 			//TODO: Throw ERRORS;
@@ -645,8 +645,8 @@ 
 
 					$link = (string) $nodes_chapter[0]->getAttribute('href');
 					$chapterURLSegments = explode('/', preg_replace('/\?.*$/', '', $link));
-					$titleData['latest_chapter'] = $chapterURLSegments[3] . ':--:' . preg_replace('/.*?([0-9]+)$/', '$1', $link);
-					$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->textContent));
+					$titleData['latest_chapter'] = $chapterURLSegments[3].':--:'.preg_replace('/.*?([0-9]+)$/', '$1', $link);
+					$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) $nodes_latest[0]->textContent));
 				}
 			} else {
 				//TODO: Throw ERRORS;
@@ -681,7 +681,7 @@ 
 		$chapter_parts = explode('/', $chapter);
 		return [
 			'url'    => "https://reader.kireicake.com/read/{$title_url}/{$chapter}",
-			'number' => ($chapter_parts[1] !== '0' ? "v{$chapter_parts[1]}/" : '') . "c{$chapter_parts[2]}" . (isset($chapter_parts[3]) ? ".{$chapter_parts[3]}" : '')/*)*/
+			'number' => ($chapter_parts[1] !== '0' ? "v{$chapter_parts[1]}/" : '')."c{$chapter_parts[2]}".(isset($chapter_parts[3]) ? ".{$chapter_parts[3]}" : '')/*)*/
 		];
 	}
 
@@ -712,7 +712,7 @@ 
 
 				$link = (string) $nodes_chapter[0]->getAttribute('href');
 				$titleData['latest_chapter'] = preg_replace('/.*\/read\/.*?\/(.*?)\/$/', '$1', $link);
-				$titleData['last_updated'] =  date("Y-m-d H:i:s", strtotime((string) str_replace('.', '', explode(',', $nodes_latest[0]->textContent)[1])));
+				$titleData['last_updated'] = date("Y-m-d H:i:s", strtotime((string) str_replace('.', '', explode(',', $nodes_latest[0]->textContent)[1])));
 			}
 		} else {
 			//TODO: Throw ERRORS;

Braces +61 added lines, -21 removed lines

@@ -16,13 +16,17 @@ 
 	public function isValidTitleURL(string $title_url) : bool {}
 	public function isValidChapter(string $chapter): bool {}
 
-	protected function get_content(string $url, string $cookie_string = "", string $cookiejar_path = ""){
+	protected function get_content(string $url, string $cookie_string = "", string $cookiejar_path = "") {
 		$ch = curl_init();
 		curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
 		curl_setopt($ch, CURLOPT_ENCODING , "gzip");
 
-		if(!empty($cookie_string))  curl_setopt($ch, CURLOPT_COOKIE, $cookie_string);
-		if(!empty($cookiejar_path)) curl_setopt($ch, CURLOPT_COOKIEFILE, $cookiejar_path);
+		if(!empty($cookie_string)) {
+			curl_setopt($ch, CURLOPT_COOKIE, $cookie_string);
+		}
+		if(!empty($cookiejar_path)) {
+			curl_setopt($ch, CURLOPT_COOKIEFILE, $cookiejar_path);
+		}
 
 		//Some sites check the useragent for stuff, use a pre-defined user-agent to avoid stuff.
 		curl_setopt($ch, CURLOPT_USERAGENT, 'Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2824.0 Safari/537.36');
@@ -68,12 +72,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9_]+$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (MangaFox): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (MangaFox): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^(?:v[0-9a-zA-Z]+\/)?c[0-9\.]+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (MangaFox): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (MangaFox): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -129,12 +137,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9_]+$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (MangaFox): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (MangaFox): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^(?:v[0-9]+\/)?c[0-9]+(?:\.[0-9]+)?$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (MangaFox): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (MangaFox): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -195,7 +207,9 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9%-]+:--:(?:English|Spanish|French|German|Portuguese|Turkish|Indonesian|Greek|Filipino|Italian|Polish|Thai|Malay|Hungarian|Romanian|Arabic|Hebrew|Russian|Vietnamese|Dutch)$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (Batoto): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (Batoto): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
@@ -203,7 +217,9 @@ 
 		//       Preferably we'd just use /^[0-9a-z]+:--:(v[0-9]+\/)?c[0-9]+(\.[0-9]+)?$/
 
 		$success = (bool) preg_match('/^[0-9a-z]+:--:.+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (Batoto): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (Batoto): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -296,12 +312,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9_]+:--:(?:0|1)$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (DynastyScans): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (DynastyScans): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^[0-9a-z_]+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (DynastyScans): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (DynastyScans): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -407,12 +427,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9-]+$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (MangaPanda): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (MangaPanda): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^[0-9]+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (MangaPanda): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (MangaPanda): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -459,12 +483,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9_]+$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (MangaStream): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (MangaStream): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^(.*?)\/[0-9]+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (MangaStream): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (MangaStream): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -533,12 +561,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[0-9]+:--:(?:en|zh-hant|zh-hans|th|id):--:[a-z0-9-]+:--:(?:drama|fantasy|comedy|action|slice-of-life|romance|superhero|thriller|sports|sci-fi)$/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (WebToons): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (WebToons): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^[0-9]+:--:.*$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (WebToons): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (WebToons): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -595,12 +627,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[A-Za-z0-9-]+/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (KissManga): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (KissManga): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^.*?:--:[0-9]+$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (KissManga): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (KissManga): '.$chapter);
+		}
 		return $success;
 	}
 
@@ -667,12 +703,16 @@ 
 
 	public function isValidTitleURL(string $title_url) : bool {
 		$success = (bool) preg_match('/^[a-z0-9_]+/', $title_url);
-		if(!$success) log_message('error', "Invalid Title URL (KireiCake): {$title_url}");
+		if(!$success) {
+			log_message('error', "Invalid Title URL (KireiCake): {$title_url}");
+		}
 		return $success;
 	}
 	public function isValidChapter(string $chapter) : bool {
 		$success = (bool) preg_match('/^en\/[0-9]+(?:\/[0-9]+(?:\/[0-9]+(?:\/[0-9]+)?)?)?$/', $chapter);
-		if(!$success) log_message('error', 'Invalid Chapter (KireiCake): '.$chapter);
+		if(!$success) {
+			log_message('error', 'Invalid Chapter (KireiCake): '.$chapter);
+		}
 		return $success;
 	}
 

Upper-Lower-Casing +15 added lines, -15 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');
+<?php declare(strict_types=1); defined('BASEPATH') or exit('No direct script access allowed');
 
 class Site_Model extends CI_Model {
 	public function __construct() {
@@ -94,9 +94,9 @@ 
 			//$data = preg_replace('/^[\S\s]*(<body id="body">[\S\s]*<\/body>)[\S\s]*$/', '$1', $data);
 
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 
@@ -155,9 +155,9 @@ 
 			//$data = preg_replace('/^[\S\s]*(<body id="body">[\S\s]*<\/body>)[\S\s]*$/', '$1', $data);
 
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 			$nodes_title = $xpath->query("//meta[@property='og:title']");
@@ -243,9 +243,9 @@ 
 		if(strpos($data, '>Register now<') === FALSE) {
 			//Auth was successful
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 			$nodes = $xpath->query("(//div/div)[last()]/table/tbody/tr[2]");
@@ -426,9 +426,9 @@ 
 			//$data = preg_replace('/^[\S\s]*(<body id="body">[\S\s]*<\/body>)[\S\s]*$/', '$1', $data);
 
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 
@@ -485,9 +485,9 @@ 
 			//$data = preg_replace('/^[\S\s]*(<body id="body">[\S\s]*<\/body>)[\S\s]*$/', '$1', $data);
 
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 
@@ -629,9 +629,9 @@ 
 				$data = preg_replace('/^[\S\s]*(<div id="leftside">[\S\s]*)<div id="rightside">[\S\s]*$/', '$1', $data);
 
 				$dom = new DOMDocument();
-				libxml_use_internal_errors(true);
+				libxml_use_internal_errors(TRUE);
 				$dom->loadHTML($data);
-				libxml_use_internal_errors(false);
+				libxml_use_internal_errors(FALSE);
 
 				$xpath = new DOMXPath($dom);
 
@@ -695,9 +695,9 @@ 
 			$data = preg_replace('/^[\S\s]*(<article>[\S\s]*)<\/article>[\S\s]*$/', '$1', $data);
 
 			$dom = new DOMDocument();
-			libxml_use_internal_errors(true);
+			libxml_use_internal_errors(TRUE);
 			$dom->loadHTML($data);
-			libxml_use_internal_errors(false);
+			libxml_use_internal_errors(FALSE);
 
 			$xpath = new DOMXPath($dom);
 

application/models/User_Model.php

Indentation +12 added lines, -12 removed lines

@@ -40,9 +40,9 @@ 
 		$this->load->database();
 
 		$query = $this->db->select('*')
-		                  ->from('auth_users')
-		                  ->where('username', $username)
-		                  ->get();
+						  ->from('auth_users')
+						  ->where('username', $username)
+						  ->get();
 
 		return (bool) $query->num_rows();
 	}
@@ -64,9 +64,9 @@ 
 			$this->load->database();
 
 			$query = $this->db->select('email')
-			                  ->from('auth_users')
-			                  ->where('username', $identity)
-			                  ->get();
+							  ->from('auth_users')
+							  ->where('username', $identity)
+							  ->get();
 
 			if($query->num_rows() > 0) {
 				//username exists, grab email
@@ -84,9 +84,9 @@ 
 		$user = NULL;
 
 		$query = $this->db->select('*')
-		                  ->from('auth_users')
-		                  ->where('username', $username)
-		                  ->get();
+						  ->from('auth_users')
+						  ->where('username', $username)
+						  ->get();
 
 		if($query->num_rows() > 0) {
 			$user = $query->row();
@@ -114,9 +114,9 @@ 
 	
 	public function get_id_from_api_key(string $api_key) {
 		$query = $this->db->select('id')
-		                  ->from('auth_users')
-		                  ->where('api_key', $api_key)
-		                  ->get();
+						  ->from('auth_users')
+						  ->where('api_key', $api_key)
+						  ->get();
 
 		if($query->num_rows() > 0) {
 			$userID = $query->row('id');

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');
+<?php declare(strict_types = 1); defined('BASEPATH') OR exit('No direct script access allowed');
 
 class User_Model extends CI_Model {
 	public $id;
@@ -71,7 +71,7 @@ 
 			if($query->num_rows() > 0) {
 				//username exists, grab email
 				$email = $query->row('email');
-			}else{
+			} else {
 				//username doesn't exist, return FALSE
 				$email = FALSE;
 			}
@@ -103,7 +103,7 @@ 
 	public function get_new_api_key() : string {
 		$api_key = NULL;
 		if($this->logged_in()) {
-			$api_key = substr("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ", mt_rand(0, 51), 1) . substr(md5((string) time()), 1);
+			$api_key = substr("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ", mt_rand(0, 51), 1).substr(md5((string) time()), 1);
 
 			$this->db->where('id', $this->id);
 			$this->db->update('auth_users', ['api_key' => $api_key]);

Braces +1 added lines, -1 removed lines

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

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare(strict_types=1); defined('BASEPATH') OR exit('No direct script access allowed');
+<?php declare(strict_types=1); defined('BASEPATH') or exit('No direct script access allowed');
 
 class User_Model extends CI_Model {
 	public $id;

application/config/email.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 /*
 | -------------------------------------------------------------------

application/config/migration.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-defined('BASEPATH') OR exit('No direct script access allowed');
+defined('BASEPATH') or exit('No direct script access allowed');
 
 /*
 |--------------------------------------------------------------------------

application/config/testing/migration.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('BASEPATH') OR exit('No direct script access allowed');
+<?php defined('BASEPATH') or exit('No direct script access allowed');
 
 /*
 |--------------------------------------------------------------------------