wp-pay-gateways / ideal-basic

src/Listener.php

Spacing +10 added lines, -10 removed lines

@@ -17,33 +17,33 @@
 class Listener {
 	public static function listen() {
 		// Also check for typo 'xml_notifaction', as this has been used in the past.
-		if ( ! filter_has_var( INPUT_GET, 'xml_notification' ) && ! filter_has_var( INPUT_GET, 'xml_notifaction' ) ) {
+		if ( ! filter_has_var(INPUT_GET, 'xml_notification') && ! filter_has_var(INPUT_GET, 'xml_notifaction')) {
 			return;
 		}
 
 		$notification = Util::get_notification();
 
-		if ( ! $notification ) {
+		if ( ! $notification) {
 			return;
 		}
 
-		$payment = get_pronamic_payment_by_purchase_id( $notification->get_purchase_id() );
+		$payment = get_pronamic_payment_by_purchase_id($notification->get_purchase_id());
 
-		if ( null !== $payment ) {
+		if (null !== $payment) {
 			// Add note.
 			$note = sprintf(
 				/* translators: %s: iDEAL Basic */
-				__( 'Webhook requested by %s.', 'pronamic_ideal' ),
-				__( 'iDEAL Basic', 'pronamic_ideal' )
+				__('Webhook requested by %s.', 'pronamic_ideal'),
+				__('iDEAL Basic', 'pronamic_ideal')
 			);
 
-			$payment->add_note( $note );
+			$payment->add_note($note);
 
-			$payment->set_status( $notification->get_status() );
-			$payment->set_transaction_id( $notification->get_transaction_id() );
+			$payment->set_status($notification->get_status());
+			$payment->set_transaction_id($notification->get_transaction_id());
 
 			// Update payment.
-			Plugin::update_payment( $payment );
+			Plugin::update_payment($payment);
 		}
 	}
 }

src/Client.php

Spacing +47 added lines, -47 removed lines

@@ -176,10 +176,10 @@ 
 
 		$this->forbidden_characters = array();
 
-		$this->set_payment_type( self::PAYMENT_TYPE_IDEAL );
-		$this->set_expire_date_format( self::DATE_EXPIRE_FORMAT );
-		$this->set_expire_date_modifier( self::EXPIRE_DATE_MODIFIER );
-		$this->set_forbidden_characters( self::FORBIDDEN_CHARACHTERS );
+		$this->set_payment_type(self::PAYMENT_TYPE_IDEAL);
+		$this->set_expire_date_format(self::DATE_EXPIRE_FORMAT);
+		$this->set_expire_date_modifier(self::EXPIRE_DATE_MODIFIER);
+		$this->set_forbidden_characters(self::FORBIDDEN_CHARACHTERS);
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	 *
 	 * @param string $url Payment server URL.
 	 */
-	public function set_payment_server_url( $url ) {
+	public function set_payment_server_url($url) {
 		$this->payment_server_url = $url;
 	}
 
@@ -214,7 +214,7 @@ 
 	 *
 	 * @param string $merchant_id Merchant ID.
 	 */
-	public function set_merchant_id( $merchant_id ) {
+	public function set_merchant_id($merchant_id) {
 		$this->merchant_id = $merchant_id;
 	}
 
@@ -232,7 +232,7 @@ 
 	 *
 	 * @param string $sub_id Sub ID.
 	 */
-	public function set_sub_id( $sub_id ) {
+	public function set_sub_id($sub_id) {
 		$this->sub_id = $sub_id;
 	}
 
@@ -251,7 +251,7 @@ 
 	 *
 	 * @param string $hash_key Hash key.
 	 */
-	public function set_hash_key( $hash_key ) {
+	public function set_hash_key($hash_key) {
 		$this->hash_key = $hash_key;
 	}
 
@@ -270,8 +270,8 @@ 
 	 *
 	 * @param string $purchase_id Purchase ID.
 	 */
-	public function set_purchase_id( $purchase_id ) {
-		$this->purchase_id = substr( $purchase_id, 0, 16 );
+	public function set_purchase_id($purchase_id) {
+		$this->purchase_id = substr($purchase_id, 0, 16);
 	}
 
 	/**
@@ -288,7 +288,7 @@ 
 	 *
 	 * @param string $language Language.
 	 */
-	public function set_language( $language ) {
+	public function set_language($language) {
 		$this->language = $language;
 	}
 
@@ -307,8 +307,8 @@ 
 	 *
 	 * @param string $description Description.
 	 */
-	public function set_description( $description ) {
-		$this->description = DataHelper::an32( $description );
+	public function set_description($description) {
+		$this->description = DataHelper::an32($description);
 	}
 
 	/**
@@ -325,7 +325,7 @@ 
 	 *
 	 * @param string $currency Currency.
 	 */
-	public function set_currency( $currency ) {
+	public function set_currency($currency) {
 		$this->currency = $currency;
 	}
 
@@ -344,7 +344,7 @@ 
 	 *
 	 * @param string $payment_type Payment type.
 	 */
-	public function set_payment_type( $payment_type ) {
+	public function set_payment_type($payment_type) {
 		$this->payment_type = $payment_type;
 	}
 
@@ -355,10 +355,10 @@ 
 	 *
 	 * @return DateTime
 	 */
-	public function get_expire_date( $create_new = false ) {
-		if ( null === $this->expire_date || $create_new ) {
-			$this->expire_date = new DateTime( null, new DateTimeZone( Plugin::TIMEZONE ) );
-			$this->expire_date->modify( $this->expire_date_modifier );
+	public function get_expire_date($create_new = false) {
+		if (null === $this->expire_date || $create_new) {
+			$this->expire_date = new DateTime(null, new DateTimeZone(Plugin::TIMEZONE));
+			$this->expire_date->modify($this->expire_date_modifier);
 		}
 
 		return $this->expire_date;
@@ -378,7 +378,7 @@ 
 	 *
 	 * @param string $expire_date_format Expire date format.
 	 */
-	public function set_expire_date_format( $expire_date_format ) {
+	public function set_expire_date_format($expire_date_format) {
 		$this->expire_date_format = $expire_date_format;
 	}
 
@@ -396,7 +396,7 @@ 
 	 *
 	 * @param string $expire_date_modifier Expire date modifier.
 	 */
-	public function set_expire_date_modifier( $expire_date_modifier ) {
+	public function set_expire_date_modifier($expire_date_modifier) {
 		$this->expire_date_modifier = $expire_date_modifier;
 	}
 
@@ -405,7 +405,7 @@ 
 	 *
 	 * @param DateTime $date Expire date.
 	 */
-	public function set_expire_date( DateTime $date ) {
+	public function set_expire_date(DateTime $date) {
 		$this->expire_date = $date;
 	}
 
@@ -425,13 +425,13 @@ 
 	 *
 	 * @throws InvalidArgumentException Passed characters is not an array or string.
 	 */
-	public function set_forbidden_characters( $characters ) {
-		if ( ! is_array( $characters ) && ! is_string( $characters ) ) {
-			throw new InvalidArgumentException( 'Invalid characters argument.' );
+	public function set_forbidden_characters($characters) {
+		if ( ! is_array($characters) && ! is_string($characters)) {
+			throw new InvalidArgumentException('Invalid characters argument.');
 		}
 
-		if ( is_string( $characters ) ) {
-			$characters = str_split( $characters );
+		if (is_string($characters)) {
+			$characters = str_split($characters);
 		}
 
 		$this->forbidden_characters = $characters;
@@ -451,7 +451,7 @@ 
 	 *
 	 * @param string $url Success URL.
 	 */
-	public function set_success_url( $url ) {
+	public function set_success_url($url) {
 		$this->success_url = $url;
 	}
 
@@ -469,7 +469,7 @@ 
 	 *
 	 * @param string $url Cancel URL.
 	 */
-	public function set_cancel_url( $url ) {
+	public function set_cancel_url($url) {
 		$this->cancel_url = $url;
 	}
 
@@ -487,7 +487,7 @@ 
 	 *
 	 * @param string $url Error URL.
 	 */
-	public function set_error_url( $url ) {
+	public function set_error_url($url) {
 		$this->error_url = $url;
 	}
 
@@ -505,7 +505,7 @@ 
 	 *
 	 * @param Items $items Items.
 	 */
-	public function set_items( Items $items ) {
+	public function set_items(Items $items) {
 		$this->items = $items;
 	}
 
@@ -534,7 +534,7 @@ 
 		$string[] = $this->get_sub_id();
 
 		// Total amount of transaction.
-		$string[] = intval( $this->get_amount()->get_cents() );
+		$string[] = intval($this->get_amount()->get_cents());
 
 		// The online shop's unique order number, also known as purchase id.
 		$string[] = $this->get_purchase_id();
@@ -544,10 +544,10 @@ 
 
 		// yyyy-MMddTHH:mm:ss.SSS Z Time at which the transaction expires (maximum of 1 hour later).
 		// The consumer has time until then to pay with iDEAL.
-		$string[] = $this->get_expire_date()->format( $this->get_expire_date_format() );
+		$string[] = $this->get_expire_date()->format($this->get_expire_date_format());
 
 		// Iterate through the items and concat.
-		foreach ( $this->get_items() as $item ) {
+		foreach ($this->get_items() as $item) {
 			// Article number. <n> is 1 for the first product, 2 for the second, etc.
 			// N.B. Note that for every product type the parameters
 			// itemNumber<n>, itemDescription<n>, itemQuantity<n> and itemPrice<n> are mandatory.
@@ -560,17 +560,17 @@ 
 			$string[] = $item->get_quantity();
 
 			// Price of article <n> in whole eurocents.
-			$string[] = intval( $item->get_price()->get_cents() );
+			$string[] = intval($item->get_price()->get_cents());
 		}
 
-		$concat_string = implode( '', $string );
+		$concat_string = implode('', $string);
 
 		// The characters "\t", "\n", "\r", " " (spaces) may not exist in the string.
 		$forbidden_characters = $this->get_forbidden_characters();
-		$concat_string        = str_replace( $forbidden_characters, '', $concat_string );
+		$concat_string        = str_replace($forbidden_characters, '', $concat_string);
 
 		// Delete special HTML entities.
-		$concat_string = html_entity_decode( $concat_string, ENT_COMPAT, 'UTF-8' );
+		$concat_string = html_entity_decode($concat_string, ENT_COMPAT, 'UTF-8');
 
 		return $concat_string;
 	}
@@ -581,7 +581,7 @@ 
 	 * @return string Hash
 	 */
 	public function create_hash() {
-		return sha1( $this->create_hash_string() );
+		return sha1($this->create_hash_string());
 	}
 
 	/**
@@ -596,23 +596,23 @@ 
 		$fields['merchantID'] = $this->get_merchant_id();
 		$fields['subID']      = $this->get_sub_id();
 
-		$fields['amount']      = intval( $this->get_amount()->get_cents() );
+		$fields['amount']      = intval($this->get_amount()->get_cents());
 		$fields['purchaseID']  = $this->get_purchase_id();
 		$fields['language']    = $this->get_language();
 		$fields['currency']    = $this->get_currency();
 		$fields['description'] = $this->get_description();
 		$fields['hash']        = $this->create_hash();
 		$fields['paymentType'] = $this->get_payment_type();
-		$fields['validUntil']  = $this->get_expire_date()->format( $this->get_expire_date_format() );
+		$fields['validUntil']  = $this->get_expire_date()->format($this->get_expire_date_format());
 
 		$serial_number = 1;
-		foreach ( $this->get_items() as $item ) {
-			$fields[ 'itemNumber' . $serial_number ]      = $item->get_number();
-			$fields[ 'itemDescription' . $serial_number ] = $item->get_description();
-			$fields[ 'itemQuantity' . $serial_number ]    = $item->get_quantity();
-			$fields[ 'itemPrice' . $serial_number ]       = intval( $item->get_price()->get_cents() );
+		foreach ($this->get_items() as $item) {
+			$fields['itemNumber' . $serial_number]      = $item->get_number();
+			$fields['itemDescription' . $serial_number] = $item->get_description();
+			$fields['itemQuantity' . $serial_number]    = $item->get_quantity();
+			$fields['itemPrice' . $serial_number]       = intval($item->get_price()->get_cents());
 
-			$serial_number ++;
+			$serial_number++;
 		}
 
 		$fields['urlCancel']  = $this->get_cancel_url();

src/Items.php

Spacing +8 added lines, -8 removed lines

@@ -40,7 +40,7 @@ 
 	// Function name "getIterator" is in camel caps format, try 'get_iterator'
 	public function getIterator() {
 		// @codingStandardsIgnoreEnd
-		return new ArrayIterator( $this->items );
+		return new ArrayIterator($this->items);
 	}
 
 	/**
@@ -48,7 +48,7 @@ 
 	 *
 	 * @param Item $item Item.
 	 */
-	public function add_item( Item $item ) {
+	public function add_item(Item $item) {
 		$this->items[] = $item;
 	}
 
@@ -58,20 +58,20 @@ 
 	public function get_amount() {
 		$amount = 0;
 
-		$use_bcmath = extension_loaded( 'bcmath' );
+		$use_bcmath = extension_loaded('bcmath');
 
-		foreach ( $this->items as $item ) {
-			if ( $use_bcmath ) {
+		foreach ($this->items as $item) {
+			if ($use_bcmath) {
 				// Use non-locale aware float value.
 				// @link http://php.net/sprintf.
-				$item_amount = sprintf( '%F', $item->get_amount() );
+				$item_amount = sprintf('%F', $item->get_amount());
 
-				$amount = bcadd( $amount, $item_amount, 8 );
+				$amount = bcadd($amount, $item_amount, 8);
 			} else {
 				$amount += $item->get_amount();
 			}
 		}
 
-		return new Money( $amount );
+		return new Money($amount);
 	}
 }

src/ConfigFactory.php

Spacing +10 added lines, -10 removed lines

@@ -17,22 +17,22 @@
 class ConfigFactory extends GatewayConfigFactory {
 	private $config_class;
 
-	public function __construct( $config_class = null, $config_test_class = null ) {
-		$this->config_class      = is_null( $config_class ) ? __NAMESPACE__ . '\Config' : $config_class;
-		$this->config_test_class = is_null( $config_test_class ) ? __NAMESPACE__ . '\Config' : $config_test_class;
+	public function __construct($config_class = null, $config_test_class = null) {
+		$this->config_class      = is_null($config_class) ? __NAMESPACE__ . '\Config' : $config_class;
+		$this->config_test_class = is_null($config_test_class) ? __NAMESPACE__ . '\Config' : $config_test_class;
 	}
 
-	public function get_config( $post_id ) {
-		$mode = get_post_meta( $post_id, '_pronamic_gateway_mode', true );
+	public function get_config($post_id) {
+		$mode = get_post_meta($post_id, '_pronamic_gateway_mode', true);
 
-		$config_class = ( Gateway::MODE_TEST === $mode ) ? $this->config_test_class : $this->config_class;
+		$config_class = (Gateway::MODE_TEST === $mode) ? $this->config_test_class : $this->config_class;
 
 		$config = new $config_class();
 
-		$config->merchant_id = get_post_meta( $post_id, '_pronamic_gateway_ideal_merchant_id', true );
-		$config->sub_id      = get_post_meta( $post_id, '_pronamic_gateway_ideal_sub_id', true );
-		$config->hash_key    = get_post_meta( $post_id, '_pronamic_gateway_ideal_hash_key', true );
-		$config->purchase_id = get_post_meta( $post_id, '_pronamic_gateway_ideal_purchase_id', true );
+		$config->merchant_id = get_post_meta($post_id, '_pronamic_gateway_ideal_merchant_id', true);
+		$config->sub_id      = get_post_meta($post_id, '_pronamic_gateway_ideal_sub_id', true);
+		$config->hash_key    = get_post_meta($post_id, '_pronamic_gateway_ideal_hash_key', true);
+		$config->purchase_id = get_post_meta($post_id, '_pronamic_gateway_ideal_purchase_id', true);
 
 		return $config;
 	}

src/Gateway.php

Spacing +25 added lines, -25 removed lines

@@ -30,17 +30,17 @@ 
 	 *
 	 * @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);
 
 		$this->client = new Client();
 
-		$this->client->set_payment_server_url( $config->get_payment_server_url() );
-		$this->client->set_merchant_id( $config->merchant_id );
-		$this->client->set_sub_id( $config->sub_id );
-		$this->client->set_hash_key( $config->hash_key );
+		$this->client->set_payment_server_url($config->get_payment_server_url());
+		$this->client->set_merchant_id($config->merchant_id);
+		$this->client->set_sub_id($config->sub_id);
+		$this->client->set_hash_key($config->hash_key);
 	}
 
 	/**
@@ -70,34 +70,34 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
-		$payment->set_action_url( $this->client->get_payment_server_url() );
+	public function start(Payment $payment) {
+		$payment->set_action_url($this->client->get_payment_server_url());
 
 		// Purchase ID.
-		$purchase_id = $payment->format_string( $this->config->purchase_id );
+		$purchase_id = $payment->format_string($this->config->purchase_id);
 
-		$payment->set_meta( 'purchase_id', $purchase_id );
+		$payment->set_meta('purchase_id', $purchase_id);
 
 		// General.
-		$this->client->set_currency( $payment->get_total_amount()->get_currency()->get_alphabetic_code() );
-		$this->client->set_purchase_id( $purchase_id );
-		$this->client->set_description( $payment->get_description() );
+		$this->client->set_currency($payment->get_total_amount()->get_currency()->get_alphabetic_code());
+		$this->client->set_purchase_id($purchase_id);
+		$this->client->set_description($payment->get_description());
 
-		if ( null !== $payment->get_customer() ) {
-			$this->client->set_language( $payment->get_customer()->get_language() );
+		if (null !== $payment->get_customer()) {
+			$this->client->set_language($payment->get_customer()->get_language());
 		}
 
 		// Items.
 		$items = new Items();
 
-		$items->add_item( new Item( 1, $payment->get_description(), 1, $payment->get_total_amount() ) );
+		$items->add_item(new Item(1, $payment->get_description(), 1, $payment->get_total_amount()));
 
-		$this->client->set_items( $items );
+		$this->client->set_items($items);
 
 		// URLs.
-		$this->client->set_cancel_url( add_query_arg( 'status', Statuses::CANCELLED, $payment->get_return_url() ) );
-		$this->client->set_success_url( add_query_arg( 'status', Statuses::SUCCESS, $payment->get_return_url() ) );
-		$this->client->set_error_url( add_query_arg( 'status', Statuses::FAILURE, $payment->get_return_url() ) );
+		$this->client->set_cancel_url(add_query_arg('status', Statuses::CANCELLED, $payment->get_return_url()));
+		$this->client->set_success_url(add_query_arg('status', Statuses::SUCCESS, $payment->get_return_url()));
+		$this->client->set_error_url(add_query_arg('status', Statuses::FAILURE, $payment->get_return_url()));
 	}
 
 	/**
@@ -105,14 +105,14 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
-		if ( ! filter_has_var( INPUT_GET, 'status' ) ) {
+	public function update_status(Payment $payment) {
+		if ( ! filter_has_var(INPUT_GET, 'status')) {
 			return;
 		}
 
-		$status = filter_input( INPUT_GET, 'status', FILTER_SANITIZE_STRING );
+		$status = filter_input(INPUT_GET, 'status', FILTER_SANITIZE_STRING);
 
 		// Update payment status.
-		$payment->set_status( $status );
+		$payment->set_status($status);
 	}
 }