wp-pay-gateways / multisafepay

src/Connect/XML/RedirectTransactionRequestMessage.php

Spacing +12 added lines, -12 removed lines

@@ -25,8 +25,8 @@ 
 	/**
 	 * Constructs and initialize an directory response message
 	 */
-	public function __construct( $merchant, $customer, $transaction ) {
-		parent::__construct( self::NAME );
+	public function __construct($merchant, $customer, $transaction) {
+		parent::__construct(self::NAME);
 
 		$this->merchant    = $merchant;
 		$this->customer    = $customer;
@@ -42,9 +42,9 @@ 
 		$document = parent::get_document();
 
 		// Merchant
-		$merchant = XML_Util::add_element( $document, $document->documentElement, 'merchant' );
+		$merchant = XML_Util::add_element($document, $document->documentElement, 'merchant');
 
-		XML_Util::add_elements( $document, $merchant, array(
+		XML_Util::add_elements($document, $merchant, array(
 			'account'          => $this->merchant->account,
 			'site_id'          => $this->merchant->site_id,
 			'site_secure_code' => $this->merchant->site_secure_code,
@@ -52,12 +52,12 @@ 
 			'redirect_url'     => $this->merchant->redirect_url,
 			'cancel_url'       => $this->merchant->cancel_url,
 			'close_window'     => $this->merchant->close_window,
-		) );
+		));
 
 		// Customer
-		$customer = XML_Util::add_element( $document, $document->documentElement, 'customer' );
+		$customer = XML_Util::add_element($document, $document->documentElement, 'customer');
 
-		XML_Util::add_elements( $document, $customer, array(
+		XML_Util::add_elements($document, $customer, array(
 			'locale'      => $this->customer->locale,
 			'ipaddress'   => $this->customer->ip_address,
 			'forwardedip' => $this->customer->forwarded_ip,
@@ -71,12 +71,12 @@ 
 			'country'     => $this->customer->country,
 			'phone'       => $this->customer->phone,
 			'email'       => $this->customer->email,
-		) );
+		));
 
 		// Transaction
-		$transaction = XML_Util::add_element( $document, $document->documentElement, 'transaction' );
+		$transaction = XML_Util::add_element($document, $document->documentElement, 'transaction');
 
-		XML_Util::add_elements( $document, $transaction, array(
+		XML_Util::add_elements($document, $transaction, array(
 			'id'          => $this->transaction->id,
 			'currency'    => $this->transaction->currency,
 			'amount'      => $this->transaction->amount,
@@ -88,10 +88,10 @@ 
 			'manual'      => $this->transaction->manual,
 			'gateway'     => $this->transaction->gateway,
 			'daysactive'  => $this->transaction->days_active,
-		) );
+		));
 
 		// Signature
-		XML_Util::add_element( $document, $document->documentElement, 'signature', $this->signature );
+		XML_Util::add_element($document, $document->documentElement, 'signature', $this->signature);
 
 		return $document;
 	}

src/Connect/XML/DirectTransactionRequestMessage.php

Spacing +16 added lines, -16 removed lines

@@ -41,8 +41,8 @@ 
 	 * @param Transaction $transaction
 	 * @param GatewayInfo $gateway_info
 	 */
-	public function __construct( Merchant $merchant, Customer $customer, Transaction $transaction, GatewayInfo $gateway_info = null ) {
-		parent::__construct( self::NAME );
+	public function __construct(Merchant $merchant, Customer $customer, Transaction $transaction, GatewayInfo $gateway_info = null) {
+		parent::__construct(self::NAME);
 
 		$this->merchant     = $merchant;
 		$this->customer     = $customer;
@@ -59,9 +59,9 @@ 
 		$document = parent::get_document();
 
 		// Merchant
-		$merchant = XML_Util::add_element( $document, $document->documentElement, 'merchant' );
+		$merchant = XML_Util::add_element($document, $document->documentElement, 'merchant');
 
-		XML_Util::add_elements( $document, $merchant, array(
+		XML_Util::add_elements($document, $merchant, array(
 			'account'          => $this->merchant->account,
 			'site_id'          => $this->merchant->site_id,
 			'site_secure_code' => $this->merchant->site_secure_code,
@@ -69,12 +69,12 @@ 
 			'redirect_url'     => $this->merchant->redirect_url,
 			'cancel_url'       => $this->merchant->cancel_url,
 			'close_window'     => $this->merchant->close_window,
-		) );
+		));
 
 		// Customer
-		$customer = XML_Util::add_element( $document, $document->documentElement, 'customer' );
+		$customer = XML_Util::add_element($document, $document->documentElement, 'customer');
 
-		XML_Util::add_elements( $document, $customer, array(
+		XML_Util::add_elements($document, $customer, array(
 			'locale'      => $this->customer->locale,
 			'ipaddress'   => $this->customer->ip_address,
 			'forwardedip' => $this->customer->forwarded_ip,
@@ -88,12 +88,12 @@ 
 			'country'     => $this->customer->country,
 			'phone'       => $this->customer->phone,
 			'email'       => $this->customer->email,
-		) );
+		));
 
 		// Transaction
-		$transaction = XML_Util::add_element( $document, $document->documentElement, 'transaction' );
+		$transaction = XML_Util::add_element($document, $document->documentElement, 'transaction');
 
-		XML_Util::add_elements( $document, $transaction, array(
+		XML_Util::add_elements($document, $transaction, array(
 			'id'          => $this->transaction->id,
 			'currency'    => $this->transaction->currency,
 			'amount'      => $this->transaction->amount,
@@ -105,19 +105,19 @@ 
 			'manual'      => $this->transaction->manual,
 			'gateway'     => $this->transaction->gateway,
 			'daysactive'  => $this->transaction->days_active,
-		) );
+		));
 
 		// Gateway info
-		if ( $this->gateway_info ) {
-			$gateway_info = XML_Util::add_element( $document, $document->documentElement, 'gatewayinfo' );
+		if ($this->gateway_info) {
+			$gateway_info = XML_Util::add_element($document, $document->documentElement, 'gatewayinfo');
 
-			XML_Util::add_elements( $document, $gateway_info, array(
+			XML_Util::add_elements($document, $gateway_info, array(
 				'issuerid' => $this->gateway_info->issuer_id,
-			) );
+			));
 		}
 
 		// Signature
-		XML_Util::add_element( $document, $document->documentElement, 'signature', $this->signature );
+		XML_Util::add_element($document, $document->documentElement, 'signature', $this->signature);
 
 		return $document;
 	}

src/Connect/Gateway.php

Spacing +51 added lines, -51 removed lines

@@ -47,15 +47,15 @@ 
 	 *
 	 * @param Config $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( self::METHOD_HTTP_REDIRECT );
-		$this->set_slug( self::SLUG );
+		$this->set_method(self::METHOD_HTTP_REDIRECT);
+		$this->set_slug(self::SLUG);
 
 		$this->client = new Client();
 
@@ -77,9 +77,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,
 			);
@@ -96,10 +96,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'),
 			),
 		);
 
@@ -124,18 +124,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;
 			}
 		}
@@ -170,12 +170,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();
 		}
 
@@ -191,13 +191,13 @@ 
 
 		// Customer.
 		$customer               = new Customer();
-		$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);
 
-		if ( null !== $payment->get_customer() ) {
+		if (null !== $payment->get_customer()) {
 			$name = $payment->get_customer()->get_name();
 
-			if ( null !== $name ) {
+			if (null !== $name) {
 				$customer->first_name = $name->get_first_name();
 				$customer->last_name  = $name->get_last_name();
 			}
@@ -213,71 +213,71 @@ 
 		$transaction->amount      = $payment->get_total_amount()->get_cents();
 		$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();
@@ -289,25 +289,25 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	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/Signature.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
 	 *
 	 * @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(
 			$amount,
 			$currency,
@@ -49,9 +49,9 @@ 
 			$transaction_id,
 		);
 
-		$string = implode( '', $values );
+		$string = implode('', $values);
 
-		$signature = md5( $string );
+		$signature = md5($string);
 
 		return $signature;
 	}