Form submission filter flaw (or feature request ;-))

HI Guys,

In the “class-form-generator.php” file, starting at line 794 you have a filter callback that allows us to change the value of $proceed (true/false). However, you have code immediately following the callback that can alter the value of $proceed (line 798). So if I want to prevent emails that contain certain words from being sent and I return false from my filter callback when a bad word is detected, $this->check_recaptcha_token( $new_post[‘label_input’] ) could potentially switch $proceed back to true and render my filter useless:

			//hook to stop execution here and do something different with the data
			$proceed = apply_filters( 'avf_form_send', true, $new_post, $this->form_params, $this );
			
			if( $this->is_recaptcha() ) {
				$proceed = $this->check_recaptcha_token( $new_post['label_input'] ); // <--- this line renders any change to the value of $proceed in the callback filter useless!
				
				if( $proceed ) {
					delete_transient( 'avia_recaptcha_transient_' . $proceed );
				}
			}
			if( ! $proceed )
			{
				if( is_null( $proceed ) )
				{
					return false;
				}
				else
				{
					return true;
				}
			}

The filter can only return true or false, so the only thing I can do is exit(); instead of allowing Enfold’s process for preventing the submission.

Great product. I have purchased several installs because it’s so versatile! Keep up the good work.

Thanks,
Andy