Strategy11 / formidable-forms

classes/models/FrmNotification.php

Indentation +268 added lines, -268 removed lines

@@ -2,165 +2,165 @@ 
 
 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 ) {
 		if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING  ) {
-            return;
-        }
-
-        global $wpdb;
-
-        $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
+			return;
+		}
+
+		global $wpdb;
+
+		$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);
-            $mail_body .= "\r\n\r\n" . __( 'User Information', 'formidable' ) ."\r\n";
-            $mail_body .= __( 'IP Address', 'formidable' ) . ': '. $entry->ip ."\r\n";
+		// 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,
-                ) );
+			$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( $to_emails[ $email_key ] );
-            }
-        }
-
-        // 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,
-        ) );
-
-        return $sent_to;
-    }
+			}
+		}
+
+		// 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,
+		) );
+
+		return $sent_to;
+	}
 
 	public function entry_created( $entry_id, $form_id ) {
-        _deprecated_function( __FUNCTION__, '2.0', 'FrmFormActionsController::trigger_actions("create", '. $form_id .', '. $entry_id .', "email")');
-        FrmFormActionsController::trigger_actions('create', $form_id, $entry_id, 'email');
-    }
+		_deprecated_function( __FUNCTION__, '2.0', '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 ;.
@@ -178,53 +178,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 ) {
@@ -234,48 +234,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' ) ) {
@@ -288,110 +288,110 @@ 
 			$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(),
-        );
-        $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 );
-
-        $recipient      = $atts['to_email']; //recipient
-        $header         = array();
-        $header[]       = 'From: ' . $atts['from'];
-
-        //Allow for cc and bcc arrays
+		$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(),
+		);
+		$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 );
+
+		$recipient      = $atts['to_email']; //recipient
+		$header         = array();
+		$header[]       = 'From: ' . $atts['from'];
+
+		//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';
-        $charset        = get_option('blog_charset');
+		$content_type   = $atts['plain_text'] ? 'text/plain' : 'text/html';
+		$charset        = get_option('blog_charset');
 
-        $header[]       = 'Reply-To: '. $atts['reply_to'];
+		$header[]       = 'Reply-To: '. $atts['reply_to'];
 		$header[]       = 'Content-Type: ' . $content_type . '; charset="' . esc_attr( $charset ) . '"';
-        $atts['subject'] = wp_specialchars_decode(strip_tags(stripslashes($atts['subject'])), ENT_QUOTES );
+		$atts['subject'] = wp_specialchars_decode(strip_tags(stripslashes($atts['subject'])), ENT_QUOTES );
 
-        $message        = do_shortcode($atts['message']);
+		$message        = do_shortcode($atts['message']);
 
-        if ( $atts['plain_text'] ) {
-            //$message    = wordwrap($message, 70, "\r\n"); //in case any lines are longer than 70 chars
-            $message    = wp_specialchars_decode(strip_tags($message), ENT_QUOTES );
-        } else {
+		if ( $atts['plain_text'] ) {
+			//$message    = wordwrap($message, 70, "\r\n"); //in case any lines are longer than 70 chars
+			$message    = wp_specialchars_decode(strip_tags($message), ENT_QUOTES );
+		} 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' );
+		}
 		$message = apply_filters( 'frm_email_message', $message, $atts );
 
-        $header         = apply_filters('frm_email_header', $header, array(
+		$header         = apply_filters('frm_email_header', $header, array(
 			'to_email' => $atts['to_email'], 'subject' => $atts['subject'],
 		) );
 
-        if ( apply_filters('frm_encode_subject', 1, $atts['subject'] ) ) {
-            $atts['subject'] = '=?'. $charset .'?B?'. base64_encode($atts['subject']) .'?=';
-        }
+		if ( apply_filters('frm_encode_subject', 1, $atts['subject'] ) ) {
+			$atts['subject'] = '=?'. $charset .'?B?'. base64_encode($atts['subject']) .'?=';
+		}
 
-        remove_filter('wp_mail_from', 'bp_core_email_from_address_filter' );
-        remove_filter('wp_mail_from_name', 'bp_core_email_from_name_filter');
+		remove_filter('wp_mail_from', 'bp_core_email_from_address_filter' );
+		remove_filter('wp_mail_from_name', 'bp_core_email_from_name_filter');
 
-        $sent = wp_mail($recipient, $atts['subject'], $message, $header, $atts['attachments']);
-        if ( ! $sent ) {
-            $header = 'From: '. $atts['from'] ."\r\n";
-            $recipient = implode(',', (array) $recipient);
-            $sent = mail($recipient, $atts['subject'], $message, $header);
-        }
+		$sent = wp_mail($recipient, $atts['subject'], $message, $header, $atts['attachments']);
+		if ( ! $sent ) {
+			$header = 'From: '. $atts['from'] ."\r\n";
+			$recipient = implode(',', (array) $recipient);
+			$sent = mail($recipient, $atts['subject'], $message, $header);
+		}
 
 		// remove the filter now so other emails can still use it
 		remove_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
 
-        do_action('frm_notification', $recipient, $atts['subject'], $message);
+		do_action('frm_notification', $recipient, $atts['subject'], $message);
 
-        if ( $sent ) {
+		if ( $sent ) {
 			$sent_to = array_merge( (array) $atts['to_email'], (array) $atts['cc'], (array) $atts['bcc'] );
-            $sent_to = array_filter( $sent_to );
-            if ( apply_filters('frm_echo_emails', false) ) {
-                $temp = str_replace('<', '&lt;', $sent_to);
+			$sent_to = array_filter( $sent_to );
+			if ( apply_filters('frm_echo_emails', false) ) {
+				$temp = str_replace('<', '&lt;', $sent_to);
 				echo ' ' . FrmAppHelper::kses( implode(', ', (array) $temp ) );
-            }
-            return $sent_to;
-        }
-    }
+			}
+			return $sent_to;
+		}
+	}
 
 	/**
 	 * This function should only be fired when Mandrill is sending an HTML email

Spacing +80 added lines, -80 removed lines

@@ -2,14 +2,14 @@ 
 
 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 ) {
-		if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING  ) {
+		if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING ) {
             return;
         }
 
@@ -19,8 +19,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;
@@ -32,7 +32,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' );
@@ -43,29 +43,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
@@ -73,7 +73,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 ) ) {
@@ -83,39 +83,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);
-            $mail_body .= "\r\n\r\n" . __( 'User Information', 'formidable' ) ."\r\n";
-            $mail_body .= __( 'IP Address', 'formidable' ) . ': '. $entry->ip ."\r\n";
+            $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";
+            $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,
@@ -127,7 +127,7 @@ 
                     'email_key' => $email_key,
                 ) );
 
-				unset( $to_emails[ $email_key ] );
+				unset( $to_emails[$email_key] );
             }
         }
 
@@ -148,18 +148,18 @@ 
     }
 
 	public function entry_created( $entry_id, $form_id ) {
-        _deprecated_function( __FUNCTION__, '2.0', 'FrmFormActionsController::trigger_actions("create", '. $form_id .', '. $entry_id .', "email")');
-        FrmFormActionsController::trigger_actions('create', $form_id, $entry_id, 'email');
+        _deprecated_function( __FUNCTION__, '2.0', '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'
-        ));
+        ) );
     }
 
 	/**
@@ -192,28 +192,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
@@ -221,7 +221,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;
         }
     }
@@ -244,20 +244,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 ) ) {
@@ -265,14 +265,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;
             }
         }
@@ -289,18 +289,18 @@ 
 		}
 
         // Set up formatted value
-		$final_val = str_replace( '"', '', $part_1 ) . ' <'. $part_2 .'>';
+		$final_val = str_replace( '"', '', $part_1 ) . ' <' . $part_2 . '>';
 
         // 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'       => '',
@@ -313,7 +313,7 @@ 
             'reply_to'      => $admin_email,
             'attachments'   => array(),
         );
-        $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 );
@@ -326,68 +326,68 @@ 
 		$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
 
 		foreach ( $cc as $k => $v ) {
-			$header[] = $k . ': '. implode( ',', $v );
+			$header[] = $k . ': ' . implode( ',', $v );
 		}
 
         $content_type   = $atts['plain_text'] ? 'text/plain' : 'text/html';
-        $charset        = get_option('blog_charset');
+        $charset        = get_option( 'blog_charset' );
 
-        $header[]       = 'Reply-To: '. $atts['reply_to'];
-		$header[]       = 'Content-Type: ' . $content_type . '; charset="' . esc_attr( $charset ) . '"';
-        $atts['subject'] = wp_specialchars_decode(strip_tags(stripslashes($atts['subject'])), ENT_QUOTES );
+        $header[]       = 'Reply-To: ' . $atts['reply_to'];
+		$header[] = 'Content-Type: ' . $content_type . '; charset="' . esc_attr( $charset ) . '"';
+        $atts['subject'] = wp_specialchars_decode( strip_tags( stripslashes( $atts['subject'] ) ), ENT_QUOTES );
 
-        $message        = do_shortcode($atts['message']);
+        $message = do_shortcode( $atts['message'] );
 
         if ( $atts['plain_text'] ) {
             //$message    = wordwrap($message, 70, "\r\n"); //in case any lines are longer than 70 chars
-            $message    = wp_specialchars_decode(strip_tags($message), ENT_QUOTES );
+            $message = wp_specialchars_decode( strip_tags( $message ), ENT_QUOTES );
         } else {
 			// remove line breaks in HTML emails to prevent conflicts with Mandrill
         	add_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
         }
 		$message = apply_filters( 'frm_email_message', $message, $atts );
 
-        $header         = apply_filters('frm_email_header', $header, array(
+        $header = apply_filters( 'frm_email_header', $header, array(
 			'to_email' => $atts['to_email'], 'subject' => $atts['subject'],
 		) );
 
-        if ( apply_filters('frm_encode_subject', 1, $atts['subject'] ) ) {
-            $atts['subject'] = '=?'. $charset .'?B?'. base64_encode($atts['subject']) .'?=';
+        if ( apply_filters( 'frm_encode_subject', 1, $atts['subject'] ) ) {
+            $atts['subject'] = '=?' . $charset . '?B?' . base64_encode( $atts['subject'] ) . '?=';
         }
 
-        remove_filter('wp_mail_from', 'bp_core_email_from_address_filter' );
-        remove_filter('wp_mail_from_name', 'bp_core_email_from_name_filter');
+        remove_filter( 'wp_mail_from', 'bp_core_email_from_address_filter' );
+        remove_filter( 'wp_mail_from_name', 'bp_core_email_from_name_filter' );
 
-        $sent = wp_mail($recipient, $atts['subject'], $message, $header, $atts['attachments']);
+        $sent = wp_mail( $recipient, $atts['subject'], $message, $header, $atts['attachments'] );
         if ( ! $sent ) {
-            $header = 'From: '. $atts['from'] ."\r\n";
-            $recipient = implode(',', (array) $recipient);
-            $sent = mail($recipient, $atts['subject'], $message, $header);
+            $header = 'From: ' . $atts['from'] . "\r\n";
+            $recipient = implode( ',', (array) $recipient );
+            $sent = mail( $recipient, $atts['subject'], $message, $header );
         }
 
 		// remove the filter now so other emails can still use it
 		remove_filter( 'mandrill_nl2br', 'FrmNotification::remove_mandrill_br' );
 
-        do_action('frm_notification', $recipient, $atts['subject'], $message);
+        do_action( 'frm_notification', $recipient, $atts['subject'], $message );
 
         if ( $sent ) {
 			$sent_to = array_merge( (array) $atts['to_email'], (array) $atts['cc'], (array) $atts['bcc'] );
             $sent_to = array_filter( $sent_to );
-            if ( apply_filters('frm_echo_emails', false) ) {
-                $temp = str_replace('<', '&lt;', $sent_to);
-				echo ' ' . FrmAppHelper::kses( implode(', ', (array) $temp ) );
+            if ( apply_filters( 'frm_echo_emails', false ) ) {
+                $temp = str_replace( '<', '&lt;', $sent_to );
+				echo ' ' . FrmAppHelper::kses( implode( ', ', (array) $temp ) );
             }
             return $sent_to;
         }

classes/models/FrmSettings.php

Spacing +54 added lines, -54 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class FrmSettings{
+class FrmSettings {
     public $option_name = 'frm_options';
     public $menu;
     public $mu_menu;
@@ -28,19 +28,19 @@ 
     public $re_msg;
 
     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.' );
         }
 
-        $settings = get_transient($this->option_name);
+        $settings = get_transient( $this->option_name );
 
-        if ( ! is_object($settings) ) {
-            $settings = $this->translate_settings($settings);
+        if ( ! is_object( $settings ) ) {
+            $settings = $this->translate_settings( $settings );
         }
 
         foreach ( $settings as $setting_name => $setting ) {
             $this->{$setting_name} = $setting;
-            unset($setting_name, $setting);
+            unset( $setting_name, $setting );
         }
 
         $this->set_default_options();
@@ -48,24 +48,24 @@ 
 
 	private function translate_settings( $settings ) {
         if ( $settings ) { //workaround for W3 total cache conflict
-            return unserialize(serialize($settings));
+            return unserialize( serialize( $settings ) );
         }
 
-        $settings = get_option($this->option_name);
-        if ( is_object($settings) ) {
-            set_transient($this->option_name, $settings);
+        $settings = get_option( $this->option_name );
+        if ( is_object( $settings ) ) {
+            set_transient( $this->option_name, $settings );
             return $settings;
         }
 
         // If unserializing didn't work
         if ( $settings ) { //workaround for W3 total cache conflict
-            $settings = unserialize(serialize($settings));
+            $settings = unserialize( serialize( $settings ) );
         } else {
             $settings = $this;
         }
 
-        update_option($this->option_name, $settings);
-        set_transient($this->option_name, $settings);
+        update_option( $this->option_name, $settings );
+        set_transient( $this->option_name, $settings );
 
         return $settings;
     }
@@ -98,8 +98,8 @@ 
 	private function set_default_options() {
         $this->fill_recaptcha_settings();
 
-        if ( ! isset($this->load_style) ) {
-            if ( ! isset($this->custom_style) ) {
+        if ( ! isset( $this->load_style ) ) {
+            if ( ! isset( $this->custom_style ) ) {
                 $this->custom_style = true;
             }
 
@@ -109,16 +109,16 @@ 
         $this->fill_with_defaults();
 
         if ( is_multisite() && is_admin() ) {
-            $mu_menu = get_site_option('frm_admin_menu_name');
-            if ( $mu_menu && ! empty($mu_menu) ) {
+            $mu_menu = get_site_option( 'frm_admin_menu_name' );
+            if ( $mu_menu && ! empty( $mu_menu ) ) {
                 $this->menu = $mu_menu;
                 $this->mu_menu = 1;
             }
         }
 
-        $frm_roles = FrmAppHelper::frm_capabilities('pro');
+        $frm_roles = FrmAppHelper::frm_capabilities( 'pro' );
         foreach ( $frm_roles as $frm_role => $frm_role_description ) {
-            if ( ! isset($this->$frm_role) ) {
+            if ( ! isset( $this->$frm_role ) ) {
                 $this->$frm_role = 'administrator';
             }
         }
@@ -128,12 +128,12 @@ 
         $settings = $this->default_options();
 
         foreach ( $settings as $setting => $default ) {
-			if ( isset( $params[ 'frm_' . $setting ] ) ) {
-				$this->{$setting} = $params[ 'frm_' . $setting ];
-            } else if ( ! isset($this->{$setting}) ) {
+			if ( isset( $params['frm_' . $setting] ) ) {
+				$this->{$setting} = $params['frm_' . $setting];
+            } else if ( ! isset( $this->{$setting}) ) {
                 $this->{$setting} = $default;
             }
-            unset($setting, $default);
+            unset( $setting, $default );
         }
     }
 
@@ -141,23 +141,23 @@ 
         $privkey = '';
         $re_lang = 'en';
 
-        if ( ! isset($this->pubkey) ) {
+        if ( ! isset( $this->pubkey ) ) {
             // get the options from the database
-            $recaptcha_opt = is_multisite() ? get_site_option('recaptcha') : get_option('recaptcha');
-            $this->pubkey = isset($recaptcha_opt['pubkey']) ? $recaptcha_opt['pubkey'] : '';
-            $privkey = isset($recaptcha_opt['privkey']) ? $recaptcha_opt['privkey'] : $privkey;
-            $re_lang = isset($recaptcha_opt['re_lang']) ? $recaptcha_opt['re_lang'] : $re_lang;
+            $recaptcha_opt = is_multisite() ? get_site_option( 'recaptcha' ) : get_option( 'recaptcha' );
+            $this->pubkey = isset( $recaptcha_opt['pubkey'] ) ? $recaptcha_opt['pubkey'] : '';
+            $privkey = isset( $recaptcha_opt['privkey'] ) ? $recaptcha_opt['privkey'] : $privkey;
+            $re_lang = isset( $recaptcha_opt['re_lang'] ) ? $recaptcha_opt['re_lang'] : $re_lang;
         }
 
-        if ( ! isset($this->re_msg) || empty($this->re_msg) ) {
+        if ( ! isset( $this->re_msg ) || empty( $this->re_msg ) ) {
             $this->re_msg = __( 'The reCAPTCHA was not entered correctly', 'formidable' );
         }
 
-        if ( ! isset($this->privkey) ) {
+        if ( ! isset( $this->privkey ) ) {
             $this->privkey = $privkey;
         }
 
-        if ( ! isset($this->re_lang) ) {
+        if ( ! isset( $this->re_lang ) ) {
             $this->re_lang = $re_lang;
         }
     }
@@ -168,31 +168,31 @@ 
     }
 
 	public function update( $params ) {
-        $this->fill_with_defaults($params);
-        $this->update_settings($params);
+        $this->fill_with_defaults( $params );
+        $this->update_settings( $params );
 
         if ( $this->mu_menu ) {
-            update_site_option('frm_admin_menu_name', $this->menu);
-        } else if ( current_user_can('administrator') ) {
-            update_site_option('frm_admin_menu_name', false);
+            update_site_option( 'frm_admin_menu_name', $this->menu );
+        } else if ( current_user_can( 'administrator' ) ) {
+            update_site_option( 'frm_admin_menu_name', false );
         }
 
-        $this->update_roles($params);
+        $this->update_roles( $params );
 
         do_action( 'frm_update_settings', $params );
     }
 
 	private function update_settings( $params ) {
-        $this->mu_menu = isset($params['frm_mu_menu']) ? $params['frm_mu_menu'] : 0;
+        $this->mu_menu = isset( $params['frm_mu_menu'] ) ? $params['frm_mu_menu'] : 0;
 
-        $this->pubkey = trim($params['frm_pubkey']);
+        $this->pubkey = trim( $params['frm_pubkey'] );
         $this->privkey = $params['frm_privkey'];
         $this->re_lang = $params['frm_re_lang'];
 
         $this->load_style = $params['frm_load_style'];
         $this->preview_page_id = (int) $params['frm-preview-page-id'];
 
-        $this->use_html = isset($params['frm_use_html']) ? $params['frm_use_html'] : 0;
+        $this->use_html = isset( $params['frm_use_html'] ) ? $params['frm_use_html'] : 0;
         //$this->custom_style = isset($params['frm_custom_style']) ? $params['frm_custom_style'] : 0;
 		$this->jquery_css = isset( $params['frm_jquery_css'] ) ? absint( $params['frm_jquery_css'] ) : 0;
 		$this->accordion_js = isset( $params['frm_accordion_js'] ) ? absint( $params['frm_accordion_js'] ) : 0;
@@ -205,30 +205,30 @@ 
         $frm_roles = FrmAppHelper::frm_capabilities();
         $roles = get_editable_roles();
         foreach ( $frm_roles as $frm_role => $frm_role_description ) {
-            $this->$frm_role = (array) ( isset( $params[ $frm_role ] ) ? $params[ $frm_role ] : 'administrator' );
+            $this->$frm_role = (array) ( isset( $params[$frm_role] ) ? $params[$frm_role] : 'administrator' );
 
-            if ( count($this->$frm_role) === 1 ) {
-                $set_role = reset($this->$frm_role);
+            if ( count( $this->$frm_role ) === 1 ) {
+                $set_role = reset( $this->$frm_role );
                 switch ( $set_role ) {
                     case 'subscriber':
-                        array_push($this->$frm_role, 'contributor');
+                        array_push( $this->$frm_role, 'contributor' );
                     case 'contributor':
-                        array_push($this->$frm_role, 'author');
+                        array_push( $this->$frm_role, 'author' );
                     case 'author':
-                        array_push($this->$frm_role, 'editor');
+                        array_push( $this->$frm_role, 'editor' );
                     case 'editor':
-                        array_push($this->$frm_role, 'administrator');
+                        array_push( $this->$frm_role, 'administrator' );
                 }
-                unset($set_role);
+                unset( $set_role );
             }
 
             foreach ( $roles as $role => $details ) {
-                if ( in_array($role, $this->$frm_role) ) {
+                if ( in_array( $role, $this->$frm_role ) ) {
     			    $wp_roles->add_cap( $role, $frm_role );
     			} else {
     			    $wp_roles->remove_cap( $role, $frm_role );
     			}
-    			unset($role, $details);
+    			unset( $role, $details );
     		}
 		}
     }
@@ -236,10 +236,10 @@ 
 	public function store() {
         // Save the posted value in the database
 
-        update_option('frm_options', $this);
+        update_option( 'frm_options', $this );
 
-        delete_transient('frm_options');
-        set_transient('frm_options', $this);
+        delete_transient( 'frm_options' );
+        set_transient( 'frm_options', $this );
 
         do_action( 'frm_store_settings' );
     }

Braces +5 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class FrmSettings{
+class FrmSettings {
     public $option_name = 'frm_options';
     public $menu;
     public $mu_menu;
@@ -47,7 +47,8 @@ 
     }
 
 	private function translate_settings( $settings ) {
-        if ( $settings ) { //workaround for W3 total cache conflict
+        if ( $settings ) {
+//workaround for W3 total cache conflict
             return unserialize(serialize($settings));
         }
 
@@ -58,7 +59,8 @@ 
         }
 
         // If unserializing didn't work
-        if ( $settings ) { //workaround for W3 total cache conflict
+        if ( $settings ) {
+//workaround for W3 total cache conflict
             $settings = unserialize(serialize($settings));
         } else {
             $settings = $this;

Indentation +213 added lines, -213 removed lines

@@ -1,246 +1,246 @@
 <?php
 
 class FrmSettings{
-    public $option_name = 'frm_options';
-    public $menu;
-    public $mu_menu;
-    public $preview_page_id;
-    public $use_html;
-    public $jquery_css;
-    public $accordion_js;
-
-    public $success_msg;
-    public $blank_msg;
-    public $unique_msg;
-    public $invalid_msg;
-    public $failed_msg;
-    public $submit_value;
-    public $login_msg;
-    public $admin_permission;
-
-    public $email_to;
-    public $load_style;
-    public $custom_style;
-
-    public $pubkey;
-    public $privkey;
-    public $re_lang;
-    public $re_msg;
-
-    public function __construct() {
-        if ( ! defined('ABSPATH') ) {
-            die('You are not allowed to call this page directly.');
-        }
-
-        $settings = get_transient($this->option_name);
-
-        if ( ! is_object($settings) ) {
-            $settings = $this->translate_settings($settings);
-        }
-
-        foreach ( $settings as $setting_name => $setting ) {
-            $this->{$setting_name} = $setting;
-            unset($setting_name, $setting);
-        }
-
-        $this->set_default_options();
-    }
+	public $option_name = 'frm_options';
+	public $menu;
+	public $mu_menu;
+	public $preview_page_id;
+	public $use_html;
+	public $jquery_css;
+	public $accordion_js;
+
+	public $success_msg;
+	public $blank_msg;
+	public $unique_msg;
+	public $invalid_msg;
+	public $failed_msg;
+	public $submit_value;
+	public $login_msg;
+	public $admin_permission;
+
+	public $email_to;
+	public $load_style;
+	public $custom_style;
+
+	public $pubkey;
+	public $privkey;
+	public $re_lang;
+	public $re_msg;
+
+	public function __construct() {
+		if ( ! defined('ABSPATH') ) {
+			die('You are not allowed to call this page directly.');
+		}
+
+		$settings = get_transient($this->option_name);
+
+		if ( ! is_object($settings) ) {
+			$settings = $this->translate_settings($settings);
+		}
+
+		foreach ( $settings as $setting_name => $setting ) {
+			$this->{$setting_name} = $setting;
+			unset($setting_name, $setting);
+		}
+
+		$this->set_default_options();
+	}
 
 	private function translate_settings( $settings ) {
-        if ( $settings ) { //workaround for W3 total cache conflict
-            return unserialize(serialize($settings));
-        }
-
-        $settings = get_option($this->option_name);
-        if ( is_object($settings) ) {
-            set_transient($this->option_name, $settings);
-            return $settings;
-        }
-
-        // If unserializing didn't work
-        if ( $settings ) { //workaround for W3 total cache conflict
-            $settings = unserialize(serialize($settings));
-        } else {
-            $settings = $this;
-        }
-
-        update_option($this->option_name, $settings);
-        set_transient($this->option_name, $settings);
-
-        return $settings;
-    }
-
-    /**
-     * @return array
-     */
+		if ( $settings ) { //workaround for W3 total cache conflict
+			return unserialize(serialize($settings));
+		}
+
+		$settings = get_option($this->option_name);
+		if ( is_object($settings) ) {
+			set_transient($this->option_name, $settings);
+			return $settings;
+		}
+
+		// If unserializing didn't work
+		if ( $settings ) { //workaround for W3 total cache conflict
+			$settings = unserialize(serialize($settings));
+		} else {
+			$settings = $this;
+		}
+
+		update_option($this->option_name, $settings);
+		set_transient($this->option_name, $settings);
+
+		return $settings;
+	}
+
+	/**
+	 * @return array
+	 */
 	public function default_options() {
-        return array(
-            'menu'      => __( 'Forms', 'formidable' ),
-            'mu_menu'   => 0,
-            'preview_page_id' => 0,
-            'use_html'  => true,
-            'jquery_css' => false,
-            'accordion_js' => false,
-
-            'success_msg' => __( 'Your responses were successfully submitted. Thank you!', 'formidable' ),
-            'blank_msg' => __( 'This field cannot be blank.', 'formidable' ),
-            'unique_msg' => __( 'This value must be unique.', 'formidable' ),
-            'invalid_msg' => __( 'There was a problem with your submission. Errors are marked below.', 'formidable' ),
-            'failed_msg' => __( 'We\'re sorry. It looks like you\'ve  already submitted that.', 'formidable' ),
-            'submit_value' => __( 'Submit', 'formidable' ),
-            'login_msg' => __( 'You do not have permission to view this form.', 'formidable' ),
-            'admin_permission' => __( 'You do not have permission to do that', 'formidable' ),
-
-            'email_to' => '[admin_email]',
-        );
-    }
+		return array(
+			'menu'      => __( 'Forms', 'formidable' ),
+			'mu_menu'   => 0,
+			'preview_page_id' => 0,
+			'use_html'  => true,
+			'jquery_css' => false,
+			'accordion_js' => false,
+
+			'success_msg' => __( 'Your responses were successfully submitted. Thank you!', 'formidable' ),
+			'blank_msg' => __( 'This field cannot be blank.', 'formidable' ),
+			'unique_msg' => __( 'This value must be unique.', 'formidable' ),
+			'invalid_msg' => __( 'There was a problem with your submission. Errors are marked below.', 'formidable' ),
+			'failed_msg' => __( 'We\'re sorry. It looks like you\'ve  already submitted that.', 'formidable' ),
+			'submit_value' => __( 'Submit', 'formidable' ),
+			'login_msg' => __( 'You do not have permission to view this form.', 'formidable' ),
+			'admin_permission' => __( 'You do not have permission to do that', 'formidable' ),
+
+			'email_to' => '[admin_email]',
+		);
+	}
 
 	private function set_default_options() {
-        $this->fill_recaptcha_settings();
-
-        if ( ! isset($this->load_style) ) {
-            if ( ! isset($this->custom_style) ) {
-                $this->custom_style = true;
-            }
-
-            $this->load_style = 'all';
-        }
-
-        $this->fill_with_defaults();
-
-        if ( is_multisite() && is_admin() ) {
-            $mu_menu = get_site_option('frm_admin_menu_name');
-            if ( $mu_menu && ! empty($mu_menu) ) {
-                $this->menu = $mu_menu;
-                $this->mu_menu = 1;
-            }
-        }
-
-        $frm_roles = FrmAppHelper::frm_capabilities('pro');
-        foreach ( $frm_roles as $frm_role => $frm_role_description ) {
-            if ( ! isset($this->$frm_role) ) {
-                $this->$frm_role = 'administrator';
-            }
-        }
-    }
+		$this->fill_recaptcha_settings();
+
+		if ( ! isset($this->load_style) ) {
+			if ( ! isset($this->custom_style) ) {
+				$this->custom_style = true;
+			}
+
+			$this->load_style = 'all';
+		}
+
+		$this->fill_with_defaults();
+
+		if ( is_multisite() && is_admin() ) {
+			$mu_menu = get_site_option('frm_admin_menu_name');
+			if ( $mu_menu && ! empty($mu_menu) ) {
+				$this->menu = $mu_menu;
+				$this->mu_menu = 1;
+			}
+		}
+
+		$frm_roles = FrmAppHelper::frm_capabilities('pro');
+		foreach ( $frm_roles as $frm_role => $frm_role_description ) {
+			if ( ! isset($this->$frm_role) ) {
+				$this->$frm_role = 'administrator';
+			}
+		}
+	}
 
 	public function fill_with_defaults( $params = array() ) {
-        $settings = $this->default_options();
+		$settings = $this->default_options();
 
-        foreach ( $settings as $setting => $default ) {
+		foreach ( $settings as $setting => $default ) {
 			if ( isset( $params[ 'frm_' . $setting ] ) ) {
 				$this->{$setting} = $params[ 'frm_' . $setting ];
-            } else if ( ! isset($this->{$setting}) ) {
-                $this->{$setting} = $default;
-            }
-            unset($setting, $default);
-        }
-    }
-
-    private function fill_recaptcha_settings() {
-        $privkey = '';
-        $re_lang = 'en';
-
-        if ( ! isset($this->pubkey) ) {
-            // get the options from the database
-            $recaptcha_opt = is_multisite() ? get_site_option('recaptcha') : get_option('recaptcha');
-            $this->pubkey = isset($recaptcha_opt['pubkey']) ? $recaptcha_opt['pubkey'] : '';
-            $privkey = isset($recaptcha_opt['privkey']) ? $recaptcha_opt['privkey'] : $privkey;
-            $re_lang = isset($recaptcha_opt['re_lang']) ? $recaptcha_opt['re_lang'] : $re_lang;
-        }
-
-        if ( ! isset($this->re_msg) || empty($this->re_msg) ) {
-            $this->re_msg = __( 'The reCAPTCHA was not entered correctly', 'formidable' );
-        }
-
-        if ( ! isset($this->privkey) ) {
-            $this->privkey = $privkey;
-        }
-
-        if ( ! isset($this->re_lang) ) {
-            $this->re_lang = $re_lang;
-        }
-    }
-
-    public function validate( $params, $errors ) {
-        $errors = apply_filters( 'frm_validate_settings', $errors, $params );
-        return $errors;
-    }
+			} else if ( ! isset($this->{$setting}) ) {
+				$this->{$setting} = $default;
+			}
+			unset($setting, $default);
+		}
+	}
+
+	private function fill_recaptcha_settings() {
+		$privkey = '';
+		$re_lang = 'en';
+
+		if ( ! isset($this->pubkey) ) {
+			// get the options from the database
+			$recaptcha_opt = is_multisite() ? get_site_option('recaptcha') : get_option('recaptcha');
+			$this->pubkey = isset($recaptcha_opt['pubkey']) ? $recaptcha_opt['pubkey'] : '';
+			$privkey = isset($recaptcha_opt['privkey']) ? $recaptcha_opt['privkey'] : $privkey;
+			$re_lang = isset($recaptcha_opt['re_lang']) ? $recaptcha_opt['re_lang'] : $re_lang;
+		}
+
+		if ( ! isset($this->re_msg) || empty($this->re_msg) ) {
+			$this->re_msg = __( 'The reCAPTCHA was not entered correctly', 'formidable' );
+		}
+
+		if ( ! isset($this->privkey) ) {
+			$this->privkey = $privkey;
+		}
+
+		if ( ! isset($this->re_lang) ) {
+			$this->re_lang = $re_lang;
+		}
+	}
+
+	public function validate( $params, $errors ) {
+		$errors = apply_filters( 'frm_validate_settings', $errors, $params );
+		return $errors;
+	}
 
 	public function update( $params ) {
-        $this->fill_with_defaults($params);
-        $this->update_settings($params);
+		$this->fill_with_defaults($params);
+		$this->update_settings($params);
 
-        if ( $this->mu_menu ) {
-            update_site_option('frm_admin_menu_name', $this->menu);
-        } else if ( current_user_can('administrator') ) {
-            update_site_option('frm_admin_menu_name', false);
-        }
+		if ( $this->mu_menu ) {
+			update_site_option('frm_admin_menu_name', $this->menu);
+		} else if ( current_user_can('administrator') ) {
+			update_site_option('frm_admin_menu_name', false);
+		}
 
-        $this->update_roles($params);
+		$this->update_roles($params);
 
-        do_action( 'frm_update_settings', $params );
-    }
+		do_action( 'frm_update_settings', $params );
+	}
 
 	private function update_settings( $params ) {
-        $this->mu_menu = isset($params['frm_mu_menu']) ? $params['frm_mu_menu'] : 0;
+		$this->mu_menu = isset($params['frm_mu_menu']) ? $params['frm_mu_menu'] : 0;
 
-        $this->pubkey = trim($params['frm_pubkey']);
-        $this->privkey = $params['frm_privkey'];
-        $this->re_lang = $params['frm_re_lang'];
+		$this->pubkey = trim($params['frm_pubkey']);
+		$this->privkey = $params['frm_privkey'];
+		$this->re_lang = $params['frm_re_lang'];
 
-        $this->load_style = $params['frm_load_style'];
-        $this->preview_page_id = (int) $params['frm-preview-page-id'];
+		$this->load_style = $params['frm_load_style'];
+		$this->preview_page_id = (int) $params['frm-preview-page-id'];
 
-        $this->use_html = isset($params['frm_use_html']) ? $params['frm_use_html'] : 0;
-        //$this->custom_style = isset($params['frm_custom_style']) ? $params['frm_custom_style'] : 0;
+		$this->use_html = isset($params['frm_use_html']) ? $params['frm_use_html'] : 0;
+		//$this->custom_style = isset($params['frm_custom_style']) ? $params['frm_custom_style'] : 0;
 		$this->jquery_css = isset( $params['frm_jquery_css'] ) ? absint( $params['frm_jquery_css'] ) : 0;
 		$this->accordion_js = isset( $params['frm_accordion_js'] ) ? absint( $params['frm_accordion_js'] ) : 0;
-    }
+	}
 
 	private function update_roles( $params ) {
-        //update roles
-        global $wp_roles;
-
-        $frm_roles = FrmAppHelper::frm_capabilities();
-        $roles = get_editable_roles();
-        foreach ( $frm_roles as $frm_role => $frm_role_description ) {
-            $this->$frm_role = (array) ( isset( $params[ $frm_role ] ) ? $params[ $frm_role ] : 'administrator' );
-
-            if ( count($this->$frm_role) === 1 ) {
-                $set_role = reset($this->$frm_role);
-                switch ( $set_role ) {
-                    case 'subscriber':
-                        array_push($this->$frm_role, 'contributor');
-                    case 'contributor':
-                        array_push($this->$frm_role, 'author');
-                    case 'author':
-                        array_push($this->$frm_role, 'editor');
-                    case 'editor':
-                        array_push($this->$frm_role, 'administrator');
-                }
-                unset($set_role);
-            }
-
-            foreach ( $roles as $role => $details ) {
-                if ( in_array($role, $this->$frm_role) ) {
-    			    $wp_roles->add_cap( $role, $frm_role );
-    			} else {
-    			    $wp_roles->remove_cap( $role, $frm_role );
-    			}
-    			unset($role, $details);
-    		}
-		}
-    }
+		//update roles
+		global $wp_roles;
+
+		$frm_roles = FrmAppHelper::frm_capabilities();
+		$roles = get_editable_roles();
+		foreach ( $frm_roles as $frm_role => $frm_role_description ) {
+			$this->$frm_role = (array) ( isset( $params[ $frm_role ] ) ? $params[ $frm_role ] : 'administrator' );
+
+			if ( count($this->$frm_role) === 1 ) {
+				$set_role = reset($this->$frm_role);
+				switch ( $set_role ) {
+					case 'subscriber':
+						array_push($this->$frm_role, 'contributor');
+					case 'contributor':
+						array_push($this->$frm_role, 'author');
+					case 'author':
+						array_push($this->$frm_role, 'editor');
+					case 'editor':
+						array_push($this->$frm_role, 'administrator');
+				}
+				unset($set_role);
+			}
+
+			foreach ( $roles as $role => $details ) {
+				if ( in_array($role, $this->$frm_role) ) {
+					$wp_roles->add_cap( $role, $frm_role );
+				} else {
+					$wp_roles->remove_cap( $role, $frm_role );
+				}
+				unset($role, $details);
+			}
+		}
+	}
 
 	public function store() {
-        // Save the posted value in the database
+		// Save the posted value in the database
 
-        update_option('frm_options', $this);
+		update_option('frm_options', $this);
 
-        delete_transient('frm_options');
-        set_transient('frm_options', $this);
+		delete_transient('frm_options');
+		set_transient('frm_options', $this);
 
-        do_action( 'frm_store_settings' );
-    }
+		do_action( 'frm_store_settings' );
+	}
 }

classes/views/addons/settings.php

Spacing +2 added lines, -2 removed lines

@@ -4,8 +4,8 @@
 	<?php
 
 	foreach ( $plugins as $slug => $plugin ) {
-		$license = get_option( 'edd_'. $slug .'_license_key' );
-		$status  = get_option( 'edd_'. $slug .'_license_active' );
+		$license = get_option( 'edd_' . $slug . '_license_key' );
+		$status  = get_option( 'edd_' . $slug . '_license_active' );
 		$activate = ( false !== $license && $status == 'valid' ) ? 'deactivate' : 'activate';
 		$icon_class = ( empty( $license ) ) ? 'frm_hidden' : '';
 		?>

classes/views/frm-entries/_sidebar-shared-pub.php

Indentation +2 added lines, -2 removed lines

@@ -1,12 +1,12 @@
 <?php
 if ( ! isset( $entry) ) {
-    $entry = $record;
+	$entry = $record;
 } ?>
 
 <div class="misc-pub-section curtime misc-pub-curtime">
     <span id="timestamp">
     <?php
-    $date_format = __( 'M j, Y @ G:i' );
+	$date_format = __( 'M j, Y @ G:i' );
 	printf( __( 'Published on: <b>%1$s</b>' ), FrmAppHelper::get_localized_date( $date_format, $entry->created_at ) ); ?>
     </span>
 </div>

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! isset( $entry) ) {
+if ( ! isset( $entry ) ) {
     $entry = $record;
 } ?>
 
@@ -18,4 +18,4 @@ 
 </div>
 <?php } ?>
 
-<?php do_action('frm_entry_shared_sidebar', $entry); ?>
+<?php do_action( 'frm_entry_shared_sidebar', $entry ); ?>

classes/views/frm-entries/errors.php

Indentation +8 added lines, -8 removed lines

@@ -4,30 +4,30 @@
 <?php
 }
 if ( isset( $message ) && $message != '' ) {
-    if ( FrmAppHelper::is_admin() ) {
+	if ( FrmAppHelper::is_admin() ) {
 		?><div id="message" class="frm_message updated frm_msg_padding"><?php echo wp_kses_post( $message ) ?></div><?php
 	} else {
-        FrmFormsHelper::get_scroll_js($form->id);
+		FrmFormsHelper::get_scroll_js($form->id);
 
 		// we need to allow scripts here for javascript in the success message
 		echo $message;
-    }
+	}
 }
 
 if ( isset($errors) && is_array( $errors ) && ! empty( $errors ) ) {
 
 	if ( isset( $form ) && is_object( $form ) ) {
-    	FrmFormsHelper::get_scroll_js( $form->id );
+		FrmFormsHelper::get_scroll_js( $form->id );
 	} ?>
 <div class="frm_error_style">
 <?php
 $img = '';
 if ( ! FrmAppHelper::is_admin() ) {
-    $img = apply_filters('frm_error_icon', $img);
-    if ( $img && ! empty($img) ) {
-    ?><img src="<?php echo esc_attr( $img ) ?>" alt="" />
+	$img = apply_filters('frm_error_icon', $img);
+	if ( $img && ! empty($img) ) {
+	?><img src="<?php echo esc_attr( $img ) ?>" alt="" />
 <?php
-    }
+	}
 }
 
 FrmFormsHelper::show_errors( compact( 'img', 'errors' ) );

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( isset($include_extra_container) ) { ?>
+if ( isset( $include_extra_container ) ) { ?>
 <div class="<?php echo esc_attr( $include_extra_container ) ?>" id="frm_form_<?php echo esc_attr( $form->id ) ?>_container">
 <?php
 }
@@ -7,14 +7,14 @@ 
     if ( FrmAppHelper::is_admin() ) {
 		?><div id="message" class="frm_message updated frm_msg_padding"><?php echo wp_kses_post( $message ) ?></div><?php
 	} else {
-        FrmFormsHelper::get_scroll_js($form->id);
+        FrmFormsHelper::get_scroll_js( $form->id );
 
 		// we need to allow scripts here for javascript in the success message
 		echo $message;
     }
 }
 
-if ( isset($errors) && is_array( $errors ) && ! empty( $errors ) ) {
+if ( isset( $errors ) && is_array( $errors ) && ! empty( $errors ) ) {
 
 	if ( isset( $form ) && is_object( $form ) ) {
     	FrmFormsHelper::get_scroll_js( $form->id );
@@ -23,8 +23,8 @@ 
 <?php
 $img = '';
 if ( ! FrmAppHelper::is_admin() ) {
-    $img = apply_filters('frm_error_icon', $img);
-    if ( $img && ! empty($img) ) {
+    $img = apply_filters( 'frm_error_icon', $img );
+    if ( $img && ! empty( $img ) ) {
     ?><img src="<?php echo esc_attr( $img ) ?>" alt="" />
 <?php
     }
@@ -37,7 +37,7 @@ 
 <?php
 }
 
-if ( isset($include_extra_container) ) { ?>
+if ( isset( $include_extra_container ) ) { ?>
 </div>
 <?php
 }

classes/views/frm-entries/form.php

Indentation +6 added lines, -6 removed lines

@@ -5,7 +5,7 @@ 
     </div>
 </div>
 <?php
-    return;
+	return;
 }
 
 global $frm_vars;
@@ -38,7 +38,7 @@ 
 		} else {
 			do_action( 'frm_show_other_field_type', $field, $form, array( 'action' => $form_action ) );
 		}
-    	do_action('frm_get_field_scripts', $field, $form, $form->id);
+		do_action('frm_get_field_scripts', $field, $form, $form->id);
 	}
 }
 
@@ -63,8 +63,8 @@ 
 
 // close open collapsible toggle div
 if ( isset($frm_vars['collapse_div']) && $frm_vars['collapse_div'] ) {
-    echo "</div>\n";
-    unset($frm_vars['collapse_div']);
+	echo "</div>\n";
+	unset($frm_vars['collapse_div']);
 }
 
 echo FrmFormsHelper::replace_shortcodes($values['after_html'], $form);
@@ -77,8 +77,8 @@ 
 }
 
 if ( FrmForm::show_submit( $form ) ) {
-    unset($values['fields']);
-    FrmFormsHelper::get_custom_submit($values['submit_html'], $form, $submit, $form_action, $values);
+	unset($values['fields']);
+	FrmFormsHelper::get_custom_submit($values['submit_html'], $form, $submit, $form_action, $values);
 }
 ?>
 </fieldset>

Spacing +17 added lines, -17 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( empty($values) || ! isset($values['fields']) || empty($values['fields']) ) { ?>
-<div class="frm_forms <?php echo FrmFormsHelper::get_form_style_class($form); ?>" id="frm_form_<?php echo esc_attr( $form->id ) ?>_container">
+if ( empty( $values ) || ! isset( $values['fields'] ) || empty( $values['fields'] ) ) { ?>
+<div class="frm_forms <?php echo FrmFormsHelper::get_form_style_class( $form ); ?>" id="frm_form_<?php echo esc_attr( $form->id ) ?>_container">
 	<div class="frm_error_style"><strong><?php _e( 'Oops!', 'formidable' ) ?></strong> <?php printf( __( 'You did not add any fields to your form. %1$sGo back%2$s and add some.', 'formidable' ), '<a href="' . esc_url( admin_url( '?page=formidable&frm_action=edit&id=' . $form->id ) ) . '">', '</a>' ) ?>
     </div>
 </div>
@@ -21,11 +21,11 @@ 
 <div class="frm_form_fields <?php echo esc_attr( apply_filters( 'frm_form_fields_class', '', $values ) ); ?>">
 <fieldset>
 <?php echo FrmFormsHelper::replace_shortcodes( $values['before_html'], $form, $title, $description ); ?>
-<input type="hidden" name="frm_action" value="<?php echo esc_attr($form_action) ?>" />
-<input type="hidden" name="form_id" value="<?php echo esc_attr($form->id) ?>" />
-<input type="hidden" name="frm_hide_fields_<?php echo esc_attr( $form->id ) ?>" id="frm_hide_fields_<?php echo esc_attr( $form->id ) ?>" value="<?php echo esc_attr($frm_hide_fields) ?>" />
+<input type="hidden" name="frm_action" value="<?php echo esc_attr( $form_action ) ?>" />
+<input type="hidden" name="form_id" value="<?php echo esc_attr( $form->id ) ?>" />
+<input type="hidden" name="frm_hide_fields_<?php echo esc_attr( $form->id ) ?>" id="frm_hide_fields_<?php echo esc_attr( $form->id ) ?>" value="<?php echo esc_attr( $frm_hide_fields ) ?>" />
 <input type="hidden" name="frm_helpers_<?php echo esc_attr( $form->id ) ?>" id="frm_helpers_<?php echo esc_attr( $form->id ) ?>" value="<?php echo esc_attr( $frm_helpers ) ?>" />
-<input type="hidden" name="form_key" value="<?php echo esc_attr($form->form_key) ?>" />
+<input type="hidden" name="form_key" value="<?php echo esc_attr( $form->form_key ) ?>" />
 <input type="hidden" name="item_meta[0]" value="" />
 <?php wp_nonce_field( 'frm_submit_entry_nonce', 'frm_submit_entry_' . $form->id ); ?>
 
@@ -38,7 +38,7 @@ 
 		} else {
 			do_action( 'frm_show_other_field_type', $field, $form, array( 'action' => $form_action ) );
 		}
-    	do_action('frm_get_field_scripts', $field, $form, $form->id);
+    	do_action( 'frm_get_field_scripts', $field, $form, $form->id );
 	}
 }
 
@@ -46,13 +46,13 @@ 
 if ( FrmAppHelper::is_admin() ) { ?>
 <div class="frm_form_field form-field">
 <label class="frm_primary_label"><?php _e( 'Entry Key', 'formidable' ) ?></label>
-<input type="text" name="item_key" value="<?php echo esc_attr($values['item_key']) ?>" />
+<input type="text" name="item_key" value="<?php echo esc_attr( $values['item_key'] ) ?>" />
 </div>
 <?php } else { ?>
-<input type="hidden" name="item_key" value="<?php echo esc_attr($values['item_key']) ?>" />
+<input type="hidden" name="item_key" value="<?php echo esc_attr( $values['item_key'] ) ?>" />
 <?php }
 
-do_action('frm_entry_form', $form, $form_action, $errors);
+do_action( 'frm_entry_form', $form, $form_action, $errors );
 
 global $frm_vars;
 // close open section div
@@ -62,23 +62,23 @@ 
 }
 
 // close open collapsible toggle div
-if ( isset($frm_vars['collapse_div']) && $frm_vars['collapse_div'] ) {
+if ( isset( $frm_vars['collapse_div'] ) && $frm_vars['collapse_div'] ) {
     echo "</div>\n";
-    unset($frm_vars['collapse_div']);
+    unset( $frm_vars['collapse_div'] );
 }
 
-echo FrmFormsHelper::replace_shortcodes($values['after_html'], $form);
+echo FrmFormsHelper::replace_shortcodes( $values['after_html'], $form );
 
 
-if ( has_action('frm_entries_footer_scripts') ) { ?>
+if ( has_action( 'frm_entries_footer_scripts' ) ) { ?>
 <script type="text/javascript">
-<?php do_action('frm_entries_footer_scripts', $values['fields'], $form); ?>
+<?php do_action( 'frm_entries_footer_scripts', $values['fields'], $form ); ?>
 </script><?php
 }
 
 if ( FrmForm::show_submit( $form ) ) {
-    unset($values['fields']);
-    FrmFormsHelper::get_custom_submit($values['submit_html'], $form, $submit, $form_action, $values);
+    unset( $values['fields'] );
+    FrmFormsHelper::get_custom_submit( $values['submit_html'], $form, $submit, $form_action, $values );
 }
 ?>
 </fieldset>

classes/views/frm-entries/list.php

Spacing +3 added lines, -3 removed lines

@@ -1,16 +1,16 @@
 <div id="form_entries_page" class="wrap">
     <div class="frmicon icon32"><br/></div>
     <h2><?php _e( 'Entries', 'formidable' ); ?>
-        <?php do_action('frm_entry_inside_h2', $form); ?>
+        <?php do_action( 'frm_entry_inside_h2', $form ); ?>
     </h2>
 
-    <?php require(FrmAppHelper::plugin_path() .'/classes/views/shared/errors.php'); ?>
+    <?php require( FrmAppHelper::plugin_path() . '/classes/views/shared/errors.php' ); ?>
 
     <form id="posts-filter" method="get">
         <div id="poststuff">
             <div id="post-body" class="metabox-holder columns-2">
             <div id="post-body-content">
-                <?php FrmAppController::get_form_nav($form, true, 'hide'); ?>
+                <?php FrmAppController::get_form_nav( $form, true, 'hide' ); ?>
             </div>
             <div id="postbox-container-1" class="postbox-container">
                 <input type="hidden" name="page" value="formidable-entries" />

classes/views/frm-entries/new.php

Spacing +5 added lines, -5 removed lines

@@ -1,17 +1,17 @@
-<div class="frm_forms <?php echo FrmFormsHelper::get_form_style_class($values); ?>" id="frm_form_<?php echo esc_attr( $form->id ) ?>_container">
+<div class="frm_forms <?php echo FrmFormsHelper::get_form_style_class( $values ); ?>" id="frm_form_<?php echo esc_attr( $form->id ) ?>_container">
 <?php
 if ( ! isset( $include_form_tag ) || $include_form_tag ) {
 ?>
-<form enctype="<?php echo esc_attr( apply_filters( 'frm_form_enctype', 'multipart/form-data', $form ) ) ?>" method="post" class="frm-show-form <?php do_action('frm_form_classes', $form) ?>" id="form_<?php echo esc_attr( $form->form_key ) ?>" <?php echo $frm_settings->use_html ? '' : 'action=""'; ?>>
+<form enctype="<?php echo esc_attr( apply_filters( 'frm_form_enctype', 'multipart/form-data', $form ) ) ?>" method="post" class="frm-show-form <?php do_action( 'frm_form_classes', $form ) ?>" id="form_<?php echo esc_attr( $form->form_key ) ?>" <?php echo $frm_settings->use_html ? '' : 'action=""'; ?>>
 <?php
 } else { ?>
-<div id="form_<?php echo esc_attr( $form->form_key ) ?>" class="frm-show-form <?php do_action('frm_form_classes', $form) ?>" >
+<div id="form_<?php echo esc_attr( $form->form_key ) ?>" class="frm-show-form <?php do_action( 'frm_form_classes', $form ) ?>" >
 <?php
 }
 
-include(FrmAppHelper::plugin_path() .'/classes/views/frm-entries/errors.php');
+include( FrmAppHelper::plugin_path() . '/classes/views/frm-entries/errors.php' );
 $form_action = 'create';
-require(FrmAppHelper::plugin_path() .'/classes/views/frm-entries/form.php');
+require( FrmAppHelper::plugin_path() . '/classes/views/frm-entries/form.php' );
 
 if ( ! isset( $include_form_tag ) || $include_form_tag ) {
 ?>

classes/views/frm-entries/no_entries.php

Spacing +5 added lines, -5 removed lines

@@ -1,18 +1,18 @@
 <div class="frmcenter frm_no_entries_form">
 <?php
-if ( $form && isset($form->options['no_save']) && $form->options['no_save'] ) { ?>
+if ( $form && isset( $form->options['no_save'] ) && $form->options['no_save'] ) { ?>
 <h3><?php _e( 'This form is not set to save any entries.', 'formidable' ) ?></h3>
-<p>If you would like entries in this form to be saved, go to the <a href="<?php echo esc_url(admin_url('admin.php?page=formidable') . '&frm_action=settings&id='. $form->id) ?>">form Settings</a> page and uncheck the "Do not store any entries submitted from this form" box.</p>
+<p>If you would like entries in this form to be saved, go to the <a href="<?php echo esc_url( admin_url( 'admin.php?page=formidable' ) . '&frm_action=settings&id=' . $form->id ) ?>">form Settings</a> page and uncheck the "Do not store any entries submitted from this form" box.</p>
 <?php
 } else if ( $form ) {
 ?>
-<div class="frm_no_entries_header"><?php printf(__( 'No Entries for form: %s', 'formidable' ), $form->name); ?></div>
-<p class="frm_no_entries_text"><?php printf( __( 'For instructions on publishing your form see %1$sthis page%2$s <br/> or click "%3$sAdd New%4$s" above to add an entry from here (Requires License)', 'formidable' ), '<a href="https://formidablepro.com/knowledgebase/publish-your-forms/" target="_blank">', '</a>', '<a href="' . esc_url( admin_url('admin.php?page=formidable-entries&frm_action=new&form=' . $form->id ) ) . '">', '</a>' ); ?></p>
+<div class="frm_no_entries_header"><?php printf( __( 'No Entries for form: %s', 'formidable' ), $form->name ); ?></div>
+<p class="frm_no_entries_text"><?php printf( __( 'For instructions on publishing your form see %1$sthis page%2$s <br/> or click "%3$sAdd New%4$s" above to add an entry from here (Requires License)', 'formidable' ), '<a href="https://formidablepro.com/knowledgebase/publish-your-forms/" target="_blank">', '</a>', '<a href="' . esc_url( admin_url( 'admin.php?page=formidable-entries&frm_action=new&form=' . $form->id ) ) . '">', '</a>' ); ?></p>
 <?php
 } else {
 ?>
 <div class="frm_no_entries_header"><?php _e( 'You have not created any forms yet.', 'formidable' ); ?></div>
-<p class="frm_no_entries_text"><?php printf( __( 'To view entries, you must first %1$sbuild a form%2$s', 'formidable' ), '<a href="'. esc_url( admin_url( 'admin.php?page=formidable&frm_action=new' ) ) . '">', '</a>' ); ?></p>
+<p class="frm_no_entries_text"><?php printf( __( 'To view entries, you must first %1$sbuild a form%2$s', 'formidable' ), '<a href="' . esc_url( admin_url( 'admin.php?page=formidable&frm_action=new' ) ) . '">', '</a>' ); ?></p>
 <?php
 } ?>
 </div>