Strategy11 / formidable-forms

classes/controllers/FrmUsageController.php

Spacing +5 added lines, -5 removed lines

@@ -158,15 +158,15 @@
 
 		$flows_data = self::get_flows_data();
 
-		if ( ! isset( $flows_data[ $key ] ) ) {
-			$flows_data[ $key ] = array();
+		if ( ! isset( $flows_data[$key] ) ) {
+			$flows_data[$key] = array();
 		}
 
-		if ( ! isset( $flows_data[ $key ][ $value ] ) ) {
-			$flows_data[ $key ][ $value ] = 0;
+		if ( ! isset( $flows_data[$key][$value] ) ) {
+			$flows_data[$key][$value] = 0;
 		}
 
-		$flows_data[ $key ][ $value ]++;
+		$flows_data[$key][$value] ++;
 		update_option( self::FLOWS_ACTION_NAME, $flows_data );
 	}
 

phpcs-sniffs/Formidable/Sniffs/CodeAnalysis/MoveVariableBelowEarlyReturnSniff.php

Spacing +27 added lines, -27 removed lines

@@ -60,12 +60,12 @@ 
 		$tokens = $phpcsFile->getTokens();
 
 		// Make sure this function has a scope.
-		if ( ! isset( $tokens[ $stackPtr ]['scope_opener'] ) || ! isset( $tokens[ $stackPtr ]['scope_closer'] ) ) {
+		if ( ! isset( $tokens[$stackPtr]['scope_opener'] ) || ! isset( $tokens[$stackPtr]['scope_closer'] ) ) {
 			return;
 		}
 
-		$functionOpener = $tokens[ $stackPtr ]['scope_opener'];
-		$functionCloser = $tokens[ $stackPtr ]['scope_closer'];
+		$functionOpener = $tokens[$stackPtr]['scope_opener'];
+		$functionCloser = $tokens[$stackPtr]['scope_closer'];
 
 		// Find the first statement in the function.
 		$firstStatement = $phpcsFile->findNext( T_WHITESPACE, $functionOpener + 1, $functionCloser, true );
@@ -75,12 +75,12 @@ 
 		}
 
 		// Check if the first statement is a variable assignment.
-		if ( $tokens[ $firstStatement ]['code'] !== T_VARIABLE ) {
+		if ( $tokens[$firstStatement]['code'] !== T_VARIABLE ) {
 			return;
 		}
 
 		// Get the variable name.
-		$variableName = $tokens[ $firstStatement ]['content'];
+		$variableName = $tokens[$firstStatement]['content'];
 
 		// Check if this variable is a function parameter (skip those, especially by-reference params).
 		if ( $this->isVariableAParameter( $phpcsFile, $tokens, $stackPtr, $variableName ) ) {
@@ -102,22 +102,22 @@ 
 		}
 
 		// Check if the next statement is an if with an early return.
-		if ( $tokens[ $nextStatement ]['code'] !== T_IF ) {
+		if ( $tokens[$nextStatement]['code'] !== T_IF ) {
 			return;
 		}
 
 		// Make sure the if has a scope.
-		if ( ! isset( $tokens[ $nextStatement ]['scope_opener'] ) || ! isset( $tokens[ $nextStatement ]['scope_closer'] ) ) {
+		if ( ! isset( $tokens[$nextStatement]['scope_opener'] ) || ! isset( $tokens[$nextStatement]['scope_closer'] ) ) {
 			return;
 		}
 
-		$ifOpener = $tokens[ $nextStatement ]['scope_opener'];
-		$ifCloser = $tokens[ $nextStatement ]['scope_closer'];
+		$ifOpener = $tokens[$nextStatement]['scope_opener'];
+		$ifCloser = $tokens[$nextStatement]['scope_closer'];
 
 		// Check if the if body contains only a return statement.
 		$ifFirstStatement = $phpcsFile->findNext( T_WHITESPACE, $ifOpener + 1, $ifCloser, true );
 
-		if ( false === $ifFirstStatement || $tokens[ $ifFirstStatement ]['code'] !== T_RETURN ) {
+		if ( false === $ifFirstStatement || $tokens[$ifFirstStatement]['code'] !== T_RETURN ) {
 			return;
 		}
 
@@ -138,7 +138,7 @@ 
 		// Check if the if has an else/elseif - skip those.
 		$afterIfClose = $phpcsFile->findNext( T_WHITESPACE, $ifCloser + 1, $functionCloser, true );
 
-		if ( false !== $afterIfClose && in_array( $tokens[ $afterIfClose ]['code'], array( T_ELSE, T_ELSEIF ), true ) ) {
+		if ( false !== $afterIfClose && in_array( $tokens[$afterIfClose]['code'], array( T_ELSE, T_ELSEIF ), true ) ) {
 			return;
 		}
 
@@ -194,15 +194,15 @@ 
 		// Get the variable declaration content.
 		$varDeclaration = '';
 
-		for ( $i = $varStart; $i <= $assignmentEnd; $i++ ) {
-			$varDeclaration .= $tokens[ $i ]['content'];
+		for ( $i = $varStart; $i <= $assignmentEnd; $i ++ ) {
+			$varDeclaration .= $tokens[$i]['content'];
 		}
 
 		// Find the start of the line for the variable (to get indentation).
 		$lineStart = $varStart;
 
-		for ( $i = $varStart - 1; $i >= 0; $i-- ) {
-			if ( $tokens[ $i ]['line'] < $tokens[ $varStart ]['line'] ) {
+		for ( $i = $varStart - 1; $i >= 0; $i -- ) {
+			if ( $tokens[$i]['line'] < $tokens[$varStart]['line'] ) {
 				break;
 			}
 			$lineStart = $i;
@@ -210,18 +210,18 @@ 
 
 		// Get the indentation based on the variable's column position.
 		// Column is 1-indexed, so column 2 means 1 character of indentation.
-		$column   = $tokens[ $varStart ]['column'];
+		$column   = $tokens[$varStart]['column'];
 		$tabCount = (int) floor( ( $column - 1 ) / 4 );
 		$indent   = str_repeat( "\t", $tabCount );
 
 		// Remove the variable declaration line (from line start to semicolon).
-		for ( $i = $lineStart; $i <= $assignmentEnd; $i++ ) {
+		for ( $i = $lineStart; $i <= $assignmentEnd; $i ++ ) {
 			$phpcsFile->fixer->replaceToken( $i, '' );
 		}
 
 		// Also remove the newline after the semicolon if it exists.
-		if ( isset( $tokens[ $assignmentEnd + 1 ] ) && $tokens[ $assignmentEnd + 1 ]['code'] === T_WHITESPACE ) {
-			$wsContent = $tokens[ $assignmentEnd + 1 ]['content'];
+		if ( isset( $tokens[$assignmentEnd + 1] ) && $tokens[$assignmentEnd + 1]['code'] === T_WHITESPACE ) {
+			$wsContent = $tokens[$assignmentEnd + 1]['content'];
 
 			// Remove one newline from the whitespace.
 			if ( strpos( $wsContent, "\n" ) === 0 ) {
@@ -233,8 +233,8 @@ 
 		// Find the whitespace after the if closer.
 		$afterIfCloser = $ifCloser + 1;
 
-		if ( isset( $tokens[ $afterIfCloser ] ) && $tokens[ $afterIfCloser ]['code'] === T_WHITESPACE ) {
-			$existingWs = $tokens[ $afterIfCloser ]['content'];
+		if ( isset( $tokens[$afterIfCloser] ) && $tokens[$afterIfCloser]['code'] === T_WHITESPACE ) {
+			$existingWs = $tokens[$afterIfCloser]['content'];
 			// Add the variable declaration with proper formatting.
 			// Keep the existing whitespace structure but insert the variable declaration.
 			$newContent = "\n\n" . $indent . $varDeclaration . $existingWs;
@@ -259,8 +259,8 @@ 
 	 * @return bool
 	 */
 	private function isVariableUsedInRange( File $phpcsFile, array $tokens, $start, $end, $variableName ) {
-		for ( $i = $start; $i <= $end; $i++ ) {
-			if ( $tokens[ $i ]['code'] === T_VARIABLE && $tokens[ $i ]['content'] === $variableName ) {
+		for ( $i = $start; $i <= $end; $i ++ ) {
+			if ( $tokens[$i]['code'] === T_VARIABLE && $tokens[$i]['content'] === $variableName ) {
 				return true;
 			}
 		}
@@ -282,15 +282,15 @@ 
 		// Find the opening parenthesis of the function.
 		$openParen = $phpcsFile->findNext( T_OPEN_PARENTHESIS, $functionPtr + 1 );
 
-		if ( false === $openParen || ! isset( $tokens[ $openParen ]['parenthesis_closer'] ) ) {
+		if ( false === $openParen || ! isset( $tokens[$openParen]['parenthesis_closer'] ) ) {
 			return false;
 		}
 
-		$closeParen = $tokens[ $openParen ]['parenthesis_closer'];
+		$closeParen = $tokens[$openParen]['parenthesis_closer'];
 
 		// Look for the variable in the parameter list.
-		for ( $i = $openParen + 1; $i < $closeParen; $i++ ) {
-			if ( $tokens[ $i ]['code'] === T_VARIABLE && $tokens[ $i ]['content'] === $variableName ) {
+		for ( $i = $openParen + 1; $i < $closeParen; $i ++ ) {
+			if ( $tokens[$i]['code'] === T_VARIABLE && $tokens[$i]['content'] === $variableName ) {
 				return true;
 			}
 		}