wp-pay-gateways / ideal-basic

src/Item.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 	/**
 	 * Constructs and initialize a iDEAL basic item
 	 */
-	public function __construct( $number, $description, $quantity, $price ) {
+	public function __construct($number, $description, $quantity, $price) {
 		$this->number      = $number;
 		$this->description = $description;
 		$this->quantity    = $quantity;

src/Items.php

Spacing +4 added lines, -4 removed lines

@@ -40,13 +40,13 @@ 
 	// 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);
 	}
 
 	/**
 	 * Add item
 	 */
-	public function add_item( Item $item ) {
+	public function add_item(Item $item) {
 		$this->items[] = $item;
 	}
 
@@ -56,10 +56,10 @@ 
 	public function get_amount() {
 		$amount = 0;
 
-		foreach ( $this->items as $item ) {
+		foreach ($this->items as $item) {
 			$amount += $item->get_amount();
 		}
 
-		return new Money( $amount );
+		return new Money($amount);
 	}
 }

src/Gateway.php

Spacing +29 added lines, -29 removed lines

@@ -23,19 +23,19 @@ 
 	 *
 	 * @param Config $config
 	 */
-	public function __construct( Config $config ) {
-		parent::__construct( $config );
+	public function __construct(Config $config) {
+		parent::__construct($config);
 
-		$this->set_method( Gateway::METHOD_HTML_FORM );
-		$this->set_has_feedback( false );
-		$this->set_amount_minimum( 0.01 );
+		$this->set_method(Gateway::METHOD_HTML_FORM);
+		$this->set_has_feedback(false);
+		$this->set_amount_minimum(0.01);
 
 		$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);
 	}
 
 	/**
@@ -65,31 +65,31 @@ 
 	 *
 	 * @see Pronamic_WP_Pay_Gateway::start()
 	 */
-	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_language( $payment->get_language() );
-		$this->client->set_currency( $payment->get_currency() );
-		$this->client->set_purchase_id( $purchase_id );
-		$this->client->set_description( $payment->get_description() );
+		$this->client->set_language($payment->get_language());
+		$this->client->set_currency($payment->get_currency());
+		$this->client->set_purchase_id($purchase_id);
+		$this->client->set_description($payment->get_description());
 
 		// Items
 		$items = new Items();
 
-		$items->add_item( new Item( 1, $payment->get_description(), 1, $payment->get_amount()->get_amount() ) );
+		$items->add_item(new Item(1, $payment->get_description(), 1, $payment->get_amount()->get_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()));
 	}
 
 	/**
@@ -97,23 +97,23 @@ 
 	 *
 	 * @param Payment $payment
 	 */
-	public function update_status( Payment $payment ) {
+	public function update_status(Payment $payment) {
 		$status = null;
 
 		// Handle XML notification
 		$notification = Util::get_notification();
 
-		if ( $notification ) {
-			$payment->set_transaction_id( $notification->get_transaction_id() );
+		if ($notification) {
+			$payment->set_transaction_id($notification->get_transaction_id());
 
 			$status = $notification->get_status();
-		} elseif ( filter_has_var( INPUT_GET, 'status' ) ) {
-			$status = filter_input( INPUT_GET, 'status', FILTER_SANITIZE_STRING );
+		} elseif (filter_has_var(INPUT_GET, 'status')) {
+			$status = filter_input(INPUT_GET, 'status', FILTER_SANITIZE_STRING);
 		}
 
 		// Set payment status
-		if ( $status ) {
-			$payment->set_status( $status );
+		if ($status) {
+			$payment->set_status($status);
 		}
 	}
 }

src/Util.php

Spacing +5 added lines, -5 removed lines

@@ -20,19 +20,19 @@
 	 * Get parsed notification.
 	 */
 	public static function get_notification() {
-		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')) {
 			// Also check for typo 'xml_notifaction', as this has been used in the past.
 			return;
 		}
 
-		$data = file_get_contents( 'php://input' );
+		$data = file_get_contents('php://input');
 
-		$xml = Core_Util::simplexml_load_string( $data );
+		$xml = Core_Util::simplexml_load_string($data);
 
-		if ( is_wp_error( $xml ) ) {
+		if (is_wp_error($xml)) {
 			return;
 		}
 
-		return NotificationParser::parse( $xml );
+		return NotificationParser::parse($xml);
 	}
 }

src/Listener.php

Spacing +8 added lines, -8 removed lines

@@ -17,30 +17,30 @@
 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);
 
 			// Update payment.
-			Plugin::update_payment( $payment );
+			Plugin::update_payment($payment);
 		}
 	}
 }

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 = ( 'test' === $mode ) ? $this->config_test_class : $this->config_class;
+		$config_class = ('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/AbstractIntegration.php

Spacing +3 added lines, -3 removed lines

@@ -17,10 +17,10 @@
 abstract class AbstractIntegration extends IDeal_AbstractIntegration {
 	public function __construct() {
 		// 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/Settings.php

Spacing +20 added lines, -20 removed lines

@@ -16,61 +16,61 @@
  */
 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) {
 		// Transaction feedback
 		$sections['IDealBasic_feedback'] = array(
-			'title'       => __( 'Transaction feedback', 'pronamic_ideal' ),
-			'methods'     => array( 'ideal-basic' ),
-			'description' => __( 'The URL below needs to be copied to the payment provider dashboard to receive automatic transaction status updates.', 'pronamic_ideal' ),
+			'title'       => __('Transaction feedback', 'pronamic_ideal'),
+			'methods'     => array('ideal-basic'),
+			'description' => __('The URL below needs to be copied to the payment provider dashboard to receive automatic transaction status updates.', 'pronamic_ideal'),
 		);
 
 		// Return sections
 		return $sections;
 	}
 
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		// Hash Key
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
 			'meta_key' => '_pronamic_gateway_ideal_hash_key',
-			'title'    => __( 'Hash Key', 'pronamic_ideal' ),
+			'title'    => __('Hash Key', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'regular-text', 'code' ),
-			'tooltip'  => __( 'Hash key (also known as: key or secret key) as mentioned in the payment provider dashboard.', 'pronamic_ideal' ),
-			'methods'  => array( 'ideal-basic' ),
+			'classes'  => array('regular-text', 'code'),
+			'tooltip'  => __('Hash key (also known as: key or secret key) as mentioned in the payment provider dashboard.', 'pronamic_ideal'),
+			'methods'  => array('ideal-basic'),
 		);
 
 		// Transaction feedback
 		$fields[] = array(
 			'section' => 'ideal',
-			'methods' => array( 'ideal-basic' ),
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'methods' => array('ideal-basic'),
+			'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')
 			),
 		);
 
 		// XML Notification URL
 		$fields[] = array(
 			'section'  => 'IDealBasic_feedback',
-			'title'    => __( 'XML Notification URL', 'pronamic_ideal' ),
+			'title'    => __('XML Notification URL', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'regular-text', 'code' ),
-			'value'    => add_query_arg( array(
+			'classes'  => array('regular-text', 'code'),
+			'value'    => add_query_arg(array(
 				'gateway'          => 'IDealBasic',
 				'xml_notification' => 'true',
-			), site_url( '/' ) ),
-			'methods'  => array( 'ideal-basic' ),
+			), site_url('/')),
+			'methods'  => array('ideal-basic'),
 			'readonly' => true,
 			'size'     => 200,
-			'tooltip'  => __( 'Copy the XML notification URL to the payment provider dashboard to receive automatic transaction status updates.', 'pronamic_ideal' ),
+			'tooltip'  => __('Copy the XML notification URL to the payment provider dashboard to receive automatic transaction status updates.', 'pronamic_ideal'),
 		);
 
 		// Return fields

src/XML/NotificationParser.php

Spacing +10 added lines, -10 removed lines

@@ -26,25 +26,25 @@
 	 *
 	 * @return Notification
 	 */
-	public static function parse( SimpleXMLElement $xml, $notification = null ) {
-		if ( null === $notification ) {
+	public static function parse(SimpleXMLElement $xml, $notification = null) {
+		if (null === $notification) {
 			$notification = new Notification();
 		}
 
-		if ( $xml->createDateTimeStamp ) {
-			$notification->set_date( new DateTime( Security::filter( $xml->createDateTimeStamp ) ) );
+		if ($xml->createDateTimeStamp) {
+			$notification->set_date(new DateTime(Security::filter($xml->createDateTimeStamp)));
 		}
 
-		if ( $xml->transactionID ) {
-			$notification->set_transaction_id( Security::filter( $xml->transactionID ) );
+		if ($xml->transactionID) {
+			$notification->set_transaction_id(Security::filter($xml->transactionID));
 		}
 
-		if ( $xml->purchaseID ) {
-			$notification->set_purchase_id( Security::filter( $xml->purchaseID ) );
+		if ($xml->purchaseID) {
+			$notification->set_purchase_id(Security::filter($xml->purchaseID));
 		}
 
-		if ( $xml->status ) {
-			$notification->set_status( Security::filter( $xml->status ) );
+		if ($xml->status) {
+			$notification->set_status(Security::filter($xml->status));
 		}
 
 		return $notification;