Passed
Push — feature/post-pay ( cf1132...b725c4 )
by Remco
03:48
created
src/Client.php 1 patch
Spacing   +32 added lines, -32 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,7 +98,7 @@  discard block
 block discarded – undo
98 98
 	 * @param array  $parameters Parameters.
99 99
 	 * @return false|SimpleXMLElement
100 100
 	 */
101
-	private function send_request( $method, array $parameters = array() ) {
101
+	private function send_request($method, array $parameters = array()) {
102 102
 		$url = self::API_URL . '/' . $method;
103 103
 
104 104
 		$result = Core_Util::remote_get_body(
@@ -110,20 +110,20 @@  discard block
 block discarded – undo
110 110
 			)
111 111
 		);
112 112
 
113
-		if ( $result instanceof WP_Error ) {
113
+		if ($result instanceof WP_Error) {
114 114
 			$this->error = $result;
115 115
 
116 116
 			return false;
117 117
 		}
118 118
 
119
-		if ( ! is_string( $result ) ) {
119
+		if ( ! is_string($result)) {
120 120
 			return false;
121 121
 		}
122 122
 
123 123
 		// XML.
124
-		$xml = Core_Util::simplexml_load_string( $result );
124
+		$xml = Core_Util::simplexml_load_string($result);
125 125
 
126
-		if ( $xml instanceof WP_Error ) {
126
+		if ($xml instanceof WP_Error) {
127 127
 			$this->error = $xml;
128 128
 
129 129
 			return false;
@@ -138,31 +138,31 @@  discard block
 block discarded – undo
138 138
 	 * @param SimpleXMLElement $document Document.
139 139
 	 * @return WP_Error|Transaction|Error
140 140
 	 */
141
-	private function parse_document( SimpleXMLElement $document ) {
141
+	private function parse_document(SimpleXMLElement $document) {
142 142
 		$this->error = null;
143 143
 
144 144
 		$name = $document->getName();
145 145
 
146
-		switch ( $name ) {
146
+		switch ($name) {
147 147
 			case 'errorresponse':
148
-				$sisow_error = ErrorParser::parse( $document->error );
148
+				$sisow_error = ErrorParser::parse($document->error);
149 149
 
150
-				$this->error = new WP_Error( 'ideal_sisow_error', $sisow_error->message, $sisow_error );
150
+				$this->error = new WP_Error('ideal_sisow_error', $sisow_error->message, $sisow_error);
151 151
 
152 152
 				return $sisow_error;
153 153
 			case 'transactionrequest':
154
-				$transaction = TransactionParser::parse( $document->transaction );
154
+				$transaction = TransactionParser::parse($document->transaction);
155 155
 
156 156
 				return $transaction;
157 157
 			case 'statusresponse':
158
-				$transaction = TransactionParser::parse( $document->transaction );
158
+				$transaction = TransactionParser::parse($document->transaction);
159 159
 
160 160
 				return $transaction;
161 161
 			default:
162 162
 				return new WP_Error(
163 163
 					'ideal_sisow_error',
164 164
 					/* translators: %s: XML document element name */
165
-					sprintf( __( 'Unknwon Sisow message (%s)', 'pronamic_ideal' ), $name )
165
+					sprintf(__('Unknwon Sisow message (%s)', 'pronamic_ideal'), $name)
166 166
 				);
167 167
 		}
168 168
 	}
@@ -173,27 +173,27 @@  discard block
 block discarded – undo
173 173
 	 * @return array<int|string, string>|false
174 174
 	 */
175 175
 	public function get_directory() {
176
-		if ( $this->test_mode ) {
176
+		if ($this->test_mode) {
177 177
 			$return = array(
178
-				'99' => __( 'Sisow Bank (test)', 'pronamic_ideal' ),
178
+				'99' => __('Sisow Bank (test)', 'pronamic_ideal'),
179 179
 			);
180 180
 		}
181 181
 
182 182
 		// Request.
183
-		$result = $this->send_request( RequestMethods::DIRECTORY_REQUEST );
183
+		$result = $this->send_request(RequestMethods::DIRECTORY_REQUEST);
184 184
 
185
-		if ( false === $result ) {
185
+		if (false === $result) {
186 186
 			return false;
187 187
 		}
188 188
 
189 189
 		// Parse.
190 190
 		$directory = array();
191 191
 
192
-		foreach ( $result->directory->issuer as $issuer ) {
192
+		foreach ($result->directory->issuer as $issuer) {
193 193
 			$id   = (string) $issuer->issuerid;
194 194
 			$name = (string) $issuer->issuername;
195 195
 
196
-			$directory[ $id ] = $name;
196
+			$directory[$id] = $name;
197 197
 		}
198 198
 
199 199
 		return $directory;
@@ -205,22 +205,22 @@  discard block
 block discarded – undo
205 205
 	 * @param TransactionRequest $request Transaction request.
206 206
 	 * @return Transaction|false
207 207
 	 */
208
-	public function create_transaction( TransactionRequest $request ) {
208
+	public function create_transaction(TransactionRequest $request) {
209 209
 		$result = false;
210 210
 
211 211
 		// Request.
212
-		$request->sign( $this->merchant_key );
212
+		$request->sign($this->merchant_key);
213 213
 
214
-		$response = $this->send_request( RequestMethods::TRANSACTION_REQUEST, $request->get_parameters() );
214
+		$response = $this->send_request(RequestMethods::TRANSACTION_REQUEST, $request->get_parameters());
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,22 +233,22 @@  discard block
 block discarded – undo
233 233
 	 * @param StatusRequest $request Status request object.
234 234
 	 * @return Transaction|Error|WP_Error|false
235 235
 	 */
236
-	public function get_status( StatusRequest $request ) {
236
+	public function get_status(StatusRequest $request) {
237 237
 		$status = false;
238 238
 
239 239
 		// Request.
240
-		$request->sign( $this->merchant_key );
240
+		$request->sign($this->merchant_key);
241 241
 
242
-		$response = $this->send_request( RequestMethods::STATUS_REQUEST, $request->get_parameters() );
242
+		$response = $this->send_request(RequestMethods::STATUS_REQUEST, $request->get_parameters());
243 243
 
244
-		if ( false === $response ) {
244
+		if (false === $response) {
245 245
 			return false;
246 246
 		}
247 247
 
248 248
 		// Parse.
249
-		$message = $this->parse_document( $response );
249
+		$message = $this->parse_document($response);
250 250
 
251
-		if ( $message instanceof Transaction ) {
251
+		if ($message instanceof Transaction) {
252 252
 			return $message;
253 253
 		}
254 254
 
Please login to merge, or discard this patch.