wp-pay-gateways / ideal-advanced-v3

src/ConfigFactory.php

Spacing +10 added lines, -10 removed lines

@@ -16,25 +16,25 @@
 class ConfigFactory extends GatewayConfigFactory {
 	private $config_class;
 
-	public function __construct( $config_class = 'Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\Config', $config_test_class = 'Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\Config' ) {
+	public function __construct($config_class = 'Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\Config', $config_test_class = 'Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\Config') {
 		$this->config_class      = $config_class;
 		$this->config_test_class = $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->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->purchase_id = get_post_meta($post_id, '_pronamic_gateway_ideal_purchase_id', true);
 
-		$config->private_key          = get_post_meta( $post_id, '_pronamic_gateway_ideal_private_key', true );
-		$config->private_key_password = get_post_meta( $post_id, '_pronamic_gateway_ideal_private_key_password', true );
-		$config->private_certificate  = get_post_meta( $post_id, '_pronamic_gateway_ideal_private_certificate', true );
+		$config->private_key          = get_post_meta($post_id, '_pronamic_gateway_ideal_private_key', true);
+		$config->private_key_password = get_post_meta($post_id, '_pronamic_gateway_ideal_private_key_password', true);
+		$config->private_certificate  = get_post_meta($post_id, '_pronamic_gateway_ideal_private_certificate', true);
 
 		return $config;
 	}

src/Settings.php

Spacing +153 added lines, -153 removed lines

@@ -17,14 +17,14 @@ 
 class Settings extends GatewaySettings {
 	public function __construct() {
 		// Filters
-		add_filter( 'pronamic_pay_gateway_fields', array( $this, 'fields' ) );
+		add_filter('pronamic_pay_gateway_fields', array($this, 'fields'));
 
 		// Actions
-		add_action( 'current_screen', array( $this, 'maybe_download_private_certificate' ) );
-		add_action( 'current_screen', array( $this, 'maybe_download_private_key' ) );
+		add_action('current_screen', array($this, 'maybe_download_private_certificate'));
+		add_action('current_screen', array($this, 'maybe_download_private_key'));
 	}
 
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		/*
 		 * Private Key and Certificate
 		 */
@@ -32,83 +32,83 @@ 
 		// Private key and certificate information
 		$fields[] = array(
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
-			'title'    => __( 'Private key and certificate', 'pronamic_ideal' ),
+			'methods'  => array('ideal-advanced-v3'),
+			'title'    => __('Private key and certificate', 'pronamic_ideal'),
 			'type'     => 'description',
-			'callback' => array( $this, 'field_security' ),
+			'callback' => array($this, 'field_security'),
 		);
 
 		// Organization
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_organization',
-			'title'    => __( 'Organization', 'pronamic_ideal' ),
+			'title'    => __('Organization', 'pronamic_ideal'),
 			'type'     => 'text',
-			'tooltip'  => __( 'Organization name, e.g. Pronamic', 'pronamic_ideal' ),
+			'tooltip'  => __('Organization name, e.g. Pronamic', 'pronamic_ideal'),
 		);
 
 		// Organization Unit
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_organization_unit',
-			'title'    => __( 'Organization Unit', 'pronamic_ideal' ),
+			'title'    => __('Organization Unit', 'pronamic_ideal'),
 			'type'     => 'text',
-			'tooltip'  => __( 'Organization unit, e.g. Administration', 'pronamic_ideal' ),
+			'tooltip'  => __('Organization unit, e.g. Administration', 'pronamic_ideal'),
 		);
 
 		// Locality
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_locality',
-			'title'    => __( 'City', 'pronamic_ideal' ),
+			'title'    => __('City', 'pronamic_ideal'),
 			'type'     => 'text',
-			'tooltip'  => __( 'City, e.g. Amsterdam', 'pronamic_ideal' ),
+			'tooltip'  => __('City, e.g. Amsterdam', 'pronamic_ideal'),
 		);
 
 		// State or Province
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_state_or_province',
-			'title'    => __( 'State / province', 'pronamic_ideal' ),
+			'title'    => __('State / province', 'pronamic_ideal'),
 			'type'     => 'text',
-			'tooltip'  => __( 'State or province, e.g. Friesland', 'pronamic_ideal' ),
+			'tooltip'  => __('State or province, e.g. Friesland', 'pronamic_ideal'),
 		);
 
 		// Country
-		$locale = explode( '_', get_locale() );
+		$locale = explode('_', get_locale());
 
-		$locale = array_pop( $locale );
+		$locale = array_pop($locale);
 
 		$fields[] = array(
 			'filter'      => FILTER_SANITIZE_STRING,
 			'section'     => 'ideal',
-			'methods'     => array( 'ideal-advanced-v3' ),
+			'methods'     => array('ideal-advanced-v3'),
 			'group'       => 'pk-cert',
 			'meta_key'    => '_pronamic_gateway_country',
-			'title'       => __( 'Country', 'pronamic_ideal' ),
+			'title'       => __('Country', 'pronamic_ideal'),
 			'type'        => 'text',
 			'tooltip'     => sprintf(
 				'%s %s (ISO-3166-1 alpha-2)',
-				__( '2 letter country code, e.g.', 'pronamic_ideal' ),
-				strtoupper( $locale )
+				__('2 letter country code, e.g.', 'pronamic_ideal'),
+				strtoupper($locale)
 			),
 			'size'        => 2,
 			'description' => sprintf(
 				'%s %s',
-				__( '2 letter country code, e.g.', 'pronamic_ideal' ),
-				strtoupper( $locale )
+				__('2 letter country code, e.g.', 'pronamic_ideal'),
+				strtoupper($locale)
 			),
 		);
 
@@ -116,14 +116,14 @@ 
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_email',
-			'title'    => __( 'E-mail address', 'pronamic_ideal' ),
+			'title'    => __('E-mail address', 'pronamic_ideal'),
 			'tooltip'  => sprintf(
 				/* translators: %s: admin email */
-				__( 'E-mail address, e.g. %s', 'pronamic_ideal' ),
-				get_option( 'admin_email' )
+				__('E-mail address, e.g. %s', 'pronamic_ideal'),
+				get_option('admin_email')
 			),
 			'type'     => 'text',
 		);
@@ -132,66 +132,66 @@ 
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_NUMBER_INT,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_number_days_valid',
-			'title'    => __( 'Number Days Valid', 'pronamic_ideal' ),
+			'title'    => __('Number Days Valid', 'pronamic_ideal'),
 			'type'     => 'text',
 			'default'  => 1825,
-			'tooltip'  => __( 'Number of days the generated certificate will be valid for, e.g. 1825 days for the maximum duration of 5 years.', 'pronamic_ideal' ),
+			'tooltip'  => __('Number of days the generated certificate will be valid for, e.g. 1825 days for the maximum duration of 5 years.', 'pronamic_ideal'),
 		);
 
 		// Private Key Password
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_ideal_private_key_password',
-			'title'    => __( 'Private Key Password', 'pronamic_ideal' ),
+			'title'    => __('Private Key Password', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'regular-text', 'code' ),
+			'classes'  => array('regular-text', 'code'),
 			'default'  => wp_generate_password(),
-			'tooltip'  => __( 'A random password which will be used for the generation of the private key and certificate.', 'pronamic_ideal' ),
+			'tooltip'  => __('A random password which will be used for the generation of the private key and certificate.', 'pronamic_ideal'),
 		);
 
 		// Private Key
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_ideal_private_key',
-			'title'    => __( 'Private Key', 'pronamic_ideal' ),
+			'title'    => __('Private Key', 'pronamic_ideal'),
 			'type'     => 'textarea',
-			'callback' => array( $this, 'field_private_key' ),
-			'classes'  => array( 'code' ),
-			'tooltip'  => __( 'The private key is used for secure communication with the payment provider. If left empty, the private key will be generated using the given private key password.', 'pronamic_ideal' ),
+			'callback' => array($this, 'field_private_key'),
+			'classes'  => array('code'),
+			'tooltip'  => __('The private key is used for secure communication with the payment provider. If left empty, the private key will be generated using the given private key password.', 'pronamic_ideal'),
 		);
 
 		// Private Certificate
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'ideal',
-			'methods'  => array( 'ideal-advanced-v3' ),
+			'methods'  => array('ideal-advanced-v3'),
 			'group'    => 'pk-cert',
 			'meta_key' => '_pronamic_gateway_ideal_private_certificate',
-			'title'    => __( 'Private Certificate', 'pronamic_ideal' ),
+			'title'    => __('Private Certificate', 'pronamic_ideal'),
 			'type'     => 'textarea',
-			'callback' => array( $this, 'field_private_certificate' ),
-			'classes'  => array( 'code' ),
-			'tooltip'  => __( 'The certificate is used for secure communication with the payment provider. If left empty, the certificate will be generated using the private key and given organization details.', 'pronamic_ideal' ),
+			'callback' => array($this, 'field_private_certificate'),
+			'classes'  => array('code'),
+			'tooltip'  => __('The certificate is used for secure communication with the payment provider. If left empty, the certificate will be generated using the private key and given organization details.', 'pronamic_ideal'),
 		);
 
 		// Transaction feedback
 		$fields[] = array(
 			'section' => 'ideal',
-			'methods' => array( 'ideal-advanced-v3' ),
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'methods' => array('ideal-advanced-v3'),
+			'title'   => __('Transaction feedback', 'pronamic_ideal'),
 			'type'    => 'description',
 			'html'    => sprintf(
 				'<span class="dashicons dashicons-yes"></span> %s',
-				__( 'Payment status updates will be processed without any additional configuration.', 'pronamic_ideal' )
+				__('Payment status updates will be processed without any additional configuration.', 'pronamic_ideal')
 			),
 		);
 
@@ -204,25 +204,25 @@ 
 	 *
 	 * @param array $field
 	 */
-	public function field_security( $field ) {
-		$certificate = get_post_meta( get_the_ID(), '_pronamic_gateway_ideal_private_certificate', true );
+	public function field_security($field) {
+		$certificate = get_post_meta(get_the_ID(), '_pronamic_gateway_ideal_private_certificate', true);
 
 		?>
 		<p>
-			<?php if ( empty( $certificate ) ) : ?>
+			<?php if (empty($certificate)) : ?>
 
 				<span
-					class="dashicons dashicons-no"></span> <?php esc_html_e( 'The private key and certificate have not yet been configured.', 'pronamic_ideal' ); ?>
+					class="dashicons dashicons-no"></span> <?php esc_html_e('The private key and certificate have not yet been configured.', 'pronamic_ideal'); ?>
 				<br/>
 
 				<br/>
 
-				<?php esc_html_e( 'A private key and certificate are required for communication with the payment provider. Enter the organization details from the iDEAL account below to generate these required files.', 'pronamic_ideal' ); ?>
+				<?php esc_html_e('A private key and certificate are required for communication with the payment provider. Enter the organization details from the iDEAL account below to generate these required files.', 'pronamic_ideal'); ?>
 
 			<?php else : ?>
 
 				<span
-					class="dashicons dashicons-yes"></span> <?php esc_html_e( 'A private key and certificate have been configured. The certificate must be uploaded to the payment provider dashboard to complete configuration.', 'pronamic_ideal' ); ?>
+					class="dashicons dashicons-yes"></span> <?php esc_html_e('A private key and certificate have been configured. The certificate must be uploaded to the payment provider dashboard to complete configuration.', 'pronamic_ideal'); ?>
 				<br/>
 
 				<br/>
@@ -230,7 +230,7 @@ 
 				<?php
 
 				submit_button(
-					__( 'Download certificate', 'pronamic_ideal' ),
+					__('Download certificate', 'pronamic_ideal'),
 					'secondary',
 					'download_private_certificate',
 					false
@@ -238,37 +238,37 @@ 
 
 				?>
 
-				<a class="pronamic-pay-btn-link" href="#" id="pk-cert-fields-toggle"><?php esc_html_e( 'Show details…', 'pronamic_ideal' ); ?></a>
+				<a class="pronamic-pay-btn-link" href="#" id="pk-cert-fields-toggle"><?php esc_html_e('Show details…', 'pronamic_ideal'); ?></a>
 
 			<?php endif; ?>
 		</p>
 		<?php
 	}
 
-	public function field_private_key( $field ) {
-		$private_key          = get_post_meta( get_the_ID(), '_pronamic_gateway_ideal_private_key', true );
-		$private_key_password = get_post_meta( get_the_ID(), '_pronamic_gateway_ideal_private_key_password', true );
-		$number_days_valid    = get_post_meta( get_the_ID(), '_pronamic_gateway_number_days_valid', true );
+	public function field_private_key($field) {
+		$private_key          = get_post_meta(get_the_ID(), '_pronamic_gateway_ideal_private_key', true);
+		$private_key_password = get_post_meta(get_the_ID(), '_pronamic_gateway_ideal_private_key_password', true);
+		$number_days_valid    = get_post_meta(get_the_ID(), '_pronamic_gateway_number_days_valid', true);
 
-		$filename = __( 'ideal.key', 'pronamic_ideal' );
+		$filename = __('ideal.key', 'pronamic_ideal');
 
-		if ( ! empty( $private_key_password ) && ! empty( $number_days_valid ) ) {
+		if ( ! empty($private_key_password) && ! empty($number_days_valid)) {
 			$command = sprintf(
 				'openssl genrsa -aes128 -out %s -passout pass:%s 2048',
-				escapeshellarg( $filename ),
-				escapeshellarg( $private_key_password )
+				escapeshellarg($filename),
+				escapeshellarg($private_key_password)
 			);
 
 			?>
 
-			<p><?php esc_html_e( 'OpenSSL command', 'pronamic_ideal' ); ?></p>
-			<input id="pronamic_ideal_openssl_command_key" name="pronamic_ideal_openssl_command_key" value="<?php echo esc_attr( $command ); ?>" type="text" class="large-text code" readonly="readonly"/>
+			<p><?php esc_html_e('OpenSSL command', 'pronamic_ideal'); ?></p>
+			<input id="pronamic_ideal_openssl_command_key" name="pronamic_ideal_openssl_command_key" value="<?php echo esc_attr($command); ?>" type="text" class="large-text code" readonly="readonly"/>
 
 			<?php
 		} else {
 			printf(
 				'<p class="pronamic-pay-description description">%s</p>',
-				esc_html__( 'Leave empty and save the configuration to generate the private key or view the OpenSSL command.', 'pronamic_ideal' )
+				esc_html__('Leave empty and save the configuration to generate the private key or view the OpenSSL command.', 'pronamic_ideal')
 			);
 		}
 
@@ -276,9 +276,9 @@ 
 		<p>
 			<?php
 
-			if ( ! empty( $private_key ) ) {
+			if ( ! empty($private_key)) {
 				submit_button(
-					__( 'Download', 'pronamic_ideal' ),
+					__('Download', 'pronamic_ideal'),
 					'secondary',
 					'download_private_key',
 					false
@@ -289,7 +289,7 @@ 
 
 			printf(
 				'<label class="pronamic-pay-form-control-file-button button">%s <input type="file" name="%s" /></label>',
-				esc_html__( 'Upload', 'pronamic_ideal' ),
+				esc_html__('Upload', 'pronamic_ideal'),
 				'_pronamic_gateway_ideal_private_key_file'
 			);
 
@@ -298,79 +298,79 @@ 
 		<?php
 	}
 
-	public function field_private_certificate( $field ) {
-		$certificate = get_post_meta( get_the_ID(), '_pronamic_gateway_ideal_private_certificate', true );
+	public function field_private_certificate($field) {
+		$certificate = get_post_meta(get_the_ID(), '_pronamic_gateway_ideal_private_certificate', true);
 
-		$private_key_password = get_post_meta( get_the_ID(), '_pronamic_gateway_ideal_private_key_password', true );
-		$number_days_valid    = get_post_meta( get_the_ID(), '_pronamic_gateway_number_days_valid', true );
+		$private_key_password = get_post_meta(get_the_ID(), '_pronamic_gateway_ideal_private_key_password', true);
+		$number_days_valid    = get_post_meta(get_the_ID(), '_pronamic_gateway_number_days_valid', true);
 
-		$filename_key = __( 'ideal.key', 'pronamic_ideal' );
-		$filename_cer = __( 'ideal.cer', 'pronamic_ideal' );
+		$filename_key = __('ideal.key', 'pronamic_ideal');
+		$filename_cer = __('ideal.cer', 'pronamic_ideal');
 
 		// @link http://www.openssl.org/docs/apps/req.html
 		$subj_args = array(
-			'C'            => get_post_meta( get_the_ID(), '_pronamic_gateway_country', true ),
-			'ST'           => get_post_meta( get_the_ID(), '_pronamic_gateway_state_or_province', true ),
-			'L'            => get_post_meta( get_the_ID(), '_pronamic_gateway_locality', true ),
-			'O'            => get_post_meta( get_the_ID(), '_pronamic_gateway_organization', true ),
-			'OU'           => get_post_meta( get_the_ID(), '_pronamic_gateway_organization_unit', true ),
-			'CN'           => get_post_meta( get_the_ID(), '_pronamic_gateway_organization', true ),
-			'emailAddress' => get_post_meta( get_the_ID(), '_pronamic_gateway_email', true ),
+			'C'            => get_post_meta(get_the_ID(), '_pronamic_gateway_country', true),
+			'ST'           => get_post_meta(get_the_ID(), '_pronamic_gateway_state_or_province', true),
+			'L'            => get_post_meta(get_the_ID(), '_pronamic_gateway_locality', true),
+			'O'            => get_post_meta(get_the_ID(), '_pronamic_gateway_organization', true),
+			'OU'           => get_post_meta(get_the_ID(), '_pronamic_gateway_organization_unit', true),
+			'CN'           => get_post_meta(get_the_ID(), '_pronamic_gateway_organization', true),
+			'emailAddress' => get_post_meta(get_the_ID(), '_pronamic_gateway_email', true),
 		);
 
-		$subj_args = array_filter( $subj_args );
+		$subj_args = array_filter($subj_args);
 
 		$subj = '';
-		foreach ( $subj_args as $type => $value ) {
-			$subj .= '/' . $type . '=' . addslashes( $value );
+		foreach ($subj_args as $type => $value) {
+			$subj .= '/' . $type . '=' . addslashes($value);
 		}
 
-		if ( ! empty( $subj ) ) {
-			$command = trim( sprintf(
+		if ( ! empty($subj)) {
+			$command = trim(sprintf(
 				'openssl req -x509 -sha256 -new -key %s -passin pass:%s -days %s -out %s %s',
-				escapeshellarg( $filename_key ),
-				escapeshellarg( $private_key_password ),
-				escapeshellarg( $number_days_valid ),
-				escapeshellarg( $filename_cer ),
-				empty( $subj ) ? '' : sprintf( '-subj %s', escapeshellarg( $subj ) )
-			) );
+				escapeshellarg($filename_key),
+				escapeshellarg($private_key_password),
+				escapeshellarg($number_days_valid),
+				escapeshellarg($filename_cer),
+				empty($subj) ? '' : sprintf('-subj %s', escapeshellarg($subj))
+			));
 
 			?>
 
-			<p><?php esc_html_e( 'OpenSSL command', 'pronamic_ideal' ); ?></p>
-			<input id="pronamic_ideal_openssl_command_certificate" name="pronamic_ideal_openssl_command_certificate" value="<?php echo esc_attr( $command ); ?>" type="text" class="large-text code" readonly="readonly"/>
+			<p><?php esc_html_e('OpenSSL command', 'pronamic_ideal'); ?></p>
+			<input id="pronamic_ideal_openssl_command_certificate" name="pronamic_ideal_openssl_command_certificate" value="<?php echo esc_attr($command); ?>" type="text" class="large-text code" readonly="readonly"/>
 
 			<?php
 		} else {
 			printf(
 				'<p class="pronamic-pay-description description">%s</p>',
-				esc_html__( 'Leave empty and save the configuration to generate the certificate or view the OpenSSL command.', 'pronamic_ideal' )
+				esc_html__('Leave empty and save the configuration to generate the certificate or view the OpenSSL command.', 'pronamic_ideal')
 			);
 		}
 
-		if ( ! empty( $certificate ) ) {
-			$fingerprint = Security::get_sha_fingerprint( $certificate );
-			$fingerprint = str_split( $fingerprint, 2 );
-			$fingerprint = implode( ':', $fingerprint );
+		if ( ! empty($certificate)) {
+			$fingerprint = Security::get_sha_fingerprint($certificate);
+			$fingerprint = str_split($fingerprint, 2);
+			$fingerprint = implode(':', $fingerprint);
 
 			echo '<dl>';
 
-			echo '<dt>', esc_html__( 'SHA Fingerprint', 'pronamic_ideal' ), '</dt>';
-			echo '<dd>', esc_html( $fingerprint ), '</dd>';
+			echo '<dt>', esc_html__('SHA Fingerprint', 'pronamic_ideal'), '</dt>';
+			echo '<dd>', esc_html($fingerprint), '</dd>';
 
-			$info = openssl_x509_parse( $certificate );
+			$info = openssl_x509_parse($certificate);
 
-			if ( $info ) {
-				$date_format = __( 'M j, Y @ G:i', 'pronamic_ideal' );
+			if ($info) {
+				$date_format = __('M j, Y @ G:i', 'pronamic_ideal');
 
-				if ( isset( $info['validFrom_time_t'] ) ) {
-					echo '<dt>', esc_html__( 'Valid From', 'pronamic_ideal' ), '</dt>';
-					echo '<dd>', esc_html( date_i18n( $date_format, $info['validFrom_time_t'] ) ), '</dd>';
+				if (isset($info['validFrom_time_t'])) {
+					echo '<dt>', esc_html__('Valid From', 'pronamic_ideal'), '</dt>';
+					echo '<dd>', esc_html(date_i18n($date_format, $info['validFrom_time_t'])), '</dd>';
 				}
 
-				if ( isset( $info['validTo_time_t'] ) ) {
-					echo '<dt>', esc_html__( 'Valid To', 'pronamic_ideal' ), '</dt>';
-					echo '<dd>', esc_html( date_i18n( $date_format, $info['validTo_time_t'] ) ), '</dd>';
+				if (isset($info['validTo_time_t'])) {
+					echo '<dt>', esc_html__('Valid To', 'pronamic_ideal'), '</dt>';
+					echo '<dd>', esc_html(date_i18n($date_format, $info['validTo_time_t'])), '</dd>';
 				}
 			}
 
@@ -381,9 +381,9 @@ 
 		<p>
 			<?php
 
-			if ( ! empty( $certificate ) ) {
+			if ( ! empty($certificate)) {
 				submit_button(
-					__( 'Download', 'pronamic_ideal' ),
+					__('Download', 'pronamic_ideal'),
 					'secondary',
 					'download_private_certificate',
 					false
@@ -394,7 +394,7 @@ 
 
 			printf(
 				'<label class="pronamic-pay-form-control-file-button button">%s <input type="file" name="%s" /></label>',
-				esc_html__( 'Upload', 'pronamic_ideal' ),
+				esc_html__('Upload', 'pronamic_ideal'),
 				'_pronamic_gateway_ideal_private_certificate_file'
 			);
 
@@ -407,16 +407,16 @@ 
 	 * Download private certificate
 	 */
 	public function maybe_download_private_certificate() {
-		if ( filter_has_var( INPUT_POST, 'download_private_certificate' ) ) {
-			$post_id = filter_input( INPUT_POST, 'post_ID', FILTER_SANITIZE_STRING );
+		if (filter_has_var(INPUT_POST, 'download_private_certificate')) {
+			$post_id = filter_input(INPUT_POST, 'post_ID', FILTER_SANITIZE_STRING);
 
-			$filename = sprintf( 'ideal-private-certificate-%s.cer', $post_id );
+			$filename = sprintf('ideal-private-certificate-%s.cer', $post_id);
 
-			header( 'Content-Description: File Transfer' );
-			header( 'Content-Disposition: attachment; filename=' . $filename );
-			header( 'Content-Type: application/x-x509-ca-cert; charset=' . get_option( 'blog_charset' ), true );
+			header('Content-Description: File Transfer');
+			header('Content-Disposition: attachment; filename=' . $filename);
+			header('Content-Type: application/x-x509-ca-cert; charset=' . get_option('blog_charset'), true);
 
-			echo get_post_meta( $post_id, '_pronamic_gateway_ideal_private_certificate', true ); //xss ok
+			echo get_post_meta($post_id, '_pronamic_gateway_ideal_private_certificate', true); //xss ok
 
 			exit;
 		}
@@ -426,16 +426,16 @@ 
 	 * Download private key
 	 */
 	public function maybe_download_private_key() {
-		if ( filter_has_var( INPUT_POST, 'download_private_key' ) ) {
-			$post_id = filter_input( INPUT_POST, 'post_ID', FILTER_SANITIZE_STRING );
+		if (filter_has_var(INPUT_POST, 'download_private_key')) {
+			$post_id = filter_input(INPUT_POST, 'post_ID', FILTER_SANITIZE_STRING);
 
-			$filename = sprintf( 'ideal-private-key-%s.key', $post_id );
+			$filename = sprintf('ideal-private-key-%s.key', $post_id);
 
-			header( 'Content-Description: File Transfer' );
-			header( 'Content-Disposition: attachment; filename=' . $filename );
-			header( 'Content-Type: application/pgp-keys; charset=' . get_option( 'blog_charset' ), true );
+			header('Content-Description: File Transfer');
+			header('Content-Disposition: attachment; filename=' . $filename);
+			header('Content-Type: application/pgp-keys; charset=' . get_option('blog_charset'), true);
 
-			echo get_post_meta( $post_id, '_pronamic_gateway_ideal_private_key', true ); //xss ok
+			echo get_post_meta($post_id, '_pronamic_gateway_ideal_private_key', true); //xss ok
 
 			exit;
 		}
@@ -444,41 +444,41 @@ 
 	/***
 	 * Save post.
 	 */
-	public function save_post( $data ) {
+	public function save_post($data) {
 		// Files
 		$files = array(
 			'_pronamic_gateway_ideal_private_key_file' => '_pronamic_gateway_ideal_private_key',
 			'_pronamic_gateway_ideal_private_certificate_file' => '_pronamic_gateway_ideal_private_certificate',
 		);
 
-		foreach ( $files as $name => $meta_key ) {
-			if ( isset( $_FILES[ $name ] ) && UPLOAD_ERR_OK === $_FILES[ $name ]['error'] ) { // WPCS: input var okay.
-				$value = file_get_contents( $_FILES[ $name ]['tmp_name'] ); // WPCS: input var okay. // WPCS: sanitization ok.
+		foreach ($files as $name => $meta_key) {
+			if (isset($_FILES[$name]) && UPLOAD_ERR_OK === $_FILES[$name]['error']) { // WPCS: input var okay.
+				$value = file_get_contents($_FILES[$name]['tmp_name']); // WPCS: input var okay. // WPCS: sanitization ok.
 
-				$data[ $meta_key ] = $value;
+				$data[$meta_key] = $value;
 			}
 		}
 
 		// Generate private key and certificate
-		if ( empty( $data['_pronamic_gateway_ideal_private_key_password'] ) ) {
+		if (empty($data['_pronamic_gateway_ideal_private_key_password'])) {
 			// Without private key password we can't create private key and certifiate.
 			return $data;
 		}
 
-		if ( ! in_array( 'aes-128-cbc', openssl_get_cipher_methods(), true ) ) {
+		if ( ! in_array('aes-128-cbc', openssl_get_cipher_methods(), true)) {
 			// Without AES-128-CBC ciphter method we can't create private key and certificate.
 			return $data;
 		}
 
 		// Private key
-		$pkey = openssl_pkey_get_private( $data['_pronamic_gateway_ideal_private_key'], $data['_pronamic_gateway_ideal_private_key_password'] );
+		$pkey = openssl_pkey_get_private($data['_pronamic_gateway_ideal_private_key'], $data['_pronamic_gateway_ideal_private_key_password']);
 
-		if ( false === $pkey ) {
+		if (false === $pkey) {
 			// If we can't open the private key we will create a new private key and certificate.
 
-			if ( defined( 'OPENSSL_CIPHER_AES_128_CBC' ) ) {
+			if (defined('OPENSSL_CIPHER_AES_128_CBC')) {
 				$cipher = OPENSSL_CIPHER_AES_128_CBC;
-			} elseif ( defined( 'OPENSSL_CIPHER_3DES' ) ) {
+			} elseif (defined('OPENSSL_CIPHER_3DES')) {
 				// @link https://www.pronamic.nl/wp-content/uploads/2011/12/iDEAL_Advanced_PHP_EN_V2.2.pdf
 				$cipher = OPENSSL_CIPHER_3DES;
 			} else {
@@ -497,16 +497,16 @@ 
 				'basicConstraints'       => 'CA:true',
 			);
 
-			$pkey = openssl_pkey_new( $args );
+			$pkey = openssl_pkey_new($args);
 
-			if ( false === $pkey ) {
+			if (false === $pkey) {
 				return $data;
 			}
 
 			// Export key
-			$result = openssl_pkey_export( $pkey, $private_key, $data['_pronamic_gateway_ideal_private_key_password'], $args );
+			$result = openssl_pkey_export($pkey, $private_key, $data['_pronamic_gateway_ideal_private_key_password'], $args);
 
-			if ( false === $result ) {
+			if (false === $result) {
 				return $data;
 			}
 
@@ -516,7 +516,7 @@ 
 		}
 
 		// Certificate
-		if ( empty( $data['_pronamic_gateway_ideal_private_certificate'] ) ) {
+		if (empty($data['_pronamic_gateway_ideal_private_certificate'])) {
 			$required_keys = array(
 				'countryName',
 				'stateOrProvinceName',
@@ -536,16 +536,16 @@ 
 				'emailAddress'           => $data['_pronamic_gateway_email'],
 			);
 
-			$distinguished_name = array_filter( $distinguished_name );
+			$distinguished_name = array_filter($distinguished_name);
 
 			// Create certificate only if distinguished name contains all required elements
 			// @link http://stackoverflow.com/questions/13169588/how-to-check-if-multiple-array-keys-exists
-			if ( count( array_intersect_key( array_flip( $required_keys ), $distinguished_name ) ) === count( $required_keys ) ) {
-				$csr = openssl_csr_new( $distinguished_name, $pkey );
+			if (count(array_intersect_key(array_flip($required_keys), $distinguished_name)) === count($required_keys)) {
+				$csr = openssl_csr_new($distinguished_name, $pkey);
 
-				$cert = openssl_csr_sign( $csr, null, $pkey, $data['_pronamic_gateway_number_days_valid'], $args, time() );
+				$cert = openssl_csr_sign($csr, null, $pkey, $data['_pronamic_gateway_number_days_valid'], $args, time());
 
-				openssl_x509_export( $cert, $certificate );
+				openssl_x509_export($cert, $certificate);
 
 				$data['_pronamic_gateway_ideal_private_certificate'] = $certificate;
 			}

src/Client.php

Spacing +55 added lines, -55 removed lines

@@ -122,7 +122,7 @@ 
 	 *
 	 * @param string $url
 	 */
-	public function set_acquirer_url( $url ) {
+	public function set_acquirer_url($url) {
 		$this->acquirer_url = $url;
 
 		$this->directory_request_url   = $url;
@@ -138,14 +138,14 @@ 
 	 *
 	 * @return ResponseMessage
 	 */
-	private function send_message( $url, RequestMessage $message ) {
+	private function send_message($url, RequestMessage $message) {
 		$result = false;
 
 		// Sign
 		$document = $message->get_document();
-		$document = $this->sign_document( $document );
+		$document = $this->sign_document($document);
 
-		if ( false !== $document ) {
+		if (false !== $document) {
 			// Stringify
 			$data = $document->saveXML();
 
@@ -154,32 +154,32 @@ 
 			 *
 			 * @since 1.1.11
 			 */
-			if ( 'https://www.ideal-checkout.nl/simulator/' === $url ) {
-				$data = $document->C14N( true, false );
+			if ('https://www.ideal-checkout.nl/simulator/' === $url) {
+				$data = $document->C14N(true, false);
 			}
 
 			// Remote post
-			$response = wp_remote_post( $url, array(
+			$response = wp_remote_post($url, array(
 				'method'  => 'POST',
 				'headers' => array(
 					'Content-Type' => 'text/xml; charset=' . Message::XML_ENCODING,
 				),
 				'body'    => $data,
-			) );
+			));
 
 			// Handle response
-			if ( ! is_wp_error( $response ) ) {
-				if ( 200 === wp_remote_retrieve_response_code( $response ) ) {
-					$body = wp_remote_retrieve_body( $response );
+			if ( ! is_wp_error($response)) {
+				if (200 === wp_remote_retrieve_response_code($response)) {
+					$body = wp_remote_retrieve_body($response);
 
-					$xml = Core_Util::simplexml_load_string( $body );
+					$xml = Core_Util::simplexml_load_string($body);
 
-					if ( is_wp_error( $xml ) ) {
+					if (is_wp_error($xml)) {
 						$this->error = $xml;
 					} else {
-						$document = self::parse_document( $xml );
+						$document = self::parse_document($xml);
 
-						if ( is_wp_error( $document ) ) {
+						if (is_wp_error($document)) {
 							$this->error = $document;
 						} else {
 							$result = $document;
@@ -190,8 +190,8 @@ 
 						'wrong_response_code',
 						sprintf(
 							/* translators: %s: response code */
-							__( 'The response code (<code>%s<code>) from the iDEAL provider was incorrect.', 'pronamic_ideal' ),
-							wp_remote_retrieve_response_code( $response )
+							__('The response code (<code>%s<code>) from the iDEAL provider was incorrect.', 'pronamic_ideal'),
+							wp_remote_retrieve_response_code($response)
 						)
 					);
 				}
@@ -210,33 +210,33 @@ 
 	 *
 	 * @return ResponseMessage|WP_Error
 	 */
-	private function parse_document( SimpleXMLElement $document ) {
+	private function parse_document(SimpleXMLElement $document) {
 		$this->error = null;
 
 		$name = $document->getName();
 
-		switch ( $name ) {
+		switch ($name) {
 			case AcquirerErrorResMessage::NAME:
-				$message = AcquirerErrorResMessage::parse( $document );
+				$message = AcquirerErrorResMessage::parse($document);
 
 				$this->error = new WP_Error(
 					'IDealAdvancedV3_error',
-					sprintf( '%s. %s', $message->error->get_message(), $message->error->get_detail() ),
+					sprintf('%s. %s', $message->error->get_message(), $message->error->get_detail()),
 					$message->error
 				);
 
 				return $message;
 			case DirectoryResponseMessage::NAME:
-				return DirectoryResponseMessage::parse( $document );
+				return DirectoryResponseMessage::parse($document);
 			case TransactionResponseMessage::NAME:
-				return TransactionResponseMessage::parse( $document );
+				return TransactionResponseMessage::parse($document);
 			case AcquirerStatusResMessage::NAME:
-				return AcquirerStatusResMessage::parse( $document );
+				return AcquirerStatusResMessage::parse($document);
 			default:
 				return new WP_Error(
 					'IDealAdvancedV3_error',
 					/* translators: %s: XML document element name */
-					sprintf( __( 'Unknwon iDEAL message (%s)', 'pronamic_ideal' ), $name )
+					sprintf(__('Unknwon iDEAL message (%s)', 'pronamic_ideal'), $name)
 				);
 		}
 	}
@@ -252,12 +252,12 @@ 
 		$request_dir_message = new DirectoryRequestMessage();
 
 		$merchant = $request_dir_message->get_merchant();
-		$merchant->set_id( $this->merchant_id );
-		$merchant->set_sub_id( $this->sub_id );
+		$merchant->set_id($this->merchant_id);
+		$merchant->set_sub_id($this->sub_id);
 
-		$response_dir_message = $this->send_message( $this->directory_request_url, $request_dir_message );
+		$response_dir_message = $this->send_message($this->directory_request_url, $request_dir_message);
 
-		if ( $response_dir_message instanceof DirectoryResponseMessage ) {
+		if ($response_dir_message instanceof DirectoryResponseMessage) {
 			$directory = $response_dir_message->get_directory();
 		}
 
@@ -272,20 +272,20 @@ 
 	 *
 	 * @return TransactionResponseMessage
 	 */
-	public function create_transaction( Transaction $transaction, $return_url, $issuer_id ) {
+	public function create_transaction(Transaction $transaction, $return_url, $issuer_id) {
 		$message = new TransactionRequestMessage();
 
 		$merchant = $message->get_merchant();
-		$merchant->set_id( $this->merchant_id );
-		$merchant->set_sub_id( $this->sub_id );
-		$merchant->set_return_url( $return_url );
+		$merchant->set_id($this->merchant_id);
+		$merchant->set_sub_id($this->sub_id);
+		$merchant->set_return_url($return_url);
 
 		$message->issuer = new Issuer();
-		$message->issuer->set_id( $issuer_id );
+		$message->issuer->set_id($issuer_id);
 
 		$message->transaction = $transaction;
 
-		return $this->send_message( $this->transaction_request_url, $message );
+		return $this->send_message($this->transaction_request_url, $message);
 	}
 
 	/**
@@ -295,17 +295,17 @@ 
 	 *
 	 * @return TransactionResponseMessage
 	 */
-	public function get_status( $transaction_id ) {
+	public function get_status($transaction_id) {
 		$message = new AcquirerStatusReqMessage();
 
 		$merchant = $message->get_merchant();
-		$merchant->set_id( $this->merchant_id );
-		$merchant->set_sub_id( $this->sub_id );
+		$merchant->set_id($this->merchant_id);
+		$merchant->set_sub_id($this->sub_id);
 
 		$message->transaction = new Transaction();
-		$message->transaction->set_id( $transaction_id );
+		$message->transaction->set_id($transaction_id);
 
-		return $this->send_message( $this->status_request_url, $message );
+		return $this->send_message($this->status_request_url, $message);
 	}
 
 	/**
@@ -317,7 +317,7 @@ 
 	 *
 	 * @return DOMDocument
 	 */
-	private function sign_document( DOMDocument $document ) {
+	private function sign_document(DOMDocument $document) {
 		$result = false;
 
 		try {
@@ -325,14 +325,14 @@ 
 
 			// For canonicalization purposes the exclusive (9) algorithm must be used.
 			// @link http://pronamic.nl/wp-content/uploads/2012/12/iDEAL-Merchant-Integration-Guide-ENG-v3.3.1.pdf #page 30
-			$dsig->setCanonicalMethod( XMLSecurityDSig::EXC_C14N );
+			$dsig->setCanonicalMethod(XMLSecurityDSig::EXC_C14N);
 
 			// For hashing purposes the SHA-256 (11) algorithm must be used.
 			// @link http://pronamic.nl/wp-content/uploads/2012/12/iDEAL-Merchant-Integration-Guide-ENG-v3.3.1.pdf #page 30
 			$dsig->addReference(
 				$document,
 				XMLSecurityDSig::SHA256,
-				array( 'http://www.w3.org/2000/09/xmldsig#enveloped-signature' ),
+				array('http://www.w3.org/2000/09/xmldsig#enveloped-signature'),
 				array(
 					'force_uri' => true,
 				)
@@ -340,39 +340,39 @@ 
 
 			// For signature purposes the RSAWithSHA 256 (12) algorithm must be used.
 			// @link http://pronamic.nl/wp-content/uploads/2012/12/iDEAL-Merchant-Integration-Guide-ENG-v3.3.1.pdf #page 31
-			$key = new XMLSecurityKey( XMLSecurityKey::RSA_SHA256, array(
+			$key = new XMLSecurityKey(XMLSecurityKey::RSA_SHA256, array(
 				'type' => 'private',
-			) );
+			));
 
 			$key->passphrase = $this->private_key_password;
 
-			$key->loadKey( $this->private_key );
+			$key->loadKey($this->private_key);
 
 			// Test if we can get an private key object, to prefent the following errors:
 			// Warning: openssl_sign() [function.openssl-sign]: supplied key param cannot be coerced into a private key
-			$result = openssl_get_privatekey( $this->private_key, $this->private_key_password );
+			$result = openssl_get_privatekey($this->private_key, $this->private_key_password);
 
-			if ( false !== $result ) {
+			if (false !== $result) {
 				// Sign
-				$dsig->sign( $key );
+				$dsig->sign($key);
 
 				// The public key must be referenced using a fingerprint of an X.509
 				// certificate. The fingerprint must be calculated according
 				// to the following formula HEX(SHA-1(DER certificate)) (13)
 				// @link http://pronamic.nl/wp-content/uploads/2012/12/iDEAL-Merchant-Integration-Guide-ENG-v3.3.1.pdf #page 31
-				$fingerprint = Security::get_sha_fingerprint( $this->private_certificate );
+				$fingerprint = Security::get_sha_fingerprint($this->private_certificate);
 
-				$dsig->addKeyInfoAndName( $fingerprint );
+				$dsig->addKeyInfoAndName($fingerprint);
 
 				// Add the signature
-				$dsig->appendSignature( $document->documentElement );
+				$dsig->appendSignature($document->documentElement);
 
 				$result = $document;
 			} else {
-				throw new Exception( 'Can not load private key' );
+				throw new Exception('Can not load private key');
 			}
-		} catch ( Exception $e ) {
-			$this->error = new WP_Error( 'xml_security', $e->getMessage(), $e );
+		} catch (Exception $e) {
+			$this->error = new WP_Error('xml_security', $e->getMessage(), $e);
 		}
 
 		return $result;

src/Gateway.php

Spacing +33 added lines, -33 removed lines

@@ -29,19 +29,19 @@ 
 	 *
 	 * @param Config $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( self::METHOD_HTTP_REDIRECT );
+		$this->set_method(self::METHOD_HTTP_REDIRECT);
 
 		// Client.
 		$client = new Client();
 
-		$client->set_acquirer_url( $config->get_payment_server_url() );
+		$client->set_acquirer_url($config->get_payment_server_url());
 
 		$client->merchant_id          = $config->merchant_id;
 		$client->sub_id               = $config->sub_id;
@@ -62,7 +62,7 @@ 
 	public function get_issuers() {
 		$directory = $this->client->get_directory();
 
-		if ( ! $directory ) {
+		if ( ! $directory) {
 			$this->error = $this->client->get_error();
 
 			return array();
@@ -70,11 +70,11 @@ 
 
 		$groups = array();
 
-		foreach ( $directory->get_countries() as $country ) {
+		foreach ($directory->get_countries() as $country) {
 			$issuers = array();
 
-			foreach ( $country->get_issuers() as $issuer ) {
-				$issuers[ $issuer->get_id() ] = $issuer->get_name();
+			foreach ($country->get_issuers() as $issuer) {
+				$issuers[$issuer->get_id()] = $issuer->get_name();
 			}
 
 			$groups[] = array(
@@ -114,38 +114,38 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
+	public function start(Payment $payment) {
 		// 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);
 
 		// Transaction.
 		$transaction = new Transaction();
-		$transaction->set_purchase_id( $purchase_id );
-		$transaction->set_amount( $payment->get_total_amount()->get_value() );
-		$transaction->set_currency( $payment->get_total_amount()->get_currency()->get_alphabetic_code() );
-		$transaction->set_expiration_period( 'PT30M' );
-		$transaction->set_description( $payment->get_description() );
-		$transaction->set_entrance_code( $payment->get_entrance_code() );
-
-		if ( null !== $payment->get_customer() ) {
-			$transaction->set_language( $payment->get_customer()->get_language() );
+		$transaction->set_purchase_id($purchase_id);
+		$transaction->set_amount($payment->get_total_amount()->get_value());
+		$transaction->set_currency($payment->get_total_amount()->get_currency()->get_alphabetic_code());
+		$transaction->set_expiration_period('PT30M');
+		$transaction->set_description($payment->get_description());
+		$transaction->set_entrance_code($payment->get_entrance_code());
+
+		if (null !== $payment->get_customer()) {
+			$transaction->set_language($payment->get_customer()->get_language());
 		}
 
 		// Create transaction.
-		$result = $this->client->create_transaction( $transaction, $payment->get_return_url(), $payment->get_issuer() );
+		$result = $this->client->create_transaction($transaction, $payment->get_return_url(), $payment->get_issuer());
 
 		$error = $this->client->get_error();
 
-		if ( is_wp_error( $error ) ) {
-			$this->set_error( $error );
+		if (is_wp_error($error)) {
+			$this->set_error($error);
 
 			return;
 		}
 
-		$payment->set_action_url( $result->issuer->get_authentication_url() );
-		$payment->set_transaction_id( $result->transaction->get_id() );
+		$payment->set_action_url($result->issuer->get_authentication_url());
+		$payment->set_transaction_id($result->transaction->get_id());
 	}
 
 	/**
@@ -153,20 +153,20 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
-		$result = $this->client->get_status( $payment->get_transaction_id() );
+	public function update_status(Payment $payment) {
+		$result = $this->client->get_status($payment->get_transaction_id());
 
 		$error = $this->client->get_error();
 
-		if ( is_wp_error( $error ) ) {
-			$this->set_error( $error );
+		if (is_wp_error($error)) {
+			$this->set_error($error);
 		} else {
 			$transaction = $result->transaction;
 
-			$payment->set_status( $transaction->get_status() );
-			$payment->set_consumer_name( $transaction->get_consumer_name() );
-			$payment->set_consumer_iban( $transaction->get_consumer_iban() );
-			$payment->set_consumer_bic( $transaction->get_consumer_bic() );
+			$payment->set_status($transaction->get_status());
+			$payment->set_consumer_name($transaction->get_consumer_name());
+			$payment->set_consumer_iban($transaction->get_consumer_iban());
+			$payment->set_consumer_bic($transaction->get_consumer_bic());
 		}
 	}
 }