wp-pay-gateways / multisafepay

src/Settings.php

Spacing +25 added lines, -25 removed lines

@@ -16,18 +16,18 @@ 
  */
 class Settings extends GatewaySettings {
 	public function __construct() {
-		add_filter( 'pronamic_pay_gateway_sections', array( $this, 'sections' ) );
-		add_filter( 'pronamic_pay_gateway_fields', array( $this, 'fields' ) );
+		add_filter('pronamic_pay_gateway_sections', array($this, 'sections'));
+		add_filter('pronamic_pay_gateway_fields', array($this, 'fields'));
 	}
 
-	public function sections( array $sections ) {
+	public function sections(array $sections) {
 		$sections['multisafepay'] = array(
-			'title'       => __( 'MultiSafepay', 'pronamic_ideal' ),
-			'methods'     => array( 'multisafepay' ),
+			'title'       => __('MultiSafepay', 'pronamic_ideal'),
+			'methods'     => array('multisafepay'),
 			'description' => sprintf(
 				/* translators: 1: MultiSafepay */
-				__( 'Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal' ),
-				__( 'MultiSafepay', 'pronamic_ideal' )
+				__('Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal'),
+				__('MultiSafepay', 'pronamic_ideal')
 			),
 		);
 
@@ -35,21 +35,21 @@ 
 		return $sections;
 	}
 
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		// Account ID
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'multisafepay',
 			'meta_key' => '_pronamic_gateway_multisafepay_account_id',
-			'title'    => __( 'Account ID', 'pronamic_ideal' ),
+			'title'    => __('Account ID', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'code' ),
-			'methods'  => array( 'multisafepay_connect' ),
+			'classes'  => array('code'),
+			'methods'  => array('multisafepay_connect'),
 			'tooltip'  => sprintf(
 				'%s %s.',
-				__( 'Account ID', 'pronamic_ideal' ),
+				__('Account ID', 'pronamic_ideal'),
 				/* translators: %s: MultiSafepay */
-				sprintf( __( 'as mentioned in the %s dashboard', 'pronamic_ideal' ), __( 'MultiSafepay', 'pronamic_ideal' ) )
+				sprintf(__('as mentioned in the %s dashboard', 'pronamic_ideal'), __('MultiSafepay', 'pronamic_ideal'))
 			),
 		);
 
@@ -58,15 +58,15 @@ 
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'multisafepay',
 			'meta_key' => '_pronamic_gateway_multisafepay_site_id',
-			'title'    => __( 'Site ID', 'pronamic_ideal' ),
+			'title'    => __('Site ID', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'code' ),
-			'methods'  => array( 'multisafepay_connect' ),
+			'classes'  => array('code'),
+			'methods'  => array('multisafepay_connect'),
 			'tooltip'  => sprintf(
 				'%s %s.',
-				__( 'Site ID', 'pronamic_ideal' ),
+				__('Site ID', 'pronamic_ideal'),
 				/* translators: %s: MultiSafepay */
-				sprintf( __( 'as mentioned in the %s dashboard', 'pronamic_ideal' ), __( 'MultiSafepay', 'pronamic_ideal' ) )
+				sprintf(__('as mentioned in the %s dashboard', 'pronamic_ideal'), __('MultiSafepay', 'pronamic_ideal'))
 			),
 		);
 
@@ -75,26 +75,26 @@ 
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'multisafepay',
 			'meta_key' => '_pronamic_gateway_multisafepay_site_code',
-			'title'    => __( 'Site Security Code', 'pronamic_ideal' ),
+			'title'    => __('Site Security Code', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'code' ),
-			'methods'  => array( 'multisafepay_connect' ),
+			'classes'  => array('code'),
+			'methods'  => array('multisafepay_connect'),
 			'tooltip'  => sprintf(
 				'%s %s.',
-				__( 'Site Security Code', 'pronamic_ideal' ),
+				__('Site Security Code', 'pronamic_ideal'),
 				/* translators: %s: MultiSafepay */
-				sprintf( __( 'as mentioned in the %s dashboard', 'pronamic_ideal' ), __( 'MultiSafepay', 'pronamic_ideal' ) )
+				sprintf(__('as mentioned in the %s dashboard', 'pronamic_ideal'), __('MultiSafepay', 'pronamic_ideal'))
 			),
 		);
 
 		// Transaction feedback
 		$fields[] = array(
 			'section' => 'multisafepay',
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'title'   => __('Transaction feedback', 'pronamic_ideal'),
 			'type'    => 'description',
 			'html'    => sprintf(
 				'<span class="dashicons dashicons-yes"></span> %s',
-				__( 'Payment status updates will be processed without any additional configuration.', 'pronamic_ideal' )
+				__('Payment status updates will be processed without any additional configuration.', 'pronamic_ideal')
 			),
 		);
 

src/Connect/Methods.php

Spacing +8 added lines, -8 removed lines

@@ -170,13 +170,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function transform( $payment_method, $default = null ) {
-		if ( ! is_scalar( $payment_method ) ) {
+	public static function transform($payment_method, $default = null) {
+		if ( ! is_scalar($payment_method)) {
 			return null;
 		}
 
-		if ( isset( self::$map[ $payment_method ] ) ) {
-			return self::$map[ $payment_method ];
+		if (isset(self::$map[$payment_method])) {
+			return self::$map[$payment_method];
 		}
 
 		return $default;
@@ -191,14 +191,14 @@ 
 	 *
 	 * @return string
 	 */
-	public static function transform_gateway_method( $method ) {
-		if ( ! is_scalar( $method ) ) {
+	public static function transform_gateway_method($method) {
+		if ( ! is_scalar($method)) {
 			return null;
 		}
 
-		$payment_method = array_search( $method, self::$map, true );
+		$payment_method = array_search($method, self::$map, true);
 
-		if ( ! $payment_method ) {
+		if ( ! $payment_method) {
 			return null;
 		}
 

src/Connect/Signature.php

Spacing +4 added lines, -4 removed lines

@@ -40,18 +40,18 @@
 	 *
 	 * @return string
 	 */
-	public static function generate( $amount, $currency, $account, $site_id, $transaction_id ) {
+	public static function generate($amount, $currency, $account, $site_id, $transaction_id) {
 		$values = array(
-			Util::amount_to_cents( $amount ),
+			Util::amount_to_cents($amount),
 			$currency,
 			$account,
 			$site_id,
 			$transaction_id,
 		);
 
-		$string = implode( '', $values );
+		$string = implode('', $values);
 
-		$signature = md5( $string );
+		$signature = md5($string);
 
 		return $signature;
 	}

src/Connect/ConfigFactory.php

Spacing +6 added lines, -6 removed lines

@@ -24,15 +24,15 @@
 	 *
 	 * @return Config
 	 */
-	public function get_config( $post_id ) {
+	public function get_config($post_id) {
 		$config = new Config();
 
-		$config->mode       = get_post_meta( $post_id, '_pronamic_gateway_mode', true );
-		$config->account_id = get_post_meta( $post_id, '_pronamic_gateway_multisafepay_account_id', true );
-		$config->site_id    = get_post_meta( $post_id, '_pronamic_gateway_multisafepay_site_id', true );
-		$config->site_code  = get_post_meta( $post_id, '_pronamic_gateway_multisafepay_site_code', true );
+		$config->mode       = get_post_meta($post_id, '_pronamic_gateway_mode', true);
+		$config->account_id = get_post_meta($post_id, '_pronamic_gateway_multisafepay_account_id', true);
+		$config->site_id    = get_post_meta($post_id, '_pronamic_gateway_multisafepay_site_id', true);
+		$config->site_code  = get_post_meta($post_id, '_pronamic_gateway_multisafepay_site_code', true);
 
-		if ( Gateway::MODE_TEST === $config->mode ) {
+		if (Gateway::MODE_TEST === $config->mode) {
 			$config->api_url = MultiSafepay::API_TEST_URL;
 		} else {
 			$config->api_url = MultiSafepay::API_PRODUCTION_URL;

src/Connect/Gateway.php

Spacing +54 added lines, -54 removed lines

@@ -40,17 +40,17 @@ 
 	 *
 	 * @param Config $config
 	 */
-	public function __construct( Config $config ) {
-		parent::__construct( $config );
+	public function __construct(Config $config) {
+		parent::__construct($config);
 
 		$this->supports = array(
 			'payment_status_request',
 		);
 
-		$this->set_method( Gateway::METHOD_HTTP_REDIRECT );
-		$this->set_has_feedback( true );
-		$this->set_amount_minimum( 0 );
-		$this->set_slug( self::SLUG );
+		$this->set_method(Gateway::METHOD_HTTP_REDIRECT);
+		$this->set_has_feedback(true);
+		$this->set_amount_minimum(0);
+		$this->set_slug(self::SLUG);
 
 		$this->client = new Client();
 
@@ -72,9 +72,9 @@ 
 		$merchant->site_id          = $this->config->site_id;
 		$merchant->site_secure_code = $this->config->site_code;
 
-		$result = $this->client->get_ideal_issuers( $merchant );
+		$result = $this->client->get_ideal_issuers($merchant);
 
-		if ( $result ) {
+		if ($result) {
 			$groups[] = array(
 				'options' => $result,
 			);
@@ -91,10 +91,10 @@ 
 	public function get_credit_card_issuers() {
 		$groups[] = array(
 			'options' => array(
-				Methods::AMEX       => _x( 'AMEX', 'Payment method name', 'pronamic_ideal' ),
-				Methods::MAESTRO    => _x( 'Maestro', 'Payment method name', 'pronamic_ideal' ),
-				Methods::MASTERCARD => _x( 'MASTER', 'Payment method name', 'pronamic_ideal' ),
-				Methods::VISA       => _x( 'VISA', 'Payment method name', 'pronamic_ideal' ),
+				Methods::AMEX       => _x('AMEX', 'Payment method name', 'pronamic_ideal'),
+				Methods::MAESTRO    => _x('Maestro', 'Payment method name', 'pronamic_ideal'),
+				Methods::MASTERCARD => _x('MASTER', 'Payment method name', 'pronamic_ideal'),
+				Methods::VISA       => _x('VISA', 'Payment method name', 'pronamic_ideal'),
 			),
 		);
 
@@ -107,12 +107,12 @@ 
 	 * @since 1.2.0
 	 */
 	public function get_issuer_field() {
-		switch ( $this->get_payment_method() ) {
+		switch ($this->get_payment_method()) {
 			case PaymentMethods::IDEAL:
 				return array(
 					'id'       => 'pronamic_ideal_issuer_id',
 					'name'     => 'pronamic_ideal_issuer_id',
-					'label'    => __( 'Choose your bank', 'pronamic_ideal' ),
+					'label'    => __('Choose your bank', 'pronamic_ideal'),
 					'required' => true,
 					'type'     => 'select',
 					'choices'  => $this->get_transient_issuers(),
@@ -122,7 +122,7 @@ 
 				return array(
 					'id'       => 'pronamic_credit_card_issuer_id',
 					'name'     => 'pronamic_credit_card_issuer_id',
-					'label'    => __( 'Choose your credit card issuer', 'pronamic_ideal' ),
+					'label'    => __('Choose your credit card issuer', 'pronamic_ideal'),
 					'required' => true,
 					'type'     => 'select',
 					'choices'  => $this->get_transient_credit_card_issuers(),
@@ -148,18 +148,18 @@ 
 		$customer = new Customer();
 
 		// Get gateways.
-		$result = $this->client->get_gateways( $merchant, $customer );
+		$result = $this->client->get_gateways($merchant, $customer);
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			$this->error = $this->client->get_error();
 
 			return $payment_methods;
 		}
 
-		foreach ( $result as $method => $title ) {
-			$payment_method = Methods::transform_gateway_method( $method );
+		foreach ($result as $method => $title) {
+			$payment_method = Methods::transform_gateway_method($method);
 
-			if ( $payment_method ) {
+			if ($payment_method) {
 				$payment_methods[] = $payment_method;
 			}
 		}
@@ -194,12 +194,12 @@ 
 	 *
 	 * @param Payment $payment payment object
 	 */
-	public function start( Payment $payment ) {
+	public function start(Payment $payment) {
 		$payment_method = $payment->get_method();
 
 		$transaction_description = $payment->get_description();
 
-		if ( empty( $transaction_description ) ) {
+		if (empty($transaction_description)) {
 			$transaction_description = $payment->get_id();
 		}
 
@@ -216,8 +216,8 @@ 
 		// Customer
 		$customer               = new Customer();
 		$customer->locale       = $payment->get_locale();
-		$customer->ip_address   = Server::get( 'REMOTE_ADDR', FILTER_VALIDATE_IP );
-		$customer->forwarded_ip = Server::get( 'HTTP_X_FORWARDED_FOR', FILTER_VALIDATE_IP );
+		$customer->ip_address   = Server::get('REMOTE_ADDR', FILTER_VALIDATE_IP);
+		$customer->forwarded_ip = Server::get('HTTP_X_FORWARDED_FOR', FILTER_VALIDATE_IP);
 		$customer->first_name   = $payment->get_first_name();
 		$customer->last_name    = $payment->get_last_name();
 		$customer->email        = $payment->get_email();
@@ -229,96 +229,96 @@ 
 		$transaction->amount      = $payment->get_amount()->get_amount();
 		$transaction->description = $transaction_description;
 
-		switch ( $payment_method ) {
+		switch ($payment_method) {
 			case PaymentMethods::IDEAL:
 				$transaction->gateway = Methods::IDEAL;
 
 				$issuer = $payment->get_issuer();
 
-				if ( empty( $issuer ) ) {
-					$message = new RedirectTransactionRequestMessage( $merchant, $customer, $transaction );
+				if (empty($issuer)) {
+					$message = new RedirectTransactionRequestMessage($merchant, $customer, $transaction);
 				} else {
 					$gateway_info = new GatewayInfo();
 
 					$gateway_info->issuer_id = $issuer;
 
-					$message = new DirectTransactionRequestMessage( $merchant, $customer, $transaction, $gateway_info );
+					$message = new DirectTransactionRequestMessage($merchant, $customer, $transaction, $gateway_info);
 				}
 
 				break;
 			case PaymentMethods::CREDIT_CARD:
-				$gateway = Methods::transform( $payment_method );
+				$gateway = Methods::transform($payment_method);
 
 				$issuer = $payment->get_issuer();
 
-				if ( empty( $issuer ) ) {
-					if ( $gateway ) {
+				if (empty($issuer)) {
+					if ($gateway) {
 						$transaction->gateway = $gateway;
 					}
 				} else {
 					$transaction->gateway = $issuer;
 				}
 
-				$message = new RedirectTransactionRequestMessage( $merchant, $customer, $transaction );
+				$message = new RedirectTransactionRequestMessage($merchant, $customer, $transaction);
 
 				break;
 			default:
-				$gateway = Methods::transform( $payment_method );
+				$gateway = Methods::transform($payment_method);
 
-				if ( $gateway ) {
+				if ($gateway) {
 					$transaction->gateway = $gateway;
 				}
 
-				if ( ! isset( $transaction->gateway ) && ! empty( $payment_method ) ) {
+				if ( ! isset($transaction->gateway) && ! empty($payment_method)) {
 					// Leap of faith if the WordPress payment method could not transform to a Mollie method?
 					$transaction->gateway = $payment_method;
 				}
 
-				$message = new RedirectTransactionRequestMessage( $merchant, $customer, $transaction );
+				$message = new RedirectTransactionRequestMessage($merchant, $customer, $transaction);
 		}
 
-		$signature = Signature::generate( $transaction->amount, $transaction->currency, $merchant->account, $merchant->site_id, $transaction->id );
+		$signature = Signature::generate($transaction->amount, $transaction->currency, $merchant->account, $merchant->site_id, $transaction->id);
 
 		$message->signature = $signature;
 
-		$response = $this->client->start_transaction( $message );
+		$response = $this->client->start_transaction($message);
 
-		if ( $response ) {
+		if ($response) {
 			$transaction = $response->transaction;
 
-			$payment->set_transaction_id( $transaction->id );
+			$payment->set_transaction_id($transaction->id);
 
-			if ( isset( $transaction->payment_url ) ) {
-				$payment->set_action_url( $transaction->payment_url );
+			if (isset($transaction->payment_url)) {
+				$payment->set_action_url($transaction->payment_url);
 			}
 
-			if ( isset( $response->gateway_info->redirect_url ) ) {
-				$payment->set_action_url( $response->gateway_info->redirect_url );
+			if (isset($response->gateway_info->redirect_url)) {
+				$payment->set_action_url($response->gateway_info->redirect_url);
 			}
 		} else {
 			$this->error = $this->client->get_error();
 		}
 	}
 
-	public function update_status( Payment $payment ) {
+	public function update_status(Payment $payment) {
 		$merchant = new Merchant();
 
 		$merchant->account          = $this->config->account_id;
 		$merchant->site_id          = $this->config->site_id;
 		$merchant->site_secure_code = $this->config->site_code;
 
-		$message = new StatusRequestMessage( $merchant, $payment->get_transaction_id() );
+		$message = new StatusRequestMessage($merchant, $payment->get_transaction_id());
 
-		$result = $this->client->get_status( $message );
+		$result = $this->client->get_status($message);
 
-		if ( $result ) {
-			$status = Statuses::transform( $result->ewallet->status );
+		if ($result) {
+			$status = Statuses::transform($result->ewallet->status);
 
-			$payment->set_status( $status );
-			$payment->set_consumer_name( $result->payment_details->account_holder_name );
-			$payment->set_consumer_iban( $result->payment_details->account_iban );
-			$payment->set_consumer_bic( $result->payment_details->account_bic );
-			$payment->set_consumer_account_number( $result->payment_details->account_id );
+			$payment->set_status($status);
+			$payment->set_consumer_name($result->payment_details->account_holder_name);
+			$payment->set_consumer_iban($result->payment_details->account_iban);
+			$payment->set_consumer_bic($result->payment_details->account_bic);
+			$payment->set_consumer_account_number($result->payment_details->account_id);
 		} else {
 			$this->error = $this->client->get_error();
 		}

src/Connect/Statuses.php

Spacing +2 added lines, -2 removed lines

@@ -70,8 +70,8 @@
 	 *
 	 * @return null|string
 	 */
-	public static function transform( $status ) {
-		switch ( $status ) {
+	public static function transform($status) {
+		switch ($status) {
 			case self::COMPLETED:
 				return Core_Statuses::SUCCESS;
 

src/Connect/Integration.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 		$this->id            = 'multisafepay-connect';
 		$this->name          = 'MultiSafepay - Connect';
 		$this->url           = 'http://www.multisafepay.com/';
-		$this->product_url   = __( 'http://www.multisafepay.com/', 'pronamic_ideal' );
+		$this->product_url   = __('http://www.multisafepay.com/', 'pronamic_ideal');
 		$this->dashboard_url = 'https://merchant.multisafepay.com/';
 		$this->provider      = 'multisafepay';
 	}

src/Connect/Client.php

Spacing +30 added lines, -30 removed lines

@@ -57,50 +57,50 @@ 
 		return $this->error;
 	}
 
-	private function parse_xml( $xml ) {
-		switch ( $xml->getName() ) {
+	private function parse_xml($xml) {
+		switch ($xml->getName()) {
 			case IDealIssuersRequestMessage::NAME:
-				return IDealIssuersResponseMessage::parse( $xml );
+				return IDealIssuersResponseMessage::parse($xml);
 
 			case GatewaysRequestMessage::NAME:
-				return GatewaysResponseMessage::parse( $xml );
+				return GatewaysResponseMessage::parse($xml);
 
 			case DirectTransactionRequestMessage::NAME:
-				return DirectTransactionResponseMessage::parse( $xml );
+				return DirectTransactionResponseMessage::parse($xml);
 
 			case RedirectTransactionRequestMessage::NAME:
-				return RedirectTransactionResponseMessage::parse( $xml );
+				return RedirectTransactionResponseMessage::parse($xml);
 
 			case StatusRequestMessage::NAME:
-				return StatusResponseMessage::parse( $xml );
+				return StatusResponseMessage::parse($xml);
 		}
 
 		return false;
 	}
 
-	private function request( $message ) {
+	private function request($message) {
 		$return = false;
 
-		$result = Core_Util::remote_get_body( $this->api_url, 200, array(
+		$result = Core_Util::remote_get_body($this->api_url, 200, array(
 			'method' => 'POST',
 			'body'   => (string) $message,
-		) );
+		));
 
-		if ( is_wp_error( $result ) ) {
+		if (is_wp_error($result)) {
 			$this->error = $result;
 
 			return false;
 		}
 
-		$xml = Core_Util::simplexml_load_string( $result );
+		$xml = Core_Util::simplexml_load_string($result);
 
-		if ( is_wp_error( $xml ) ) {
+		if (is_wp_error($xml)) {
 			$this->error = $xml;
 		} else {
-			$return = $this->parse_xml( $xml );
+			$return = $this->parse_xml($xml);
 
-			if ( is_object( $return ) && isset( $return->result ) && 'error' === $return->result ) {
-				$this->error = new WP_Error( 'multisafepay_error', $xml->error->description, $xml->error );
+			if (is_object($return) && isset($return->result) && 'error' === $return->result) {
+				$this->error = new WP_Error('multisafepay_error', $xml->error->description, $xml->error);
 				$return      = false;
 			}
 		}
@@ -113,14 +113,14 @@ 
 	 *
 	 * @since 1.2.0
 	 */
-	public function get_ideal_issuers( $merchant ) {
+	public function get_ideal_issuers($merchant) {
 		$return = false;
 
-		$request = new IDealIssuersRequestMessage( $merchant );
+		$request = new IDealIssuersRequestMessage($merchant);
 
-		$response = $this->request( $request );
+		$response = $this->request($request);
 
-		if ( $response ) {
+		if ($response) {
 			$return = $response->issuers;
 		}
 
@@ -132,14 +132,14 @@ 
 	 *
 	 * @since 1.2.0
 	 */
-	public function get_gateways( $merchant, $customer ) {
+	public function get_gateways($merchant, $customer) {
 		$return = false;
 
-		$request = new GatewaysRequestMessage( $merchant, $customer );
+		$request = new GatewaysRequestMessage($merchant, $customer);
 
-		$response = $this->request( $request );
+		$response = $this->request($request);
 
-		if ( $response ) {
+		if ($response) {
 			$return = $response->gateways;
 		}
 
@@ -151,12 +151,12 @@ 
 	 *
 	 * @param array $message
 	 */
-	public function start_transaction( $message ) {
+	public function start_transaction($message) {
 		$return = false;
 
-		$response = $this->request( $message );
+		$response = $this->request($message);
 
-		if ( $response ) {
+		if ($response) {
 			$return = $response;
 		}
 
@@ -168,12 +168,12 @@ 
 	 *
 	 * @param array $message
 	 */
-	public function get_status( $message ) {
+	public function get_status($message) {
 		$return = false;
 
-		$response = $this->request( $message );
+		$response = $this->request($message);
 
-		if ( $response ) {
+		if ($response) {
 			$return = $response;
 		}
 

src/Connect/XML/Message.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	/**
 	 * Constructs and initialize an message
 	 */
-	public function __construct( $name ) {
+	public function __construct($name) {
 		$this->name = $name;
 	}