PhpAbac\Loader\AbacLoader

Complexity

Total Complexity

3

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
	load()	0	1	?
A	supportsExtension()	0	3	1
A	getExtensionAllowed()	0	3	1

<?php
/**
 * Projet :  php-abac.
 * User: mvedie
 * Date: 24/11/2016
 * Time: 18:51
 */

namespace PhpAbac\Loader;

use PhpAbac\Manager\ConfigurationManager;
use Symfony\Component\Config\FileLocatorInterface;
use Symfony\Component\Config\Loader\FileLoader;

abstract class AbacLoader extends FileLoader {
	/**
	 * Must be overrided to contains an array of allowed extension
	 */
	protected static $_EXTENSION_ALLOWED_A = [];
	
	/** @var  ConfigurationManager The configuration manage instanciator and user of this AbacLoader Instance */
	protected $configurationManger;
	
	public function __construct( FileLocatorInterface $locator ) {

Expected 0 spaces between opening bracket and type hint "FileLocatorInterface"; 1 found

Expected 0 spaces between argument "$locator" and closing bracket; 1 found

		parent::__construct( $locator );
	}
	
	/**
	 * Method to load a resource and return an array that contains decoded data of the resource
	 *
	 * @param string $resource The path of the resource to load
	 * @param null   $type     ??
	 *
	 * @return string
	 */
	abstract public function load( $resource, $type = null );

Expected 0 spaces between opening bracket and argument "$resource"; 1 found

Expected 0 spaces between argument "$type" and closing bracket; 1 found

	
//	/**
//	 * Method to check if a resource ( by his path ) is supported by the loader
//	 *
//	 * @param string $resource The path of the resource to check
//	 * @param null   $type     ??
//	 *
//	 * @return boolean Return true if the resource is supported by the loader
//	 */
//	abstract public function supports( $resource, $type = null );
	
	/**
	 * Method to check if a resource is supported by the loader
	 * This method check only extension file
	 *
	 * @param $resource
	 *
	 * @return boolean Return true if the extension of the ressource is supported by the loader
	 */
	public static final function supportsExtension( $resource ) {

As per PSR2, final should precede the visibility keyword.

Expected 0 spaces between opening bracket and argument "$resource"; 1 found

Expected 0 spaces between argument "$resource" and closing bracket; 1 found

		return in_array( pathinfo( $resource, PATHINFO_EXTENSION ), self::getExtensionAllowed() );
	}

    /**
	 * Method to return allowed extension for file to load with the loader
     * @return mixed
     */
	private static final function getExtensionAllowed() {

As per PSR2, final should precede the visibility keyword.

		return static::$_EXTENSION_ALLOWED_A;
	}
}

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.