dosamigos\flysystem\LocalFsComponent

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	8	2
A	initAdapter()	0	4	1

<?php
/**
 * This file is part of the 2amigos/yii2-flysystem-component project.
 *
 * (c) 2amigOS! <http://2amigos.us/>
 *
 * For the full copyright and license information, please view
 * the LICENSE file that was distributed with this source code.
 */

namespace dosamigos\flysystem;

use League\Flysystem\Adapter\Local;
use Yii;
use yii\base\InvalidConfigException;

class LocalFsComponent extends AbstractFsComponent
{
    /**
     * @var string
     */
    public $path;
    /**
     * @var int
     */
    public $writeFlags = LOCK_EX;
    /**
     * @var int
     */
    public $linkHandling = Local::DISALLOW_LINKS;
    /**
     * @var array
     */
    public $permissions = [];

    /**
     * @inheritdoc
     */
    public function init()
    {
        if ($this->path === null) {
            throw new InvalidConfigException('The "path" property must be set.');
        }
        $this->path = Yii::getAlias($this->path);

It seems like \Yii::getAlias($this->path) can also be of type boolean. However, the property $path is declared as type string. Maybe add an additional type check?

        parent::init();
    }

    /**
     * @return Local
     */
    protected function initAdapter()
    {
        return new Local($this->path, $this->writeFlags, $this->linkHandling, $this->permissions);
    }
}