ShiftOneLabs\LaravelDbEvents\Traits\ConnectorConnectTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	connect()	0	12	2
A	parentConnect()	0	4	1

<?php
namespace ShiftOneLabs\LaravelDbEvents\Traits;

use ShiftOneLabs\LaravelDbEvents\Exceptions\ConnectingException;
use ShiftOneLabs\LaravelDbEvents\Extension\Database\Events\DatabaseConnected;
use ShiftOneLabs\LaravelDbEvents\Extension\Database\Events\DatabaseConnecting;

trait ConnectorConnectTrait
{
    use SupportsEvents;

    /**
     * Establish a database connection.
     *
     * @param  array  $config
     * @return \PDO
     */
    public function connect(array $config)
    {
        if ($this->fireEvent(new DatabaseConnecting($this, array_get($config, 'name'), $config), true) === false) {
            throw new ConnectingException();
        }

        $connection = $this->parentConnect($config);

        $this->fireEvent(new DatabaseConnected($this, array_get($config, 'name'), $config, $connection));

        return $connection;
    }

    /**
     * Call connect on the parent class to establish a database connection.
     *
     * @param  array  $config
     * @return \PDO
     */
    protected function parentConnect(array $config)
    {
        return parent::connect($config);

It seems like you call parent on a different method (connect() instead of parentConnect()). Are you sure this is correct? If so, you might want to change this to $this->connect().

    }
}