wp-pay-gateways / sisow

src/Gateway.php

Spacing +45 added lines, -45 removed lines

@@ -38,17 +38,17 @@ 
 	 *
 	 * @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_method(self::METHOD_HTTP_REDIRECT);
 
-		$this->client = new Client( $config->merchant_id, $config->merchant_key );
-		$this->client->set_test_mode( self::MODE_TEST === $config->mode );
+		$this->client = new Client($config->merchant_id, $config->merchant_key);
+		$this->client->set_test_mode(self::MODE_TEST === $config->mode);
 	}
 
 	/**
@@ -61,7 +61,7 @@ 
 
 		$result = $this->client->get_directory();
 
-		if ( $result ) {
+		if ($result) {
 			$groups[] = array(
 				'options' => $result,
 			);
@@ -112,14 +112,14 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
+	public function start(Payment $payment) {
 		// Order and purchase ID.
 		$order_id    = $payment->get_order_id();
-		$purchase_id = strval( empty( $order_id ) ? $payment->get_id() : $order_id );
+		$purchase_id = strval(empty($order_id) ? $payment->get_id() : $order_id);
 
 		// Maximum length for purchase ID is 16 characters, otherwise an error will occur:
 		// ideal_sisow_error - purchaseid too long (16).
-		$purchase_id = substr( $purchase_id, 0, 16 );
+		$purchase_id = substr($purchase_id, 0, 16);
 
 		// New transaction request.
 		$request = new TransactionRequest(
@@ -129,12 +129,12 @@ 
 
 		$request->merge_parameters(
 			array(
-				'payment'      => Methods::transform( $payment->get_method(), $payment->get_method() ),
-				'purchaseid'   => substr( $purchase_id, 0, 16 ),
+				'payment'      => Methods::transform($payment->get_method(), $payment->get_method()),
+				'purchaseid'   => substr($purchase_id, 0, 16),
 				'entrancecode' => $payment->get_entrance_code(),
 				'amount'       => $payment->get_amount()->get_cents(),
-				'description'  => substr( $payment->get_description(), 0, 32 ),
-				'testmode'     => ( self::MODE_TEST === $this->config->mode ) ? 'true' : 'false',
+				'description'  => substr($payment->get_description(), 0, 32),
+				'testmode'     => (self::MODE_TEST === $this->config->mode) ? 'true' : 'false',
 				'returnurl'    => $payment->get_return_url(),
 				'cancelurl'    => $payment->get_return_url(),
 				'notifyurl'    => $payment->get_return_url(),
@@ -146,15 +146,15 @@ 
 		);
 
 		// Payment method.
-		$this->set_payment_method( null === $payment->get_method() ? PaymentMethods::IDEAL : $payment->get_method() );
+		$this->set_payment_method(null === $payment->get_method() ? PaymentMethods::IDEAL : $payment->get_method());
 
 		// Additional parameters for payment method.
-		if ( PaymentMethods::IDEALQR === $payment->get_method() ) {
-			$request->set_parameter( 'qrcode', 'true' );
+		if (PaymentMethods::IDEALQR === $payment->get_method()) {
+			$request->set_parameter('qrcode', 'true');
 		}
 
 		// Customer.
-		if ( null !== $payment->get_customer() ) {
+		if (null !== $payment->get_customer()) {
 			$customer = $payment->get_customer();
 
 			$request->merge_parameters(
@@ -164,25 +164,25 @@ 
 				)
 			);
 
-			if ( null !== $customer->get_locale() ) {
+			if (null !== $customer->get_locale()) {
 				/*
 				 * @link https://github.com/wp-pay-gateways/sisow/tree/feature/post-pay/documentation#parameter-locale
 				 */
-				$sisow_locale = strtoupper( substr( $customer->get_locale(), -2 ) );
+				$sisow_locale = strtoupper(substr($customer->get_locale(), -2));
 
-				$request->set_parameter( 'locale', $sisow_locale );
+				$request->set_parameter('locale', $sisow_locale);
 			}
 
-			if ( null !== $customer->get_birth_date() ) {
-				$request->set_parameter( 'birth_date', $customer->get_birth_date()->format( 'dmY' ) );
+			if (null !== $customer->get_birth_date()) {
+				$request->set_parameter('birth_date', $customer->get_birth_date()->format('dmY'));
 			}
 		}
 
 		// Billing address.
-		if ( null !== $payment->get_billing_address() ) {
+		if (null !== $payment->get_billing_address()) {
 			$address = $payment->get_billing_address();
 
-			if ( null !== $address->get_name() ) {
+			if (null !== $address->get_name()) {
 				$name = $address->get_name();
 
 				$request->merge_parameters(
@@ -210,10 +210,10 @@ 
 		}
 
 		// Shipping address.
-		if ( null !== $payment->get_shipping_address() ) {
+		if (null !== $payment->get_shipping_address()) {
 			$address = $payment->get_shipping_address();
 
-			if ( null !== $address->get_name() ) {
+			if (null !== $address->get_name()) {
 				$name = $address->get_name();
 
 				$request->merge_parameters(
@@ -240,22 +240,22 @@ 
 		}
 
 		// Lines.
-		if ( null !== $payment->get_lines() ) {
+		if (null !== $payment->get_lines()) {
 			$lines = $payment->get_lines();
 
 			$x = 1;
 
-			foreach ( $lines as $line ) {
-				$net_price = ( null === $line->get_unit_price_excluding_tax() ) ? null : $line->get_unit_price_excluding_tax()->get_cents();
-				$total     = ( null === $line->get_total_amount() ) ? null : $line->get_total_amount()->get_cents();
-				$net_total = ( null === $line->get_total_amount_excluding_tax() ) ? null : $line->get_total_amount_excluding_tax()->get_cents();
-				$tax       = ( null === $line->get_tax_amount() ) ? null : $line->get_tax_amount()->get_cents();
+			foreach ($lines as $line) {
+				$net_price = (null === $line->get_unit_price_excluding_tax()) ? null : $line->get_unit_price_excluding_tax()->get_cents();
+				$total     = (null === $line->get_total_amount()) ? null : $line->get_total_amount()->get_cents();
+				$net_total = (null === $line->get_total_amount_excluding_tax()) ? null : $line->get_total_amount_excluding_tax()->get_cents();
+				$tax       = (null === $line->get_tax_amount()) ? null : $line->get_tax_amount()->get_cents();
 
 				$product_id = $line->get_id();
 
-				if ( PaymentLineType::SHIPPING === $line->get_type() ) {
+				if (PaymentLineType::SHIPPING === $line->get_type()) {
 					$product_id = 'shipping';
-				} elseif ( PaymentLineType::FEE === $line->get_type() ) {
+				} elseif (PaymentLineType::FEE === $line->get_type()) {
 					$product_id = 'paymentfee';
 				}
 
@@ -277,11 +277,11 @@ 
 		}
 
 		// Create transaction.
-		$result = $this->client->create_transaction( $request );
+		$result = $this->client->create_transaction($request);
 
-		if ( false !== $result ) {
-			$payment->set_transaction_id( $result->id );
-			$payment->set_action_url( $result->issuer_url );
+		if (false !== $result) {
+			$payment->set_transaction_id($result->id);
+			$payment->set_action_url($result->issuer_url);
 		} else {
 			$this->error = $this->client->get_error();
 		}
@@ -292,16 +292,16 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
+	public function update_status(Payment $payment) {
 		$request = new StatusRequest(
 			$payment->get_transaction_id(),
 			$this->config->merchant_id,
 			$this->config->shop_id
 		);
 
-		$result = $this->client->get_status( $request );
+		$result = $this->client->get_status($request);
 
-		if ( false === $result ) {
+		if (false === $result) {
 			$this->error = $this->client->get_error();
 
 			return;
@@ -309,9 +309,9 @@ 
 
 		$transaction = $result;
 
-		$payment->set_status( $transaction->status );
-		$payment->set_consumer_name( $transaction->consumer_name );
-		$payment->set_consumer_account_number( $transaction->consumer_account );
-		$payment->set_consumer_city( $transaction->consumer_city );
+		$payment->set_status($transaction->status);
+		$payment->set_consumer_name($transaction->consumer_name);
+		$payment->set_consumer_account_number($transaction->consumer_account);
+		$payment->set_consumer_city($transaction->consumer_city);
 	}
 }

src/Util.php

Spacing +4 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
 	 * $            Dollar
 	 */
 	public static function get_pattern() {
-		if ( null === self::$pattern ) {
+		if (null === self::$pattern) {
 			$characters = array(
 				'A-Z',
 				'a-z',
@@ -90,7 +90,7 @@ 
 			 * We use a # as a regex delimiter instead of a / so we don't have to escape the slash.
 			 * @link http://stackoverflow.com/q/12239424
 			 */
-			self::$pattern = '#[^' . implode( $characters ) . ']#';
+			self::$pattern = '#[^' . implode($characters) . ']#';
 		}
 
 		return self::$pattern;
@@ -102,7 +102,7 @@ 
 	 * @param string $string String to filter.
 	 * @return mixed
 	 */
-	public static function filter( $string ) {
-		return preg_replace( self::get_pattern(), '', $string );
+	public static function filter($string) {
+		return preg_replace(self::get_pattern(), '', $string);
 	}
 }

src/XML/TransactionParser.php

Spacing +25 added lines, -25 removed lines

@@ -33,57 +33,57 @@
 	 * @param SimpleXMLElement $xml XML element to parse.
 	 * @return Transaction
 	 */
-	public static function parse( SimpleXMLElement $xml ) {
+	public static function parse(SimpleXMLElement $xml) {
 		$transaction = new Transaction();
 
 		// Transaction request.
-		if ( isset( $xml->trxid ) ) {
-			$transaction->id = Security::filter( $xml->trxid );
+		if (isset($xml->trxid)) {
+			$transaction->id = Security::filter($xml->trxid);
 		}
 
-		if ( isset( $xml->issuerurl ) ) {
-			$transaction->issuer_url = urldecode( Security::filter( $xml->issuerurl ) );
+		if (isset($xml->issuerurl)) {
+			$transaction->issuer_url = urldecode(Security::filter($xml->issuerurl));
 		}
 
 		// Status response.
-		if ( isset( $xml->status ) ) {
-			$transaction->status = Security::filter( $xml->status );
+		if (isset($xml->status)) {
+			$transaction->status = Security::filter($xml->status);
 		}
 
-		if ( isset( $xml->amount ) ) {
-			$transaction->amount = Util::cents_to_amount( Security::filter( $xml->amount ) );
+		if (isset($xml->amount)) {
+			$transaction->amount = Util::cents_to_amount(Security::filter($xml->amount));
 		}
 
-		if ( isset( $xml->purchaseid ) ) {
-			$transaction->purchase_id = Security::filter( $xml->purchaseid );
+		if (isset($xml->purchaseid)) {
+			$transaction->purchase_id = Security::filter($xml->purchaseid);
 		}
 
-		if ( isset( $xml->description ) ) {
-			$transaction->description = Security::filter( $xml->description );
+		if (isset($xml->description)) {
+			$transaction->description = Security::filter($xml->description);
 		}
 
-		if ( isset( $xml->entrancecode ) ) {
-			$transaction->entrance_code = Security::filter( $xml->entrancecode );
+		if (isset($xml->entrancecode)) {
+			$transaction->entrance_code = Security::filter($xml->entrancecode);
 		}
 
-		if ( isset( $xml->issuerid ) ) {
-			$transaction->issuer_id = Security::filter( $xml->issuerid );
+		if (isset($xml->issuerid)) {
+			$transaction->issuer_id = Security::filter($xml->issuerid);
 		}
 
-		if ( isset( $xml->timestamp ) ) {
-			$transaction->timestamp = new DateTime( Security::filter( $xml->timestamp ) );
+		if (isset($xml->timestamp)) {
+			$transaction->timestamp = new DateTime(Security::filter($xml->timestamp));
 		}
 
-		if ( isset( $xml->consumername ) ) {
-			$transaction->consumer_name = Security::filter( $xml->consumername );
+		if (isset($xml->consumername)) {
+			$transaction->consumer_name = Security::filter($xml->consumername);
 		}
 
-		if ( isset( $xml->consumeraccount ) ) {
-			$transaction->consumer_account = Security::filter( $xml->consumeraccount );
+		if (isset($xml->consumeraccount)) {
+			$transaction->consumer_account = Security::filter($xml->consumeraccount);
 		}
 
-		if ( isset( $xml->consumercity ) ) {
-			$transaction->consumer_city = Security::filter( $xml->consumercity );
+		if (isset($xml->consumercity)) {
+			$transaction->consumer_city = Security::filter($xml->consumercity);
 		}
 
 		return $transaction;