Client::get_error() - Code Metrics - Inspection of "Updated packages." - wp-pay-gateways/targetpay - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — develop ( 9671b5...d3774b )

by Remco

created 2019-08-28 12:02 UTC

Client::get_error() A

↳ Parent: Client

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	2
ccs	0
cts	2
cp	0
crap	2
rs	10
c	0
b	0
f	0

<?php

namespace Pronamic\WordPress\Pay\Gateways\TargetPay;

use Pronamic\WordPress\Pay\Core\Util;
use Pronamic\WordPress\Pay\Core\XML\Security;
use stdClass;
use WP_Error;

/**
 * Title: TargetPay gateway
 * Description:
 * Copyright: 2005-2019 Pronamic
 * Company: Pronamic
 *
 * @author  Remco Tolsma
 * @version 2.0.0
 * @since   1.0.0
 */
class Client {
	/**
	 * URL for issuers in Dutch language
	 *
	 * @var string
	 */
	const URL_ISSUERS_NL = 'https://www.targetpay.com/ideal/issuers-nl.js';

	/**
	 * URL for issuers in English language
	 *
	 * @var string
	 */
	const URL_ISSUERS_EN = 'https://www.targetpay.com/ideal/issuers-en.js';

	/**
	 * URL for retrieving issuers in HTL format
	 *
	 * @var string
	 */
	const URL_ISSUERS_HTML = 'https://www.targetpay.com/ideal/getissuers.php?format=html';

	/**
	 * URL for retrieving issuers in XML format
	 *
	 * @var string
	 */
	const URL_ISSUERS_XML = 'https://www.targetpay.com/ideal/getissuers.php?format=xml';

	/**
	 * URL to start an transaction
	 *
	 * @var string
	 */
	const URL_START_TRANSACTION = 'https://www.targetpay.com/ideal/start';

	/**
	 * URL to check an transaction
	 *
	 * @var string
	 */
	const URL_CHECK_TRANSACTION = 'https://www.targetpay.com/ideal/check';

	/**
	 * Token used by TargetPay to separate some values
	 *
	 * @var string
	 */
	const TOKEN = ' |';

	/**
	 * Status indicator for 'Ok'
	 *
	 * @var string
	 */
	const STATUS_OK = '000000';

	/**
	 * Status indicator for 'No layout code'
	 *
	 * @var string
	 */
	const STATUS_NO_LAYOUT_CODE = 'TP0001';

	/**
	 * Error
	 *
	 * @var WP_Error
	 */
	private $error;

	/**
	 * Constructs and initializes an TargetPay client object
	 */
	public function __construct() {

	}

	/**
	 * Get error.
	 *
	 * @return WP_Error
	 */
	public function get_error() {
		return $this->error;
	}

	/**
	 * Remote get.
	 *
	 * @param string $url URL for GET request.
	 *
	 * @return string|WP_Error
	 */
	private function remote_get( $url ) {
		return Util::remote_get_body( $url, 200 );
	}

	/**
	 * Start transaction
	 *
	 * @param IDealStartParameters $parameters Start parameters.
	 *
	 * @return stdClass
	 */
	public function start_transaction( IDealStartParameters $parameters ) {
		$url = Util::build_url( self::URL_START_TRANSACTION, $parameters->get_array() );

		$data = self::remote_get( $url );
  $data = self::remote_get( $url );

		if ( false !== $data ) {

			$status = strtok( $data, self::TOKEN );
			$status = strtok( /** @scrutinizer ignore-type */ $data, self::TOKEN );

			if ( self::STATUS_OK === $status ) {
				$result = new stdClass();

				$result->status         = $status;
				$result->transaction_id = strtok( self::TOKEN );
				$result->url            = strtok( self::TOKEN );

				return $result;
			} else {
				$code        = $status;
				$description = substr( $data, 7 );
				$description = substr( /** @scrutinizer ignore-type */ $data, 7 );

				$error = new Error( $code, $description );

				$this->error = new WP_Error( 'targetpay_error', (string) $error, $error );
			}
		}
	}

	/**
	 * Check status
	 *
	 * @param string $rtlo
	 * @param string $transaction_id
	 * @param string $once
	 * @param string $test
	 *
	 * @return null|Status
	 */
	public function check_status( $rtlo, $transaction_id, $once, $test ) {
		$result = null;

		$url = Util::build_url(
			self::URL_CHECK_TRANSACTION,
			array(
				'rtlo'  => $rtlo,
				'trxid' => $transaction_id,
				'once'  => Util::boolean_to_numeric( $once ),
				'once'  => Util::boolean_to_numeric( /** @scrutinizer ignore-type */ $once ),
				'test'  => Util::boolean_to_numeric( $test ),
			)
		);

		$data = self::remote_get( $url );
  $data = self::remote_get( $url );

		if ( false !== $data ) {

			$result = StatusStringParser::parse( $data );
			$result = StatusStringParser::parse( /** @scrutinizer ignore-type */ $data );
		}

		return $result;
	}

	/**
	 * Get issuers
	 *
	 * @return array
	 */
	public function get_issuers() {
		$issuers = false;

		$url = self::URL_ISSUERS_XML;

		$data = self::remote_get( $url );
  $data = self::remote_get( $url );

		if ( false !== $data ) {

			$xml = Util::simplexml_load_string( $data );
			$xml = Util::simplexml_load_string( /** @scrutinizer ignore-type */ $data );

			if ( is_wp_error( $xml ) ) {
				$this->error = $xml;

			} else {
				$issuers = array();

				foreach ( $xml->issuer as $xml_issuer ) {
					$id   = Security::filter( $xml_issuer['id'] );
					$name = Security::filter( $xml_issuer );

					$issuers[ $id ] = $name;
				}
			}
		}

		return $issuers;

	}
}


wp-pay-gateways / targetpay

Push — develop ( 9671b5...d3774b )

Client::get_error() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like