wp-pay-gateways / sisow

src/Util.php

Spacing +4 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
 	 * $            Dollar
 	 */
 	public static function get_pattern() {
-		if ( null === self::$pattern ) {
+		if (null === self::$pattern) {
 			$characters = array(
 				'A-Z',
 				'a-z',
@@ -90,7 +90,7 @@ 
 			 * We use a # as a regex delimiter instead of a / so we don't have to escape the slash.
 			 * @see http://stackoverflow.com/q/12239424
 			 */
-			self::$pattern = '#[^' . implode( $characters ) . ']#';
+			self::$pattern = '#[^' . implode($characters) . ']#';
 		}
 
 		return self::$pattern;
@@ -102,7 +102,7 @@ 
 	 * @param string $string String to filter.
 	 * @return mixed
 	 */
-	public static function filter( $string ) {
-		return preg_replace( self::get_pattern(), '', $string );
+	public static function filter($string) {
+		return preg_replace(self::get_pattern(), '', $string);
 	}
 }

src/Error.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	 * @param string $code    Code.
 	 * @param string $message Message.
 	 */
-	public function __construct( $code, $message ) {
+	public function __construct($code, $message) {
 		$this->code    = $code;
 		$this->message = $message;
 	}

src/Methods.php

Spacing +4 added lines, -4 removed lines

@@ -192,13 +192,13 @@
 	 * @param string|null $default        Value to return if method could not be transformed.
 	 * @return string|null
 	 */
-	public static function transform( $payment_method, $default = null ) {
-		if ( ! is_scalar( $payment_method ) ) {
+	public static function transform($payment_method, $default = null) {
+		if ( ! is_scalar($payment_method)) {
 			return null;
 		}
 
-		if ( isset( self::$map[ $payment_method ] ) ) {
-			return self::$map[ $payment_method ];
+		if (isset(self::$map[$payment_method])) {
+			return self::$map[$payment_method];
 		}
 
 		return $default;

src/XML/Parser.php

Spacing +1 added lines, -1 removed lines

@@ -28,5 +28,5 @@
 	 *
 	 * @param SimpleXMLElement $xml XML element to parse.
 	 */
-	public static function parse( SimpleXMLElement $xml );
+	public static function parse(SimpleXMLElement $xml);
 }

src/XML/TransactionParser.php

Spacing +25 added lines, -25 removed lines

@@ -33,57 +33,57 @@
 	 * @param SimpleXMLElement $xml XML element to parse.
 	 * @return Transaction
 	 */
-	public static function parse( SimpleXMLElement $xml ) {
+	public static function parse(SimpleXMLElement $xml) {
 		$transaction = new Transaction();
 
 		// Transaction request.
-		if ( isset( $xml->trxid ) ) {
-			$transaction->id = Security::filter( $xml->trxid );
+		if (isset($xml->trxid)) {
+			$transaction->id = Security::filter($xml->trxid);
 		}
 
-		if ( isset( $xml->issuerurl ) ) {
-			$transaction->issuer_url = urldecode( Security::filter( $xml->issuerurl ) );
+		if (isset($xml->issuerurl)) {
+			$transaction->issuer_url = urldecode(Security::filter($xml->issuerurl));
 		}
 
 		// Status response.
-		if ( isset( $xml->status ) ) {
-			$transaction->status = Security::filter( $xml->status );
+		if (isset($xml->status)) {
+			$transaction->status = Security::filter($xml->status);
 		}
 
-		if ( isset( $xml->amount ) ) {
-			$transaction->amount = Pay_Util::cents_to_amount( Security::filter( $xml->amount ) );
+		if (isset($xml->amount)) {
+			$transaction->amount = Pay_Util::cents_to_amount(Security::filter($xml->amount));
 		}
 
-		if ( isset( $xml->purchaseid ) ) {
-			$transaction->purchase_id = Security::filter( $xml->purchaseid );
+		if (isset($xml->purchaseid)) {
+			$transaction->purchase_id = Security::filter($xml->purchaseid);
 		}
 
-		if ( isset( $xml->description ) ) {
-			$transaction->description = Security::filter( $xml->description );
+		if (isset($xml->description)) {
+			$transaction->description = Security::filter($xml->description);
 		}
 
-		if ( isset( $xml->entrancecode ) ) {
-			$transaction->entrance_code = Security::filter( $xml->entrancecode );
+		if (isset($xml->entrancecode)) {
+			$transaction->entrance_code = Security::filter($xml->entrancecode);
 		}
 
-		if ( isset( $xml->issuerid ) ) {
-			$transaction->issuer_id = Security::filter( $xml->issuerid );
+		if (isset($xml->issuerid)) {
+			$transaction->issuer_id = Security::filter($xml->issuerid);
 		}
 
-		if ( isset( $xml->timestamp ) ) {
-			$transaction->timestamp = new DateTime( Security::filter( $xml->timestamp ) );
+		if (isset($xml->timestamp)) {
+			$transaction->timestamp = new DateTime(Security::filter($xml->timestamp));
 		}
 
-		if ( isset( $xml->consumername ) ) {
-			$transaction->consumer_name = Security::filter( $xml->consumername );
+		if (isset($xml->consumername)) {
+			$transaction->consumer_name = Security::filter($xml->consumername);
 		}
 
-		if ( isset( $xml->consumeraccount ) ) {
-			$transaction->consumer_account = Security::filter( $xml->consumeraccount );
+		if (isset($xml->consumeraccount)) {
+			$transaction->consumer_account = Security::filter($xml->consumeraccount);
 		}
 
-		if ( isset( $xml->consumercity ) ) {
-			$transaction->consumer_city = Security::filter( $xml->consumercity );
+		if (isset($xml->consumercity)) {
+			$transaction->consumer_city = Security::filter($xml->consumercity);
 		}
 
 		return $transaction;

src/XML/ErrorParser.php

Spacing +3 added lines, -3 removed lines

@@ -30,10 +30,10 @@
 	 *
 	 * @param SimpleXMLElement $xml XML element to parse.
 	 */
-	public static function parse( SimpleXMLElement $xml ) {
+	public static function parse(SimpleXMLElement $xml) {
 		$error = new Sisow_Error(
-			Security::filter( $xml->errorcode ),
-			Security::filter( $xml->errormessage )
+			Security::filter($xml->errorcode),
+			Security::filter($xml->errormessage)
 		);
 
 		return $error;

src/Settings.php

Spacing +24 added lines, -24 removed lines

@@ -27,8 +27,8 @@ 
 	 * Construct settings.
 	 */
 	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'));
 	}
 
 	/**
@@ -37,15 +37,15 @@ 
 	 * @param array $sections Sections.
 	 * @return array
 	 */
-	public function sections( array $sections ) {
+	public function sections(array $sections) {
 		// Sisow.
 		$sections['sisow'] = array(
-			'title'       => __( 'Sisow', 'pronamic_ideal' ),
-			'methods'     => array( 'sisow' ),
+			'title'       => __('Sisow', 'pronamic_ideal'),
+			'methods'     => array('sisow'),
 			'description' => sprintf(
 				/* translators: %s: Sisow */
-				__( 'Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal' ),
-				__( 'Sisow', 'pronamic_ideal' )
+				__('Account details are provided by %1$s after registration. These settings need to match with the %1$s dashboard.', 'pronamic_ideal'),
+				__('Sisow', 'pronamic_ideal')
 			),
 		);
 
@@ -58,55 +58,55 @@ 
 	 * @param array $fields Fields.
 	 * @return array
 	 */
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		// Merchant ID.
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'sisow',
-			'methods'  => array( 'sisow' ),
+			'methods'  => array('sisow'),
 			'meta_key' => '_pronamic_gateway_sisow_merchant_id',
-			'title'    => _x( 'Merchant ID', 'sisow', 'pronamic_ideal' ),
+			'title'    => _x('Merchant ID', 'sisow', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'code' ),
-			'tooltip'  => __( 'Merchant ID as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal' ),
+			'classes'  => array('code'),
+			'tooltip'  => __('Merchant ID as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal'),
 		);
 
 		// Merchant Key.
 		$fields[] = array(
 			'filter'   => FILTER_SANITIZE_STRING,
 			'section'  => 'sisow',
-			'methods'  => array( 'sisow' ),
+			'methods'  => array('sisow'),
 			'meta_key' => '_pronamic_gateway_sisow_merchant_key',
-			'title'    => _x( 'Merchant Key', 'sisow', 'pronamic_ideal' ),
+			'title'    => _x('Merchant Key', 'sisow', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'regular-text', 'code' ),
-			'tooltip'  => __( 'Merchant Key as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal' ),
+			'classes'  => array('regular-text', 'code'),
+			'tooltip'  => __('Merchant Key as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal'),
 		);
 
 		// Shop ID.
 		$fields[] = array(
 			'filter'      => FILTER_SANITIZE_STRING,
 			'section'     => 'sisow',
-			'methods'     => array( 'sisow' ),
+			'methods'     => array('sisow'),
 			'meta_key'    => '_pronamic_gateway_sisow_shop_id',
-			'title'       => _x( 'Shop ID', 'sisow', 'pronamic_ideal' ),
+			'title'       => _x('Shop ID', 'sisow', 'pronamic_ideal'),
 			'type'        => 'text',
-			'classes'     => array( 'regular-text', 'code' ),
-			'tooltip'     => __( 'Shop ID as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal' ),
+			'classes'     => array('regular-text', 'code'),
+			'tooltip'     => __('Shop ID as mentioned at <strong>My Profile</strong> in the Sisow dashboard.', 'pronamic_ideal'),
 			/* translators: %s: 0 */
-			'description' => sprintf( __( 'Default: <code>%s</code>', 'pronamic_ideal' ), 0 ),
+			'description' => sprintf(__('Default: <code>%s</code>', 'pronamic_ideal'), 0),
 			'default'     => 0,
 		);
 
 		// Transaction feedback.
 		$fields[] = array(
 			'section' => 'sisow',
-			'methods' => array( 'sisow' ),
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'methods' => array('sisow'),
+			'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')
 			),
 		);
 

src/ConfigFactory.php

Spacing +5 added lines, -5 removed lines

@@ -29,13 +29,13 @@
 	 * @param int $post_id Post ID.
 	 * @return Config
 	 */
-	public function get_config( $post_id ) {
+	public function get_config($post_id) {
 		$config = new Config();
 
-		$config->merchant_id  = $this->get_meta( $post_id, 'sisow_merchant_id' );
-		$config->merchant_key = $this->get_meta( $post_id, 'sisow_merchant_key' );
-		$config->shop_id      = $this->get_meta( $post_id, 'sisow_shop_id' );
-		$config->mode         = $this->get_meta( $post_id, 'mode' );
+		$config->merchant_id  = $this->get_meta($post_id, 'sisow_merchant_id');
+		$config->merchant_key = $this->get_meta($post_id, 'sisow_merchant_key');
+		$config->shop_id      = $this->get_meta($post_id, 'sisow_shop_id');
+		$config->mode         = $this->get_meta($post_id, 'mode');
 
 		return $config;
 	}

src/Gateway.php

Spacing +42 added lines, -42 removed lines

@@ -37,17 +37,17 @@ 
 	 *
 	 * @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);
 
-		$this->client = new Client( $config->merchant_id, $config->merchant_key );
-		$this->client->set_test_mode( self::MODE_TEST === $config->mode );
+		$this->client = new Client($config->merchant_id, $config->merchant_key);
+		$this->client->set_test_mode(self::MODE_TEST === $config->mode);
 	}
 
 	/**
@@ -60,7 +60,7 @@ 
 
 		$result = $this->client->get_directory();
 
-		if ( $result ) {
+		if ($result) {
 			$groups[] = array(
 				'options' => $result,
 			);
@@ -111,14 +111,14 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
+	public function start(Payment $payment) {
 		// Order and purchase ID.
 		$order_id    = $payment->get_order_id();
-		$purchase_id = strval( empty( $order_id ) ? $payment->get_id() : $order_id );
+		$purchase_id = strval(empty($order_id) ? $payment->get_id() : $order_id);
 
 		// Maximum length for purchase ID is 16 characters, otherwise an error will occur:
 		// ideal_sisow_error - purchaseid too long (16).
-		$purchase_id = substr( $purchase_id, 0, 16 );
+		$purchase_id = substr($purchase_id, 0, 16);
 
 		// New transaction request.
 		$request = new TransactionRequest(
@@ -128,12 +128,12 @@ 
 
 		$request->merge_parameters(
 			array(
-				'payment'      => Methods::transform( $payment->get_method(), $payment->get_method() ),
-				'purchaseid'   => substr( $purchase_id, 0, 16 ),
+				'payment'      => Methods::transform($payment->get_method(), $payment->get_method()),
+				'purchaseid'   => substr($purchase_id, 0, 16),
 				'entrancecode' => $payment->get_entrance_code(),
 				'amount'       => $payment->get_amount()->get_cents(),
-				'description'  => substr( $payment->get_description(), 0, 32 ),
-				'testmode'     => ( self::MODE_TEST === $this->config->mode ) ? 'true' : 'false',
+				'description'  => substr($payment->get_description(), 0, 32),
+				'testmode'     => (self::MODE_TEST === $this->config->mode) ? 'true' : 'false',
 				'returnurl'    => $payment->get_return_url(),
 				'cancelurl'    => $payment->get_return_url(),
 				'notifyurl'    => $payment->get_return_url(),
@@ -145,15 +145,15 @@ 
 		);
 
 		// Payment method.
-		$this->set_payment_method( null === $payment->get_method() ? PaymentMethods::IDEAL : $payment->get_method() );
+		$this->set_payment_method(null === $payment->get_method() ? PaymentMethods::IDEAL : $payment->get_method());
 
 		// Additional parameters for payment method.
-		if ( PaymentMethods::IDEALQR === $payment->get_method() ) {
-			$request->set_parameter( 'qrcode', 'true' );
+		if (PaymentMethods::IDEALQR === $payment->get_method()) {
+			$request->set_parameter('qrcode', 'true');
 		}
 
 		// Customer.
-		if ( null !== $payment->get_customer() ) {
+		if (null !== $payment->get_customer()) {
 			$customer = $payment->get_customer();
 
 			$request->merge_parameters(
@@ -164,16 +164,16 @@ 
 				)
 			);
 
-			if ( null !== $customer->get_birth_date() ) {
-				$request->set_parameter( 'birth_date', $customer->get_birth_date()->format( 'ddmmYYYY' ) );
+			if (null !== $customer->get_birth_date()) {
+				$request->set_parameter('birth_date', $customer->get_birth_date()->format('ddmmYYYY'));
 			}
 		}
 
 		// Billing address.
-		if ( null !== $payment->get_billing_address() ) {
+		if (null !== $payment->get_billing_address()) {
 			$address = $payment->get_billing_address();
 
-			if ( null !== $address->get_name() ) {
+			if (null !== $address->get_name()) {
 				$name = $address->get_name();
 
 				$request->merge_parameters(
@@ -201,10 +201,10 @@ 
 		}
 
 		// Shipping address.
-		if ( null !== $payment->get_shipping_address() ) {
+		if (null !== $payment->get_shipping_address()) {
 			$address = $payment->get_shipping_address();
 
-			if ( null !== $address->get_name() ) {
+			if (null !== $address->get_name()) {
 				$name = $address->get_name();
 
 				$request->merge_parameters(
@@ -231,16 +231,16 @@ 
 		}
 
 		// Lines.
-		if ( null !== $payment->get_lines() ) {
+		if (null !== $payment->get_lines()) {
 			$lines = $payment->get_lines();
 
 			$x = 1;
 
-			foreach ( $lines as $line ) {
-				$net_price = ( null === $line->get_unit_price() ) ? null : $line->get_unit_price()->get_cents();
-				$total     = ( null === $line->get_total_amount() ) ? null : $line->get_total_amount()->get_cents();
-				$tax       = ( null === $line->get_tax_amount() ) ? null : $line->get_tax_amount()->get_cents();
-				$net_total = ( $total - $tax );
+			foreach ($lines as $line) {
+				$net_price = (null === $line->get_unit_price()) ? null : $line->get_unit_price()->get_cents();
+				$total     = (null === $line->get_total_amount()) ? null : $line->get_total_amount()->get_cents();
+				$tax       = (null === $line->get_tax_amount()) ? null : $line->get_tax_amount()->get_cents();
+				$net_total = ($total - $tax);
 
 				$request->merge_parameters(
 					array(
@@ -260,11 +260,11 @@ 
 		}
 
 		// Create transaction.
-		$result = $this->client->create_transaction( $request );
+		$result = $this->client->create_transaction($request);
 
-		if ( false !== $result ) {
-			$payment->set_transaction_id( $result->id );
-			$payment->set_action_url( $result->issuer_url );
+		if (false !== $result) {
+			$payment->set_transaction_id($result->id);
+			$payment->set_action_url($result->issuer_url);
 		} else {
 			$this->error = $this->client->get_error();
 		}
@@ -275,34 +275,34 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
+	public function update_status(Payment $payment) {
 		$request = new StatusRequest(
 			$payment->get_transaction_id(),
 			$this->config->merchant_id,
 			$this->config->shop_id
 		);
 
-		$result = $this->client->get_status( $request );
+		$result = $this->client->get_status($request);
 
-		if ( $result instanceof Error ) {
+		if ($result instanceof Error) {
 			$this->error = $this->client->get_error();
 
 			return;
 		}
 
-		if ( false === $result ) {
+		if (false === $result) {
 			$this->error = $this->client->get_error();
 
 			return;
 		}
 
-		if ( $result instanceof Transaction ) {
+		if ($result instanceof Transaction) {
 			$transaction = $result;
 
-			$payment->set_status( $transaction->status );
-			$payment->set_consumer_name( $transaction->consumer_name );
-			$payment->set_consumer_account_number( $transaction->consumer_account );
-			$payment->set_consumer_city( $transaction->consumer_city );
+			$payment->set_status($transaction->status);
+			$payment->set_consumer_name($transaction->consumer_name);
+			$payment->set_consumer_account_number($transaction->consumer_account);
+			$payment->set_consumer_city($transaction->consumer_city);
 		}
 	}
 }