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

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

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

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

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

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

build/CodeIgniter/UnusedSniffs/NamingConventions/ValidVariableNameSniff.php

Braces +118 added lines, -56 removed lines

@@ -40,8 +40,7 @@ 
 use PHP_CodeSniffer\Sniffs\AbstractVariableSniff;
 use PHP_CodeSniffer\Files\File;
 
-class ValidVariableNameSniff extends AbstractVariableSniff
-{
+class ValidVariableNameSniff extends AbstractVariableSniff {
 
 
     /**
@@ -61,13 +60,16 @@ 
         $varName = substr($varTk['content'], 1);
         $varProps = $phpcsFile->getMemberProperties($stackPtr);
         // check(s)
-        if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName) ) {
+        if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName) )
+        {
             return;
         }
-        if ( ! $this->checkVisibilityPrefix($phpcsFile, $stackPtr, $varName, $varProps)) {
+        if ( ! $this->checkVisibilityPrefix($phpcsFile, $stackPtr, $varName, $varProps))
+        {
             return;
         }
-        if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName)) {
+        if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName))
+        {
             return;
         }
 
@@ -89,14 +91,17 @@ 
         $varTk = $tokens[$stackPtr];
         $varName = substr($varTk['content'], 1);
         // skip the current object variable, i.e. $this
-        if (0 === strcmp($varName, 'this')) {
+        if (0 === strcmp($varName, 'this'))
+        {
             return;
         }
         // check(s)
-        if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName)) {
+        if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName))
+        {
             return;
         }
-        if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName)) {
+        if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName))
+        {
             return;
         }
 
@@ -117,16 +122,19 @@ 
         $stringTk = $tokens[$stackPtr];
         $stringString = $stringTk['content'];
         $varAt = self::_getVariablePosition($stringString, 0);
-        while (false !== $varAt) {
+        while (false !== $varAt)
+        {
             // get variable name
             $matches = array();
             preg_match('/^\$\{?([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\}?/', substr($stringString, $varAt), $matches);
             $varName = $matches[1];
             // check(s)
-            if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName)) {
+            if ( ! $this->checkLowerCase($phpcsFile, $stackPtr, $varName))
+            {
                 return;
             }
-            if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName)) {
+            if ( ! $this->checkLength($phpcsFile, $stackPtr, $varName))
+            {
                 return;
             }
             // prepare checking next variable
@@ -152,14 +160,18 @@ 
     protected function checkLowerCase(File $phpcsFile, $stackPtr, $varName)
     {
         $isInLowerCase = true;
-        if (0 !== strcmp($varName, strtolower($varName))) {
+        if (0 !== strcmp($varName, strtolower($varName)))
+        {
             // get the expected variable name
             $varNameWithUnderscores = preg_replace('/([A-Z])/', '_${1}', $varName);
             $expectedVarName = strtolower(ltrim($varNameWithUnderscores, '_'));
             // adapts the error message to the error case
-            if (strlen($varNameWithUnderscores) > strlen($varName)) {
+            if (strlen($varNameWithUnderscores) > strlen($varName))
+            {
                 $error = 'Variables should not use CamelCasing or start with a Capital.';
-            } else {
+            }
+            else
+            {
                 $error = 'Variables should be entirely lowercased.';
             }
             $error = $error . 'Please consider "' . $expectedVarName
@@ -194,16 +206,22 @@ 
         $isVisibilityPrefixRight = true;
         $scope = $varProps['scope'];
         // If it's a private variable, it must have an underscore on the front.
-        if ($scope === 'private' && $varName{0} !== '_') {
+        if ($scope === 'private' && $varName{0} !== '_')
+        {
             $error = "Private variable name \"$varName\" must be prefixed with an underscore";
             $phpcsFile->addError($error, $stackPtr);
             $isVisibilityPrefixRight = false;
-        } else if ($scope !== 'private' && $varName{0} === '_') {
+        }
+        else if ($scope !== 'private' && $varName{0} === '_')
+        {
             // If it's not a private variable,
             // then it must not start with an underscore.
-            if (isset ($scopeSpecified) && true === $scopeSpecified) {
+            if (isset ($scopeSpecified) && true === $scopeSpecified)
+            {
                 $error = "Public variable name \"$varName\" must not be prefixed with an underscore";
-            } else {
+            }
+            else
+            {
                 $error = ucfirst($scope) . " variable name \"$varName\" must not be prefixed with an underscore";
             }
             $phpcsFile->addError($error, $stackPtr);
@@ -241,13 +259,16 @@ 
         $isLengthRight = true;
         // cleans variable name
         $varName = ltrim($varName, '_');
-        if (strlen($varName) <= $minLength) {
+        if (strlen($varName) <= $minLength)
+        {
             // skips adding an error, if it is a specific variable name
-            if (in_array($varName, $allowedShortName)) {
+            if (in_array($varName, $allowedShortName))
+            {
                 return $isLengthRight;
             }
             // skips adding an error, if the variable is in a for loop
-            if (false !== self::_isInForLoop($phpcsFile, $stackPtr, $varName)) {
+            if (false !== self::_isInForLoop($phpcsFile, $stackPtr, $varName))
+            {
                 return $isLengthRight;
             }
             // adds the error message finally
@@ -281,19 +302,25 @@ 
     private static function _isInForLoop(File $phpcsFile, $stackPtr, $varName)
     {
         $keepLookingFromPtr = $stackPtr;
-        while (false !== $keepLookingFromPtr) {
+        while (false !== $keepLookingFromPtr)
+        {
             // looks if it is in (head or body) of a for loop
             $forPtr = self::_isInForLoopHead($phpcsFile, $keepLookingFromPtr);
-            if (false === $forPtr) {
+            if (false === $forPtr)
+            {
                 $forPtr = self::_isInForLoopBody($phpcsFile, $keepLookingFromPtr);
             }
             // checks if it is declared in here and prepares next step
-            if (false !== $forPtr) {
-                if (false !== self::_isDeclaredInForLoop($phpcsFile, $forPtr, $varName)) {
+            if (false !== $forPtr)
+            {
+                if (false !== self::_isDeclaredInForLoop($phpcsFile, $forPtr, $varName))
+                {
                     return $forPtr;
                 }
                 $keepLookingFromPtr = $forPtr;
-            } else {
+            }
+            else
+            {
                 $keepLookingFromPtr = false;
             }
         }
@@ -318,9 +345,11 @@ 
         $isInForLoop = false;
         $tokens = $phpcsFile->getTokens();
         $currentTk = $tokens[$stackPtr];
-        if (array_key_exists('nested_parenthesis', $currentTk)) {
+        if (array_key_exists('nested_parenthesis', $currentTk))
+        {
             $nestedParenthesis = $currentTk['nested_parenthesis'];
-            foreach ( $nestedParenthesis as $openParPtr => $closeParPtr) {
+            foreach ( $nestedParenthesis as $openParPtr => $closeParPtr)
+            {
                 $nonWhitspacePtr = $phpcsFile->findPrevious(
                     array(T_WHITESPACE),
                     $openParPtr - 1,
@@ -329,9 +358,11 @@ 
                     null,
                     true
                 );
-                if (false !== $nonWhitspacePtr) {
+                if (false !== $nonWhitspacePtr)
+                {
                     $isFor = T_FOR === $tokens[$nonWhitspacePtr]['code'];
-                    if ($isFor) {
+                    if ($isFor)
+                    {
                         $isInForLoop = $nonWhitspacePtr;
                         break;
                     }
@@ -365,11 +396,14 @@ 
 
         // looks for a for loop having a body not enclosed with curly brackets,
         // which involves that its body contains only one instruction.
-        if (is_array($parentPtrAndCode) && ! empty($parentPtrAndCode)) {
+        if (is_array($parentPtrAndCode) && ! empty($parentPtrAndCode))
+        {
             $parentCode = reset($parentPtrAndCode);
             $parentPtr = key($parentPtrAndCode);
             $openBracketPtr = $tokens[$parentPtr]['scope_opener'];
-        } else {
+        }
+        else
+        {
             $parentCode = 0;
             $parentPtr = 0;
             $openBracketPtr = 0;
@@ -377,7 +411,8 @@ 
         $openResearchScopePtr = $stackPtr;
         // recursive search, since a for statement may englobe other inline
         // control statement or may be near to function calls, etc...
-        while (false !== $openResearchScopePtr) {
+        while (false !== $openResearchScopePtr)
+        {
             $closeParPtr = $phpcsFile->findPrevious(
                 array(T_CLOSE_PARENTHESIS),
                 $openResearchScopePtr,
@@ -388,46 +423,58 @@ 
             );
             // is there a closing parenthesis with a control statement before
             // the previous instruction ?
-            if (false !== $closeParPtr) {
+            if (false !== $closeParPtr)
+            {
                 // is there no opening curly bracket specific to
                 // set of instructions, between the closing parenthesis
                 // and the current token ?
-                if ($openBracketPtr < $closeParPtr) {
+                if ($openBracketPtr < $closeParPtr)
+                {
                     // starts the search from the token before the closing
                     // parenthesis, if it isn't a for statement
                     $openResearchScopePtr = $closeParPtr - 1;
                     // is this parenthesis associated with a for statement ?
                     $closeParenthesisTk = $tokens[$closeParPtr];
-                    if (array_key_exists('parenthesis_owner', $closeParenthesisTk)) {
+                    if (array_key_exists('parenthesis_owner', $closeParenthesisTk))
+                    {
                         $mayBeForPtr = $closeParenthesisTk['parenthesis_owner'];
                         $mayBeForTk = $tokens[$mayBeForPtr];
-                        if (T_FOR === $mayBeForTk['code']) {
+                        if (T_FOR === $mayBeForTk['code'])
+                        {
                             return $mayBeForPtr;
                         }
                     }
-                } else {
+                }
+                else
+                {
                     // if it is about a for loop, don't go further
                     // and detect it after one more loop execution, do it now
-                    if (T_FOR === $parentCode) {
+                    if (T_FOR === $parentCode)
+                    {
                         return $parentPtr;
                     }
                     // starts the search from the token before the one
                     // englobing the current statement
                     $openResearchScopePtr = $parentPtr - 1;
                     // re-initialize variables about the englobing structure
-                    if (is_array($parentPtrAndCode)) {
+                    if (is_array($parentPtrAndCode))
+                    {
                         $parentCode = next($parentPtrAndCode);
                         $parentPtr = key($parentPtrAndCode);
                         $openBracketPtr = $tokens[$parentPtr]['scope_opener'];
                     }
                 }
-            } else {
+            }
+            else
+            {
                 $openResearchScopePtr = false;
             }
         }
         // looks for a for loop having a body enclosed with curly brackets
-        foreach ($parentPtrAndCode as $parentPtr => $parentCode) {
-            if (T_FOR === $parentCode) {
+        foreach ($parentPtrAndCode as $parentPtr => $parentCode)
+        {
+            if (T_FOR === $parentCode)
+            {
                 return $parentPtr;
             }
         }
@@ -452,13 +499,15 @@ 
         $isDeclaredInFor = false;
         $tokens = $phpcsFile->getTokens();
         $forVarPtrs = self::_getVarDeclaredInFor($phpcsFile, $forPtr);
-        foreach ($forVarPtrs as $forVarPtr) {
+        foreach ($forVarPtrs as $forVarPtr)
+        {
             $forVarTk = $tokens[$forVarPtr];
             // get variable name
             $matches = array();
             preg_match('/^\$\{?([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\}?/', $forVarTk['content'], $matches);
             $forVarName = $matches[1];
-            if (0 === strcmp($forVarName, $varName)) {
+            if (0 === strcmp($forVarName, $varName))
+            {
                 $isDeclaredInFor = $forVarPtr;
                 break;
             }
@@ -484,7 +533,8 @@ 
     {
         $tokens = $phpcsFile->getTokens();
         $forTk = $tokens[$forPtr];
-        if (T_FOR !== $forTk['code']) {
+        if (T_FOR !== $forTk['code'])
+        {
             throw new PHP_CodeSniffer_Exception('$forPtr must be of type T_FOR');
         }
         $openParPtr = $forTk['parenthesis_opener'];
@@ -496,7 +546,8 @@ 
             $openParPtr + 1,
             $endOfDeclPtr
         );
-        while (false !== $varPtr) {
+        while (false !== $varPtr)
+        {
             $forVarPtrs [] = $varPtr;
             $varPtr = $phpcsFile->findNext(
                 array(T_VARIABLE),
@@ -524,20 +575,27 @@ 
     {
         $var_starts_at = strpos($haystack, '$', $offset);
         $is_a_var = false;
-        while (false !== $var_starts_at && ! $is_a_var) {
+        while (false !== $var_starts_at && ! $is_a_var)
+        {
             // makes sure that $ is used for a variable and not as a symbol,
             // if $ is protected with the escape char, then it is a symbol.
-            if (0 !== strcmp($haystack[$var_starts_at - 1], '\\')) {
-                if (0 === strcmp($haystack[$var_starts_at + 1], '{')) {
+            if (0 !== strcmp($haystack[$var_starts_at - 1], '\\'))
+            {
+                if (0 === strcmp($haystack[$var_starts_at + 1], '{'))
+                {
                     // there is an opening brace in the right place
                     // so it looks for the closing brace in the right place
                     $hsChunk2 = substr($haystack, $var_starts_at + 2);
-                    if (1 === preg_match('/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*\}/', $hsChunk2)) {
+                    if (1 === preg_match('/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*\}/', $hsChunk2))
+                    {
                         $is_a_var = true;
                     }
-                } else {
+                }
+                else
+                {
                     $hsChunk1 = substr($haystack, $var_starts_at + 1);
-                    if (1 === preg_match('/^[a-zA-Z_\x7f-\xff]/', $hsChunk1)) {
+                    if (1 === preg_match('/^[a-zA-Z_\x7f-\xff]/', $hsChunk1))
+                    {
                         // $ is used for a variable and not as a symbol,
                         // since what follows $ matchs the definition of
                         // a variable label for PHP.
@@ -547,13 +605,17 @@ 
             }
             // update $var_starts_at for the next variable
             // only if no variable was found, since it is returned otherwise.
-            if ( ! $is_a_var) {
+            if ( ! $is_a_var)
+            {
                 $var_starts_at = strpos($haystack, '$', $var_starts_at + 1);
             }
         }
-        if ($is_a_var) {
+        if ($is_a_var)
+        {
             return $var_starts_at;
-        } else {
+        }
+        else
+        {
             return false;
         }
     }//end _getVariablePosition()