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

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 Auth_Model extends CI_Model {
 	public function __construct() {
@@ -60,7 +60,7 @@ 
 			}
 
 			$success = TRUE;
-		} catch (Exception $e) {
+		} catch(Exception $e) {
 			//echo 'Caught exception: ',  $e->getMessage(), "\n";
 
 			//revert verification
@@ -91,7 +91,7 @@ 
 				//TODO: Remove from DB, send user error that verification expired.
 			} else {
 				//not expired, verification is valid, return email
-				$return =  $result->email;
+				$return = $result->email;
 			}
 		}
 		return $return;

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

Spacing +7 added lines, -7 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 Tracker_Model extends CI_Model {
 	public $sites;
@@ -49,8 +49,8 @@ 
 			];
 		}
 		if($query->num_rows() > 0) {
-			foreach ($query->result() as $row) {
-				$is_unread     = intval($row->latest_chapter == $row->current_chapter ? '1' : '0');
+			foreach($query->result() as $row) {
+				$is_unread = intval($row->latest_chapter == $row->current_chapter ? '1' : '0');
 				$arr[$row->category]['unread_count'] = (($arr[$row->category]['unread_count'] ?? 0) + !$is_unread);
 				$arr[$row->category]['manga'][] = [
 					'id' => $row->id,
@@ -79,8 +79,8 @@ 
 			}
 
 			//NOTE: This does not sort in the same way as tablesorter, but it works better.
-			foreach (array_keys($arr) as $category) {
-				usort($arr[$category]['manga'], function ($a, $b) {
+			foreach(array_keys($arr) as $category) {
+				usort($arr[$category]['manga'], function($a, $b) {
 					return strtolower("{$a['new_chapter_exists']} - {$a['title_data']['title']}") <=> strtolower("{$b['new_chapter_exists']} - {$b['title_data']['title']}");
 				});
 			}
@@ -226,7 +226,7 @@ 
 		                  ->get();
 
 		if($query->num_rows() > 0) {
-			foreach ($query->result() as $row) {
+			foreach($query->result() as $row) {
 				print "> {$row->title} <{$row->site_class}>"; //Print this prior to doing anything so we can more easily find out if something went wrong
 				$titleData = $this->sites->{$row->site_class}->getTitleData($row->title_url);
 				if(!is_null($titleData['latest_chapter'])) {
@@ -261,7 +261,7 @@ 
 
 		$arr = [];
 		if($query->num_rows() > 0) {
-			foreach ($query->result() as $row) {
+			foreach($query->result() as $row) {
 				$arr[$row->category][] = [
 					'site'            => $row->site,
 					'title_url'       => $row->title_url,

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

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/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 {}

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_Options_Model extends CI_Model {
 	public $options = array(
@@ -140,7 +140,7 @@ 
 		$query = $this->db->select('value_str, value_int')
 		                  ->from('user_options')
 		                  ->where('user_id', $userID)
-		                  ->where('name',    $option)
+		                  ->where('name', $option)
 		                  ->limit(1);
 		return $query->get()->row_array();
 	}
@@ -179,7 +179,7 @@ 
 			);
 			//FIXME: Get a better solution than str_replace for removing special characters
 			$elements = array();
-			foreach (array_values($this->options[$option]['valid_options']) as $valid_option) {
+			foreach(array_values($this->options[$option]['valid_options']) as $valid_option) {
 				$elements[$option.'_'.str_replace(',', '_', $valid_option)] = array_merge($base_attributes, array(
 					'value' => $valid_option
 				));

Braces +7 added lines, -2 removed lines

@@ -94,7 +94,9 @@ 
 			}
 
 			//Overall fallback method.
-			if(!isset($value)) $value = $this->options[$option]['default'];
+			if(!isset($value)) {
+				$value = $this->options[$option]['default'];
+			}
 		} else {
 			$value = FALSE;
 		}
@@ -165,7 +167,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/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?

Spacing +12 added lines, -12 removed lines

@@ -63,7 +63,7 @@ 
  * Different environments will require different levels of error reporting.
  * By default development will show errors but testing and live will hide them.
  */
-switch (ENVIRONMENT)
+switch(ENVIRONMENT)
 {
 	case 'development':
 		error_reporting(-1);
@@ -79,7 +79,7 @@ 
 	case 'testing':
 	case 'production':
 		ini_set('display_errors', 0);
-		if (version_compare(PHP_VERSION, '5.3', '>='))
+		if(version_compare(PHP_VERSION, '5.3', '>='))
 		{
 			error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
 		}
@@ -197,12 +197,12 @@ 
  */
 
 // Set the current directory correctly for CLI requests
-if (defined('STDIN'))
+if(defined('STDIN'))
 {
 	chdir(dirname(__FILE__));
 }
 
-if (($_temp = realpath($system_path)) !== FALSE)
+if(($_temp = realpath($system_path)) !== FALSE)
 {
 	$system_path = $_temp.DIRECTORY_SEPARATOR;
 }
@@ -217,7 +217,7 @@ 
 }
 
 // Is the system path correct?
-if ( ! is_dir($system_path))
+if(!is_dir($system_path))
 {
 	header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 	echo 'Your system folder path does not appear to be set correctly. Please open the following file and correct this: '.pathinfo(__FILE__, PATHINFO_BASENAME);
@@ -242,9 +242,9 @@ 
 define('SYSDIR', basename(BASEPATH));
 
 // The path to the "application" directory
-if (is_dir($application_folder))
+if(is_dir($application_folder))
 {
-	if (($_temp = realpath($application_folder)) !== FALSE)
+	if(($_temp = realpath($application_folder)) !== FALSE)
 	{
 		$application_folder = $_temp;
 	}
@@ -257,7 +257,7 @@ 
 		);
 	}
 }
-elseif (is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR))
+elseif(is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR))
 {
 	$application_folder = BASEPATH.strtr(
 			trim($application_folder, '/\\'),
@@ -275,13 +275,13 @@ 
 define('APPPATH', $application_folder.DIRECTORY_SEPARATOR);
 
 // The path to the "views" directory
-if ( ! isset($view_folder[0]) && is_dir(APPPATH.'views'.DIRECTORY_SEPARATOR))
+if(!isset($view_folder[0]) && is_dir(APPPATH.'views'.DIRECTORY_SEPARATOR))
 {
 	$view_folder = APPPATH.'views';
 }
-elseif (is_dir($view_folder))
+elseif(is_dir($view_folder))
 {
-	if (($_temp = realpath($view_folder)) !== FALSE)
+	if(($_temp = realpath($view_folder)) !== FALSE)
 	{
 		$view_folder = $_temp;
 	}
@@ -294,7 +294,7 @@ 
 		);
 	}
 }
-elseif (is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR))
+elseif(is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR))
 {
 	$view_folder = APPPATH.strtr(
 			trim($view_folder, '/\\'),

Braces +18 added lines, -45 removed lines

@@ -63,8 +63,7 @@ 
  * Different environments will require different levels of error reporting.
  * By default development will show errors but testing and live will hide them.
  */
-switch (ENVIRONMENT)
-{
+switch (ENVIRONMENT) {
 	case 'development':
 		error_reporting(-1);
 		ini_set('display_errors', 1);
@@ -79,12 +78,9 @@ 
 	case 'testing':
 	case 'production':
 		ini_set('display_errors', 0);
-		if (version_compare(PHP_VERSION, '5.3', '>='))
-		{
+		if (version_compare(PHP_VERSION, '5.3', '>=')) {
 			error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
-		}
-		else
-		{
+		} else {
 			error_reporting(E_ALL & ~E_NOTICE & ~E_STRICT & ~E_USER_NOTICE);
 		}
 		break;
@@ -197,17 +193,13 @@ 
  */
 
 // Set the current directory correctly for CLI requests
-if (defined('STDIN'))
-{
+if (defined('STDIN')) {
 	chdir(dirname(__FILE__));
 }
 
-if (($_temp = realpath($system_path)) !== FALSE)
-{
+if (($_temp = realpath($system_path)) !== FALSE) {
 	$system_path = $_temp.DIRECTORY_SEPARATOR;
-}
-else
-{
+} else {
 	// Ensure there's a trailing slash
 	$system_path = strtr(
 		               rtrim($system_path, '/\\'),
@@ -217,8 +209,7 @@ 
 }
 
 // Is the system path correct?
-if ( ! is_dir($system_path))
-{
+if ( ! is_dir($system_path)) {
 	header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 	echo 'Your system folder path does not appear to be set correctly. Please open the following file and correct this: '.pathinfo(__FILE__, PATHINFO_BASENAME);
 	exit(3); // EXIT_CONFIG
@@ -242,31 +233,23 @@ 
 define('SYSDIR', basename(BASEPATH));
 
 // The path to the "application" directory
-if (is_dir($application_folder))
-{
-	if (($_temp = realpath($application_folder)) !== FALSE)
-	{
+if (is_dir($application_folder)) {
+	if (($_temp = realpath($application_folder)) !== FALSE) {
 		$application_folder = $_temp;
-	}
-	else
-	{
+	} else {
 		$application_folder = strtr(
 			rtrim($application_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
 	}
-}
-elseif (is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR))
-{
+} elseif (is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR)) {
 	$application_folder = BASEPATH.strtr(
 			trim($application_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
-}
-else
-{
+} else {
 	header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 	echo 'Your application folder path does not appear to be set correctly. Please open the following file and correct this: '.SELF;
 	exit(3); // EXIT_CONFIG
@@ -275,35 +258,25 @@ 
 define('APPPATH', $application_folder.DIRECTORY_SEPARATOR);
 
 // The path to the "views" directory
-if ( ! isset($view_folder[0]) && is_dir(APPPATH.'views'.DIRECTORY_SEPARATOR))
-{
+if ( ! isset($view_folder[0]) && is_dir(APPPATH.'views'.DIRECTORY_SEPARATOR)) {
 	$view_folder = APPPATH.'views';
-}
-elseif (is_dir($view_folder))
-{
-	if (($_temp = realpath($view_folder)) !== FALSE)
-	{
+} elseif (is_dir($view_folder)) {
+	if (($_temp = realpath($view_folder)) !== FALSE) {
 		$view_folder = $_temp;
-	}
-	else
-	{
+	} else {
 		$view_folder = strtr(
 			rtrim($view_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
 	}
-}
-elseif (is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR))
-{
+} elseif (is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR)) {
 	$view_folder = APPPATH.strtr(
 			trim($view_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
-}
-else
-{
+} else {
 	header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 	echo 'Your view folder path does not appear to be set correctly. Please open the following file and correct this: '.SELF;
 	exit(3); // EXIT_CONFIG

application/tests/controllers/User/Auth/Login_test.php

Spacing +10 added lines, -10 removed lines

@@ -9,7 +9,7 @@ 
 	public function test_index_logged_in() {
 		//user is already logged in, redirect to dashboard
 		$this->request->setCallablePreConstructor(
-			function () {
+			function() {
 				$ion_auth = $this->getMock_ion_auth_logged_in();
 				load_class_instance('ion_auth', $ion_auth);
 			}
@@ -30,7 +30,7 @@ 
 		$this->markTestIncomplete('This test is broken for now.');
 
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -44,7 +44,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$user = $this->getDouble(
 					'User_Model',
 					['find_email_from_identity' => 'foobar']
@@ -56,7 +56,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['login' => TRUE]
@@ -77,7 +77,7 @@ 
 		$this->markTestIncomplete('This test is broken for now.');
 
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -91,7 +91,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$user = $this->getDouble(
 					'User_Model',
 					['find_email_from_identity' => 'foobar']
@@ -103,7 +103,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['login' => TRUE]
@@ -115,7 +115,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				// Get mock object
 				$session = $this->getDouble(
 					'CI_Session',
@@ -134,7 +134,7 @@ 
 	public function test_login_validation_fail() {
 		//user isn't logged in, validation failed, this is happens on the first time loading the page
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				// Get mock object
 				$validation = $this->getDouble(
 					'CI_Form_validation',
@@ -156,7 +156,7 @@ 
 	public function test_login_validation_pass_but_login_fail() {
 		//user isn't logged in, tried to login, validation succeeded but login failed
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				// Get mock object
 				$validation = $this->getDouble(
 					'CI_Form_validation',

application/tests/controllers/User/Auth/Signup_test.php

Spacing +16 added lines, -16 removed lines

@@ -6,7 +6,7 @@ 
 		$this->assertResponseCode(200);
 
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble('Auth_Model', ['verification_check' => 'foobar']);
 				$this->verifyInvokedOnce($auth, 'verification_check');
 				$CI->Auth = $auth;
@@ -19,7 +19,7 @@ 
 	public function test_index_logged_in() {
 		//user is already logged in, redirect to dashboard
 		$this->request->setCallablePreConstructor(
-			function () {
+			function() {
 				$ion_auth = $this->getMock_ion_auth_logged_in();
 				load_class_instance('ion_auth', $ion_auth);
 			}
@@ -37,7 +37,7 @@ 
 
 	public function test_signup_p1_validation_pass_verification_pass() {
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -51,7 +51,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-				function ($CI) {
+				function($CI) {
 					$auth = $this->getDouble('Auth_Model', ['verification_start' => TRUE]);
 					$this->verifyInvokedOnce($auth, 'verification_start');
 					$CI->Auth = $auth;
@@ -65,7 +65,7 @@ 
 	}
 	public function test_signup_p1_validation_pass_verification_fail() {
 		$this->request->setCallable(
-				function ($CI) {
+				function($CI) {
 					$validation = $this->getDouble(
 							'CI_Form_validation',
 							['set_rules' => NULL, 'run' => TRUE]
@@ -79,7 +79,7 @@ 
 				}
 		);
 		$this->request->addCallable(
-				function ($CI) {
+				function($CI) {
 					$auth = $this->getDouble('Auth_Model', ['verification_start' => FALSE]);
 					$this->verifyInvokedOnce($auth, 'verification_start');
 					$CI->Auth = $auth;
@@ -93,7 +93,7 @@ 
 	}
 	public function test_signup_p1_validation_fail() {
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => FALSE, 'set_value' => '']
@@ -115,14 +115,14 @@ 
 		//form was valid, and register was successful, user is redirected to dashboard
 
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble('Auth_Model', ['verification_check' => 'foo@bar.com']);
 				$this->verifyInvokedOnce($auth, 'verification_check');
 				$CI->Auth = $auth;
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -136,7 +136,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['register' => TRUE]
@@ -154,14 +154,14 @@ 
 	public function test_signup_p2_verification_pass_validation_pass_register_fail() {
 		//form was valid, and register was unsuccessful, reshow form
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble('Auth_Model', ['verification_check' => 'foo@bar.com']);
 				$this->verifyInvokedOnce($auth, 'verification_check');
 				$CI->Auth = $auth;
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE, 'set_value' => 'foobar']
@@ -176,7 +176,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['register' => FALSE, 'errors' => FALSE, 'logged_in' => FALSE]
@@ -194,14 +194,14 @@ 
 	public function test_signup_p2_verification_pass_validation_fail() {
 		//validation failed, this is probably the first time the user visits the page, show form
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble('Auth_Model', ['verification_check' => 'foo@bar.com']);
 				$this->verifyInvokedOnce($auth, 'verification_check');
 				$CI->Auth = $auth;
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => FALSE, 'set_value' => '']
@@ -222,7 +222,7 @@ 
 	public function test_signup_p2_verification_fail() {
 		//validation failed, this is probably the first time the user visits the page, show form
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble('Auth_Model', ['verification_check' => FALSE]);
 				$this->verifyInvokedOnce($auth, 'verification_check');
 				$CI->Auth = $auth;

application/tests/controllers/User/Auth/Forgot_Password_test.php

Spacing +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
 
 
 		$this->request->setCallablePreConstructor(
-			function () {
+			function() {
 				$ion_auth = $this->getMock_ion_auth_logged_in();
 				load_class_instance('ion_auth', $ion_auth);
 			}
@@ -38,7 +38,7 @@ 
 	public function test_forgot_password_validation_pass() {
 		//user isn't logged in, form is valid, user is shown success page regardless of if email is used
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -52,7 +52,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['forgotten_password' => TRUE, 'row' => (object) ['email' => 'foo@bar.com'], 'logged_in' => FALSE]
@@ -72,7 +72,7 @@ 
 	public function test_forgot_password_validation_fail() {
 		//user isn't logged in, form is invalid, this is usually the case when first viewing the page
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => FALSE]
@@ -93,7 +93,7 @@ 
 	public function test_reset_password_user_pass_validation_pass_change_fail() {
 		//reset code is valid, form is invalid (or new)
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['forgotten_password_check' => TRUE, 'reset_password' => FALSE]
@@ -105,7 +105,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -126,7 +126,7 @@ 
 	public function test_reset_password_user_pass_validation_pass_change_pass() {
 		//reset code is valid, form is valid, password change was successful, redirect to login
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['forgotten_password_check' => TRUE, 'reset_password' => TRUE]
@@ -138,7 +138,7 @@ 
 			}
 		);
 		$this->request->addCallable(
-			function ($CI) {
+			function($CI) {
 				$validation = $this->getDouble(
 					'CI_Form_validation',
 					['set_rules' => NULL, 'run' => TRUE]
@@ -159,7 +159,7 @@ 
 	public function test_reset_password_user_pass_validation_fail() {
 		//reset code is valid, form is valid, password change was unsuccessful, send back to reset_password page
 		$this->request->setCallable(
-			function ($CI) {
+			function($CI) {
 				$auth = $this->getDouble(
 					'Ion_auth_model',
 					['forgotten_password_check' => TRUE, 'logged_in' => FALSE]

application/tests/controllers/IndexC_test.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
 	public function test_APPPATH() {
 		$actual   = realpath(APPPATH);
-		$expected = realpath(__DIR__ . '/../..');
+		$expected = realpath(__DIR__.'/../..');
 		$this->assertEquals(
 			$expected,
 			$actual,