Passed
Push — feature/post-pay ( cef525...ced2a4 )
by Remco
03:32
created
src/Client.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 	 * @param string $merchant_id  Merchant ID.
69 69
 	 * @param string $merchant_key Merchant key.
70 70
 	 */
71
-	public function __construct( $merchant_id, $merchant_key ) {
71
+	public function __construct($merchant_id, $merchant_key) {
72 72
 		$this->merchant_id  = $merchant_id;
73 73
 		$this->merchant_key = $merchant_key;
74 74
 	}
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 	 *
88 88
 	 * @param boolean $test_mode True if test mode, false otherwise.
89 89
 	 */
90
-	public function set_test_mode( $test_mode ) {
90
+	public function set_test_mode($test_mode) {
91 91
 		$this->test_mode = $test_mode;
92 92
 	}
93 93
 
@@ -98,11 +98,11 @@  discard block
 block discarded – undo
98 98
 	 * @param Request|null $request Request.
99 99
 	 * @return false|SimpleXMLElement
100 100
 	 */
101
-	private function send_request( $method, Request $request = null ) {
101
+	private function send_request($method, Request $request = null) {
102 102
 		$url = self::API_URL . '/' . $method;
103 103
 
104
-		if ( null !== $request ) {
105
-			$request->sign( $this->merchant_key );
104
+		if (null !== $request) {
105
+			$request->sign($this->merchant_key);
106 106
 		}
107 107
 
108 108
 		$result = Core_Util::remote_get_body(
@@ -110,24 +110,24 @@  discard block
 block discarded – undo
110 110
 			200,
111 111
 			array(
112 112
 				'method' => 'POST',
113
-				'body'   => ( null === $request ) ? null : $request->get_parameters(),
113
+				'body'   => (null === $request) ? null : $request->get_parameters(),
114 114
 			)
115 115
 		);
116 116
 
117
-		if ( $result instanceof WP_Error ) {
117
+		if ($result instanceof WP_Error) {
118 118
 			$this->error = $result;
119 119
 
120 120
 			return false;
121 121
 		}
122 122
 
123
-		if ( ! is_string( $result ) ) {
123
+		if ( ! is_string($result)) {
124 124
 			return false;
125 125
 		}
126 126
 
127 127
 		// XML.
128
-		$xml = Core_Util::simplexml_load_string( $result );
128
+		$xml = Core_Util::simplexml_load_string($result);
129 129
 
130
-		if ( $xml instanceof WP_Error ) {
130
+		if ($xml instanceof WP_Error) {
131 131
 			$this->error = $xml;
132 132
 
133 133
 			return false;
@@ -142,31 +142,31 @@  discard block
 block discarded – undo
142 142
 	 * @param SimpleXMLElement $document Document.
143 143
 	 * @return WP_Error|Transaction|Error
144 144
 	 */
145
-	private function parse_document( SimpleXMLElement $document ) {
145
+	private function parse_document(SimpleXMLElement $document) {
146 146
 		$this->error = null;
147 147
 
148 148
 		$name = $document->getName();
149 149
 
150
-		switch ( $name ) {
150
+		switch ($name) {
151 151
 			case 'errorresponse':
152
-				$sisow_error = ErrorParser::parse( $document->error );
152
+				$sisow_error = ErrorParser::parse($document->error);
153 153
 
154
-				$this->error = new WP_Error( 'ideal_sisow_error', $sisow_error->message, $sisow_error );
154
+				$this->error = new WP_Error('ideal_sisow_error', $sisow_error->message, $sisow_error);
155 155
 
156 156
 				return $sisow_error;
157 157
 			case 'transactionrequest':
158
-				$transaction = TransactionParser::parse( $document->transaction );
158
+				$transaction = TransactionParser::parse($document->transaction);
159 159
 
160 160
 				return $transaction;
161 161
 			case 'statusresponse':
162
-				$transaction = TransactionParser::parse( $document->transaction );
162
+				$transaction = TransactionParser::parse($document->transaction);
163 163
 
164 164
 				return $transaction;
165 165
 			default:
166 166
 				return new WP_Error(
167 167
 					'ideal_sisow_error',
168 168
 					/* translators: %s: XML document element name */
169
-					sprintf( __( 'Unknwon Sisow message (%s)', 'pronamic_ideal' ), $name )
169
+					sprintf(__('Unknwon Sisow message (%s)', 'pronamic_ideal'), $name)
170 170
 				);
171 171
 		}
172 172
 	}
@@ -177,27 +177,27 @@  discard block
 block discarded – undo
177 177
 	 * @return array|false
178 178
 	 */
179 179
 	public function get_directory() {
180
-		if ( $this->test_mode ) {
180
+		if ($this->test_mode) {
181 181
 			return array(
182
-				'99' => __( 'Sisow Bank (test)', 'pronamic_ideal' ),
182
+				'99' => __('Sisow Bank (test)', 'pronamic_ideal'),
183 183
 			);
184 184
 		}
185 185
 
186 186
 		// Request.
187
-		$result = $this->send_request( RequestMethods::DIRECTORY_REQUEST );
187
+		$result = $this->send_request(RequestMethods::DIRECTORY_REQUEST);
188 188
 
189
-		if ( false === $result ) {
189
+		if (false === $result) {
190 190
 			return false;
191 191
 		}
192 192
 
193 193
 		// Parse.
194 194
 		$directory = array();
195 195
 
196
-		foreach ( $result->directory->issuer as $issuer ) {
196
+		foreach ($result->directory->issuer as $issuer) {
197 197
 			$id   = (string) $issuer->issuerid;
198 198
 			$name = (string) $issuer->issuername;
199 199
 
200
-			$directory[ $id ] = $name;
200
+			$directory[$id] = $name;
201 201
 		}
202 202
 
203 203
 		return $directory;
@@ -209,18 +209,18 @@  discard block
 block discarded – undo
209 209
 	 * @param TransactionRequest $request Transaction request.
210 210
 	 * @return Transaction|false
211 211
 	 */
212
-	public function create_transaction( TransactionRequest $request ) {
212
+	public function create_transaction(TransactionRequest $request) {
213 213
 		// Request.
214
-		$response = $this->send_request( RequestMethods::TRANSACTION_REQUEST, $request );
214
+		$response = $this->send_request(RequestMethods::TRANSACTION_REQUEST, $request);
215 215
 
216
-		if ( false === $response ) {
216
+		if (false === $response) {
217 217
 			return false;
218 218
 		}
219 219
 
220 220
 		// Parse.
221
-		$message = $this->parse_document( $response );
221
+		$message = $this->parse_document($response);
222 222
 
223
-		if ( $message instanceof Transaction ) {
223
+		if ($message instanceof Transaction) {
224 224
 			return $message;
225 225
 		}
226 226
 
@@ -233,18 +233,18 @@  discard block
 block discarded – undo
233 233
 	 * @param StatusRequest $request Status request object.
234 234
 	 * @return Transaction|false
235 235
 	 */
236
-	public function get_status( StatusRequest $request ) {
236
+	public function get_status(StatusRequest $request) {
237 237
 		// Request.
238
-		$response = $this->send_request( RequestMethods::STATUS_REQUEST, $request );
238
+		$response = $this->send_request(RequestMethods::STATUS_REQUEST, $request);
239 239
 
240
-		if ( false === $response ) {
240
+		if (false === $response) {
241 241
 			return false;
242 242
 		}
243 243
 
244 244
 		// Parse.
245
-		$message = $this->parse_document( $response );
245
+		$message = $this->parse_document($response);
246 246
 
247
-		if ( $message instanceof Transaction ) {
247
+		if ($message instanceof Transaction) {
248 248
 			return $message;
249 249
 		}
250 250
 
Please login to merge, or discard this patch.