Strategy11 / formidable-forms

classes/models/FrmNotification.php

Indentation +229 added lines, -229 removed lines

@@ -2,128 +2,128 @@ 
 
 class FrmNotification {
 	public function __construct() {
-        if ( ! defined('ABSPATH') ) {
-            die('You are not allowed to call this page directly.');
-        }
-        add_action('frm_trigger_email_action', 'FrmNotification::trigger_email', 10, 3);
-    }
+		if ( ! defined('ABSPATH') ) {
+			die('You are not allowed to call this page directly.');
+		}
+		add_action('frm_trigger_email_action', 'FrmNotification::trigger_email', 10, 3);
+	}
 
 	public static function trigger_email( $action, $entry, $form ) {
-        $notification = $action->post_content;
-        $email_key = $action->ID;
-
-        // Set the subject
-        if ( empty($notification['email_subject']) ) {
-            $notification['email_subject'] = sprintf(__( '%1$s Form submitted on %2$s', 'formidable' ), $form->name, '[sitename]');
-        }
-
-        $plain_text = $notification['plain_text'] ? true : false;
-
-        //Filter these fields
-        $filter_fields = array(
-            'email_to', 'cc', 'bcc',
-            'reply_to', 'from',
-            'email_subject', 'email_message',
-        );
-
-        add_filter('frm_plain_text_email', ($plain_text ? '__return_true' : '__return_false'));
-
-        //Get all values in entry in order to get User ID field ID
-        $values = FrmEntryMeta::getAll( array( 'it.field_id !' => 0, 'it.item_id' => $entry->id ), ' ORDER BY fi.field_order' );
-        $user_id_field = $user_id_key = '';
-        foreach ( $values as $value ) {
-            if ( $value->field_type == 'user_id' ) {
-                $user_id_field = $value->field_id;
-                $user_id_key = $value->field_key;
-                break;
-            }
-            unset($value);
-        }
-
-        //Filter and prepare the email fields
-        foreach ( $filter_fields as $f ) {
-            //Don't allow empty From
+		$notification = $action->post_content;
+		$email_key = $action->ID;
+
+		// Set the subject
+		if ( empty($notification['email_subject']) ) {
+			$notification['email_subject'] = sprintf(__( '%1$s Form submitted on %2$s', 'formidable' ), $form->name, '[sitename]');
+		}
+
+		$plain_text = $notification['plain_text'] ? true : false;
+
+		//Filter these fields
+		$filter_fields = array(
+			'email_to', 'cc', 'bcc',
+			'reply_to', 'from',
+			'email_subject', 'email_message',
+		);
+
+		add_filter('frm_plain_text_email', ($plain_text ? '__return_true' : '__return_false'));
+
+		//Get all values in entry in order to get User ID field ID
+		$values = FrmEntryMeta::getAll( array( 'it.field_id !' => 0, 'it.item_id' => $entry->id ), ' ORDER BY fi.field_order' );
+		$user_id_field = $user_id_key = '';
+		foreach ( $values as $value ) {
+			if ( $value->field_type == 'user_id' ) {
+				$user_id_field = $value->field_id;
+				$user_id_key = $value->field_key;
+				break;
+			}
+			unset($value);
+		}
+
+		//Filter and prepare the email fields
+		foreach ( $filter_fields as $f ) {
+			//Don't allow empty From
 			if ( $f == 'from' && empty( $notification[ $f ] ) ) {
 				$notification[ $f ] = '[admin_email]';
 			} else if ( in_array( $f, array( 'email_to', 'cc', 'bcc', 'reply_to', 'from' ) ) ) {
 				//Remove brackets
-                //Add a space in case there isn't one
+				//Add a space in case there isn't one
 				$notification[ $f ] = str_replace( '<', ' ', $notification[ $f ] );
 				$notification[ $f ] = str_replace( array( '"', '>' ), '', $notification[ $f ] );
 
-                //Switch userID shortcode to email address
+				//Switch userID shortcode to email address
 				if ( strpos( $notification[ $f ], '[' . $user_id_field . ']' ) !== false || strpos( $notification[ $f ], '[' . $user_id_key . ']' ) !== false ) {
 					$user_data = get_userdata( $entry->metas[ $user_id_field ] );
-                    $user_email = $user_data->user_email;
+					$user_email = $user_data->user_email;
 					$notification[ $f ] = str_replace( array( '[' . $user_id_field . ']', '[' . $user_id_key . ']' ), $user_email, $notification[ $f ] );
-                }
-            }
+				}
+			}
 
 			$notification[ $f ] = FrmFieldsHelper::basic_replace_shortcodes( $notification[ $f ], $form, $entry );
-        }
+		}
 
-        //Put recipients, cc, and bcc into an array if they aren't empty
+		//Put recipients, cc, and bcc into an array if they aren't empty
 		$to_emails = self::explode_emails( $notification['email_to'] );
 		$cc = self::explode_emails( $notification['cc'] );
 		$bcc = self::explode_emails( $notification['bcc'] );
 
-        $to_emails = apply_filters('frm_to_email', $to_emails, $values, $form->id, compact('email_key', 'entry', 'form'));
+		$to_emails = apply_filters('frm_to_email', $to_emails, $values, $form->id, compact('email_key', 'entry', 'form'));
 
-        // Stop now if there aren't any recipients
-        if ( empty( $to_emails ) && empty( $cc ) && empty( $bcc ) ) {
-            return;
-        }
+		// Stop now if there aren't any recipients
+		if ( empty( $to_emails ) && empty( $cc ) && empty( $bcc ) ) {
+			return;
+		}
 
-        $to_emails = array_unique( (array) $to_emails );
+		$to_emails = array_unique( (array) $to_emails );
 
-        $prev_mail_body = $mail_body = $notification['email_message'];
-        $mail_body = FrmEntriesHelper::replace_default_message($mail_body, array(
-            'id' => $entry->id, 'entry' => $entry, 'plain_text' => $plain_text,
-            'user_info' => (isset($notification['inc_user_info']) ? $notification['inc_user_info'] : false),
-        ) );
+		$prev_mail_body = $mail_body = $notification['email_message'];
+		$mail_body = FrmEntriesHelper::replace_default_message($mail_body, array(
+			'id' => $entry->id, 'entry' => $entry, 'plain_text' => $plain_text,
+			'user_info' => (isset($notification['inc_user_info']) ? $notification['inc_user_info'] : false),
+		) );
 
-        // Add the user info if it isn't already included
-        if ( $notification['inc_user_info'] && $prev_mail_body == $mail_body ) {
-            $data = maybe_unserialize($entry->description);
+		// Add the user info if it isn't already included
+		if ( $notification['inc_user_info'] && $prev_mail_body == $mail_body ) {
+			$data = maybe_unserialize($entry->description);
 			$mail_body .= "\r\n\r\n" . __( 'User Information', 'formidable' ) . "\r\n";
 			$mail_body .= __( 'IP Address', 'formidable' ) . ': ' . $entry->ip . "\r\n";
 			$mail_body .= __( 'User-Agent (Browser/OS)', 'formidable' ) . ': ' . FrmEntryFormat::get_browser( $data['browser'] ) . "\r\n";
 			$mail_body .= __( 'Referrer', 'formidable' ) . ': ' . $data['referrer'] . "\r\n";
-        }
-        unset($prev_mail_body);
-
-        // Add attachments
-        $attachments = apply_filters('frm_notification_attachment', array(), $form, compact('entry', 'email_key') );
-
-        if ( ! empty($notification['email_subject']) ) {
-            $notification['email_subject'] = apply_filters('frm_email_subject', $notification['email_subject'], compact('form', 'entry', 'email_key'));
-        }
-
-        // check for a phone number
-        foreach ( (array) $to_emails as $email_key => $e ) {
-            if ( $e != '[admin_email]' && ! is_email($e) ) {
-                $e = explode(' ', $e);
-
-                //If to_email has name <test@mail.com> format
-                if ( is_email(end($e)) ) {
-                    continue;
-                }
-
-                do_action('frm_send_to_not_email', array(
-                    'e'         => $e,
-                    'subject'   => $notification['email_subject'],
-                    'mail_body' => $mail_body,
-                    'reply_to'  => $notification['reply_to'],
-                    'from'      => $notification['from'],
-                    'plain_text' => $plain_text,
-                    'attachments' => $attachments,
-                    'form'      => $form,
-                    'email_key' => $email_key,
-                ) );
+		}
+		unset($prev_mail_body);
+
+		// Add attachments
+		$attachments = apply_filters('frm_notification_attachment', array(), $form, compact('entry', 'email_key') );
+
+		if ( ! empty($notification['email_subject']) ) {
+			$notification['email_subject'] = apply_filters('frm_email_subject', $notification['email_subject'], compact('form', 'entry', 'email_key'));
+		}
+
+		// check for a phone number
+		foreach ( (array) $to_emails as $email_key => $e ) {
+			if ( $e != '[admin_email]' && ! is_email($e) ) {
+				$e = explode(' ', $e);
+
+				//If to_email has name <test@mail.com> format
+				if ( is_email(end($e)) ) {
+					continue;
+				}
+
+				do_action('frm_send_to_not_email', array(
+					'e'         => $e,
+					'subject'   => $notification['email_subject'],
+					'mail_body' => $mail_body,
+					'reply_to'  => $notification['reply_to'],
+					'from'      => $notification['from'],
+					'plain_text' => $plain_text,
+					'attachments' => $attachments,
+					'form'      => $form,
+					'email_key' => $email_key,
+				) );
 
 				unset( $to_emails[ $email_key ] );
-            }
-        }
+			}
+		}
 
 		/**
 		 * Send a separate email for email address in the "to" section
@@ -131,38 +131,38 @@ 
 		 */
 		$send_single_recipient = apply_filters( 'frm_send_separate_emails', false, compact( 'action', 'entry', 'form' ) );
 
-        // Send the email now
-        $sent_to = self::send_email( array(
-            'to_email'      => $to_emails,
-            'subject'       => $notification['email_subject'],
-            'message'       => $mail_body,
-            'from'          => $notification['from'],
-            'plain_text'    => $plain_text,
-            'reply_to'      => $notification['reply_to'],
-            'attachments'   => $attachments,
-            'cc'            => $cc,
-            'bcc'           => $bcc,
+		// Send the email now
+		$sent_to = self::send_email( array(
+			'to_email'      => $to_emails,
+			'subject'       => $notification['email_subject'],
+			'message'       => $mail_body,
+			'from'          => $notification['from'],
+			'plain_text'    => $plain_text,
+			'reply_to'      => $notification['reply_to'],
+			'attachments'   => $attachments,
+			'cc'            => $cc,
+			'bcc'           => $bcc,
 			'single_recipient' => $send_single_recipient,
-        ) );
+		) );
 
-        return $sent_to;
-    }
+		return $sent_to;
+	}
 
 	public function entry_created( $entry_id, $form_id ) {
 		$new_function = 'FrmFormActionsController::trigger_actions("create", ' . $form_id . ', ' . $entry_id . ', "email")';
 		_deprecated_function( __FUNCTION__, '2.0', $new_function );
-        FrmFormActionsController::trigger_actions('create', $form_id, $entry_id, 'email');
-    }
+		FrmFormActionsController::trigger_actions('create', $form_id, $entry_id, 'email');
+	}
 
 	public function send_notification_email( $to_email, $subject, $message, $from = '', $from_name = '', $plain_text = true, $attachments = array(), $reply_to = '' ) {
-        _deprecated_function( __FUNCTION__, '2.0', 'FrmNotification::send_email' );
+		_deprecated_function( __FUNCTION__, '2.0', 'FrmNotification::send_email' );
 
-        return self::send_email(compact(
-            'to_email', 'subject', 'message',
-            'from', 'from_name', 'plain_text',
-            'attachments', 'reply_to'
-        ));
-    }
+		return self::send_email(compact(
+			'to_email', 'subject', 'message',
+			'from', 'from_name', 'plain_text',
+			'attachments', 'reply_to'
+		));
+	}
 
 	/**
 	 * Extract the emails from cc and bcc. Allow separation by , or ;.
@@ -180,53 +180,53 @@ 
 		return $emails;
 	}
 
-    /**
-    * Put To, BCC, CC, Reply To, and From fields in Name <test@mail.com> format
-    * Formats that should work: Name, "Name", test@mail.com, <test@mail.com>, Name <test@mail.com>,
-    * "Name" <test@mail.com>, Name test@mail.com, "Name" test@mail.com, Name<test@mail.com>, "Name"<test@mail.com>
-    * "First Last" <test@mail.com>
-    *
-    * Things that won't work: First Last (with no email entered)
-    * @since 2.0
-    * @param array $atts array of email fields, pass by reference
-    * @param $admin_email
-    */
-    private static function format_email_fields( &$atts, $admin_email ) {
-
-        // If from is empty or is set to admin_email, set it now
-        $atts['from'] = ( empty($atts['from']) || $atts['from'] == '[admin_email]' ) ? $admin_email : $atts['from'];
-
-        // Filter values in these fields
+	/**
+	 * Put To, BCC, CC, Reply To, and From fields in Name <test@mail.com> format
+	 * Formats that should work: Name, "Name", test@mail.com, <test@mail.com>, Name <test@mail.com>,
+	 * "Name" <test@mail.com>, Name test@mail.com, "Name" test@mail.com, Name<test@mail.com>, "Name"<test@mail.com>
+	 * "First Last" <test@mail.com>
+	 *
+	 * Things that won't work: First Last (with no email entered)
+	 * @since 2.0
+	 * @param array $atts array of email fields, pass by reference
+	 * @param $admin_email
+	 */
+	private static function format_email_fields( &$atts, $admin_email ) {
+
+		// If from is empty or is set to admin_email, set it now
+		$atts['from'] = ( empty($atts['from']) || $atts['from'] == '[admin_email]' ) ? $admin_email : $atts['from'];
+
+		// Filter values in these fields
 		$filter_fields = array( 'to_email', 'bcc', 'cc', 'from', 'reply_to' );
 
-        foreach ( $filter_fields as $f ) {
-            // If empty, just skip it
+		foreach ( $filter_fields as $f ) {
+			// If empty, just skip it
 			if ( empty( $atts[ $f ] ) ) {
-                continue;
-            }
+				continue;
+			}
 
-            // to_email, cc, and bcc can be an array
+			// to_email, cc, and bcc can be an array
 			if ( is_array( $atts[ $f ] ) ) {
 				foreach ( $atts[ $f ] as $key => $val ) {
-                    self::format_single_field( $atts, $f, $val, $key );
-                    unset( $key, $val );
-                }
-                unset($f);
-                continue;
-            }
+					self::format_single_field( $atts, $f, $val, $key );
+					unset( $key, $val );
+				}
+				unset($f);
+				continue;
+			}
 
 			self::format_single_field( $atts, $f, $atts[ $f ] );
-        }
+		}
 
-        // If reply-to isn't set, make it match the from settings
-        if ( empty( $atts['reply_to'] ) ) {
-            $atts['reply_to'] = self::get_email_from_formatted_string( $atts['from'] );
-        }
+		// If reply-to isn't set, make it match the from settings
+		if ( empty( $atts['reply_to'] ) ) {
+			$atts['reply_to'] = self::get_email_from_formatted_string( $atts['from'] );
+		}
 
-        if ( ! is_array($atts['to_email']) && '[admin_email]' == $atts['to_email'] ) {
-            $atts['to_email'] = $admin_email;
-        }
-    }
+		if ( ! is_array($atts['to_email']) && '[admin_email]' == $atts['to_email'] ) {
+			$atts['to_email'] = $admin_email;
+		}
+	}
 
 	private static function get_email_from_formatted_string( $value ) {
 		if ( strpos( $value, '<' ) !== false ) {
@@ -236,48 +236,48 @@ 
 		return $value;
 	}
 
-    /**
-    * Format individual email fields
-    *
-    * @since 2.0
-    * @param array $atts pass by reference
-    * @param string $f (to, from, reply_to, etc)
-    * @param string $val value saved in field
-    * @param int $key if in array, this will be set
-    */
-    private static function format_single_field( &$atts, $f, $val, $key = false ) {
-        $val = trim($val);
-
-        // If just a plain email is used
-        if ( is_email($val) ) {
-            // add sender's name if not included in $from
-            if ( $f == 'from' ) {
+	/**
+	 * Format individual email fields
+	 *
+	 * @since 2.0
+	 * @param array $atts pass by reference
+	 * @param string $f (to, from, reply_to, etc)
+	 * @param string $val value saved in field
+	 * @param int $key if in array, this will be set
+	 */
+	private static function format_single_field( &$atts, $f, $val, $key = false ) {
+		$val = trim($val);
+
+		// If just a plain email is used
+		if ( is_email($val) ) {
+			// add sender's name if not included in $from
+			if ( $f == 'from' ) {
 				$part_2 = $atts[ $f ];
-                $part_1  = $atts['from_name'] ? $atts['from_name'] : wp_specialchars_decode( FrmAppHelper::site_name(), ENT_QUOTES );
-            } else {
-                return;
-            }
-        } else {
-            $parts = explode(' ', $val);
-            $part_2 = end($parts);
-
-            // If inputted correcly, $part_2 should be an email
-            if ( is_email( $part_2 ) ) {
-                $part_1 = trim( str_replace( $part_2, '', $val ) );
-            } else if ( in_array( $f, array( 'from', 'reply_to' ) ) ) {
+				$part_1  = $atts['from_name'] ? $atts['from_name'] : wp_specialchars_decode( FrmAppHelper::site_name(), ENT_QUOTES );
+			} else {
+				return;
+			}
+		} else {
+			$parts = explode(' ', $val);
+			$part_2 = end($parts);
+
+			// If inputted correcly, $part_2 should be an email
+			if ( is_email( $part_2 ) ) {
+				$part_1 = trim( str_replace( $part_2, '', $val ) );
+			} else if ( in_array( $f, array( 'from', 'reply_to' ) ) ) {
 				// In case someone just puts a name in the From or Reply To field
 				$part_1 = $val;
-                $part_2 = get_option('admin_email');
-            } else {
+				$part_2 = get_option('admin_email');
+			} else {
 				// In case someone just puts a name in any other email field
-                if ( false !== $key ) {
+				if ( false !== $key ) {
 					unset( $atts[ $f ][ $key ] );
-                    return;
-                }
+					return;
+				}
 				$atts[ $f ] = '';
-                return;
-            }
-        }
+				return;
+			}
+		}
 
 		// if sending the email from a yahoo address, change it to the WordPress default
 		if ( $f == 'from' && strpos( $part_2, '@yahoo.com' ) ) {
@@ -290,60 +290,60 @@ 
 			$part_2 = 'wordpress@' . $sitename;
 		}
 
-        // Set up formatted value
+		// Set up formatted value
 		$final_val = str_replace( '"', '', $part_1 ) . ' <' . $part_2 . '>';
 
-        // If value is an array
-        if ( false !== $key ) {
+		// If value is an array
+		if ( false !== $key ) {
 			$atts[ $f ][ $key ] = $final_val;
-            return;
-        }
+			return;
+		}
 		$atts[ $f ] = $final_val;
-    }
+	}
 
 	public static function send_email( $atts ) {
-        $admin_email = get_option('admin_email');
-        $defaults = array(
-            'to_email'      => $admin_email,
-            'subject'       => '',
-            'message'       => '',
-            'from'          => $admin_email,
-            'from_name'     => '',
-            'cc'            => '',
-            'bcc'           => '',
-            'plain_text'    => true,
-            'reply_to'      => $admin_email,
-            'attachments'   => array(),
+		$admin_email = get_option('admin_email');
+		$defaults = array(
+			'to_email'      => $admin_email,
+			'subject'       => '',
+			'message'       => '',
+			'from'          => $admin_email,
+			'from_name'     => '',
+			'cc'            => '',
+			'bcc'           => '',
+			'plain_text'    => true,
+			'reply_to'      => $admin_email,
+			'attachments'   => array(),
 			'single_recipient' => false,
-        );
-        $atts = wp_parse_args($atts, $defaults);
+		);
+		$atts = wp_parse_args($atts, $defaults);
 
-        // Put To, BCC, CC, Reply To, and From fields in the correct format
-        self::format_email_fields( $atts, $admin_email );
+		// Put To, BCC, CC, Reply To, and From fields in the correct format
+		self::format_email_fields( $atts, $admin_email );
 
-        $header         = array();
-        $header[]       = 'From: ' . $atts['from'];
+		$header         = array();
+		$header[]       = 'From: ' . $atts['from'];
 
-        //Allow for cc and bcc arrays
+		//Allow for cc and bcc arrays
 		$array_fields = array( 'CC' => $atts['cc'], 'BCC' => $atts['bcc'] );
 		$cc = array( 'CC' => array(), 'BCC' => array() );
-        foreach ( $array_fields as $key => $a_field ) {
-            if ( empty($a_field) ) {
-                continue;
-            }
+		foreach ( $array_fields as $key => $a_field ) {
+			if ( empty($a_field) ) {
+				continue;
+			}
 
 			foreach ( (array) $a_field as $email ) {
 				$cc[ $key ][] = $email;
-            }
-            unset($key, $a_field);
-        }
+			}
+			unset($key, $a_field);
+		}
 		$cc = array_filter( $cc ); // remove cc and bcc if they are empty
 
 		foreach ( $cc as $k => $v ) {
 			$header[] = $k . ': ' . implode( ',', $v );
 		}
 
-        $content_type   = $atts['plain_text'] ? 'text/plain' : 'text/html';
+		$content_type   = $atts['plain_text'] ? 'text/plain' : 'text/html';
 		$atts['charset'] = get_option('blog_charset');
 
 		$header[]       = 'Reply-To: ' . $atts['reply_to'];
@@ -352,12 +352,12 @@ 
 
 		$atts['message'] = do_shortcode( $atts['message'] );
 
-        if ( $atts['plain_text'] ) {
+		if ( $atts['plain_text'] ) {
 			$atts['message'] = wp_specialchars_decode( strip_tags( $atts['message'] ), ENT_QUOTES );
-        } else {
+		} else {
 			// remove line breaks in HTML emails to prevent conflicts with Mandrill
-        	add_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
-        }
+			add_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
+		}
 		$atts['message'] = apply_filters( 'frm_email_message', $atts['message'], $atts );
 
 		/**
@@ -386,10 +386,10 @@ 
 		// remove the filter now so other emails can still use it
 		remove_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
 
-        if ( $sent ) {
+		if ( $sent ) {
 			return self::return_emails_sent( $atts );
-        }
-    }
+		}
+	}
 
 	private static function send_single_email( $recipient, $atts, $header ) {
 		$header = apply_filters( 'frm_email_header', $header, array(

Spacing +58 added lines, -58 removed lines

@@ -2,10 +2,10 @@ 
 
 class FrmNotification {
 	public function __construct() {
-        if ( ! defined('ABSPATH') ) {
-            die('You are not allowed to call this page directly.');
+        if ( ! defined( 'ABSPATH' ) ) {
+            die( 'You are not allowed to call this page directly.' );
         }
-        add_action('frm_trigger_email_action', 'FrmNotification::trigger_email', 10, 3);
+        add_action( 'frm_trigger_email_action', 'FrmNotification::trigger_email', 10, 3 );
     }
 
 	public static function trigger_email( $action, $entry, $form ) {
@@ -13,8 +13,8 @@ 
         $email_key = $action->ID;
 
         // Set the subject
-        if ( empty($notification['email_subject']) ) {
-            $notification['email_subject'] = sprintf(__( '%1$s Form submitted on %2$s', 'formidable' ), $form->name, '[sitename]');
+        if ( empty( $notification['email_subject'] ) ) {
+            $notification['email_subject'] = sprintf( __( '%1$s Form submitted on %2$s', 'formidable' ), $form->name, '[sitename]' );
         }
 
         $plain_text = $notification['plain_text'] ? true : false;
@@ -26,7 +26,7 @@ 
             'email_subject', 'email_message',
         );
 
-        add_filter('frm_plain_text_email', ($plain_text ? '__return_true' : '__return_false'));
+        add_filter( 'frm_plain_text_email', ( $plain_text ? '__return_true' : '__return_false' ) );
 
         //Get all values in entry in order to get User ID field ID
         $values = FrmEntryMeta::getAll( array( 'it.field_id !' => 0, 'it.item_id' => $entry->id ), ' ORDER BY fi.field_order' );
@@ -37,29 +37,29 @@ 
                 $user_id_key = $value->field_key;
                 break;
             }
-            unset($value);
+            unset( $value );
         }
 
         //Filter and prepare the email fields
         foreach ( $filter_fields as $f ) {
             //Don't allow empty From
-			if ( $f == 'from' && empty( $notification[ $f ] ) ) {
-				$notification[ $f ] = '[admin_email]';
+			if ( $f == 'from' && empty( $notification[$f] ) ) {
+				$notification[$f] = '[admin_email]';
 			} else if ( in_array( $f, array( 'email_to', 'cc', 'bcc', 'reply_to', 'from' ) ) ) {
 				//Remove brackets
                 //Add a space in case there isn't one
-				$notification[ $f ] = str_replace( '<', ' ', $notification[ $f ] );
-				$notification[ $f ] = str_replace( array( '"', '>' ), '', $notification[ $f ] );
+				$notification[$f] = str_replace( '<', ' ', $notification[$f] );
+				$notification[$f] = str_replace( array( '"', '>' ), '', $notification[$f] );
 
                 //Switch userID shortcode to email address
-				if ( strpos( $notification[ $f ], '[' . $user_id_field . ']' ) !== false || strpos( $notification[ $f ], '[' . $user_id_key . ']' ) !== false ) {
-					$user_data = get_userdata( $entry->metas[ $user_id_field ] );
+				if ( strpos( $notification[$f], '[' . $user_id_field . ']' ) !== false || strpos( $notification[$f], '[' . $user_id_key . ']' ) !== false ) {
+					$user_data = get_userdata( $entry->metas[$user_id_field] );
                     $user_email = $user_data->user_email;
-					$notification[ $f ] = str_replace( array( '[' . $user_id_field . ']', '[' . $user_id_key . ']' ), $user_email, $notification[ $f ] );
+					$notification[$f] = str_replace( array( '[' . $user_id_field . ']', '[' . $user_id_key . ']' ), $user_email, $notification[$f] );
                 }
             }
 
-			$notification[ $f ] = FrmFieldsHelper::basic_replace_shortcodes( $notification[ $f ], $form, $entry );
+			$notification[$f] = FrmFieldsHelper::basic_replace_shortcodes( $notification[$f], $form, $entry );
         }
 
         //Put recipients, cc, and bcc into an array if they aren't empty
@@ -67,7 +67,7 @@ 
 		$cc = self::explode_emails( $notification['cc'] );
 		$bcc = self::explode_emails( $notification['bcc'] );
 
-        $to_emails = apply_filters('frm_to_email', $to_emails, $values, $form->id, compact('email_key', 'entry', 'form'));
+        $to_emails = apply_filters( 'frm_to_email', $to_emails, $values, $form->id, compact( 'email_key', 'entry', 'form' ) );
 
         // Stop now if there aren't any recipients
         if ( empty( $to_emails ) && empty( $cc ) && empty( $bcc ) ) {
@@ -77,39 +77,39 @@ 
         $to_emails = array_unique( (array) $to_emails );
 
         $prev_mail_body = $mail_body = $notification['email_message'];
-        $mail_body = FrmEntriesHelper::replace_default_message($mail_body, array(
+        $mail_body = FrmEntriesHelper::replace_default_message( $mail_body, array(
             'id' => $entry->id, 'entry' => $entry, 'plain_text' => $plain_text,
-            'user_info' => (isset($notification['inc_user_info']) ? $notification['inc_user_info'] : false),
+            'user_info' => ( isset( $notification['inc_user_info'] ) ? $notification['inc_user_info'] : false ),
         ) );
 
         // Add the user info if it isn't already included
         if ( $notification['inc_user_info'] && $prev_mail_body == $mail_body ) {
-            $data = maybe_unserialize($entry->description);
+            $data = maybe_unserialize( $entry->description );
 			$mail_body .= "\r\n\r\n" . __( 'User Information', 'formidable' ) . "\r\n";
 			$mail_body .= __( 'IP Address', 'formidable' ) . ': ' . $entry->ip . "\r\n";
 			$mail_body .= __( 'User-Agent (Browser/OS)', 'formidable' ) . ': ' . FrmEntryFormat::get_browser( $data['browser'] ) . "\r\n";
 			$mail_body .= __( 'Referrer', 'formidable' ) . ': ' . $data['referrer'] . "\r\n";
         }
-        unset($prev_mail_body);
+        unset( $prev_mail_body );
 
         // Add attachments
-        $attachments = apply_filters('frm_notification_attachment', array(), $form, compact('entry', 'email_key') );
+        $attachments = apply_filters( 'frm_notification_attachment', array(), $form, compact( 'entry', 'email_key' ) );
 
-        if ( ! empty($notification['email_subject']) ) {
-            $notification['email_subject'] = apply_filters('frm_email_subject', $notification['email_subject'], compact('form', 'entry', 'email_key'));
+        if ( ! empty( $notification['email_subject'] ) ) {
+            $notification['email_subject'] = apply_filters( 'frm_email_subject', $notification['email_subject'], compact( 'form', 'entry', 'email_key' ) );
         }
 
         // check for a phone number
         foreach ( (array) $to_emails as $email_key => $e ) {
-            if ( $e != '[admin_email]' && ! is_email($e) ) {
-                $e = explode(' ', $e);
+            if ( $e != '[admin_email]' && ! is_email( $e ) ) {
+                $e = explode( ' ', $e );
 
                 //If to_email has name <test@mail.com> format
-                if ( is_email(end($e)) ) {
+                if ( is_email( end( $e ) ) ) {
                     continue;
                 }
 
-                do_action('frm_send_to_not_email', array(
+                do_action( 'frm_send_to_not_email', array(
                     'e'         => $e,
                     'subject'   => $notification['email_subject'],
                     'mail_body' => $mail_body,
@@ -121,7 +121,7 @@ 
                     'email_key' => $email_key,
                 ) );
 
-				unset( $to_emails[ $email_key ] );
+				unset( $to_emails[$email_key] );
             }
         }
 
@@ -151,17 +151,17 @@ 
 	public function entry_created( $entry_id, $form_id ) {
 		$new_function = 'FrmFormActionsController::trigger_actions("create", ' . $form_id . ', ' . $entry_id . ', "email")';
 		_deprecated_function( __FUNCTION__, '2.0', $new_function );
-        FrmFormActionsController::trigger_actions('create', $form_id, $entry_id, 'email');
+        FrmFormActionsController::trigger_actions( 'create', $form_id, $entry_id, 'email' );
     }
 
 	public function send_notification_email( $to_email, $subject, $message, $from = '', $from_name = '', $plain_text = true, $attachments = array(), $reply_to = '' ) {
         _deprecated_function( __FUNCTION__, '2.0', 'FrmNotification::send_email' );
 
-        return self::send_email(compact(
+        return self::send_email( compact(
             'to_email', 'subject', 'message',
             'from', 'from_name', 'plain_text',
             'attachments', 'reply_to'
-        ));
+        ) );
     }
 
 	/**
@@ -194,28 +194,28 @@ 
     private static function format_email_fields( &$atts, $admin_email ) {
 
         // If from is empty or is set to admin_email, set it now
-        $atts['from'] = ( empty($atts['from']) || $atts['from'] == '[admin_email]' ) ? $admin_email : $atts['from'];
+        $atts['from'] = ( empty( $atts['from'] ) || $atts['from'] == '[admin_email]' ) ? $admin_email : $atts['from'];
 
         // Filter values in these fields
 		$filter_fields = array( 'to_email', 'bcc', 'cc', 'from', 'reply_to' );
 
         foreach ( $filter_fields as $f ) {
             // If empty, just skip it
-			if ( empty( $atts[ $f ] ) ) {
+			if ( empty( $atts[$f] ) ) {
                 continue;
             }
 
             // to_email, cc, and bcc can be an array
-			if ( is_array( $atts[ $f ] ) ) {
-				foreach ( $atts[ $f ] as $key => $val ) {
+			if ( is_array( $atts[$f] ) ) {
+				foreach ( $atts[$f] as $key => $val ) {
                     self::format_single_field( $atts, $f, $val, $key );
                     unset( $key, $val );
                 }
-                unset($f);
+                unset( $f );
                 continue;
             }
 
-			self::format_single_field( $atts, $f, $atts[ $f ] );
+			self::format_single_field( $atts, $f, $atts[$f] );
         }
 
         // If reply-to isn't set, make it match the from settings
@@ -223,7 +223,7 @@ 
             $atts['reply_to'] = self::get_email_from_formatted_string( $atts['from'] );
         }
 
-        if ( ! is_array($atts['to_email']) && '[admin_email]' == $atts['to_email'] ) {
+        if ( ! is_array( $atts['to_email'] ) && '[admin_email]' == $atts['to_email'] ) {
             $atts['to_email'] = $admin_email;
         }
     }
@@ -246,20 +246,20 @@ 
     * @param int $key if in array, this will be set
     */
     private static function format_single_field( &$atts, $f, $val, $key = false ) {
-        $val = trim($val);
+        $val = trim( $val );
 
         // If just a plain email is used
-        if ( is_email($val) ) {
+        if ( is_email( $val ) ) {
             // add sender's name if not included in $from
             if ( $f == 'from' ) {
-				$part_2 = $atts[ $f ];
-                $part_1  = $atts['from_name'] ? $atts['from_name'] : wp_specialchars_decode( FrmAppHelper::site_name(), ENT_QUOTES );
+				$part_2 = $atts[$f];
+                $part_1 = $atts['from_name'] ? $atts['from_name'] : wp_specialchars_decode( FrmAppHelper::site_name(), ENT_QUOTES );
             } else {
                 return;
             }
         } else {
-            $parts = explode(' ', $val);
-            $part_2 = end($parts);
+            $parts = explode( ' ', $val );
+            $part_2 = end( $parts );
 
             // If inputted correcly, $part_2 should be an email
             if ( is_email( $part_2 ) ) {
@@ -267,14 +267,14 @@ 
             } else if ( in_array( $f, array( 'from', 'reply_to' ) ) ) {
 				// In case someone just puts a name in the From or Reply To field
 				$part_1 = $val;
-                $part_2 = get_option('admin_email');
+                $part_2 = get_option( 'admin_email' );
             } else {
 				// In case someone just puts a name in any other email field
                 if ( false !== $key ) {
-					unset( $atts[ $f ][ $key ] );
+					unset( $atts[$f][$key] );
                     return;
                 }
-				$atts[ $f ] = '';
+				$atts[$f] = '';
                 return;
             }
         }
@@ -295,14 +295,14 @@ 
 
         // If value is an array
         if ( false !== $key ) {
-			$atts[ $f ][ $key ] = $final_val;
+			$atts[$f][$key] = $final_val;
             return;
         }
-		$atts[ $f ] = $final_val;
+		$atts[$f] = $final_val;
     }
 
 	public static function send_email( $atts ) {
-        $admin_email = get_option('admin_email');
+        $admin_email = get_option( 'admin_email' );
         $defaults = array(
             'to_email'      => $admin_email,
             'subject'       => '',
@@ -316,7 +316,7 @@ 
             'attachments'   => array(),
 			'single_recipient' => false,
         );
-        $atts = wp_parse_args($atts, $defaults);
+        $atts = wp_parse_args( $atts, $defaults );
 
         // Put To, BCC, CC, Reply To, and From fields in the correct format
         self::format_email_fields( $atts, $admin_email );
@@ -328,14 +328,14 @@ 
 		$array_fields = array( 'CC' => $atts['cc'], 'BCC' => $atts['bcc'] );
 		$cc = array( 'CC' => array(), 'BCC' => array() );
         foreach ( $array_fields as $key => $a_field ) {
-            if ( empty($a_field) ) {
+            if ( empty( $a_field ) ) {
                 continue;
             }
 
 			foreach ( (array) $a_field as $email ) {
-				$cc[ $key ][] = $email;
+				$cc[$key][] = $email;
             }
-            unset($key, $a_field);
+            unset( $key, $a_field );
         }
 		$cc = array_filter( $cc ); // remove cc and bcc if they are empty
 
@@ -343,8 +343,8 @@ 
 			$header[] = $k . ': ' . implode( ',', $v );
 		}
 
-        $content_type   = $atts['plain_text'] ? 'text/plain' : 'text/html';
-		$atts['charset'] = get_option('blog_charset');
+        $content_type = $atts['plain_text'] ? 'text/plain' : 'text/html';
+		$atts['charset'] = get_option( 'blog_charset' );
 
 		$header[]       = 'Reply-To: ' . $atts['reply_to'];
 		$header[]       = 'Content-Type: ' . $content_type . '; charset="' . esc_attr( $atts['charset'] ) . '"';
@@ -411,7 +411,7 @@ 
 	}
 
 	private static function encode_subject( $charset, &$subject ) {
-		if ( apply_filters('frm_encode_subject', 1, $subject ) ) {
+		if ( apply_filters( 'frm_encode_subject', 1, $subject ) ) {
 			$subject = '=?' . $charset . '?B?' . base64_encode( $subject ) . '?=';
 		}
 	}
@@ -427,7 +427,7 @@ 
 
 		if ( apply_filters( 'frm_echo_emails', false ) ) {
 			$temp = str_replace( '<', '&lt;', $sent_to );
-			echo ' ' . FrmAppHelper::kses( implode(', ', (array) $temp ) );
+			echo ' ' . FrmAppHelper::kses( implode( ', ', (array) $temp ) );
 		}
 		return $sent_to;
 	}

classes/controllers/FrmEntriesController.php

Indentation +291 added lines, -291 removed lines

@@ -2,201 +2,201 @@ 
 
 class FrmEntriesController {
 
-    public static function menu() {
+	public static function menu() {
 		FrmAppHelper::force_capability( 'frm_view_entries' );
 
-        add_submenu_page('formidable', 'Formidable | ' . __( 'Entries', 'formidable' ), __( 'Entries', 'formidable' ), 'frm_view_entries', 'formidable-entries', 'FrmEntriesController::route' );
+		add_submenu_page('formidable', 'Formidable | ' . __( 'Entries', 'formidable' ), __( 'Entries', 'formidable' ), 'frm_view_entries', 'formidable-entries', 'FrmEntriesController::route' );
 
 		if ( ! in_array( FrmAppHelper::simple_get( 'frm_action', 'sanitize_title' ), array( 'edit', 'show' ) ) ) {
 			$menu_name = FrmAppHelper::get_menu_name();
 			add_filter( 'manage_' . sanitize_title( $menu_name ) . '_page_formidable-entries_columns', 'FrmEntriesController::manage_columns' );
 			add_filter( 'get_user_option_manage' . sanitize_title( $menu_name ) . '_page_formidable-entriescolumnshidden', 'FrmEntriesController::hidden_columns' );
 			add_filter( 'manage_' . sanitize_title( $menu_name ) . '_page_formidable-entries_sortable_columns', 'FrmEntriesController::sortable_columns' );
-        }
-    }
+		}
+	}
 
-    /* Display in Back End */
-    public static function route() {
+	/* Display in Back End */
+	public static function route() {
 		$action = FrmAppHelper::get_param( 'frm_action', '', 'get', 'sanitize_title' );
 
-        switch ( $action ) {
-            case 'show':
-            case 'destroy':
-            case 'destroy_all':
-                return self::$action();
+		switch ( $action ) {
+			case 'show':
+			case 'destroy':
+			case 'destroy_all':
+				return self::$action();
 
-            default:
-                do_action( 'frm_entry_action_route', $action );
-                if ( apply_filters( 'frm_entry_stop_action_route', false, $action ) ) {
-                    return;
-                }
+			default:
+				do_action( 'frm_entry_action_route', $action );
+				if ( apply_filters( 'frm_entry_stop_action_route', false, $action ) ) {
+					return;
+				}
 
-                return self::display_list();
-        }
-    }
+				return self::display_list();
+		}
+	}
 
 	public static function contextual_help( $help, $screen_id, $screen ) {
-        // Only add to certain screens. add_help_tab was introduced in WordPress 3.3
-        if ( ! method_exists( $screen, 'add_help_tab' ) ) {
-            return $help;
-        }
+		// Only add to certain screens. add_help_tab was introduced in WordPress 3.3
+		if ( ! method_exists( $screen, 'add_help_tab' ) ) {
+			return $help;
+		}
 
 		$action = FrmAppHelper::simple_get( 'frm_action', 'sanitize_title' );
 		$page = FrmAppHelper::simple_get( 'page', 'sanitize_title' );
 		if ( $page != 'formidable-entries' || ( ! empty( $action ) && $action != 'list' ) ) {
-            return $help;
-        }
+			return $help;
+		}
 
 		unset( $action, $page );
 
-        $screen->add_help_tab( array(
-            'id'      => 'formidable-entries-tab',
-            'title'   => __( 'Overview', 'formidable' ),
+		$screen->add_help_tab( array(
+			'id'      => 'formidable-entries-tab',
+			'title'   => __( 'Overview', 'formidable' ),
 			'content' => '<p>' . esc_html__( 'This screen provides access to all of your entries. You can customize the display of this screen to suit your workflow.', 'formidable' ) . '</p> <p>' . esc_html__( 'Hovering over a row in the entries list will display action links that allow you to manage your entry.', 'formidable' ) . '</p>',
-        ));
+		));
 
-        $screen->set_help_sidebar(
+		$screen->set_help_sidebar(
 			'<p><strong>' . esc_html__( 'For more information:', 'formidable' ) . '</strong></p>' .
 			'<p><a href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/knowledgebase/manage-entries-from-the-back-end/' ) ) . '" target="_blank">' . esc_html__( 'Documentation on Entries', 'formidable' ) . '</a></p>' .
 			'<p><a href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/help-desk/' ) ) . '" target="_blank">' . esc_html__( 'Support', 'formidable' ) . '</a></p>'
-    	);
+		);
 
-        return $help;
-    }
+		return $help;
+	}
 
 	public static function manage_columns( $columns ) {
-        global $frm_vars, $wpdb;
+		global $frm_vars, $wpdb;
 		$form_id = FrmForm::get_current_form_id();
 
 		$columns[ $form_id . '_id' ] = 'ID';
 		$columns[ $form_id . '_item_key' ] = esc_html__( 'Entry Key', 'formidable' );
 
-        if ( ! $form_id ) {
-            return $columns;
-        }
+		if ( ! $form_id ) {
+			return $columns;
+		}
 
-        $form_cols = FrmField::get_all_for_form($form_id, '', 'include');
+		$form_cols = FrmField::get_all_for_form($form_id, '', 'include');
 
-        foreach ( $form_cols as $form_col ) {
+		foreach ( $form_cols as $form_col ) {
 			if ( FrmField::is_no_save_field( $form_col->type ) ) {
-                continue;
-            }
+				continue;
+			}
 
-            if ( $form_col->type == 'form' && isset( $form_col->field_options['form_select'] ) && ! empty( $form_col->field_options['form_select'] ) ) {
+			if ( $form_col->type == 'form' && isset( $form_col->field_options['form_select'] ) && ! empty( $form_col->field_options['form_select'] ) ) {
 				$sub_form_cols = FrmField::get_all_for_form( $form_col->field_options['form_select'] );
 
-                if ( $sub_form_cols ) {
-                    foreach ( $sub_form_cols as $k => $sub_form_col ) {
+				if ( $sub_form_cols ) {
+					foreach ( $sub_form_cols as $k => $sub_form_col ) {
 						if ( FrmField::is_no_save_field( $sub_form_col->type ) ) {
-                            unset( $sub_form_cols[ $k ] );
-                            continue;
-                        }
+							unset( $sub_form_cols[ $k ] );
+							continue;
+						}
 						$columns[ $form_id . '_' . $sub_form_col->field_key . '-_-' . $form_col->id ] = FrmAppHelper::truncate( $sub_form_col->name, 35 );
-                        unset($sub_form_col);
-                    }
-                }
-                unset($sub_form_cols);
-            } else {
-                $col_id = $form_col->field_key;
-                if ( $form_col->form_id != $form_id ) {
+						unset($sub_form_col);
+					}
+				}
+				unset($sub_form_cols);
+			} else {
+				$col_id = $form_col->field_key;
+				if ( $form_col->form_id != $form_id ) {
 					$col_id .= '-_-form' . $form_col->form_id;
-                }
+				}
 
-                if ( isset($form_col->field_options['separate_value']) && $form_col->field_options['separate_value'] ) {
+				if ( isset($form_col->field_options['separate_value']) && $form_col->field_options['separate_value'] ) {
 					$columns[ $form_id . '_frmsep_' . $col_id ] = FrmAppHelper::truncate( $form_col->name, 35 );
-                }
+				}
 				$columns[ $form_id . '_' . $col_id ] = FrmAppHelper::truncate( $form_col->name, 35 );
-            }
-        }
+			}
+		}
 
 		$columns[ $form_id . '_created_at' ] = __( 'Entry creation date', 'formidable' );
 		$columns[ $form_id . '_updated_at' ] = __( 'Entry update date', 'formidable' );
 		$columns[ $form_id . '_ip' ] = 'IP';
 
-        $frm_vars['cols'] = $columns;
+		$frm_vars['cols'] = $columns;
 
 		$action = FrmAppHelper::simple_get( 'frm_action', 'sanitize_title' );
 		if ( FrmAppHelper::is_admin_page( 'formidable-entries' ) && in_array( $action, array( '', 'list', 'destroy' ) ) ) {
 			add_screen_option( 'per_page', array( 'label' => __( 'Entries', 'formidable' ), 'default' => 20, 'option' => 'formidable_page_formidable_entries_per_page' ) );
-        }
+		}
 
-        return $columns;
-    }
+		return $columns;
+	}
 
 	public static function check_hidden_cols( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 		$menu_name = FrmAppHelper::get_menu_name();
 		$this_page_name = 'manage' . sanitize_title( $menu_name ) . '_page_formidable-entriescolumnshidden';
 		if ( $meta_key != $this_page_name || $meta_value == $prev_value ) {
-            return $check;
-        }
+			return $check;
+		}
 
 		if ( empty( $prev_value ) ) {
 			$prev_value = get_metadata( 'user', $object_id, $meta_key, true );
 		}
 
-        global $frm_vars;
-        //add a check so we don't create a loop
-        $frm_vars['prev_hidden_cols'] = ( isset($frm_vars['prev_hidden_cols']) && $frm_vars['prev_hidden_cols'] ) ? false : $prev_value;
+		global $frm_vars;
+		//add a check so we don't create a loop
+		$frm_vars['prev_hidden_cols'] = ( isset($frm_vars['prev_hidden_cols']) && $frm_vars['prev_hidden_cols'] ) ? false : $prev_value;
 
-        return $check;
-    }
+		return $check;
+	}
 
-    //add hidden columns back from other forms
+	//add hidden columns back from other forms
 	public static function update_hidden_cols( $meta_id, $object_id, $meta_key, $meta_value ) {
 		$menu_name = FrmAppHelper::get_menu_name();
 		$sanitized = sanitize_title( $menu_name );
 		$this_page_name = 'manage' . $sanitized . '_page_formidable-entriescolumnshidden';
 		if ( $meta_key != $this_page_name ) {
-            return;
-        }
-
-        global $frm_vars;
-        if ( ! isset($frm_vars['prev_hidden_cols']) || ! $frm_vars['prev_hidden_cols'] ) {
-            return; //don't continue if there's no previous value
-        }
-
-        foreach ( $meta_value as $mk => $mv ) {
-            //remove blank values
-            if ( empty( $mv )  ) {
-                unset( $meta_value[ $mk ] );
-            }
-        }
-
-        $cur_form_prefix = reset($meta_value);
-        $cur_form_prefix = explode('_', $cur_form_prefix);
-        $cur_form_prefix = $cur_form_prefix[0];
-        $save = false;
-
-        foreach ( (array) $frm_vars['prev_hidden_cols'] as $prev_hidden ) {
+			return;
+		}
+
+		global $frm_vars;
+		if ( ! isset($frm_vars['prev_hidden_cols']) || ! $frm_vars['prev_hidden_cols'] ) {
+			return; //don't continue if there's no previous value
+		}
+
+		foreach ( $meta_value as $mk => $mv ) {
+			//remove blank values
+			if ( empty( $mv )  ) {
+				unset( $meta_value[ $mk ] );
+			}
+		}
+
+		$cur_form_prefix = reset($meta_value);
+		$cur_form_prefix = explode('_', $cur_form_prefix);
+		$cur_form_prefix = $cur_form_prefix[0];
+		$save = false;
+
+		foreach ( (array) $frm_vars['prev_hidden_cols'] as $prev_hidden ) {
 			if ( empty( $prev_hidden ) || in_array( $prev_hidden, $meta_value ) ) {
-                //don't add blank cols or process included cols
-                continue;
-            }
+				//don't add blank cols or process included cols
+				continue;
+			}
 
 			$form_prefix = explode( '_', $prev_hidden );
-            $form_prefix = $form_prefix[0];
-            if ( $form_prefix == $cur_form_prefix ) {
-                //don't add back columns that are meant to be hidden
-                continue;
-            }
+			$form_prefix = $form_prefix[0];
+			if ( $form_prefix == $cur_form_prefix ) {
+				//don't add back columns that are meant to be hidden
+				continue;
+			}
 
-            $meta_value[] = $prev_hidden;
-            $save = true;
-            unset($form_prefix);
-        }
+			$meta_value[] = $prev_hidden;
+			$save = true;
+			unset($form_prefix);
+		}
 
 		if ( $save ) {
-            $user = wp_get_current_user();
+			$user = wp_get_current_user();
 			update_user_option( $user->ID, $this_page_name, $meta_value, true );
-        }
-    }
+		}
+	}
 
 	public static function save_per_page( $save, $option, $value ) {
-        if ( $option == 'formidable_page_formidable_entries_per_page' ) {
-            $save = (int) $value;
-        }
-        return $save;
-    }
+		if ( $option == 'formidable_page_formidable_entries_per_page' ) {
+			$save = (int) $value;
+		}
+		return $save;
+	}
 
 	public static function sortable_columns() {
 		$form_id = FrmForm::get_current_form_id();
@@ -222,122 +222,122 @@ 
 	}
 
 	public static function hidden_columns( $result ) {
-        global $frm_vars;
+		global $frm_vars;
 
 		$form_id = FrmForm::get_current_form_id();
 
-        $return = false;
-        foreach ( (array) $result as $r ) {
-            if ( ! empty( $r ) ) {
-                $form_prefix = explode( '_', $r );
-                $form_prefix = $form_prefix[0];
+		$return = false;
+		foreach ( (array) $result as $r ) {
+			if ( ! empty( $r ) ) {
+				$form_prefix = explode( '_', $r );
+				$form_prefix = $form_prefix[0];
 
-                if ( (int) $form_prefix == (int) $form_id ) {
-                    $return = true;
-                    break;
-                }
+				if ( (int) $form_prefix == (int) $form_id ) {
+					$return = true;
+					break;
+				}
 
-                unset($form_prefix);
-            }
-        }
+				unset($form_prefix);
+			}
+		}
 
-        if ( $return ) {
+		if ( $return ) {
 			return $result;
 		}
 
-        $i = isset($frm_vars['cols']) ? count($frm_vars['cols']) : 0;
-        $max_columns = 8;
-        if ( $i <= $max_columns ) {
+		$i = isset($frm_vars['cols']) ? count($frm_vars['cols']) : 0;
+		$max_columns = 8;
+		if ( $i <= $max_columns ) {
 			return $result;
 		}
 
-        global $frm_vars;
-        if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] ) {
-            $frm_vars['current_form']->options = maybe_unserialize($frm_vars['current_form']->options);
-        }
+		global $frm_vars;
+		if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] ) {
+			$frm_vars['current_form']->options = maybe_unserialize($frm_vars['current_form']->options);
+		}
 
-        if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] && isset($frm_vars['current_form']->options['hidden_cols']) && ! empty($frm_vars['current_form']->options['hidden_cols']) ) {
-            $result = $frm_vars['current_form']->options['hidden_cols'];
-        } else {
-            $cols = $frm_vars['cols'];
-            $cols = array_reverse($cols, true);
+		if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] && isset($frm_vars['current_form']->options['hidden_cols']) && ! empty($frm_vars['current_form']->options['hidden_cols']) ) {
+			$result = $frm_vars['current_form']->options['hidden_cols'];
+		} else {
+			$cols = $frm_vars['cols'];
+			$cols = array_reverse($cols, true);
 
 			$result[] = $form_id . '_id';
-            $i--;
+			$i--;
 
 			$result[] = $form_id . '_item_key';
-            $i--;
+			$i--;
 
 			foreach ( $cols as $col_key => $col ) {
-                if ( $i > $max_columns ) {
+				if ( $i > $max_columns ) {
 					$result[] = $col_key;
 				}
-                //remove some columns by default
-                $i--;
-                unset($col_key, $col);
-            }
-        }
+				//remove some columns by default
+				$i--;
+				unset($col_key, $col);
+			}
+		}
 
-        return $result;
-    }
+		return $result;
+	}
 
 	public static function display_list( $message = '', $errors = array() ) {
-        global $wpdb, $frm_vars;
+		global $wpdb, $frm_vars;
 
 		$form = FrmForm::get_current_form();
 		$params = FrmForm::get_admin_params( $form );
 
-        if ( $form ) {
-            $params['form'] = $form->id;
-            $frm_vars['current_form'] = $form;
+		if ( $form ) {
+			$params['form'] = $form->id;
+			$frm_vars['current_form'] = $form;
 
-	        if ( 'trash' == $form->status ) {
-	            $delete_timestamp = time() - ( DAY_IN_SECONDS * EMPTY_TRASH_DAYS );
-	            $time_to_delete = FrmAppHelper::human_time_diff( $delete_timestamp, ( isset( $form->options['trash_time'] ) ? ( $form->options['trash_time'] ) : time() ) );
-	            $errors['trash'] = sprintf( __( 'This form is in the trash and is scheduled to be deleted permanently in %s along with any entries.', 'formidable' ), $time_to_delete );
-	            unset( $time_to_delete, $delete_timestamp );
-	        }
+			if ( 'trash' == $form->status ) {
+				$delete_timestamp = time() - ( DAY_IN_SECONDS * EMPTY_TRASH_DAYS );
+				$time_to_delete = FrmAppHelper::human_time_diff( $delete_timestamp, ( isset( $form->options['trash_time'] ) ? ( $form->options['trash_time'] ) : time() ) );
+				$errors['trash'] = sprintf( __( 'This form is in the trash and is scheduled to be deleted permanently in %s along with any entries.', 'formidable' ), $time_to_delete );
+				unset( $time_to_delete, $delete_timestamp );
+			}
 		}
 
-        $table_class = apply_filters( 'frm_entries_list_class', 'FrmEntriesListHelper' );
+		$table_class = apply_filters( 'frm_entries_list_class', 'FrmEntriesListHelper' );
 
-        $wp_list_table = new $table_class( array( 'params' => $params ) );
+		$wp_list_table = new $table_class( array( 'params' => $params ) );
 
-        $pagenum = $wp_list_table->get_pagenum();
+		$pagenum = $wp_list_table->get_pagenum();
 
-        $wp_list_table->prepare_items();
+		$wp_list_table->prepare_items();
 
-        $total_pages = $wp_list_table->get_pagination_arg( 'total_pages' );
-        if ( $pagenum > $total_pages && $total_pages > 0 ) {
+		$total_pages = $wp_list_table->get_pagination_arg( 'total_pages' );
+		if ( $pagenum > $total_pages && $total_pages > 0 ) {
 			$url = add_query_arg( 'paged', $total_pages );
-            if ( headers_sent() ) {
-                echo FrmAppHelper::js_redirect($url);
-            } else {
-                wp_redirect( esc_url_raw( $url ) );
-            }
-            die();
-        }
-
-        if ( empty($message) && isset($_GET['import-message']) ) {
-            $message = __( 'Your import is complete', 'formidable' );
-        }
+			if ( headers_sent() ) {
+				echo FrmAppHelper::js_redirect($url);
+			} else {
+				wp_redirect( esc_url_raw( $url ) );
+			}
+			die();
+		}
+
+		if ( empty($message) && isset($_GET['import-message']) ) {
+			$message = __( 'Your import is complete', 'formidable' );
+		}
 
 		require( FrmAppHelper::plugin_path() . '/classes/views/frm-entries/list.php' );
-    }
+	}
 
-    /* Back End CRUD */
+	/* Back End CRUD */
 	public static function show( $id = 0 ) {
-        FrmAppHelper::permission_check('frm_view_entries');
+		FrmAppHelper::permission_check('frm_view_entries');
 
-        if ( ! $id ) {
+		if ( ! $id ) {
 			$id = FrmAppHelper::get_param( 'id', 0, 'get', 'absint' );
 
-            if ( ! $id ) {
+			if ( ! $id ) {
 				$id = FrmAppHelper::get_param( 'item_id', 0, 'get', 'absint' );
-            }
-        }
+			}
+		}
 
-        $entry = FrmEntry::getOne($id, true);
+		$entry = FrmEntry::getOne($id, true);
 		if ( ! $entry ) {
 			echo '<div id="form_show_entry_page" class="wrap">' .
 				__( 'You are trying to view an entry that does not exist.', 'formidable' ) .
@@ -345,110 +345,110 @@ 
 			return;
 		}
 
-        $data = maybe_unserialize($entry->description);
+		$data = maybe_unserialize($entry->description);
 		if ( ! is_array( $data ) || ! isset( $data['referrer'] ) ) {
 			$data = array( 'referrer' => $data );
 		}
 
 		$fields = FrmField::get_all_for_form( $entry->form_id, '', 'include' );
-        $to_emails = array();
+		$to_emails = array();
 
 		include( FrmAppHelper::plugin_path() . '/classes/views/frm-entries/show.php' );
-    }
+	}
 
-    public static function destroy() {
-        FrmAppHelper::permission_check('frm_delete_entries');
+	public static function destroy() {
+		FrmAppHelper::permission_check('frm_delete_entries');
 
 		$params = FrmForm::get_admin_params();
 
-        if ( isset($params['keep_post']) && $params['keep_post'] ) {
-            //unlink entry from post
-            global $wpdb;
+		if ( isset($params['keep_post']) && $params['keep_post'] ) {
+			//unlink entry from post
+			global $wpdb;
 			$wpdb->update( $wpdb->prefix . 'frm_items', array( 'post_id' => '' ), array( 'id' => $params['id'] ) );
-        }
+		}
 
-        $message = '';
-        if ( FrmEntry::destroy( $params['id'] ) ) {
-            $message = __( 'Entry was Successfully Destroyed', 'formidable' );
-        }
+		$message = '';
+		if ( FrmEntry::destroy( $params['id'] ) ) {
+			$message = __( 'Entry was Successfully Destroyed', 'formidable' );
+		}
 
-        self::display_list( $message );
-    }
+		self::display_list( $message );
+	}
 
-    public static function destroy_all() {
-        if ( ! current_user_can( 'frm_delete_entries' ) ) {
-            $frm_settings = FrmAppHelper::get_settings();
-            wp_die( $frm_settings->admin_permission );
-        }
+	public static function destroy_all() {
+		if ( ! current_user_can( 'frm_delete_entries' ) ) {
+			$frm_settings = FrmAppHelper::get_settings();
+			wp_die( $frm_settings->admin_permission );
+		}
 
-        global $wpdb;
+		global $wpdb;
 		$params = FrmForm::get_admin_params();
-        $message = '';
-        $errors = array();
-        $form_id = (int) $params['form'];
+		$message = '';
+		$errors = array();
+		$form_id = (int) $params['form'];
 
-        if ( $form_id ) {
-            $entry_ids = FrmDb::get_col( 'frm_items', array( 'form_id' => $form_id ) );
+		if ( $form_id ) {
+			$entry_ids = FrmDb::get_col( 'frm_items', array( 'form_id' => $form_id ) );
 			$action = FrmFormAction::get_action_for_form( $form_id, 'wppost', 1 );
 
-            if ( $action ) {
-                // this action takes a while, so only trigger it if there are posts to delete
-                foreach ( $entry_ids as $entry_id ) {
-                    do_action( 'frm_before_destroy_entry', $entry_id );
-                    unset( $entry_id );
-                }
-            }
-
-            $wpdb->query( $wpdb->prepare( "DELETE em.* FROM {$wpdb->prefix}frm_item_metas as em INNER JOIN {$wpdb->prefix}frm_items as e on (em.item_id=e.id) and form_id=%d", $form_id ) );
-            $results = $wpdb->query( $wpdb->prepare( "DELETE FROM {$wpdb->prefix}frm_items WHERE form_id=%d", $form_id ) );
-            if ( $results ) {
+			if ( $action ) {
+				// this action takes a while, so only trigger it if there are posts to delete
+				foreach ( $entry_ids as $entry_id ) {
+					do_action( 'frm_before_destroy_entry', $entry_id );
+					unset( $entry_id );
+				}
+			}
+
+			$wpdb->query( $wpdb->prepare( "DELETE em.* FROM {$wpdb->prefix}frm_item_metas as em INNER JOIN {$wpdb->prefix}frm_items as e on (em.item_id=e.id) and form_id=%d", $form_id ) );
+			$results = $wpdb->query( $wpdb->prepare( "DELETE FROM {$wpdb->prefix}frm_items WHERE form_id=%d", $form_id ) );
+			if ( $results ) {
 				FrmEntry::clear_cache();
-                $message = __( 'Entries were Successfully Destroyed', 'formidable' );
-            }
-        } else {
-            $errors = __( 'No entries were specified', 'formidable' );
-        }
-
-        self::display_list( $message, $errors );
-    }
-
-    public static function show_form( $id = '', $key = '', $title = false, $description = false ) {
-        _deprecated_function( __FUNCTION__, '1.07.05', 'FrmFormsController::show_form()' );
-        return FrmFormsController::show_form( $id, $key, $title, $description );
-    }
-
-    public static function get_form( $filename, $form, $title, $description ) {
-        _deprecated_function( __FUNCTION__, '1.07.05', 'FrmFormsController::get_form()' );
-        return FrmFormsController::get_form( $form, $title, $description );
-    }
-
-    public static function process_entry( $errors = '', $ajax = false ) {
+				$message = __( 'Entries were Successfully Destroyed', 'formidable' );
+			}
+		} else {
+			$errors = __( 'No entries were specified', 'formidable' );
+		}
+
+		self::display_list( $message, $errors );
+	}
+
+	public static function show_form( $id = '', $key = '', $title = false, $description = false ) {
+		_deprecated_function( __FUNCTION__, '1.07.05', 'FrmFormsController::show_form()' );
+		return FrmFormsController::show_form( $id, $key, $title, $description );
+	}
+
+	public static function get_form( $filename, $form, $title, $description ) {
+		_deprecated_function( __FUNCTION__, '1.07.05', 'FrmFormsController::get_form()' );
+		return FrmFormsController::get_form( $form, $title, $description );
+	}
+
+	public static function process_entry( $errors = '', $ajax = false ) {
 		$form_id = FrmAppHelper::get_post_param( 'form_id', '', 'absint' );
 		if ( FrmAppHelper::is_admin() || empty( $_POST ) || empty( $form_id ) || ! isset( $_POST['item_key'] ) ) {
-            return;
-        }
+			return;
+		}
 
-        global $frm_vars;
+		global $frm_vars;
 
 		$form = FrmForm::getOne( $form_id );
-        if ( ! $form ) {
-            return;
-        }
+		if ( ! $form ) {
+			return;
+		}
 
 		$params = FrmForm::get_params( $form );
 
-        if ( ! isset( $frm_vars['form_params'] ) ) {
-            $frm_vars['form_params'] = array();
-        }
+		if ( ! isset( $frm_vars['form_params'] ) ) {
+			$frm_vars['form_params'] = array();
+		}
 		$frm_vars['form_params'][ $form->id ] = $params;
 
 		if ( isset( $frm_vars['created_entries'][ $form_id ] ) ) {
-            return;
-        }
+			return;
+		}
 
-        if ( $errors == '' && ! $ajax ) {
+		if ( $errors == '' && ! $ajax ) {
 			$errors = FrmEntryValidate::validate( $_POST );
-        }
+		}
 
 		/**
 		 * Use this filter to add trigger actions and add errors after
@@ -459,54 +459,54 @@ 
 
 		$frm_vars['created_entries'][ $form_id ] = array( 'errors' => $errors );
 
-        if ( empty( $errors ) ) {
+		if ( empty( $errors ) ) {
 			$_POST['frm_skip_cookie'] = 1;
-            if ( $params['action'] == 'create' ) {
+			if ( $params['action'] == 'create' ) {
 				if ( apply_filters( 'frm_continue_to_create', true, $form_id ) && ! isset( $frm_vars['created_entries'][ $form_id ]['entry_id'] ) ) {
 					$frm_vars['created_entries'][ $form_id ]['entry_id'] = FrmEntry::create( $_POST );
-                }
-            }
+				}
+			}
 
-            do_action( 'frm_process_entry', $params, $errors, $form, array( 'ajax' => $ajax ) );
+			do_action( 'frm_process_entry', $params, $errors, $form, array( 'ajax' => $ajax ) );
 			unset( $_POST['frm_skip_cookie'] );
-        }
-    }
-
-    public static function delete_entry_before_redirect( $url, $form, $atts ) {
-        self::_delete_entry( $atts['id'], $form );
-        return $url;
-    }
-
-    //Delete entry if not redirected
-    public static function delete_entry_after_save( $atts ) {
-        self::_delete_entry( $atts['entry_id'], $atts['form'] );
-    }
-
-    private static function _delete_entry( $entry_id, $form ) {
-        if ( ! $form ) {
-            return;
-        }
-
-        $form->options = maybe_unserialize( $form->options );
-        if ( isset( $form->options['no_save'] ) && $form->options['no_save'] ) {
-            FrmEntry::destroy( $entry_id );
-        }
-    }
+		}
+	}
+
+	public static function delete_entry_before_redirect( $url, $form, $atts ) {
+		self::_delete_entry( $atts['id'], $form );
+		return $url;
+	}
+
+	//Delete entry if not redirected
+	public static function delete_entry_after_save( $atts ) {
+		self::_delete_entry( $atts['entry_id'], $atts['form'] );
+	}
+
+	private static function _delete_entry( $entry_id, $form ) {
+		if ( ! $form ) {
+			return;
+		}
+
+		$form->options = maybe_unserialize( $form->options );
+		if ( isset( $form->options['no_save'] ) && $form->options['no_save'] ) {
+			FrmEntry::destroy( $entry_id );
+		}
+	}
 
 	public static function show_entry_shortcode( $atts ) {
 		return FrmEntryFormat::show_entry( $atts );
 	}
 
 	public static function filter_shortcode_value( $value, $tag, $atts, $field ) {
-        $plain_text = add_filter('frm_plain_text_email', true);
+		$plain_text = add_filter('frm_plain_text_email', true);
 		FrmEntryFormat::textarea_display_value( $field->type, $plain_text, $value );
 
-        if ( isset($atts['show']) && $atts['show'] == 'value' ) {
-            return $value;
-        }
+		if ( isset($atts['show']) && $atts['show'] == 'value' ) {
+			return $value;
+		}
 
-        return $value;
-    }
+		return $value;
+	}
 
 	public static function get_params( $form = null ) {
 		_deprecated_function( __FUNCTION__, '2.0.9', 'FrmForm::get_params' );
@@ -514,15 +514,15 @@ 
 	}
 
 	public static function entry_sidebar( $entry ) {
-        $data = maybe_unserialize($entry->description);
-        $date_format = get_option('date_format');
-        $time_format = get_option('time_format');
+		$data = maybe_unserialize($entry->description);
+		$date_format = get_option('date_format');
+		$time_format = get_option('time_format');
 		if ( isset( $data['browser'] ) ) {
 			$browser = FrmEntryFormat::get_browser( $data['browser'] );
 		}
 
 		include( FrmAppHelper::plugin_path() . '/classes/views/frm-entries/sidebar-shared.php' );
-    }
+	}
 
 	/***********************************************************************
 	 * Deprecated Functions

Spacing +50 added lines, -50 removed lines

@@ -5,7 +5,7 @@ 
     public static function menu() {
 		FrmAppHelper::force_capability( 'frm_view_entries' );
 
-        add_submenu_page('formidable', 'Formidable | ' . __( 'Entries', 'formidable' ), __( 'Entries', 'formidable' ), 'frm_view_entries', 'formidable-entries', 'FrmEntriesController::route' );
+        add_submenu_page( 'formidable', 'Formidable | ' . __( 'Entries', 'formidable' ), __( 'Entries', 'formidable' ), 'frm_view_entries', 'formidable-entries', 'FrmEntriesController::route' );
 
 		if ( ! in_array( FrmAppHelper::simple_get( 'frm_action', 'sanitize_title' ), array( 'edit', 'show' ) ) ) {
 			$menu_name = FrmAppHelper::get_menu_name();
@@ -53,7 +53,7 @@ 
             'id'      => 'formidable-entries-tab',
             'title'   => __( 'Overview', 'formidable' ),
 			'content' => '<p>' . esc_html__( 'This screen provides access to all of your entries. You can customize the display of this screen to suit your workflow.', 'formidable' ) . '</p> <p>' . esc_html__( 'Hovering over a row in the entries list will display action links that allow you to manage your entry.', 'formidable' ) . '</p>',
-        ));
+        ) );
 
         $screen->set_help_sidebar(
 			'<p><strong>' . esc_html__( 'For more information:', 'formidable' ) . '</strong></p>' .
@@ -68,14 +68,14 @@ 
         global $frm_vars, $wpdb;
 		$form_id = FrmForm::get_current_form_id();
 
-		$columns[ $form_id . '_id' ] = 'ID';
-		$columns[ $form_id . '_item_key' ] = esc_html__( 'Entry Key', 'formidable' );
+		$columns[$form_id . '_id'] = 'ID';
+		$columns[$form_id . '_item_key'] = esc_html__( 'Entry Key', 'formidable' );
 
         if ( ! $form_id ) {
             return $columns;
         }
 
-        $form_cols = FrmField::get_all_for_form($form_id, '', 'include');
+        $form_cols = FrmField::get_all_for_form( $form_id, '', 'include' );
 
         foreach ( $form_cols as $form_col ) {
 			if ( FrmField::is_no_save_field( $form_col->type ) ) {
@@ -88,30 +88,30 @@ 
                 if ( $sub_form_cols ) {
                     foreach ( $sub_form_cols as $k => $sub_form_col ) {
 						if ( FrmField::is_no_save_field( $sub_form_col->type ) ) {
-                            unset( $sub_form_cols[ $k ] );
+                            unset( $sub_form_cols[$k] );
                             continue;
                         }
-						$columns[ $form_id . '_' . $sub_form_col->field_key . '-_-' . $form_col->id ] = FrmAppHelper::truncate( $sub_form_col->name, 35 );
-                        unset($sub_form_col);
+						$columns[$form_id . '_' . $sub_form_col->field_key . '-_-' . $form_col->id] = FrmAppHelper::truncate( $sub_form_col->name, 35 );
+                        unset( $sub_form_col );
                     }
                 }
-                unset($sub_form_cols);
+                unset( $sub_form_cols );
             } else {
                 $col_id = $form_col->field_key;
                 if ( $form_col->form_id != $form_id ) {
 					$col_id .= '-_-form' . $form_col->form_id;
                 }
 
-                if ( isset($form_col->field_options['separate_value']) && $form_col->field_options['separate_value'] ) {
-					$columns[ $form_id . '_frmsep_' . $col_id ] = FrmAppHelper::truncate( $form_col->name, 35 );
+                if ( isset( $form_col->field_options['separate_value'] ) && $form_col->field_options['separate_value'] ) {
+					$columns[$form_id . '_frmsep_' . $col_id] = FrmAppHelper::truncate( $form_col->name, 35 );
                 }
-				$columns[ $form_id . '_' . $col_id ] = FrmAppHelper::truncate( $form_col->name, 35 );
+				$columns[$form_id . '_' . $col_id] = FrmAppHelper::truncate( $form_col->name, 35 );
             }
         }
 
-		$columns[ $form_id . '_created_at' ] = __( 'Entry creation date', 'formidable' );
-		$columns[ $form_id . '_updated_at' ] = __( 'Entry update date', 'formidable' );
-		$columns[ $form_id . '_ip' ] = 'IP';
+		$columns[$form_id . '_created_at'] = __( 'Entry creation date', 'formidable' );
+		$columns[$form_id . '_updated_at'] = __( 'Entry update date', 'formidable' );
+		$columns[$form_id . '_ip'] = 'IP';
 
         $frm_vars['cols'] = $columns;
 
@@ -136,7 +136,7 @@ 
 
         global $frm_vars;
         //add a check so we don't create a loop
-        $frm_vars['prev_hidden_cols'] = ( isset($frm_vars['prev_hidden_cols']) && $frm_vars['prev_hidden_cols'] ) ? false : $prev_value;
+        $frm_vars['prev_hidden_cols'] = ( isset( $frm_vars['prev_hidden_cols'] ) && $frm_vars['prev_hidden_cols'] ) ? false : $prev_value;
 
         return $check;
     }
@@ -151,19 +151,19 @@ 
         }
 
         global $frm_vars;
-        if ( ! isset($frm_vars['prev_hidden_cols']) || ! $frm_vars['prev_hidden_cols'] ) {
+        if ( ! isset( $frm_vars['prev_hidden_cols'] ) || ! $frm_vars['prev_hidden_cols'] ) {
             return; //don't continue if there's no previous value
         }
 
         foreach ( $meta_value as $mk => $mv ) {
             //remove blank values
-            if ( empty( $mv )  ) {
-                unset( $meta_value[ $mk ] );
+            if ( empty( $mv ) ) {
+                unset( $meta_value[$mk] );
             }
         }
 
-        $cur_form_prefix = reset($meta_value);
-        $cur_form_prefix = explode('_', $cur_form_prefix);
+        $cur_form_prefix = reset( $meta_value );
+        $cur_form_prefix = explode( '_', $cur_form_prefix );
         $cur_form_prefix = $cur_form_prefix[0];
         $save = false;
 
@@ -182,7 +182,7 @@ 
 
             $meta_value[] = $prev_hidden;
             $save = true;
-            unset($form_prefix);
+            unset( $form_prefix );
         }
 
 		if ( $save ) {
@@ -214,7 +214,7 @@ 
 		foreach ( $fields as $field ) {
 			if ( $field->type != 'checkbox' && ( ! isset( $field->field_options['post_field'] ) || $field->field_options['post_field'] == '' ) ) {
 				// Can't sort on checkboxes because they are stored serialized, or post fields
-				$columns[ $form_id . '_' . $field->field_key ] = 'meta_' . $field->id;
+				$columns[$form_id . '_' . $field->field_key] = 'meta_' . $field->id;
 			}
 		}
 
@@ -237,7 +237,7 @@ 
                     break;
                 }
 
-                unset($form_prefix);
+                unset( $form_prefix );
             }
         }
 
@@ -245,36 +245,36 @@ 
 			return $result;
 		}
 
-        $i = isset($frm_vars['cols']) ? count($frm_vars['cols']) : 0;
+        $i = isset( $frm_vars['cols'] ) ? count( $frm_vars['cols'] ) : 0;
         $max_columns = 8;
         if ( $i <= $max_columns ) {
 			return $result;
 		}
 
         global $frm_vars;
-        if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] ) {
-            $frm_vars['current_form']->options = maybe_unserialize($frm_vars['current_form']->options);
+        if ( isset( $frm_vars['current_form'] ) && $frm_vars['current_form'] ) {
+            $frm_vars['current_form']->options = maybe_unserialize( $frm_vars['current_form']->options );
         }
 
-        if ( isset($frm_vars['current_form']) && $frm_vars['current_form'] && isset($frm_vars['current_form']->options['hidden_cols']) && ! empty($frm_vars['current_form']->options['hidden_cols']) ) {
+        if ( isset( $frm_vars['current_form'] ) && $frm_vars['current_form'] && isset( $frm_vars['current_form']->options['hidden_cols'] ) && ! empty( $frm_vars['current_form']->options['hidden_cols'] ) ) {
             $result = $frm_vars['current_form']->options['hidden_cols'];
         } else {
             $cols = $frm_vars['cols'];
-            $cols = array_reverse($cols, true);
+            $cols = array_reverse( $cols, true );
 
 			$result[] = $form_id . '_id';
-            $i--;
+            $i --;
 
 			$result[] = $form_id . '_item_key';
-            $i--;
+            $i --;
 
 			foreach ( $cols as $col_key => $col ) {
                 if ( $i > $max_columns ) {
 					$result[] = $col_key;
 				}
                 //remove some columns by default
-                $i--;
-                unset($col_key, $col);
+                $i --;
+                unset( $col_key, $col );
             }
         }
 
@@ -311,14 +311,14 @@ 
         if ( $pagenum > $total_pages && $total_pages > 0 ) {
 			$url = add_query_arg( 'paged', $total_pages );
             if ( headers_sent() ) {
-                echo FrmAppHelper::js_redirect($url);
+                echo FrmAppHelper::js_redirect( $url );
             } else {
                 wp_redirect( esc_url_raw( $url ) );
             }
             die();
         }
 
-        if ( empty($message) && isset($_GET['import-message']) ) {
+        if ( empty( $message ) && isset( $_GET['import-message'] ) ) {
             $message = __( 'Your import is complete', 'formidable' );
         }
 
@@ -327,7 +327,7 @@ 
 
     /* Back End CRUD */
 	public static function show( $id = 0 ) {
-        FrmAppHelper::permission_check('frm_view_entries');
+        FrmAppHelper::permission_check( 'frm_view_entries' );
 
         if ( ! $id ) {
 			$id = FrmAppHelper::get_param( 'id', 0, 'get', 'absint' );
@@ -337,7 +337,7 @@ 
             }
         }
 
-        $entry = FrmEntry::getOne($id, true);
+        $entry = FrmEntry::getOne( $id, true );
 		if ( ! $entry ) {
 			echo '<div id="form_show_entry_page" class="wrap">' .
 				__( 'You are trying to view an entry that does not exist.', 'formidable' ) .
@@ -345,7 +345,7 @@ 
 			return;
 		}
 
-        $data = maybe_unserialize($entry->description);
+        $data = maybe_unserialize( $entry->description );
 		if ( ! is_array( $data ) || ! isset( $data['referrer'] ) ) {
 			$data = array( 'referrer' => $data );
 		}
@@ -357,11 +357,11 @@ 
     }
 
     public static function destroy() {
-        FrmAppHelper::permission_check('frm_delete_entries');
+        FrmAppHelper::permission_check( 'frm_delete_entries' );
 
 		$params = FrmForm::get_admin_params();
 
-        if ( isset($params['keep_post']) && $params['keep_post'] ) {
+        if ( isset( $params['keep_post'] ) && $params['keep_post'] ) {
             //unlink entry from post
             global $wpdb;
 			$wpdb->update( $wpdb->prefix . 'frm_items', array( 'post_id' => '' ), array( 'id' => $params['id'] ) );
@@ -440,9 +440,9 @@ 
         if ( ! isset( $frm_vars['form_params'] ) ) {
             $frm_vars['form_params'] = array();
         }
-		$frm_vars['form_params'][ $form->id ] = $params;
+		$frm_vars['form_params'][$form->id] = $params;
 
-		if ( isset( $frm_vars['created_entries'][ $form_id ] ) ) {
+		if ( isset( $frm_vars['created_entries'][$form_id] ) ) {
             return;
         }
 
@@ -457,13 +457,13 @@ 
 		 */
 		$errors = apply_filters( 'frm_entries_before_create', $errors, $form );
 
-		$frm_vars['created_entries'][ $form_id ] = array( 'errors' => $errors );
+		$frm_vars['created_entries'][$form_id] = array( 'errors' => $errors );
 
         if ( empty( $errors ) ) {
 			$_POST['frm_skip_cookie'] = 1;
             if ( $params['action'] == 'create' ) {
-				if ( apply_filters( 'frm_continue_to_create', true, $form_id ) && ! isset( $frm_vars['created_entries'][ $form_id ]['entry_id'] ) ) {
-					$frm_vars['created_entries'][ $form_id ]['entry_id'] = FrmEntry::create( $_POST );
+				if ( apply_filters( 'frm_continue_to_create', true, $form_id ) && ! isset( $frm_vars['created_entries'][$form_id]['entry_id'] ) ) {
+					$frm_vars['created_entries'][$form_id]['entry_id'] = FrmEntry::create( $_POST );
                 }
             }
 
@@ -498,10 +498,10 @@ 
 	}
 
 	public static function filter_shortcode_value( $value, $tag, $atts, $field ) {
-        $plain_text = add_filter('frm_plain_text_email', true);
+        $plain_text = add_filter( 'frm_plain_text_email', true );
 		FrmEntryFormat::textarea_display_value( $field->type, $plain_text, $value );
 
-        if ( isset($atts['show']) && $atts['show'] == 'value' ) {
+        if ( isset( $atts['show'] ) && $atts['show'] == 'value' ) {
             return $value;
         }
 
@@ -514,9 +514,9 @@ 
 	}
 
 	public static function entry_sidebar( $entry ) {
-        $data = maybe_unserialize($entry->description);
-        $date_format = get_option('date_format');
-        $time_format = get_option('time_format');
+        $data = maybe_unserialize( $entry->description );
+        $date_format = get_option( 'date_format' );
+        $time_format = get_option( 'time_format' );
 		if ( isset( $data['browser'] ) ) {
 			$browser = FrmEntryFormat::get_browser( $data['browser'] );
 		}