wp-pay-gateways / ogone

src/Listener.php

Spacing +12 added lines, -12 removed lines

@@ -18,27 +18,27 @@
 	public static function listen() {
 		$data = Security::get_request_data();
 
-		$data = array_change_key_case( $data, CASE_UPPER );
+		$data = array_change_key_case($data, CASE_UPPER);
 
-		if ( isset(
-			$data[ Parameters::NC_ERROR ],
+		if (isset(
+			$data[Parameters::NC_ERROR],
 			$data['PAYID'],
-			$data[ Parameters::ORDERID ],
-			$data[ Parameters::STATUS ]
-		) ) {
-			$payment_id = $data[ Parameters::ORDERID ];
+			$data[Parameters::ORDERID],
+			$data[Parameters::STATUS]
+		)) {
+			$payment_id = $data[Parameters::ORDERID];
 
-			if ( isset( $data['PAYMENT_ID'] ) ) {
-				$payment_id = filter_var( $data['PAYMENT_ID'], FILTER_SANITIZE_NUMBER_INT );
+			if (isset($data['PAYMENT_ID'])) {
+				$payment_id = filter_var($data['PAYMENT_ID'], FILTER_SANITIZE_NUMBER_INT);
 			}
 
-			$payment = get_pronamic_payment( $payment_id );
+			$payment = get_pronamic_payment($payment_id);
 
 			// Add note.
-			$payment->add_note( __( 'Webhook requested.', 'pronamic_ideal' ) );
+			$payment->add_note(__('Webhook requested.', 'pronamic_ideal'));
 
 			// Update payment.
-			Plugin::update_payment( $payment );
+			Plugin::update_payment($payment);
 		}
 	}
 }

src/AbstractIntegration.php

Spacing +4 added lines, -4 removed lines

@@ -7,15 +7,15 @@
 abstract class AbstractIntegration extends Common_AbstractIntegration {
 	public function __construct() {
 		$this->url           = 'https://secure.ogone.com/';
-		$this->product_url   = __( 'https://payment-services.ingenico.com/nl/en', 'pronamic_ideal' );
+		$this->product_url   = __('https://payment-services.ingenico.com/nl/en', 'pronamic_ideal');
 		$this->dashboard_url = 'https://secure.ogone.com/';
 		$this->provider      = 'ogone';
 
 		// 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/Security.php

Spacing +24 added lines, -24 removed lines

@@ -30,12 +30,12 @@ 
 	 * Get calculations parameters in
 	 */
 	public static function get_calculations_parameters_in() {
-		if ( ! isset( self::$calculations_parameters_in ) ) {
+		if ( ! isset(self::$calculations_parameters_in)) {
 			self::$calculations_parameters_in = array();
 
-			$file = dirname( __FILE__ ) . '/../data/calculations-parameters-sha-in.txt';
-			if ( is_readable( $file ) ) {
-				self::$calculations_parameters_in = file( $file, FILE_IGNORE_NEW_LINES );
+			$file = dirname(__FILE__) . '/../data/calculations-parameters-sha-in.txt';
+			if (is_readable($file)) {
+				self::$calculations_parameters_in = file($file, FILE_IGNORE_NEW_LINES);
 			}
 		}
 
@@ -46,12 +46,12 @@ 
 	 * Get calculations parameters in
 	 */
 	public static function get_calculations_parameters_out() {
-		if ( ! isset( self::$calculations_parameters_out ) ) {
+		if ( ! isset(self::$calculations_parameters_out)) {
 			self::$calculations_parameters_out = array();
 
-			$file = dirname( __FILE__ ) . '/../data/calculations-parameters-sha-out.txt';
-			if ( is_readable( $file ) ) {
-				self::$calculations_parameters_out = file( $file, FILE_IGNORE_NEW_LINES );
+			$file = dirname(__FILE__) . '/../data/calculations-parameters-sha-out.txt';
+			if (is_readable($file)) {
+				self::$calculations_parameters_out = file($file, FILE_IGNORE_NEW_LINES);
 			}
 		}
 
@@ -66,8 +66,8 @@ 
 	public static function get_request_data() {
 		$data = array();
 
-		if ( isset( $_SERVER['REQUEST_METHOD'] ) ) { // WPCS: input var okay
-			switch ( $_SERVER['REQUEST_METHOD'] ) { // WPCS: input var okay
+		if (isset($_SERVER['REQUEST_METHOD'])) { // WPCS: input var okay
+			switch ($_SERVER['REQUEST_METHOD']) { // WPCS: input var okay
 				case 'GET':
 					// @todo see how we can improve security around this
 					$data = $_GET; // WPCS: input var okay
@@ -84,48 +84,48 @@ 
 		return $data;
 	}
 
-	public static function get_calculation_fields( $calculation_fields, $fields ) {
-		$calculation_fields = array_flip( $calculation_fields );
+	public static function get_calculation_fields($calculation_fields, $fields) {
+		$calculation_fields = array_flip($calculation_fields);
 
-		return array_intersect_key( $fields, $calculation_fields );
+		return array_intersect_key($fields, $calculation_fields);
 	}
 
-	public static function get_signature( $fields, $passphrase, $hash_algorithm ) {
+	public static function get_signature($fields, $passphrase, $hash_algorithm) {
 		// This string is constructed by concatenating the values of the fields sent with the order (sorted
 		// alphabetically, in the format ‘parameter=value’), separated by a passphrase.
 		$string = '';
 
 		// All parameters need to be put alphabetically
-		ksort( $fields );
+		ksort($fields);
 
 		// Loop
-		foreach ( $fields as $name => $value ) {
+		foreach ($fields as $name => $value) {
 			$value = (string) $value;
 
 			// Use of empty will fail, value can be string '0'
-			if ( strlen( $value ) > 0 ) {
-				$name = strtoupper( $name );
+			if (strlen($value) > 0) {
+				$name = strtoupper($name);
 
 				$string .= $name . '=' . $value . $passphrase;
 			}
 		}
 
 		// Hash
-		$result = hash( $hash_algorithm, $string );
+		$result = hash($hash_algorithm, $string);
 
 		// String to uppercase
-		$result = strtoupper( $result );
+		$result = strtoupper($result);
 
 		return $result;
 	}
 
-	public static function sign_data( Data $data, $pass_phrase, $hash_algorithm ) {
+	public static function sign_data(Data $data, $pass_phrase, $hash_algorithm) {
 		$calculation_fields = Security::get_calculations_parameters_in();
 
-		$fields = Security::get_calculation_fields( $calculation_fields, $data->get_fields() );
+		$fields = Security::get_calculation_fields($calculation_fields, $data->get_fields());
 
-		$signature = Security::get_signature( $fields, $pass_phrase, $hash_algorithm );
+		$signature = Security::get_signature($fields, $pass_phrase, $hash_algorithm);
 
-		$data->set_field( 'SHASign', $signature );
+		$data->set_field('SHASign', $signature);
 	}
 }

src/Statuses.php

Spacing +2 added lines, -2 removed lines

@@ -286,8 +286,8 @@
 	 *
 	 * @param string $status
 	 */
-	public static function transform( $status ) {
-		switch ( $status ) {
+	public static function transform($status) {
+		switch ($status) {
 			case self::INCOMPLETE_OR_INVALID :
 			case self::AUTHORIZATION_REFUSED :
 			case self::AUTHOR_DELETION_REFUSED :

src/Config.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
 	 * Constructs and initializes Ogone config object.
 	 */
 	public function __construct() {
-		$this->set_form_action_url( $this->get_default_form_action_url() );
+		$this->set_form_action_url($this->get_default_form_action_url());
 	}
 
 	/**
@@ -51,9 +51,9 @@ 
 	 * @return string
 	 */
 	protected function get_default_form_action_url() {
-		$is_utf8 = strcasecmp( get_bloginfo( 'charset' ), 'UTF-8' ) === 0;
+		$is_utf8 = strcasecmp(get_bloginfo('charset'), 'UTF-8') === 0;
 
-		if ( $is_utf8 ) {
+		if ($is_utf8) {
 			return 'https://secure.ogone.com/ncol/prod/orderstandard_utf8.asp';
 		}
 
@@ -77,7 +77,7 @@ 
 	 *
 	 * @param string $url Ogone e-Commerce form action URL.
 	 */
-	public function set_form_action_url( $url ) {
+	public function set_form_action_url($url) {
 		$this->form_action_url = $url;
 	}
 
@@ -98,7 +98,7 @@ 
 	 * @return string
 	 */
 	public function get_direct_query_url() {
-		if ( 'test' === $this->mode ) {
+		if ('test' === $this->mode) {
 			return 'https://secure.ogone.com/ncol/test/querydirect.asp';
 		}
 

src/Data.php

Spacing +5 added lines, -5 removed lines

@@ -41,11 +41,11 @@ 
 	 *
 	 * @param string $name
 	 */
-	public function get_field( $name ) {
+	public function get_field($name) {
 		$value = null;
 
-		if ( isset( $this->fields[ $name ] ) ) {
-			$value = $this->fields[ $name ];
+		if (isset($this->fields[$name])) {
+			$value = $this->fields[$name];
 		}
 
 		return $value;
@@ -59,8 +59,8 @@ 
 	 *
 	 * @return Data
 	 */
-	public function set_field( $name, $value ) {
-		$this->fields[ $name ] = $value;
+	public function set_field($name, $value) {
+		$this->fields[$name] = $value;
 
 		return $this;
 	}

src/OrderStandardEasy/Client.php

Spacing +4 added lines, -4 removed lines

@@ -42,10 +42,10 @@ 
 	 *
 	 * @param string $psp_id
 	 */
-	public function __construct( $psp_id ) {
+	public function __construct($psp_id) {
 		$this->data = new Data();
-		$this->data->set_field( Parameters::PSPID, $psp_id );
-		$this->data->set_field( 'PM', self::PAYMENT_TYPE_IDEAL );
+		$this->data->set_field(Parameters::PSPID, $psp_id);
+		$this->data->set_field('PM', self::PAYMENT_TYPE_IDEAL);
 	}
 
 	/**
@@ -62,7 +62,7 @@ 
 	 *
 	 * @param string $url an URL
 	 */
-	public function set_payment_server_url( $url ) {
+	public function set_payment_server_url($url) {
 		$this->payment_server_url = $url;
 	}
 

src/OrderStandardEasy/Gateway.php

Spacing +35 added lines, -35 removed lines

@@ -26,15 +26,15 @@ 
 	 *
 	 * @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( $config->psp_id );
-		$this->client->set_payment_server_url( $config->get_form_action_url() );
+		$this->client = new Client($config->psp_id);
+		$this->client->set_payment_server_url($config->get_form_action_url());
 	}
 
 	/**
@@ -64,44 +64,44 @@ 
 	 *
 	 * @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());
 
 		$ogone_data = $this->client->get_data();
 
 		// General
-		$ogone_data_general = new DataGeneralHelper( $ogone_data );
+		$ogone_data_general = new DataGeneralHelper($ogone_data);
 
 		$ogone_data_general
-			->set_order_id( $payment->format_string( $this->config->order_id ) )
-			->set_order_description( $payment->get_description() )
-			->set_param_plus( 'payment_id=' . $payment->get_id() )
-			->set_currency( $payment->get_currency() )
-			->set_amount( $payment->get_amount()->get_amount() )
-			->set_language( $payment->get_locale() );
+			->set_order_id($payment->format_string($this->config->order_id))
+			->set_order_description($payment->get_description())
+			->set_param_plus('payment_id=' . $payment->get_id())
+			->set_currency($payment->get_currency())
+			->set_amount($payment->get_amount()->get_amount())
+			->set_language($payment->get_locale());
 
 		// Customer
-		$ogone_data_customer = new DataCustomerHelper( $ogone_data );
+		$ogone_data_customer = new DataCustomerHelper($ogone_data);
 
 		$ogone_data_customer
-			->set_name( $payment->get_customer_name() )
-			->set_email( $payment->get_email() )
-			->set_address( $payment->get_address() )
-			->set_zip( $payment->get_zip() )
-			->set_town( $payment->get_city() )
-			->set_country( $payment->get_country() )
-			->set_telephone_number( $payment->get_telephone_number() );
+			->set_name($payment->get_customer_name())
+			->set_email($payment->get_email())
+			->set_address($payment->get_address())
+			->set_zip($payment->get_zip())
+			->set_town($payment->get_city())
+			->set_country($payment->get_country())
+			->set_telephone_number($payment->get_telephone_number());
 
 		// URL's
-		$ogone_url_helper = new DataUrlHelper( $ogone_data );
+		$ogone_url_helper = new DataUrlHelper($ogone_data);
 
 		$ogone_url_helper
-			->set_accept_url( add_query_arg( 'status', Statuses::SUCCESS, $payment->get_return_url() ) )
-			->set_cancel_url( add_query_arg( 'status', Statuses::CANCELLED, $payment->get_return_url() ) )
-			->set_decline_url( add_query_arg( 'status', Statuses::FAILURE, $payment->get_return_url() ) )
-			->set_exception_url( add_query_arg( 'status', Statuses::FAILURE, $payment->get_return_url() ) )
-			->set_back_url( home_url( '/' ) )
-			->set_home_url( home_url( '/' ) );
+			->set_accept_url(add_query_arg('status', Statuses::SUCCESS, $payment->get_return_url()))
+			->set_cancel_url(add_query_arg('status', Statuses::CANCELLED, $payment->get_return_url()))
+			->set_decline_url(add_query_arg('status', Statuses::FAILURE, $payment->get_return_url()))
+			->set_exception_url(add_query_arg('status', Statuses::FAILURE, $payment->get_return_url()))
+			->set_back_url(home_url('/'))
+			->set_home_url(home_url('/'));
 	}
 
 	/**
@@ -109,13 +109,13 @@ 
 	 *
 	 * @param 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);
 
-		$payment->set_status( $status );
+		$payment->set_status($status);
 	}
 }

src/OrderStandardEasy/ConfigFactory.php

Spacing +11 added lines, -11 removed lines

@@ -17,27 +17,27 @@
 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();
 
-		$form_action_url = get_post_meta( $post_id, '_pronamic_gateway_ogone_form_action_url', true );
+		$form_action_url = get_post_meta($post_id, '_pronamic_gateway_ogone_form_action_url', true);
 
-		if ( '' !== $form_action_url ) {
-			$config->set_form_action_url( $form_action_url );
+		if ('' !== $form_action_url) {
+			$config->set_form_action_url($form_action_url);
 		}
 
 		$config->mode     = $mode;
-		$config->psp_id   = get_post_meta( $post_id, '_pronamic_gateway_ogone_psp_id', true );
-		$config->order_id = get_post_meta( $post_id, '_pronamic_gateway_ogone_order_id', true );
+		$config->psp_id   = get_post_meta($post_id, '_pronamic_gateway_ogone_psp_id', true);
+		$config->order_id = get_post_meta($post_id, '_pronamic_gateway_ogone_order_id', true);
 
 		return $config;
 	}