Humweb\Sociable\Auth\Manager

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	2
A	createLaravelDriver()	0	4	1
A	createSentinelDriver()	0	4	1
A	getDefaultDriver()	0	4	1

<?php

namespace Humweb\Sociable\Auth;

use Illuminate\Container\Container;
use Illuminate\Support\Manager as BaseManager;

/**
 * Manager
 *
 * @package Humweb\SociableConnection\Auth
 */
class Manager extends BaseManager
{

    /**
     * Create a new manager instance.
     *
     * @param  \Illuminate\Foundation\Application  $app
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($app = null)
    {
        $this->app = $app ?: Container::getInstance();

$app ?: \Illuminate\Container\Container::getInstance() is of type object<Illuminate\Container\Container>, but the property $app was declared to be of type object<Illuminate\Foundation\Application>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }


    /**
     * Create Laravel driver
     *
     * @return \Humweb\Sociable\Auth\LaravelDriver
     */
    protected function createLaravelDriver()
    {
        return new LaravelDriver($this->app);
    }


    /**
     * @return \Humweb\Sociable\Auth\SentinelDriver
     */
    protected function createSentinelDriver()
    {
        return new SentinelDriver($this->app);
    }


    /**
     * Get the default driver name.
     *
     * @return string
     */
    public function getDefaultDriver()
    {
        return $this->app['config']->get('social.auth_provider', 'laravel');
    }
}