timw4mail / HummingBirdAnimeClient

build/CodeIgniter/Sniffs/Commenting/InlineCommentSniff.php

Doc Comments +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -118,7 +118,7 @@ 
      * @param File $phpcsFile The current file being scanned.
      * @param int                  $stackPtr  Pointer to the first comment line.
      *
-     * @return type array Pointers to tokens making up the comment block.
+     * @return integer[] array Pointers to tokens making up the comment block.
      */
     private function _getCommentBlock(File $phpcsFile, $stackPtr)
     {
@@ -145,7 +145,7 @@ 
      * Add errors to $phpcsFile, if $commentLines isn't enclosed with blank lines.
      *
      * @param File $phpcsFile    The current file being scanned.
-     * @param array                $commentLines Lines of the comment block being checked.
+     * @param integer[]                $commentLines Lines of the comment block being checked.
      *
      * @return bool TRUE if $commentLines is enclosed with at least a blank line
      * before and after, FALSE otherwise.

Spacing +5 added lines, -5 removed lines

@@ -94,17 +94,17 @@ 
     {
         $tokens = $phpcsFile->getTokens();
         if ($tokens[$stackPtr]['content']{0} === '#') {
-            $error  = 'Perl-style comments are not allowed; use "// Comment" or DocBlock comments instead';
+            $error = 'Perl-style comments are not allowed; use "// Comment" or DocBlock comments instead';
             $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
             return FALSE;
         } else if (substr($tokens[$stackPtr]['content'], 0, 2) === '/*'
             || $tokens[$stackPtr]['content']{0} === '*'
         ) {
-            $error  = 'Multi lines comments are not allowed; use "// Comment" DocBlock comments instead';
+            $error = 'Multi lines comments are not allowed; use "// Comment" DocBlock comments instead';
             $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
             return FALSE;
         } else if (substr($tokens[$stackPtr]['content'], 0, 2) !== '//') {
-            $error  = 'Use single line or DocBlock comments within code';
+            $error = 'Use single line or DocBlock comments within code';
             $phpcsFile->addError($error, $stackPtr, 'WrongStyle');
             return FALSE;
         }
@@ -162,7 +162,7 @@ 
             $firstPreviousSpacePtr--;
         }
         if ($tokens[$firstPreviousSpacePtr]['line'] >= $tokens[$firstCommentPtr]['line'] - 1) {
-            $error  = "Please add a blank line before comments counting more than {$this->longCommentLimit} lines.";
+            $error = "Please add a blank line before comments counting more than {$this->longCommentLimit} lines.";
             $phpcsFile->addError($error, $firstCommentPtr, 'LongCommentWithoutSpacing');
             $hasBlankLinesAround = FALSE;
         }
@@ -174,7 +174,7 @@ 
             $lastNextSpacePtr++;
         }
         if ($tokens[$lastNextSpacePtr]['line'] <= $tokens[$lastCommentPtr]['line'] + 1) {
-            $error  = "Please add a blank line after comments counting more than {$this->longCommentLimit} lines.";
+            $error = "Please add a blank line after comments counting more than {$this->longCommentLimit} lines.";
             $phpcsFile->addError($error, $lastCommentPtr, 'LongCommentWithoutSpacing');
             $hasBlankLinesAround = FALSE;
         }

build/CodeIgniter/Sniffs/Files/ByteOrderMarkSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function register()
     {
-        return array( T_OPEN_TAG );
+        return array(T_OPEN_TAG);
     }//end register()
 
     /**
@@ -74,7 +74,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr )
+    public function process(File $phpcsFile, $stackPtr)
     {
         // We are only interested if this is the first open tag.
         if ($stackPtr !== 0) {

build/CodeIgniter/Sniffs/Files/Utf8EncodingSniff.php

Doc Comments +2 added lines, -1 removed lines

@@ -36,7 +36,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -174,6 +174,7 @@ 
      *
      * @param string $str String to split.
 	 * @param int $len number of characters per chunk
+	 * @param string $glue
      *
      * @return array string array after splitting
      *

Spacing +10 added lines, -10 removed lines

@@ -96,10 +96,10 @@ 
      */
     private static function _checkUtf8W3c($content)
     {
-        $content_chunks=self::mb_chunk_split($content, 4096, '');
-    	foreach($content_chunks as $content_chunk)
+        $content_chunks = self::mb_chunk_split($content, 4096, '');
+    	foreach ($content_chunks as $content_chunk)
 		{
-			$preg_result= preg_match(
+			$preg_result = preg_match(
             '%^(?:
                   [\x09\x0A\x0D\x20-\x7E]            # ASCII
                 | [\xC2-\xDF][\x80-\xBF]             # non-overlong 2-byte
@@ -112,7 +112,7 @@ 
             )*$%xs',
             $content_chunk
 			);
-			if($preg_result!==1)
+			if ($preg_result !== 1)
 			{
 				return false;
 			}
@@ -141,9 +141,9 @@ 
                 if (($c >= 254)) {
                     return false;
                 } elseif ($c >= 252) {
-                    $bits=6;
+                    $bits = 6;
                 } elseif ($c >= 248) {
-                    $bits=5;
+                    $bits = 5;
                 } elseif ($c >= 240) {
                     $bytes = 4;
                 } elseif ($c >= 224) {
@@ -182,14 +182,14 @@ 
 	private static function mb_chunk_split($str, $len, $glue)
 	{
 		if (empty($str)) return false;
-		$array = self::mbStringToArray ($str);
+		$array = self::mbStringToArray($str);
 		$n = -1;
 		$new = Array();
 		foreach ($array as $char) {
 			$n++;
-			if ($n < $len) $new []= $char;
+			if ($n < $len) $new [] = $char;
 			elseif ($n == $len) {
-				$new []= $glue . $char;
+				$new [] = $glue . $char;
 				$n = 0;
 			}
 		}
@@ -204,7 +204,7 @@ 
      *
      * @see http://php.net/manual/en/function.chunk-split.php
      */
-	private static function mbStringToArray ($str)
+	private static function mbStringToArray($str)
 	{
 		if (empty($str)) return false;
 		$len = mb_strlen($str);

build/CodeIgniter/Sniffs/Operators/LogicalOperatorAndSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 	/**
      * Returns an array of tokens this test wants to listen for: symbolic and literal operators and.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

build/CodeIgniter/Sniffs/Operators/UppercaseLogicalOperatorOrSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * Returns an array of tokens this test wants to listen for: literal and symbolic operators or.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

build/CodeIgniter/Sniffs/Strings/DoubleQuoteUsageSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
 	 * @param string               $dblQtString The double-quoted string content,
 	 *                                          i.e. without quotes.
 	 *
-	 * @return void
+	 * @return boolean
 	 */
 	protected function processDoubleQuotedString (File $phpcsFile, $stackPtr, $dblQtString)
 	{
@@ -134,7 +134,7 @@ 
 	 * @param string               $sglQtString The single-quoted string content,
 	 *                                          i.e. without quotes.
 	 *
-	 * @return void
+	 * @return boolean
 	 */
 	protected function processSingleQuotedString (File $phpcsFile, $stackPtr, $sglQtString)
 	{

Spacing +24 added lines, -24 removed lines

@@ -89,10 +89,10 @@ 
 	 *
 	 * @return void
 	 */
-	protected function processDoubleQuotedString (File $phpcsFile, $stackPtr, $dblQtString)
+	protected function processDoubleQuotedString(File $phpcsFile, $stackPtr, $dblQtString)
 	{
 		$variableFound = FALSE;
-		$strTokens = token_get_all('<?php '.$dblQtString);
+		$strTokens = token_get_all('<?php ' . $dblQtString);
 		$strPtr = 1; // skip php opening tag added by ourselves
 		$requireDblQuotes = FALSE;
 		while ($strPtr < count($strTokens)) {
@@ -136,10 +136,10 @@ 
 	 *
 	 * @return void
 	 */
-	protected function processSingleQuotedString (File $phpcsFile, $stackPtr, $sglQtString)
+	protected function processSingleQuotedString(File $phpcsFile, $stackPtr, $sglQtString)
 	{
 		$variableFound = FALSE;
-		$strTokens = token_get_all('<?php '.$sglQtString);
+		$strTokens = token_get_all('<?php ' . $sglQtString);
 		$strPtr = 1; // skip php opening tag added by ourselves
 		while ($strPtr < count($strTokens)) {
 			$strToken = $strTokens[$strPtr];
@@ -171,17 +171,17 @@ 
 	 * @return array The attribute name associated to index 'var', an array with
 	 * indexes 'obj' and 'attr' or an array with indexes 'arr' and 'idx'.
 	 */
-	private function _parseVariable ($strTokens, &$strPtr)
+	private function _parseVariable($strTokens, &$strPtr)
 	{
 		if ( ! in_array($strTokens[$strPtr][0], array(T_VARIABLE, T_STRING_VARNAME))) {
-			throw new Exception ('Expected variable name.');
+			throw new Exception('Expected variable name.');
 		}
 		$var = $strTokens[$strPtr][1];
 		$strPtr++;
 		$startStrPtr = $strPtr;
 		try {
 			$attr = $this->_parseObjectAttribute($strTokens, $strPtr);
-			return array ('obj' => $var, 'attr' => $attr);
+			return array('obj' => $var, 'attr' => $attr);
 		} catch (Exception $err) {
 			if ($strPtr !== $startStrPtr) {
 				throw $err;
@@ -189,13 +189,13 @@ 
 		}
 		try {
 			$idx = $this->_parseArrayIndexes($strTokens, $strPtr);
-			return array ('arr' => $var, 'idx' => $idx);
+			return array('arr' => $var, 'idx' => $idx);
 		} catch (Exception $err) {
 			if ($strPtr !== $startStrPtr) {
 				throw $err;
 			}
 		}
-		return array ('var' => $var);
+		return array('var' => $var);
 	}//end _parseVariable()
 
 
@@ -215,21 +215,21 @@ 
 	 * @return mixed The attribute name as a string, an array with indexes
 	 * 'obj' and 'attr' or an array with indexes 'arr' and 'idx'.
 	 */
-	private function _parseObjectAttribute ($strTokens, &$strPtr)
+	private function _parseObjectAttribute($strTokens, &$strPtr)
 	{
 		if (T_OBJECT_OPERATOR !== $strTokens[$strPtr][0]) {
-			throw new Exception ('Expected ->.');
+			throw new Exception('Expected ->.');
 		}
 		$strPtr++;
 		if (T_STRING !== $strTokens[$strPtr][0]) {
-			throw new Exception ('Expected an object attribute.');
+			throw new Exception('Expected an object attribute.');
 		}
 		$attr = $strTokens[$strPtr][1];
 		$strPtr++;
 		$startStrPtr = $strPtr;
 		try {
 			$sub_attr = $this->_parseObjectAttribute($strTokens, $strPtr);
-			return array ('obj' => $attr, 'attr' => $sub_attr);
+			return array('obj' => $attr, 'attr' => $sub_attr);
 		} catch (Exception $err) {
 			if ($strPtr !== $startStrPtr) {
 				throw $err;
@@ -237,7 +237,7 @@ 
 		}
 		try {
 			$idx = $this->_parseArrayIndexes($strTokens, $strPtr);
-			return array ('arr' => $attr, 'idx' => $idx);
+			return array('arr' => $attr, 'idx' => $idx);
 		} catch (Exception $err) {
 			if ($strPtr !== $startStrPtr) {
 				throw $err;
@@ -260,7 +260,7 @@ 
 	 *
 	 * @return array Indexes in the same order as in the string.
 	 */
-	private function _parseArrayIndexes ($strTokens, &$strPtr)
+	private function _parseArrayIndexes($strTokens, &$strPtr)
 	{
 		$indexes = array($this->_parseArrayIndex($strTokens, $strPtr));
 		try {
@@ -290,19 +290,19 @@ 
 	 *
 	 * @return string Index between the 2 square brackets
 	 */
-	private function _parseArrayIndex ($strTokens, &$strPtr)
+	private function _parseArrayIndex($strTokens, &$strPtr)
 	{
 		if ('[' !== $strTokens[$strPtr]) {
-			throw new Exception ('Expected [.');
+			throw new Exception('Expected [.');
 		}
 		$strPtr++;
-		if (! in_array($strTokens[$strPtr][0], array(T_CONSTANT_ENCAPSED_STRING, T_LNUMBER))) {
-			throw new Exception ('Expected an array index.');
+		if ( ! in_array($strTokens[$strPtr][0], array(T_CONSTANT_ENCAPSED_STRING, T_LNUMBER))) {
+			throw new Exception('Expected an array index.');
 		}
 		$index = $strTokens[$strPtr][1];
 		$strPtr++;
 		if (']' !== $strTokens[$strPtr]) {
-			throw new Exception ('Expected ].');
+			throw new Exception('Expected ].');
 		}
 		$strPtr++;
 		return $index;
@@ -380,11 +380,11 @@ 
      *
      * @return void
      */
-    protected function processDoubleQuotedString (File $phpcsFile, $stackPtr, $qtString)
+    protected function processDoubleQuotedString(File $phpcsFile, $stackPtr, $qtString)
     {
         // so there should be at least a single quote or a special char
         // if there are the 2 kinds of quote and no special char, then add a warning
-        $has_variable = parent::processDoubleQuotedString($phpcsFile, $stackPtr, '"'.$qtString.'"');
+        $has_variable = parent::processDoubleQuotedString($phpcsFile, $stackPtr, '"' . $qtString . '"');
         $has_specific_sequence = $this->_hasSpecificSequence($qtString);
         $dbl_qt_at = strpos($qtString, '"');
         $smpl_qt_at = strpos($qtString, "'");
@@ -413,14 +413,14 @@ 
      *
      * @return void
      */
-    protected function processSingleQuotedString (File $phpcsFile, $stackPtr, $qtString)
+    protected function processSingleQuotedString(File $phpcsFile, $stackPtr, $qtString)
     {
         // if there is single quotes without additional double quotes,
         // then user is allowed to use double quote to avoid having to
         // escape single quotes. Don't add the warning, if an error was
         // already added, because a variable was found in a single-quoted
         // string.
-        $has_variable = parent::processSingleQuotedString($phpcsFile, $stackPtr, "'".$qtString."'");
+        $has_variable = parent::processSingleQuotedString($phpcsFile, $stackPtr, "'" . $qtString . "'");
         $dbl_qt_at = strpos($qtString, '"');
         $smpl_qt_at = strpos($qtString, "'");
         if (false === $has_variable && false !== $smpl_qt_at && false === $dbl_qt_at) {

build/CodeIgniter/Sniffs/WhiteSpace/LogicalNotSpacingSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return string[]
      */
     public function register()
     {

build/CodeIgniter/UnusedSniffs/Files/ClosingLocationCommentSniff.php

Doc Comments +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -134,7 +134,7 @@ 
      * application root. Parent directory of the application root are allowed
      * but not mandatory.
      *
-     * @return string|bool The relative path from $appRoot to $filePath, or
+     * @return false|string The relative path from $appRoot to $filePath, or
      * false if $appRoot cannot be found in $filePath.
      */
     private static function _getLocationPath ($filePath, $appRoot)

Spacing +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
      * @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)
+    private static function _getLocationPath($filePath, $appRoot)
     {
         // removes the path to application root
         // from the beginning of the file path

RoboFile.php

Doc Comments +5 added lines, -1 removed lines

@@ -2,6 +2,10 @@ 
 if ( ! function_exists('glob_recursive'))
 {
 	// Does not support flag GLOB_BRACE
+
+	/**
+	 * @param string $pattern
+	 */
 	function glob_recursive($pattern, $flags = 0)
 	{
 		$files = glob($pattern, $flags);
@@ -320,7 +324,7 @@ 
 	 * Shortcut for joining an array of command arguments
 	 * and then running it
 	 *
-	 * @param array $cmd_parts - command arguments
+	 * @param string[] $cmd_parts - command arguments
 	 * @param string $join_on - what to join the command arguments with
 	 */
 	protected function _run(array $cmd_parts, $join_on = ' ')

Spacing +8 added lines, -8 removed lines

@@ -6,9 +6,9 @@ 
 	{
 		$files = glob($pattern, $flags);
 
-		foreach (glob(dirname($pattern).'/*', GLOB_ONLYDIR|GLOB_NOSORT) as $dir)
+		foreach (glob(dirname($pattern) . '/*', GLOB_ONLYDIR | GLOB_NOSORT) as $dir)
 		{
-			$files = array_merge($files, glob_recursive($dir.'/'.basename($pattern), $flags));
+			$files = array_merge($files, glob_recursive($dir . '/' . basename($pattern), $flags));
 		}
 
 		return $files;
@@ -80,13 +80,13 @@ 
 			'build/humbug.json',
 			'build/humbug-log.txt',
 		];
-		array_map(function ($file) {
+		array_map(function($file) {
 			@unlink($file);
 		}, $cleanFiles);
 
 		// So the task doesn't complain,
 		// make any 'missing' dirs to cleanup
-		array_map(function ($dir) {
+		array_map(function($dir) {
 			if ( ! is_dir($dir))
 			{
 				`mkdir -p {$dir}`;
@@ -130,7 +130,7 @@ 
 
 		$chunks = array_chunk($files, 6);
 
-		foreach($chunks as $chunk)
+		foreach ($chunks as $chunk)
 		{
 			$this->parallelLint($chunk);
 		}
@@ -241,10 +241,10 @@ 
 			->monitor('composer.json', function() {
 				$this->taskComposerUpdate()->run();
 			})
-			->monitor('src', function () {
+			->monitor('src', function() {
 				$this->taskExec('test')->run();
 			})
-			->monitor('tests', function () {
+			->monitor('tests', function() {
 				$this->taskExec('test')->run();
 			})
 			->run();
@@ -295,7 +295,7 @@ 
 			->timeout(5)
 			->printed(FALSE);
 
-		foreach($chunk as $file)
+		foreach ($chunk as $file)
 		{
 			$task = $task->process("php -l {$file}");
 		}