DefaultThrowExceptionProxyFactory::getExceptionFromResult()

Complexity

Conditions	8
Paths	60

Size

Total Lines	68
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	38
CRAP Score	8.001

Importance

Changes

0

<?php
    private function getExceptionFromResult(Pjb62\Exception\JavaException $result): Exception\JavaExceptionInterface
    {
        $message = $result->__get('message')->__toString();

        $found = false;

        foreach ($this->msgPatternsMapping as $exceptionClass => $pattern) {
            if (preg_match($pattern, $message)) {
                $found = true;
                break;
            }
        }

        if (!$found) {
            $exceptionClass = $this->defaultException;
        } else {
            $exceptionClass = '';
        }

        $cls = '\\Soluble\\Japha\\Bridge\\Exception\\'.$exceptionClass;

        $stackTrace = $result->getCause()->__toString();

The method getCause() does not exist on Soluble\Japha\Bridge\Driver\Pjb62\Exception\JavaException. Since you implemented __call, consider adding a @method annotation.

        $code = $result->getCode();

        $driverException = null;
        if ($result instanceof \Exception) {

$result is always a sub-type of Exception.

            $driverException = $result;
        }

        // Public message, mask any login/passwords
        $message = preg_replace('/user=([^&\ ]+)|password=([^&\ ]+)/', '****', $message);

        // Getting original class name from cause
        $javaExceptionClass = 'Unkwown java exception';
        $cause = 'Unknown cause';

        if (is_string($message)) {

The condition is_string($message) is always true.

            preg_match('/Cause: ([^:]+):/', $message, $matches);
            if (count($matches) > 1) {
                $javaExceptionClass = $matches[1];
            }

            // Getting cause from message
            $tmp = explode('Cause: ', $message);
            if (count($tmp) > 1) {
                array_shift($tmp);
                $cause = trim(implode(', ', $tmp));
            } else {
                $cause = $message;
            }
        } else {
            $message = 'Empty message';
        }


        $e = new $cls(
            $message,
            $cause,
            $stackTrace,
            $javaExceptionClass,
            $code,
            $driverException,
            null
        );

        $this->logException($e, $exceptionClass);

        return $e;
    }