wp-pay-gateways / omnikassa

src/Integration.php

Spacing +5 added lines, -5 removed lines

@@ -20,17 +20,17 @@
 		$this->name          = 'Rabobank - OmniKassa';
 		$this->product_url   = 'https://www.rabobank.nl/bedrijven/betalen/geld-ontvangen/rabo-omnikassa/';
 		$this->dashboard_url = array(
-			__( 'admin', 'pronamic_ideal' )    => 'https://dashboard.omnikassa.rabobank.nl/',
-			__( 'download', 'pronamic_ideal' ) => 'https://download.omnikassa.rabobank.nl/',
+			__('admin', 'pronamic_ideal')    => 'https://dashboard.omnikassa.rabobank.nl/',
+			__('download', 'pronamic_ideal') => 'https://download.omnikassa.rabobank.nl/',
 		);
 		$this->provider      = 'rabobank';
 		$this->deprecated    = true;
 
 		// 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 +37 added lines, -37 removed lines

@@ -57,10 +57,10 @@ 
 		);
 	}
 
-	public static function is_supported_language( $language ) {
+	public static function is_supported_language($language) {
 		$languages = self::get_supported_language_codes();
 
-		return in_array( $language, $languages, true );
+		return in_array($language, $languages, true);
 	}
 
 	/**
@@ -192,7 +192,7 @@ 
 	public function __construct() {
 		$this->payment_mean_brand_list = array();
 
-		$this->set_interface_version( self::INTERFACE_VERSION_HP_1_0 );
+		$this->set_interface_version(self::INTERFACE_VERSION_HP_1_0);
 	}
 
 	/**
@@ -209,7 +209,7 @@ 
 	 *
 	 * @param string $url an URL
 	 */
-	public function set_action_url( $url ) {
+	public function set_action_url($url) {
 		$this->action_url = $url;
 	}
 
@@ -227,7 +227,7 @@ 
 	 *
 	 * @param string $interface_version
 	 */
-	public function set_interface_version( $interface_version ) {
+	public function set_interface_version($interface_version) {
 		$this->interface_version = $interface_version;
 	}
 
@@ -245,7 +245,7 @@ 
 	 *
 	 * @param string $currency_numeric_code
 	 */
-	public function set_currency_numeric_code( $currency_numeric_code ) {
+	public function set_currency_numeric_code($currency_numeric_code) {
 		$this->currency_numeric_code = $currency_numeric_code;
 	}
 
@@ -263,7 +263,7 @@ 
 	 *
 	 * @param string $merchant_id
 	 */
-	public function set_merchant_id( $merchant_id ) {
+	public function set_merchant_id($merchant_id) {
 		$this->merchant_id = $merchant_id;
 	}
 
@@ -283,7 +283,7 @@ 
 	 *
 	 * @param string $normal_return_url
 	 */
-	public function set_normal_return_url( $normal_return_url ) {
+	public function set_normal_return_url($normal_return_url) {
 		$this->normal_return_url = $normal_return_url;
 	}
 
@@ -301,7 +301,7 @@ 
 	 *
 	 * @param float $amount
 	 */
-	public function set_amount( $amount ) {
+	public function set_amount($amount) {
 		$this->amount = $amount;
 	}
 
@@ -320,8 +320,8 @@ 
 	 *
 	 * @param string $transaction_reference
 	 */
-	public function set_transaction_reference( $transaction_reference ) {
-		$this->transaction_reference = DataHelper::filter_an( $transaction_reference, 35 );
+	public function set_transaction_reference($transaction_reference) {
+		$this->transaction_reference = DataHelper::filter_an($transaction_reference, 35);
 	}
 
 	/**
@@ -338,7 +338,7 @@ 
 	 *
 	 * @param string $key_version
 	 */
-	public function set_key_version( $key_version ) {
+	public function set_key_version($key_version) {
 		$this->key_version = $key_version;
 	}
 
@@ -358,7 +358,7 @@ 
 	 *
 	 * @param string $automatic_response_url
 	 */
-	public function set_automatic_response_url( $automatic_response_url ) {
+	public function set_automatic_response_url($automatic_response_url) {
 		$this->automatic_response_url = $automatic_response_url;
 	}
 
@@ -376,7 +376,7 @@ 
 	 *
 	 * @param string $customer_language
 	 */
-	public function set_customer_language( $customer_language ) {
+	public function set_customer_language($customer_language) {
 		$this->customer_language = $customer_language;
 	}
 
@@ -385,7 +385,7 @@ 
 	 *
 	 * @param string $payment_mean_brand
 	 */
-	public function add_payment_mean_brand( $payment_mean_brand ) {
+	public function add_payment_mean_brand($payment_mean_brand) {
 		$this->payment_mean_brand_list[] = $payment_mean_brand;
 	}
 
@@ -395,7 +395,7 @@ 
 	 * @return string ANS128 listString comma separated list
 	 */
 	public function get_payment_mean_brand_list() {
-		return apply_filters( 'pronamic_pay_omnikassa_payment_mean_brand_list', implode( ', ', $this->payment_mean_brand_list ) );
+		return apply_filters('pronamic_pay_omnikassa_payment_mean_brand_list', implode(', ', $this->payment_mean_brand_list));
 	}
 
 	/**
@@ -412,7 +412,7 @@ 
 	 *
 	 * @param string $order_id
 	 */
-	public function set_order_id( $order_id ) {
+	public function set_order_id($order_id) {
 		$this->order_id = $order_id;
 	}
 
@@ -433,8 +433,8 @@ 
 	public function get_formatted_expiration_date() {
 		$result = null;
 
-		if ( null !== $this->expiration_date ) {
-			$result = $this->expiration_date->format( DATE_ISO8601 );
+		if (null !== $this->expiration_date) {
+			$result = $this->expiration_date->format(DATE_ISO8601);
 		}
 
 		return $result;
@@ -445,19 +445,19 @@ 
 	 *
 	 * @param DateTime $date
 	 */
-	public function set_expiration_date( DateTime $date = null ) {
+	public function set_expiration_date(DateTime $date = null) {
 		$this->expiration_date = $date;
 	}
 
 	public function get_data_array() {
 		// Payment Request - required fields
 		$required_fields = array(
-			'amount'               => strval( $this->get_amount() ),
-			'currencyCode'         => strval( $this->get_currency_numeric_code() ),
+			'amount'               => strval($this->get_amount()),
+			'currencyCode'         => strval($this->get_currency_numeric_code()),
 			'merchantId'           => $this->get_merchant_id(),
 			'normalReturnUrl'      => $this->get_normal_return_url(),
 			'transactionReference' => $this->get_transaction_reference(),
-			'keyVersion'           => strval( $this->get_key_version() ),
+			'keyVersion'           => strval($this->get_key_version()),
 		);
 
 		// Payment request - optional fields
@@ -470,7 +470,7 @@ 
 		);
 
 		// @link http://briancray.com/2009/04/25/remove-null-values-php-arrays/
-		$optional_fields = array_filter( $optional_fields );
+		$optional_fields = array_filter($optional_fields);
 
 		// Data
 		$data = $required_fields + $optional_fields;
@@ -486,7 +486,7 @@ 
 	public function get_data() {
 		$data = $this->get_data_array();
 
-		return self::create_piped_string( $data );
+		return self::create_piped_string($data);
 	}
 
 	/**
@@ -503,7 +503,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function set_secret_key( $secret_key ) {
+	public function set_secret_key($secret_key) {
 		$this->secret_key = $secret_key;
 	}
 
@@ -516,7 +516,7 @@ 
 		$data       = $this->get_data();
 		$secret_key = $this->get_secret_key();
 
-		return self::compute_seal( $data, $secret_key );
+		return self::compute_seal($data, $secret_key);
 	}
 
 	/**
@@ -527,11 +527,11 @@ 
 	 *
 	 * @return string seal
 	 */
-	public static function compute_seal( $data, $secret_key ) {
+	public static function compute_seal($data, $secret_key) {
 		$value = $data . $secret_key;
-		$value = utf8_encode( $value );
+		$value = utf8_encode($value);
 
-		return hash( self::HASH_ALGORITHM_SHA256, $value );
+		return hash(self::HASH_ALGORITHM_SHA256, $value);
 	}
 
 	/**
@@ -555,9 +555,9 @@ 
 	 *
 	 * @return string
 	 */
-	public static function create_piped_string( array $data ) {
+	public static function create_piped_string(array $data) {
 		// @link http://core.trac.wordpress.org/browser/tags/3.3.1/wp-includes/functions.php#L1385
-		return _http_build_query( $data, null, '|', '', false );
+		return _http_build_query($data, null, '|', '', false);
 	}
 
 	/**
@@ -567,14 +567,14 @@ 
 	 *
 	 * @return array
 	 */
-	public static function parse_piped_string( $string ) {
+	public static function parse_piped_string($string) {
 		$data = array();
 
-		$pairs = explode( '|', $string );
-		foreach ( $pairs as $pair ) {
-			list( $key, $value ) = explode( '=', $pair );
+		$pairs = explode('|', $string);
+		foreach ($pairs as $pair) {
+			list($key, $value) = explode('=', $pair);
 
-			$data[ $key ] = $value;
+			$data[$key] = $value;
 		}
 
 		return $data;

src/Gateway.php

Spacing +68 added lines, -68 removed lines

@@ -29,24 +29,24 @@ 
 	 *
 	 * @param Config $config Config.
 	 */
-	public function __construct( Config $config ) {
-		parent::__construct( $config );
+	public function __construct(Config $config) {
+		parent::__construct($config);
 
-		$this->set_method( self::METHOD_HTML_FORM );
+		$this->set_method(self::METHOD_HTML_FORM);
 
 		// Client.
 		$this->client = new Client();
 
 		$action_url = Client::ACTION_URL_PRUDCTION;
 
-		if ( self::MODE_TEST === $config->mode ) {
+		if (self::MODE_TEST === $config->mode) {
 			$action_url = Client::ACTION_URL_TEST;
 		}
 
-		$this->client->set_action_url( $action_url );
-		$this->client->set_merchant_id( $config->merchant_id );
-		$this->client->set_key_version( $config->key_version );
-		$this->client->set_secret_key( $config->secret_key );
+		$this->client->set_action_url($action_url);
+		$this->client->set_merchant_id($config->merchant_id);
+		$this->client->set_key_version($config->key_version);
+		$this->client->set_secret_key($config->secret_key);
 	}
 
 	/**
@@ -70,33 +70,33 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
-		$transaction_reference = $payment->get_meta( 'omnikassa_transaction_reference' );
+	public function start(Payment $payment) {
+		$transaction_reference = $payment->get_meta('omnikassa_transaction_reference');
 
-		if ( empty( $transaction_reference ) ) {
-			$transaction_reference = md5( uniqid( '', true ) );
+		if (empty($transaction_reference)) {
+			$transaction_reference = md5(uniqid('', true));
 
-			$payment->set_meta( 'omnikassa_transaction_reference', $transaction_reference );
+			$payment->set_meta('omnikassa_transaction_reference', $transaction_reference);
 		}
 
-		$payment->set_transaction_id( $transaction_reference );
-		$payment->set_action_url( $this->client->get_action_url() );
+		$payment->set_transaction_id($transaction_reference);
+		$payment->set_action_url($this->client->get_action_url());
 
 		$language = null;
 
-		if ( null !== $payment->get_customer() ) {
+		if (null !== $payment->get_customer()) {
 			$language = $payment->get_customer()->get_language();
 		}
 
-		$this->client->set_customer_language( LocaleHelper::transform( $language ) );
-		$this->client->set_currency_numeric_code( $payment->get_total_amount()->get_currency()->get_numeric_code() );
-		$this->client->set_order_id( $payment->format_string( $this->config->order_id ) );
-		$this->client->set_normal_return_url( home_url( '/' ) );
-		$this->client->set_automatic_response_url( home_url( '/' ) );
-		$this->client->set_amount( $payment->get_total_amount()->get_cents() );
-		$this->client->set_transaction_reference( $transaction_reference );
+		$this->client->set_customer_language(LocaleHelper::transform($language));
+		$this->client->set_currency_numeric_code($payment->get_total_amount()->get_currency()->get_numeric_code());
+		$this->client->set_order_id($payment->format_string($this->config->order_id));
+		$this->client->set_normal_return_url(home_url('/'));
+		$this->client->set_automatic_response_url(home_url('/'));
+		$this->client->set_amount($payment->get_total_amount()->get_cents());
+		$this->client->set_transaction_reference($transaction_reference);
 
-		switch ( $payment->get_method() ) {
+		switch ($payment->get_method()) {
 			/*
 			 * If this field is not supplied in the payment request, then
 			 * by default the customer will be redirected to the Rabo
@@ -128,38 +128,38 @@ 
 			 */
 			case PaymentMethods::BANCONTACT:
 			case PaymentMethods::MISTER_CASH:
-				$this->client->add_payment_mean_brand( Methods::BCMC );
+				$this->client->add_payment_mean_brand(Methods::BCMC);
 
 				break;
 			case PaymentMethods::CREDIT_CARD:
-				$this->client->add_payment_mean_brand( Methods::MAESTRO );
-				$this->client->add_payment_mean_brand( Methods::MASTERCARD );
-				$this->client->add_payment_mean_brand( Methods::VISA );
-				$this->client->add_payment_mean_brand( Methods::VPAY );
+				$this->client->add_payment_mean_brand(Methods::MAESTRO);
+				$this->client->add_payment_mean_brand(Methods::MASTERCARD);
+				$this->client->add_payment_mean_brand(Methods::VISA);
+				$this->client->add_payment_mean_brand(Methods::VPAY);
 
 				break;
 			case PaymentMethods::DIRECT_DEBIT:
-				$this->client->add_payment_mean_brand( Methods::INCASSO );
+				$this->client->add_payment_mean_brand(Methods::INCASSO);
 
 				break;
 			case PaymentMethods::MAESTRO:
-				$this->client->add_payment_mean_brand( Methods::MAESTRO );
+				$this->client->add_payment_mean_brand(Methods::MAESTRO);
 
 				break;
 			case PaymentMethods::IDEAL:
-				$this->client->add_payment_mean_brand( Methods::IDEAL );
+				$this->client->add_payment_mean_brand(Methods::IDEAL);
 
 				break;
 			default:
-				$this->client->add_payment_mean_brand( Methods::IDEAL );
-				$this->client->add_payment_mean_brand( Methods::VISA );
-				$this->client->add_payment_mean_brand( Methods::MASTERCARD );
-				$this->client->add_payment_mean_brand( Methods::MAESTRO );
-				$this->client->add_payment_mean_brand( Methods::VPAY );
-				$this->client->add_payment_mean_brand( Methods::BCMC );
-				$this->client->add_payment_mean_brand( Methods::INCASSO );
-				$this->client->add_payment_mean_brand( Methods::ACCEPTGIRO );
-				$this->client->add_payment_mean_brand( Methods::REMBOURS );
+				$this->client->add_payment_mean_brand(Methods::IDEAL);
+				$this->client->add_payment_mean_brand(Methods::VISA);
+				$this->client->add_payment_mean_brand(Methods::MASTERCARD);
+				$this->client->add_payment_mean_brand(Methods::MAESTRO);
+				$this->client->add_payment_mean_brand(Methods::VPAY);
+				$this->client->add_payment_mean_brand(Methods::BCMC);
+				$this->client->add_payment_mean_brand(Methods::INCASSO);
+				$this->client->add_payment_mean_brand(Methods::ACCEPTGIRO);
+				$this->client->add_payment_mean_brand(Methods::REMBOURS);
 
 				break;
 		}
@@ -180,57 +180,57 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
-		$input_data = filter_input( INPUT_POST, 'Data' );
-		$input_seal = filter_input( INPUT_POST, 'Seal', FILTER_SANITIZE_STRING );
+	public function update_status(Payment $payment) {
+		$input_data = filter_input(INPUT_POST, 'Data');
+		$input_seal = filter_input(INPUT_POST, 'Seal', FILTER_SANITIZE_STRING);
 
-		$data = Client::parse_piped_string( $input_data );
+		$data = Client::parse_piped_string($input_data);
 
-		$seal = Client::compute_seal( $input_data, $this->config->secret_key );
+		$seal = Client::compute_seal($input_data, $this->config->secret_key);
 
 		// Check if the posted seal is equal to our seal.
-		if ( 0 === strcasecmp( $input_seal, $seal ) ) {
+		if (0 === strcasecmp($input_seal, $seal)) {
 			$response_code = $data['responseCode'];
 
-			$status = ResponseCodes::transform( $response_code );
+			$status = ResponseCodes::transform($response_code);
 
 			// Set the status of the payment.
-			$payment->set_status( $status );
+			$payment->set_status($status);
 
 			$labels = array(
-				'amount'               => __( 'Amount', 'pronamic_ideal' ),
-				'captureDay'           => _x( 'Capture Day', 'creditcard', 'pronamic_ideal' ),
-				'captureMode'          => _x( 'Capture Mode', 'creditcard', 'pronamic_ideal' ),
-				'currencyCode'         => __( 'Currency Code', 'pronamic_ideal' ),
-				'merchantId'           => __( 'Merchant ID', 'pronamic_ideal' ),
-				'orderId'              => __( 'Order ID', 'pronamic_ideal' ),
-				'transactionDateTime'  => __( 'Transaction Date Time', 'pronamic_ideal' ),
-				'transactionReference' => __( 'Transaction Reference', 'pronamic_ideal' ),
-				'keyVersion'           => __( 'Key Version', 'pronamic_ideal' ),
-				'authorisationId'      => __( 'Authorisation ID', 'pronamic_ideal' ),
-				'paymentMeanBrand'     => __( 'Payment Mean Brand', 'pronamic_ideal' ),
-				'paymentMeanType'      => __( 'Payment Mean Type', 'pronamic_ideal' ),
-				'responseCode'         => __( 'Response Code', 'pronamic_ideal' ),
+				'amount'               => __('Amount', 'pronamic_ideal'),
+				'captureDay'           => _x('Capture Day', 'creditcard', 'pronamic_ideal'),
+				'captureMode'          => _x('Capture Mode', 'creditcard', 'pronamic_ideal'),
+				'currencyCode'         => __('Currency Code', 'pronamic_ideal'),
+				'merchantId'           => __('Merchant ID', 'pronamic_ideal'),
+				'orderId'              => __('Order ID', 'pronamic_ideal'),
+				'transactionDateTime'  => __('Transaction Date Time', 'pronamic_ideal'),
+				'transactionReference' => __('Transaction Reference', 'pronamic_ideal'),
+				'keyVersion'           => __('Key Version', 'pronamic_ideal'),
+				'authorisationId'      => __('Authorisation ID', 'pronamic_ideal'),
+				'paymentMeanBrand'     => __('Payment Mean Brand', 'pronamic_ideal'),
+				'paymentMeanType'      => __('Payment Mean Type', 'pronamic_ideal'),
+				'responseCode'         => __('Response Code', 'pronamic_ideal'),
 			);
 
 			$note = '';
 
 			$note .= '<p>';
-			$note .= __( 'OmniKassa transaction data in response message:', 'pronamic_ideal' );
+			$note .= __('OmniKassa transaction data in response message:', 'pronamic_ideal');
 			$note .= '</p>';
 
 			$note .= '<dl>';
 
-			foreach ( $labels as $key => $label ) {
-				if ( isset( $data[ $key ] ) ) {
-					$note .= sprintf( '<dt>%s</dt>', esc_html( $label ) );
-					$note .= sprintf( '<dd>%s</dd>', esc_html( $data[ $key ] ) );
+			foreach ($labels as $key => $label) {
+				if (isset($data[$key])) {
+					$note .= sprintf('<dt>%s</dt>', esc_html($label));
+					$note .= sprintf('<dd>%s</dd>', esc_html($data[$key]));
 				}
 			}
 
 			$note .= '</dl>';
 
-			$payment->add_note( $note );
+			$payment->add_note($note);
 		}
 	}
 }

src/LocaleHelper.php

Spacing +5 added lines, -5 removed lines

@@ -20,8 +20,8 @@ 
 	 *
 	 * @return string|null
 	 */
-	public static function transform( $language ) {
-		if ( ! is_string( $language ) ) {
+	public static function transform($language) {
+		if ( ! is_string($language)) {
 			return null;
 		}
 
@@ -38,13 +38,13 @@ 
 		);
 
 		// Sub string.
-		$locale = substr( $language, 0, 2 );
+		$locale = substr($language, 0, 2);
 
 		// Upper case.
-		$locale = strtoupper( $locale );
+		$locale = strtoupper($locale);
 
 		// Is supported?
-		if ( in_array( $locale, $supported, true ) ) {
+		if (in_array($locale, $supported, true)) {
 			return $locale;
 		}