DakuTree / manga-tracker

application/models/Auth_Model.php

Doc Comments +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 
 	/**
 	 * @param string $verificationCode
-	 * @return mixed
+	 * @return string
 	 */
 	public function verification_check(string $verificationCode) {
 		//user is trying to validate their email for signup, check if verification code is still valid/exists

Indentation +3 added lines, -3 removed lines

@@ -77,9 +77,9 @@
 	public function verification_check(string $verificationCode) {
 		//user is trying to validate their email for signup, check if verification code is still valid/exists
 		$query = $this->db->select('email, verification_code_time')
-		                  ->from('auth_signup_verification')
-		                  ->where(array('verification_code' => $verificationCode))
-		                  ->get();
+						  ->from('auth_signup_verification')
+						  ->where(array('verification_code' => $verificationCode))
+						  ->get();
 
 		$return = FALSE;
 		if($query->num_rows() > 0) {

application/models/Tracker_Model.php

Doc Comments +3 added lines

@@ -88,6 +88,9 @@
 		return $arr;
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getSiteDataFromURL(string $site_url) {
 		$query = $this->db->select('id, site_class')
 		                  ->from('tracker_sites')

Indentation +48 added lines, -48 removed lines

@@ -90,9 +90,9 @@ 
 
 	public function getSiteDataFromURL(string $site_url) {
 		$query = $this->db->select('id, site_class')
-		                  ->from('tracker_sites')
-		                  ->where('site', $site_url)
-		                  ->get();
+						  ->from('tracker_sites')
+						  ->where('site', $site_url)
+						  ->get();
 
 		if($query->num_rows() > 0) {
 			$siteData = $query->row();
@@ -103,10 +103,10 @@ 
 
 	public function getTitleID(string $titleURL, int $siteID) {
 		$query = $this->db->select('id')
-		                  ->from('tracker_titles')
-		                  ->where('title_url', $titleURL)
-		                  ->where('site_id', $siteID)
-		                  ->get();
+						  ->from('tracker_titles')
+						  ->where('title_url', $titleURL)
+						  ->where('site_id', $siteID)
+						  ->get();
 
 		if($query->num_rows() > 0) {
 			$titleID = $query->row('id');
@@ -140,9 +140,9 @@ 
 			}
 			if($this->db->select('*')->where('user_id', $userID)->where('title_id', $titleID)->get('tracker_chapters')->num_rows() > 0) {
 				$success = $this->db->set(['current_chapter' => $chapter, 'last_updated' => NULL])
-				                    ->where('user_id', $userID)
-				                    ->where('title_id', $titleID)
-				                    ->update('tracker_chapters');
+									->where('user_id', $userID)
+									->where('title_id', $titleID)
+									->update('tracker_chapters');
 			} else {
 				$success = $this->db->insert('tracker_chapters', [
 					'user_id'         => $userID,
@@ -157,31 +157,31 @@ 
 
 	public function updateTrackerByID(int $userID, int $chapterID, string $chapter) : bool {
 		$success = $this->db->set(['current_chapter' => $chapter, 'last_updated' => NULL])
-		                    ->where('user_id', $userID)
-		                    ->where('id', $chapterID)
-		                    ->update('tracker_chapters');
+							->where('user_id', $userID)
+							->where('id', $chapterID)
+							->update('tracker_chapters');
 
 		return (bool) $success;
 	}
 
 	public function deleteTrackerByID(int $userID, int $chapterID) {
 		$success = $this->db->where('user_id', $userID)
-		                    ->where('id', $chapterID)
-		                    ->delete('tracker_chapters');
+							->where('id', $chapterID)
+							->delete('tracker_chapters');
 
 		return (bool) $success;
 	}
 	private function updateTitleById(int $id, string $latestChapter) {
 		//FIXME: Really not too happy with how we're doing history stuff here, it just feels messy.
 		$query = $this->db->select('latest_chapter, last_updated')
-		                  ->from('tracker_titles')
-		                  ->where('id', $id)
-		                  ->get();
+						  ->from('tracker_titles')
+						  ->where('id', $id)
+						  ->get();
 		$row = $query->row();
 
 		$success = $this->db->set(['latest_chapter' => $latestChapter]) //last_updated gets updated via a trigger if something changes
-		                    ->where('id', $id)
-		                    ->update('tracker_titles');
+							->where('id', $id)
+							->update('tracker_titles');
 
 		//Update History
 		$this->History->updateTitleHistory($id, $row->latest_chapter, $row->last_updated);
@@ -190,16 +190,16 @@ 
 	}
 	private function updateTitleDataById(int $id, array $titleData) {
 		$success = $this->db->set($titleData)
-		                    ->where('id', $id)
-		                    ->update('tracker_titles');
+							->where('id', $id)
+							->update('tracker_titles');
 
 		return (bool) $success;
 	}
 	private function addTitle(string $titleURL, int $siteID) {
 		$query = $this->db->select('site, site_class')
-		                  ->from('tracker_sites')
-		                  ->where('id', $siteID)
-		                  ->get();
+						  ->from('tracker_sites')
+						  ->where('id', $siteID)
+						  ->get();
 
 		$titleData = $this->sites->{$query->row()->site_class}->getTitleData($titleURL);
 		$this->db->insert('tracker_titles', array_merge($titleData, ['title_url' => $titleURL, 'site_id' => $siteID]));
@@ -224,18 +224,18 @@ 
 		                      tracker_titles.last_updated,
 		                      from_unixtime(MAX(auth_users.last_login)) AS timestamp
 		                  ')
-		                  ->from('tracker_titles')
-		                  ->join('tracker_sites', 'tracker_sites.id = tracker_titles.site_id', 'left')
-		                  ->join('tracker_chapters', 'tracker_titles.id = tracker_chapters.title_id', 'left')
-		                  ->join('auth_users', 'tracker_chapters.user_id = auth_users.id', 'left')
-		                  ->where('tracker_sites.status = "enabled"')
-		                  ->where('(`complete` = "N" AND (`latest_chapter` = NULL OR `last_checked` < DATE_SUB(NOW(), INTERVAL 12 HOUR)))', NULL, FALSE) //TODO: Each title should have specific interval time?
-		                  ->or_where('(`complete` = "Y" AND `last_checked` < DATE_SUB(NOW(), INTERVAL 1 WEEK))', NULL, FALSE)
-		                  ->group_by('tracker_titles.id')
-		                  ->having('timestamp IS NOT NULL')
-		                  ->having('timestamp > DATE_SUB(NOW(), INTERVAL 120 HOUR)')
-		                  ->order_by('tracker_titles.title', 'ASC')
-		                  ->get();
+						  ->from('tracker_titles')
+						  ->join('tracker_sites', 'tracker_sites.id = tracker_titles.site_id', 'left')
+						  ->join('tracker_chapters', 'tracker_titles.id = tracker_chapters.title_id', 'left')
+						  ->join('auth_users', 'tracker_chapters.user_id = auth_users.id', 'left')
+						  ->where('tracker_sites.status = "enabled"')
+						  ->where('(`complete` = "N" AND (`latest_chapter` = NULL OR `last_checked` < DATE_SUB(NOW(), INTERVAL 12 HOUR)))', NULL, FALSE) //TODO: Each title should have specific interval time?
+						  ->or_where('(`complete` = "Y" AND `last_checked` < DATE_SUB(NOW(), INTERVAL 1 WEEK))', NULL, FALSE)
+						  ->group_by('tracker_titles.id')
+						  ->having('timestamp IS NOT NULL')
+						  ->having('timestamp > DATE_SUB(NOW(), INTERVAL 120 HOUR)')
+						  ->order_by('tracker_titles.title', 'ASC')
+						  ->get();
 
 		if($query->num_rows() > 0) {
 			foreach ($query->result() as $row) {
@@ -248,8 +248,8 @@ 
 						//Make sure last_checked is always updated on successful run.
 						//CHECK: Is there a reason we aren't just doing this in updateTitleById?
 						$this->db->set('last_checked', 'CURRENT_TIMESTAMP', FALSE)
-						         ->where('id', $row->id)
-						         ->update('tracker_titles');
+								 ->where('id', $row->id)
+								 ->update('tracker_titles');
 
 						//Update History
 						$this->History->updateTitleHistory((int) $row->id, $titleData['latest_chapter'], $titleData['last_updated']);
@@ -362,9 +362,9 @@ 
 	}
 	public function setCategoryTrackerByID(int $userID, int $chapterID, string $category) : bool {
 		$success = $this->db->set(['category' => $category, 'last_updated' => NULL])
-		                    ->where('user_id', $userID)
-		                    ->where('id', $chapterID)
-		                    ->update('tracker_chapters');
+							->where('user_id', $userID)
+							->where('id', $chapterID)
+							->update('tracker_chapters');
 
 		return (bool) $success;
 	}
@@ -374,9 +374,9 @@ 
 		$success = FALSE;
 		if(preg_match("/^[a-z0-9-_,]{0,255}$/", $tag_string)) {
 			$success = $this->db->set(['tags' => $tag_string, 'last_updated' => NULL])
-			                    ->where('user_id', $userID)
-			                    ->where('id', $chapterID)
-			                    ->update('tracker_chapters');
+								->where('user_id', $userID)
+								->where('id', $chapterID)
+								->update('tracker_chapters');
 		}
 
 		return (bool) $success;
@@ -386,9 +386,9 @@ 
 		$usedCategories = [];
 
 		$query = $this->db->distinct()
-		                  ->select('category')
-		                  ->from('tracker_chapters')
-		                  ->get();
+						  ->select('category')
+						  ->from('tracker_chapters')
+						  ->get();
 
 		return array_column($query->result_array(), 'category');
 	}

application/models/User_Options_Model.php

Doc Comments +4 added lines, -1 removed lines

@@ -73,7 +73,7 @@ 
 	/**
 	 * Get user option, or default option if it does not exist.
 	 * @param string $option
-	 * @return mixed Returns option value as STRING, or FALSE if option does not exist.
+	 * @return string Returns option value as STRING, or FALSE if option does not exist.
 	 */
 	public function get(string $option) {
 		return $this->get_by_userid($option, (int) $this->User->id);
@@ -135,6 +135,9 @@ 
 		return $success;
 	}
 
+	/**
+	 * @return string
+	 */
 	private function get_db(string $option, int $userID) {
 		//This function assumes we've already done some basic validation.
 		$query = $this->db->select('value_str, value_int')

Indentation +4 added lines, -4 removed lines

@@ -138,10 +138,10 @@
 	private function get_db(string $option, int $userID) {
 		//This function assumes we've already done some basic validation.
 		$query = $this->db->select('value_str, value_int')
-		                  ->from('user_options')
-		                  ->where('user_id', $userID)
-		                  ->where('name',    $option)
-		                  ->limit(1);
+						  ->from('user_options')
+						  ->where('user_id', $userID)
+						  ->where('name',    $option)
+						  ->limit(1);
 		return $query->get()->row_array();
 	}
 	private function set_db(string $option, $value) : bool {}

application/tests/libraries/MY_Form_validation_test.php

Doc Comments +4 added lines

@@ -144,6 +144,10 @@
 	}
 
 	//utility functions
+
+	/**
+	 * @param string $field
+	 */
 	function get_error_message(/*str*/$field, /*obj*/ $customObj = FALSE) {
 		$obj = $customObj ?: $this->form_validation;
 		$error_messages = ReflectionHelper::getPrivateProperty(

public/index.php

Indentation +4 added lines, -4 removed lines

@@ -210,10 +210,10 @@
 {
 	// Ensure there's a trailing slash
 	$system_path = strtr(
-		               rtrim($system_path, '/\\'),
-		               '/\\',
-		               DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
-	               ).DIRECTORY_SEPARATOR;
+					   rtrim($system_path, '/\\'),
+					   '/\\',
+					   DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
+				   ).DIRECTORY_SEPARATOR;
 }
 
 // Is the system path correct?

application/tests/TestCase.php

Indentation +13 added lines, -13 removed lines

@@ -29,27 +29,27 @@ 
 
 	public function getMock_ion_auth_logged_in() {
 		$ion_auth = $this->getMockBuilder('Ion_auth')
-		                 ->disableOriginalConstructor()
-		                 ->getMock();
+						 ->disableOriginalConstructor()
+						 ->getMock();
 
 		$ion_auth->expects($this->at(0))
-		         ->method('logged_in')
-		         ->willReturn(TRUE);
+				 ->method('logged_in')
+				 ->willReturn(TRUE);
 		$ion_auth->expects($this->at(1))
-		         ->method('logged_in')
-		         ->willReturn(TRUE);
+				 ->method('logged_in')
+				 ->willReturn(TRUE);
 
 		$ion_auth->expects($this->any())
-		         ->method($this->anything())
-		         ->will($this->returnSelf());
+				 ->method($this->anything())
+				 ->will($this->returnSelf());
 		
 		return $ion_auth;
 	}
 
 	public function getMock_CI_DB_result(array $methods) {
 		$db_result = $this->getMockBuilder('CI_DB_result')
-		                  ->disableOriginalConstructor()
-		                  ->getMock();
+						  ->disableOriginalConstructor()
+						  ->getMock();
 
 		foreach ($methods as $method => $return) {
 			$db_result->method($method)->willReturn($return);
@@ -59,9 +59,9 @@ 
 
 	public function getMock_CI_DB($return) {
 		$db = $this->getMockBuilder('CI_DB')
-		           ->disableOriginalConstructor()
-		           ->setMethods(array('select', 'from', 'where', 'get'))
-		           ->getMock();
+				   ->disableOriginalConstructor()
+				   ->setMethods(array('select', 'from', 'where', 'get'))
+				   ->getMock();
 
 		$db->expects($this->at(3))
 		   ->method('get')

application/tests/models/User_Model_test.php

Indentation +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
 		$User_Model = $this->User_Model;
 
 		$ion_auth = $this->getMockBuilder('ion_auth')
-		                 ->disableOriginalConstructor()
-		                 ->getMock();
+						 ->disableOriginalConstructor()
+						 ->getMock();
 		$ion_auth->method('logged_in')->willReturn(FALSE);
 
 		$User_Model->ion_auth = $ion_auth;

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) {

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