wp-pay-gateways / sisow

src/Client.php

Spacing +38 added lines, -38 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,7 +98,7 @@ 
 	 * @param array  $parameters Parameters.
 	 * @return string|WP_Error
 	 */
-	private function send_request( $method, array $parameters = array() ) {
+	private function send_request($method, array $parameters = array()) {
 		$url = self::API_URL . '/' . $method;
 
 		return Core_Util::remote_get_body(
@@ -117,31 +117,31 @@ 
 	 * @param SimpleXMLElement $document Document.
 	 * @return WP_Error|Transaction
 	 */
-	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)
 				);
 		}
 	}
@@ -154,36 +154,36 @@ 
 	public function get_directory() {
 		$directory = false;
 
-		if ( $this->test_mode ) {
-			$directory = array( '99' => __( 'Sisow Bank (test)', 'pronamic_ideal' ) );
+		if ($this->test_mode) {
+			$directory = array('99' => __('Sisow Bank (test)', 'pronamic_ideal'));
 		} else {
 			// Request.
-			$result = $this->send_request( RequestMethods::DIRECTORY_REQUEST );
+			$result = $this->send_request(RequestMethods::DIRECTORY_REQUEST);
 
-			if ( $result instanceof WP_Error ) {
+			if ($result instanceof WP_Error) {
 				$this->error = $result;
 
 				return $directory;
 			}
 
 			// XML.
-			$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;
 
 				return $directory;
 			}
 
 			// Parse.
-			if ( $xml instanceof SimpleXMLElement ) {
+			if ($xml instanceof SimpleXMLElement) {
 				$directory = array();
 
-				foreach ( $xml->directory->issuer as $issuer ) {
+				foreach ($xml->directory->issuer as $issuer) {
 					$id   = (string) $issuer->issuerid;
 					$name = (string) $issuer->issuername;
 
-					$directory[ $id ] = $name;
+					$directory[$id] = $name;
 				}
 			}
 		}
@@ -197,34 +197,34 @@ 
 	 * @param TransactionRequest $request Transaction request.
 	 * @return Transaction|false
 	 */
-	public function create_transaction( TransactionRequest $request ) {
+	public function create_transaction(TransactionRequest $request) {
 		$result = false;
 
 		// Request.
-		$request->sign( $this->merchant_key );
+		$request->sign($this->merchant_key);
 
-		$response = $this->send_request( RequestMethods::TRANSACTION_REQUEST, $request->get_parameters() );
+		$response = $this->send_request(RequestMethods::TRANSACTION_REQUEST, $request->get_parameters());
 
-		if ( $response instanceof WP_Error ) {
+		if ($response instanceof WP_Error) {
 			$this->error = $response;
 
 			return $result;
 		}
 
 		// XML.
-		$xml = Core_Util::simplexml_load_string( $response );
+		$xml = Core_Util::simplexml_load_string($response);
 
-		if ( $xml instanceof WP_Error) ) {
+		if ($xml instanceof WP_Error) ) {
 			$this->error = $xml;
 
 			return $result;
 		}
 
 		// Parse.
-		if ( $xml instanceof SimpleXMLElement ) {
-			$message = $this->parse_document( $xml );
+		if ($xml instanceof SimpleXMLElement) {
+			$message = $this->parse_document($xml);
 
-			if ( $message instanceof Transaction ) {
+			if ($message instanceof Transaction) {
 				$result = $message;
 			}
 		}
@@ -240,7 +240,7 @@ 
 	 * @param string $merchant_id    Merchant ID.
 	 * @param string $merchant_key   Merchant key.
 	 */
-	public static function create_status_sha1( $transaction_id, $shop_id, $merchant_id, $merchant_key ) {
+	public static function create_status_sha1($transaction_id, $shop_id, $merchant_id, $merchant_key) {
 		return sha1(
 			$transaction_id .
 			$shop_id .
@@ -255,10 +255,10 @@ 
 	 * @param string $transaction_id Transaction ID.
 	 * @return Transaction|false
 	 */
-	public function get_status( $transaction_id ) {
+	public function get_status($transaction_id) {
 		$status = false;
 
-		if ( '' === $transaction_id ) {
+		if ('' === $transaction_id) {
 			return $status;
 		}
 
@@ -266,30 +266,30 @@ 
 		$parameters = array(
 			'merchantid' => $this->merchant_id,
 			'trxid'      => $transaction_id,
-			'sha1'       => self::create_status_sha1( $transaction_id, '', $this->merchant_id, $this->merchant_key ),
+			'sha1'       => self::create_status_sha1($transaction_id, '', $this->merchant_id, $this->merchant_key),
 		);
 
 		// Request.
-		$result = $this->send_request( RequestMethods::STATUS_REQUEST, $parameters );
+		$result = $this->send_request(RequestMethods::STATUS_REQUEST, $parameters);
 
-		if ( $result instanceof WP_Error ) {
+		if ($result instanceof WP_Error) {
 			$this->error = $result;
 
 			return $status;
 		}
 
 		// 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 $status;
 		}
 
 		// Parse.
-		if ( $xml instanceof SimpleXMLElement ) {
-			$status = $this->parse_document( $xml );
+		if ($xml instanceof SimpleXMLElement) {
+			$status = $this->parse_document($xml);
 
 			return $status;
 		}

Braces +3 added lines, -1 removed lines

@@ -214,8 +214,10 @@
 		// XML.
 		$xml = Core_Util::simplexml_load_string( $response );
 
-		if ( $xml instanceof WP_Error) ) {
+		if ( $xml instanceof WP_Error) {
+			) {
 			$this->error = $xml;
+		}
 
 			return $result;
 		}