wp-pay-extensions / memberpress

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();
 	}

src/PaymentData.php

Spacing +27 added lines, -27 removed lines

@@ -67,7 +67,7 @@ 
 	 * @param MeprTransaction $transaction MemberPress transaction object.
 	 * @param Gateway         $gateway     MemberPress gateway object.
 	 */
-	public function __construct( MeprTransaction $transaction, Gateway $gateway ) {
+	public function __construct(MeprTransaction $transaction, Gateway $gateway) {
 		parent::__construct();
 
 		$this->transaction = $transaction;
@@ -135,12 +135,12 @@ 
 		$items = new Items();
 
 		$item = new Item();
-		$item->set_number( $this->get_order_id() );
-		$item->set_description( $this->get_description() );
-		$item->set_price( $this->transaction->total );
-		$item->set_quantity( 1 );
+		$item->set_number($this->get_order_id());
+		$item->set_description($this->get_description());
+		$item->set_price($this->transaction->total);
+		$item->set_quantity(1);
 
-		$items->addItem( $item );
+		$items->addItem($item);
 
 		return $items;
 	}
@@ -216,9 +216,9 @@ 
 	 * @return string|null
 	 */
 	public function get_address() {
-		$value = $this->user->address( 'one', false );
+		$value = $this->user->address('one', false);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return null;
 		}
 
@@ -233,9 +233,9 @@ 
 	 * @return string|null
 	 */
 	public function get_city() {
-		$value = $this->user->address( 'city', false );
+		$value = $this->user->address('city', false);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return null;
 		}
 
@@ -250,9 +250,9 @@ 
 	 * @return string|null
 	 */
 	public function get_zip() {
-		$value = $this->user->address( 'zip', false );
+		$value = $this->user->address('zip', false);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return null;
 		}
 
@@ -267,9 +267,9 @@ 
 	 * @return string|null
 	 */
 	public function get_country() {
-		$value = $this->user->address( 'country', false );
+		$value = $this->user->address('country', false);
 
-		if ( false === $value ) {
+		if (false === $value) {
 			return null;
 		}
 
@@ -288,7 +288,7 @@ 
 
 		// @link https://gitlab.com/pronamic/memberpress/blob/1.2.4/app/models/MeprOptions.php#L768-782
 		// @link https://github.com/wp-premium/memberpress-basic/blob/1.3.18/app/models/MeprOptions.php#L806-L835
-		return $mepr_options->thankyou_page_url( 'trans_num=' . $this->transaction->id );
+		return $mepr_options->thankyou_page_url('trans_num=' . $this->transaction->id);
 	}
 
 	/**
@@ -300,19 +300,19 @@ 
 	 * @return string
 	 */
 	public function get_cancel_url() {
-		if ( isset( $this->transaction->product_id ) && $this->transaction->product_id > 0 ) {
-			$product = new MeprProduct( $this->transaction->product_id );
+		if (isset($this->transaction->product_id) && $this->transaction->product_id > 0) {
+			$product = new MeprProduct($this->transaction->product_id);
 
-			$url = $this->gateway->message_page_url( $product, 'cancel' );
+			$url = $this->gateway->message_page_url($product, 'cancel');
 
-			if ( false !== $url ) {
+			if (false !== $url) {
 				return $url;
 			}
 		}
 
 		$mepr_options = MeprOptions::fetch();
 
-		return $mepr_options->account_page_url( 'action=subscriptions' );
+		return $mepr_options->account_page_url('action=subscriptions');
 	}
 
 	/**
@@ -332,7 +332,7 @@ 
 	public function get_error_url() {
 		$mepr_options = MeprOptions::fetch();
 
-		return $mepr_options->account_page_url( 'action=subscriptions' );
+		return $mepr_options->account_page_url('action=subscriptions');
 	}
 
 	/**
@@ -345,26 +345,26 @@ 
 	public function get_subscription() {
 		$product = $this->transaction->product();
 
-		if ( $product->is_one_time_payment() ) {
+		if ($product->is_one_time_payment()) {
 			return false;
 		}
 
 		$mp_subscription = $this->transaction->subscription();
 
-		if ( ! $mp_subscription ) {
+		if ( ! $mp_subscription) {
 			return false;
 		}
 
 		$frequency = '';
 
-		if ( $mp_subscription->limit_cycles ) {
+		if ($mp_subscription->limit_cycles) {
 			$frequency = $mp_subscription->limit_cycles;
 		}
 
 		$subscription                  = new Subscription();
 		$subscription->frequency       = $frequency;
 		$subscription->interval        = $product->period;
-		$subscription->interval_period = Core_Util::to_period( $product->period_type );
+		$subscription->interval_period = Core_Util::to_period($product->period_type);
 		$subscription->description     = sprintf(
 			'Order #%s - %s',
 			$this->get_source_id(),
@@ -390,11 +390,11 @@ 
 	public function get_subscription_source_id() {
 		$subscription = $this->get_subscription();
 
-		if ( ! $subscription ) {
+		if ( ! $subscription) {
 			return false;
 		}
 
-		if ( ! empty( $this->transaction->subscription_id ) ) {
+		if ( ! empty($this->transaction->subscription_id)) {
 			return $this->transaction->subscription_id;
 		}