Strategy11 / formidable-forms

phpcs-sniffs/Formidable/Sniffs/WhiteSpace/NoBlankLineInShortIfSniff.php

Spacing +11 added lines, -11 removed lines

@@ -51,16 +51,16 @@ 
 		$tokens = $phpcsFile->getTokens();
 
 		// Make sure this if/else has a scope (curly braces).
-		if ( ! isset( $tokens[ $stackPtr ]['scope_opener'] ) || ! isset( $tokens[ $stackPtr ]['scope_closer'] ) ) {
+		if ( ! isset( $tokens[$stackPtr]['scope_opener'] ) || ! isset( $tokens[$stackPtr]['scope_closer'] ) ) {
 			return;
 		}
 
-		$scopeOpener = $tokens[ $stackPtr ]['scope_opener'];
-		$scopeCloser = $tokens[ $stackPtr ]['scope_closer'];
+		$scopeOpener = $tokens[$stackPtr]['scope_opener'];
+		$scopeCloser = $tokens[$stackPtr]['scope_closer'];
 
 		// Calculate the number of lines in the block body.
-		$openerLine = $tokens[ $scopeOpener ]['line'];
-		$closerLine = $tokens[ $scopeCloser ]['line'];
+		$openerLine = $tokens[$scopeOpener]['line'];
+		$closerLine = $tokens[$scopeCloser]['line'];
 
 		// Body lines = closerLine - openerLine - 1.
 		// We want to trigger when there are exactly 3 body lines (2 code + 1 blank).
@@ -76,9 +76,9 @@ 
 		// A blank line is a whitespace token that contains 2+ consecutive newlines.
 		$blankLineToken = null;
 
-		for ( $i = $scopeOpener + 1; $i < $scopeCloser; $i++ ) {
-			if ( $tokens[ $i ]['code'] === T_WHITESPACE ) {
-				$content      = $tokens[ $i ]['content'];
+		for ( $i = $scopeOpener + 1; $i < $scopeCloser; $i ++ ) {
+			if ( $tokens[$i]['code'] === T_WHITESPACE ) {
+				$content      = $tokens[$i]['content'];
 				$newlineCount = substr_count( $content, "\n" );
 
 				// If this whitespace has 2+ newlines, it contains a blank line.
@@ -93,8 +93,8 @@ 
 			// Also check for consecutive newline tokens (PHPCS sometimes splits them).
 			$lastWasNewline = false;
 
-			for ( $i = $scopeOpener + 1; $i < $scopeCloser; $i++ ) {
-				if ( $tokens[ $i ]['code'] === T_WHITESPACE && $tokens[ $i ]['content'] === "\n" ) {
+			for ( $i = $scopeOpener + 1; $i < $scopeCloser; $i ++ ) {
+				if ( $tokens[$i]['code'] === T_WHITESPACE && $tokens[$i]['content'] === "\n" ) {
 					if ( $lastWasNewline ) {
 						$blankLineToken = $i;
 						break;
@@ -119,7 +119,7 @@ 
 		if ( true === $fix ) {
 			$phpcsFile->fixer->beginChangeset();
 
-			$content = $tokens[ $blankLineToken ]['content'];
+			$content = $tokens[$blankLineToken]['content'];
 
 			// Remove one newline from the whitespace.
 			if ( substr_count( $content, "\n" ) >= 2 ) {

classes/models/FrmStyle.php

Spacing +12 added lines, -12 removed lines

@@ -135,22 +135,22 @@ 
 			$default_settings = $this->get_defaults();
 
 			foreach ( $default_settings as $setting => $default ) {
-				if ( ! isset( $new_instance['post_content'][ $setting ] ) ) {
-					$new_instance['post_content'][ $setting ] = $default;
+				if ( ! isset( $new_instance['post_content'][$setting] ) ) {
+					$new_instance['post_content'][$setting] = $default;
 				}
 
 				if ( $this->is_color( $setting ) ) {
-					$color_val = $new_instance['post_content'][ $setting ];
+					$color_val = $new_instance['post_content'][$setting];
 
 					if ( $color_val !== '' && str_contains( $color_val, 'rgb' ) ) {
 						// Maybe sanitize if invalid rgba value is entered.
 						$this->maybe_sanitize_rgba_value( $color_val );
 					}
-					$new_instance['post_content'][ $setting ] = str_replace( '#', '', $color_val );
-				} elseif ( in_array( $setting, array( 'submit_style', 'important_style', 'auto_width' ), true ) && ! isset( $new_instance['post_content'][ $setting ] ) ) {
-					$new_instance['post_content'][ $setting ] = 0;
+					$new_instance['post_content'][$setting] = str_replace( '#', '', $color_val );
+				} elseif ( in_array( $setting, array( 'submit_style', 'important_style', 'auto_width' ), true ) && ! isset( $new_instance['post_content'][$setting] ) ) {
+					$new_instance['post_content'][$setting] = 0;
 				} elseif ( $setting === 'font' ) {
-					$new_instance['post_content'][ $setting ] = $this->force_balanced_quotation( $new_instance['post_content'][ $setting ] );
+					$new_instance['post_content'][$setting] = $this->force_balanced_quotation( $new_instance['post_content'][$setting] );
 				}
 			}
 
@@ -264,10 +264,10 @@ 
 		$sanitized_settings = array();
 
 		foreach ( $valid_keys as $key ) {
-			$sanitized_settings[ $key ] = isset( $settings[ $key ] ) ? sanitize_textarea_field( $settings[ $key ] ) : $defaults[ $key ];
+			$sanitized_settings[$key] = isset( $settings[$key] ) ? sanitize_textarea_field( $settings[$key] ) : $defaults[$key];
 
 			if ( 'custom_css' !== $key && 'single_style_custom_css' !== $key ) {
-				$sanitized_settings[ $key ] = $this->strip_invalid_characters( $sanitized_settings[ $key ] );
+				$sanitized_settings[$key] = $this->strip_invalid_characters( $sanitized_settings[$key] );
 			}
 		}
 
@@ -562,12 +562,12 @@ 
 			$style->post_content = $this->override_defaults( $style->post_content );
 			$style->post_content = wp_parse_args( $style->post_content, $default_values );
 
-			$styles[ $style->ID ] = $style;
+			$styles[$style->ID] = $style;
 		}//end foreach
 
 		if ( ! $default_style ) {
 			$default_style                            = reset( $styles );
-			$styles[ $default_style->ID ]->menu_order = 1;
+			$styles[$default_style->ID]->menu_order = 1;
 		}
 
 		return $styles;
@@ -824,7 +824,7 @@ 
 				continue;
 			}
 
-			if ( $value && $char === $value[ strlen( $value ) - 1 ] ) {
+			if ( $value && $char === $value[strlen( $value ) - 1] ) {
 				$value = $char . $value;
 			} else {
 				$value .= $char;

classes/helpers/FrmFormsHelper.php

Spacing +23 added lines, -23 removed lines

@@ -50,7 +50,7 @@ 
 			'class'        => '',
 			'inc_children' => 'exclude',
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args( $args, $defaults );
 
 		if ( ! $args['field_id'] ) {
 			$args['field_id'] = $field_name;
@@ -95,7 +95,7 @@ 
 	 */
 	public static function add_html_attr( $class, $param, &$add_html ) {
 		if ( $class ) {
-			$add_html[ $param ] = sanitize_title( $param ) . '="' . esc_attr( trim( sanitize_text_field( $class ) ) ) . '"';
+			$add_html[$param] = sanitize_title( $param ) . '="' . esc_attr( trim( sanitize_text_field( $class ) ) ) . '"';
 		}
 	}
 
@@ -350,8 +350,8 @@ 
 		);
 
 		foreach ( $defaults as $var => $default ) {
-			if ( ! isset( $values[ $var ] ) ) {
-				$values[ $var ] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' );
+			if ( ! isset( $values[$var] ) ) {
+				$values[$var] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' );
 			}
 		}
 
@@ -367,8 +367,8 @@ 
 		);
 
 		foreach ( $defaults as $var => $default ) {
-			if ( ! isset( $values[ $var ] ) ) {
-				$values[ $var ] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' );
+			if ( ! isset( $values[$var] ) ) {
+				$values[$var] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' );
 			}
 		}
 		unset( $defaults );
@@ -419,16 +419,16 @@ 
 
 		foreach ( $defaults as $var => $default ) {
 			if ( is_array( $default ) ) {
-				if ( ! isset( $values[ $var ] ) ) {
-					$values[ $var ] = $record && isset( $record->options[ $var ] ) ? $record->options[ $var ] : array();
+				if ( ! isset( $values[$var] ) ) {
+					$values[$var] = $record && isset( $record->options[$var] ) ? $record->options[$var] : array();
 				}
 
 				foreach ( $default as $k => $v ) {
-					$values[ $var ][ $k ] = $post_values && isset( $post_values[ $var ][ $k ] ) ? $post_values[ $var ][ $k ] : ( $record && isset( $record->options[ $var ] ) && isset( $record->options[ $var ][ $k ] ) ? $record->options[ $var ][ $k ] : $v );
+					$values[$var][$k] = $post_values && isset( $post_values[$var][$k] ) ? $post_values[$var][$k] : ( $record && isset( $record->options[$var] ) && isset( $record->options[$var][$k] ) ? $record->options[$var][$k] : $v );
 
 					if ( is_array( $v ) ) {
 						foreach ( $v as $k1 => $v1 ) {
-							$values[ $var ][ $k ][ $k1 ] = $post_values && isset( $post_values[ $var ][ $k ][ $k1 ] ) ? $post_values[ $var ][ $k ][ $k1 ] : ( $record && isset( $record->options[ $var ] ) && isset( $record->options[ $var ][ $k ] ) && isset( $record->options[ $var ][ $k ][ $k1 ] ) ? $record->options[ $var ][ $k ][ $k1 ] : $v1 );
+							$values[$var][$k][$k1] = $post_values && isset( $post_values[$var][$k][$k1] ) ? $post_values[$var][$k][$k1] : ( $record && isset( $record->options[$var] ) && isset( $record->options[$var][$k] ) && isset( $record->options[$var][$k][$k1] ) ? $record->options[$var][$k][$k1] : $v1 );
 							unset( $k1, $v1 );
 						}
 					}
@@ -436,7 +436,7 @@ 
 					unset( $k, $v );
 				}
 			} else {
-				$values[ $var ] = $post_values && isset( $post_values['options'][ $var ] ) ? $post_values['options'][ $var ] : ( $record && isset( $record->options[ $var ] ) ? $record->options[ $var ] : $default );
+				$values[$var] = $post_values && isset( $post_values['options'][$var] ) ? $post_values['options'][$var] : ( $record && isset( $record->options[$var] ) ? $record->options[$var] : $default );
 			}
 
 			unset( $var, $default );
@@ -485,7 +485,7 @@ 
 		$defaults = self::get_default_opts();
 
 		foreach ( $defaults as $var => $default ) {
-			$options[ $var ] = $values['options'][ $var ] ?? $default;
+			$options[$var] = $values['options'][$var] ?? $default;
 			unset( $var, $default );
 		}
 	}
@@ -681,7 +681,7 @@ 
 	public static function insert_opt_html( $args ) {
 		$class  = $args['class'] ?? '';
 		$fields = self::get_field_type_data_for_insert_opt_html();
-		$field  = $fields[ $args['type'] ] ?? array();
+		$field  = $fields[$args['type']] ?? array();
 
 		self::prepare_field_type( $field );
 
@@ -822,7 +822,7 @@ 
 
 		foreach ( $fields as $field ) {
 			if ( $prev_order === $field->field_order ) {
-				++$add_order;
+				++ $add_order;
 			}
 
 			if ( $add_order ) {
@@ -1359,9 +1359,9 @@ 
 				$link .= ' onclick="return confirm(\'' . esc_attr( $link_details['confirm'] ) . '\')"';
 			}
 
-			$label = $link_details[ $length ] ?? $link_details['label'];
+			$label = $link_details[$length] ?? $link_details['label'];
 
-			if ( $length === 'icon' && isset( $link_details[ $length ] ) ) {
+			if ( $length === 'icon' && isset( $link_details[$length] ) ) {
 				$label = '<span class="' . $label . '" title="' . esc_attr( $link_details['label'] ) . '" aria-hidden="true"></span>';
 				$link .= ' aria-label="' . esc_attr( $link_details['label'] ) . '"';
 			}
@@ -1529,7 +1529,7 @@ 
 			$status = 'publish';
 		}
 
-		return $nice_names[ $status ];
+		return $nice_names[$status];
 	}
 
 	/**
@@ -1593,7 +1593,7 @@ 
 
 		if ( count( $categories ) === 1 ) {
 			$category = reset( $categories );
-			$icon     = $icons[ $category ] ?? $icon;
+			$icon     = $icons[$category] ?? $icon;
 		} elseif ( $categories ) {
 			$icons = array_intersect_key( $icons, array_flip( $categories ) );
 			$icon  = reset( $icons );
@@ -1725,7 +1725,7 @@ 
 
 		foreach ( $item['categories'] as $k => $category ) {
 			if ( in_array( $category, $plans, true ) ) {
-				unset( $item['categories'][ $k ] );
+				unset( $item['categories'][$k] );
 				return self::convert_legacy_package_names( $category );
 			}
 		}
@@ -1760,7 +1760,7 @@ 
 	 * @return array
 	 */
 	public static function get_license_types( $args = array() ) {
-		$defaults      = array(
+		$defaults = array(
 			'include_all' => true,
 			'case_lower'  => false,
 		);
@@ -1969,9 +1969,9 @@ 
 		}
 
 		foreach ( $shortcodes[0] as $key => $shortcode ) {
-			$options = trim( $shortcodes[3][ $key ] );
+			$options = trim( $shortcodes[3][$key] );
 
-			if ( in_array( $shortcodes[1][ $key ], array( 'if ' ), true ) ) {
+			if ( in_array( $shortcodes[1][$key], array( 'if ' ), true ) ) {
 				// Skip if shortcodes.
 				continue;
 			}
@@ -1981,7 +1981,7 @@ 
 				continue;
 			}
 
-			$new_shortcode = '[' . $shortcodes[2][ $key ];
+			$new_shortcode = '[' . $shortcodes[2][$key];
 
 			if ( $options ) {
 				$new_shortcode .= ' ' . $options;