wp-pay-gateways / ems-e-commerce

src/PaymentMethods.php

Spacing +2 added lines, -2 removed lines

@@ -80,8 +80,8 @@
 	 *
 	 * @return string
 	 */
-	public static function transform( $payment_method ) {
-		switch ( $payment_method ) {
+	public static function transform($payment_method) {
+		switch ($payment_method) {
 			case Core_PaymentMethods::BANCONTACT:
 				return self::BANCONTACT;
 

src/Settings.php

Spacing +22 added lines, -22 removed lines

@@ -16,37 +16,37 @@ 
  */
 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) {
 		// iDEAL
 		$sections['ems_ecommerce'] = array(
-			'title'       => __( 'EMS e-Commerce', 'pronamic_ideal' ),
-			'methods'     => array( 'ems_ecommerce' ),
-			'description' => __( 'Account details are provided by the payment provider after registration. These settings need to match with the payment provider dashboard.', 'pronamic_ideal' ),
+			'title'       => __('EMS e-Commerce', 'pronamic_ideal'),
+			'methods'     => array('ems_ecommerce'),
+			'description' => __('Account details are provided by the payment provider after registration. These settings need to match with the payment provider dashboard.', 'pronamic_ideal'),
 		);
 
 		// Advanced
 		$sections['ems_ecommerce_advanced'] = array(
-			'title'       => __( 'Advanced', 'pronamic_ideal' ),
-			'methods'     => array( 'ems_ecommerce' ),
-			'description' => __( 'Optional settings for advanced usage only.', 'pronamic_ideal' ),
+			'title'       => __('Advanced', 'pronamic_ideal'),
+			'methods'     => array('ems_ecommerce'),
+			'description' => __('Optional settings for advanced usage only.', 'pronamic_ideal'),
 		);
 
 		return $sections;
 	}
 
-	public function fields( array $fields ) {
+	public function fields(array $fields) {
 		// Storename
 		$fields[] = array(
 			'filter'   => FILTER_UNSAFE_RAW,
 			'section'  => 'ems_ecommerce',
 			'meta_key' => '_pronamic_gateway_ems_ecommerce_storename',
-			'title'    => _x( 'Storename', 'ems', 'pronamic_ideal' ),
+			'title'    => _x('Storename', 'ems', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'code' ),
+			'classes'  => array('code'),
 		);
 
 		// Shared secret
@@ -54,19 +54,19 @@ 
 			'filter'   => FILTER_UNSAFE_RAW,
 			'section'  => 'ems_ecommerce',
 			'meta_key' => '_pronamic_gateway_ems_ecommerce_secret',
-			'title'    => _x( 'Shared Secret', 'ems', 'pronamic_ideal' ),
+			'title'    => _x('Shared Secret', 'ems', 'pronamic_ideal'),
 			'type'     => 'text',
-			'classes'  => array( 'large-text', 'code' ),
+			'classes'  => array('large-text', 'code'),
 		);
 
 		// Transaction feedback
 		$fields[] = array(
 			'section' => 'ems_ecommerce',
-			'title'   => __( 'Transaction feedback', 'pronamic_ideal' ),
+			'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')
 			),
 		);
 
@@ -78,23 +78,23 @@ 
 			),
 			'section'     => 'ems_ecommerce_advanced',
 			'meta_key'    => '_pronamic_gateway_ems_ecommerce_order_id',
-			'title'       => __( 'Order ID', 'pronamic_ideal' ),
+			'title'       => __('Order ID', 'pronamic_ideal'),
 			'type'        => 'text',
-			'classes'     => array( 'regular-text', 'code' ),
+			'classes'     => array('regular-text', 'code'),
 			'tooltip'     => sprintf(
-				__( 'The EMS e-Commerce %s parameter.', 'pronamic_ideal' ),
-				sprintf( '<code>%s</code>', 'orderId' )
+				__('The EMS e-Commerce %s parameter.', 'pronamic_ideal'),
+				sprintf('<code>%s</code>', 'orderId')
 			),
 			'description' => sprintf(
 				'%s %s<br />%s',
-				__( 'Available tags:', 'pronamic_ideal' ),
+				__('Available tags:', 'pronamic_ideal'),
 				sprintf(
 					'<code>%s</code> <code>%s</code>',
 					'{order_id}',
 					'{payment_id}'
 				),
 				sprintf(
-					__( 'Default: <code>%s</code>', 'pronamic_ideal' ),
+					__('Default: <code>%s</code>', 'pronamic_ideal'),
 					'{order_id}'
 				)
 			),

src/Statuses.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
 	 *
 	 * @return null|string
 	 */
-	public static function transform( $status ) {
-		switch ( $status ) {
+	public static function transform($status) {
+		switch ($status) {
 			case self::OPEN :
 				return Core_Statuses::OPEN;
 

src/Integration.php

Spacing +6 added lines, -6 removed lines

@@ -20,16 +20,16 @@
 		$this->name          = 'EMS e-Commerce';
 		$this->product_url   = '';
 		$this->dashboard_url = array(
-			__( 'test', 'pronamic_ideal' ) => 'https://test.ipg-online.com/vt/login',
-			__( 'live', 'pronamic_ideal' ) => 'https://www.ipg-online.com/vt/login',
+			__('test', 'pronamic_ideal') => 'https://test.ipg-online.com/vt/login',
+			__('live', 'pronamic_ideal') => 'https://www.ipg-online.com/vt/login',
 		);
-		$this->provider      = 'ems';
+		$this->provider = 'ems';
 
 		// 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/Listener.php

Spacing +7 added lines, -7 removed lines

@@ -16,24 +16,24 @@
  */
 class Listener {
 	public static function listen() {
-		if ( ! filter_has_var( INPUT_POST, 'ems_notify_payment_id' ) ) {
+		if ( ! filter_has_var(INPUT_POST, 'ems_notify_payment_id')) {
 			return;
 		}
 
-		$payment_id = filter_input( INPUT_POST, 'ems_notify_payment_id' );
+		$payment_id = filter_input(INPUT_POST, 'ems_notify_payment_id');
 
-		$payment = get_pronamic_payment( $payment_id );
+		$payment = get_pronamic_payment($payment_id);
 
 		// Add note.
 		$note = sprintf(
 			/* translators: %s: EMS */
-			__( 'Webhook requested by %s.', 'pronamic_ideal' ),
-			__( 'EMS', 'pronamic_ideal' )
+			__('Webhook requested by %s.', 'pronamic_ideal'),
+			__('EMS', 'pronamic_ideal')
 		);
 
-		$payment->add_note( $note );
+		$payment->add_note($note);
 
 		// Update payment.
-		Plugin::update_payment( $payment );
+		Plugin::update_payment($payment);
 	}
 }

src/ConfigFactory.php

Spacing +5 added lines, -5 removed lines

@@ -15,13 +15,13 @@
  * @since 1.0.0
  */
 class ConfigFactory extends GatewayConfigFactory {
-	public function get_config( $post_id ) {
+	public function get_config($post_id) {
 		$config = new Config();
 
-		$config->storename = get_post_meta( $post_id, '_pronamic_gateway_ems_ecommerce_storename', true );
-		$config->secret    = get_post_meta( $post_id, '_pronamic_gateway_ems_ecommerce_secret', true );
-		$config->mode      = get_post_meta( $post_id, '_pronamic_gateway_mode', true );
-		$config->order_id  = get_post_meta( $post_id, '_pronamic_gateway_ems_ecommerce_order_id', true );
+		$config->storename = get_post_meta($post_id, '_pronamic_gateway_ems_ecommerce_storename', true);
+		$config->secret    = get_post_meta($post_id, '_pronamic_gateway_ems_ecommerce_secret', true);
+		$config->mode      = get_post_meta($post_id, '_pronamic_gateway_mode', true);
+		$config->order_id  = get_post_meta($post_id, '_pronamic_gateway_ems_ecommerce_order_id', true);
 
 		return $config;
 	}

src/Client.php

Spacing +24 added lines, -24 removed lines

@@ -156,7 +156,7 @@ 
 	 *
 	 * @param string $url Action URL.
 	 */
-	public function set_action_url( $url ) {
+	public function set_action_url($url) {
 		$this->action_url = $url;
 	}
 
@@ -174,7 +174,7 @@ 
 	 *
 	 * @param string $code Currency numeric code.
 	 */
-	public function set_currency_numeric_code( $code ) {
+	public function set_currency_numeric_code($code) {
 		$this->currency_numeric_code = $code;
 	}
 
@@ -192,7 +192,7 @@ 
 	 *
 	 * @param string $storename Storename.
 	 */
-	public function set_storename( $storename ) {
+	public function set_storename($storename) {
 		$this->storename = $storename;
 	}
 
@@ -212,7 +212,7 @@ 
 	 *
 	 * @param string $return_url Return URL.
 	 */
-	public function set_return_url( $return_url ) {
+	public function set_return_url($return_url) {
 		$this->return_url = $return_url;
 	}
 
@@ -230,7 +230,7 @@ 
 	 *
 	 * @param float $amount Amount.
 	 */
-	public function set_amount( $amount ) {
+	public function set_amount($amount) {
 		$this->amount = $amount;
 	}
 
@@ -248,7 +248,7 @@ 
 	 *
 	 * @param string $notification_url Notification URL.
 	 */
-	public function set_notification_url( $notification_url ) {
+	public function set_notification_url($notification_url) {
 		$this->notification_url = $notification_url;
 	}
 
@@ -266,7 +266,7 @@ 
 	 *
 	 * @param string $language Language.
 	 */
-	public function set_language( $language ) {
+	public function set_language($language) {
 		$this->language = $language;
 	}
 
@@ -275,7 +275,7 @@ 
 	 *
 	 * @param string $payment_method Payment method.
 	 */
-	public function set_payment_method( $payment_method ) {
+	public function set_payment_method($payment_method) {
 		$this->payment_method = $payment_method;
 	}
 
@@ -302,7 +302,7 @@ 
 	 *
 	 * @param string $order_id Order ID.
 	 */
-	public function set_order_id( $order_id ) {
+	public function set_order_id($order_id) {
 		$this->order_id = $order_id;
 	}
 
@@ -320,7 +320,7 @@ 
 	 *
 	 * @param int $payment_id Payment ID.
 	 */
-	public function set_payment_id( $payment_id ) {
+	public function set_payment_id($payment_id) {
 		$this->payment_id = $payment_id;
 	}
 
@@ -331,9 +331,9 @@ 
 	 *
 	 * @return DateTime
 	 */
-	public function get_transaction_datetime( $create_new = false ) {
-		if ( null === $this->transaction_datetime || $create_new ) {
-			$this->transaction_datetime = new DateTime( null, new DateTimeZone( 'UTC' ) );
+	public function get_transaction_datetime($create_new = false) {
+		if (null === $this->transaction_datetime || $create_new) {
+			$this->transaction_datetime = new DateTime(null, new DateTimeZone('UTC'));
 		}
 
 		return $this->transaction_datetime;
@@ -344,7 +344,7 @@ 
 	 *
 	 * @param DateTime $datetime Transaction date time.
 	 */
-	public function set_transaction_datetime( DateTime $datetime ) {
+	public function set_transaction_datetime(DateTime $datetime) {
 		$this->transaction_datetime = $datetime;
 	}
 
@@ -360,11 +360,11 @@ 
 			// According the EMS documentation the timezone should be in `Area/Location` notation, but it seems like `UTC` is also working.
 			'timezone'       => 'UTC',
 			// In WordPress, PHP's `time()` will always return `UTC` and is the same as calling `current_time( 'timestamp', true )`.
-			'txndatetime'    => $this->get_transaction_datetime()->format( 'Y:m:d-H:i:s' ),
+			'txndatetime'    => $this->get_transaction_datetime()->format('Y:m:d-H:i:s'),
 			'hash_algorithm' => 'SHA256',
 			'storename'      => $this->get_storename(),
 			'mode'           => 'payonly',
-			'chargetotal'    => number_format( ( $this->get_amount() / 100 ), 2, '.', '' ),
+			'chargetotal'    => number_format(($this->get_amount() / 100), 2, '.', ''),
 			'currency'       => $this->get_currency_numeric_code(),
 		);
 
@@ -381,7 +381,7 @@ 
 		);
 
 		// @link http://briancray.com/2009/04/25/remove-null-values-php-arrays/
-		$optional_fields = array_filter( $optional_fields );
+		$optional_fields = array_filter($optional_fields);
 
 		// Data.
 		$data = $required_fields + $optional_fields;
@@ -405,7 +405,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function set_secret( $secret ) {
+	public function set_secret($secret) {
 		$this->secret = $secret;
 	}
 
@@ -426,7 +426,7 @@ 
 			$secret,
 		);
 
-		return self::compute_hash( $values );
+		return self::compute_hash($values);
 	}
 
 	/**
@@ -436,11 +436,11 @@ 
 	 *
 	 * @return string
 	 */
-	public static function compute_hash( $values ) {
-		$value = implode( '', $values );
-		$value = bin2hex( $value );
+	public static function compute_hash($values) {
+		$value = implode('', $values);
+		$value = bin2hex($value);
 
-		return hash( self::HASH_ALGORITHM_SHA256, $value );
+		return hash(self::HASH_ALGORITHM_SHA256, $value);
 	}
 
 	/**
@@ -462,7 +462,7 @@ 
 	 *
 	 * @param string $issuer_id Issuer ID.
 	 */
-	public function set_issuer_id( $issuer_id ) {
+	public function set_issuer_id($issuer_id) {
 		$this->issuer_id = $issuer_id;
 	}
 

src/Gateway.php

Spacing +59 added lines, -59 removed lines

@@ -30,23 +30,23 @@ 
 	 *
 	 * @param Config $config Config.
 	 */
-	public function __construct( Config $config ) {
-		parent::__construct( $config );
+	public function __construct(Config $config) {
+		parent::__construct($config);
 
-		$this->set_method( self::METHOD_HTML_FORM );
+		$this->set_method(self::METHOD_HTML_FORM);
 
 		// Client.
 		$this->client = new Client();
 
 		$action_url = Client::ACTION_URL_PRODUCTION;
 
-		if ( self::MODE_TEST === $config->mode ) {
+		if (self::MODE_TEST === $config->mode) {
 			$action_url = Client::ACTION_URL_TEST;
 		}
 
-		$this->client->set_action_url( $action_url );
-		$this->client->set_storename( $config->storename );
-		$this->client->set_secret( $config->secret );
+		$this->client->set_action_url($action_url);
+		$this->client->set_storename($config->storename);
+		$this->client->set_secret($config->secret);
 	}
 
 	/**
@@ -70,31 +70,31 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function start( Payment $payment ) {
-		$payment->set_action_url( $this->client->get_action_url() );
+	public function start(Payment $payment) {
+		$payment->set_action_url($this->client->get_action_url());
 
-		$this->client->set_payment_id( $payment->get_id() );
-		$this->client->set_currency_numeric_code( $payment->get_total_amount()->get_currency()->get_numeric_code() );
-		$this->client->set_order_id( $payment->format_string( $this->config->order_id ) );
-		$this->client->set_return_url( home_url( '/' ) );
-		$this->client->set_notification_url( home_url( '/' ) );
-		$this->client->set_amount( $payment->get_total_amount()->get_cents() );
-		$this->client->set_issuer_id( $payment->get_issuer() );
+		$this->client->set_payment_id($payment->get_id());
+		$this->client->set_currency_numeric_code($payment->get_total_amount()->get_currency()->get_numeric_code());
+		$this->client->set_order_id($payment->format_string($this->config->order_id));
+		$this->client->set_return_url(home_url('/'));
+		$this->client->set_notification_url(home_url('/'));
+		$this->client->set_amount($payment->get_total_amount()->get_cents());
+		$this->client->set_issuer_id($payment->get_issuer());
 
 		// Language.
-		if ( null !== $payment->get_customer() ) {
-			$this->client->set_language( $payment->get_customer()->get_locale() );
+		if (null !== $payment->get_customer()) {
+			$this->client->set_language($payment->get_customer()->get_locale());
 		}
 
 		// Payment method.
-		$payment_method = PaymentMethods::transform( $payment->get_method() );
+		$payment_method = PaymentMethods::transform($payment->get_method());
 
-		if ( null === $payment_method && '' !== $payment->get_method() ) {
+		if (null === $payment_method && '' !== $payment->get_method()) {
 			// Leap of faith if the WordPress payment method could not transform to a EMS method?
 			$payment_method = $payment->get_method();
 		}
 
-		$this->client->set_payment_method( $payment_method );
+		$this->client->set_payment_method($payment_method);
 	}
 
 	/**
@@ -112,40 +112,40 @@ 
 	 *
 	 * @param Payment $payment Payment.
 	 */
-	public function update_status( Payment $payment ) {
-		$approval_code = filter_input( INPUT_POST, 'approval_code', FILTER_SANITIZE_STRING );
+	public function update_status(Payment $payment) {
+		$approval_code = filter_input(INPUT_POST, 'approval_code', FILTER_SANITIZE_STRING);
 
-		$input_hash = filter_input( INPUT_POST, 'response_hash' );
+		$input_hash = filter_input(INPUT_POST, 'response_hash');
 
 		$hash_values = array(
 			$this->client->get_secret(),
 			$approval_code,
-			filter_input( INPUT_POST, 'chargetotal', FILTER_SANITIZE_STRING ),
-			filter_input( INPUT_POST, 'currency', FILTER_SANITIZE_STRING ),
-			filter_input( INPUT_POST, 'txndatetime', FILTER_SANITIZE_STRING ),
+			filter_input(INPUT_POST, 'chargetotal', FILTER_SANITIZE_STRING),
+			filter_input(INPUT_POST, 'currency', FILTER_SANITIZE_STRING),
+			filter_input(INPUT_POST, 'txndatetime', FILTER_SANITIZE_STRING),
 			$this->client->get_storename(),
 		);
 
-		if ( filter_has_var( INPUT_POST, 'notification_hash' ) ) {
-			$input_hash = filter_input( INPUT_POST, 'notification_hash' );
+		if (filter_has_var(INPUT_POST, 'notification_hash')) {
+			$input_hash = filter_input(INPUT_POST, 'notification_hash');
 
 			$hash_values = array(
-				filter_input( INPUT_POST, 'chargetotal', FILTER_SANITIZE_STRING ),
+				filter_input(INPUT_POST, 'chargetotal', FILTER_SANITIZE_STRING),
 				$this->client->get_secret(),
-				filter_input( INPUT_POST, 'currency', FILTER_SANITIZE_STRING ),
-				filter_input( INPUT_POST, 'txndatetime', FILTER_SANITIZE_STRING ),
+				filter_input(INPUT_POST, 'currency', FILTER_SANITIZE_STRING),
+				filter_input(INPUT_POST, 'txndatetime', FILTER_SANITIZE_STRING),
 				$this->client->get_storename(),
 				$approval_code,
 			);
 		}
 
-		$hash = Client::compute_hash( $hash_values );
+		$hash = Client::compute_hash($hash_values);
 
 		// Check if the posted hash is equal to the calculated response or notification hash.
-		if ( 0 === strcasecmp( $input_hash, $hash ) ) {
-			$response_code = substr( $approval_code, 0, 1 );
+		if (0 === strcasecmp($input_hash, $hash)) {
+			$response_code = substr($approval_code, 0, 1);
 
-			switch ( $response_code ) {
+			switch ($response_code) {
 				case 'Y':
 					$status = Statuses::SUCCESS;
 
@@ -153,9 +153,9 @@ 
 				case 'N':
 					$status = Statuses::FAILURE;
 
-					$fail_code = filter_input( INPUT_POST, 'fail_rc', FILTER_SANITIZE_NUMBER_INT );
+					$fail_code = filter_input(INPUT_POST, 'fail_rc', FILTER_SANITIZE_NUMBER_INT);
 
-					if ( '5993' === $fail_code ) {
+					if ('5993' === $fail_code) {
 						$status = Statuses::CANCELLED;
 					}
 
@@ -168,43 +168,43 @@ 
 			}
 
 			// Set the status of the payment.
-			$payment->set_status( $status );
+			$payment->set_status($status);
 
 			$labels = array(
-				'approval_code'           => __( 'Approval code', 'pronamic_ideal' ),
-				'oid'                     => __( 'Order ID', 'pronamic_ideal' ),
-				'refnumber'               => _x( 'Reference number', 'creditcard', 'pronamic_ideal' ),
-				'status'                  => __( 'Status', 'pronamic_ideal' ),
-				'txndate_processed'       => __( 'Time of transaction processing', 'pronamic_ideal' ),
-				'tdate'                   => __( 'Identification for transaction', 'pronamic_ideal' ),
-				'fail_reason'             => __( 'Fail reason', 'pronamic_ideal' ),
-				'response_hash'           => __( 'Response hash', 'pronamic_ideal' ),
-				'processor_response_code' => __( 'Processor response code', 'pronamic_ideal' ),
-				'fail_rc'                 => __( 'Fail code', 'pronamic_ideal' ),
-				'terminal_id'             => __( 'Terminal ID', 'pronamic_ideal' ),
-				'ccbin'                   => __( 'Creditcard issuing bank', 'pronamic_ideal' ),
-				'cccountry'               => __( 'Creditcard country', 'pronamic_ideal' ),
-				'ccbrand'                 => __( 'Creditcard brand', 'pronamic_ideal' ),
+				'approval_code'           => __('Approval code', 'pronamic_ideal'),
+				'oid'                     => __('Order ID', 'pronamic_ideal'),
+				'refnumber'               => _x('Reference number', 'creditcard', 'pronamic_ideal'),
+				'status'                  => __('Status', 'pronamic_ideal'),
+				'txndate_processed'       => __('Time of transaction processing', 'pronamic_ideal'),
+				'tdate'                   => __('Identification for transaction', 'pronamic_ideal'),
+				'fail_reason'             => __('Fail reason', 'pronamic_ideal'),
+				'response_hash'           => __('Response hash', 'pronamic_ideal'),
+				'processor_response_code' => __('Processor response code', 'pronamic_ideal'),
+				'fail_rc'                 => __('Fail code', 'pronamic_ideal'),
+				'terminal_id'             => __('Terminal ID', 'pronamic_ideal'),
+				'ccbin'                   => __('Creditcard issuing bank', 'pronamic_ideal'),
+				'cccountry'               => __('Creditcard country', 'pronamic_ideal'),
+				'ccbrand'                 => __('Creditcard brand', 'pronamic_ideal'),
 			);
 
 			$note = '';
 
 			$note .= '<p>';
-			$note .= __( 'EMS e-Commerce transaction data in response message:', 'pronamic_ideal' );
+			$note .= __('EMS e-Commerce transaction data in response message:', 'pronamic_ideal');
 			$note .= '</p>';
 
 			$note .= '<dl>';
 
-			foreach ( $labels as $key => $label ) {
-				if ( filter_has_var( INPUT_POST, $key ) ) {
-					$note .= sprintf( '<dt>%s</dt>', esc_html( $label ) );
-					$note .= sprintf( '<dd>%s</dd>', esc_html( filter_input( INPUT_POST, $key, FILTER_SANITIZE_STRING ) ) );
+			foreach ($labels as $key => $label) {
+				if (filter_has_var(INPUT_POST, $key)) {
+					$note .= sprintf('<dt>%s</dt>', esc_html($label));
+					$note .= sprintf('<dd>%s</dd>', esc_html(filter_input(INPUT_POST, $key, FILTER_SANITIZE_STRING)));
 				}
 			}
 
 			$note .= '</dl>';
 
-			$payment->add_note( $note );
+			$payment->add_note($note);
 		}
 	}
 }