Equip\Configuration\SessionConfiguration

Complexity

Total Complexity

7

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	apply()	0	11	1
A	prepareSession()	0	16	3
A	startSession()	0	4	1
A	canSessionStart()	0	4	1
A	isCli()	0	4	1

<?php

namespace Equip\Configuration;

use Auryn\Injector;
use Equip\Configuration\ConfigurationInterface;
use Equip\NativeSession;
use Equip\SessionInterface;

class SessionConfiguration implements ConfigurationInterface
{
    /**
     * @inheritDoc
     */
    public function apply(Injector $injector)
    {
        $injector->share(SessionInterface::class);

        $injector->alias(
            SessionInterface::class,
            NativeSession::class
        );

        $injector->prepare(NativeSession::class, [$this, 'prepareSession']);
    }

    /**
     * Initialize the session
     *
     * @param NativeSession $session
     * @param Injector $injector
     *
     * @return void
     */
    public function prepareSession(NativeSession $session, Injector $injector)

The parameter $session is not used and could be removed.

The parameter $injector is not used and could be removed.

    {
        // It seems impossible to test this, since it has to be run in a web
        // server to really be useful and PHPUnit will prevent the session
        // headers working properly. One possible way to test it is by using
        // runkit to mock the [native function][1], but that seems extreme and
        // requires installation of a PECL extension.
        //
        // [1]: https://github.com/tcz/phpunit-mockfunction

58% of this comment could be valid code. Did you maybe forget this after debugging?

        //
        // @codeCoverageIgnoreStart
        if ($this->canSessionStart() && $this->isCli() === false) {
            $this->startSession();
        }
        // @codeCoverageIgnoreEnd
    }

    /**
     * Start a new session
     *
     * @return void
     *
     * @codeCoverageIgnore
     */
    protected function startSession()
    {
        session_start();
    }

    /**
     * Check if a session can be started
     *
     * @return boolean
     */
    private function canSessionStart()
    {
        return session_status() === PHP_SESSION_NONE;
    }

    /**
     * Check if we are running in a command line context
     *
     * @return boolean
     */
    private function isCli()
    {
        return php_sapi_name() === 'cli';
    }
}