DefaultThrowExceptionProxyFactory::getExceptionFromResult()

Complexity

Conditions	6
Paths	24

Size

Total Lines	54
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	31
CRAP Score	6.008

Importance

Changes	2
Bugs	0	Features	0

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

        $exceptionClass = $this->defaultException;

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

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

        // Public message, mask any login/passwords
        $message = preg_replace('/user=([^&\ ]+)|password=([^&\ ]+)/', '****', $message);
        $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;
        }

        // Getting original class name from cause
        preg_match('/Cause: ([^:]+):/', $message, $matches);
        if (count($matches) > 1) {
            $javaExceptionClass = $matches[1];
        } else {
            $javaExceptionClass = 'Unknown java exception class';
        }

        // Getting cause from message
        $tmp = explode('Cause: ', $message);
        if (count($tmp) > 1) {
            array_shift($tmp);
            $cause = trim(implode(', ', $tmp));
        } else {
            $cause = $message;
        }
        $e = new $cls(
            $message,
            $cause,
            $stackTrace,
            $javaExceptionClass,
            $code,
            $driverException,
            null
        );

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

        return $e;
    }