Strategy11 / formidable-forms

classes/helpers/FrmFormActionsHelper.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	die( 'You are not allowed to call this page directly.' );
 }
 

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

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 ); ?>

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( esc_html__( 'Published on: %1$s' ), '<b>' . FrmAppHelper::get_localized_date( $date_format, $entry->created_at ) . '</b>' );
 	?>
     </span>

classes/views/frm-entries/sidebar-shared.php

Indentation +7 added lines, -7 removed lines

@@ -66,18 +66,18 @@
         <?php } ?>
 
         <?php
-        foreach ( (array) $data as $k => $d ) {
+		foreach ( (array) $data as $k => $d ) {
 			if ( in_array( $k, array( 'browser', 'referrer' ) ) ) {
-                continue;
-            }
-        ?>
+				continue;
+			}
+		?>
         <div class="misc-pub-section">
 			<b><?php echo sanitize_text_field( ucfirst( str_replace( '-', ' ', $k ) ) ); ?></b>:
 			<?php echo wp_kses_post( implode( ', ', (array) $d ) ); ?>
         </div>
         <?php
-            unset($k, $d);
-        }
-        ?>
+			unset($k, $d);
+		}
+		?>
     </div>
 </div>

Spacing +3 added lines, -3 removed lines

@@ -6,7 +6,7 @@ 
         <div class="misc-pub-section">
             <span class="dashicons dashicons-admin-post wp-media-buttons-icon"></span>
             <?php esc_html_e( 'Post', 'formidable' ) ?>:
-            <b><?php echo get_the_title($entry->post_id) ?></b>
+            <b><?php echo get_the_title( $entry->post_id ) ?></b>
 			<span>
 				<a href="<?php echo esc_url( admin_url( 'post.php?post=' . $entry->post_id . '&action=edit' ) ) ?>">
 					<?php esc_html_e( 'Edit', 'formidable' ) ?>
@@ -66,7 +66,7 @@ 
         </div>
         <?php } ?>
 
-        <?php if ( isset($data['referrer']) ) { ?>
+        <?php if ( isset( $data['referrer'] ) ) { ?>
         <div class="misc-pub-section">
 			<b><?php esc_html_e( 'Referrer', 'formidable' ) ?></b>:<br/>
 			<?php echo wp_kses_post( str_replace( "\r\n", '<br/>', $data['referrer'] ) ); ?>
@@ -84,7 +84,7 @@ 
 			<?php echo wp_kses_post( implode( ', ', (array) $d ) ); ?>
         </div>
         <?php
-            unset($k, $d);
+            unset( $k, $d );
         }
         ?>
     </div>

classes/views/frm-form-actions/default_actions.php

Indentation +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 class FrmDefPostAction extends FrmFormAction {
 	public function __construct() {
 		$action_ops = FrmFormAction::default_action_opts();
-	    $action_ops['classes'] = 'ab-icon frm_dashicon_font dashicons-before';
+		$action_ops['classes'] = 'ab-icon frm_dashicon_font dashicons-before';
 
 		parent::__construct( 'wppost', __( 'Create Post', 'formidable' ), $action_ops );
 	}

Spacing +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
 // add register action
 class FrmDefRegAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_register_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_register_icon' );
 		parent::__construct( 'register', __( 'Register User', 'formidable' ), $action_ops );
 	}
 }
@@ -20,7 +20,7 @@ 
 // add paypal action
 class FrmDefPayPalAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_paypal_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_paypal_icon' );
 		parent::__construct( 'paypal', __( 'Collect Payment', 'formidable' ), $action_ops );
 	}
 }
@@ -28,7 +28,7 @@ 
 // add aweber action
 class FrmDefAweberAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_aweber_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_aweber_icon' );
 		parent::__construct( 'aweber', __( 'Aweber', 'formidable' ), $action_ops );
 	}
 }
@@ -36,7 +36,7 @@ 
 // add mailchimp action
 class FrmDefMlcmpAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_mailchimp_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_mailchimp_icon' );
 		parent::__construct( 'mailchimp', __( 'MailChimp', 'formidable' ), $action_ops );
 	}
 }
@@ -44,7 +44,7 @@ 
 // add twilio action
 class FrmDefTwilioAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_sms_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_sms_icon' );
 		parent::__construct( 'twilio', __( 'Twilio', 'formidable' ), $action_ops );
 	}
 }
@@ -52,7 +52,7 @@ 
 // add highrise action
 class FrmDefHrsAction extends FrmFormAction {
 	public function __construct() {
-		$action_ops = FrmFormAction::default_action_opts('frm_highrise_icon');
+		$action_ops = FrmFormAction::default_action_opts( 'frm_highrise_icon' );
 		parent::__construct( 'highrise', __( 'Highrise', 'formidable' ), $action_ops );
 	}
 }

classes/views/shared/head.php

Spacing +1 added lines, -1 removed lines

@@ -1,2 +1,2 @@
 <?php
-_deprecated_file( basename(__FILE__), '2.0', null, __( 'Use wp_enqueue_script and wp_enqueue_style.', 'formidable' ) );
+_deprecated_file( basename( __FILE__ ), '2.0', null, __( 'Use wp_enqueue_script and wp_enqueue_style.', 'formidable' ) );

classes/views/xml/forms_xml.php

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 if ( ! $item_ids ) {
-    return;
+	return;
 }
 
 // fetch 20 posts at a time rather than loading the entire table into memory
@@ -45,6 +45,6 @@ 
 <?php	} ?>
 	</form>
 <?php
-    	unset( $fields );
+		unset( $fields );
 	}
 }

Spacing +12 added lines, -12 removed lines

@@ -13,16 +13,16 @@ 
 ?>
 	<form>
 		<id><?php echo absint( $form->id ) ?></id>
-		<form_key><?php echo FrmXMLHelper::cdata($form->form_key) ?></form_key>
-		<name><?php echo FrmXMLHelper::cdata($form->name) ?></name>
-		<description><?php echo FrmXMLHelper::cdata($form->description) ?></description>
+		<form_key><?php echo FrmXMLHelper::cdata( $form->form_key ) ?></form_key>
+		<name><?php echo FrmXMLHelper::cdata( $form->name ) ?></name>
+		<description><?php echo FrmXMLHelper::cdata( $form->description ) ?></description>
 		<created_at><?php echo esc_html( $form->created_at ) ?></created_at>
 		<logged_in><?php echo esc_html( $form->logged_in ) ?></logged_in>
 		<is_template><?php echo esc_html( $form->is_template ) ?></is_template>
 		<default_template><?php echo esc_html( $form->default_template ) ?></default_template>
 		<editable><?php echo esc_html( $form->editable ) ?></editable>
-		<options><?php echo FrmXMLHelper::prepare_form_options_for_export($form->options) ?></options>
-		<status><?php echo FrmXMLHelper::cdata($form->status) ?></status>
+		<options><?php echo FrmXMLHelper::prepare_form_options_for_export( $form->options ) ?></options>
+		<status><?php echo FrmXMLHelper::cdata( $form->status ) ?></status>
         <parent_form_id><?php echo esc_html( $form->parent_form_id ) ?></parent_form_id>
 <?php
 
@@ -32,16 +32,16 @@ 
 		?>
 		<field>
 		    <id><?php echo absint( $field->id ) ?></id>
-            <field_key><?php echo FrmXMLHelper::cdata($field->field_key) ?></field_key>
-            <name><?php echo FrmXMLHelper::cdata($field->name) ?></name>
-            <description><?php echo FrmXMLHelper::cdata($field->description) ?></description>
-            <type><?php echo FrmXMLHelper::cdata($field->type) ?></type>
-            <default_value><?php echo FrmXMLHelper::cdata($field->default_value) ?></default_value>
+            <field_key><?php echo FrmXMLHelper::cdata( $field->field_key ) ?></field_key>
+            <name><?php echo FrmXMLHelper::cdata( $field->name ) ?></name>
+            <description><?php echo FrmXMLHelper::cdata( $field->description ) ?></description>
+            <type><?php echo FrmXMLHelper::cdata( $field->type ) ?></type>
+            <default_value><?php echo FrmXMLHelper::cdata( $field->default_value ) ?></default_value>
             <field_order><?php echo absint( $field->field_order ) ?></field_order>
             <form_id><?php echo absint( $field->form_id ) ?></form_id>
             <required><?php echo absint( $field->required ) ?></required>
-            <options><?php echo FrmXMLHelper::cdata($field->options) ?></options>
-            <field_options><?php echo FrmXMLHelper::cdata($field->field_options) ?></field_options>
+            <options><?php echo FrmXMLHelper::cdata( $field->options ) ?></options>
+            <field_options><?php echo FrmXMLHelper::cdata( $field->field_options ) ?></field_options>
 		</field>
 <?php	} ?>
 	</form>

deprecated.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 // 2.0
 if ( ! isset($frm_vars['pro_is_installed']) ) {
-    $frm_vars['pro_is_installed'] = false;
+	$frm_vars['pro_is_installed'] = false;
 }
 
 // Instansiate Models

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 // 2.0
-if ( ! isset($frm_vars['pro_is_installed']) ) {
+if ( ! isset( $frm_vars['pro_is_installed'] ) ) {
     $frm_vars['pro_is_installed'] = false;
 }
 

classes/views/shared/form-nav.php

Spacing +1 added lines, -1 removed lines

@@ -17,6 +17,6 @@
 </ul>
 
 <?php if ( $form && $title == 'show' ) { ?>
-    <input id="title" type="text" value="<?php echo esc_attr( $form->name == '' ? __( '(no title)') : $form->name ) ?>" readonly="readonly" disabled="disabled" />
+    <input id="title" type="text" value="<?php echo esc_attr( $form->name == '' ? __( '(no title)' ) : $form->name ) ?>" readonly="readonly" disabled="disabled" />
 <?php } ?>
 </div>

classes/models/FrmPointers.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	/**
 	 * Get the singleton instance of this class
 	 *
-	 * @return object
+	 * @return FrmPointers
 	 */
 	public static function get_instance() {
 		if ( ! ( self::$instance instanceof self ) ) {

Indentation +16 added lines, -16 removed lines

@@ -239,16 +239,16 @@ 
 
 		return array(
 			'content'   => '<h3>' . __( 'Forms', 'formidable' ) . '</h3>'
-			               . '<p>' . __( 'All your forms will be listed on this page. Create your first form by clicking on the "Add New" button.', 'formidable' ) . '</p>'
-			               . '<p><strong>' . __( 'Subscribe to our Newsletter', 'formidable' ) . '</strong><br/>'
-			               . sprintf( __( 'If you would like to hear about new features and updates for %1$s, subscribe to our newsletter:', 'formidable' ), 'Formidable' ) . '</p>'
-			               . '<form target="_blank" action="//formidablepro.us1.list-manage.com/subscribe/post?u=a4a913790ffb892daacc6f271&amp;id=7e7df15967" method="post" selector="newsletter-form" accept-charset="' . esc_attr( get_bloginfo( 'charset' ) ) . '">'
-			               . '<p>'
-			               . '<input style="margin: 5px; color:#666" name="EMAIL" value="' . esc_attr( $current_user->user_email ) . '" selector="newsletter-email" placeholder="' . esc_attr__( 'Email', 'formidable' ) . '"/>'
+						   . '<p>' . __( 'All your forms will be listed on this page. Create your first form by clicking on the "Add New" button.', 'formidable' ) . '</p>'
+						   . '<p><strong>' . __( 'Subscribe to our Newsletter', 'formidable' ) . '</strong><br/>'
+						   . sprintf( __( 'If you would like to hear about new features and updates for %1$s, subscribe to our newsletter:', 'formidable' ), 'Formidable' ) . '</p>'
+						   . '<form target="_blank" action="//formidablepro.us1.list-manage.com/subscribe/post?u=a4a913790ffb892daacc6f271&amp;id=7e7df15967" method="post" selector="newsletter-form" accept-charset="' . esc_attr( get_bloginfo( 'charset' ) ) . '">'
+						   . '<p>'
+						   . '<input style="margin: 5px; color:#666" name="EMAIL" value="' . esc_attr( $current_user->user_email ) . '" selector="newsletter-email" placeholder="' . esc_attr__( 'Email', 'formidable' ) . '"/>'
 						   . '<input type="hidden" name="group[4505]" value="4" />'
-			               . '<button type="submit" class="button-primary">' . esc_html__( 'Subscribe', 'formidable' ) . '</button>'
-			               . '</p>'
-			               . '</form>',
+						   . '<button type="submit" class="button-primary">' . esc_html__( 'Subscribe', 'formidable' ) . '</button>'
+						   . '</p>'
+						   . '</form>',
 			'next_page' => 'entries',
 		);
 	}
@@ -261,7 +261,7 @@ 
 	private function entries_pointer() {
 		return array(
 			'content'   => '<h3>' . __( 'Entries', 'formidable' ) . '</h3>'
-			               . '<p>' . __( 'Each time one of your forms is submitted, an entry is created. You will find every form submission listed here so you will always have a backup if an email fails.', 'formidable' ) . '</p>',
+						   . '<p>' . __( 'Each time one of your forms is submitted, an entry is created. You will find every form submission listed here so you will always have a backup if an email fails.', 'formidable' ) . '</p>',
 			'prev_page' => '',
 			'next_page' => 'styles',
 			'selector'  => '.wp-list-table',
@@ -277,7 +277,7 @@ 
 	private function styles_pointer() {
 		return array(
 			'content'   => '<h3>' . __( 'Styles', 'formidable' ) . '</h3>'
-			               . '<p>' . __( 'Want to make changes to the way your forms look? Make all the changes you would like right here, and watch the sample form change before your eyes.', 'formidable' ) . '</p>',
+						   . '<p>' . __( 'Want to make changes to the way your forms look? Make all the changes you would like right here, and watch the sample form change before your eyes.', 'formidable' ) . '</p>',
 			'prev_page' => 'entries',
 			'next_page' => 'import',
 			'selector'  => '.general-style',
@@ -293,7 +293,7 @@ 
 	private function import_pointer() {
 		return array(
 			'content'   => '<h3>' . __( 'Import/Export', 'formidable' ) . '</h3>'
-			               . '<p>' . __( 'Import and export forms and styles when copying from one site to another or sharing with someone else. Your entries can be exported to a CSV as well. The Premium version also includes the option to import entries to your site from a CSV.', 'formidable' ) . '</p>',
+						   . '<p>' . __( 'Import and export forms and styles when copying from one site to another or sharing with someone else. Your entries can be exported to a CSV as well. The Premium version also includes the option to import entries to your site from a CSV.', 'formidable' ) . '</p>',
 			'prev_page' => 'styles',
 			'next_page' => 'settings',
 			'selector'  => '.inside.with_frm_style',
@@ -313,7 +313,7 @@ 
 				. __( 'Turn stylesheets and scripts off, set which user roles have access to change and create forms, setup your reCaptcha, and set default messages for new forms and fields.', 'formidable' )
 				. '<p><strong>' . __( 'Plugin Licenses', 'formidable' ) . '</strong><br/>'
 				. sprintf( __( 'Once you&#8217;ve purchased %1$s or any addons, you&#8217;ll have to enter a license key to get access to all of their powerful features. A Plugin Licenses tab will appear here for you to enter your license key.', 'formidable' ), 'Formidable Pro' )
-           	    . '</p>',
+		   		. '</p>',
 			'prev_page' => 'import',
 			'next_page' => 'addons',
 		);
@@ -327,10 +327,10 @@ 
 	private function addons_pointer() {
 		return array(
 			'content'   => '<h3>' . __( 'Addons', 'formidable' ) . '</h3>'
-			               . '<p>' . sprintf( __( 'The powerful functions of %1$s can be extended with %2$spremium plugins%3$s. You can read all about the Formidable Premium Plugins %2$shere%3$s.', 'formidable' ), 'Formidable', '<a target="_blank" href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/' ) ) . '">', '</a>' )
+						   . '<p>' . sprintf( __( 'The powerful functions of %1$s can be extended with %2$spremium plugins%3$s. You can read all about the Formidable Premium Plugins %2$shere%3$s.', 'formidable' ), 'Formidable', '<a target="_blank" href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/' ) ) . '">', '</a>' )
 						   . '</p>'
-			               . '<p><strong>' . __( 'Like this plugin?', 'formidable' ) . '</strong><br/>' . sprintf( __( 'So, we&#8217;ve come to the end of the tour. If you like the plugin, please %1$srate it 5 stars on WordPress.org%2$s!', 'formidable' ), '<a target="_blank" href="https://wordpress.org/plugins/formidable/">', '</a>' ) . '</p>'
-			               . '<p>' . sprintf( __( 'Thank you for using our plugin and good luck with your forms!<br/><br/>Best,<br/>Team Formidable - %1$sformidableforms.com%2$s', 'formidable' ), '<a target="_blank" href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/' ) ) . '">', '</a>' ) . '</p>',
+						   . '<p><strong>' . __( 'Like this plugin?', 'formidable' ) . '</strong><br/>' . sprintf( __( 'So, we&#8217;ve come to the end of the tour. If you like the plugin, please %1$srate it 5 stars on WordPress.org%2$s!', 'formidable' ), '<a target="_blank" href="https://wordpress.org/plugins/formidable/">', '</a>' ) . '</p>'
+						   . '<p>' . sprintf( __( 'Thank you for using our plugin and good luck with your forms!<br/><br/>Best,<br/>Team Formidable - %1$sformidableforms.com%2$s', 'formidable' ), '<a target="_blank" href="' . esc_url( FrmAppHelper::make_affiliate_url( 'https://formidableforms.com/' ) ) . '">', '</a>' ) . '</p>',
 			'prev_page' => 'settings',
 		);
 	}

Spacing +1 added lines, -1 removed lines

@@ -188,7 +188,7 @@
 	 * @param string $page Admin page key.
 	 */
 	private function do_page_pointer( $page ) {
-		$pointer = call_user_func( array( $this, $this->admin_pages[ $page ] ) );
+		$pointer = call_user_func( array( $this, $this->admin_pages[$page] ) );
 
 		$opt_arr = array(
 			'content'      => $pointer['content'],