timw4mail / HummingBirdAnimeClient

src/Model/Manga.php

Braces +5 added lines, -4 removed lines

@@ -23,8 +23,7 @@ 
 /**
  * Model for handling requests dealing with the manga list
  */
-class Manga extends API
-{
+class Manga extends API {
 
 	const READING = 'Reading';
 	const PLAN_TO_READ = 'Plan to Read';
@@ -160,12 +159,14 @@ 
 			self::COMPLETED => [],
 		];
 
-		foreach ($data as &$entry) {
+		foreach ($data as &$entry)
+		{
 			$key = $this->status_map[$entry['reading_status']];
 			$output[$key][] = $entry;
 		}
 
-		foreach ($output as &$val) {
+		foreach ($output as &$val)
+		{
 			$this->sortByName($val, 'manga');
 		}
 

src/Model/Anime.php

Braces +2 added lines, -1 removed lines

@@ -53,7 +53,8 @@
 	 * Anime constructor.
 	 * @param ContainerInterface $container
 	 */
-	public function __construct(ContainerInterface $container) {
+	public function __construct(ContainerInterface $container)
+	{
 		parent::__construct($container);
 
 		$config = $container->get('config');

src/API/GuzzleTrait.php

Indentation +16 added lines, -16 removed lines

@@ -20,22 +20,22 @@
  * Base trait for api interaction

  */

 trait GuzzleTrait {

-    /**

-     * The Guzzle http client object

-     * @var object

-     */

-    protected $client;

+	/**

+	 * The Guzzle http client object

+	 * @var object

+	 */

+	protected $client;

 

-    /**

-     * Cookie jar object for api requests

-     * @var object

-     */

-    protected $cookieJar;

+	/**

+	 * Cookie jar object for api requests

+	 * @var object

+	 */

+	protected $cookieJar;

 

-    /**

-     * Set up the class properties

-     *

-     * @return void

-     */

-    abstract protected function init();

+	/**

+	 * Set up the class properties

+	 *

+	 * @return void

+	 */

+	abstract protected function init();

 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 /**

  * Base trait for api interaction

  */

-trait GuzzleTrait {

+trait GuzzleTrait {
     /**

      * The Guzzle http client object

      * @var object

src/Dispatcher.php

Braces +2 added lines, -1 removed lines

@@ -289,7 +289,8 @@
 
 		$params = [];
 
-		switch($failure->failedRule) {
+		switch($failure->failedRule)
+		{
 			case 'Aura\Router\Rule\Allows':
 				$params = [
 					'http_code' => 405,

build/CodeIgniter/UnusedSniffs/Files/ClosingLocationCommentSniff.php

Indentation +132 added lines, -132 removed lines

@@ -45,138 +45,138 @@
 
 class ClosingLocationCommentSniff extends AbstractClosingCommentSniff
 {
-    public $applicationRoot = '/application/';
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_OPEN_TAG
-        );
-
-    }//end register()
-
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        // We are only interested if this is the first open tag.
-        if ($stackPtr !== 0) {
-            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
-                return;
-            }
-        }
-
-        $filePath = $phpcsFile->getFilename();
-        $tokens = $phpcsFile->getTokens();
-        // removes the application root from the beginning of the file path
-        $locationPath = self::_getLocationPath($filePath, $this->_getAppRoot());
-        // add an error, if application root doesn't exist in current file path
-        if (false === $locationPath) {
-            $error = 'Unable to find "' . $this->_getAppRoot() . '" in file path "' . $filePath . '". Please set your project\'s application root.';
-            $phpcsFile->addError($error, count($tokens) - 1);
-            return;
-        }
-        // generates the expected comment
-        $commentTemplate = "Location: $locationPath";
-
-        $currentToken = count($tokens) - 1;
-        $hasClosingLocationComment = false;
-        $isNotAWhitespaceOrAComment = false;
-        while ($currentToken >= 0
-            && ! $isNotAWhitespaceOrAComment
-            && ! $hasClosingLocationComment
-        ) {
-            $token = $tokens[$currentToken];
-            $tokenCode = $token['code'];
-            if (T_COMMENT === $tokenCode) {
-                $commentString = self::_getCommentContent($token['content']);
-                if (0 === strcmp($commentString, $commentTemplate)) {
-                    $hasClosingLocationComment = true;
-                }
-            } else if (T_WHITESPACE === $tokenCode) {
-                // Whitespaces are allowed between the closing file comment,
-                //other comments and end of file
-            } else {
-                $isNotAWhitespaceOrAComment = true;
-            }
-            $currentToken--;
-        }
-
-        if ( ! $hasClosingLocationComment) {
-            $error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
-            $phpcsFile->addError($error, $currentToken);
-        }
-    }//end process()
-
-
-    /**
-     * Returns the relative path from $appRoot to $filePath, or false if
-     * $appRoot cannot be found in $filePath, because $appRoot is not a parent
-     * of $filePath.
-     *
-     * @param string $filePath Full path to the file being proceed.
-     * @param string $appRoot  Partial or full path to the CodeIgniter
-     * application root of the file being proceed. It must not contain the
-     * full path to the application root, but at least the name of the
-     * application root. Parent directory of the application root are allowed
-     * but not mandatory.
-     *
-     * @return string|bool The relative path from $appRoot to $filePath, or
-     * false if $appRoot cannot be found in $filePath.
-     */
-    private static function _getLocationPath ($filePath, $appRoot)
-    {
-        // removes the path to application root
-        // from the beginning of the file path
-        $appRootAt = strpos($filePath, $appRoot);
-        if (false === $appRootAt) {
-            return false;
-        }
-        $localPath = substr($filePath, $appRootAt + strlen($appRoot));
-        // ensures the location path to be a relative path starting with "./".
-        if ( ! self::_stringStartsWith($localPath, './')) {
-            $localPath = './' . $localPath;
-        } else if ( ! self::_stringStartsWith($localPath, '.')
-            && self::_stringStartsWith($localPath, '/')
-        ) {
-            $localPath = '.' . $localPath;
-        }
-        return $localPath;
-    }//end _getLocationPath()
-
-
-    /**
-     * Returns the application root that should be used first.
-     *
-     * There are several ways to configure the application root.
-     * In order of priority :
-     *   - Configuration variable ci_application_root.
-     *   - Rule property applicationRoot.
-     *   - Default value '/application/'
-     *
-     * @return string Path to your project application root.
-     */
-    private function _getAppRoot()
-    {
-        $appRoot = Common::getConfigData('ci_application_root');
-        if (null === $appRoot) {
-            $appRoot = $this->applicationRoot;
-        }
-        return $appRoot;
-    }//end _getAppRoot()
+	public $applicationRoot = '/application/';
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_OPEN_TAG
+		);
+
+	}//end register()
+
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		// We are only interested if this is the first open tag.
+		if ($stackPtr !== 0) {
+			if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
+				return;
+			}
+		}
+
+		$filePath = $phpcsFile->getFilename();
+		$tokens = $phpcsFile->getTokens();
+		// removes the application root from the beginning of the file path
+		$locationPath = self::_getLocationPath($filePath, $this->_getAppRoot());
+		// add an error, if application root doesn't exist in current file path
+		if (false === $locationPath) {
+			$error = 'Unable to find "' . $this->_getAppRoot() . '" in file path "' . $filePath . '". Please set your project\'s application root.';
+			$phpcsFile->addError($error, count($tokens) - 1);
+			return;
+		}
+		// generates the expected comment
+		$commentTemplate = "Location: $locationPath";
+
+		$currentToken = count($tokens) - 1;
+		$hasClosingLocationComment = false;
+		$isNotAWhitespaceOrAComment = false;
+		while ($currentToken >= 0
+			&& ! $isNotAWhitespaceOrAComment
+			&& ! $hasClosingLocationComment
+		) {
+			$token = $tokens[$currentToken];
+			$tokenCode = $token['code'];
+			if (T_COMMENT === $tokenCode) {
+				$commentString = self::_getCommentContent($token['content']);
+				if (0 === strcmp($commentString, $commentTemplate)) {
+					$hasClosingLocationComment = true;
+				}
+			} else if (T_WHITESPACE === $tokenCode) {
+				// Whitespaces are allowed between the closing file comment,
+				//other comments and end of file
+			} else {
+				$isNotAWhitespaceOrAComment = true;
+			}
+			$currentToken--;
+		}
+
+		if ( ! $hasClosingLocationComment) {
+			$error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
+			$phpcsFile->addError($error, $currentToken);
+		}
+	}//end process()
+
+
+	/**
+	 * Returns the relative path from $appRoot to $filePath, or false if
+	 * $appRoot cannot be found in $filePath, because $appRoot is not a parent
+	 * of $filePath.
+	 *
+	 * @param string $filePath Full path to the file being proceed.
+	 * @param string $appRoot  Partial or full path to the CodeIgniter
+	 * application root of the file being proceed. It must not contain the
+	 * full path to the application root, but at least the name of the
+	 * application root. Parent directory of the application root are allowed
+	 * but not mandatory.
+	 *
+	 * @return string|bool The relative path from $appRoot to $filePath, or
+	 * false if $appRoot cannot be found in $filePath.
+	 */
+	private static function _getLocationPath ($filePath, $appRoot)
+	{
+		// removes the path to application root
+		// from the beginning of the file path
+		$appRootAt = strpos($filePath, $appRoot);
+		if (false === $appRootAt) {
+			return false;
+		}
+		$localPath = substr($filePath, $appRootAt + strlen($appRoot));
+		// ensures the location path to be a relative path starting with "./".
+		if ( ! self::_stringStartsWith($localPath, './')) {
+			$localPath = './' . $localPath;
+		} else if ( ! self::_stringStartsWith($localPath, '.')
+			&& self::_stringStartsWith($localPath, '/')
+		) {
+			$localPath = '.' . $localPath;
+		}
+		return $localPath;
+	}//end _getLocationPath()
+
+
+	/**
+	 * Returns the application root that should be used first.
+	 *
+	 * There are several ways to configure the application root.
+	 * In order of priority :
+	 *   - Configuration variable ci_application_root.
+	 *   - Rule property applicationRoot.
+	 *   - Default value '/application/'
+	 *
+	 * @return string Path to your project application root.
+	 */
+	private function _getAppRoot()
+	{
+		$appRoot = Common::getConfigData('ci_application_root');
+		if (null === $appRoot) {
+			$appRoot = $this->applicationRoot;
+		}
+		return $appRoot;
+	}//end _getAppRoot()
 }//end class
 
 ?>

Braces +27 added lines, -14 removed lines

@@ -43,8 +43,7 @@ 
 use PHP_CodeSniffer\Files\File;
 use PHP_CodeSniffer\Util\Common;
 
-class ClosingLocationCommentSniff extends AbstractClosingCommentSniff
-{
+class ClosingLocationCommentSniff extends AbstractClosingCommentSniff {
     public $applicationRoot = '/application/';
 
     /**
@@ -73,8 +72,10 @@ 
     public function process(File $phpcsFile, $stackPtr)
     {
         // We are only interested if this is the first open tag.
-        if ($stackPtr !== 0) {
-            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
+        if ($stackPtr !== 0)
+        {
+            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false)
+            {
                 return;
             }
         }
@@ -84,7 +85,8 @@ 
         // removes the application root from the beginning of the file path
         $locationPath = self::_getLocationPath($filePath, $this->_getAppRoot());
         // add an error, if application root doesn't exist in current file path
-        if (false === $locationPath) {
+        if (false === $locationPath)
+        {
             $error = 'Unable to find "' . $this->_getAppRoot() . '" in file path "' . $filePath . '". Please set your project\'s application root.';
             $phpcsFile->addError($error, count($tokens) - 1);
             return;
@@ -101,21 +103,28 @@ 
         ) {
             $token = $tokens[$currentToken];
             $tokenCode = $token['code'];
-            if (T_COMMENT === $tokenCode) {
+            if (T_COMMENT === $tokenCode)
+            {
                 $commentString = self::_getCommentContent($token['content']);
-                if (0 === strcmp($commentString, $commentTemplate)) {
+                if (0 === strcmp($commentString, $commentTemplate))
+                {
                     $hasClosingLocationComment = true;
                 }
-            } else if (T_WHITESPACE === $tokenCode) {
+            }
+            else if (T_WHITESPACE === $tokenCode)
+            {
                 // Whitespaces are allowed between the closing file comment,
                 //other comments and end of file
-            } else {
+            }
+            else
+            {
                 $isNotAWhitespaceOrAComment = true;
             }
             $currentToken--;
         }
 
-        if ( ! $hasClosingLocationComment) {
+        if ( ! $hasClosingLocationComment)
+        {
             $error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
             $phpcsFile->addError($error, $currentToken);
         }
@@ -142,14 +151,17 @@ 
         // removes the path to application root
         // from the beginning of the file path
         $appRootAt = strpos($filePath, $appRoot);
-        if (false === $appRootAt) {
+        if (false === $appRootAt)
+        {
             return false;
         }
         $localPath = substr($filePath, $appRootAt + strlen($appRoot));
         // ensures the location path to be a relative path starting with "./".
-        if ( ! self::_stringStartsWith($localPath, './')) {
+        if ( ! self::_stringStartsWith($localPath, './'))
+        {
             $localPath = './' . $localPath;
-        } else if ( ! self::_stringStartsWith($localPath, '.')
+        }
+        else if ( ! self::_stringStartsWith($localPath, '.')
             && self::_stringStartsWith($localPath, '/')
         ) {
             $localPath = '.' . $localPath;
@@ -172,7 +184,8 @@ 
     private function _getAppRoot()
     {
         $appRoot = Common::getConfigData('ci_application_root');
-        if (null === $appRoot) {
+        if (null === $appRoot)
+        {
             $appRoot = $this->applicationRoot;
         }
         return $appRoot;

build/CodeIgniter/UnusedSniffs/Files/AbstractClosingCommentSniff.php

Indentation +61 added lines, -61 removed lines

@@ -34,71 +34,71 @@
 
 class AbstractClosingCommentSniff implements Sniff
 {
-    /**
-     * As an abstract class, this sniff is not associated to any token.
-     */
-    public function register()
-    {
-      return array();
-    }
+	/**
+	 * As an abstract class, this sniff is not associated to any token.
+	 */
+	public function register()
+	{
+	  return array();
+	}
 
-    /**
-     * As an abstract class, this sniff is not dedicated to process a token.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-      $error = __CLASS__.'::'.__METHOD__.' is abstract. Please develop this method in a child class.';
-      throw new PHP_CodeSniffer_Exception($error);
-    }
+	/**
+	 * As an abstract class, this sniff is not dedicated to process a token.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+	  $error = __CLASS__.'::'.__METHOD__.' is abstract. Please develop this method in a child class.';
+	  throw new PHP_CodeSniffer_Exception($error);
+	}
 
-    /**
-     * Returns the comment without its delimiter(s) as well as leading
-     * and traling whitespaces.
-     *
-     * It removes the first #, the two first / (i.e. //) or the first /*
-     * and last \*\/. If a comment starts with /**, then the last * will remain
-     * as well as whitespaces between this star and the comment content.
-     *
-     * @param string $comment Comment containing either comment delimiter(s) and
-     * trailing or leading whitspaces to clean.
-     *
-     * @return string Comment without comment delimiter(s) and whitespaces.
-     */
-    protected static function _getCommentContent ($comment)
-    {
-        if (self::_stringStartsWith($comment, '#')) {
-            $comment = substr($comment, 1);
-        } else if (self::_stringStartsWith($comment, '//')) {
-            $comment = substr($comment, 2);
-        } else if (self::_stringStartsWith($comment, '/*')) {
-            $comment = substr($comment, 2, strlen($comment) - 2 - 2);
-        }
-        $comment = trim($comment);
-        return $comment;
-    }//_getCommentContent()
+	/**
+	 * Returns the comment without its delimiter(s) as well as leading
+	 * and traling whitespaces.
+	 *
+	 * It removes the first #, the two first / (i.e. //) or the first /*
+	 * and last \*\/. If a comment starts with /**, then the last * will remain
+	 * as well as whitespaces between this star and the comment content.
+	 *
+	 * @param string $comment Comment containing either comment delimiter(s) and
+	 * trailing or leading whitspaces to clean.
+	 *
+	 * @return string Comment without comment delimiter(s) and whitespaces.
+	 */
+	protected static function _getCommentContent ($comment)
+	{
+		if (self::_stringStartsWith($comment, '#')) {
+			$comment = substr($comment, 1);
+		} else if (self::_stringStartsWith($comment, '//')) {
+			$comment = substr($comment, 2);
+		} else if (self::_stringStartsWith($comment, '/*')) {
+			$comment = substr($comment, 2, strlen($comment) - 2 - 2);
+		}
+		$comment = trim($comment);
+		return $comment;
+	}//_getCommentContent()
 
 
-    /**
-     * Binary safe string comparison between $needle and
-     * the beginning of $haystack. Returns true if $haystack starts with
-     * $needle, false otherwise.
-     *
-     * @param string $haystack The string to search in.
-     * @param string $needle   The string to search for.
-     *
-     * @return bool true if $haystack starts with $needle, false otherwise.
-     */
-    protected static function _stringStartsWith ($haystack, $needle)
-    {
-        $startsWith = false;
-        if (strlen($needle) <= strlen($haystack)) {
-            $haystackBeginning = substr($haystack, 0, strlen($needle));
-            if (0 === strcmp($haystackBeginning, $needle)) {
-                $startsWith = true;
-            }
-        }
-        return $startsWith;
-    }//_stringStartsWith()
+	/**
+	 * Binary safe string comparison between $needle and
+	 * the beginning of $haystack. Returns true if $haystack starts with
+	 * $needle, false otherwise.
+	 *
+	 * @param string $haystack The string to search in.
+	 * @param string $needle   The string to search for.
+	 *
+	 * @return bool true if $haystack starts with $needle, false otherwise.
+	 */
+	protected static function _stringStartsWith ($haystack, $needle)
+	{
+		$startsWith = false;
+		if (strlen($needle) <= strlen($haystack)) {
+			$haystackBeginning = substr($haystack, 0, strlen($needle));
+			if (0 === strcmp($haystackBeginning, $needle)) {
+				$startsWith = true;
+			}
+		}
+		return $startsWith;
+	}//_stringStartsWith()
 }//end class
 
 ?>

Braces +13 added lines, -7 removed lines

@@ -32,8 +32,7 @@ 
 use PHP_CodeSniffer\Sniffs\Sniff;
 use PHP_CodeSniffer\Files\File;
 
-class AbstractClosingCommentSniff implements Sniff
-{
+class AbstractClosingCommentSniff implements Sniff {
     /**
      * As an abstract class, this sniff is not associated to any token.
      */
@@ -66,11 +65,16 @@ 
      */
     protected static function _getCommentContent ($comment)
     {
-        if (self::_stringStartsWith($comment, '#')) {
+        if (self::_stringStartsWith($comment, '#'))
+        {
             $comment = substr($comment, 1);
-        } else if (self::_stringStartsWith($comment, '//')) {
+        }
+        else if (self::_stringStartsWith($comment, '//'))
+        {
             $comment = substr($comment, 2);
-        } else if (self::_stringStartsWith($comment, '/*')) {
+        }
+        else if (self::_stringStartsWith($comment, '/*'))
+        {
             $comment = substr($comment, 2, strlen($comment) - 2 - 2);
         }
         $comment = trim($comment);
@@ -91,9 +95,11 @@ 
     protected static function _stringStartsWith ($haystack, $needle)
     {
         $startsWith = false;
-        if (strlen($needle) <= strlen($haystack)) {
+        if (strlen($needle) <= strlen($haystack))
+        {
             $haystackBeginning = substr($haystack, 0, strlen($needle));
-            if (0 === strcmp($haystackBeginning, $needle)) {
+            if (0 === strcmp($haystackBeginning, $needle))
+            {
                 $startsWith = true;
             }
         }

build/CodeIgniter/UnusedSniffs/Files/ClosingFileCommentSniff.php

Indentation +59 added lines, -59 removed lines

@@ -39,71 +39,71 @@
 class ClosingFileCommentSniff extends AbstractClosingCommentSniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_OPEN_TAG,
-        );
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_OPEN_TAG,
+		);
 
-    }//end register()
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        // We are only interested if this is the first open tag.
-        if ($stackPtr !== 0) {
-            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
-                return;
-            }
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		// We are only interested if this is the first open tag.
+		if ($stackPtr !== 0) {
+			if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
+				return;
+			}
+		}
 
-        $fullFilename = $phpcsFile->getFilename();
-        $filename = basename($fullFilename);
-        $commentTemplate = "End of file $filename";
+		$fullFilename = $phpcsFile->getFilename();
+		$filename = basename($fullFilename);
+		$commentTemplate = "End of file $filename";
 
-        $tokens = $phpcsFile->getTokens();
-        $currentToken = count($tokens) - 1;
-        $hasClosingFileComment = false;
-        $isNotAWhitespaceOrAComment = false;
-        while ($currentToken >= 0
-            && ! $isNotAWhitespaceOrAComment
-            && ! $hasClosingFileComment
-        ) {
-            $token = $tokens[$currentToken];
-            $tokenCode = $token['code'];
-            if (T_COMMENT === $tokenCode) {
-                $commentString = self::_getCommentContent($token['content']);
-                if (0 === strcmp($commentString, $commentTemplate)) {
-                    $hasClosingFileComment = true;
-                }
-            } else if (T_WHITESPACE === $tokenCode) {
-                // Whitespaces are allowed between the closing file comment,
-                // other comments and end of file
-            } else {
-                $isNotAWhitespaceOrAComment = true;
-            }
-            $currentToken--;
-        }
+		$tokens = $phpcsFile->getTokens();
+		$currentToken = count($tokens) - 1;
+		$hasClosingFileComment = false;
+		$isNotAWhitespaceOrAComment = false;
+		while ($currentToken >= 0
+			&& ! $isNotAWhitespaceOrAComment
+			&& ! $hasClosingFileComment
+		) {
+			$token = $tokens[$currentToken];
+			$tokenCode = $token['code'];
+			if (T_COMMENT === $tokenCode) {
+				$commentString = self::_getCommentContent($token['content']);
+				if (0 === strcmp($commentString, $commentTemplate)) {
+					$hasClosingFileComment = true;
+				}
+			} else if (T_WHITESPACE === $tokenCode) {
+				// Whitespaces are allowed between the closing file comment,
+				// other comments and end of file
+			} else {
+				$isNotAWhitespaceOrAComment = true;
+			}
+			$currentToken--;
+		}
 
-        if ( ! $hasClosingFileComment) {
-            $error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
-            $phpcsFile->addError($error, $currentToken);
-        }
-    }//end process()
+		if ( ! $hasClosingFileComment) {
+			$error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
+			$phpcsFile->addError($error, $currentToken);
+		}
+	}//end process()
 }//end class
 
 ?>

Braces +17 added lines, -9 removed lines

@@ -36,8 +36,7 @@ 
 
 use PHP_CodeSniffer\Files\File;
 
-class ClosingFileCommentSniff extends AbstractClosingCommentSniff
-{
+class ClosingFileCommentSniff extends AbstractClosingCommentSniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
@@ -65,8 +64,10 @@ 
     public function process(File $phpcsFile, $stackPtr)
     {
         // We are only interested if this is the first open tag.
-        if ($stackPtr !== 0) {
-            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
+        if ($stackPtr !== 0)
+        {
+            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false)
+            {
                 return;
             }
         }
@@ -85,21 +86,28 @@ 
         ) {
             $token = $tokens[$currentToken];
             $tokenCode = $token['code'];
-            if (T_COMMENT === $tokenCode) {
+            if (T_COMMENT === $tokenCode)
+            {
                 $commentString = self::_getCommentContent($token['content']);
-                if (0 === strcmp($commentString, $commentTemplate)) {
+                if (0 === strcmp($commentString, $commentTemplate))
+                {
                     $hasClosingFileComment = true;
                 }
-            } else if (T_WHITESPACE === $tokenCode) {
+            }
+            else if (T_WHITESPACE === $tokenCode)
+            {
                 // Whitespaces are allowed between the closing file comment,
                 // other comments and end of file
-            } else {
+            }
+            else
+            {
                 $isNotAWhitespaceOrAComment = true;
             }
             $currentToken--;
         }
 
-        if ( ! $hasClosingFileComment) {
+        if ( ! $hasClosingFileComment)
+        {
             $error = 'No comment block marks the end of file instead of the closing PHP tag. Please add a comment block containing only "' . $commentTemplate . '".';
             $phpcsFile->addError($error, $currentToken);
         }

build/CodeIgniter/UnusedSniffs/NamingConventions/ConstructorNameSniff.php

Indentation +92 added lines, -92 removed lines

@@ -36,101 +36,101 @@
 {
 
 
-    public $php5Constructors = '1';
-
-
-    /**
-     * Constructs the test with the tokens it wishes to listen for.
-     *
-     * @return void
-     */
-    public function __construct()
-    {
-        parent::__construct(array(T_CLASS, T_INTERFACE), array(T_FUNCTION), true);
-
-    }//end __construct()
-
-
-    /**
-     * Processes this test when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being scanned.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     * @param int                  $currScope A pointer to the start of the scope.
-     *
-     * @return void
-     */
-    protected function processTokenWithinScope(
-        File $phpcsFile,
-        $stackPtr,
-        $currScope
-    ) {
-        $methodName = $phpcsFile->getDeclarationName($stackPtr);
-        $className  = $phpcsFile->getDeclarationName($currScope);
+	public $php5Constructors = '1';
+
+
+	/**
+	 * Constructs the test with the tokens it wishes to listen for.
+	 *
+	 * @return void
+	 */
+	public function __construct()
+	{
+		parent::__construct(array(T_CLASS, T_INTERFACE), array(T_FUNCTION), true);
+
+	}//end __construct()
+
+
+	/**
+	 * Processes this test when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being scanned.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 * @param int                  $currScope A pointer to the start of the scope.
+	 *
+	 * @return void
+	 */
+	protected function processTokenWithinScope(
+		File $phpcsFile,
+		$stackPtr,
+		$currScope
+	) {
+		$methodName = $phpcsFile->getDeclarationName($stackPtr);
+		$className  = $phpcsFile->getDeclarationName($currScope);
 
 	$isPhp4Constructor = strcasecmp($methodName, $className) === 0;
 	$isPhp5Constructor = strcasecmp($methodName, '__construct') === 0;
-        if ($this->php5Constructors != '0') {
-            if ($isPhp4Constructor) {
-                $error = "PHP4 style constructors are not allowed; use \"__construct\" instead";
-                $phpcsFile->addError($error, $stackPtr);
-            }
-        } else {
-            if ($isPhp5Constructor) {
-                $error = "PHP5 style constructors are not allowed; use \"$className\" instead";
-                $phpcsFile->addError($error, $stackPtr);
-            }
-        }
-        if ( ! $isPhp4Constructor && ! $isPhp5Constructor ) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        $parentClassName = $phpcsFile->findExtendedClassName($currScope);
-        $wrongConstructor = '';
-        // prepares the error message and wrong constructor
-        if ($this->php5Constructors != '0') {
-            $error = 'PHP4 style calls to parent constructors are not allowed.';
-            $error = "$error Please use \"parent::__construct\" instead.";
-            if (false !== $parentClassName) {
-                $wrongConstructor = $parentClassName;
-            }
-            // Else $wrongConstructor will be empty
-            // and the test expression will always be false.
-            // It doesn't check that no parent method should be called
-            // when no parent class is defined.
-        } else {
-            $error = 'PHP5 style calls to parent constructors are not allowed.';
-            if (false !== $parentClassName) {
-                $error = "$error Please use \"parent::$parentClassName\" instead.";
-            }
-            $wrongConstructor = '__construct';
-        }
-
-        // looks for the use of a wrong constructor.
-        $endFunctionIndex = $tokens[$stackPtr]['scope_closer'];
-        $doubleColonIndex = $phpcsFile->findNext(
-            array(T_DOUBLE_COLON),
-            $stackPtr,
-            $endFunctionIndex
-        );
-        while ($doubleColonIndex) {
-            if ($tokens[($doubleColonIndex + 1)]['code'] === T_STRING
-                && $tokens[($doubleColonIndex + 1)]['content'] === $wrongConstructor
-            ) {
-                $phpcsFile->addError($error, ($doubleColonIndex + 1));
-            }
-
-            $doubleColonIndex = $phpcsFile->findNext(
-                array(T_DOUBLE_COLON),
-                $doubleColonIndex + 1,
-                $endFunctionIndex
-            );
-        }
-
-    }//end processTokenWithinScope()
+		if ($this->php5Constructors != '0') {
+			if ($isPhp4Constructor) {
+				$error = "PHP4 style constructors are not allowed; use \"__construct\" instead";
+				$phpcsFile->addError($error, $stackPtr);
+			}
+		} else {
+			if ($isPhp5Constructor) {
+				$error = "PHP5 style constructors are not allowed; use \"$className\" instead";
+				$phpcsFile->addError($error, $stackPtr);
+			}
+		}
+		if ( ! $isPhp4Constructor && ! $isPhp5Constructor ) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		$parentClassName = $phpcsFile->findExtendedClassName($currScope);
+		$wrongConstructor = '';
+		// prepares the error message and wrong constructor
+		if ($this->php5Constructors != '0') {
+			$error = 'PHP4 style calls to parent constructors are not allowed.';
+			$error = "$error Please use \"parent::__construct\" instead.";
+			if (false !== $parentClassName) {
+				$wrongConstructor = $parentClassName;
+			}
+			// Else $wrongConstructor will be empty
+			// and the test expression will always be false.
+			// It doesn't check that no parent method should be called
+			// when no parent class is defined.
+		} else {
+			$error = 'PHP5 style calls to parent constructors are not allowed.';
+			if (false !== $parentClassName) {
+				$error = "$error Please use \"parent::$parentClassName\" instead.";
+			}
+			$wrongConstructor = '__construct';
+		}
+
+		// looks for the use of a wrong constructor.
+		$endFunctionIndex = $tokens[$stackPtr]['scope_closer'];
+		$doubleColonIndex = $phpcsFile->findNext(
+			array(T_DOUBLE_COLON),
+			$stackPtr,
+			$endFunctionIndex
+		);
+		while ($doubleColonIndex) {
+			if ($tokens[($doubleColonIndex + 1)]['code'] === T_STRING
+				&& $tokens[($doubleColonIndex + 1)]['content'] === $wrongConstructor
+			) {
+				$phpcsFile->addError($error, ($doubleColonIndex + 1));
+			}
+
+			$doubleColonIndex = $phpcsFile->findNext(
+				array(T_DOUBLE_COLON),
+				$doubleColonIndex + 1,
+				$endFunctionIndex
+			);
+		}
+
+	}//end processTokenWithinScope()
 
 	protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
 	{

Braces +23 added lines, -12 removed lines

@@ -32,8 +32,7 @@ 
  * @license   http://thomas.ernest.fr/developement/php_cs/licence GNU General Public License
  * @link      http://pear.php.net/package/PHP_CodeSniffer
  */
-class ConstructorNameSniff extends AbstractScopeSniff
-{
+class ConstructorNameSniff extends AbstractScopeSniff {
 
 
     public $php5Constructors = '1';
@@ -71,18 +70,24 @@ 
 
 	$isPhp4Constructor = strcasecmp($methodName, $className) === 0;
 	$isPhp5Constructor = strcasecmp($methodName, '__construct') === 0;
-        if ($this->php5Constructors != '0') {
-            if ($isPhp4Constructor) {
+        if ($this->php5Constructors != '0')
+        {
+            if ($isPhp4Constructor)
+            {
                 $error = "PHP4 style constructors are not allowed; use \"__construct\" instead";
                 $phpcsFile->addError($error, $stackPtr);
             }
-        } else {
-            if ($isPhp5Constructor) {
+        }
+        else
+        {
+            if ($isPhp5Constructor)
+            {
                 $error = "PHP5 style constructors are not allowed; use \"$className\" instead";
                 $phpcsFile->addError($error, $stackPtr);
             }
         }
-        if ( ! $isPhp4Constructor && ! $isPhp5Constructor ) {
+        if ( ! $isPhp4Constructor && ! $isPhp5Constructor )
+        {
             return;
         }
 
@@ -91,19 +96,24 @@ 
         $parentClassName = $phpcsFile->findExtendedClassName($currScope);
         $wrongConstructor = '';
         // prepares the error message and wrong constructor
-        if ($this->php5Constructors != '0') {
+        if ($this->php5Constructors != '0')
+        {
             $error = 'PHP4 style calls to parent constructors are not allowed.';
             $error = "$error Please use \"parent::__construct\" instead.";
-            if (false !== $parentClassName) {
+            if (false !== $parentClassName)
+            {
                 $wrongConstructor = $parentClassName;
             }
             // Else $wrongConstructor will be empty
             // and the test expression will always be false.
             // It doesn't check that no parent method should be called
             // when no parent class is defined.
-        } else {
+        }
+        else
+        {
             $error = 'PHP5 style calls to parent constructors are not allowed.';
-            if (false !== $parentClassName) {
+            if (false !== $parentClassName)
+            {
                 $error = "$error Please use \"parent::$parentClassName\" instead.";
             }
             $wrongConstructor = '__construct';
@@ -116,7 +126,8 @@ 
             $stackPtr,
             $endFunctionIndex
         );
-        while ($doubleColonIndex) {
+        while ($doubleColonIndex)
+        {
             if ($tokens[($doubleColonIndex + 1)]['code'] === T_STRING
                 && $tokens[($doubleColonIndex + 1)]['content'] === $wrongConstructor
             ) {

build/CodeIgniter/UnusedSniffs/NamingConventions/ValidClassNameSniff.php

Indentation +37 added lines, -37 removed lines

@@ -39,45 +39,45 @@
 {
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            T_CLASS,
-            T_INTERFACE,
-        );
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			T_CLASS,
+			T_INTERFACE,
+		);
 
-    }//end register()
+	}//end register()
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param File $phpcsFile The current file being processed.
-     * @param int                  $stackPtr  The position of the current token
-     *                                        in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        // get the class name
-        $className = trim($phpcsFile->getDeclarationName($stackPtr));
-        // compute the expected class name
-        // [^_] means "something different from _", but not "nothing or something different from _"
-        $lcClassNameChunk = preg_replace('/([^_])([A-Z])/', '${1}_${2}', $className);
-        $expectedClassName
-            = strtoupper($className[0]) . strtolower(substr($lcClassNameChunk,1));
-        // ensures that the current class name
-        // and the expected class name are identical
-        if (0 !== strcmp($className, $expectedClassName)) {
-            $error =  'Class names should always have their first letter uppercase. Multiple words should be separated with an underscore, and not CamelCased. Please consider ' . $expectedClassName . ' instead of ' . $className . '.';
-            $phpcsFile->addError($error, $stackPtr);
-        }
-    }//end process()
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param File $phpcsFile The current file being processed.
+	 * @param int                  $stackPtr  The position of the current token
+	 *                                        in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		// get the class name
+		$className = trim($phpcsFile->getDeclarationName($stackPtr));
+		// compute the expected class name
+		// [^_] means "something different from _", but not "nothing or something different from _"
+		$lcClassNameChunk = preg_replace('/([^_])([A-Z])/', '${1}_${2}', $className);
+		$expectedClassName
+			= strtoupper($className[0]) . strtolower(substr($lcClassNameChunk,1));
+		// ensures that the current class name
+		// and the expected class name are identical
+		if (0 !== strcmp($className, $expectedClassName)) {
+			$error =  'Class names should always have their first letter uppercase. Multiple words should be separated with an underscore, and not CamelCased. Please consider ' . $expectedClassName . ' instead of ' . $className . '.';
+			$phpcsFile->addError($error, $stackPtr);
+		}
+	}//end process()
 
 }//end class
 

Braces +3 added lines, -3 removed lines

@@ -35,8 +35,7 @@ 
 use PHP_CodeSniffer\Sniffs\Sniff;
 use PHP_CodeSniffer\Files\File;
 
-class ValidClassNameSniff implements Sniff
-{
+class ValidClassNameSniff implements Sniff {
 
 
     /**
@@ -73,7 +72,8 @@ 
             = strtoupper($className[0]) . strtolower(substr($lcClassNameChunk,1));
         // ensures that the current class name
         // and the expected class name are identical
-        if (0 !== strcmp($className, $expectedClassName)) {
+        if (0 !== strcmp($className, $expectedClassName))
+        {
             $error =  'Class names should always have their first letter uppercase. Multiple words should be separated with an underscore, and not CamelCased. Please consider ' . $expectedClassName . ' instead of ' . $className . '.';
             $phpcsFile->addError($error, $stackPtr);
         }