csphere\core\service\Drivers

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	2
A	driver()	0	4	1

<?php

/**
 * Pattern for basic functionality of drivers
 *
 * PHP Version 5
 *
 * @category  Core
 * @package   Service
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

namespace csphere\core\service;

/**
 * Pattern for basic functionality of drivers
 *
 * @category  Core
 * @package   Service
 * @author    Hans-Joachim Piepereit <contact@csphere.eu>
 * @copyright 2013 cSphere Team
 * @license   http://opensource.org/licenses/bsd-license Simplified BSD License
 * @link      http://www.csphere.eu
 **/

abstract class Drivers
{
    /**
     * Stores the loader object
     **/
    protected $loader = null;

    /**
     * Stores the driver configuration
     **/
    protected $config = [];

    /**
     * Creates the driver handler object
     *
     * @param array $config Configuration details as an array
     *
     * @return \csphere\core\service\Drivers
     **/

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

There must be no blank lines after the function comment

    public function __construct(array $config)
    {
        // Store loader object
        $this->loader = \csphere\core\service\Locator::get();

        // Check for empty driver
        if (empty($config['driver'])) {

            $config['driver'] = 'none';
        }

        $this->config = $config;
    }

    /**
     * Returns the name of the driver
     *
     * @return string
     **/

There must be no blank lines after the function comment

    public function driver()
    {
        return $this->config['driver'];
    }
}