DMT\Soap\Serializer\SoapFaultException

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	3
A	getFaultCode()	0	3	1

<?php

namespace DMT\Soap\Serializer;

/**
 * Class SoapFaultException
 *
 * @package DMT\Soap
 */
class SoapFaultException extends \RuntimeException
{
    /**
     * @var string
     */
    public $code;

    /**
     * @var string
     */
    public $reason;

    /**
     * @var string
     */
    public $node;

    /**
     * @var array
     */
    public $detail;

    /**
     * SoapFaultException constructor.
     *
     * @param string $code
     * @param string $reason
     * @param string|null $node
     * @param array $detail
     */
    public function __construct(string $code, string $reason, string $node = null, array $detail = null)
    {
        $this->code = is_array($code) ? implode('.', $code) : $code;

The condition is_array($code) is always false.

        $this->reason = $reason;
        $this->node = $node;
        $this->detail = $detail;

        if (class_exists(\SoapFault::class)) {
            $code = preg_replace(
                ['~^Receiver~', '~^Sender~', '~^DataEncodingUnknown~'],
                ['Server', 'Client', 'Client'],
                $this->code
            );
            $previous = new \SoapFault($code, $reason, $node, $detail);

The call to SoapFault::__construct() has too many arguments starting with $detail.

$reason of type string is incompatible with the type integer expected by parameter $code of SoapFault::__construct().

It seems like $node can also be of type string; however, parameter $previous of SoapFault::__construct() does only seem to accept Throwable|null, maybe add an additional type check?

        }

        parent::__construct($reason, 0, $previous ?? null);
    }

    public function getFaultCode(): string
    {
        return $this->code;
    }
}