Strategy11 / formidable-forms

classes/models/fields/FrmFieldNumber.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
 
 		//validate the number format
 		if ( ! is_numeric( $args['value'] ) && '' !== $args['value'] ) {
-			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $this->field, 'invalid' );
+			$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $this->field, 'invalid' );
 		}
 
 		// validate number settings
@@ -55,9 +55,9 @@ 
 			if ( $frm_settings->use_html && $maxnum !== '' && $minnum !== '' ) {
 				$value = (float) $args['value'];
 				if ( $value < $minnum ) {
-					$errors[ 'field' . $args['id'] ] = __( 'Please select a higher number', 'formidable' );
+					$errors['field' . $args['id']] = __( 'Please select a higher number', 'formidable' );
 				} elseif ( $value > $maxnum ) {
-					$errors[ 'field' . $args['id'] ] = __( 'Please select a lower number', 'formidable' );
+					$errors['field' . $args['id']] = __( 'Please select a lower number', 'formidable' );
 				}
 			}
 		}

classes/models/FrmMigrate.php

Spacing +2 added lines, -2 removed lines

@@ -326,7 +326,7 @@ 
 				continue;
 			}
 
-			$this->maybe_convert_migrated_size( $widgets[ $k ]['size'] );
+			$this->maybe_convert_migrated_size( $widgets[$k]['size'] );
 		}
 		update_option( 'widget_frm_show_form', $widgets );
 	}
@@ -422,7 +422,7 @@ 
 			if ( ! is_array( $widget ) || ! isset( $widget['size'] ) ) {
 				continue;
 			}
-			$this->convert_character_to_px( $widgets[ $k ]['size'] );
+			$this->convert_character_to_px( $widgets[$k]['size'] );
 		}
 		update_option( 'widget_frm_show_form', $widgets );
 	}

classes/models/FrmInstallerSkin.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 */
 	public function set_upgrader( &$upgrader ) {
 		if ( is_object( $upgrader ) ) {
-			$this->upgrader =& $upgrader;
+			$this->upgrader = & $upgrader;
 		}
 	}
 

deprecated/FrmDeprecated.php

Doc Comments +5 added lines

@@ -10,6 +10,8 @@ 
 
 	/**
 	 * @deprecated 2.3
+	 * @param string $function
+	 * @param string $version
 	 */
 	public static function deprecated( $function, $version ) {
 		_deprecated_function( $function, $version );
@@ -68,6 +70,7 @@ 
 
 	/**
 	 * @deprecated 3.04.03
+	 * @param string $url
 	 */
 	private static function send_api_request( $url, $transient = array() ) {
 		$data = get_transient( $transient['name'] );
@@ -318,6 +321,7 @@ 
 
 	/**
 	 * @deprecated 3.0
+	 * @param string $field
 	 */
 	private static function edit_in_place_value( $field ) {
 		_deprecated_function( __FUNCTION__, '3.0' );
@@ -714,6 +718,7 @@ 
 
 	/**
 	 * @deprecated 3.02.03
+	 * @return string
 	 */
 	public static function get_form_for_page() {
 		_deprecated_function( __FUNCTION__, '3.02.03' );

Indentation +48 added lines, -48 removed lines

@@ -624,8 +624,8 @@ 
 	 */
 	public static function get_shortcode_tag( $shortcodes, $short_key, $args ) {
 		_deprecated_function( __FUNCTION__, '3.0', 'FrmShortcodeHelper::get_shortcode_tag' );
-        return FrmShortcodeHelper::get_shortcode_tag( $shortcodes, $short_key, $args );
-    }
+		return FrmShortcodeHelper::get_shortcode_tag( $shortcodes, $short_key, $args );
+	}
 
 	/**
 	 * @deprecated 3.01
@@ -638,52 +638,52 @@ 
 	/**
 	 * @deprecated 3.02.03
 	 */
-    public static function jquery_themes() {
+	public static function jquery_themes() {
 		_deprecated_function( __FUNCTION__, '3.02.03', 'FrmProStylesController::jquery_themes' );
 
-        $themes = array(
-            'ui-lightness'  => 'UI Lightness',
-            'ui-darkness'   => 'UI Darkness',
-            'smoothness'    => 'Smoothness',
-            'start'         => 'Start',
-            'redmond'       => 'Redmond',
-            'sunny'         => 'Sunny',
-            'overcast'      => 'Overcast',
-            'le-frog'       => 'Le Frog',
-            'flick'         => 'Flick',
+		$themes = array(
+			'ui-lightness'  => 'UI Lightness',
+			'ui-darkness'   => 'UI Darkness',
+			'smoothness'    => 'Smoothness',
+			'start'         => 'Start',
+			'redmond'       => 'Redmond',
+			'sunny'         => 'Sunny',
+			'overcast'      => 'Overcast',
+			'le-frog'       => 'Le Frog',
+			'flick'         => 'Flick',
 			'pepper-grinder' => 'Pepper Grinder',
-            'eggplant'      => 'Eggplant',
-            'dark-hive'     => 'Dark Hive',
-            'cupertino'     => 'Cupertino',
-            'south-street'  => 'South Street',
-            'blitzer'       => 'Blitzer',
-            'humanity'      => 'Humanity',
-            'hot-sneaks'    => 'Hot Sneaks',
-            'excite-bike'   => 'Excite Bike',
-            'vader'         => 'Vader',
-            'dot-luv'       => 'Dot Luv',
-            'mint-choc'     => 'Mint Choc',
-            'black-tie'     => 'Black Tie',
-            'trontastic'    => 'Trontastic',
-            'swanky-purse'  => 'Swanky Purse',
-        );
+			'eggplant'      => 'Eggplant',
+			'dark-hive'     => 'Dark Hive',
+			'cupertino'     => 'Cupertino',
+			'south-street'  => 'South Street',
+			'blitzer'       => 'Blitzer',
+			'humanity'      => 'Humanity',
+			'hot-sneaks'    => 'Hot Sneaks',
+			'excite-bike'   => 'Excite Bike',
+			'vader'         => 'Vader',
+			'dot-luv'       => 'Dot Luv',
+			'mint-choc'     => 'Mint Choc',
+			'black-tie'     => 'Black Tie',
+			'trontastic'    => 'Trontastic',
+			'swanky-purse'  => 'Swanky Purse',
+		);
 
 		$themes = apply_filters( 'frm_jquery_themes', $themes );
-        return $themes;
-    }
+		return $themes;
+	}
 
 	/**
 	 * @deprecated 3.02.03
 	 */
-    public static function enqueue_jquery_css() {
+	public static function enqueue_jquery_css() {
 		_deprecated_function( __FUNCTION__, '3.02.03', 'FrmProStylesController::enqueue_jquery_css' );
 
 		$form = self::get_form_for_page();
 		$theme_css = FrmStylesController::get_style_val( 'theme_css', $form );
-        if ( $theme_css != -1 ) {
+		if ( $theme_css != -1 ) {
 			wp_enqueue_style( 'jquery-theme', self::jquery_css_url( $theme_css ), array(), FrmAppHelper::plugin_version() );
-        }
-    }
+		}
+	}
 
 	/**
 	 * @deprecated 3.02.03
@@ -691,26 +691,26 @@ 
 	public static function jquery_css_url( $theme_css ) {
 		_deprecated_function( __FUNCTION__, '3.02.03', 'FrmProStylesController::jquery_css_url' );
 
-        if ( $theme_css == -1 ) {
-            return;
-        }
+		if ( $theme_css == -1 ) {
+			return;
+		}
 
-        if ( ! $theme_css || $theme_css == '' || $theme_css == 'ui-lightness' ) {
-            $css_file = FrmAppHelper::plugin_url() . '/css/ui-lightness/jquery-ui.css';
+		if ( ! $theme_css || $theme_css == '' || $theme_css == 'ui-lightness' ) {
+			$css_file = FrmAppHelper::plugin_url() . '/css/ui-lightness/jquery-ui.css';
 		} elseif ( preg_match( '/^http.?:\/\/.*\..*$/', $theme_css ) ) {
-            $css_file = $theme_css;
-        } else {
-            $uploads = FrmStylesHelper::get_upload_base();
+			$css_file = $theme_css;
+		} else {
+			$uploads = FrmStylesHelper::get_upload_base();
 			$file_path = '/formidable/css/' . $theme_css . '/jquery-ui.css';
 			if ( file_exists( $uploads['basedir'] . $file_path ) ) {
-                $css_file = $uploads['baseurl'] . $file_path;
-            } else {
+				$css_file = $uploads['baseurl'] . $file_path;
+			} else {
 				$css_file = FrmAppHelper::jquery_ui_base_url() . '/themes/' . $theme_css . '/jquery-ui.min.css';
-            }
-        }
+			}
+		}
 
-        return $css_file;
-    }
+		return $css_file;
+	}
 
 	/**
 	 * @deprecated 3.02.03

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
 		FrmAppHelper::permission_check( 'frm_edit_forms' );
 
 		$action = isset( $_REQUEST['frm_action'] ) ? 'frm_action' : 'action';
-		$action = empty( $values ) ? FrmAppHelper::get_param( $action, '', 'get', 'sanitize_title' ) : $values[ $action ];
+		$action = empty( $values ) ? FrmAppHelper::get_param( $action, '', 'get', 'sanitize_title' ) : $values[$action];
 
 		if ( $action === 'create' ) {
 			self::create( $values );
@@ -458,8 +458,8 @@ 
 		$path = untrailingslashit( trim( $path ) );
 		$templates = glob( $path . '/*.php' );
 
-		for ( $i = count( $templates ) - 1; $i >= 0; $i-- ) {
-			$filename = str_replace( '.php', '', str_replace( $path . '/', '', $templates[ $i ] ) );
+		for ( $i = count( $templates ) - 1; $i >= 0; $i -- ) {
+			$filename = str_replace( '.php', '', str_replace( $path . '/', '', $templates[$i] ) );
 			$template_query = array( 'form_key' => $filename );
 			if ( $template ) {
 				$template_query['is_template'] = 1;
@@ -474,7 +474,7 @@ 
 			$values['is_template'] = $template;
 			$values['status'] = 'published';
 
-			include( $templates[ $i ] );
+			include( $templates[$i] );
 
 			//get updated form
 			if ( isset( $form ) && ! empty( $form ) ) {

deprecated/FrmEDD_SL_Plugin_Updater.php

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
 	 * @uses api_request()
 	 *
 	 * @param array   $_transient_data Update array build by WordPress.
-	 * @return array Modified update array with custom plugin data.
+	 * @return stdClass Modified update array with custom plugin data.
 	 */
 	public function check_update( $_transient_data ) {
 
@@ -219,7 +219,7 @@ 
 	 *
 	 * @param string  $_action The requested action.
 	 * @param array   $_data   Parameters for the API action.
-	 * @return false|object
+	 * @return string
 	 */
 	private function api_request( $_action, $_data ) {
 

Braces +4 added lines, -2 removed lines

@@ -297,11 +297,13 @@
 
 		global $frm_edd_plugin_data;
 
-		if ( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' !== $_REQUEST['edd_sl_action'] ) { // WPCS: CSRF ok.
+		if ( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' !== $_REQUEST['edd_sl_action'] ) {
+// WPCS: CSRF ok.
 			return;
 		}
 
-		if ( empty( $_REQUEST['plugin'] ) || empty( $_REQUEST['slug'] ) ) { // WPCS: CSRF ok.
+		if ( empty( $_REQUEST['plugin'] ) || empty( $_REQUEST['slug'] ) ) {
+// WPCS: CSRF ok.
 			return;
 		}
 

Spacing +7 added lines, -7 removed lines

@@ -44,7 +44,7 @@ 
 		$this->beta        = ! empty( $this->api_data['beta'] ) ? true : false;
 		$this->cache_key   = md5( serialize( $this->slug . $this->version . $this->api_data['license'] . $this->beta ) );
 
-		$frm_edd_plugin_data[ $this->slug ] = $this->api_data;
+		$frm_edd_plugin_data[$this->slug] = $this->api_data;
 
 		/**
 		 * Fires after the $frm_edd_plugin_data is setup.
@@ -93,7 +93,7 @@ 
 			$_transient_data = new stdClass();
 		}
 
-		if ( ! empty( $_transient_data->response ) && ! empty( $_transient_data->response[ $this->name ] ) && false === $this->wp_override ) {
+		if ( ! empty( $_transient_data->response ) && ! empty( $_transient_data->response[$this->name] ) && false === $this->wp_override ) {
 			return $_transient_data;
 		}
 
@@ -120,12 +120,12 @@ 
 					$version_info->plugin = $this->name;
 				}
 
-				$_transient_data->response[ $this->name ] = $version_info;
+				$_transient_data->response[$this->name] = $version_info;
 
 			}
 
 			$_transient_data->last_checked           = time();
-			$_transient_data->checked[ $this->name ] = $this->version;
+			$_transient_data->checked[$this->name] = $this->version;
 
 		}
 
@@ -189,7 +189,7 @@ 
 		if ( isset( $_data->sections ) && ! is_array( $_data->sections ) ) {
 			$new_sections = array();
 			foreach ( $_data->sections as $key => $value ) {
-				$new_sections[ $key ] = $value;
+				$new_sections[$key] = $value;
 			}
 
 			$_data->sections = $new_sections;
@@ -199,7 +199,7 @@ 
 		if ( isset( $_data->banners ) && ! is_array( $_data->banners ) ) {
 			$new_banners = array();
 			foreach ( $_data->banners as $key => $value ) {
-				$new_banners[ $key ] = $value;
+				$new_banners[$key] = $value;
 			}
 
 			$_data->banners = $new_banners;
@@ -316,7 +316,7 @@ 
 		}
 
 		$slug         = sanitize_text_field( $_REQUEST['slug'] ); // WPCS: CSRF ok.
-		$data         = $frm_edd_plugin_data[ $slug ];
+		$data         = $frm_edd_plugin_data[$slug];
 		$beta         = ! empty( $data['beta'] ) ? true : false;
 		$cache_key    = md5( 'edd_plugin_' . sanitize_key( $_REQUEST['plugin'] ) . '_' . $beta . '_version_info' ); // WPCS: CSRF ok.
 		$version_info = $this->get_cached_version_info( $cache_key );

deprecated/FrmEntryFormat.php

Doc Comments +1 added lines

@@ -151,6 +151,7 @@
 
 	/**
 	 * @deprecated 2.04
+	 * @param string $val
 	 */
 	private static function get_field_value( $atts, &$val ) {
 		_deprecated_function( __FUNCTION__, '2.04', 'instance of FrmEntryValues or FrmProEntryValues' );

Spacing +8 added lines, -8 removed lines

@@ -98,15 +98,15 @@ 
 		self::prepare_field_output( $atts, $val );
 
 		if ( $atts['format'] != 'text' ) {
-			$values[ $f->field_key ] = $val;
+			$values[$f->field_key] = $val;
 			if ( $atts['entry'] && $f->type != 'textarea' ) {
-				$prev_val = maybe_unserialize( $atts['entry']->metas[ $f->id ] );
+				$prev_val = maybe_unserialize( $atts['entry']->metas[$f->id] );
 				if ( $prev_val != $val ) {
-					$values[ $f->field_key . '-value' ] = $prev_val;
+					$values[$f->field_key . '-value'] = $prev_val;
 				}
 			}
 		} else {
-			$values[ $f->id ] = array(
+			$values[$f->id] = array(
 				'label' => $f->name,
 				'val'   => $val,
 				'type'  => $f->type,
@@ -120,9 +120,9 @@ 
 	private static function fill_missing_fields( $atts, &$values ) {
 		_deprecated_function( __FUNCTION__, '2.04', 'instance of FrmEntryValues or FrmProEntryValues' );
 
-		if ( $atts['entry'] && ! isset( $atts['entry']->metas[ $atts['field']->id ] ) ) {
+		if ( $atts['entry'] && ! isset( $atts['entry']->metas[$atts['field']->id] ) ) {
 			// In case include_blank is set
-			$atts['entry']->metas[ $atts['field']->id ] = '';
+			$atts['entry']->metas[$atts['field']->id] = '';
 			$atts['entry'] = apply_filters( 'frm_prepare_entry_content', $atts['entry'], array( 'field' => $atts['field'] ) );
 			self::fill_values_from_entry( $atts, $values );
 		}
@@ -150,7 +150,7 @@ 
 			'type'  => $f->type,
 		);
 
-		$values[ $f->id ] = apply_filters( 'frm_field_shortcodes_for_default_html_email', $field_shortcodes, $f );
+		$values[$f->id] = apply_filters( 'frm_field_shortcodes_for_default_html_email', $field_shortcodes, $f );
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 
 		$f = $atts['field'];
 		if ( $atts['entry'] ) {
-			$prev_val = maybe_unserialize( $atts['entry']->metas[ $f->id ] );
+			$prev_val = maybe_unserialize( $atts['entry']->metas[$f->id] );
 			$meta = array(
 				'item_id'    => $atts['id'],
 				'field_id'   => $f->id,

deprecated/FrmPointers.php

Doc Comments +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	/**
 	 * Get the singleton instance of this class
 	 *
-	 * @return object
+	 * @return FrmPointers
 	 */
 	public static function get_instance() {
 		_deprecated_function( __FUNCTION__, '3.01.03' );

classes/models/FrmAddon.php

Braces +4 added lines, -2 removed lines

@@ -306,7 +306,8 @@ 
 
 		if ( isset( $version_info->new_version ) && ! empty( $version_info->new_version ) ) {
 			$this->clear_old_plugin_version( $version_info );
-			if ( $version_info === false ) { // was cleared with timeout
+			if ( $version_info === false ) {
+// was cleared with timeout
 				$transient = false;
 			} else {
 				$this->maybe_use_beta_url( $version_info );
@@ -393,7 +394,8 @@ 
 	}
 
 	private function is_license_revoked() {
-		if ( empty( $this->license ) || empty( $this->plugin_slug ) || isset( $_POST['license'] ) ) { // WPCS: CSRF ok.
+		if ( empty( $this->license ) || empty( $this->plugin_slug ) || isset( $_POST['license'] ) ) {
+// WPCS: CSRF ok.
 			return;
 		}
 

Doc Comments +7 added lines

@@ -234,6 +234,9 @@ 
 		$this->delete_cache();
 	}
 
+	/**
+	 * @param string|false $is_active
+	 */
 	public function set_active( $is_active ) {
 		update_option( $this->option_name . 'active', $is_active );
 		$this->delete_cache();
@@ -352,6 +355,7 @@ 
 	 * Get the API info for this plugin
 	 *
 	 * @since 3.04.03
+	 * @param string $license
 	 */
 	protected function get_api_info( $license ) {
 		$api   = new FrmFormApi( $license );
@@ -582,6 +586,9 @@ 
 		wp_die();
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function send_mothership_request( $action ) {
 		$api_params = array(
 			'edd_action' => $action,

Spacing +14 added lines, -14 removed lines

@@ -41,7 +41,7 @@ 
 	}
 
 	public function insert_installed_addon( $plugins ) {
-		$plugins[ $this->plugin_slug ] = $this;
+		$plugins[$this->plugin_slug] = $this;
 
 		return $plugins;
 	}
@@ -49,8 +49,8 @@ 
 	public static function get_addon( $plugin_slug ) {
 		$plugins = apply_filters( 'frm_installed_addons', array() );
 		$plugin  = false;
-		if ( isset( $plugins[ $plugin_slug ] ) ) {
-			$plugin = $plugins[ $plugin_slug ];
+		if ( isset( $plugins[$plugin_slug] ) ) {
+			$plugin = $plugins[$plugin_slug];
 		}
 
 		return $plugin;
@@ -109,7 +109,7 @@ 
 		} else {
 			$api     = new FrmFormApi( $this->license );
 			$plugins = $api->get_api_info();
-			$_data   = $plugins[ $item_id ];
+			$_data   = $plugins[$item_id];
 		}
 
 		$_data['sections'] = array(
@@ -301,19 +301,19 @@ 
 
 		if ( $this->is_current_version( $transient ) ) {
 			//make sure it doesn't show there is an update if plugin is up-to-date
-			if ( isset( $transient->response[ $this->plugin_folder ] ) ) {
-				unset( $transient->response[ $this->plugin_folder ] );
+			if ( isset( $transient->response[$this->plugin_folder] ) ) {
+				unset( $transient->response[$this->plugin_folder] );
 			}
-		} elseif ( isset( $transient->response ) && isset( $transient->response[ $this->plugin_folder ] ) ) {
-			$this->prepare_update_details( $transient->response[ $this->plugin_folder ] );
+		} elseif ( isset( $transient->response ) && isset( $transient->response[$this->plugin_folder] ) ) {
+			$this->prepare_update_details( $transient->response[$this->plugin_folder] );
 
 			// if the transient has expired, clear the update and trigger it again
-			if ( $transient->response[ $this->plugin_folder ] === false ) {
+			if ( $transient->response[$this->plugin_folder] === false ) {
 				if ( ! $this->has_been_cleared() ) {
 					$this->cleared_plugins();
 					$this->manually_queue_update();
 				}
-				unset( $transient->response[ $this->plugin_folder ] );
+				unset( $transient->response[$this->plugin_folder] );
 			}
 		}
 
@@ -401,7 +401,7 @@ 
 	}
 
 	private function is_current_version( $transient ) {
-		if ( empty( $transient->checked ) || ! isset( $transient->checked[ $this->plugin_folder ] ) ) {
+		if ( empty( $transient->checked ) || ! isset( $transient->checked[$this->plugin_folder] ) ) {
 			return false;
 		}
 
@@ -410,7 +410,7 @@ 
 			return true;
 		}
 
-		return isset( $transient->response ) && isset( $transient->response[ $this->plugin_folder ] ) && $transient->checked[ $this->plugin_folder ] === $transient->response[ $this->plugin_folder ]->new_version;
+		return isset( $transient->response ) && isset( $transient->response[$this->plugin_folder] ) && $transient->checked[$this->plugin_folder] === $transient->response[$this->plugin_folder]->new_version;
 	}
 
 	private function has_been_cleared() {
@@ -491,8 +491,8 @@ 
 			$response['message'] = $response['status'];
 		} else {
 			$messages = $this->get_messages();
-			if ( is_string( $response['status'] ) && isset( $messages[ $response['status'] ] ) ) {
-				$response['message'] = $messages[ $response['status'] ];
+			if ( is_string( $response['status'] ) && isset( $messages[$response['status']] ) ) {
+				$response['message'] = $messages[$response['status']];
 			} else {
 				$response['message'] = FrmAppHelper::kses( $response['status'], array( 'a' ) );
 			}

classes/controllers/FrmAppController.php

Spacing +1 added lines, -1 removed lines

@@ -235,7 +235,7 @@
 		if ( is_array( $upsell ) ) {
 			foreach ( $upsell as $k => $plugin ) {
 				if ( strpos( $plugin['slug'], 'wpforms' ) !== false ) {
-					unset( $upsell[ $k ] );
+					unset( $upsell[$k] );
 				}
 			}
 		}