yikesinc / yikes-inc-easy-mailchimp-extender

public/partials/shortcodes/process/process_form_submission.php

Spacing +45 added lines, -45 removed lines

@@ -14,7 +14,7 @@ 
 $data = $_POST;
 
 // Confirm we have a form id to work with
-$form_id = ( isset( $data['yikes-mailchimp-submitted-form'] ) ) ? absint( $data['yikes-mailchimp-submitted-form'] ) : false;
+$form_id = ( isset( $data[ 'yikes-mailchimp-submitted-form' ] ) ) ? absint( $data[ 'yikes-mailchimp-submitted-form' ] ) : false;
 
 // Set the form id in our class
 $submission_handler->set_form_id( $form_id );
@@ -28,7 +28,7 @@ 
 }
 
 // Check our nonce
-if ( $submission_handler->handle_nonce( $_POST['yikes_easy_mc_new_subscriber'], 'yikes_easy_mc_form_submit' ) === false ) {
+if ( $submission_handler->handle_nonce( $_POST[ 'yikes_easy_mc_new_subscriber' ], 'yikes_easy_mc_form_submit' ) === false ) {
 	$process_submission_response = $submission_handler->wrap_form_submission_response( $submission_handler->handle_nonce_message, $is_success = false );
 	return;
 }
@@ -44,10 +44,10 @@ 
 }
 
 // Set up some variables from the form data -- these are required
-$list_id             = isset( $form_data['list_id'] ) ? $form_data['list_id'] : null;
-$submission_settings = isset( $form_data['submission_settings'] ) ? $form_data['submission_settings'] : null;
-$optin_settings      = isset( $form_data['optin_settings'] ) ? $form_data['optin_settings'] : null;
-$form_fields         = isset( $form_data['fields'] ) ? $form_data['fields'] : null;
+$list_id             = isset( $form_data[ 'list_id' ] ) ? $form_data[ 'list_id' ] : null;
+$submission_settings = isset( $form_data[ 'submission_settings' ] ) ? $form_data[ 'submission_settings' ] : null;
+$optin_settings      = isset( $form_data[ 'optin_settings' ] ) ? $form_data[ 'optin_settings' ] : null;
+$form_fields         = isset( $form_data[ 'fields' ] ) ? $form_data[ 'fields' ] : null;
 
 // Send an error if for some reason we can't find the required form data
 if ( $submission_handler->handle_empty_fields_generic( array( $list_id, $submission_settings, $optin_settings, $form_fields ) ) === false ) {
@@ -70,8 +70,8 @@ 
 $submission_handler->set_list_id( $list_id );
 
 // Set up some variables from the form data -- these are not required
-$error_messages      = isset( $form_data['error_messages'] ) ? $form_data['error_messages'] : array();
-$notifications       = isset( $form_data['custom_notifications'] ) ? $form_data['custom_notifications'] : array(); // Do we need this?
+$error_messages      = isset( $form_data[ 'error_messages' ] ) ? $form_data[ 'error_messages' ] : array();
+$notifications       = isset( $form_data[ 'custom_notifications' ] ) ? $form_data[ 'custom_notifications' ] : array(); // Do we need this?
 
 // Set the error messages in our class
 $submission_handler->set_error_messages( $error_messages );
@@ -87,7 +87,7 @@ 
 }
 
 // Get, sanitize, and lowercasify the email
-$submitted_email = isset( $data['EMAIL'] ) ? $data['EMAIL'] : '';
+$submitted_email = isset( $data[ 'EMAIL' ] ) ? $data[ 'EMAIL' ] : '';
 $sanitized_email = $submission_handler->get_sanitized_email( $submitted_email ); 
 $submission_handler->set_email( $sanitized_email );
 
@@ -98,7 +98,7 @@ 
 }
 
 // Check for Honeypot filled
-$honey_pot_filled = ( isset( $data['yikes-mailchimp-honeypot'] ) && '' !== $data['yikes-mailchimp-honeypot'] ) ? true : false;
+$honey_pot_filled = ( isset( $data[ 'yikes-mailchimp-honeypot' ] ) && '' !== $data[ 'yikes-mailchimp-honeypot' ] ) ? true : false;
 
 // Send an error if honey pot is not empty
 if ( $submission_handler->handle_non_empty_honeypot( $honey_pot_filled ) === false ) {
@@ -107,11 +107,11 @@ 
 }
 
 // Check if reCAPTCHA Response was submitted with the form data, and handle it if needed
-if ( isset( $data['g-recaptcha-response'] ) ) {
-	$recaptcha_response = $data['g-recaptcha-response'];
+if ( isset( $data[ 'g-recaptcha-response' ] ) ) {
+	$recaptcha_response = $data[ 'g-recaptcha-response' ];
 	$recaptcha_handle = $submission_handler->handle_recaptcha( $recaptcha_response );
-	if ( isset( $recaptcha_handle['success'] ) && $recaptcha_handle['success'] === false ) {
-		$process_submission_response = $submission_handler->wrap_form_submission_response( $recaptcha_handle['message'], $is_success = false );
+	if ( isset( $recaptcha_handle[ 'success' ] ) && $recaptcha_handle[ 'success' ] === false ) {
+		$process_submission_response = $submission_handler->wrap_form_submission_response( $recaptcha_handle[ 'message' ], $is_success = false );
 		return;
 	}
 }
@@ -120,7 +120,7 @@ 
 $merge_variables = $submission_handler->get_submitted_merge_values( $data, $form_fields );
 
 // Submission Setting: Replace interest groups or update interest groups
-$replace_interests = isset( $submission_settings['replace_interests'] ) ? (bool) $submission_settings['replace_interests'] : true;
+$replace_interests = isset( $submission_settings[ 'replace_interests' ] ) ? (bool) $submission_settings[ 'replace_interests' ] : true;
 
 // Get the default groups
 $groups = $submission_handler->get_default_interest_groups( $replace_interests, $list_handler );
@@ -135,17 +135,17 @@ 
  *
  * @param $merge_variables array Array of merge variable to use
  */
-do_action( 'yikes-mailchimp-before-submission',            $merge_variables );
+do_action( 'yikes-mailchimp-before-submission', $merge_variables );
 do_action( "yikes-mailchimp-before-submission-{$form_id}", $merge_variables );
 
 // Allow users to check for form values (using the `yikes-mailchimp-filter-before-submission` filter hook in function `get_submitted_merge_values`) 
 // and pass back an error and message to the user
 // If error is set and no message, default to our class variable's default error message
-if ( isset( $merge_variables['error'] ) ) {
-	$merge_error_message = isset( $merge_variables['message'] ) ? $merge_variables['message'] : $submission_handler->default_error_response_message;
-	$merge_vars_error_array = $submission_handler->handle_merge_variables_error( $merge_variables['error'], $merge_error_message );
-	if ( $merge_vars_error_array['success'] === false ) {
-		$process_submission_response = $submission_handler->wrap_form_submission_response( $merge_vars_error_array['message'], $is_success = false );
+if ( isset( $merge_variables[ 'error' ] ) ) {
+	$merge_error_message = isset( $merge_variables[ 'message' ] ) ? $merge_variables[ 'message' ] : $submission_handler->default_error_response_message;
+	$merge_vars_error_array = $submission_handler->handle_merge_variables_error( $merge_variables[ 'error' ], $merge_error_message );
+	if ( $merge_vars_error_array[ 'success' ] === false ) {
+		$process_submission_response = $submission_handler->wrap_form_submission_response( $merge_vars_error_array[ 'message' ], $is_success = false );
 		return;
 	}
 }
@@ -158,7 +158,7 @@ 
 
 // Only add groups if they exist
 if ( ! empty( $groups ) ) {
-	$member_data['interests'] = $groups;
+	$member_data[ 'interests' ] = $groups;
 }
 
 // Check if this member already exists
@@ -169,12 +169,12 @@ 
 // Likewise, if this member exists but their status is 'pending' it means we're dealing with a double opt-in list and they never confirmed
 // Or, if this member but their status is 'unsubscribed' it means we're dealing with someone who unsubscribed and they need to re-subscribe
 // Continue as if they're a new member to force another double opt-in email.
-if ( is_array( $member_exists ) && isset( $member_exists['status'] ) ) {
-	$double_optin_resubscribe = $member_exists['status'] === 'pending' || $member_exists['status'] === 'unsubscribed';
+if ( is_array( $member_exists ) && isset( $member_exists[ 'status' ] ) ) {
+	$double_optin_resubscribe = $member_exists[ 'status' ] === 'pending' || $member_exists[ 'status' ] === 'unsubscribed';
 } else {
 
 	// If this member doesn't exist, set their timestamp signup to the current time.
-	$member_data['timestamp_signup'] = current_time( 'Y-m-d H:i:s', 1 );
+	$member_data[ 'timestamp_signup' ] = current_time( 'Y-m-d H:i:s', 1 );
 }
 
 if ( is_wp_error( $member_exists ) || $double_optin_resubscribe === true ) {
@@ -182,24 +182,24 @@ 
 
 	// Check the opt-in value - is it double or single?
 	// Double opt-in means 'status_if_new' => 'pending'
-	$double_optin = isset( $optin_settings['optin'] ) ? (int) $optin_settings['optin'] : 0;
+	$double_optin = isset( $optin_settings[ 'optin' ] ) ? (int) $optin_settings[ 'optin' ] : 0;
 
 	// If the user was unsubscribed and is re-subscribing, we set the status to 'pending', which
 	// causes Mailchimp to send them a confirmation email.  This is the only way Mailchimp will
 	// allow us to re-subscribe the user.
-	$was_unsubscribed = is_array( $member_exists ) && isset( $member_exists['status'] ) && $member_exists['status'] === 'unsubscribed';
+	$was_unsubscribed = is_array( $member_exists ) && isset( $member_exists[ 'status' ] ) && $member_exists[ 'status' ] === 'unsubscribed';
 	$was_unsubscribed = apply_filters( 'yikes_mailchimp_single_resubscribe', $was_unsubscribed );
 
 	if ( $double_optin === 1 || $was_unsubscribed === true ) {
 
 		// Double opt-in
-		$member_data['status_if_new'] = 'pending';
-		$member_data['status']        = 'pending';
+		$member_data[ 'status_if_new' ] = 'pending';
+		$member_data[ 'status' ]        = 'pending';
 	} else {
 
 		// Single opt-in
-		$member_data['status_if_new'] = 'subscribed';
-		$member_data['status']        = 'subscribed';
+		$member_data[ 'status_if_new' ] = 'subscribed';
+		$member_data[ 'status' ]        = 'subscribed';
 	}
 
 } else {
@@ -208,28 +208,28 @@ 
 
 	// But first let's set our flag, and set the Mailchimp status flag
 	$new_subscriber = false;
-	$member_data['status'] = 'subscribed';
+	$member_data[ 'status' ] = 'subscribed';
 
 	// Check our update_existing_user optin setting
-	$update_existing_user = ( $optin_settings['update_existing_user'] === '1' ) ? true : false;
+	$update_existing_user = ( $optin_settings[ 'update_existing_user' ] === '1' ) ? true : false;
 
 	// If update_existing_user is false (not allowed) then simply fail and return a response message
 	if ( $update_existing_user === false ) {
 		$disallow_update_array = $submission_handler->handle_disallowed_existing_user_update();
-		if ( $disallow_update_array['success'] === false ) {
-			$process_submission_response = $submission_handler->wrap_form_submission_response( $disallow_update_array['message'], $is_success = false );
+		if ( $disallow_update_array[ 'success' ] === false ) {
+			$process_submission_response = $submission_handler->wrap_form_submission_response( $disallow_update_array[ 'message' ], $is_success = false );
 			return;
 		}
 	}
 
 	// If update_existing_user is true, we need to check our 'send_update_email' option
-	$send_update_email = ( $optin_settings['send_update_email'] === '1' ) ? true : false;
+	$send_update_email = ( $optin_settings[ 'send_update_email' ] === '1' ) ? true : false;
 
 	// If $send_update_email is true (we send the email) then we need to fire off the 'send update email' logic
 	if ( $send_update_email === true ) {
 		$update_existing_user_array = $submission_handler->handle_updating_existing_user();
-		if ( $update_existing_user_array['success'] === false ) {
-			$process_submission_response = $submission_handler->wrap_form_submission_response( $update_existing_user_array['message'], $is_success = false );
+		if ( $update_existing_user_array[ 'success' ] === false ) {
+			$process_submission_response = $submission_handler->wrap_form_submission_response( $update_existing_user_array[ 'message' ], $is_success = false );
 			return;
 		}
 	}
@@ -250,9 +250,9 @@ 
 
 // If this is the first time subscriber add the tags along with the subscribe request.
 // This allows for double opt-in forms submissions to be tagged.
-if ( ! empty( $form_data['tags'] ) && ! is_array( $member_exists ) ) {
-	foreach( $form_data['tags'] as $tag ) {
-		$member_data['tags'][] = $tag['name'];
+if ( ! empty( $form_data[ 'tags' ] ) && ! is_array( $member_exists ) ) {
+	foreach ( $form_data[ 'tags' ] as $tag ) {
+		$member_data[ 'tags' ][ ] = $tag[ 'name' ];
 	}
 }
 
@@ -272,9 +272,9 @@ 
 }
 
 // Handle errors in the response
-if ( isset( $success_array ) && isset( $success_array['success'] ) && $success_array['success'] === false ) {
-	$process_submission_response = isset( $success_array['message'] ) ? $success_array['message'] : '';
-	$process_submission_response = $submission_handler->wrap_form_submission_response( $success_array['message'], $is_success = false );
+if ( isset( $success_array ) && isset( $success_array[ 'success' ] ) && $success_array[ 'success' ] === false ) {
+	$process_submission_response = isset( $success_array[ 'message' ] ) ? $success_array[ 'message' ] : '';
+	$process_submission_response = $submission_handler->wrap_form_submission_response( $success_array[ 'message' ], $is_success = false );
 	return;
 }
 
@@ -284,7 +284,7 @@ 
 // For non-AJAX submissions, if we have a new subscriber we need to increment our submissions count by 1
 // For AJAX, this is an AJAX call that gets fired off after form submission
 if ( $new_subscriber === true ) {
-	$submissions = (int) $form_settings['submissions'] + 1;	
+	$submissions = (int) $form_settings[ 'submissions' ] + 1;	
 	$interface->update_form_field( $form_id, 'submissions', $submissions );
 }
 

src/Renderable.php

Spacing +1 added lines, -1 removed lines

@@ -29,5 +29,5 @@
 	 *
 	 * @return string Rendered HTML.
 	 */
-	public function render( array $context = [] );
+	public function render( array $context = [ ] );
 }

src/Autoloader.php

Spacing +7 added lines, -7 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @var array
 	 */
-	private $namespaces = [];
+	private $namespaces = [ ];
 
 	/**
 	 * Destructor for the Autoloader class.
@@ -84,7 +84,7 @@ 
 		$lowercase = false,
 		$underscores = false
 	) {
-		$this->namespaces[] = [
+		$this->namespaces[ ] = [
 			'root'        => $this->normalize_root( (string) $root ),
 			'base_dir'    => trailingslashit( (string) $base_dir ),
 			'prefix'      => (string) $prefix,
@@ -110,12 +110,12 @@ 
 		foreach ( $this->namespaces as $namespace ) {
 
 			// Move on if the object does not belong to the current namespace.
-			if ( 0 !== strpos( $class, $namespace['root'] ) ) {
+			if ( 0 !== strpos( $class, $namespace[ 'root' ] ) ) {
 				continue;
 			}
 
 			// Remove namespace root level to correspond with root filesystem.
-			$filename = str_replace( $namespace['root'], '', $class );
+			$filename = str_replace( $namespace[ 'root' ], '', $class );
 
 			// Replace the namespace separator "\" by the system-dependent directory separator.
 			$filename = str_replace( '\\', DIRECTORY_SEPARATOR, $filename );
@@ -124,17 +124,17 @@ 
 			$filename = $this->remove_leading_backslash( $filename );
 
 			// Change to lower case if requested.
-			if ( true === $namespace['lowercase'] ) {
+			if ( true === $namespace[ 'lowercase' ] ) {
 				$filename = strtolower( $filename );
 			}
 
 			// Change underscores into hyphens if requested.
-			if ( true === $namespace['underscores'] ) {
+			if ( true === $namespace[ 'underscores' ] ) {
 				$filename = str_replace( '_', '-', $filename );
 			}
 
 			// Add base_dir, prefix and suffix.
-			$filepath = "{$namespace['base_dir']}{$namespace['prefix']}{$filename}{$namespace['suffix']}";
+			$filepath = "{$namespace[ 'base_dir' ]}{$namespace[ 'prefix' ]}{$filename}{$namespace[ 'suffix' ]}";
 
 			// Require the file if it exists and is readable.
 			if ( is_readable( $filepath ) ) {

src/Container.php

Spacing +3 added lines, -3 removed lines

@@ -23,15 +23,15 @@
 	 * @since %VERSION%
 	 * @var array
 	 */
-	protected $services = [];
+	protected $services = [ ];
 
 	/**
 	 * Container constructor.
 	 *
 	 * @param array $services Services to register with the container.
 	 */
-	public function __construct( array $services = [] ) {
-		$this->services = $services ?: [];
+	public function __construct( array $services = [ ] ) {
+		$this->services = $services ? : [ ];
 	}
 
 	/**

src/Assets/AssetsAwareness.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 	 * @since %VERSION%
 	 * @var Asset[]
 	 */
-	protected $assets = [];
+	protected $assets = [ ];
 
 	/**
 	 * Get the array of known assets.
@@ -107,6 +107,6 @@ 
 	 * @since %VERSION%
 	 */
 	protected function load_assets() {
-		$this->assets = [];
+		$this->assets = [ ];
 	}
 }

src/Assets/BlockAsset.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	 * @return Closure
 	 */
 	protected function get_enqueue_closure() {
-		return function () {
+		return function() {
 			call_user_func( parent::get_enqueue_closure() );
 			wp_set_script_translations( $this->handle, 'yikes-inc-easy-mailchimp-extender' );
 		};

src/Assets/StyleAsset.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 	const MEDIA_ALL    = 'all';
 	const MEDIA_PRINT  = 'print';
 	const MEDIA_SCREEN = 'screen';
-	const DEPENDENCIES = [];
+	const DEPENDENCIES = [ ];
 	const VERSION      = Plugin::VERSION;
 	const DISABLEABLE  = false;
 
@@ -113,7 +113,7 @@ 
 	 * @return Closure
 	 */
 	protected function get_register_closure() {
-		return function () {
+		return function() {
 			if ( wp_script_is( $this->handle, 'registered' ) ) {
 				return;
 			}
@@ -140,7 +140,7 @@ 
 	 * @return Closure
 	 */
 	protected function get_enqueue_closure() {
-		return function () {
+		return function() {
 			wp_enqueue_style( $this->handle );
 		};
 	}
@@ -153,7 +153,7 @@ 
 	 * @return Closure
 	 */
 	protected function get_dequeue_closure() {
-		return function () {
+		return function() {
 			wp_dequeue_style( $this->handle );
 		};
 	}

src/Assets/AssetsHandler.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 	 *
 	 * @var Asset[]
 	 */
-	private $assets = [];
+	private $assets = [ ];
 
 	/**
 	 * Add a single asset to the asset handler.

src/Plugin.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
 	 * @since %VERSION%
 	 * @var Service[]
 	 */
-	private $services = [];
+	private $services = [ ];
 
 	/**
 	 * Instantiate a Plugin object.
@@ -66,7 +66,7 @@ 
 	 */
 	public function __construct( Container $container, AssetsHandler $assets_handler = null ) {
 		$this->container      = $container;
-		$this->assets_handler = $assets_handler ?: new AssetsHandler();
+		$this->assets_handler = $assets_handler ? : new AssetsHandler();
 	}
 
 	/**