@@ -16,7 +16,7 @@ |
||
16 | 16 | 'title' => esc_attr__( 'Toggle Options', 'formidable' ), |
17 | 17 | ) |
18 | 18 | ); |
19 | - $e_args = array( |
|
19 | + $e_args = array( |
|
20 | 20 | 'textarea_name' => 'field_options[description_' . absint( $field['id'] ) . ']', |
21 | 21 | 'textarea_rows' => 8, |
22 | 22 | ); |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | $metas = FrmEntryMeta::get_entry_meta_info( $entry_exist ); |
81 | 81 | $field_metas = array(); |
82 | 82 | foreach ( $metas as $meta ) { |
83 | - $field_metas[ $meta->field_id ] = $meta->meta_value; |
|
83 | + $field_metas[$meta->field_id] = $meta->meta_value; |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | $filtered_vals = array_filter( $values['item_meta'] ); |
@@ -136,10 +136,10 @@ discard block |
||
136 | 136 | $reduced = array(); |
137 | 137 | foreach ( $filter_vals as $field_id => $value ) { |
138 | 138 | $field = FrmFieldFactory::get_field_object( $field_id ); |
139 | - $reduced[ $field_id ] = $field->get_value_to_save( $value, array( 'entry_id' => $entry_id ) ); |
|
140 | - $reduced[ $field_id ] = $field->set_value_before_save( $reduced[ $field_id ] ); |
|
141 | - if ( '' === $reduced[ $field_id ] || ( is_array( $reduced[ $field_id ] ) && 0 === count( $reduced[ $field_id ] ) ) ) { |
|
142 | - unset( $reduced[ $field_id ] ); |
|
139 | + $reduced[$field_id] = $field->get_value_to_save( $value, array( 'entry_id' => $entry_id ) ); |
|
140 | + $reduced[$field_id] = $field->set_value_before_save( $reduced[$field_id] ); |
|
141 | + if ( '' === $reduced[$field_id] || ( is_array( $reduced[$field_id] ) && 0 === count( $reduced[$field_id] ) ) ) { |
|
142 | + unset( $reduced[$field_id] ); |
|
143 | 143 | } |
144 | 144 | } |
145 | 145 | return $reduced; |
@@ -332,7 +332,7 @@ discard block |
||
332 | 332 | $query = "SELECT it.*, fr.name as form_name, fr.form_key as form_key FROM {$wpdb->prefix}frm_items it |
333 | 333 | LEFT OUTER JOIN {$wpdb->prefix}frm_forms fr ON it.form_id=fr.id WHERE "; |
334 | 334 | |
335 | - $query .= is_numeric( $id ) ? 'it.id=%d' : 'it.item_key=%s'; |
|
335 | + $query .= is_numeric( $id ) ? 'it.id=%d' : 'it.item_key=%s'; |
|
336 | 336 | $query_args = array( $id ); |
337 | 337 | $query = $wpdb->prepare( $query, $query_args ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared |
338 | 338 | |
@@ -377,7 +377,7 @@ discard block |
||
377 | 377 | private static function prepare_entries( &$entries ) { |
378 | 378 | foreach ( $entries as $k => $entry ) { |
379 | 379 | self::prepare_entry( $entry ); |
380 | - $entries[ $k ] = $entry; |
|
380 | + $entries[$k] = $entry; |
|
381 | 381 | } |
382 | 382 | } |
383 | 383 | |
@@ -402,20 +402,20 @@ discard block |
||
402 | 402 | foreach ( $metas as $meta_val ) { |
403 | 403 | if ( $meta_val->item_id == $entry->id ) { |
404 | 404 | FrmAppHelper::unserialize_or_decode( $meta_val->meta_value ); |
405 | - $entry->metas[ $meta_val->field_id ] = $meta_val->meta_value; |
|
405 | + $entry->metas[$meta_val->field_id] = $meta_val->meta_value; |
|
406 | 406 | if ( $include_key ) { |
407 | - $entry->metas[ $meta_val->field_key ] = $entry->metas[ $meta_val->field_id ]; |
|
407 | + $entry->metas[$meta_val->field_key] = $entry->metas[$meta_val->field_id]; |
|
408 | 408 | } |
409 | 409 | continue; |
410 | 410 | } |
411 | 411 | |
412 | 412 | // include sub entries in an array |
413 | - if ( ! isset( $entry->metas[ $meta_val->field_id ] ) ) { |
|
414 | - $entry->metas[ $meta_val->field_id ] = array(); |
|
413 | + if ( ! isset( $entry->metas[$meta_val->field_id] ) ) { |
|
414 | + $entry->metas[$meta_val->field_id] = array(); |
|
415 | 415 | } |
416 | 416 | |
417 | 417 | FrmAppHelper::unserialize_or_decode( $meta_val->meta_value ); |
418 | - $entry->metas[ $meta_val->field_id ][] = $meta_val->meta_value; |
|
418 | + $entry->metas[$meta_val->field_id][] = $meta_val->meta_value; |
|
419 | 419 | |
420 | 420 | unset( $meta_val ); |
421 | 421 | } |
@@ -462,7 +462,7 @@ discard block |
||
462 | 462 | |
463 | 463 | if ( $inc_form ) { |
464 | 464 | $fields = 'it.*, fr.name as form_name,fr.form_key as form_key'; |
465 | - $table .= 'LEFT OUTER JOIN ' . $wpdb->prefix . 'frm_forms fr ON it.form_id=fr.id '; |
|
465 | + $table .= 'LEFT OUTER JOIN ' . $wpdb->prefix . 'frm_forms fr ON it.form_id=fr.id '; |
|
466 | 466 | } |
467 | 467 | |
468 | 468 | if ( preg_match( '/ meta_([0-9]+)/', $order_by, $order_matches ) ) { |
@@ -506,16 +506,16 @@ discard block |
||
506 | 506 | } |
507 | 507 | |
508 | 508 | foreach ( $metas as $m_key => $meta_val ) { |
509 | - if ( ! isset( $entries[ $meta_val->item_id ] ) ) { |
|
509 | + if ( ! isset( $entries[$meta_val->item_id] ) ) { |
|
510 | 510 | continue; |
511 | 511 | } |
512 | 512 | |
513 | - if ( ! isset( $entries[ $meta_val->item_id ]->metas ) ) { |
|
514 | - $entries[ $meta_val->item_id ]->metas = array(); |
|
513 | + if ( ! isset( $entries[$meta_val->item_id]->metas ) ) { |
|
514 | + $entries[$meta_val->item_id]->metas = array(); |
|
515 | 515 | } |
516 | 516 | |
517 | 517 | FrmAppHelper::unserialize_or_decode( $meta_val->meta_value ); |
518 | - $entries[ $meta_val->item_id ]->metas[ $meta_val->field_id ] = $meta_val->meta_value; |
|
518 | + $entries[$meta_val->item_id]->metas[$meta_val->field_id] = $meta_val->meta_value; |
|
519 | 519 | unset( $m_key, $meta_val ); |
520 | 520 | } |
521 | 521 | |
@@ -693,7 +693,7 @@ discard block |
||
693 | 693 | } |
694 | 694 | |
695 | 695 | private static function get_entry_value( $values, $name, $default ) { |
696 | - return isset( $values[ $name ] ) ? $values[ $name ] : $default; |
|
696 | + return isset( $values[$name] ) ? $values[$name] : $default; |
|
697 | 697 | } |
698 | 698 | |
699 | 699 | /** |
@@ -874,7 +874,7 @@ discard block |
||
874 | 874 | private static function maybe_add_captcha_meta( $form_id, $entry_id ) { |
875 | 875 | global $frm_vars; |
876 | 876 | if ( array_key_exists( 'captcha_scores', $frm_vars ) && array_key_exists( $form_id, $frm_vars['captcha_scores'] ) ) { |
877 | - $captcha_score_meta = array( 'captcha_score' => $frm_vars['captcha_scores'][ $form_id ] ); |
|
877 | + $captcha_score_meta = array( 'captcha_score' => $frm_vars['captcha_scores'][$form_id] ); |
|
878 | 878 | FrmEntryMeta::add_entry_meta( $entry_id, 0, '', maybe_serialize( $captcha_score_meta ) ); |
879 | 879 | } |
880 | 880 | } |
@@ -6,7 +6,7 @@ |
||
6 | 6 | $data_keys = array_keys( $data ); |
7 | 7 | $params = array(); |
8 | 8 | foreach ( $data_keys as $key ) { |
9 | - $params[ 'data-' . $key ] = $data[ $key ]; |
|
9 | + $params['data-' . $key] = $data[$key]; |
|
10 | 10 | } |
11 | 11 | $params['class'] = 'frm_show_upgrade frm_noallow'; |
12 | 12 | $params['href'] = '#'; |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | 'class' => '', |
29 | 29 | 'inc_children' => 'exclude', |
30 | 30 | ); |
31 | - $args = wp_parse_args( $args, $defaults ); |
|
31 | + $args = wp_parse_args( $args, $defaults ); |
|
32 | 32 | |
33 | 33 | if ( ! $args['field_id'] ) { |
34 | 34 | $args['field_id'] = $field_name; |
@@ -70,7 +70,7 @@ discard block |
||
70 | 70 | */ |
71 | 71 | public static function add_html_attr( $class, $param, &$add_html ) { |
72 | 72 | if ( ! empty( $class ) ) { |
73 | - $add_html[ $param ] = sanitize_title( $param ) . '="' . esc_attr( trim( sanitize_text_field( $class ) ) ) . '"'; |
|
73 | + $add_html[$param] = sanitize_title( $param ) . '="' . esc_attr( trim( sanitize_text_field( $class ) ) ) . '"'; |
|
74 | 74 | } |
75 | 75 | } |
76 | 76 | |
@@ -276,8 +276,8 @@ discard block |
||
276 | 276 | 'description' => '', |
277 | 277 | ); |
278 | 278 | foreach ( $defaults as $var => $default ) { |
279 | - if ( ! isset( $values[ $var ] ) ) { |
|
280 | - $values[ $var ] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' ); |
|
279 | + if ( ! isset( $values[$var] ) ) { |
|
280 | + $values[$var] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' ); |
|
281 | 281 | } |
282 | 282 | } |
283 | 283 | |
@@ -292,8 +292,8 @@ discard block |
||
292 | 292 | 'parent_form_id' => 0, |
293 | 293 | ); |
294 | 294 | foreach ( $defaults as $var => $default ) { |
295 | - if ( ! isset( $values[ $var ] ) ) { |
|
296 | - $values[ $var ] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' ); |
|
295 | + if ( ! isset( $values[$var] ) ) { |
|
296 | + $values[$var] = FrmAppHelper::get_param( $var, $default, 'get', 'sanitize_text_field' ); |
|
297 | 297 | } |
298 | 298 | } |
299 | 299 | unset( $defaults ); |
@@ -330,16 +330,16 @@ discard block |
||
330 | 330 | $defaults = self::get_default_opts(); |
331 | 331 | foreach ( $defaults as $var => $default ) { |
332 | 332 | if ( is_array( $default ) ) { |
333 | - if ( ! isset( $values[ $var ] ) ) { |
|
334 | - $values[ $var ] = ( $record && isset( $record->options[ $var ] ) ) ? $record->options[ $var ] : array(); |
|
333 | + if ( ! isset( $values[$var] ) ) { |
|
334 | + $values[$var] = ( $record && isset( $record->options[$var] ) ) ? $record->options[$var] : array(); |
|
335 | 335 | } |
336 | 336 | |
337 | 337 | foreach ( $default as $k => $v ) { |
338 | - $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 ); |
|
338 | + $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 ); |
|
339 | 339 | |
340 | 340 | if ( is_array( $v ) ) { |
341 | 341 | foreach ( $v as $k1 => $v1 ) { |
342 | - $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 ); |
|
342 | + $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 ); |
|
343 | 343 | unset( $k1, $v1 ); |
344 | 344 | } |
345 | 345 | } |
@@ -347,7 +347,7 @@ discard block |
||
347 | 347 | unset( $k, $v ); |
348 | 348 | } |
349 | 349 | } else { |
350 | - $values[ $var ] = ( $post_values && isset( $post_values['options'][ $var ] ) ) ? $post_values['options'][ $var ] : ( ( $record && isset( $record->options[ $var ] ) ) ? $record->options[ $var ] : $default ); |
|
350 | + $values[$var] = ( $post_values && isset( $post_values['options'][$var] ) ) ? $post_values['options'][$var] : ( ( $record && isset( $record->options[$var] ) ) ? $record->options[$var] : $default ); |
|
351 | 351 | } |
352 | 352 | |
353 | 353 | unset( $var, $default ); |
@@ -387,7 +387,7 @@ discard block |
||
387 | 387 | public static function fill_form_options( &$options, $values ) { |
388 | 388 | $defaults = self::get_default_opts(); |
389 | 389 | foreach ( $defaults as $var => $default ) { |
390 | - $options[ $var ] = isset( $values['options'][ $var ] ) ? $values['options'][ $var ] : $default; |
|
390 | + $options[$var] = isset( $values['options'][$var] ) ? $values['options'][$var] : $default; |
|
391 | 391 | unset( $var, $default ); |
392 | 392 | } |
393 | 393 | } |
@@ -549,7 +549,7 @@ discard block |
||
549 | 549 | public static function insert_opt_html( $args ) { |
550 | 550 | $class = isset( $args['class'] ) ? $args['class'] : ''; |
551 | 551 | $fields = FrmField::all_field_selection(); |
552 | - $field = isset( $fields[ $args['type'] ] ) ? $fields[ $args['type'] ] : array(); |
|
552 | + $field = isset( $fields[$args['type']] ) ? $fields[$args['type']] : array(); |
|
553 | 553 | |
554 | 554 | self::prepare_field_type( $field ); |
555 | 555 | |
@@ -705,7 +705,7 @@ discard block |
||
705 | 705 | FrmField::update( $field->id, array( 'field_order' => $field->field_order + 2 ) ); |
706 | 706 | } |
707 | 707 | |
708 | - $add_order += 2; |
|
708 | + $add_order += 2; |
|
709 | 709 | $open = false; |
710 | 710 | $reset_fields = true; |
711 | 711 | } |
@@ -1108,10 +1108,10 @@ discard block |
||
1108 | 1108 | $link .= ' onclick="return confirm(\'' . esc_attr( $link_details['confirm'] ) . '\')"'; |
1109 | 1109 | } |
1110 | 1110 | |
1111 | - $label = ( isset( $link_details[ $length ] ) ? $link_details[ $length ] : $link_details['label'] ); |
|
1112 | - if ( $length == 'icon' && isset( $link_details[ $length ] ) ) { |
|
1111 | + $label = ( isset( $link_details[$length] ) ? $link_details[$length] : $link_details['label'] ); |
|
1112 | + if ( $length == 'icon' && isset( $link_details[$length] ) ) { |
|
1113 | 1113 | $label = '<span class="' . $label . '" title="' . esc_attr( $link_details['label'] ) . '" aria-hidden="true"></span>'; |
1114 | - $link .= ' aria-label="' . esc_attr( $link_details['label'] ) . '"'; |
|
1114 | + $link .= ' aria-label="' . esc_attr( $link_details['label'] ) . '"'; |
|
1115 | 1115 | } |
1116 | 1116 | |
1117 | 1117 | $link .= '>' . $label . '</a>'; |
@@ -1250,7 +1250,7 @@ discard block |
||
1250 | 1250 | $status = 'publish'; |
1251 | 1251 | } |
1252 | 1252 | |
1253 | - $name = $nice_names[ $status ]; |
|
1253 | + $name = $nice_names[$status]; |
|
1254 | 1254 | |
1255 | 1255 | return $name; |
1256 | 1256 | } |
@@ -1283,17 +1283,17 @@ discard block |
||
1283 | 1283 | '' => array( 'align_right' ), |
1284 | 1284 | ); |
1285 | 1285 | |
1286 | - $icons[ __( 'My Templates', 'formidable' ) ] = array( 'user', 'var(--orange)' ); |
|
1286 | + $icons[__( 'My Templates', 'formidable' )] = array( 'user', 'var(--orange)' ); |
|
1287 | 1287 | |
1288 | 1288 | $icon = $icons['']; |
1289 | 1289 | |
1290 | 1290 | if ( count( $categories ) === 1 ) { |
1291 | 1291 | $category = reset( $categories ); |
1292 | - $icon = isset( $icons[ $category ] ) ? $icons[ $category ] : $icon; |
|
1292 | + $icon = isset( $icons[$category] ) ? $icons[$category] : $icon; |
|
1293 | 1293 | } elseif ( ! empty( $categories ) ) { |
1294 | 1294 | foreach ( $icons as $cat => $icon ) { |
1295 | 1295 | if ( ! in_array( $cat, $categories ) ) { |
1296 | - unset( $icons[ $cat ] ); |
|
1296 | + unset( $icons[$cat] ); |
|
1297 | 1297 | } |
1298 | 1298 | } |
1299 | 1299 | $icon = reset( $icons ); |
@@ -1422,7 +1422,7 @@ discard block |
||
1422 | 1422 | |
1423 | 1423 | foreach ( $item['categories'] as $k => $category ) { |
1424 | 1424 | if ( in_array( $category, $plans, true ) ) { |
1425 | - unset( $item['categories'][ $k ] ); |
|
1425 | + unset( $item['categories'][$k] ); |
|
1426 | 1426 | |
1427 | 1427 | if ( in_array( $category, array( 'Creator', 'Personal' ), true ) ) { |
1428 | 1428 | // Show the current package name. |
@@ -1650,9 +1650,9 @@ discard block |
||
1650 | 1650 | } |
1651 | 1651 | |
1652 | 1652 | foreach ( $shortcodes[0] as $key => $shortcode ) { |
1653 | - $options = trim( $shortcodes[3][ $key ] ); |
|
1653 | + $options = trim( $shortcodes[3][$key] ); |
|
1654 | 1654 | |
1655 | - if ( in_array( $shortcodes[1][ $key ], array( 'if ' ), true ) ) { |
|
1655 | + if ( in_array( $shortcodes[1][$key], array( 'if ' ), true ) ) { |
|
1656 | 1656 | // Skip if shortcodes. |
1657 | 1657 | continue; |
1658 | 1658 | } |
@@ -1662,7 +1662,7 @@ discard block |
||
1662 | 1662 | continue; |
1663 | 1663 | } |
1664 | 1664 | |
1665 | - $new_shortcode = '[' . $shortcodes[2][ $key ]; |
|
1665 | + $new_shortcode = '[' . $shortcodes[2][$key]; |
|
1666 | 1666 | if ( $options ) { |
1667 | 1667 | $new_shortcode .= ' ' . $options; |
1668 | 1668 | } |
@@ -41,7 +41,7 @@ discard block |
||
41 | 41 | $sub_field_class = "frm_form_field form-field frm_form_subfield-{$name} {$sub_field['wrapper_classes']}"; |
42 | 42 | $sub_field_desc = FrmField::get_option( $field, $name . '_desc' ); |
43 | 43 | |
44 | - if ( isset( $errors[ 'field' . $field_id . '-' . $name ] ) ) { |
|
44 | + if ( isset( $errors['field' . $field_id . '-' . $name] ) ) { |
|
45 | 45 | $sub_field_class .= ' frm_blank_field'; |
46 | 46 | } |
47 | 47 | ?> |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | <input |
62 | 62 | type="<?php echo esc_attr( $sub_field['type'] ); ?>" |
63 | 63 | id="<?php echo esc_attr( $html_id . '_' . $name ); ?>" |
64 | - value="<?php echo esc_attr( isset( $field_value[ $name ] ) ? $field_value[ $name ] : '' ); ?>" |
|
64 | + value="<?php echo esc_attr( isset( $field_value[$name] ) ? $field_value[$name] : '' ); ?>" |
|
65 | 65 | <?php |
66 | 66 | if ( empty( $args['remove_names'] ) ) { |
67 | 67 | echo 'name="' . esc_attr( $field_name ) . '[' . esc_attr( $name ) . ']" '; |
@@ -78,9 +78,9 @@ discard block |
||
78 | 78 | } |
79 | 79 | |
80 | 80 | // Don't show individual field errors when there is a combo field error. |
81 | - if ( ! empty( $errors ) && isset( $errors[ 'field' . $field_id . '-' . $name ] ) && ! isset( $errors[ 'field' . $field_id ] ) ) { |
|
81 | + if ( ! empty( $errors ) && isset( $errors['field' . $field_id . '-' . $name] ) && ! isset( $errors['field' . $field_id] ) ) { |
|
82 | 82 | ?> |
83 | - <div class="frm_error" role="alert"><?php echo esc_html( $errors[ 'field' . $field_id . '-' . $name ] ); ?></div> |
|
83 | + <div class="frm_error" role="alert"><?php echo esc_html( $errors['field' . $field_id . '-' . $name] ); ?></div> |
|
84 | 84 | <?php } ?> |
85 | 85 | </div> |
86 | 86 | <?php } ?> |
@@ -443,11 +443,11 @@ discard block |
||
443 | 443 | ), |
444 | 444 | ); |
445 | 445 | |
446 | - if ( ! isset( $available_status[ $status ] ) ) { |
|
446 | + if ( ! isset( $available_status[$status] ) ) { |
|
447 | 447 | return; |
448 | 448 | } |
449 | 449 | |
450 | - FrmAppHelper::permission_check( $available_status[ $status ]['permission'] ); |
|
450 | + FrmAppHelper::permission_check( $available_status[$status]['permission'] ); |
|
451 | 451 | |
452 | 452 | $params = FrmForm::list_page_params(); |
453 | 453 | |
@@ -455,7 +455,7 @@ discard block |
||
455 | 455 | check_admin_referer( $status . '_form_' . $params['id'] ); |
456 | 456 | |
457 | 457 | $count = 0; |
458 | - if ( FrmForm::set_status( $params['id'], $available_status[ $status ]['new_status'] ) ) { |
|
458 | + if ( FrmForm::set_status( $params['id'], $available_status[$status]['new_status'] ) ) { |
|
459 | 459 | $count ++; |
460 | 460 | } |
461 | 461 | |
@@ -472,7 +472,7 @@ discard block |
||
472 | 472 | /* translators: %1$s: Number of forms, %2$s: Start link HTML, %3$s: End link HTML */ |
473 | 473 | $available_status['trash']['message'] = sprintf( _n( '%1$s form moved to the Trash. %2$sUndo%3$s', '%1$s forms moved to the Trash. %2$sUndo%3$s', $count, 'formidable' ), $count, '<a href="' . esc_url( wp_nonce_url( '?page=formidable&frm_action=untrash&form_type=' . $form_type . '&id=' . $params['id'], 'untrash_form_' . $params['id'] ) ) . '">', '</a>' ); |
474 | 474 | |
475 | - $message = $available_status[ $status ]['message']; |
|
475 | + $message = $available_status[$status]['message']; |
|
476 | 476 | |
477 | 477 | self::display_forms_list( $params, $message ); |
478 | 478 | } |
@@ -493,7 +493,7 @@ discard block |
||
493 | 493 | 'type' => 'request', |
494 | 494 | ) |
495 | 495 | ); |
496 | - $message = sprintf( |
|
496 | + $message = sprintf( |
|
497 | 497 | /* translators: %1$s: Number of forms, %2$s: Start link HTML, %3$s: End link HTML */ |
498 | 498 | _n( '%1$s form moved to the Trash. %2$sUndo%3$s', '%1$s forms moved to the Trash. %2$sUndo%3$s', $count, 'formidable' ), |
499 | 499 | $count, |
@@ -887,11 +887,11 @@ discard block |
||
887 | 887 | } |
888 | 888 | |
889 | 889 | foreach ( $template['categories'] as $category ) { |
890 | - if ( ! isset( $templates_by_category[ $category ] ) ) { |
|
891 | - $templates_by_category[ $category ] = array(); |
|
890 | + if ( ! isset( $templates_by_category[$category] ) ) { |
|
891 | + $templates_by_category[$category] = array(); |
|
892 | 892 | } |
893 | 893 | |
894 | - $templates_by_category[ $category ][] = $template; |
|
894 | + $templates_by_category[$category][] = $template; |
|
895 | 895 | } |
896 | 896 | } |
897 | 897 | unset( $template ); |
@@ -920,7 +920,7 @@ discard block |
||
920 | 920 | $forms = FrmForm::get_published_forms( $where ); |
921 | 921 | $view_path = FrmAppHelper::plugin_path() . '/classes/views/frm-forms/'; |
922 | 922 | |
923 | - $templates_by_category[ $my_templates_translation ] = $custom_templates; |
|
923 | + $templates_by_category[$my_templates_translation] = $custom_templates; |
|
924 | 924 | |
925 | 925 | unset( $pricing, $license_type, $where ); |
926 | 926 | wp_enqueue_script( 'accordion' ); // register accordion for template groups |
@@ -1145,7 +1145,7 @@ discard block |
||
1145 | 1145 | |
1146 | 1146 | foreach ( array( 'landing', 'chat' ) as $feature ) { |
1147 | 1147 | if ( ! FrmAppHelper::show_new_feature( $feature ) ) { |
1148 | - unset( $sections[ $feature ] ); |
|
1148 | + unset( $sections[$feature] ); |
|
1149 | 1149 | } |
1150 | 1150 | } |
1151 | 1151 | |
@@ -1178,7 +1178,7 @@ discard block |
||
1178 | 1178 | $section['id'] = $section['anchor']; |
1179 | 1179 | } |
1180 | 1180 | |
1181 | - $sections[ $key ] = $section; |
|
1181 | + $sections[$key] = $section; |
|
1182 | 1182 | } |
1183 | 1183 | |
1184 | 1184 | return $sections; |
@@ -1281,7 +1281,7 @@ discard block |
||
1281 | 1281 | if ( ! empty( $user_fields ) ) { |
1282 | 1282 | $user_helpers = array(); |
1283 | 1283 | foreach ( $user_fields as $uk => $uf ) { |
1284 | - $user_helpers[ '|user_id| show="' . $uk . '"' ] = $uf; |
|
1284 | + $user_helpers['|user_id| show="' . $uk . '"'] = $uf; |
|
1285 | 1285 | unset( $uk, $uf ); |
1286 | 1286 | } |
1287 | 1287 | |
@@ -1411,7 +1411,7 @@ discard block |
||
1411 | 1411 | if ( ! isset( $frm_vars['js_validate_forms'] ) ) { |
1412 | 1412 | $frm_vars['js_validate_forms'] = array(); |
1413 | 1413 | } |
1414 | - $frm_vars['js_validate_forms'][ $form->id ] = $form; |
|
1414 | + $frm_vars['js_validate_forms'][$form->id] = $form; |
|
1415 | 1415 | } |
1416 | 1416 | |
1417 | 1417 | public static function get_email_html() { |
@@ -1532,7 +1532,7 @@ discard block |
||
1532 | 1532 | add_filter( 'frm_validate_form', 'FrmFormsController::json_error' ); |
1533 | 1533 | } else { |
1534 | 1534 | $vars = FrmAppHelper::json_to_array( $json_vars ); |
1535 | - $action = $vars[ $action ]; |
|
1535 | + $action = $vars[$action]; |
|
1536 | 1536 | unset( $_REQUEST['frm_compact_fields'], $_POST['frm_compact_fields'] ); // phpcs:ignore WordPress.Security.NonceVerification.Missing |
1537 | 1537 | $_REQUEST = array_merge( $_REQUEST, $vars ); // phpcs:ignore WordPress.Security.NonceVerification.Missing |
1538 | 1538 | $_POST = array_merge( $_POST, $_REQUEST ); // phpcs:ignore WordPress.Security.NonceVerification.Missing |
@@ -1658,7 +1658,7 @@ discard block |
||
1658 | 1658 | $actions = array(); |
1659 | 1659 | foreach ( $frm_vars['forms_loaded'] as $form ) { |
1660 | 1660 | if ( is_object( $form ) ) { |
1661 | - $actions[ $form->id ] = $form->name; |
|
1661 | + $actions[$form->id] = $form->name; |
|
1662 | 1662 | } |
1663 | 1663 | unset( $form ); |
1664 | 1664 | } |
@@ -1867,8 +1867,8 @@ discard block |
||
1867 | 1867 | private static function get_saved_errors( $form, $params ) { |
1868 | 1868 | global $frm_vars; |
1869 | 1869 | |
1870 | - if ( $params['posted_form_id'] == $form->id && $_POST && isset( $frm_vars['created_entries'][ $form->id ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1871 | - $errors = $frm_vars['created_entries'][ $form->id ]['errors']; |
|
1870 | + if ( $params['posted_form_id'] == $form->id && $_POST && isset( $frm_vars['created_entries'][$form->id] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1871 | + $errors = $frm_vars['created_entries'][$form->id]['errors']; |
|
1872 | 1872 | } else { |
1873 | 1873 | $errors = array(); |
1874 | 1874 | } |
@@ -1882,7 +1882,7 @@ discard block |
||
1882 | 1882 | public static function just_created_entry( $form_id ) { |
1883 | 1883 | global $frm_vars; |
1884 | 1884 | |
1885 | - return ( isset( $frm_vars['created_entries'] ) && isset( $frm_vars['created_entries'][ $form_id ] ) && isset( $frm_vars['created_entries'][ $form_id ]['entry_id'] ) ) ? $frm_vars['created_entries'][ $form_id ]['entry_id'] : 0; |
|
1885 | + return ( isset( $frm_vars['created_entries'] ) && isset( $frm_vars['created_entries'][$form_id] ) && isset( $frm_vars['created_entries'][$form_id]['entry_id'] ) ) ? $frm_vars['created_entries'][$form_id]['entry_id'] : 0; |
|
1886 | 1886 | } |
1887 | 1887 | |
1888 | 1888 | /** |
@@ -1890,7 +1890,7 @@ discard block |
||
1890 | 1890 | */ |
1891 | 1891 | private static function get_confirmation_method( $atts ) { |
1892 | 1892 | $opt = 'success_action'; |
1893 | - $method = ( isset( $atts['form']->options[ $opt ] ) && ! empty( $atts['form']->options[ $opt ] ) ) ? $atts['form']->options[ $opt ] : 'message'; |
|
1893 | + $method = ( isset( $atts['form']->options[$opt] ) && ! empty( $atts['form']->options[$opt] ) ) ? $atts['form']->options[$opt] : 'message'; |
|
1894 | 1894 | $method = apply_filters( 'frm_success_filter', $method, $atts['form'], 'create' ); |
1895 | 1895 | |
1896 | 1896 | if ( $method != 'message' && ( ! $atts['entry_id'] || ! is_numeric( $atts['entry_id'] ) ) ) { |
@@ -1903,7 +1903,7 @@ discard block |
||
1903 | 1903 | public static function maybe_trigger_redirect( $form, $params, $args ) { |
1904 | 1904 | if ( ! isset( $params['id'] ) ) { |
1905 | 1905 | global $frm_vars; |
1906 | - $params['id'] = $frm_vars['created_entries'][ $form->id ]['entry_id']; |
|
1906 | + $params['id'] = $frm_vars['created_entries'][$form->id]['entry_id']; |
|
1907 | 1907 | } |
1908 | 1908 | |
1909 | 1909 | $conf_method = self::get_confirmation_method( |
@@ -1947,7 +1947,7 @@ discard block |
||
1947 | 1947 | $opt = ( ! isset( $args['action'] ) || $args['action'] === 'create' ) ? 'success' : 'edit'; |
1948 | 1948 | |
1949 | 1949 | $args['success_opt'] = $opt; |
1950 | - if ( $args['conf_method'] === 'page' && is_numeric( $args['form']->options[ $opt . '_page_id' ] ) ) { |
|
1950 | + if ( $args['conf_method'] === 'page' && is_numeric( $args['form']->options[$opt . '_page_id'] ) ) { |
|
1951 | 1951 | self::load_page_after_submit( $args ); |
1952 | 1952 | } elseif ( $args['conf_method'] === 'redirect' ) { |
1953 | 1953 | self::redirect_after_submit( $args ); |
@@ -1962,8 +1962,8 @@ discard block |
||
1962 | 1962 | private static function load_page_after_submit( $args ) { |
1963 | 1963 | global $post; |
1964 | 1964 | $opt = $args['success_opt']; |
1965 | - if ( ! $post || $args['form']->options[ $opt . '_page_id' ] != $post->ID ) { |
|
1966 | - $page = get_post( $args['form']->options[ $opt . '_page_id' ] ); |
|
1965 | + if ( ! $post || $args['form']->options[$opt . '_page_id'] != $post->ID ) { |
|
1966 | + $page = get_post( $args['form']->options[$opt . '_page_id'] ); |
|
1967 | 1967 | $old_post = $post; |
1968 | 1968 | $post = $page; |
1969 | 1969 | $content = apply_filters( 'frm_content', $page->post_content, $args['form'], $args['entry_id'] ); |
@@ -1981,11 +1981,11 @@ discard block |
||
1981 | 1981 | add_filter( 'frm_use_wpautop', '__return_false' ); |
1982 | 1982 | |
1983 | 1983 | $opt = $args['success_opt']; |
1984 | - $success_url = trim( $args['form']->options[ $opt . '_url' ] ); |
|
1984 | + $success_url = trim( $args['form']->options[$opt . '_url'] ); |
|
1985 | 1985 | $success_url = apply_filters( 'frm_content', $success_url, $args['form'], $args['entry_id'] ); |
1986 | 1986 | $success_url = do_shortcode( $success_url ); |
1987 | 1987 | |
1988 | - $success_msg = isset( $args['form']->options[ $opt . '_msg' ] ) ? $args['form']->options[ $opt . '_msg' ] : __( 'Please wait while you are redirected.', 'formidable' ); |
|
1988 | + $success_msg = isset( $args['form']->options[$opt . '_msg'] ) ? $args['form']->options[$opt . '_msg'] : __( 'Please wait while you are redirected.', 'formidable' ); |
|
1989 | 1989 | |
1990 | 1990 | $redirect_msg = self::get_redirect_message( $success_url, $success_msg, $args ); |
1991 | 1991 | |
@@ -2122,7 +2122,7 @@ discard block |
||
2122 | 2122 | 'description' => false, |
2123 | 2123 | 'reset' => false, |
2124 | 2124 | ); |
2125 | - $args = wp_parse_args( $args, $defaults ); |
|
2125 | + $args = wp_parse_args( $args, $defaults ); |
|
2126 | 2126 | } |
2127 | 2127 | |
2128 | 2128 | /** |
@@ -2338,7 +2338,7 @@ discard block |
||
2338 | 2338 | |
2339 | 2339 | check_ajax_referer( 'frm_ajax', 'nonce' ); |
2340 | 2340 | |
2341 | - $html = FrmAppHelper::clip( |
|
2341 | + $html = FrmAppHelper::clip( |
|
2342 | 2342 | function() { |
2343 | 2343 | FrmAppHelper::maybe_autocomplete_pages_options( |
2344 | 2344 | array( |
@@ -1520,7 +1520,8 @@ discard block |
||
1520 | 1520 | $vars = array(); |
1521 | 1521 | FrmAppHelper::include_svg(); |
1522 | 1522 | |
1523 | - if ( isset( $_POST['frm_compact_fields'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1523 | + if ( isset( $_POST['frm_compact_fields'] ) ) { |
|
1524 | +// phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1524 | 1525 | FrmAppHelper::permission_check( 'frm_edit_forms' ); |
1525 | 1526 | |
1526 | 1527 | // Javascript needs to be allowed in some field settings. |
@@ -1828,7 +1829,8 @@ discard block |
||
1828 | 1829 | private static function maybe_get_form_to_show( $id ) { |
1829 | 1830 | $form = false; |
1830 | 1831 | |
1831 | - if ( ! empty( $id ) ) { // form id or key is set |
|
1832 | + if ( ! empty( $id ) ) { |
|
1833 | +// form id or key is set |
|
1832 | 1834 | $form = FrmForm::getOne( $id ); |
1833 | 1835 | if ( ! $form || $form->parent_form_id || $form->status === 'trash' ) { |
1834 | 1836 | $form = false; |
@@ -1910,7 +1912,8 @@ discard block |
||
1910 | 1912 | private static function get_saved_errors( $form, $params ) { |
1911 | 1913 | global $frm_vars; |
1912 | 1914 | |
1913 | - if ( $params['posted_form_id'] == $form->id && $_POST && isset( $frm_vars['created_entries'][ $form->id ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1915 | + if ( $params['posted_form_id'] == $form->id && $_POST && isset( $frm_vars['created_entries'][ $form->id ] ) ) { |
|
1916 | +// phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
1914 | 1917 | $errors = $frm_vars['created_entries'][ $form->id ]['errors']; |
1915 | 1918 | } else { |
1916 | 1919 | $errors = array(); |
@@ -5,15 +5,15 @@ discard block |
||
5 | 5 | |
6 | 6 | class FrmFormAction { |
7 | 7 | |
8 | - public $id_base; // Root id for all actions of this type. |
|
9 | - public $name; // Name for this action type. |
|
8 | + public $id_base; // Root id for all actions of this type. |
|
9 | + public $name; // Name for this action type. |
|
10 | 10 | public $option_name; |
11 | - public $action_options; // Option array passed to wp_register_sidebar_widget() |
|
11 | + public $action_options; // Option array passed to wp_register_sidebar_widget() |
|
12 | 12 | public $control_options; // Option array passed to wp_register_widget_control() |
13 | 13 | |
14 | - public $form_id; // The ID of the form to evaluate |
|
15 | - public $number = false; // Unique ID number of the current instance. |
|
16 | - public $id = ''; // Unique ID string of the current instance (id_base-number) |
|
14 | + public $form_id; // The ID of the form to evaluate |
|
15 | + public $number = false; // Unique ID number of the current instance. |
|
16 | + public $id = ''; // Unique ID string of the current instance (id_base-number) |
|
17 | 17 | public $updated = false; // Set true when we update the data after a POST submit - makes sure we don't do it twice. |
18 | 18 | |
19 | 19 | // Member functions that you must over-ride. |
@@ -151,9 +151,9 @@ discard block |
||
151 | 151 | $groups = FrmFormActionsController::form_action_groups(); |
152 | 152 | $group = 'misc'; |
153 | 153 | |
154 | - if ( isset( $action_options['group'] ) && isset( $groups[ $action_options['group'] ] ) ) { |
|
154 | + if ( isset( $action_options['group'] ) && isset( $groups[$action_options['group']] ) ) { |
|
155 | 155 | $group = $action_options['group']; |
156 | - } elseif ( isset( $groups[ $this->id_base ] ) ) { |
|
156 | + } elseif ( isset( $groups[$this->id_base] ) ) { |
|
157 | 157 | $group = $this->id_base; |
158 | 158 | } else { |
159 | 159 | foreach ( $groups as $name => $check_group ) { |
@@ -164,8 +164,8 @@ discard block |
||
164 | 164 | } |
165 | 165 | } |
166 | 166 | |
167 | - $groups[ $group ]['id'] = $group; |
|
168 | - return $groups[ $group ]; |
|
167 | + $groups[$group]['id'] = $group; |
|
168 | + return $groups[$group]; |
|
169 | 169 | } |
170 | 170 | |
171 | 171 | /** |
@@ -267,7 +267,7 @@ discard block |
||
267 | 267 | * @return integer $post_id |
268 | 268 | */ |
269 | 269 | public function maybe_create_action( $action, $forms ) { |
270 | - if ( isset( $action['ID'] ) && is_numeric( $action['ID'] ) && isset( $forms[ $action['menu_order'] ] ) && $forms[ $action['menu_order'] ] == 'updated' ) { |
|
270 | + if ( isset( $action['ID'] ) && is_numeric( $action['ID'] ) && isset( $forms[$action['menu_order']] ) && $forms[$action['menu_order']] == 'updated' ) { |
|
271 | 271 | // Update action only |
272 | 272 | $action['post_content'] = FrmAppHelper::maybe_json_decode( $action['post_content'] ); |
273 | 273 | $post_id = $this->save_settings( $action ); |
@@ -287,18 +287,18 @@ discard block |
||
287 | 287 | $switch = $this->get_global_switch_fields(); |
288 | 288 | |
289 | 289 | foreach ( (array) $action->post_content as $key => $val ) { |
290 | - if ( is_numeric( $val ) && isset( $frm_duplicate_ids[ $val ] ) ) { |
|
291 | - $action->post_content[ $key ] = $frm_duplicate_ids[ $val ]; |
|
290 | + if ( is_numeric( $val ) && isset( $frm_duplicate_ids[$val] ) ) { |
|
291 | + $action->post_content[$key] = $frm_duplicate_ids[$val]; |
|
292 | 292 | } elseif ( ! is_array( $val ) ) { |
293 | - $action->post_content[ $key ] = FrmFieldsHelper::switch_field_ids( $val ); |
|
294 | - } elseif ( isset( $switch[ $key ] ) && is_array( $switch[ $key ] ) ) { |
|
293 | + $action->post_content[$key] = FrmFieldsHelper::switch_field_ids( $val ); |
|
294 | + } elseif ( isset( $switch[$key] ) && is_array( $switch[$key] ) ) { |
|
295 | 295 | // loop through each value if empty |
296 | - if ( empty( $switch[ $key ] ) ) { |
|
297 | - $switch[ $key ] = array_keys( $val ); |
|
296 | + if ( empty( $switch[$key] ) ) { |
|
297 | + $switch[$key] = array_keys( $val ); |
|
298 | 298 | } |
299 | 299 | |
300 | - foreach ( $switch[ $key ] as $subkey ) { |
|
301 | - $action->post_content[ $key ] = $this->duplicate_array_walk( $action->post_content[ $key ], $subkey, $val ); |
|
300 | + foreach ( $switch[$key] as $subkey ) { |
|
301 | + $action->post_content[$key] = $this->duplicate_array_walk( $action->post_content[$key], $subkey, $val ); |
|
302 | 302 | } |
303 | 303 | } |
304 | 304 | |
@@ -316,20 +316,20 @@ discard block |
||
316 | 316 | foreach ( $subkey as $subkey2 ) { |
317 | 317 | foreach ( (array) $val as $ck => $cv ) { |
318 | 318 | if ( is_array( $cv ) ) { |
319 | - $action[ $ck ] = $this->duplicate_array_walk( $action[ $ck ], $subkey2, $cv ); |
|
320 | - } elseif ( isset( $cv[ $subkey ] ) && is_numeric( $cv[ $subkey ] ) && isset( $frm_duplicate_ids[ $cv[ $subkey ] ] ) ) { |
|
321 | - $action[ $ck ][ $subkey ] = $frm_duplicate_ids[ $cv[ $subkey ] ]; |
|
319 | + $action[$ck] = $this->duplicate_array_walk( $action[$ck], $subkey2, $cv ); |
|
320 | + } elseif ( isset( $cv[$subkey] ) && is_numeric( $cv[$subkey] ) && isset( $frm_duplicate_ids[$cv[$subkey]] ) ) { |
|
321 | + $action[$ck][$subkey] = $frm_duplicate_ids[$cv[$subkey]]; |
|
322 | 322 | } |
323 | 323 | } |
324 | 324 | } |
325 | 325 | } else { |
326 | 326 | foreach ( (array) $val as $ck => $cv ) { |
327 | 327 | if ( is_array( $cv ) ) { |
328 | - $action[ $ck ] = $this->duplicate_array_walk( $action[ $ck ], $subkey, $cv ); |
|
329 | - } elseif ( $ck == $subkey && isset( $frm_duplicate_ids[ $cv ] ) ) { |
|
330 | - $action[ $ck ] = $frm_duplicate_ids[ $cv ]; |
|
328 | + $action[$ck] = $this->duplicate_array_walk( $action[$ck], $subkey, $cv ); |
|
329 | + } elseif ( $ck == $subkey && isset( $frm_duplicate_ids[$cv] ) ) { |
|
330 | + $action[$ck] = $frm_duplicate_ids[$cv]; |
|
331 | 331 | } elseif ( $ck == $subkey ) { |
332 | - $action[ $ck ] = $this->maybe_switch_field_ids( $action[ $ck ] ); |
|
332 | + $action[$ck] = $this->maybe_switch_field_ids( $action[$ck] ); |
|
333 | 333 | } |
334 | 334 | } |
335 | 335 | } |
@@ -353,10 +353,10 @@ discard block |
||
353 | 353 | } |
354 | 354 | |
355 | 355 | // phpcs:ignore WordPress.Security.NonceVerification.Missing |
356 | - if ( isset( $_POST[ $this->option_name ] ) && is_array( $_POST[ $this->option_name ] ) ) { |
|
356 | + if ( isset( $_POST[$this->option_name] ) && is_array( $_POST[$this->option_name] ) ) { |
|
357 | 357 | // Sanitizing removes scripts and <email> type of values. |
358 | 358 | // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing |
359 | - $settings = wp_unslash( $_POST[ $this->option_name ] ); |
|
359 | + $settings = wp_unslash( $_POST[$this->option_name] ); |
|
360 | 360 | } else { |
361 | 361 | return; |
362 | 362 | } |
@@ -366,7 +366,7 @@ discard block |
||
366 | 366 | foreach ( $settings as $number => $new_instance ) { |
367 | 367 | $this->_set( $number ); |
368 | 368 | |
369 | - $old_instance = isset( $all_instances[ $number ] ) ? $all_instances[ $number ] : array(); |
|
369 | + $old_instance = isset( $all_instances[$number] ) ? $all_instances[$number] : array(); |
|
370 | 370 | |
371 | 371 | if ( ! isset( $new_instance['post_status'] ) ) { |
372 | 372 | $new_instance['post_status'] = 'draft'; |
@@ -406,7 +406,7 @@ discard block |
||
406 | 406 | $instance['post_content'] = apply_filters( 'frm_before_save_' . $this->id_base . '_action', $instance['post_content'], $instance, $new_instance, $old_instance, $this ); |
407 | 407 | |
408 | 408 | if ( false !== $instance ) { |
409 | - $all_instances[ $number ] = $instance; |
|
409 | + $all_instances[$number] = $instance; |
|
410 | 410 | } |
411 | 411 | |
412 | 412 | $action_ids[] = $this->save_settings( $instance ); |
@@ -482,12 +482,12 @@ discard block |
||
482 | 482 | // some plugins/themes are formatting the post_excerpt |
483 | 483 | $action->post_excerpt = sanitize_title( $action->post_excerpt ); |
484 | 484 | |
485 | - if ( ! isset( $action_controls[ $action->post_excerpt ] ) ) { |
|
485 | + if ( ! isset( $action_controls[$action->post_excerpt] ) ) { |
|
486 | 486 | continue; |
487 | 487 | } |
488 | 488 | |
489 | - $action = $action_controls[ $action->post_excerpt ]->prepare_action( $action ); |
|
490 | - $settings[ $action->ID ] = $action; |
|
489 | + $action = $action_controls[$action->post_excerpt]->prepare_action( $action ); |
|
490 | + $settings[$action->ID] = $action; |
|
491 | 491 | |
492 | 492 | if ( count( $settings ) >= $limit ) { |
493 | 493 | break; |
@@ -517,7 +517,7 @@ discard block |
||
517 | 517 | 'limit' => 99, |
518 | 518 | 'post_status' => $default_status, |
519 | 519 | ); |
520 | - $args = wp_parse_args( $args, $defaults ); |
|
520 | + $args = wp_parse_args( $args, $defaults ); |
|
521 | 521 | } |
522 | 522 | |
523 | 523 | /** |
@@ -578,7 +578,7 @@ discard block |
||
578 | 578 | |
579 | 579 | $action = $this->prepare_action( $action ); |
580 | 580 | |
581 | - $settings[ $action->ID ] = $action; |
|
581 | + $settings[$action->ID] = $action; |
|
582 | 582 | } |
583 | 583 | |
584 | 584 | if ( 1 === $limit ) { |
@@ -615,10 +615,10 @@ discard block |
||
615 | 615 | |
616 | 616 | foreach ( $default_values as $k => $vals ) { |
617 | 617 | if ( is_array( $vals ) && ! empty( $vals ) ) { |
618 | - if ( 'event' == $k && ! $this->action_options['force_event'] && ! empty( $action->post_content[ $k ] ) ) { |
|
618 | + if ( 'event' == $k && ! $this->action_options['force_event'] && ! empty( $action->post_content[$k] ) ) { |
|
619 | 619 | continue; |
620 | 620 | } |
621 | - $action->post_content[ $k ] = wp_parse_args( $action->post_content[ $k ], $vals ); |
|
621 | + $action->post_content[$k] = wp_parse_args( $action->post_content[$k], $vals ); |
|
622 | 622 | } |
623 | 623 | } |
624 | 624 | |
@@ -693,14 +693,14 @@ discard block |
||
693 | 693 | * Migrate settings from form->options into new action. |
694 | 694 | */ |
695 | 695 | public function migrate_to_2( $form, $update = 'update' ) { |
696 | - $action = $this->prepare_new( $form->id ); |
|
696 | + $action = $this->prepare_new( $form->id ); |
|
697 | 697 | FrmAppHelper::unserialize_or_decode( $form->options ); |
698 | 698 | |
699 | 699 | // fill with existing options |
700 | 700 | foreach ( $action->post_content as $name => $val ) { |
701 | - if ( isset( $form->options[ $name ] ) ) { |
|
702 | - $action->post_content[ $name ] = $form->options[ $name ]; |
|
703 | - unset( $form->options[ $name ] ); |
|
701 | + if ( isset( $form->options[$name] ) ) { |
|
702 | + $action->post_content[$name] = $form->options[$name]; |
|
703 | + unset( $form->options[$name] ); |
|
704 | 704 | } |
705 | 705 | } |
706 | 706 | |
@@ -766,7 +766,7 @@ discard block |
||
766 | 766 | $stop = $stop ? false : true; |
767 | 767 | } |
768 | 768 | |
769 | - $met[ $stop ] = $stop; |
|
769 | + $met[$stop] = $stop; |
|
770 | 770 | } |
771 | 771 | |
772 | 772 | if ( $notification['conditions']['any_all'] == 'all' && ! empty( $met ) && isset( $met[0] ) && isset( $met[1] ) ) { |
@@ -817,8 +817,8 @@ discard block |
||
817 | 817 | private static function get_value_from_entry( $entry, $field_id ) { |
818 | 818 | $observed_value = ''; |
819 | 819 | |
820 | - if ( isset( $entry->metas[ $field_id ] ) ) { |
|
821 | - $observed_value = $entry->metas[ $field_id ]; |
|
820 | + if ( isset( $entry->metas[$field_id] ) ) { |
|
821 | + $observed_value = $entry->metas[$field_id]; |
|
822 | 822 | } elseif ( $entry->post_id && FrmAppHelper::pro_is_installed() ) { |
823 | 823 | $field = FrmField::getOne( $field_id ); |
824 | 824 | $observed_value = FrmProEntryMetaHelper::get_post_or_meta_value( |
@@ -16,8 +16,8 @@ discard block |
||
16 | 16 | </li><?php |
17 | 17 | $render_icon = true; |
18 | 18 | foreach ( array( 20872734, 20874748, 20882522, 20874739, 20908981, 28109851 ) as $template ) { |
19 | - if ( isset( $templates[ $template ] ) ) { |
|
20 | - $template = $templates[ $template ]; |
|
19 | + if ( isset( $templates[$template] ) ) { |
|
20 | + $template = $templates[$template]; |
|
21 | 21 | require $view_path . 'list-template.php'; |
22 | 22 | } |
23 | 23 | } |
@@ -45,7 +45,7 @@ discard block |
||
45 | 45 | <ul class="frm-templates-list frm-categories-list"> |
46 | 46 | <?php foreach ( $categories as $category ) { ?> |
47 | 47 | <?php |
48 | - $category_templates = $templates_by_category[ $category ]; |
|
48 | + $category_templates = $templates_by_category[$category]; |
|
49 | 49 | $count = count( $category_templates ); |
50 | 50 | $available = FrmFormsHelper::available_count( $category_templates, $args ); |
51 | 51 | ?> |
@@ -14,7 +14,7 @@ discard block |
||
14 | 14 | return $errors; |
15 | 15 | } |
16 | 16 | |
17 | - 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' ) ) ) { |
|
17 | + 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' ) ) ) { |
|
18 | 18 | $frm_settings = FrmAppHelper::get_settings(); |
19 | 19 | $errors['form'] = $frm_settings->admin_permission; |
20 | 20 | } |
@@ -88,10 +88,10 @@ discard block |
||
88 | 88 | 'key_pointer' => '', // the pointer in the posted array |
89 | 89 | 'exclude' => array(), // exclude these field types from validation |
90 | 90 | ); |
91 | - $args = wp_parse_args( $args, $defaults ); |
|
91 | + $args = wp_parse_args( $args, $defaults ); |
|
92 | 92 | |
93 | 93 | if ( empty( $args['parent_field_id'] ) ) { |
94 | - $value = isset( $values['item_meta'][ $args['id'] ] ) ? $values['item_meta'][ $args['id'] ] : ''; |
|
94 | + $value = isset( $values['item_meta'][$args['id']] ) ? $values['item_meta'][$args['id']] : ''; |
|
95 | 95 | } else { |
96 | 96 | // value is from a nested form |
97 | 97 | $value = $values; |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | } |
113 | 113 | |
114 | 114 | if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) { |
115 | - $errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' ); |
|
115 | + $errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' ); |
|
116 | 116 | } elseif ( ! isset( $_POST['item_name'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
117 | 117 | self::maybe_add_item_name( $value, $posted_field ); |
118 | 118 | } |
@@ -188,7 +188,7 @@ discard block |
||
188 | 188 | $pattern = self::phone_format( $field ); |
189 | 189 | |
190 | 190 | if ( ! preg_match( $pattern, $value ) ) { |
191 | - $errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $field, 'invalid' ); |
|
191 | + $errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $field, 'invalid' ); |
|
192 | 192 | } |
193 | 193 | } |
194 | 194 | } |
@@ -298,7 +298,7 @@ discard block |
||
298 | 298 | */ |
299 | 299 | private static function form_is_in_progress( $values ) { |
300 | 300 | return FrmAppHelper::pro_is_installed() && |
301 | - ( isset( $values[ 'frm_page_order_' . $values['form_id'] ] ) || FrmAppHelper::get_post_param( 'frm_next_page' ) ) && |
|
301 | + ( isset( $values['frm_page_order_' . $values['form_id']] ) || FrmAppHelper::get_post_param( 'frm_next_page' ) ) && |
|
302 | 302 | FrmField::get_all_types_in_form( $values['form_id'], 'break' ); |
303 | 303 | } |
304 | 304 | |
@@ -543,9 +543,9 @@ discard block |
||
543 | 543 | foreach ( $datas['missing_keys'] as $key_index => $key ) { |
544 | 544 | $found = self::is_akismet_guest_info_value( $key, $value, $field_id, $datas['name_field_ids'] ); |
545 | 545 | if ( $found ) { |
546 | - $datas[ $key ] = $value; |
|
546 | + $datas[$key] = $value; |
|
547 | 547 | $datas['frm_duplicated'][] = $field_id; |
548 | - unset( $datas['missing_keys'][ $key_index ] ); |
|
548 | + unset( $datas['missing_keys'][$key_index] ); |
|
549 | 549 | } |
550 | 550 | } |
551 | 551 | } |
@@ -591,7 +591,7 @@ discard block |
||
591 | 591 | |
592 | 592 | // Send any potentially useful $_SERVER vars, but avoid sending junk we don't need. |
593 | 593 | if ( $include_value ) { |
594 | - $datas[ $key ] = $value; |
|
594 | + $datas[$key] = $value; |
|
595 | 595 | } |
596 | 596 | unset( $key, $value ); |
597 | 597 | } |
@@ -608,10 +608,10 @@ discard block |
||
608 | 608 | private static function add_comment_content_to_akismet( &$datas, $values ) { |
609 | 609 | if ( isset( $datas['frm_duplicated'] ) ) { |
610 | 610 | foreach ( $datas['frm_duplicated'] as $index ) { |
611 | - if ( isset( $values['item_meta'][ $index ] ) ) { |
|
612 | - unset( $values['item_meta'][ $index ] ); |
|
611 | + if ( isset( $values['item_meta'][$index] ) ) { |
|
612 | + unset( $values['item_meta'][$index] ); |
|
613 | 613 | } else { |
614 | - unset( $values[ $index ] ); |
|
614 | + unset( $values[$index] ); |
|
615 | 615 | } |
616 | 616 | } |
617 | 617 | unset( $datas['frm_duplicated'] ); |
@@ -632,14 +632,14 @@ discard block |
||
632 | 632 | private static function skip_adding_values_to_akismet( &$values ) { |
633 | 633 | $skipped_fields = self::get_akismet_skipped_field_ids( $values ); |
634 | 634 | foreach ( $skipped_fields as $skipped_field ) { |
635 | - if ( ! isset( $values['item_meta'][ $skipped_field->id ] ) ) { |
|
635 | + if ( ! isset( $values['item_meta'][$skipped_field->id] ) ) { |
|
636 | 636 | continue; |
637 | 637 | } |
638 | 638 | |
639 | 639 | if ( self::should_really_skip_field( $skipped_field, $values ) ) { |
640 | - unset( $values['item_meta'][ $skipped_field->id ] ); |
|
641 | - if ( isset( $values['item_meta']['other'][ $skipped_field->id ] ) ) { |
|
642 | - unset( $values['item_meta']['other'][ $skipped_field->id ] ); |
|
640 | + unset( $values['item_meta'][$skipped_field->id] ); |
|
641 | + if ( isset( $values['item_meta']['other'][$skipped_field->id] ) ) { |
|
642 | + unset( $values['item_meta']['other'][$skipped_field->id] ); |
|
643 | 643 | } |
644 | 644 | } |
645 | 645 | } |
@@ -672,14 +672,14 @@ discard block |
||
672 | 672 | } |
673 | 673 | |
674 | 674 | // If a choice field has Other option, but Other is not selected. |
675 | - if ( empty( $values['item_meta']['other'][ $field_data->id ] ) ) { |
|
675 | + if ( empty( $values['item_meta']['other'][$field_data->id] ) ) { |
|
676 | 676 | return true; |
677 | 677 | } |
678 | 678 | |
679 | 679 | // Check if submitted value is same as one of field option. |
680 | 680 | foreach ( $field_data->options as $option ) { |
681 | 681 | $option_value = ! is_array( $option ) ? $option : ( isset( $option['value'] ) ? $option['value'] : '' ); |
682 | - if ( $values['item_meta']['other'][ $field_data->id ] === $option_value ) { |
|
682 | + if ( $values['item_meta']['other'][$field_data->id] === $option_value ) { |
|
683 | 683 | return true; |
684 | 684 | } |
685 | 685 | } |
@@ -749,7 +749,7 @@ discard block |
||
749 | 749 | |
750 | 750 | // Convert name array to string. |
751 | 751 | if ( isset( $value['first'] ) && isset( $value['last'] ) ) { |
752 | - $values['item_meta'][ $field_id ] = trim( implode( ' ', $value ) ); |
|
752 | + $values['item_meta'][$field_id] = trim( implode( ' ', $value ) ); |
|
753 | 753 | $values['name_field_ids'][] = $field_id; |
754 | 754 | continue; |
755 | 755 | } |
@@ -770,8 +770,8 @@ discard block |
||
770 | 770 | continue; |
771 | 771 | } |
772 | 772 | |
773 | - if ( ! isset( $values['item_meta'][ $subsubindex ] ) ) { |
|
774 | - $values['item_meta'][ $subsubindex ] = array(); |
|
773 | + if ( ! isset( $values['item_meta'][$subsubindex] ) ) { |
|
774 | + $values['item_meta'][$subsubindex] = array(); |
|
775 | 775 | } |
776 | 776 | |
777 | 777 | // Convert name array to string. |
@@ -781,11 +781,11 @@ discard block |
||
781 | 781 | $values['name_field_ids'][] = $subsubindex; |
782 | 782 | } |
783 | 783 | |
784 | - $values['item_meta'][ $subsubindex ][] = $subsubvalue; |
|
784 | + $values['item_meta'][$subsubindex][] = $subsubvalue; |
|
785 | 785 | } |
786 | 786 | } |
787 | 787 | |
788 | - unset( $values['item_meta'][ $field_id ] ); |
|
788 | + unset( $values['item_meta'][$field_id] ); |
|
789 | 789 | } |
790 | 790 | |
791 | 791 | return $form_ids; |
@@ -113,7 +113,8 @@ discard block |
||
113 | 113 | |
114 | 114 | if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) { |
115 | 115 | $errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' ); |
116 | - } elseif ( ! isset( $_POST['item_name'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
116 | + } elseif ( ! isset( $_POST['item_name'] ) ) { |
|
117 | +// phpcs:ignore WordPress.Security.NonceVerification.Missing |
|
117 | 118 | self::maybe_add_item_name( $value, $posted_field ); |
118 | 119 | } |
119 | 120 | |
@@ -655,12 +656,14 @@ discard block |
||
655 | 656 | * @return bool |
656 | 657 | */ |
657 | 658 | private static function should_really_skip_field( $field_data, $values ) { |
658 | - if ( empty( $field_data->options ) ) { // This is skipped field types. |
|
659 | + if ( empty( $field_data->options ) ) { |
|
660 | +// This is skipped field types. |
|
659 | 661 | return true; |
660 | 662 | } |
661 | 663 | |
662 | 664 | FrmAppHelper::unserialize_or_decode( $field_data->options ); |
663 | - if ( ! $field_data->options ) { // Check if an error happens when unserializing, or empty options. |
|
665 | + if ( ! $field_data->options ) { |
|
666 | +// Check if an error happens when unserializing, or empty options. |
|
664 | 667 | return true; |
665 | 668 | } |
666 | 669 | |
@@ -743,7 +746,8 @@ discard block |
||
743 | 746 | // Blacklist check for File field in the old version doesn't contain `form_id`. |
744 | 747 | $form_ids = isset( $values['form_id'] ) ? array( absint( $values['form_id'] ) ) : array(); |
745 | 748 | foreach ( $values['item_meta'] as $field_id => $value ) { |
746 | - if ( ! is_numeric( $field_id ) ) { // Maybe `other`. |
|
749 | + if ( ! is_numeric( $field_id ) ) { |
|
750 | +// Maybe `other`. |
|
747 | 751 | continue; |
748 | 752 | } |
749 | 753 |