Strategy11 / formidable-forms

classes/models/FrmCreateFile.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
 
 		$next_dir = '';
 		foreach ( $dir_names as $dir ) {
-			$next_dir      .= '/' . $dir;
+			$next_dir .= '/' . $dir;
 			$needed_dirs[] = $this->uploads['basedir'] . $next_dir;
 		}
 

classes/models/FrmTableHTMLGenerator.php

Spacing +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
 	 * @param array $atts
 	 */
 	private function init_style_settings( $atts ) {
-		$style_settings       = array(
+		$style_settings = array(
 			'border_color' => 'dddddd',
 			'bg_color'     => 'f7f7f7',
 			'text_color'   => '444444',
@@ -82,12 +82,12 @@ 
 		$this->style_settings = apply_filters( 'frm_show_entry_styles', $style_settings );
 
 		foreach ( $this->style_settings as $key => $setting ) {
-			if ( isset( $atts[ $key ] ) && $atts[ $key ] !== '' ) {
-				$this->style_settings[ $key ] = $atts[ $key ];
+			if ( isset( $atts[$key] ) && $atts[$key] !== '' ) {
+				$this->style_settings[$key] = $atts[$key];
 			}
 
 			if ( $this->is_color_setting( $key ) ) {
-				$this->style_settings[ $key ] = $this->get_color_markup( $this->style_settings[ $key ] );
+				$this->style_settings[$key] = $this->get_color_markup( $this->style_settings[$key] );
 			}
 		}
 

classes/models/FrmEntryValidate.php

Spacing +6 added lines, -6 removed lines

@@ -11,7 +11,7 @@ 
 			return $errors;
 		}
 
-		if ( FrmAppHelper::is_admin() && is_user_logged_in() && ( ! isset( $values[ 'frm_submit_entry_' . $values['form_id'] ] ) || ! wp_verify_nonce( $values[ 'frm_submit_entry_' . $values['form_id'] ], 'frm_submit_entry_nonce' ) ) ) {
+		if ( FrmAppHelper::is_admin() && is_user_logged_in() && ( ! isset( $values['frm_submit_entry_' . $values['form_id']] ) || ! wp_verify_nonce( $values['frm_submit_entry_' . $values['form_id']], 'frm_submit_entry_nonce' ) ) ) {
 			$errors['form'] = __( 'You do not have permission to do that', 'formidable' );
 		}
 
@@ -65,10 +65,10 @@ 
 			'key_pointer'     => '', // the pointer in the posted array
 			'exclude'         => array(), // exclude these field types from validation
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args( $args, $defaults );
 
 		if ( empty( $args['parent_field_id'] ) ) {
-			$value = isset( $values['item_meta'][ $args['id'] ] ) ? $values['item_meta'][ $args['id'] ] : '';
+			$value = isset( $values['item_meta'][$args['id']] ) ? $values['item_meta'][$args['id']] : '';
 		} else {
 			// value is from a nested form
 			$value = $values;
@@ -89,7 +89,7 @@ 
 		}
 
 		if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) {
-			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
+			$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
 		} elseif ( $posted_field->type == 'text' && ! isset( $_POST['item_name'] ) ) { // WPCS: CSRF ok.
 			$_POST['item_name'] = $value;
 		}
@@ -142,7 +142,7 @@ 
 			$pattern = self::phone_format( $field );
 
 			if ( ! preg_match( $pattern, $value ) ) {
-				$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $field, 'invalid' );
+				$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $field, 'invalid' );
 			}
 		}
 	}
@@ -378,7 +378,7 @@ 
 
 			// Send any potentially useful $_SERVER vars, but avoid sending junk we don't need.
 			if ( $include_value ) {
-				$datas[ $key ] = $value;
+				$datas[$key] = $value;
 			}
 			unset( $key, $value );
 		}

Braces +2 added lines, -1 removed lines

@@ -90,7 +90,8 @@
 
 		if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) {
 			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
-		} elseif ( $posted_field->type == 'text' && ! isset( $_POST['item_name'] ) ) { // WPCS: CSRF ok.
+		} elseif ( $posted_field->type == 'text' && ! isset( $_POST['item_name'] ) ) {
+// WPCS: CSRF ok.
 			$_POST['item_name'] = $value;
 		}
 

Doc Comments +8 added lines, -1 removed lines

@@ -47,6 +47,9 @@ 
 		}
 	}
 
+	/**
+	 * @param boolean $exclude
+	 */
 	private static function get_fields_to_validate( $values, $exclude ) {
 		$where = apply_filters( 'frm_posted_field_ids', array( 'fi.form_id' => $values['form_id'] ) );
 
@@ -243,7 +246,7 @@ 
 
 	/**
 	 * @param int $form_id
-	 * @return boolean
+	 * @return boolean|string
 	 */
 	private static function is_antispam_check( $form_id ) {
 		$aspm = new FrmAntiSpam( $form_id );
@@ -310,6 +313,9 @@ 
 	 * For WP 5.5 compatibility.
 	 *
 	 * @since 4.06.02
+	 * @param string $content
+	 * @param string $ip
+	 * @param string $user_agent
 	 */
 	private static function check_disallowed_words( $author, $email, $url, $content, $ip, $user_agent ) {
 		if ( function_exists( 'wp_check_comment_disallowed_list' ) ) {
@@ -323,6 +329,7 @@ 
 	 * For WP 5.5 compatibility.
 	 *
 	 * @since 4.06.02
+	 * @return string
 	 */
 	private static function get_disallowed_words() {
 		$keys = get_option( 'disallowed_keys' );

classes/models/FrmFieldValue.php

Spacing +2 added lines, -2 removed lines

@@ -59,8 +59,8 @@
 	protected function init_saved_value( $entry ) {
 		if ( $this->field->type === 'html' ) {
 			$this->saved_value = $this->field->description;
-		} elseif ( isset( $entry->metas[ $this->field->id ] ) ) {
-			$this->saved_value = $entry->metas[ $this->field->id ];
+		} elseif ( isset( $entry->metas[$this->field->id] ) ) {
+			$this->saved_value = $entry->metas[$this->field->id];
 		} else {
 			$this->saved_value = '';
 		}

classes/views/frm-forms/form.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 		if ( isset( $values['fields'] ) && ! empty( $values['fields'] ) ) {
 			$values['count'] = 0;
 			foreach ( $values['fields'] as $field ) {
-				$values['count']++;
+				$values['count'] ++;
 				FrmFieldsController::load_single_field( $field, $values );
 				unset( $field );
 			}

classes/views/frm-form-actions/_action_inside.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 	$event_labels = FrmFormAction::trigger_labels();
 	foreach ( $action_control->action_options['event'] as $event ) {
 		?>
-		<option value="<?php echo esc_attr( $event ); ?>" <?php echo in_array( $event, (array) $form_action->post_content['event'] ) ? ' selected="selected"' : ''; ?> ><?php echo esc_html( isset( $event_labels[ $event ] ) ? $event_labels[ $event ] : $event ); ?></option>
+		<option value="<?php echo esc_attr( $event ); ?>" <?php echo in_array( $event, (array) $form_action->post_content['event'] ) ? ' selected="selected"' : ''; ?> ><?php echo esc_html( isset( $event_labels[$event] ) ? $event_labels[$event] : $event ); ?></option>
 <?php } ?>
 		</select>
 	</p>

classes/views/frm-form-actions/default_actions.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
 class FrmDefPostAction extends FrmFormAction {
 	public function __construct() {
 		$action_ops = FrmFormAction::default_action_opts( 'frm_wordpress_icon frm-inverse frm_show_upgrade' );
-		$action_ops['color']   = 'rgb(0,160,210)';
+		$action_ops['color'] = 'rgb(0,160,210)';
 
 		parent::__construct( 'wppost', __( 'Create Post', 'formidable' ), $action_ops );
 	}
@@ -42,7 +42,7 @@ 
 class FrmDefMlcmpAction extends FrmFormAction {
 	public function __construct() {
 		$action_ops = FrmFormAction::default_action_opts( 'frm_mailchimp_icon frm_show_upgrade frm-inverse' );
-		$action_ops['color']   = 'var(--dark-grey)';
+		$action_ops['color'] = 'var(--dark-grey)';
 
 		parent::__construct( 'mailchimp', 'MailChimp', $action_ops );
 	}

classes/helpers/FrmTipsHelper.php

Spacing +3 added lines, -3 removed lines

@@ -296,7 +296,7 @@ 
 	}
 
 	public static function get_banner_tip() {
-		$tips       = array(
+		$tips = array(
 			array(
 				'link' => array(
 					'medium'  => 'banner',
@@ -323,7 +323,7 @@ 
 			),
 		);
 		$random     = rand( 0, count( $tips ) - 1 );
-		$tip        = $tips[ $random ];
+		$tip        = $tips[$random];
 		$tip['num'] = $random;
 
 		return $tip;
@@ -332,6 +332,6 @@ 
 	public static function get_random_tip( $tips ) {
 		$random = rand( 0, count( $tips ) - 1 );
 
-		return $tips[ $random ];
+		return $tips[$random];
 	}
 }

classes/helpers/FrmFormsListHelper.php

Spacing +5 added lines, -5 removed lines

@@ -18,7 +18,7 @@ 
 		$page     = $this->get_pagenum();
 		$per_page = $this->get_items_per_page( 'formidable_page_formidable_per_page' );
 
-		$mode    = self::get_param(
+		$mode = self::get_param(
 			array(
 				'param'   => 'mode',
 				'default' => 'list',
@@ -30,13 +30,13 @@ 
 				'default' => 'name',
 			)
 		);
-		$order   = self::get_param(
+		$order = self::get_param(
 			array(
 				'param'   => 'order',
 				'default' => 'ASC',
 			)
 		);
-		$start   = self::get_param(
+		$start = self::get_param(
 			array(
 				'param'   => 'start',
 				'default' => ( $page - 1 ) * $per_page,
@@ -177,7 +177,7 @@ 
 
 			if ( $counts->{$status} || 'draft' !== $status ) {
 				/* translators: %1$s: Status, %2$s: Number of items */
-				$links[ $status ] = '<a href="' . esc_url( '?page=formidable&form_type=' . $status ) . '" ' . $class . '>' . sprintf( __( '%1$s <span class="count">(%2$s)</span>', 'formidable' ), $name, number_format_i18n( $counts->{$status} ) ) . '</a>';
+				$links[$status] = '<a href="' . esc_url( '?page=formidable&form_type=' . $status ) . '" ' . $class . '>' . sprintf( __( '%1$s <span class="count">(%2$s)</span>', 'formidable' ), $name, number_format_i18n( $counts->{$status} ) ) . '</a>';
 			}
 
 			unset( $status, $name );
@@ -282,7 +282,7 @@ 
 	private function get_actions( &$actions, $item, $edit_link ) {
 		$new_actions = FrmFormsHelper::get_action_links( $item->id, $item );
 		foreach ( $new_actions as $link => $action ) {
-			$new_actions[ $link ] = FrmFormsHelper::format_link_html( $action, 'short' );
+			$new_actions[$link] = FrmFormsHelper::format_link_html( $action, 'short' );
 		}
 
 		if ( 'trash' == $this->status ) {