wp-pay-gateways / sisow

src/Request.php

Spacing +16 added lines, -16 removed lines

@@ -37,9 +37,9 @@ 
 	 * @param string      $merchant_id    Merchant ID.
 	 * @param string|null $shop_id        Shop ID.
 	 */
-	public function __construct( $merchant_id, $shop_id = null ) {
-		$this->set_parameter( 'merchantid', $merchant_id );
-		$this->set_parameter( 'shopid', $shop_id );
+	public function __construct($merchant_id, $shop_id = null) {
+		$this->set_parameter('merchantid', $merchant_id);
+		$this->set_parameter('shopid', $shop_id);
 	}
 
 	/**
@@ -48,9 +48,9 @@ 
 	 * @param string $parameter Parameter.
 	 * @return string|null
 	 */
-	public function get_parameter( $parameter ) {
-		if ( isset( $this->parameters[ $parameter ] ) ) {
-			return $this->parameters[ $parameter ];
+	public function get_parameter($parameter) {
+		if (isset($this->parameters[$parameter])) {
+			return $this->parameters[$parameter];
 		}
 
 		return null;
@@ -62,8 +62,8 @@ 
 	 * @param string $parameter Parameter.
 	 * @param string $value     Value.
 	 */
-	public function set_parameter( $parameter, $value ) {
-		$this->parameters[ $parameter ] = $value;
+	public function set_parameter($parameter, $value) {
+		$this->parameters[$parameter] = $value;
 	}
 
 	/**
@@ -80,8 +80,8 @@ 
 	 *
 	 * @param array $parameters Parameters.
 	 */
-	public function merge_parameters( $parameters ) {
-		$this->parameters = array_merge( $this->parameters, $parameters );
+	public function merge_parameters($parameters) {
+		$this->parameters = array_merge($this->parameters, $parameters);
 	}
 
 	/**
@@ -99,14 +99,14 @@ 
 	 * @param string $merchant_key Merchant key.
 	 * @return string
 	 */
-	public function get_signature( $merchant_key ) {
+	public function get_signature($merchant_key) {
 		$data = $this->get_signature_data();
 
 		$data[] = $merchant_key;
 
-		$string = implode( '', $data );
+		$string = implode('', $data);
 
-		$signature = sha1( $string );
+		$signature = sha1($string);
 
 		return $signature;
 	}
@@ -116,9 +116,9 @@ 
 	 *
 	 * @param string $merchant_key Merchant key.
 	 */
-	public function sign( $merchant_key ) {
-		$signature = $this->get_signature( $merchant_key );
+	public function sign($merchant_key) {
+		$signature = $this->get_signature($merchant_key);
 
-		$this->set_parameter( 'sha1', $signature );
+		$this->set_parameter('sha1', $signature);
 	}
 }

src/StatusRequest.php

Spacing +7 added lines, -7 removed lines

@@ -28,10 +28,10 @@ 
 	 * @param string      $merchant_id    Merchant ID.
 	 * @param string|null $shop_id        Shop ID.
 	 */
-	public function __construct( $transaction_id, $merhant_id, $shop_id = null ) {
-		parent::__construct( $merhant_id, $shop_id );
+	public function __construct($transaction_id, $merhant_id, $shop_id = null) {
+		parent::__construct($merhant_id, $shop_id);
 
-		$this->set_parameter( 'trxid', $transaction_id );
+		$this->set_parameter('trxid', $transaction_id);
 	}
 
 	/**
@@ -41,16 +41,16 @@ 
 	 */
 	public function get_signature_data() {
 		return array(
-			$this->get_parameter( 'trxid' ),
+			$this->get_parameter('trxid'),
 
 			/*
 			 * Indien er geen gebruik wordt gemaakt van de shopid dan kunt u deze weglaten uit de berekening.
 			 */
-			$this->get_parameter( 'shopid' ),
+			$this->get_parameter('shopid'),
 
-			$this->get_parameter( 'merchantid' ),
+			$this->get_parameter('merchantid'),
 
-			$this->get_parameter( 'merchantkey' ),
+			$this->get_parameter('merchantkey'),
 		);
 	}
 }

src/Client.php

Spacing +36 added lines, -36 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|array
 	 */
-	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|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)
 				);
 		}
 	}
@@ -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 ( $xml instanceof WP_Error ) {
+			if ($xml instanceof WP_Error) {
 				$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;
 			}
 		}
@@ -238,32 +238,32 @@ 
 	 * @param StatusRequest $request Status request object.
 	 * @return Transaction|Error|false
 	 */
-	public function get_status( StatusRequest $request ) {
+	public function get_status(StatusRequest $request) {
 		$status = false;
 
 		// Request.
-		$request->sign( $this->merchant_key );
+		$request->sign($this->merchant_key);
 
-		$result = $this->send_request( RequestMethods::STATUS_REQUEST, $request->get_parameters() );
+		$result = $this->send_request(RequestMethods::STATUS_REQUEST, $request->get_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;
 		}