PluginProbe
Formidable Forms – WordPress Form Builder for Contact Forms, Calculators, Quizzes & More / 6.7
Formidable Forms – WordPress Form Builder for Contact Forms, Calculators, Quizzes & More v6.7
6.35 6.34 6.33.1 6.33 6.32.1 6.32 6.31 6.25 6.25.1 6.26 6.26.1 6.27 6.28 6.29 6.3 6.3.1 6.3.2 6.30 6.4 6.4.1 6.4.2 6.5 6.5.1 6.5.2 6.5.3 All 141 releases
← All changes | classes/models/FrmEntryValidate.php +113 -419 6.266.7 View file →
@@ -5,18 +5,10 @@
5 5
6 6 class FrmEntryValidate {
7 7
8 8 /**
9 - * @since 6.17
10 - *
11 - * @var array|null
12 - */
13 - private static $name_text_fields;
14 -
15 - /**
16 9 * @param array $values
17 - * @param bool|string[] $exclude
18 - *
10 + * @param string[]|bool $exclude
19 11 * @return array
20 12 */
21 13 public static function validate( $values, $exclude = false ) {
22 14 FrmEntry::sanitize_entry_post( $values );
@@ -63,9 +55,9 @@
63 55
64 56 if ( is_array( $filtered_errors ) ) {
65 57 $errors = $filtered_errors;
66 58 } else {
67 - _doing_it_wrong( __METHOD__, 'Only arrays should be returned when using the frm_validate_entry filter.', '6.3' );
59 + _doing_it_wrong( __FUNCTION__, 'Only arrays should be returned when using the frm_validate_entry filter.', '6.3' );
68 60 }
69 61
70 62 return $errors;
71 63 }
@@ -84,13 +76,8 @@
84 76 $_POST['item_meta'] = array();
85 77 }
86 78 }
87 79
88 - /**
89 - * @param array $values
90 - *
91 - * @return void
92 - */
93 80 private static function set_item_key( &$values ) {
94 81 if ( ! isset( $values['item_key'] ) || $values['item_key'] == '' ) {
95 82 global $wpdb;
96 83 $values['item_key'] = FrmAppHelper::get_unique_key( '', $wpdb->prefix . 'frm_items', 'item_key' );
@@ -97,14 +84,8 @@
97 84 $_POST['item_key'] = $values['item_key'];
98 85 }
99 86 }
100 87
101 - /**
102 - * @param array $values
103 - * @param array|string $exclude
104 - *
105 - * @return array
106 - */
107 88 private static function get_fields_to_validate( $values, $exclude ) {
108 89 $where = apply_filters( 'frm_posted_field_ids', array( 'fi.form_id' => $values['form_id'] ) );
109 90
110 91 // Don't get subfields
@@ -127,31 +108,19 @@
127 108 */
128 109 return apply_filters( 'frm_fields_to_validate', $fields, compact( 'values', 'exclude', 'where' ) );
129 110 }
130 111
131 - /**
132 - * @param object $posted_field
133 - * @param array $errors
134 - * @param array $values
135 - * @param array $args
136 - *
137 - * @return void
138 - */
139 112 public static function validate_field( $posted_field, &$errors, $values, $args = array() ) {
140 113 $defaults = array(
141 114 'id' => $posted_field->id,
142 - // The id of the repeat or embed form.
143 - 'parent_field_id' => '',
144 - // The pointer in the posted array.
145 - 'key_pointer' => '',
146 - // Exclude these field types from validation.
147 - 'exclude' => array(),
148 -
115 + 'parent_field_id' => '', // the id of the repeat or embed form
116 + 'key_pointer' => '', // the pointer in the posted array
117 + 'exclude' => array(), // exclude these field types from validation
149 118 );
150 - $args = wp_parse_args( $args, $defaults );
119 + $args = wp_parse_args( $args, $defaults );
151 120
152 121 if ( empty( $args['parent_field_id'] ) ) {
153 - $value = $values['item_meta'][ $args['id'] ] ?? '';
122 + $value = isset( $values['item_meta'][ $args['id'] ] ) ? $values['item_meta'][ $args['id'] ] : '';
154 123 } else {
155 124 // value is from a nested form
156 125 $value = $values;
157 126 }
@@ -161,9 +130,8 @@
161 130
162 131 self::maybe_clear_value_for_default_blank_setting( $posted_field, $value );
163 132
164 133 $should_trim = is_array( $value ) && count( $value ) == 1 && isset( $value[0] ) && $posted_field->type !== 'checkbox';
165 -
166 134 if ( $should_trim ) {
167 135 $value = reset( $value );
168 136 }
169 137
@@ -178,9 +146,8 @@
178 146 }
179 147
180 148 FrmEntriesHelper::set_posted_value( $posted_field, $value, $args );
181 149
182 - self::validate_options( $errors, $posted_field, $value, $args );
183 150 self::validate_field_types( $errors, $posted_field, $value, $args );
184 151
185 152 // Field might want to modify value before other parts of the system
186 153 // e.g. trim off excess values like in the case of fields with limit.
@@ -191,208 +158,19 @@
191 158 }
192 159
193 160 $errors = apply_filters( 'frm_validate_' . $posted_field->type . '_field_entry', $errors, $posted_field, $value, $args );
194 161 $errors = apply_filters( 'frm_validate_field_entry', $errors, $posted_field, $value, $args );
195 -
196 - if ( ! FrmAppHelper::pro_is_installed() && empty( $args['other'] ) ) {
197 - FrmEntriesHelper::get_posted_value( $posted_field, $value, $args );
198 - }
199 162 }
200 163
201 164 /**
202 - * @since 6.21
203 - *
204 - * @param array $errors
205 - * @param object $posted_field
206 - * @param array|string $value
207 - * @param array $args
208 - *
209 - * @return void
210 - */
211 - private static function validate_options( &$errors, $posted_field, $value, $args ) {
212 - if ( empty( $posted_field->options ) ) {
213 - return;
214 - }
215 -
216 - $option_is_valid = self::option_is_valid( $posted_field, $value, $posted_field->options );
217 -
218 - /**
219 - * @since 6.21
220 - *
221 - * @param bool $option_is_valid
222 - * @param array|string $value
223 - * @param object $field
224 - */
225 - $option_is_valid = (bool) apply_filters( 'frm_option_is_valid', $option_is_valid, $value, $posted_field );
226 -
227 - if ( ! $option_is_valid ) {
228 - $errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'invalid' );
229 - }
230 - }
231 -
232 - /**
233 - * Validate that value matches one of the options for the field.
234 - *
235 - * @since 6.21
236 - *
237 - * @param stdClass $field
238 - * @param array|string $value
239 - * @param array $options
240 - *
241 - * @return bool
242 - */
243 - private static function option_is_valid( $field, $value, $options ) {
244 - if ( '' === $value ) {
245 - return true;
246 - }
247 -
248 - $field_object = FrmFieldFactory::get_field_type( $field->type, $field );
249 -
250 - if ( ! $field_object->field_type_has_options_settings() ) {
251 - return true;
252 - }
253 -
254 - if ( in_array( $field->type, array( 'likert', 'ranking' ), true ) ) {
255 - // Ignore these field types automatically.
256 - return true;
257 - }
258 -
259 - if ( 'product' === $field->type && 'user_def' === FrmField::get_option( $field, 'data_type' ) ) {
260 - return true;
261 - }
262 -
263 - if ( ! empty( $field->field_options['post_field'] ) ) {
264 - return true;
265 - }
266 -
267 - $value = (array) $value;
268 -
269 - foreach ( $value as $current_value ) {
270 - $match = false;
271 -
272 - foreach ( $options as $key => $option ) {
273 - if ( strpos( $key, 'other_' ) === 0 ) {
274 - // Always return true if an other option is found.
275 - return true;
276 - }
277 -
278 - if ( is_array( $option ) ) {
279 - $separate_value = FrmField::get_option( $field, 'separate_value' );
280 - $option_value = $separate_value ? $option['value'] : $option['label'];
281 - } else {
282 - $option_value = $option;
283 - }
284 -
285 - $match = trim( $current_value ) === trim( $option_value );
286 -
287 - if ( $match ) {
288 - break;
289 - }
290 -
291 - $match = trim( $current_value ) === trim( do_shortcode( $option_value ) );
292 -
293 - if ( $match ) {
294 - break;
295 - }
296 -
297 - $match = self::is_filtered_match( $current_value, $option_value );
298 -
299 - if ( $match ) {
300 - break;
301 - }
302 -
303 - if ( is_numeric( $current_value ) ) {
304 - $match = (int) $current_value === (int) $option_value;
305 -
306 - if ( $match ) {
307 - break;
308 - }
309 - }
310 - }//end foreach
311 -
312 - if ( ! $match ) {
313 - return self::options_are_dynamic_based_on_hook( $field, $value );
314 - }
315 - }//end foreach
316 -
317 - return true;
318 - }
319 -
320 - /**
321 - * Make an extra check after passing $option_value through the_content filter.
322 - * This is to help catch cases where the option's formatting has been modified using
323 - * the_content filter.
324 - *
325 - * @since 6.22
326 - *
327 - * @param string $value
328 - * @param string $option_value
329 - *
330 - * @return bool
331 - */
332 - private static function is_filtered_match( $value, $option_value ) {
333 - // First remove the wpautop filter so it doesn't add extra tags to $option_value.
334 - $filter_priority = has_filter( 'the_content', 'wpautop' );
335 -
336 - if ( is_numeric( $filter_priority ) ) {
337 - remove_filter( 'the_content', 'wpautop', $filter_priority );
338 - }
339 -
340 - $filtered_option = apply_filters( 'the_content', $option_value );
341 -
342 - if ( is_numeric( $filter_priority ) ) {
343 - add_filter( 'the_content', 'wpautop', $filter_priority );
344 - }
345 - return trim( $value ) === trim( $filtered_option );
346 - }
347 -
348 - /**
349 - * Do not validate options if they have been modified with a hook.
350 - * This is to help avoid issues where the options could be based on a URL param for example.
351 - *
352 - * @since 6.21
353 - *
354 - * @param object $field_object The field object.
355 - * @param array|string $value The value to validate.
356 - *
357 - * @return bool
358 - */
359 - private static function options_are_dynamic_based_on_hook( $field_object, $value ) {
360 - $values = (array) $field_object;
361 - $values['value'] = $value;
362 - FrmFieldsHelper::prepare_new_front_field( $values, $field_object );
363 -
364 - $separate_value = FrmField::get_option( $field_object, 'separate_value' );
365 - $map_callback = function ( $option ) use ( $separate_value ) {
366 - if ( is_array( $option ) ) {
367 - $option_value = $separate_value ? $option['value'] : $option['label'];
368 - } else {
369 - $option_value = $option;
370 - }
371 -
372 - $option_value = do_shortcode( $option_value );
373 - return $option_value;
374 - };
375 -
376 - $values_options = array_map( $map_callback, $values['options'] );
377 - $field_object_options = array_map( $map_callback, $field_object->options );
378 -
379 - return $values_options !== $field_object_options;
380 - }
381 -
382 - /**
383 165 * Maybe add item_name to $_POST to save it in items table.
384 166 *
385 167 * @since 5.2.02
386 168 *
387 - * @param array|string $value Field value.
388 - * @param object $field Field object.
389 - *
390 - * @return void
169 + * @param object $field Field object.
391 170 */
392 171 private static function maybe_add_item_name( $value, $field ) {
393 172 $item_name = false;
394 -
395 173 if ( 'name' === $field->type ) {
396 174 $field_obj = FrmFieldFactory::get_field_object( $field );
397 175 $item_name = $field_obj->get_display_value( $value );
398 176 } elseif ( 'text' === $field->type ) {
@@ -409,14 +187,11 @@
409 187 * Set $value to an empty string if it matches its label
410 188 *
411 189 * @param object $field
412 190 * @param string $value
413 - *
414 - * @return void
415 191 */
416 192 private static function maybe_clear_value_for_default_blank_setting( $field, &$value ) {
417 193 $position = FrmField::get_option( $field, 'label' );
418 -
419 194 if ( ! $position ) {
420 195 $position = FrmStylesController::get_style_val( 'position', $field->form_id );
421 196 }
422 197
@@ -424,16 +199,8 @@
424 199 $value = '';
425 200 }
426 201 }
427 202
428 - /**
429 - * @param array $errors
430 - * @param object $posted_field
431 - * @param mixed $value
432 - * @param array $args
433 - *
434 - * @return void
435 - */
436 203 public static function validate_field_types( &$errors, $posted_field, $value, $args ) {
437 204 $field_obj = FrmFieldFactory::get_field_object( $posted_field );
438 205 $args['value'] = $value;
439 206 $args['errors'] = $errors;
@@ -438,26 +205,16 @@
438 205 $args['value'] = $value;
439 206 $args['errors'] = $errors;
440 207
441 208 $new_errors = $field_obj->validate( $args );
442 -
443 209 if ( ! empty( $new_errors ) ) {
444 210 $errors = array_merge( $errors, $new_errors );
445 211 }
446 212 }
447 213
448 - /**
449 - * @param array $errors
450 - * @param object $field
451 - * @param string $value
452 - * @param array $args
453 - *
454 - * @return void
455 - */
456 214 public static function validate_phone_field( &$errors, $field, $value, $args ) {
457 - $format_value = FrmField::get_option( $field, 'format' );
215 + if ( $field->type == 'phone' || ( $field->type == 'text' && FrmField::is_option_true_in_object( $field, 'format' ) ) ) {
458 216
459 - if ( $field->type === 'phone' || ( $field->type === 'text' && $format_value && ! FrmCurrencyHelper::is_currency_format( $format_value ) ) ) {
460 217 $pattern = self::phone_format( $field );
461 218
462 219 if ( ! preg_match( $pattern, $value ) ) {
463 220 $errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $field, 'invalid' );
@@ -464,13 +221,8 @@
464 221 }
465 222 }
466 223 }
467 224
468 - /**
469 - * @param object $field
470 - *
471 - * @return string
472 - */
473 225 public static function phone_format( $field ) {
474 226 if ( FrmField::is_option_empty( $field, 'format' ) ) {
475 227 $pattern = self::default_phone_format();
476 228 } else {
@@ -476,11 +228,8 @@
476 228 } else {
477 229 $pattern = FrmField::get_option( $field, 'format' );
478 230 }
479 231
480 - // Ampersands are saved as &.
481 - // Reverse it here so we are checking for the correct character.
482 - $pattern = html_entity_decode( $pattern );
483 232 $pattern = apply_filters( 'frm_phone_pattern', $pattern, $field );
484 233
485 234 // Create a regexp if format is not already a regexp
486 235 if ( strpos( $pattern, '^' ) !== 0 ) {
@@ -493,10 +242,8 @@
493 242 }
494 243
495 244 /**
496 245 * @since 3.01
497 - *
498 - * @return string
499 246 */
500 247 private static function default_phone_format() {
501 248 return '^((\+\d{1,3}(-|.| )?\(?\d\)?(-| |.)?\d{1,5})|(\(?\d{2,6}\)?))(-|.| )?(\d{3,4})(-|.| )?(\d{4})(( x| ext)\d{1,5}){0,1}$';
502 249 }
@@ -525,9 +272,8 @@
525 272
526 273 if ( strpos( $pattern, '\?' ) !== false ) {
527 274 $parts = explode( '\?', $pattern );
528 275 $pattern = '';
529 -
530 276 foreach ( $parts as $part ) {
531 277 if ( empty( $pattern ) ) {
532 278 $pattern .= $part;
533 279 } else {
@@ -534,9 +280,8 @@
534 280 $pattern .= '(' . $part . ')?';
535 281 }
536 282 }
537 283 }
538 -
539 284 $pattern = '^' . $pattern . '$';
540 285
541 286 return $pattern;
542 287 }
@@ -541,40 +286,27 @@
541 286 return $pattern;
542 287 }
543 288
544 289 /**
545 - * Check for spam.
290 + * Check for spam
546 291 *
547 - * @param bool $exclude
292 + * @param boolean $exclude
548 293 * @param array $values
549 - * @param array $errors By reference.
550 - *
551 - * @return void
294 + * @param array $errors by reference
552 295 */
553 296 public static function spam_check( $exclude, $values, &$errors ) {
554 - if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING ) {
555 - // Do not check spam on importing.
556 - return;
557 - }
558 -
559 - if ( ! empty( $exclude ) || empty( $values['item_meta'] ) || ! empty( $errors ) ) {
297 + if ( ! empty( $exclude ) || ! isset( $values['item_meta'] ) || empty( $values['item_meta'] ) || ! empty( $errors ) ) {
560 298 // only check spam if there are no other errors
561 299 return;
562 300 }
563 301
564 302 $antispam_check = self::is_antispam_check( $values['form_id'] );
565 - $spam_msg = FrmAntiSpamController::get_default_spam_message();
566 -
567 303 if ( is_string( $antispam_check ) ) {
568 304 $errors['spam'] = $antispam_check;
569 305 } elseif ( self::is_honeypot_spam( $values ) || self::is_spam_bot() ) {
570 - $errors['spam'] = $spam_msg;
571 - } else {
572 - $is_spam = FrmAntiSpamController::is_spam( $values );
573 -
574 - if ( $is_spam ) {
575 - $errors['spam'] = $is_spam;
576 - }
306 + $errors['spam'] = __( 'Your entry appears to be spam!', 'formidable' );
307 + } elseif ( self::blacklist_check( $values ) ) {
308 + $errors['spam'] = __( 'Your entry appears to be blocked spam!', 'formidable' );
577 309 }
578 310
579 311 if ( isset( $errors['spam'] ) || self::form_is_in_progress( $values ) ) {
580 312 return;
@@ -590,9 +322,8 @@
590 322 *
591 323 * @since 5.0.13
592 324 *
593 325 * @param array $values The values.
594 - *
595 326 * @return bool
596 327 */
597 328 private static function form_is_in_progress( $values ) {
598 329 return FrmAppHelper::pro_is_installed() &&
@@ -611,10 +342,9 @@
611 342 }
612 343
613 344 /**
614 345 * @param array $values
615 - *
616 - * @return bool
346 + * @return boolean
617 347 */
618 348 private static function is_honeypot_spam( $values ) {
619 349 $honeypot = new FrmHoneypot( $values['form_id'] );
620 350 return ! $honeypot->validate();
@@ -620,9 +350,9 @@
620 350 return ! $honeypot->validate();
621 351 }
622 352
623 353 /**
624 - * @return bool
354 + * @return boolean
625 355 */
626 356 private static function is_spam_bot() {
627 357 $ip = FrmAppHelper::get_ip_address();
628 358
@@ -630,10 +360,9 @@
630 360 }
631 361
632 362 /**
633 363 * @param array $values
634 - *
635 - * @return bool
364 + * @return boolean
636 365 */
637 366 private static function is_akismet_spam( $values ) {
638 367 global $wpcom_api_key;
639 368
@@ -641,9 +370,8 @@
641 370 }
642 371
643 372 /**
644 373 * @param int $form_id
645 - *
646 374 * @return bool
647 375 */
648 376 private static function is_akismet_enabled_for_user( $form_id ) {
649 377 $form = FrmForm::getOne( $form_id );
@@ -650,25 +378,59 @@
650 378
651 379 return ( ! empty( $form->options['akismet'] ) && ( $form->options['akismet'] !== 'logged' || ! is_user_logged_in() ) );
652 380 }
653 381
382 + public static function blacklist_check( $values ) {
383 + if ( ! apply_filters( 'frm_check_blacklist', true, $values ) ) {
384 + return false;
385 + }
386 +
387 + $mod_keys = trim( self::get_disallowed_words() );
388 + if ( empty( $mod_keys ) ) {
389 + return false;
390 + }
391 +
392 + $content = FrmEntriesHelper::entry_array_to_string( $values );
393 +
394 + self::prepare_values_for_spam_check( $values );
395 + $ip = FrmAppHelper::get_ip_address();
396 + $user_agent = FrmAppHelper::get_server_value( 'HTTP_USER_AGENT' );
397 + $user_info = self::get_spam_check_user_info( $values );
398 +
399 + return self::check_disallowed_words( $user_info['comment_author'], $user_info['comment_author_email'], $user_info['comment_author_url'], $content, $ip, $user_agent );
400 + }
401 +
654 402 /**
655 - * Checks spam using WordPress disallowed words and Frm denylist.
403 + * For WP 5.5 compatibility.
656 404 *
657 - * @param array $values Entry values.
405 + * @since 4.06.02
406 + */
407 + private static function check_disallowed_words( $author, $email, $url, $content, $ip, $user_agent ) {
408 + if ( function_exists( 'wp_check_comment_disallowed_list' ) ) {
409 + return wp_check_comment_disallowed_list( $author, $email, $url, $content, $ip, $user_agent );
410 + } else {
411 + return wp_blacklist_check( $author, $email, $url, $content, $ip, $user_agent );
412 + }
413 + }
414 +
415 + /**
416 + * For WP 5.5 compatibility.
658 417 *
659 - * @return bool
418 + * @since 4.06.02
660 419 */
661 - public static function blacklist_check( $values ) {
662 - return FrmAntiSpamController::contains_wp_disallowed_words( $values ) || FrmAntiSpamController::is_denylist_spam( $values );
420 + private static function get_disallowed_words() {
421 + $keys = get_option( 'disallowed_keys' );
422 + if ( false === $keys ) {
423 + // Fallback for WP < 5.5.
424 + $keys = get_option( 'blacklist_keys' );
425 + }
426 + return $keys;
663 427 }
664 428
665 429 /**
666 430 * Check entries for Akismet spam
667 431 *
668 - * @param array $values Entry values.
669 - *
670 - * @return bool true if is spam
432 + * @return boolean true if is spam
671 433 */
672 434 public static function akismet( $values ) {
673 435 if ( empty( $values['item_meta'] ) ) {
674 436 return false;
@@ -690,18 +452,13 @@
690 452
691 453 $query_string = _http_build_query( $datas, '', '&' );
692 454 $response = Akismet::http_post( $query_string, 'comment-check' );
693 455
694 - return ( is_array( $response ) && $response[1] === 'true' );
456 + return ( is_array( $response ) && $response[1] == 'true' );
695 457 }
696 458
697 459 /**
698 460 * @since 2.0
699 - *
700 - * @param array $datas The array of values being sent to Akismet.
701 - * @param array $values Entry values.
702 - *
703 - * @return void
704 461 */
705 462 private static function parse_akismet_array( &$datas, $values ) {
706 463 self::add_site_info_to_akismet( $datas );
707 464 self::add_server_values_to_akismet( $datas );
@@ -706,19 +463,13 @@
706 463 self::add_site_info_to_akismet( $datas );
707 464 self::add_server_values_to_akismet( $datas );
708 465
709 466 self::prepare_values_for_spam_check( $values );
710 - self::skip_adding_values_to_akismet( $values );
711 467
712 468 self::add_user_info_to_akismet( $datas, $values );
713 469 self::add_comment_content_to_akismet( $datas, $values );
714 470 }
715 471
716 - /**
717 - * @param array $datas
718 - *
719 - * @return void
720 - */
721 472 private static function add_site_info_to_akismet( &$datas ) {
722 473 $datas['blog'] = FrmAppHelper::site_url();
723 474 $datas['user_ip'] = preg_replace( '/[^0-9., ]/', '', FrmAppHelper::get_ip_address() );
724 475 $datas['user_agent'] = FrmAppHelper::get_server_value( 'HTTP_USER_AGENT' );
@@ -730,14 +481,8 @@
730 481 $datas['is_test'] = 'true';
731 482 }
732 483 }
733 484
734 - /**
735 - * @param array $datas
736 - * @param array $values
737 - *
738 - * @return void
739 - */
740 485 private static function add_user_info_to_akismet( &$datas, $values ) {
741 486 $user_info = self::get_spam_check_user_info( $values );
742 487 $datas = $datas + $user_info;
743 488
@@ -749,15 +494,13 @@
749 494 /**
750 495 * Gets user info for Akismet spam check.
751 496 *
752 497 * @since 5.0.13 Separate code for guest. Handle value of embedded|repeater.
753 - * @since 6.21 This changed from private to public.
754 498 *
755 499 * @param array $values Entry values after running through {@see FrmEntryValidate::prepare_values_for_spam_check()}.
756 - *
757 500 * @return array
758 501 */
759 - public static function get_spam_check_user_info( $values ) {
502 + private static function get_spam_check_user_info( $values ) {
760 503 if ( ! is_user_logged_in() ) {
761 504 return self::get_spam_check_user_info_for_guest( $values );
762 505 }
763 506
@@ -777,9 +520,8 @@
777 520 *
778 521 * @since 5.0.13
779 522 *
780 523 * @param array $values Entry values after flattened.
781 - *
782 524 * @return array
783 525 */
784 526 private static function get_spam_check_user_info_for_guest( $values ) {
785 527 $datas = array(
@@ -811,16 +553,13 @@
811 553 *
812 554 * @param array $datas Guest data.
813 555 * @param array $values The values.
814 556 * @param int|null $custom_index Custom index (or field ID).
815 - *
816 - * @return void
817 557 */
818 558 private static function recursive_add_akismet_guest_info( &$datas, $values, $custom_index = null ) {
819 559 foreach ( $values as $index => $value ) {
820 560 if ( ! $datas['missing_keys'] ) {
821 - // Found all info.
822 - return;
561 + return; // Found all info.
823 562 }
824 563
825 564 if ( is_array( $value ) ) {
826 565 self::recursive_add_akismet_guest_info( $datas, $value, $index );
@@ -827,12 +566,10 @@
827 566 continue;
828 567 }
829 568
830 569 $field_id = ! is_null( $custom_index ) ? $custom_index : $index;
831 -
832 570 foreach ( $datas['missing_keys'] as $key_index => $key ) {
833 - $found = self::is_akismet_guest_info_value( $key, $value, $field_id, $datas['name_field_ids'], $values );
834 -
571 + $found = self::is_akismet_guest_info_value( $key, $value, $field_id, $datas['name_field_ids'] );
835 572 if ( $found ) {
836 573 $datas[ $key ] = $value;
837 574 $datas['frm_duplicated'][] = $field_id;
838 575 unset( $datas['missing_keys'][ $key_index ] );
@@ -837,9 +574,9 @@
837 574 $datas['frm_duplicated'][] = $field_id;
838 575 unset( $datas['missing_keys'][ $key_index ] );
839 576 }
840 577 }
841 - }//end foreach
578 + }
842 579 }
843 580
844 581 /**
845 582 * Checks if given value is an akismet guest info.
@@ -849,13 +586,11 @@
849 586 * @param string $key Guest info key.
850 587 * @param string $value Value to check.
851 588 * @param int $field_id Field ID.
852 589 * @param array $name_field_ids Name field IDs.
853 - * @param array $values Array of posted values.
854 - *
855 590 * @return bool
856 591 */
857 - private static function is_akismet_guest_info_value( $key, &$value, $field_id, $name_field_ids, $values ) {
592 + private static function is_akismet_guest_info_value( $key, $value, $field_id, $name_field_ids ) {
858 593 if ( ! $value || is_numeric( $value ) ) {
859 594 return false;
860 595 }
861 596
@@ -866,73 +601,18 @@
866 601 case 'comment_author_url':
867 602 return 0 === strpos( $value, 'http' );
868 603
869 604 case 'comment_author':
870 - if ( $name_field_ids && in_array( $field_id, $name_field_ids, true ) ) {
605 + if ( $name_field_ids ) {
871 606 // If there is name field in the form, we should always use it as author name.
872 - return true;
607 + return in_array( $field_id, $name_field_ids, true );
873 608 }
609 + return strlen( $value ) < 200;
610 + }
874 611
875 - $form_id = FrmAppHelper::get_post_param( 'form_id', 0, 'absint' );
876 - $fields = self::get_name_text_fields( $form_id );
877 -
878 - foreach ( $fields as $index => $field ) {
879 - if ( 'Name' !== $field->name ) {
880 - continue;
881 - }
882 -
883 - if ( isset( $fields[ $index + 1 ] ) && 'Last' === $fields[ $index + 1 ]->name ) {
884 - if ( empty( $values[ absint( $fields[ $index + 1 ]->id ) ] ) ) {
885 - continue;
886 - }
887 -
888 - $value .= ' ' . $values[ $fields[ $index + 1 ]->id ];
889 - return true;
890 - }
891 - }
892 - }//end switch
893 -
894 612 return false;
895 613 }
896 614
897 - /**
898 - * Returns fields that have 'Name' and 'Last' as their name.
899 - *
900 - * @since 6.17
901 - *
902 - * @param int $form_id
903 - *
904 - * @return array
905 - */
906 - private static function get_name_text_fields( $form_id ) {
907 - $name_text_fields_is_initialized = is_array( self::$name_text_fields );
908 -
909 - if ( $name_text_fields_is_initialized && isset( self::$name_text_fields[ $form_id ] ) ) {
910 - return self::$name_text_fields[ $form_id ];
911 - }
912 -
913 - if ( ! $name_text_fields_is_initialized ) {
914 - self::$name_text_fields = array();
915 - }
916 - self::$name_text_fields[ $form_id ] = FrmDb::get_results(
917 - 'frm_fields',
918 - array(
919 - 'form_id' => $form_id,
920 - 'type' => 'text',
921 - 'name' => array( 'Name', 'Last' ),
922 - ),
923 - 'id,name',
924 - array( 'order_by' => 'field_order ASC' )
925 - );
926 -
927 - return self::$name_text_fields[ $form_id ];
928 - }
929 -
930 - /**
931 - * @param array $datas
932 - *
933 - * @return void
934 - */
935 615 private static function add_server_values_to_akismet( &$datas ) {
936 616 foreach ( $_SERVER as $key => $value ) {
937 617 $include_value = is_string( $value ) && ! preg_match( '/^HTTP_COOKIE/', $key ) && preg_match( '/^(HTTP_|REMOTE_ADDR|REQUEST_URI|DOCUMENT_URI)/', $key );
938 618
@@ -950,10 +630,8 @@
950 630 * @since 5.0.09
951 631 *
952 632 * @param array $datas The array of values being sent to Akismet.
953 633 * @param array $values Entry values.
954 - *
955 - * @return void
956 634 */
957 635 private static function add_comment_content_to_akismet( &$datas, $values ) {
958 636 if ( isset( $datas['frm_duplicated'] ) ) {
959 637 foreach ( $datas['frm_duplicated'] as $index ) {
@@ -965,8 +643,10 @@
965 643 }
966 644 unset( $datas['frm_duplicated'] );
967 645 }
968 646
647 + self::skip_adding_values_to_akismet( $values );
648 +
969 649 $datas['comment_content'] = FrmEntriesHelper::entry_array_to_string( $values );
970 650 }
971 651
972 652 /**
@@ -974,14 +654,11 @@
974 654 *
975 655 * @since 5.0.09
976 656 *
977 657 * @param array $values Entry values.
978 - *
979 - * @return void
980 658 */
981 659 private static function skip_adding_values_to_akismet( &$values ) {
982 660 $skipped_fields = self::get_akismet_skipped_field_ids( $values );
983 -
984 661 foreach ( $skipped_fields as $skipped_field ) {
985 662 if ( ! isset( $values['item_meta'][ $skipped_field->id ] ) ) {
986 663 continue;
987 664 }
@@ -987,9 +664,8 @@
987 664 }
988 665
989 666 if ( self::should_really_skip_field( $skipped_field, $values ) ) {
990 667 unset( $values['item_meta'][ $skipped_field->id ] );
991 -
992 668 if ( isset( $values['item_meta']['other'][ $skipped_field->id ] ) ) {
993 669 unset( $values['item_meta']['other'][ $skipped_field->id ] );
994 670 }
995 671 }
@@ -1002,21 +678,17 @@
1002 678 * @since 5.02.04
1003 679 *
1004 680 * @param object $field_data Object contains `id` and `options`.
1005 681 * @param array $values Entry values.
1006 - *
1007 682 * @return bool
1008 683 */
1009 684 private static function should_really_skip_field( $field_data, $values ) {
1010 - if ( empty( $field_data->options ) ) {
1011 - // This is skipped field types.
685 + if ( empty( $field_data->options ) ) { // This is skipped field types.
1012 686 return true;
1013 687 }
1014 688
1015 689 FrmAppHelper::unserialize_or_decode( $field_data->options );
1016 -
1017 - if ( ! $field_data->options ) {
1018 - // Check if an error happens when unserializing, or empty options.
690 + if ( ! $field_data->options ) { // Check if an error happens when unserializing, or empty options.
1019 691 return true;
1020 692 }
1021 693
1022 694 end( $field_data->options );
@@ -1033,10 +705,9 @@
1033 705 }
1034 706
1035 707 // Check if submitted value is same as one of field option.
1036 708 foreach ( $field_data->options as $option ) {
1037 - $option_value = ! is_array( $option ) ? $option : ( $option['value'] ?? '' );
1038 -
709 + $option_value = ! is_array( $option ) ? $option : ( isset( $option['value'] ) ? $option['value'] : '' );
1039 710 if ( $values['item_meta']['other'][ $field_data->id ] === $option_value ) {
1040 711 return true;
1041 712 }
1042 713 }
@@ -1051,9 +722,8 @@
1051 722 * @since 5.0.13 Move out get_all_form_ids_and_flatten_meta() call and get `form_ids` from `$values`.
1052 723 * @since 5.2.04 This method returns array of object contains `id` and `options` instead of array of `id` only.
1053 724 *
1054 725 * @param array $values Entry values after running through {@see FrmEntryValidate::prepare_values_for_spam_check()}.
1055 - *
1056 726 * @return array
1057 727 */
1058 728 private static function get_akismet_skipped_field_ids( $values ) {
1059 729 if ( empty( $values['form_ids'] ) ) {
@@ -1076,15 +746,12 @@
1076 746 /**
1077 747 * Prepares values array for spam check.
1078 748 *
1079 749 * @since 5.0.13
1080 - * @since 6.21 This changed from private to public.
1081 750 *
1082 751 * @param array $values Entry values.
1083 - *
1084 - * @return void
1085 752 */
1086 - public static function prepare_values_for_spam_check( &$values ) {
753 + private static function prepare_values_for_spam_check( &$values ) {
1087 754 $form_ids = self::get_all_form_ids_and_flatten_meta( $values );
1088 755 $values['form_ids'] = $form_ids;
1089 756 }
1090 757
@@ -1095,9 +762,8 @@
1095 762 * @since 5.0.09
1096 763 * @since 5.0.13 Convert name field value to string.
1097 764 *
1098 765 * @param array $values Entry values.
1099 - *
1100 766 * @return array Form IDs.
1101 767 */
1102 768 private static function get_all_form_ids_and_flatten_meta( &$values ) {
1103 769 $values['name_field_ids'] = array();
@@ -1103,12 +769,10 @@
1103 769 $values['name_field_ids'] = array();
1104 770
1105 771 // Blacklist check for File field in the old version doesn't contain `form_id`.
1106 772 $form_ids = isset( $values['form_id'] ) ? array( absint( $values['form_id'] ) ) : array();
1107 -
1108 773 foreach ( $values['item_meta'] as $field_id => $value ) {
1109 - if ( ! is_numeric( $field_id ) ) {
1110 - // Maybe `other`.
774 + if ( ! is_numeric( $field_id ) ) { // Maybe `other`.
1111 775 continue;
1112 776 }
1113 777
1114 778 // Convert name array to string.
@@ -1144,16 +808,46 @@
1144 808
1145 809 $values['name_field_ids'][] = $subsubindex;
1146 810 }
1147 811
1148 - if ( is_array( $values['item_meta'][ $subsubindex ] ) ) {
1149 - $values['item_meta'][ $subsubindex ][] = $subsubvalue;
1150 - }
812 + $values['item_meta'][ $subsubindex ][] = $subsubvalue;
1151 813 }
1152 - }//end foreach
814 + }
1153 815
1154 816 unset( $values['item_meta'][ $field_id ] );
1155 - }//end foreach
817 + }
1156 818
1157 819 return $form_ids;
820 + }
821 +
822 + /**
823 + * @deprecated 3.0
824 + * @codeCoverageIgnore
825 + */
826 + public static function validate_url_field( &$errors, $field, $value, $args ) {
827 + FrmDeprecated::validate_url_field( $errors, $field, $value, $args );
828 + }
829 +
830 + /**
831 + * @deprecated 3.0
832 + * @codeCoverageIgnore
833 + */
834 + public static function validate_email_field( &$errors, $field, $value, $args ) {
835 + FrmDeprecated::validate_email_field( $errors, $field, $value, $args );
836 + }
837 +
838 + /**
839 + * @deprecated 3.0
840 + * @codeCoverageIgnore
841 + */
842 + public static function validate_number_field( &$errors, $field, $value, $args ) {
843 + FrmDeprecated::validate_number_field( $errors, $field, $value, $args );
844 + }
845 +
846 + /**
847 + * @deprecated 3.0
848 + * @codeCoverageIgnore
849 + */
850 + public static function validate_recaptcha( &$errors, $field, $args ) {
851 + FrmDeprecated::validate_recaptcha( $errors, $field, $args );
1158 852 }
1159 853 }