wp-pay-extensions / memberpress

src/Gateway.php

Spacing +117 added lines, -117 removed lines

@@ -55,10 +55,10 @@ 
 
 		// Set the name of this gateway.
 		// @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L12-13.
-		$this->name = __( 'Pronamic', 'pronamic_ideal' );
+		$this->name = __('Pronamic', 'pronamic_ideal');
 
-		if ( ! empty( $this->payment_method ) ) {
-			$this->name = PaymentMethods::get_name( $this->payment_method );
+		if ( ! empty($this->payment_method)) {
+			$this->name = PaymentMethods::get_name($this->payment_method);
 		}
 
 		// Set the default settings.
@@ -80,7 +80,7 @@ 
 	 *
 	 * @param array $settings MemberPress gateway settings array.
 	 */
-	public function load( $settings ) {
+	public function load($settings) {
 		$this->settings = (object) $settings;
 
 		$this->set_defaults();
@@ -97,19 +97,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function send_transaction_notices( $transaction, $method ) {
+	public function send_transaction_notices($transaction, $method) {
 		$class = 'MeprUtils';
 
-		if ( ! Core_Util::class_method_exists( $class, $method ) ) {
+		if ( ! Core_Util::class_method_exists($class, $method)) {
 			$class = $this;
 		}
 
-		if ( 'MeprUtils' === $class && 'send_product_welcome_notices' === $method ) {
+		if ('MeprUtils' === $class && 'send_product_welcome_notices' === $method) {
 			// `send_product_welcome_notices` is called from `send_signup_notices` in newer versions.
 			return;
 		}
 
-		return call_user_func( array( $class, $method ), $transaction );
+		return call_user_func(array($class, $method), $transaction);
 	}
 
 	/**
@@ -137,7 +137,7 @@ 
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L72-73
 	 */
 	protected function set_defaults() {
-		if ( ! isset( $this->settings ) ) {
+		if ( ! isset($this->settings)) {
 			$this->settings = array();
 		}
 
@@ -175,7 +175,7 @@ 
 	 *
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L119-122
 	 */
-	public function process_payment( $txn ) {
+	public function process_payment($txn) {
 
 	}
 
@@ -192,8 +192,8 @@ 
 
 		$subscription = $transaction->subscription();
 
-		if ( $subscription ) {
-			if ( MeprSubscription::$active_str !== $subscription->status ) {
+		if ($subscription) {
+			if (MeprSubscription::$active_str !== $subscription->status) {
 				$subscription->status = MeprSubscription::$active_str;
 				$subscription->store();
 			}
@@ -201,7 +201,7 @@ 
 			$subscription->limit_payment_cycles();
 		}
 
-		$this->send_transaction_notices( $transaction, 'send_transaction_receipt_notices' );
+		$this->send_transaction_notices($transaction, 'send_transaction_receipt_notices');
 
 		return $transaction;
 	}
@@ -221,12 +221,12 @@ 
 		// Expire associated transactions for subscription.
 		$subscription = $transaction->subscription();
 
-		if ( $subscription ) {
+		if ($subscription) {
 			$subscription->expire_txns();
 			$subscription->store();
 		}
 
-		$this->send_transaction_notices( $transaction, 'send_failed_txn_notices' );
+		$this->send_transaction_notices($transaction, 'send_failed_txn_notices');
 
 		return $transaction;
 	}
@@ -250,17 +250,17 @@ 
 
 		$subscription = $transaction->subscription();
 
-		if ( $subscription ) {
+		if ($subscription) {
 			$subscription->status     = MeprSubscription::$active_str;
 			$subscription->created_at = $transaction->created_at;
 			$subscription->store();
 
 			// Set subscription first transaction id meta.
-			$pronamic_subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $subscription->first_txn_id );
+			$pronamic_subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $subscription->first_txn_id);
 
-			if ( $pronamic_subscription ) {
+			if ($pronamic_subscription) {
 				// Store MemberPress subscription id to be able to cancel subscription on delete.
-				$pronamic_subscription->set_meta( 'memberpress_subscription_id', $subscription->id );
+				$pronamic_subscription->set_meta('memberpress_subscription_id', $subscription->id);
 			}
 		}
 
@@ -272,9 +272,9 @@ 
 		 * @see https://github.com/wp-premium/memberpress-business/blob/1.2.7/app/lib/MeprBaseGateway.php#L609-L619
 		 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/models/MeprTransaction.php#L51
 		 */
-		$reflection = new ReflectionClass( 'MeprBaseRealGateway' );
+		$reflection = new ReflectionClass('MeprBaseRealGateway');
 
-		if ( $reflection->hasMethod( 'send_product_welcome_notices' ) && 3 === $reflection->getMethod( 'send_product_welcome_notices' )->getNumberOfParameters() ) {
+		if ($reflection->hasMethod('send_product_welcome_notices') && 3 === $reflection->getMethod('send_product_welcome_notices')->getNumberOfParameters()) {
 			$uemail = MeprEmailFactory::fetch(
 				'MeprUserProductWelcomeEmail',
 				'MeprBaseProductEmail',
@@ -287,24 +287,24 @@ 
 
 			$this->send_product_welcome_notices(
 				$uemail,
-				MeprTransactionsHelper::get_email_params( $transaction ),
+				MeprTransactionsHelper::get_email_params($transaction),
 				$transaction->user()
 			);
 		} else {
-			$this->send_transaction_notices( $transaction, 'send_product_welcome_notices' );
+			$this->send_transaction_notices($transaction, 'send_product_welcome_notices');
 		}
 
 		// Send upgrade/downgrade notices.
-		if ( $upgrade ) {
-			$this->upgraded_sub( $transaction );
-			$this->send_transaction_notices( $transaction, 'send_upgraded_txn_notices' );
-		} elseif ( $downgrade ) {
-			$this->downgraded_sub( $transaction );
-			$this->send_transaction_notices( $transaction, 'send_downgraded_txn_notices' );
+		if ($upgrade) {
+			$this->upgraded_sub($transaction);
+			$this->send_transaction_notices($transaction, 'send_upgraded_txn_notices');
+		} elseif ($downgrade) {
+			$this->downgraded_sub($transaction);
+			$this->send_transaction_notices($transaction, 'send_downgraded_txn_notices');
 		}
 
-		$this->send_transaction_notices( $transaction, 'send_signup_notices' );
-		$this->send_transaction_notices( $transaction, 'send_transaction_receipt_notices' );
+		$this->send_transaction_notices($transaction, 'send_signup_notices');
+		$this->send_transaction_notices($transaction, 'send_transaction_receipt_notices');
 
 		return $transaction;
 	}
@@ -316,7 +316,7 @@ 
 	 *
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L131-133
 	 */
-	public function process_refund( MeprTransaction $txn ) {
+	public function process_refund(MeprTransaction $txn) {
 
 	}
 
@@ -336,7 +336,7 @@ 
 	 *
 	 * @param MeprTransaction $transaction MemberPress transaction object.
 	 */
-	public function process_trial_payment( $transaction ) {
+	public function process_trial_payment($transaction) {
 
 	}
 
@@ -347,7 +347,7 @@ 
 	 *
 	 * @param MeprTransaction $transaction MemberPress transaction object.
 	 */
-	public function record_trial_payment( $transaction ) {
+	public function record_trial_payment($transaction) {
 
 	}
 
@@ -358,7 +358,7 @@ 
 	 *
 	 * @param MeprTransaction $txn MemberPress transaction object.
 	 */
-	public function process_create_subscription( $txn ) {
+	public function process_create_subscription($txn) {
 
 	}
 
@@ -379,7 +379,7 @@ 
 	 *
 	 * @param int $sub_id Subscription ID.
 	 */
-	public function process_update_subscription( $sub_id ) {
+	public function process_update_subscription($sub_id) {
 
 	}
 
@@ -399,21 +399,21 @@ 
 	 *
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L184-186
 	 */
-	public function process_suspend_subscription( $sub_id ) {
-		if ( ! MeprSubscription::exists( $sub_id ) ) {
+	public function process_suspend_subscription($sub_id) {
+		if ( ! MeprSubscription::exists($sub_id)) {
 			return;
 		}
 
-		$sub = new MeprSubscription( $sub_id );
+		$sub = new MeprSubscription($sub_id);
 
-		if ( MeprSubscription::$suspended_str === $sub->status ) {
+		if (MeprSubscription::$suspended_str === $sub->status) {
 			// Subscription is already suspended.
 			return;
 		}
 
-		$subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $sub->first_txn_id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->first_txn_id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -422,19 +422,19 @@ 
 		$sub->store();
 
 		// Send suspended subscription notices.
-		MeprUtils::send_suspended_sub_notices( $sub );
+		MeprUtils::send_suspended_sub_notices($sub);
 
 		$note = sprintf(
 			/* translators: %s: MemberPress */
-			__( '%s subscription on hold.', 'pronamic_ideal' ),
-			__( 'MemberPress', 'pronamic_ideal' )
+			__('%s subscription on hold.', 'pronamic_ideal'),
+			__('MemberPress', 'pronamic_ideal')
 		);
 
-		$subscription->add_note( $note );
+		$subscription->add_note($note);
 
 		// The status of canceled or completed subscriptions will not be changed automatically.
-		if ( ! in_array( $subscription->get_status(), array( Statuses::CANCELLED, Statuses::COMPLETED ), true ) ) {
-			$subscription->set_status( Statuses::OPEN );
+		if ( ! in_array($subscription->get_status(), array(Statuses::CANCELLED, Statuses::COMPLETED), true)) {
+			$subscription->set_status(Statuses::OPEN);
 
 			$subscription->save();
 		}
@@ -456,21 +456,21 @@ 
 	 *
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L193-195
 	 */
-	public function process_resume_subscription( $sub_id ) {
-		if ( ! MeprSubscription::exists( $sub_id ) ) {
+	public function process_resume_subscription($sub_id) {
+		if ( ! MeprSubscription::exists($sub_id)) {
 			return;
 		}
 
-		$sub = new MeprSubscription( $sub_id );
+		$sub = new MeprSubscription($sub_id);
 
-		if ( MeprSubscription::$active_str === $sub->status ) {
+		if (MeprSubscription::$active_str === $sub->status) {
 			// Subscription is already active.
 			return;
 		}
 
-		$subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $sub->first_txn_id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->first_txn_id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -481,35 +481,35 @@ 
 		// Check if prior txn is expired yet or not, if so create a temporary txn so the user can access the content immediately.
 		$prior_txn = $sub->latest_txn();
 
-		if ( false === $prior_txn || ! ( $prior_txn instanceof MeprTransaction ) || strtotime( $prior_txn->expires_at ) < time() ) {
+		if (false === $prior_txn || ! ($prior_txn instanceof MeprTransaction) || strtotime($prior_txn->expires_at) < time()) {
 			$txn                  = new MeprTransaction();
 			$txn->subscription_id = $sub->id;
 			$txn->trans_num       = $sub->subscr_id . '-' . uniqid();
 			$txn->status          = MeprTransaction::$confirmed_str;
 			$txn->txn_type        = MeprTransaction::$subscription_confirmation_str;
 			$txn->response        = (string) $sub;
-			$txn->expires_at      = MeprUtils::ts_to_mysql_date( time() + MeprUtils::days( 1 ), 'Y-m-d 23:59:59' );
+			$txn->expires_at      = MeprUtils::ts_to_mysql_date(time() + MeprUtils::days(1), 'Y-m-d 23:59:59');
 
-			$txn->set_subtotal( 0.00 ); // Just a confirmation txn.
+			$txn->set_subtotal(0.00); // Just a confirmation txn.
 
 			$txn->store();
 		}
 
 		// Send resumed subscription notices.
-		MeprUtils::send_resumed_sub_notices( $sub );
+		MeprUtils::send_resumed_sub_notices($sub);
 
 		// Add note.
 		$note = sprintf(
 			/* translators: %s: MemberPress */
-			__( '%s subscription reactivated.', 'pronamic_ideal' ),
-			__( 'MemberPress', 'pronamic_ideal' )
+			__('%s subscription reactivated.', 'pronamic_ideal'),
+			__('MemberPress', 'pronamic_ideal')
 		);
 
-		$subscription->add_note( $note );
+		$subscription->add_note($note);
 
 		// The status of canceled or completed subscriptions will not be changed automatically.
-		if ( ! in_array( $subscription->get_status(), array( Statuses::CANCELLED, Statuses::COMPLETED ), true ) ) {
-			$subscription->set_status( Statuses::ACTIVE );
+		if ( ! in_array($subscription->get_status(), array(Statuses::CANCELLED, Statuses::COMPLETED), true)) {
+			$subscription->set_status(Statuses::ACTIVE);
 
 			$subscription->save();
 		}
@@ -531,21 +531,21 @@ 
 	 *
 	 * @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprBaseGateway.php#L202-206
 	 */
-	public function process_cancel_subscription( $sub_id ) {
-		if ( ! MeprSubscription::exists( $sub_id ) ) {
+	public function process_cancel_subscription($sub_id) {
+		if ( ! MeprSubscription::exists($sub_id)) {
 			return;
 		}
 
-		$sub = new MeprSubscription( $sub_id );
+		$sub = new MeprSubscription($sub_id);
 
-		if ( MeprSubscription::$cancelled_str === $sub->status ) {
+		if (MeprSubscription::$cancelled_str === $sub->status) {
 			// Subscription is already cancelled.
 			return;
 		}
 
-		$subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $sub->first_txn_id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->first_txn_id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -554,27 +554,27 @@ 
 		$sub->store();
 
 		// Expire the grace period (confirmation) if no completed payments have come through.
-		if ( (int) $sub->txn_count <= 0 ) {
+		if ((int) $sub->txn_count <= 0) {
 			$sub->expire_txns();
 		}
 
 		$sub->limit_reached_actions();
 
 		// Send cancelled subscription notices.
-		MeprUtils::send_cancelled_sub_notices( $sub );
+		MeprUtils::send_cancelled_sub_notices($sub);
 
 		// Add note.
 		$note = sprintf(
 			/* translators: %s: MemberPress */
-			__( '%s subscription cancelled.', 'pronamic_ideal' ),
-			__( 'MemberPress', 'pronamic_ideal' )
+			__('%s subscription cancelled.', 'pronamic_ideal'),
+			__('MemberPress', 'pronamic_ideal')
 		);
 
-		$subscription->add_note( $note );
+		$subscription->add_note($note);
 
 		// The status of canceled or completed subscriptions will not be changed automatically.
-		if ( ! in_array( $subscription->get_status(), array( Statuses::CANCELLED, Statuses::COMPLETED ), true ) ) {
-			$subscription->set_status( Statuses::CANCELLED );
+		if ( ! in_array($subscription->get_status(), array(Statuses::CANCELLED, Statuses::COMPLETED), true)) {
+			$subscription->set_status(Statuses::CANCELLED);
 
 			$subscription->save();
 		}
@@ -601,7 +601,7 @@ 
 	 *
 	 * @param MeprTransaction $txn MemberPress transaction object.
 	 */
-	public function process_signup_form( $txn ) {
+	public function process_signup_form($txn) {
 
 	}
 
@@ -612,28 +612,28 @@ 
 	 *
 	 * @param MeprTransaction $txn MemberPress transaction object.
 	 */
-	public function payment_redirect( $txn ) {
-		$txn = new MeprTransaction( $txn->id );
+	public function payment_redirect($txn) {
+		$txn = new MeprTransaction($txn->id);
 
 		// Gateway.
 		$config_id = $this->settings->config_id;
 
-		$gateway = Plugin::get_gateway( $config_id );
+		$gateway = Plugin::get_gateway($config_id);
 
-		if ( ! $gateway ) {
+		if ( ! $gateway) {
 			return;
 		}
 
 		// Data.
-		$data = new PaymentData( $txn );
+		$data = new PaymentData($txn);
 
-		$payment = Plugin::start( $config_id, $gateway, $data, $this->payment_method );
+		$payment = Plugin::start($config_id, $gateway, $data, $this->payment_method);
 
 		$error = $gateway->get_error();
 
-		if ( ! is_wp_error( $error ) ) {
+		if ( ! is_wp_error($error)) {
 			// Redirect.
-			$gateway->redirect( $payment );
+			$gateway->redirect($payment);
 		}
 	}
 
@@ -647,14 +647,14 @@ 
 	 * @param  MeprTransaction $txn MemberPress transaction object.
 	 * @return bool
 	 */
-	public function display_payment_page( $txn ) {
+	public function display_payment_page($txn) {
 		// Gateway.
 		$config_id = $this->settings->config_id;
 
-		$gateway = Plugin::get_gateway( $config_id );
+		$gateway = Plugin::get_gateway($config_id);
 
-		if ( $gateway && '' === $gateway->get_input_html() ) {
-			$this->payment_redirect( $txn );
+		if ($gateway && '' === $gateway->get_input_html()) {
+			$this->payment_redirect($txn);
 		}
 	}
 
@@ -668,18 +668,18 @@ 
 	 * @param  MeprTransaction $txn MemberPress transaction object.
 	 * @return bool
 	 */
-	public function process_payment_form( $txn ) {
-		if ( ! filter_has_var( INPUT_POST, 'pronamic_pay_memberpress_pay' ) ) {
+	public function process_payment_form($txn) {
+		if ( ! filter_has_var(INPUT_POST, 'pronamic_pay_memberpress_pay')) {
 			return false;
 		}
 
 		// Gateway.
 		$config_id = $this->settings->config_id;
 
-		$gateway = Plugin::get_gateway( $config_id );
+		$gateway = Plugin::get_gateway($config_id);
 
-		if ( $gateway ) {
-			$this->payment_redirect( $txn );
+		if ($gateway) {
+			$this->payment_redirect($txn);
 		}
 	}
 
@@ -707,20 +707,20 @@ 
 	 * @param int      $product_id Product ID.
 	 * @param int      $txn_id     Transaction ID.
 	 */
-	public function display_payment_form( $amount, $user, $product_id, $txn_id ) {
-		$product = new MeprProduct( $product_id );
+	public function display_payment_form($amount, $user, $product_id, $txn_id) {
+		$product = new MeprProduct($product_id);
 
 		$coupon = false;
 
-		$txn = new MeprTransaction( $txn_id );
+		$txn = new MeprTransaction($txn_id);
 
 		// Artifically set the price of the $prd in case a coupon was used.
-		if ( $product->price !== $amount ) {
+		if ($product->price !== $amount) {
 			$coupon         = true;
 			$product->price = $amount;
 		}
 
-		$invoice = MeprTransactionsHelper::get_invoice( $txn );
+		$invoice = MeprTransactionsHelper::get_invoice($txn);
 
 		echo $invoice; // WPCS: XSS ok.
 
@@ -728,7 +728,7 @@ 
 		<div class="mp_wrapper mp_payment_form_wrapper">
 			<form action="" method="post" id="payment-form" class="mepr-form" novalidate>
 				<input type="hidden" name="mepr_process_payment_form" value="Y"/>
-				<input type="hidden" name="mepr_transaction_id" value="<?php echo esc_attr( $txn_id ); ?>"/>
+				<input type="hidden" name="mepr_transaction_id" value="<?php echo esc_attr($txn_id); ?>"/>
 				<input type="hidden" name="pronamic_pay_memberpress_pay" value="1"/>
 
 				<div class="mepr_spacer">&nbsp;</div>
@@ -738,9 +738,9 @@ 
 				// Gateway.
 				$config_id = $this->settings->config_id;
 
-				$gateway = Plugin::get_gateway( $config_id );
+				$gateway = Plugin::get_gateway($config_id);
 
-				if ( $gateway ) {
+				if ($gateway) {
 					echo $gateway->get_input_html(); // WPCS: XSS ok.
 				}
 
@@ -748,13 +748,13 @@ 
 
 				<div class="mepr_spacer">&nbsp;</div>
 
-				<input type="submit" class="mepr-submit" value="<?php esc_attr_e( 'Pay', 'pronamic_ideal' ); ?>"/>
-				<img src="<?php echo esc_attr( admin_url( 'images/loading.gif' ) ); ?>" style="display: none;" class="mepr-loading-gif"/>
-				<?php MeprView::render( '/shared/has_errors', get_defined_vars() ); ?>
+				<input type="submit" class="mepr-submit" value="<?php esc_attr_e('Pay', 'pronamic_ideal'); ?>"/>
+				<img src="<?php echo esc_attr(admin_url('images/loading.gif')); ?>" style="display: none;" class="mepr-loading-gif"/>
+				<?php MeprView::render('/shared/has_errors', get_defined_vars()); ?>
 
 				<noscript>
 					<p class="mepr_nojs">
-						<?php esc_html_e( 'JavaScript is disabled in your browser. You will not be able to complete your purchase until you either enable JavaScript in your browser, or switch to a browser that supports it.', 'pronamic_ideal' ); ?>
+						<?php esc_html_e('JavaScript is disabled in your browser. You will not be able to complete your purchase until you either enable JavaScript in your browser, or switch to a browser that supports it.', 'pronamic_ideal'); ?>
 					</p>
 				</noscript>
 			</form>
@@ -772,7 +772,7 @@ 
 	 * @param array $errors Array with errors.
 	 * @return array
 	 */
-	public function validate_payment_form( $errors ) {
+	public function validate_payment_form($errors) {
 		return $errors;
 	}
 
@@ -799,18 +799,18 @@ 
 
 				?>
 				<td>
-					<?php esc_html_e( 'Configuration', 'pronamic_ideal' ); ?>
+					<?php esc_html_e('Configuration', 'pronamic_ideal'); ?>
 				</td>
 				<td>
-					<select name="<?php echo esc_attr( $name ); ?>">
+					<select name="<?php echo esc_attr($name); ?>">
 						<?php
 
-						foreach ( Plugin::get_config_select_options( $this->payment_method ) as $value => $label ) {
+						foreach (Plugin::get_config_select_options($this->payment_method) as $value => $label) {
 							printf(
 								'<option value="%s" %s>%s</option>',
-								esc_attr( $value ),
-								selected( $value, $this->settings->config_id, false ),
-								esc_html( $label )
+								esc_attr($value),
+								selected($value, $this->settings->config_id, false),
+								esc_html($label)
 							);
 						}
 
@@ -832,7 +832,7 @@ 
 	 * @param array $errors Array with errors.
 	 * @return array
 	 */
-	public function validate_options_form( $errors ) {
+	public function validate_options_form($errors) {
 		return $errors;
 	}
 
@@ -857,7 +857,7 @@ 
 	 * @see https://github.com/wp-premium/memberpress-basic/blob/1.3.18/app/gateways/MeprStripeGateway.php#L1108-L1168
 	 * @see https://github.com/wp-premium/memberpress-basic/blob/1.3.18/app/controllers/MeprAccountCtrl.php#L388
 	 */
-	public function display_update_account_form( $sub_id, $errors = array(), $message = '' ) {
+	public function display_update_account_form($sub_id, $errors = array(), $message = '') {
 
 	}
 
@@ -870,7 +870,7 @@ 
 	 * @param  array $errors Array with errors.
 	 * @return array
 	 */
-	public function validate_update_account_form( $errors = array() ) {
+	public function validate_update_account_form($errors = array()) {
 		return $errors;
 	}
 
@@ -882,7 +882,7 @@ 
 	 *
 	 * @param int $sub_id Subscription ID.
 	 */
-	public function process_update_account_form( $sub_id ) {
+	public function process_update_account_form($sub_id) {
 
 	}