CustomerGauge\Session\NativeSessionGuard::user()

Complexity

Conditions	4
Paths	4

Size

Total Lines	24
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4.0119

Importance

Changes	1
Bugs	0	Features	0

<?php declare(strict_types=1);
    public function user()
    {
        if (! is_null($this->user)) {
            return $this->user;
        }

        $session = $this->session->retrieve();

        // Laravel implements a Chain of Responsibility on the Authentication process.
        // If this Guard cannot authenticate, we must return null to give room for
        // other Guards to attempt to authenticate the current request.
        if (! $session) {

The expression $session of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return null;
        }

        $user = $this->provider->retrieveByCredentials($session);

        if ($user) {
            $this->dispatcher->dispatch(new Authenticated('php-native-session', $user));

            $this->user = $user;
        }

        return $this->user;
    }