wp-pay-extensions / memberpress

gateways/MeprDirectDebitIDealGateway.php

Spacing +1 added lines, -1 removed lines

@@ -8,4 +8,4 @@
  * @package   Pronamic\WordPress\Pay\Extensions\MemberPress
  */
 
-class_alias( 'Pronamic\WordPress\Pay\Extensions\MemberPress\DirectDebitIDealGateway', 'MeprDirectDebitIDealGateway' );
+class_alias('Pronamic\WordPress\Pay\Extensions\MemberPress\DirectDebitIDealGateway', 'MeprDirectDebitIDealGateway');

gateways/MeprIDealGateway.php

Spacing +1 added lines, -1 removed lines

@@ -8,4 +8,4 @@
  * @package   Pronamic\WordPress\Pay\Extensions\MemberPress
  */
 
-class_alias( 'Pronamic\WordPress\Pay\Extensions\MemberPress\IDealGateway', 'MeprIDealGateway' );
+class_alias('Pronamic\WordPress\Pay\Extensions\MemberPress\IDealGateway', 'MeprIDealGateway');

gateways/MeprMisterCashGateway.php

Spacing +1 added lines, -1 removed lines

@@ -8,4 +8,4 @@
  * @package   Pronamic\WordPress\Pay\Extensions\MemberPress
  */
 
-class_alias( 'Pronamic\WordPress\Pay\Extensions\MemberPress\BancontactGateway', 'MeprMisterCashGateway' );
+class_alias('Pronamic\WordPress\Pay\Extensions\MemberPress\BancontactGateway', 'MeprMisterCashGateway');

gateways/MeprPronamicPayPalGateway.php

Spacing +1 added lines, -1 removed lines

@@ -8,4 +8,4 @@
  * @package   Pronamic\WordPress\Pay\Extensions\MemberPress
  */
 
-class_alias( 'Pronamic\WordPress\Pay\Extensions\MemberPress\PayPalGateway', 'MeprPronamicPayPalGateway' );
+class_alias('Pronamic\WordPress\Pay\Extensions\MemberPress\PayPalGateway', 'MeprPronamicPayPalGateway');

src/SubscriptionStatuses.php

Spacing +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
 	 *
 	 * @param string $status MemberPress subscription status value.
 	 */
-	public static function transform( $status ) {
-		switch ( $status ) {
+	public static function transform($status) {
+		switch ($status) {
 			case MeprSubscription::$pending_str:
 				return Statuses::OPEN;
 			case MeprSubscription::$active_str:

gateways/MeprDirectDebitBancontactGateway.php

Spacing +1 added lines, -1 removed lines

@@ -8,4 +8,4 @@
  * @package   Pronamic\WordPress\Pay\Extensions\MemberPress
  */
 
-class_alias( 'Pronamic\WordPress\Pay\Extensions\MemberPress\DirectDebitBancontactGateway', 'MeprDirectDebitBancontactGateway' );
+class_alias('Pronamic\WordPress\Pay\Extensions\MemberPress\DirectDebitBancontactGateway', 'MeprDirectDebitBancontactGateway');

src/BancontactGateway.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 	 * @return string
 	 */
 	protected function get_icon() {
-		return plugins_url( 'images/bancontact/icon-32x32.png', Plugin::$file );
+		return plugins_url('images/bancontact/icon-32x32.png', Plugin::$file);
 	}
 
 	/**

src/Gateway.php

Spacing +114 added lines, -114 removed lines

@@ -55,10 +55,10 @@ 
 	public function __construct() {
 		// 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) {
 
 	}
 
@@ -194,8 +194,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();
 			}
@@ -205,7 +205,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;
 	}
@@ -225,12 +225,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;
 	}
@@ -254,7 +254,7 @@ 
 
 		$subscription = $transaction->subscription();
 
-		if ( $subscription ) {
+		if ($subscription) {
 			$subscription->status     = MeprSubscription::$active_str;
 			$subscription->created_at = $transaction->created_at;
 			$subscription->store();
@@ -268,9 +268,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',
@@ -288,24 +288,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;
 	}
@@ -317,7 +317,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) {
 
 	}
 
@@ -337,7 +337,7 @@ 
 	 *
 	 * @param MeprTransaction $transaction MemberPress transaction object.
 	 */
-	public function process_trial_payment( $transaction ) {
+	public function process_trial_payment($transaction) {
 
 	}
 
@@ -348,7 +348,7 @@ 
 	 *
 	 * @param MeprTransaction $transaction MemberPress transaction object.
 	 */
-	public function record_trial_payment( $transaction ) {
+	public function record_trial_payment($transaction) {
 
 	}
 
@@ -359,7 +359,7 @@ 
 	 *
 	 * @param MeprTransaction $txn MemberPress transaction object.
 	 */
-	public function process_create_subscription( $txn ) {
+	public function process_create_subscription($txn) {
 
 	}
 
@@ -380,7 +380,7 @@ 
 	 *
 	 * @param int $sub_id Subscription ID.
 	 */
-	public function process_update_subscription( $sub_id ) {
+	public function process_update_subscription($sub_id) {
 
 	}
 
@@ -400,21 +400,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->id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -423,19 +423,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();
 		}
@@ -457,21 +457,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->id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -482,35 +482,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();
 		}
@@ -532,21 +532,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->id );
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $sub->id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
@@ -555,27 +555,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();
 		}
@@ -602,7 +602,7 @@ 
 	 *
 	 * @param MeprTransaction $txn MemberPress transaction object.
 	 */
-	public function process_signup_form( $txn ) {
+	public function process_signup_form($txn) {
 
 	}
 
@@ -613,28 +613,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, $this );
+		$data = new PaymentData($txn, $this);
 
-		$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.
 	 */
-	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) {
 
 	}
 

src/Extension.php

Spacing +66 added lines, -66 removed lines

@@ -47,21 +47,21 @@ 
 	 */
 	public function __construct() {
 		// @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/lib/MeprGatewayFactory.php#L48-50
-		add_filter( 'mepr-gateway-paths', array( $this, 'gateway_paths' ) );
+		add_filter('mepr-gateway-paths', array($this, 'gateway_paths'));
 
-		add_filter( 'pronamic_payment_redirect_url_' . self::SLUG, array( __CLASS__, 'redirect_url' ), 10, 2 );
-		add_action( 'pronamic_payment_status_update_' . self::SLUG, array( __CLASS__, 'status_update' ), 10, 1 );
+		add_filter('pronamic_payment_redirect_url_' . self::SLUG, array(__CLASS__, 'redirect_url'), 10, 2);
+		add_action('pronamic_payment_status_update_' . self::SLUG, array(__CLASS__, 'status_update'), 10, 1);
 
-		add_filter( 'pronamic_payment_source_text_' . self::SLUG, array( __CLASS__, 'source_text' ), 10, 2 );
-		add_filter( 'pronamic_payment_source_description_' . self::SLUG, array( __CLASS__, 'source_description' ), 10, 2 );
-		add_filter( 'pronamic_payment_source_url_' . self::SLUG, array( __CLASS__, 'source_url' ), 10, 2 );
-		add_filter( 'pronamic_subscription_source_text_' . self::SLUG, array( __CLASS__, 'subscription_source_text' ), 10, 2 );
-		add_filter( 'pronamic_subscription_source_description_' . self::SLUG, array( __CLASS__, 'subscription_source_description' ), 10, 2 );
-		add_filter( 'pronamic_subscription_source_url_' . self::SLUG, array( __CLASS__, 'subscription_source_url' ), 10, 2 );
+		add_filter('pronamic_payment_source_text_' . self::SLUG, array(__CLASS__, 'source_text'), 10, 2);
+		add_filter('pronamic_payment_source_description_' . self::SLUG, array(__CLASS__, 'source_description'), 10, 2);
+		add_filter('pronamic_payment_source_url_' . self::SLUG, array(__CLASS__, 'source_url'), 10, 2);
+		add_filter('pronamic_subscription_source_text_' . self::SLUG, array(__CLASS__, 'subscription_source_text'), 10, 2);
+		add_filter('pronamic_subscription_source_description_' . self::SLUG, array(__CLASS__, 'subscription_source_description'), 10, 2);
+		add_filter('pronamic_subscription_source_url_' . self::SLUG, array(__CLASS__, 'subscription_source_url'), 10, 2);
 
-		add_action( 'mepr_subscription_pre_delete', array( $this, 'subscription_pre_delete' ), 10, 1 );
+		add_action('mepr_subscription_pre_delete', array($this, 'subscription_pre_delete'), 10, 1);
 
-		add_action( 'mepr_subscription_transition_status', array( $this, 'memberpress_subscription_transition_status' ), 10, 3 );
+		add_action('mepr_subscription_transition_status', array($this, 'memberpress_subscription_transition_status'), 10, 3);
 	}
 
 	/**
@@ -72,8 +72,8 @@ 
 	 * @param array $paths Array with gateway paths.
 	 * @return array
 	 */
-	public function gateway_paths( $paths ) {
-		$paths[] = dirname( __FILE__ ) . '/../gateways/';
+	public function gateway_paths($paths) {
+		$paths[] = dirname(__FILE__) . '/../gateways/';
 
 		return $paths;
 	}
@@ -88,14 +88,14 @@ 
 	 *
 	 * @return string
 	 */
-	public static function redirect_url( $url, Payment $payment ) {
+	public static function redirect_url($url, Payment $payment) {
 		global $transaction;
 
 		$transaction_id = $payment->get_source_id();
 
-		$transaction = new MeprTransaction( $transaction_id );
+		$transaction = new MeprTransaction($transaction_id);
 
-		switch ( $payment->get_status() ) {
+		switch ($payment->get_status()) {
 			case Statuses::CANCELLED:
 			case Statuses::EXPIRED:
 			case Statuses::FAILURE:
@@ -106,9 +106,9 @@ 
 						'action'   => 'payment_form',
 						'txn'      => $transaction->trans_num,
 						'errors'   => array(
-							__( 'Payment failed. Please try again.', 'pronamic_ideal' ),
+							__('Payment failed. Please try again.', 'pronamic_ideal'),
 						),
-						'_wpnonce' => wp_create_nonce( 'mepr_payment_form' ),
+						'_wpnonce' => wp_create_nonce('mepr_payment_form'),
 					),
 					$product->url()
 				);
@@ -118,8 +118,8 @@ 
 				// @see https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/models/MeprOptions.php#L768-782
 				$mepr_options = MeprOptions::fetch();
 
-				$product         = new MeprProduct( $transaction->product_id );
-				$sanitized_title = sanitize_title( $product->post_title );
+				$product         = new MeprProduct($transaction->product_id);
+				$sanitized_title = sanitize_title($product->post_title);
 
 				$args = array(
 					'membership_id' => $product->ID,
@@ -127,7 +127,7 @@ 
 					'trans_num'     => $transaction->trans_num,
 				);
 
-				$url = $mepr_options->thankyou_page_url( http_build_query( $args ) );
+				$url = $mepr_options->thankyou_page_url(http_build_query($args));
 
 				break;
 			case Statuses::OPEN:
@@ -145,26 +145,26 @@ 
 	 *
 	 * @param Payment $payment The payment whose status is updated.
 	 */
-	public static function status_update( Payment $payment ) {
+	public static function status_update(Payment $payment) {
 		$transaction_id = $payment->get_source_id();
 
-		$transaction = new MeprTransaction( $transaction_id );
+		$transaction = new MeprTransaction($transaction_id);
 
-		if ( $payment->get_recurring() ) {
+		if ($payment->get_recurring()) {
 			$subscription = $transaction->subscription();
 
-			if ( empty( $subscription ) || empty( $subscription->id ) ) {
+			if (empty($subscription) || empty($subscription->id)) {
 				$subscription_id = $payment->get_subscription()->get_source_id();
 
-				$subscription = new MeprSubscription( $subscription_id );
+				$subscription = new MeprSubscription($subscription_id);
 			}
 
 			// Same source ID and first transaction ID for recurring payment means we need to add a new transaction.
-			if ( $payment->get_source_id() === $subscription->id ) {
+			if ($payment->get_source_id() === $subscription->id) {
 				// First transaction.
 				$first_txn = $subscription->first_txn();
 
-				if ( false === $first_txn || ! ( $first_txn instanceof MeprTransaction ) ) {
+				if (false === $first_txn || ! ($first_txn instanceof MeprTransaction)) {
 					$first_txn             = new MeprTransaction();
 					$first_txn->user_id    = $subscription->user_id;
 					$first_txn->product_id = $subscription->product_id;
@@ -175,7 +175,7 @@ 
 				// Transaction number.
 				$trans_num = $payment->get_transaction_id();
 
-				if ( empty( $trans_num ) ) {
+				if (empty($trans_num)) {
 					$trans_num = uniqid();
 				}
 
@@ -191,16 +191,16 @@ 
 				$transaction->status          = MeprTransaction::$pending_str;
 				$transaction->subscription_id = $subscription->id;
 
-				$transaction->set_gross( $payment->get_amount()->get_amount() );
+				$transaction->set_gross($payment->get_amount()->get_amount());
 
 				$transaction->store();
 
 				// Set source ID.
-				$payment->set_meta( 'source_id', $transaction->id );
+				$payment->set_meta('source_id', $transaction->id);
 
 				$payment->source_id = $transaction->id;
 
-				if ( MeprSubscription::$active_str === $subscription->status ) {
+				if (MeprSubscription::$active_str === $subscription->status) {
 					/*
 					 * We create a 'confirmed' 'subscription_confirmation'
 					 * transaction for a grace period of 15 days.
@@ -222,9 +222,9 @@ 
 					$subscription_confirmation->txn_type        = MeprTransaction::$subscription_confirmation_str;
 					$subscription_confirmation->status          = MeprTransaction::$confirmed_str;
 					$subscription_confirmation->subscription_id = $subscription->id;
-					$subscription_confirmation->expires_at      = MeprUtils::ts_to_mysql_date( strtotime( $payment->post->post_date_gmt ) + MeprUtils::days( 15 ), 'Y-m-d 23:59:59' );
+					$subscription_confirmation->expires_at      = MeprUtils::ts_to_mysql_date(strtotime($payment->post->post_date_gmt) + MeprUtils::days(15), 'Y-m-d 23:59:59');
 
-					$subscription_confirmation->set_subtotal( 0.00 );
+					$subscription_confirmation->set_subtotal(0.00);
 
 					$subscription_confirmation->store();
 				}
@@ -239,12 +239,12 @@ 
 			)
 		);
 
-		if ( $should_update ) {
+		if ($should_update) {
 			$gateway = new Gateway();
 
 			$gateway->mp_txn = $transaction;
 
-			switch ( $payment->get_status() ) {
+			switch ($payment->get_status()) {
 				case Statuses::CANCELLED:
 				case Statuses::EXPIRED:
 				case Statuses::FAILURE:
@@ -252,7 +252,7 @@ 
 
 					break;
 				case Statuses::SUCCESS:
-					if ( $payment->get_recurring() ) {
+					if ($payment->get_recurring()) {
 						$gateway->record_subscription_payment();
 					} else {
 						$gateway->record_payment();
@@ -271,25 +271,25 @@ 
 	 *
 	 * @param int $subscription_id MemberPress subscription id.
 	 */
-	public function subscription_pre_delete( $subscription_id ) {
-		$subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $subscription_id );
+	public function subscription_pre_delete($subscription_id) {
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $subscription_id);
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return;
 		}
 
 		// Add note.
 		$note = sprintf(
 			/* translators: %s: MemberPress */
-			__( '%s subscription deleted.', 'pronamic_ideal' ),
-			__( 'MemberPress', 'pronamic_ideal' )
+			__('%s subscription deleted.', '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();
 		}
@@ -303,8 +303,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function source_text( $text, Payment $payment ) {
-		$text = __( 'MemberPress', 'pronamic_ideal' ) . '<br />';
+	public static function source_text($text, Payment $payment) {
+		$text = __('MemberPress', 'pronamic_ideal') . '<br />';
 
 		$text .= sprintf(
 			'<a href="%s">%s</a>',
@@ -314,10 +314,10 @@ 
 					'action' => 'edit',
 					'id'     => $payment->source_id,
 				),
-				admin_url( 'admin.php' )
+				admin_url('admin.php')
 			),
 			/* translators: %s: payment source id */
-			sprintf( __( 'Transaction %s', 'pronamic_ideal' ), $payment->source_id )
+			sprintf(__('Transaction %s', 'pronamic_ideal'), $payment->source_id)
 		);
 
 		return $text;
@@ -331,8 +331,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function subscription_source_text( $text, Subscription $subscription ) {
-		$text = __( 'MemberPress', 'pronamic_ideal' ) . '<br />';
+	public static function subscription_source_text($text, Subscription $subscription) {
+		$text = __('MemberPress', 'pronamic_ideal') . '<br />';
 
 		$text .= sprintf(
 			'<a href="%s">%s</a>',
@@ -341,10 +341,10 @@ 
 					'page'         => 'memberpress-subscriptions',
 					'subscription' => $subscription->source_id,
 				),
-				admin_url( 'admin.php' )
+				admin_url('admin.php')
 			),
 			/* translators: %s: payment source id */
-			sprintf( __( 'Subscription %s', 'pronamic_ideal' ), $subscription->source_id )
+			sprintf(__('Subscription %s', 'pronamic_ideal'), $subscription->source_id)
 		);
 
 		return $text;
@@ -358,8 +358,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function source_description( $description, Payment $payment ) {
-		return __( 'MemberPress Transaction', 'pronamic_ideal' );
+	public static function source_description($description, Payment $payment) {
+		return __('MemberPress Transaction', 'pronamic_ideal');
 	}
 
 	/**
@@ -370,8 +370,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function subscription_source_description( $description, Subscription $subscription ) {
-		return __( 'MemberPress Subscription', 'pronamic_ideal' );
+	public static function subscription_source_description($description, Subscription $subscription) {
+		return __('MemberPress Subscription', 'pronamic_ideal');
 	}
 
 	/**
@@ -382,14 +382,14 @@ 
 	 *
 	 * @return string
 	 */
-	public static function source_url( $url, Payment $payment ) {
+	public static function source_url($url, Payment $payment) {
 		$url = add_query_arg(
 			array(
 				'page'   => 'memberpress-trans',
 				'action' => 'edit',
 				'id'     => $payment->source_id,
 			),
-			admin_url( 'admin.php' )
+			admin_url('admin.php')
 		);
 
 		return $url;
@@ -403,13 +403,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function subscription_source_url( $url, Subscription $subscription ) {
+	public static function subscription_source_url($url, Subscription $subscription) {
 		$url = add_query_arg(
 			array(
 				'page'         => 'memberpress-subscriptions',
 				'subscription' => $subscription->source_id,
 			),
-			admin_url( 'admin.php' )
+			admin_url('admin.php')
 		);
 
 		return $url;
@@ -426,16 +426,16 @@ 
 	 * @param string           $status_new               New status identifier.
 	 * @param MeprSubscription $memberpress_subscription MemberPress subscription object.
 	 */
-	public function memberpress_subscription_transition_status( $status_old, $status_new, $memberpress_subscription ) {
-		$subscription = get_pronamic_subscription_by_meta( '_pronamic_subscription_source_id', $memberpress_subscription->id );
+	public function memberpress_subscription_transition_status($status_old, $status_new, $memberpress_subscription) {
+		$subscription = get_pronamic_subscription_by_meta('_pronamic_subscription_source_id', $memberpress_subscription->id);
 
-		if ( empty( $subscription ) ) {
+		if (empty($subscription)) {
 			return;
 		}
 
-		$status = SubscriptionStatuses::transform( $status_new );
+		$status = SubscriptionStatuses::transform($status_new);
 
-		$subscription->set_status( $status );
+		$subscription->set_status($status);
 
 		$subscription->save();
 	}