wp-pay-gateways / sisow

src/Gateway.php

Spacing +41 added lines, -41 removed lines

@@ -37,17 +37,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);
 	}
 
 	/**
@@ -60,7 +60,7 @@ 
 
 		$result = $this->client->get_directory();
 
-		if ( $result ) {
+		if ($result) {
 			$groups[] = array(
 				'options' => $result,
 			);
@@ -111,14 +111,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(
@@ -128,12 +128,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(),
@@ -145,15 +145,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,16 +164,16 @@ 
 				)
 			);
 
-			if ( null !== $customer->get_birth_date() ) {
-				$request->set_parameter( 'birth_date', $customer->get_birth_date()->format( 'ddmmYYYY' ) );
+			if (null !== $customer->get_birth_date()) {
+				$request->set_parameter('birth_date', $customer->get_birth_date()->format('ddmmYYYY'));
 			}
 		}
 
 		// 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(
@@ -201,10 +201,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(
@@ -231,16 +231,16 @@ 
 		}
 
 		// 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() ) ? null : $line->get_unit_price()->get_cents();
-				$total     = ( null === $line->get_total_amount() ) ? null : $line->get_total_amount()->get_cents();
-				$tax       = ( null === $line->get_tax_amount() ) ? null : $line->get_tax_amount()->get_cents();
-				$net_total = ( $total - $tax );
+			foreach ($lines as $line) {
+				$net_price = (null === $line->get_unit_price()) ? null : $line->get_unit_price()->get_cents();
+				$total     = (null === $line->get_total_amount()) ? null : $line->get_total_amount()->get_cents();
+				$tax       = (null === $line->get_tax_amount()) ? null : $line->get_tax_amount()->get_cents();
+				$net_total = ($total - $tax);
 
 				$request->merge_parameters(
 					array(
@@ -260,11 +260,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();
 		}
@@ -275,28 +275,28 @@ 
 	 *
 	 * @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;
 		}
 
-		if ( $result instanceof Transaction ) {
+		if ($result instanceof Transaction) {
 			$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/Client.php

Spacing +33 added lines, -33 removed lines

@@ -68,7 +68,7 @@ 
 	 * @param string $merchant_id  Merchant ID.
 	 * @param string $merchant_key Merchant key.
 	 */
-	public function __construct( $merchant_id, $merchant_key ) {
+	public function __construct($merchant_id, $merchant_key) {
 		$this->merchant_id  = $merchant_id;
 		$this->merchant_key = $merchant_key;
 	}
@@ -87,7 +87,7 @@ 
 	 *
 	 * @param boolean $test_mode True if test mode, false otherwise.
 	 */
-	public function set_test_mode( $test_mode ) {
+	public function set_test_mode($test_mode) {
 		$this->test_mode = $test_mode;
 	}
 
@@ -98,11 +98,11 @@ 
 	 * @param Request|null $request Request.
 	 * @return false|SimpleXMLElement
 	 */
-	private function send_request( $method, Request $request = null ) {
+	private function send_request($method, Request $request = null) {
 		$url = self::API_URL . '/' . $method;
 
-		if ( null !== $request ) {
-			$request->sign( $this->merchant_key );
+		if (null !== $request) {
+			$request->sign($this->merchant_key);
 		}
 
 		$result = Core_Util::remote_get_body(
@@ -110,24 +110,24 @@ 
 			200,
 			array(
 				'method' => 'POST',
-				'body'   => ( null == $request ) ? null : $request->get_parameters(),
+				'body'   => (null == $request) ? null : $request->get_parameters(),
 			)
 		);
 
-		if ( $result instanceof WP_Error ) {
+		if ($result instanceof WP_Error) {
 			$this->error = $result;
 
 			return false;
 		}
 
-		if ( ! is_string( $result ) ) {
+		if ( ! is_string($result)) {
 			return false;
 		}
 
 		// XML.
-		$xml = Core_Util::simplexml_load_string( $result );
+		$xml = Core_Util::simplexml_load_string($result);
 
-		if ( $xml instanceof WP_Error ) {
+		if ($xml instanceof WP_Error) {
 			$this->error = $xml;
 
 			return false;
@@ -142,31 +142,31 @@ 
 	 * @param SimpleXMLElement $document Document.
 	 * @return WP_Error|Transaction|Error
 	 */
-	private function parse_document( SimpleXMLElement $document ) {
+	private function parse_document(SimpleXMLElement $document) {
 		$this->error = null;
 
 		$name = $document->getName();
 
-		switch ( $name ) {
+		switch ($name) {
 			case 'errorresponse':
-				$sisow_error = ErrorParser::parse( $document->error );
+				$sisow_error = ErrorParser::parse($document->error);
 
-				$this->error = new WP_Error( 'ideal_sisow_error', $sisow_error->message, $sisow_error );
+				$this->error = new WP_Error('ideal_sisow_error', $sisow_error->message, $sisow_error);
 
 				return $sisow_error;
 			case 'transactionrequest':
-				$transaction = TransactionParser::parse( $document->transaction );
+				$transaction = TransactionParser::parse($document->transaction);
 
 				return $transaction;
 			case 'statusresponse':
-				$transaction = TransactionParser::parse( $document->transaction );
+				$transaction = TransactionParser::parse($document->transaction);
 
 				return $transaction;
 			default:
 				return new WP_Error(
 					'ideal_sisow_error',
 					/* translators: %s: XML document element name */
-					sprintf( __( 'Unknwon Sisow message (%s)', 'pronamic_ideal' ), $name )
+					sprintf(__('Unknwon Sisow message (%s)', 'pronamic_ideal'), $name)
 				);
 		}
 	}
@@ -177,27 +177,27 @@ 
 	 * @return array|false
 	 */
 	public function get_directory() {
-		if ( $this->test_mode ) {
+		if ($this->test_mode) {
 			return array(
-				'99' => __( 'Sisow Bank (test)', 'pronamic_ideal' ),
+				'99' => __('Sisow Bank (test)', 'pronamic_ideal'),
 			);
 		}
 
 		// Request.
-		$result = $this->send_request( RequestMethods::DIRECTORY_REQUEST );
+		$result = $this->send_request(RequestMethods::DIRECTORY_REQUEST);
 
-		if ( false === $result ) {
+		if (false === $result) {
 			return false;
 		}
 
 		// Parse.
 		$directory = array();
 
-		foreach ( $result->directory->issuer as $issuer ) {
+		foreach ($result->directory->issuer as $issuer) {
 			$id   = (string) $issuer->issuerid;
 			$name = (string) $issuer->issuername;
 
-			$directory[ $id ] = $name;
+			$directory[$id] = $name;
 		}
 
 		return $directory;
@@ -209,18 +209,18 @@ 
 	 * @param TransactionRequest $request Transaction request.
 	 * @return Transaction|false
 	 */
-	public function create_transaction( TransactionRequest $request ) {
+	public function create_transaction(TransactionRequest $request) {
 		// Request.
-		$response = $this->send_request( RequestMethods::TRANSACTION_REQUEST, $request );
+		$response = $this->send_request(RequestMethods::TRANSACTION_REQUEST, $request);
 
-		if ( false === $response ) {
+		if (false === $response) {
 			return false;
 		}
 
 		// Parse.
-		$message = $this->parse_document( $response );
+		$message = $this->parse_document($response);
 
-		if ( $message instanceof Transaction ) {
+		if ($message instanceof Transaction) {
 			return $message;
 		}
 
@@ -233,18 +233,18 @@ 
 	 * @param StatusRequest $request Status request object.
 	 * @return Transaction|Error|WP_Error|false
 	 */
-	public function get_status( StatusRequest $request ) {
+	public function get_status(StatusRequest $request) {
 		// Request.
-		$response = $this->send_request( RequestMethods::STATUS_REQUEST, $request );
+		$response = $this->send_request(RequestMethods::STATUS_REQUEST, $request);
 
-		if ( false === $response ) {
+		if (false === $response) {
 			return false;
 		}
 
 		// Parse.
-		$message = $this->parse_document( $response );
+		$message = $this->parse_document($response);
 
-		if ( $message instanceof Transaction ) {
+		if ($message instanceof Transaction) {
 			return $message;
 		}