Jaxon\Container\Traits\SessionTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setSessionManager()	0	3	1
A	registerSessions()	0	5	1
A	getSessionManager()	0	3	1

<?php

Missing file doc comment

namespace Jaxon\Container\Traits;

use Closure;
use Jaxon\Contracts\Session as SessionContract;
use Jaxon\Session\Manager as SessionManager;

trait SessionTrait

Missing doc comment for trait SessionTrait

{
    /**
     * Register the values into the container
     *
     * @return void
     */
    private function registerSessions()

Expected 2 blank lines before function; 0 found

    {
        // Set the default session manager
        $this->set(SessionContract::class, function() {

It seems like set() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return new SessionManager();
        });
    }

Expected 2 blank lines after function; 1 found

    /**
     * Get the session manager
     *
     * @return SessionContract
     */
    public function getSessionManager()
    {
        return $this->g(SessionContract::class);

It seems like g() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

Expected 2 blank lines after function; 1 found

    /**
     * Set the session manager
     *
     * @param Closure      $xClosure      A closure to create the session manager instance

Expected 1 spaces after parameter type; 6 found

Expected 1 spaces after parameter name; 6 found

     *
     * @return void
     */
    public function setSessionManager(Closure $xClosure)
    {
        $this->set(SessionContract::class, $xClosure);
    }

Expected 2 blank lines after function; 0 found

}