File CardPayRequestValues.php has 495 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php declare(strict_types=1);

namespace Granam\GpWebPay;

use Granam\Boolean\Tools\ToBoolean;

CardPayRequestValues has 37 functions (exceeds 20 allowed). Consider refactoring.

Open

class CardPayRequestValues extends StrictObject
{

    // name => is required
    private static array $keysExpectedInArray = [

Function normalizeValues has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

    private static function normalizeValues(array $withUpperCasedKeys): array
    {
        $normalizedValues = [];
        foreach (self::$keysExpectedInArray as $key => $required) {
            if (($withUpperCasedKeys[$key] ?? null) === null) {

• Read up
  Read up

Method __construct has 16 arguments (exceeds 4 allowed). Consider refactoring.

Open

        CurrencyCodes $currencyCodes,
        int $orderNumber,
        float $price,
        int $currencyNumericCode,
        bool $depositFlag, // false = instant payment not required, true = requires immediate payment

Method sanitizeAsciiRange has 47 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function sanitizeAsciiRange(
        string $value,
        string $nameOfParameter,
        array $characterRanges = [[' ' /* 0x20 = space */, '~' /* 0x7E = tilde */]]
    ): string

Method __construct has 13 arguments (exceeds 4 allowed). Consider refactoring.

Open

        SettingsInterface $settings,
        DigestSignerInterface $digestSigner,
        string $operation,
        int $orderNumber,
        int $prCode,

Method buildParametersForDigest has 39 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function buildParametersForDigest(CardPayRequestValues $requestValues, SettingsInterface $settings): array
    {
        $parametersWithoutDigest = [];
        // parameters HAVE TO be in this order, see GP_webpay_HTTP_EN.pdf / GP_webpay_HTTP.pdf
        $parametersWithoutDigest[RequestDigestKeys::MERCHANTNUMBER] = $settings->getMerchantNumber();

Method normalizeValues has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private static function normalizeValues(array $withUpperCasedKeys): array
    {
        $normalizedValues = [];
        foreach (self::$keysExpectedInArray as $key => $required) {
            if (($withUpperCasedKeys[$key] ?? null) === null) {

Method setPayMethods has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function setPayMethods(array $payMethods = null)
    {
        if ($payMethods === null) {
            return;
        }

Method createFromArray has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function createFromArray(array $valuesFromGetOrPost, CurrencyCodes $currencyCodes): CardPayRequestValues
    {
        $withUpperCasedKeys = [];
        foreach ($valuesFromGetOrPost as $key => $value) {
            $withUpperCasedKeys[\strtoupper(\trim($key))] = $value;

Method __construct has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct(
        SettingsInterface $settings,
        DigestSignerInterface $digestSigner,
        string $operation,
        int $orderNumber,

Function normalizeValues has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    protected static function normalizeValues(array $values): array
    {
        $normalizedValues = [];
        foreach (self::$expectedKeys as $key => $required) {
            $values[$key] ??= null;

• Read up
  Read up

Function buildParametersForDigest has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    private function buildParametersForDigest(CardPayRequestValues $requestValues, SettingsInterface $settings): array
    {
        $parametersWithoutDigest = [];
        // parameters HAVE TO be in this order, see GP_webpay_HTTP_EN.pdf / GP_webpay_HTTP.pdf
        $parametersWithoutDigest[RequestDigestKeys::MERCHANTNUMBER] = $settings->getMerchantNumber();

• Read up
  Read up

Method getCurrentRequestUrl has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function getCurrentRequestUrl(): string
    {
        $protocol = 'http';
        if (!empty($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
            $protocol = $_SERVER['HTTP_X_FORWARDED_PROTO'];

Function __construct has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function __construct(
        SettingsInterface $settings,
        DigestSignerInterface $digestSigner,
        string $operation,
        int $orderNumber,

• Read up
  Read up

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

        string $baseUrlForRequest,
        string $privateKeyFile,
        string $privateKeyPassword,
        string $publicKeyFile,
        string $merchantNumber,

Function getCurrentRequestUrl has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    private function getCurrentRequestUrl(): string
    {
        $protocol = 'http';
        if (!empty($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
            $protocol = $_SERVER['HTTP_X_FORWARDED_PROTO'];

• Read up
  Read up

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        int $prCode,
        int $srCode,
        string $resultText = null,
        $exceptionCode = null, // intentionally without scalar type hint
        \Exception $previousException = null

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        int $prCode,
        int $srCode,
        string $resultText = null,
        $exceptionCode = null, // intentionally without scalar type hint
        \Exception $previousException = null

Method createForTest has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        string $privateKeyFile,
        string $privateKeyPassword,
        string $publicKeyFile,
        string $merchantNumber,
        string $urlForResponse = null