wp-pay-gateways / ing-kassa-compleet

src/PaymentMethods.php

Spacing +2 added lines, -2 removed lines

@@ -87,8 +87,8 @@
 	 *
 	 * @return string
 	 */
-	public static function transform( $payment_method ) {
-		switch ( $payment_method ) {
+	public static function transform($payment_method) {
+		switch ($payment_method) {
 			case Core_PaymentMethods::BANCONTACT:
 				return self::BANCONTACT;
 

src/OrderRequest.php

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
 
 	public function get_array() {
 		$array = array(
-			'amount'       => Core_Util::amount_to_cents( $this->amount ),
+			'amount'       => Core_Util::amount_to_cents($this->amount),
 			'currency'     => $this->currency,
 			'description'  => $this->description,
 			'return_url'   => $this->return_url,
@@ -51,7 +51,7 @@ 
 		);
 
 		// Array filter will remove values NULL, FALSE and empty strings ('')
-		$array = array_filter( $array );
+		$array = array_filter($array);
 
 		// Add payment method
 		$payment_method = array(
@@ -59,7 +59,7 @@ 
 		);
 
 		// Add payment method details
-		switch ( $this->method ) {
+		switch ($this->method) {
 			case PaymentMethods::IDEAL:
 				$payment_method['payment_method_details'] = array(
 					'issuer_id' => $this->issuer,
@@ -68,7 +68,7 @@ 
 				break;
 		}
 
-		$array['transactions'] = array( $payment_method );
+		$array['transactions'] = array($payment_method);
 
 		return $array;
 	}

src/Gateway.php

Spacing +38 added lines, -38 removed lines

@@ -24,19 +24,19 @@ 
 	 *
 	 * @param 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( Gateway::METHOD_HTTP_REDIRECT );
-		$this->set_has_feedback( true );
-		$this->set_amount_minimum( 0.01 );
+		$this->set_method(Gateway::METHOD_HTTP_REDIRECT);
+		$this->set_has_feedback(true);
+		$this->set_amount_minimum(0.01);
 
 		// Client
-		$this->client = new Client( $config->api_key );
+		$this->client = new Client($config->api_key);
 	}
 
 	/**
@@ -49,7 +49,7 @@ 
 
 		$result = $this->client->get_issuers();
 
-		if ( $result ) {
+		if ($result) {
 			$groups[] = array(
 				'options' => $result,
 			);
@@ -57,7 +57,7 @@ 
 
 		$error = $this->client->get_error();
 
-		if ( is_wp_error( $error ) ) {
+		if (is_wp_error($error)) {
 			$this->error = $error;
 		}
 
@@ -67,11 +67,11 @@ 
 	public function get_issuer_field() {
 		$payment_method = $this->get_payment_method();
 
-		if ( null === $payment_method || PaymentMethods::IDEAL === $payment_method ) {
+		if (null === $payment_method || PaymentMethods::IDEAL === $payment_method) {
 			return array(
 				'id'       => 'pronamic_ideal_issuer_id',
 				'name'     => 'pronamic_ideal_issuer_id',
-				'label'    => __( 'Choose your bank', 'pronamic_ideal' ),
+				'label'    => __('Choose your bank', 'pronamic_ideal'),
 				'required' => true,
 				'type'     => 'select',
 				'choices'  => $this->get_transient_issuers(),
@@ -110,7 +110,7 @@ 
 	 *
 	 * @see Pronamic_WP_Pay_Gateway::start()
 	 */
-	public function start( Payment $payment ) {
+	public function start(Payment $payment) {
 		$request = new OrderRequest();
 
 		$request->currency          = $payment->get_currency();
@@ -124,32 +124,32 @@ 
 
 		$payment_method = $payment->get_method();
 
-		if ( empty( $payment_method ) && ! empty( $issuer ) ) {
+		if (empty($payment_method) && ! empty($issuer)) {
 			$payment_method = PaymentMethods::IDEAL;
 		}
 
-		if ( PaymentMethods::IDEAL === $payment_method ) {
+		if (PaymentMethods::IDEAL === $payment_method) {
 			$request->issuer = $issuer;
 		}
 
-		$request->method = Methods::transform( $payment_method );
+		$request->method = Methods::transform($payment_method);
 
-		$order = $this->client->create_order( $request );
+		$order = $this->client->create_order($request);
 
-		if ( $order ) {
-			$payment->set_transaction_id( $order->id );
+		if ($order) {
+			$payment->set_transaction_id($order->id);
 
-			$action_url = add_query_arg( array(
+			$action_url = add_query_arg(array(
 				'payment_redirect' => $payment->get_id(),
 				'key'              => $payment->key,
-			), home_url( '/' ) );
+			), home_url('/'));
 
-			if ( PaymentMethods::BANK_TRANSFER === $payment_method ) {
+			if (PaymentMethods::BANK_TRANSFER === $payment_method) {
 				// Set payment redirect message with received transaction reference.
 				// @see https://s3-eu-west-1.amazonaws.com/wl1-apidocs/api.kassacompleet.nl/index.html#payment-methods-without-the-redirect-flow-performing_redirect-requirement
 				$message = sprintf(
 					/* translators: 1: payment provider name, 2: PSP account name, 3: PSP account number, 4: PSP account BIC, 5: formatted amount, 6: transaction reference */
-					__( 'You have chosen the payment method "Bank transfer". To complete your payment, please transfer the amount to the payment service provider (%1$s).
+					__('You have chosen the payment method "Bank transfer". To complete your payment, please transfer the amount to the payment service provider (%1$s).
 
 <strong>Account holder:</strong> %2$s
 <strong>Account IBAN:</strong> %3$s
@@ -157,8 +157,8 @@ 
 <strong>Amount:</strong> %5$s
 <strong>Transaction reference:</strong> %6$s
 
-<em>Please note: only payments with the mentioned transaction reference can be processed.</em>', 'pronamic_ideal' ),
-					__( 'ING', 'pronamic_ideal' ),
+<em>Please note: only payments with the mentioned transaction reference can be processed.</em>', 'pronamic_ideal'),
+					__('ING', 'pronamic_ideal'),
 					'ING PSP',
 					'NL13INGB0005300060',
 					'INGBNL2A',
@@ -166,19 +166,19 @@ 
 					$order->transactions[0]->payment_method_details->reference
 				);
 
-				$payment->set_meta( 'payment_redirect_message', $message );
+				$payment->set_meta('payment_redirect_message', $message);
 			}
 
-			if ( isset( $order->transactions[0]->payment_url ) ) {
+			if (isset($order->transactions[0]->payment_url)) {
 				$action_url = $order->transactions[0]->payment_url;
 			}
 
-			$payment->set_action_url( $action_url );
+			$payment->set_action_url($action_url);
 		}
 
 		$error = $this->client->get_error();
 
-		if ( is_wp_error( $error ) ) {
+		if (is_wp_error($error)) {
 			$this->error = $error;
 		}
 	}
@@ -188,36 +188,36 @@ 
 	 *
 	 * @param Payment $payment
 	 */
-	public function update_status( Payment $payment ) {
+	public function update_status(Payment $payment) {
 		$transaction_id = $payment->get_transaction_id();
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			return;
 		}
 
-		$order = $this->client->get_order( $transaction_id );
+		$order = $this->client->get_order($transaction_id);
 
-		if ( ! is_object( $order ) ) {
+		if ( ! is_object($order)) {
 			return;
 		}
 
-		$payment->set_status( Statuses::transform( $order->status ) );
+		$payment->set_status(Statuses::transform($order->status));
 
-		if ( isset( $order->transactions[0]->payment_method_details ) ) {
+		if (isset($order->transactions[0]->payment_method_details)) {
 			$details = $order->transactions[0]->payment_method_details;
 
-			if ( isset( $details->consumer_name ) ) {
-				$payment->set_consumer_name( $details->consumer_name );
+			if (isset($details->consumer_name)) {
+				$payment->set_consumer_name($details->consumer_name);
 			}
 
-			if ( isset( $details->consumer_iban ) ) {
-				$payment->set_consumer_iban( $details->consumer_iban );
+			if (isset($details->consumer_iban)) {
+				$payment->set_consumer_iban($details->consumer_iban);
 			}
 		}
 
 		$error = $this->client->get_error();
 
-		if ( is_wp_error( $error ) ) {
+		if (is_wp_error($error)) {
 			$this->error = $error;
 		}
 	}

src/Settings.php

Spacing +26 added lines, -26 removed lines

@@ -16,30 +16,30 @@ 
  */
 class Settings extends GatewaySettings {
 	public function __construct() {
-		add_filter( 'pronamic_pay_gateway_sections', array( $this, 'sections' ) );
-		add_filter( 'pronamic_pay_gateway_fields', array( $this, 'fields' ) );
+		add_filter('pronamic_pay_gateway_sections', array($this, 'sections'));
+		add_filter('pronamic_pay_gateway_fields', array($this, 'fields'));
 	}
 
-	public function sections( array $sections ) {
+	public function sections(array $sections) {
 		// iDEAL
 		$sections['ing_kassa_compleet'] = array(
-			'title'       => __( 'ING Kassa Compleet', 'pronamic_ideal' ),
-			'methods'     => array( 'ing_kassa_compleet' ),
+			'title'       => __('ING Kassa Compleet', 'pronamic_ideal'),
+			'methods'     => array('ing_kassa_compleet'),
 			'description' => sprintf(
 				/* translators: 1: ING */
-				__( 'Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal' ),
-				__( 'ING', 'pronamic_ideal' )
+				__('Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal'),
+				__('ING', 'pronamic_ideal')
 			),
 		);
 
 		// Transaction feedback
 		$sections['ing_kassa_compleet_feedback'] = array(
-			'title'       => __( 'Transaction feedback', 'pronamic_ideal' ),
-			'methods'     => array( 'ing_kassa_compleet' ),
+			'title'       => __('Transaction feedback', 'pronamic_ideal'),
+			'methods'     => array('ing_kassa_compleet'),
 			'description' => sprintf(
 				/* translators: %s: ING Kassa Compleet */
-				__( 'Set the Webhook URL in the %s dashboard to receive automatic transaction status updates.', 'pronamic_ideal' ),
-				__( 'ING Kassa Compleet', 'pronamic_ideal' )
+				__('Set the Webhook URL in the %s dashboard to receive automatic transaction status updates.', 'pronamic_ideal'),
+				__('ING Kassa Compleet', 'pronamic_ideal')
 			),
 		);
 
@@ -47,23 +47,23 @@ 
 		return $sections;
 	}
 
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		// API Key
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ing_kassa_compleet',
 			'meta_key' => '_pronamic_gateway_ing_kassa_compleet_api_key',
-			'title'    => _x( 'API Key', 'ing_kassa_compleet', 'pronamic_ideal' ),
+			'title'    => _x('API Key', 'ing_kassa_compleet', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'regular-text', 'code' ),
-			'methods'  => array( 'ing_kassa_compleet' ),
+			'classes'  => array('regular-text', 'code'),
+			'methods'  => array('ing_kassa_compleet'),
 			'tooltip'  => sprintf(
 				'%s %s.',
-				__( 'API key', 'pronamic_ideal' ),
+				__('API key', 'pronamic_ideal'),
 				sprintf(
 					/* translators: %s: ING Kassa Compleet */
-					__( 'as mentioned in the %s dashboard', 'pronamic_ideal' ),
-					__( 'ING Kassa Compleet', 'pronamic_ideal' )
+					__('as mentioned in the %s dashboard', 'pronamic_ideal'),
+					__('ING Kassa Compleet', 'pronamic_ideal')
 				)
 			),
 		);
@@ -71,27 +71,27 @@ 
 		// Transaction feedback
 		$fields[] = array(
 			'section' => 'ing_kassa_compleet',
-			'methods' => array( 'ing_kassa_compleet' ),
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'methods' => array('ing_kassa_compleet'),
+			'title'   => __('Transaction feedback', 'pronamic_ideal'),
 			'type'    => 'description',
 			'html'    => sprintf(
 				'<span class="dashicons dashicons-warning"></span> %s',
-				__( 'Receiving payment status updates needs additional configuration, if not yet completed.', 'pronamic_ideal' )
+				__('Receiving payment status updates needs additional configuration, if not yet completed.', 'pronamic_ideal')
 			),
 		);
 
 		// Webhook URL
 		$fields[] = array(
 			'section'  => 'ing_kassa_compleet_feedback',
-			'title'    => __( 'Webhook URL', 'pronamic_ideal' ),
+			'title'    => __('Webhook URL', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'large-text', 'code' ),
-			'value'    => add_query_arg( 'ing_kassa_compleet_webhook', '', home_url( '/' ) ),
+			'classes'  => array('large-text', 'code'),
+			'value'    => add_query_arg('ing_kassa_compleet_webhook', '', home_url('/')),
 			'readonly' => true,
 			'tooltip'  => sprintf(
 				/* translators: %s: ING Kassa Compleet */
-				__( 'Copy the Webhook URL to the %s dashboard to receive automatic transaction status updates.', 'pronamic_ideal' ),
-				__( 'ING Kassa Compleet', 'pronamic_ideal' )
+				__('Copy the Webhook URL to the %s dashboard to receive automatic transaction status updates.', 'pronamic_ideal'),
+				__('ING Kassa Compleet', 'pronamic_ideal')
 			),
 		);
 

src/Statuses.php

Spacing +2 added lines, -2 removed lines

@@ -66,8 +66,8 @@
 	 *
 	 * @return string|null
 	 */
-	public static function transform( $status ) {
-		switch ( $status ) {
+	public static function transform($status) {
+		switch ($status) {
 			case self::ERROR:
 				return Core_Statuses::FAILURE;
 

src/Integration.php

Spacing +3 added lines, -3 removed lines

@@ -23,10 +23,10 @@
 		$this->dashboard_url = 'https://portal.kassacompleet.nl/';
 
 		// Actions
-		$function = array( __NAMESPACE__ . '\Listener', 'listen' );
+		$function = array(__NAMESPACE__ . '\Listener', 'listen');
 
-		if ( ! has_action( 'wp_loaded', $function ) ) {
-			add_action( 'wp_loaded', $function );
+		if ( ! has_action('wp_loaded', $function)) {
+			add_action('wp_loaded', $function);
 		}
 	}
 

src/Client.php

Spacing +39 added lines, -39 removed lines

@@ -40,7 +40,7 @@ 
 	/**
 	 * Constructs and initalize an ING Kassa Compleet client object
 	 */
-	public function __construct( $api_key ) {
+	public function __construct($api_key) {
 		$this->api_key = $api_key;
 	}
 
@@ -60,44 +60,44 @@ 
 	 * @param string $method
 	 * @param array $data
 	 */
-	private function send_request( $endpoint, $method = 'POST', array $data = array() ) {
+	private function send_request($endpoint, $method = 'POST', array $data = array()) {
 		$url = self::API_URL . $endpoint;
 
 		$headers = array(
-			'Authorization' => 'Basic ' . base64_encode( $this->api_key . ':' ),
+			'Authorization' => 'Basic ' . base64_encode($this->api_key . ':'),
 		);
 
-		if ( is_array( $data ) && ! empty( $data ) ) {
-			$data = wp_json_encode( $data );
+		if (is_array($data) && ! empty($data)) {
+			$data = wp_json_encode($data);
 
 			$headers['Content-Type'] = 'application/json';
 		}
 
-		$return = wp_remote_request( $url, array(
+		$return = wp_remote_request($url, array(
 			'method'  => $method,
 			'headers' => $headers,
 			'body'    => $data,
-		) );
+		));
 
 		return $return;
 	}
 
-	public function create_order( OrderRequest $request ) {
+	public function create_order(OrderRequest $request) {
 		$result = null;
 
 		$data = $request->get_array();
 
-		$response = $this->send_request( 'orders/', 'POST', $data );
+		$response = $this->send_request('orders/', 'POST', $data);
 
-		$response_code = wp_remote_retrieve_response_code( $response );
+		$response_code = wp_remote_retrieve_response_code($response);
 
-		$body = wp_remote_retrieve_body( $response );
+		$body = wp_remote_retrieve_body($response);
 
 		// NULL is returned if the json cannot be decoded or if the encoded data is deeper than the recursion limit.
-		$ing_result = json_decode( $body );
+		$ing_result = json_decode($body);
 
-		if ( 201 === $response_code ) {
-			if ( $ing_result && 'error' === $ing_result->status ) {
+		if (201 === $response_code) {
+			if ($ing_result && 'error' === $ing_result->status) {
 				$error_msg = $ing_result->transactions[0]->reason;
 				$error     = $ing_result->transactions[0];
 			} else {
@@ -107,36 +107,36 @@ 
 			$error_msg = '';
 			$error     = '';
 
-			if ( $ing_result ) {
+			if ($ing_result) {
 				$error_msg = $ing_result->error->value;
 				$error     = $ing_result->error;
 			}
 
-			if ( 401 === $response_code ) {
+			if (401 === $response_code) {
 				// The default error message for an unauthorized API call does not mention the API key in any way.
 				$error_msg .= ' Please check the API key.';
 			}
 		}
 
-		if ( isset( $error_msg, $error ) ) {
-			$this->error = new WP_Error( 'ing_kassa_compleet_error', $error_msg, $error );
+		if (isset($error_msg, $error)) {
+			$this->error = new WP_Error('ing_kassa_compleet_error', $error_msg, $error);
 		}
 
 		return $result;
 	}
 
-	public function get_order( $order_id ) {
+	public function get_order($order_id) {
 		$result = null;
 
-		$response = $this->send_request( 'orders/' . $order_id . '/', 'GET' );
+		$response = $this->send_request('orders/' . $order_id . '/', 'GET');
 
-		$response_code = wp_remote_retrieve_response_code( $response );
+		$response_code = wp_remote_retrieve_response_code($response);
 
-		if ( 200 === $response_code ) {
-			$body = wp_remote_retrieve_body( $response );
+		if (200 === $response_code) {
+			$body = wp_remote_retrieve_body($response);
 
 			// NULL is returned if the json cannot be decoded or if the encoded data is deeper than the recursion limit.
-			$result = json_decode( $body );
+			$result = json_decode($body);
 		}
 
 		return $result;
@@ -150,42 +150,42 @@ 
 	public function get_issuers() {
 		$issuers = false;
 
-		$response = $this->send_request( 'ideal/issuers/', 'GET' );
+		$response = $this->send_request('ideal/issuers/', 'GET');
 
-		$response_code = wp_remote_retrieve_response_code( $response );
+		$response_code = wp_remote_retrieve_response_code($response);
 
-		if ( 200 === $response_code ) {
-			$body = wp_remote_retrieve_body( $response );
+		if (200 === $response_code) {
+			$body = wp_remote_retrieve_body($response);
 
 			// NULL is returned if the json cannot be decoded or if the encoded data is deeper than the recursion limit.
-			$result = json_decode( $body );
+			$result = json_decode($body);
 
-			if ( null !== $result ) {
+			if (null !== $result) {
 				$issuers = array();
 
-				foreach ( $result as $issuer ) {
-					$id   = Security::filter( $issuer->id );
-					$name = Security::filter( $issuer->name );
+				foreach ($result as $issuer) {
+					$id   = Security::filter($issuer->id);
+					$name = Security::filter($issuer->name);
 
-					$issuers[ $id ] = $name;
+					$issuers[$id] = $name;
 				}
 			}
 		} else {
-			$body = wp_remote_retrieve_body( $response );
+			$body = wp_remote_retrieve_body($response);
 
-			$ing_result = json_decode( $body );
+			$ing_result = json_decode($body);
 
 			$error_msg = $ing_result->error->value;
 
-			if ( 401 === $response_code ) {
+			if (401 === $response_code) {
 				// An unauthorized API call has nothing to do with the browser of the user in our case, remove to prevent confusion.
-				$error_msg = str_replace( "You either supplied the wrong credentials (e.g. a bad password), or your browser doesn't understand how to supply the credentials required.", '', $error_msg );
+				$error_msg = str_replace("You either supplied the wrong credentials (e.g. a bad password), or your browser doesn't understand how to supply the credentials required.", '', $error_msg);
 
 				// The default error message for an unauthorized API call does not mention the API key in any way.
 				$error_msg .= ' Please check the API key.';
 			}
 
-			$this->error = new WP_Error( 'ing_kassa_compleet_error', $error_msg, $ing_result->error );
+			$this->error = new WP_Error('ing_kassa_compleet_error', $error_msg, $ing_result->error);
 		}
 
 		return $issuers;

src/ConfigFactory.php

Spacing +3 added lines, -3 removed lines

@@ -15,11 +15,11 @@
  * @since   1.0.0
  */
 class ConfigFactory extends GatewayConfigFactory {
-	public function get_config( $post_id ) {
+	public function get_config($post_id) {
 		$config = new Config();
 
-		$config->api_key = get_post_meta( $post_id, '_pronamic_gateway_ing_kassa_compleet_api_key', true );
-		$config->mode    = get_post_meta( $post_id, '_pronamic_gateway_mode', true );
+		$config->api_key = get_post_meta($post_id, '_pronamic_gateway_ing_kassa_compleet_api_key', true);
+		$config->mode    = get_post_meta($post_id, '_pronamic_gateway_mode', true);
 
 		return $config;
 	}

src/Listener.php

Spacing +9 added lines, -9 removed lines

@@ -19,30 +19,30 @@
 	 * Listen to ING Kassa Compleet webhook requests.
 	 */
 	public static function listen() {
-		if ( ! filter_has_var( INPUT_GET, 'ing_kassa_compleet_webhook' ) ) {
+		if ( ! filter_has_var(INPUT_GET, 'ing_kassa_compleet_webhook')) {
 			return;
 		}
 
-		$data = json_decode( file_get_contents( 'php://input' ) );
+		$data = json_decode(file_get_contents('php://input'));
 
-		if ( is_object( $data ) && isset( $data->order_id ) ) {
-			$payment = get_pronamic_payment_by_transaction_id( $data->order_id );
+		if (is_object($data) && isset($data->order_id)) {
+			$payment = get_pronamic_payment_by_transaction_id($data->order_id);
 
-			if ( null === $payment ) {
+			if (null === $payment) {
 				return;
 			}
 
 			// Add note.
 			$note = sprintf(
 				/* translators: %s: ING */
-				__( 'Webhook requested by %s.', 'pronamic_ideal' ),
-				__( 'ING', 'pronamic_ideal' )
+				__('Webhook requested by %s.', 'pronamic_ideal'),
+				__('ING', 'pronamic_ideal')
 			);
 
-			$payment->add_note( $note );
+			$payment->add_note($note);
 
 			// Update payment.
-			Plugin::update_payment( $payment, false );
+			Plugin::update_payment($payment, false);
 		}
 	}
 }