rkit\filemanager\models\File

Complexity

Total Complexity

31

Size/Duplication

Total Lines	232
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	9

Test Coverage

Coverage

100%

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	tableName()	0	4	1
A	attributeLabels()	0	18	1
A	behaviors()	0	11	1
B	beforeSave()	0	17	6
A	path()	0	10	3
A	fillMetaInfo()	0	13	2
A	getExtensionByMimeType()	0	19	4
A	generateName()	0	5	1
A	isTmp()	0	4	1
A	checkAccess()	0	6	2
A	checkAccessToModel()	0	7	3
A	checkAccessToFile()	0	7	4
A	findAllByOwner()	0	7	1
A	findOneByOwner()	0	6	1

<?php

/**
 * @link https://github.com/rkit/filemanager-yii2
 * @copyright Copyright (c) 2015 Igor Romanov
 * @license [MIT](http://opensource.org/licenses/MIT)
 */

namespace rkit\filemanager\models;

use Yii;
use yii\behaviors\TimestampBehavior;
use yii\helpers\FileHelper;

/**
 * ActiveRecord for table "file"
 *
 * @property integer $id
 * @property integer $user_id
 * @property integer $owner_id
 * @property integer $owner_type
 * @property string $title
 * @property string $name
 * @property integer $size
 * @property string $extension
 * @property string $mime
 * @property string $date_create
 * @property string $date_update
 * @property integer $ip
 * @property integer $tmp
 * @property integer $position
 */
class File extends \yii\db\ActiveRecord
{
    /**
     * @inheritdoc
     * @codeCoverageIgnore
     * @internal
     */
    public static function tableName()
    {
        return 'file';
    }

    /**
     * @inheritdoc
     * @codeCoverageIgnore
     * @internal
     */
    public function attributeLabels()
    {
        return [
            'id' => Yii::t('filemanager-yii2', 'ID'),
            'user_id' => Yii::t('filemanager-yii2', 'User'),
            'owner_id' => Yii::t('filemanager-yii2', 'Owner'),
            'owner_type' => Yii::t('filemanager-yii2', 'Owner type'),
            'title' => Yii::t('filemanager-yii2', 'Title'),
            'name' => Yii::t('filemanager-yii2', 'Name'),
            'size' => Yii::t('filemanager-yii2', 'Size'),
            'extension' => Yii::t('filemanager-yii2', 'Extension'),
            'mime' => Yii::t('filemanager-yii2', 'Mime'),
            'date_create' => Yii::t('filemanager-yii2', 'Date create'),
            'date_update' => Yii::t('filemanager-yii2', 'Date update'),
            'ip' => Yii::t('filemanager-yii2', 'IP'),
            'position' => Yii::t('filemanager-yii2', 'Position'),
        ];
    }

    /**
     * @inheritdoc
     * @internal
     */
    public function behaviors()
    {
        return [
            [
                'class' => TimestampBehavior::className(),
                'createdAtAttribute' => 'date_create',
                'updatedAtAttribute' => 'date_update',
                'value' => new \yii\db\Expression('NOW()'),
            ],
        ];
    }

    /**
     * @internal
     */
    public function beforeSave($insert)
    {
        if (parent::beforeSave($insert)) {
            if ($insert) {
                if (!Yii::$app instanceof \yii\console\Application) {
                    $this->user_id = Yii::$app->user->isGuest ? 0 : Yii::$app->user->id; // @codeCoverageIgnore
                    $this->ip = ip2long(Yii::$app->request->getUserIP()); // @codeCoverageIgnore
                } // @codeCoverageIgnore

                if ($this->owner_id === null) {
                    $this->owner_id = 0;
                }
            }
            return true;
        }
        return false; // @codeCoverageIgnore
    }

    /**
     * Path to the file
     *
     * @return string
     * @SuppressWarnings(PHPMD.ElseExpression)
     */
    public function path()
    {
        if ($this->isNewRecord || is_object($this->date_create)) {
            $dateCreate = date('Ym');
        } else {
            $dateCreate = date_format(date_create($this->date_create), 'Ym');
        }

        return '/' . $dateCreate . '/' . $this->id . '/' . $this->name;
    }

    /**
     * Fill meta info
     *
     * @param string $path File path
     * @return void
     * @SuppressWarnings(PHPMD.ElseExpression)
     */
    public function fillMetaInfo($path)
    {
        $pathInfo = pathinfo($path);

        if ($this->title === null) {
            $this->title = $pathInfo['filename'];
        }

        $this->size = filesize($path);
        $this->mime = FileHelper::getMimeType($path);
        $this->extension = $this->getExtensionByMimeType($path, $this->mime);
        $this->name = $this->generateName();
    }

    /**
     * Get extension By MimeType
     *
     * $param string $path File path
     * @param string $mimeType MimeType of the file
     * @return string
     */
    private function getExtensionByMimeType($path, $mimeType)
    {
        $extensions = FileHelper::getExtensionsByMimeType($mimeType);
        $pathExtension = pathinfo($path, PATHINFO_EXTENSION);
        $titleExtension = pathinfo($this->title, PATHINFO_EXTENSION);

        if (array_search($pathExtension, $extensions) !== false) {
            return $pathExtension;
        } elseif (array_search($titleExtension, $extensions) !== false) {
            return $titleExtension;
        }
        $extension = explode('/', $mimeType);
        $extension = end($extension);
        if (array_search($extension, $extensions) !== false) {
            return $extension;
        }

        return current($extensions); // @codeCoverageIgnore

The expression current($extensions); of type integer|string|false adds false to the return on line 168 which is incompatible with the return type documented by rkit\filemanager\models\File::getExtensionByMimeType of type string. It seems like you forgot to handle an error condition.

    }

    /**
     * Generate a name
     *
     * @return string
     */
    public function generateName()
    {
        $name = date('YmdHis') . substr(md5(microtime() . uniqid()), 0, 10);
        return $name . '.' . $this->extension;
    }

    /**
     * Checks whether the file is temp
     *
     * @return bool
     */
    public function isTmp()
    {
        return (bool)$this->tmp;
    }

    /**
     * Check access
     *
     * @param int $ownerId The id of the owner
     * @param int $ownerType The type of the owner
     * @return bool
     */
    public function checkAccess($ownerId, $ownerType)
    {
        return
            $this->checkAccessToModel($ownerId, $ownerType) ||
            $this->checkAccessToFile($ownerType);
    }

    /**
     * Check access to model
     *
     * @param int $ownerId The id of the owner
     * @param int $ownerType The type of the owner
     * @return bool
     */
    public function checkAccessToModel($ownerId, $ownerType)
    {
        $equalOwnerType = $this->owner_type === $ownerType;
        $equalOwnerId = $this->owner_id === $ownerId;

        return !$this->tmp && $equalOwnerType && $equalOwnerId;
    }

    /**
     * Check access to file
     *
     * @param int $ownerId The id of the owner

There is no parameter named $ownerId. Was it maybe removed?

     * @param int $ownerType The type of the owner
     * @return bool
     */
    public function checkAccessToFile($ownerType)
    {
        $equalOwnerType = $this->owner_type === $ownerType;
        $user = $this->user_id === Yii::$app->user->id || $this->user_id === 0;

        return $this->tmp && $equalOwnerType && $user;
    }

    /**
     * Find all by owner
     *
     * @param int $ownerId The id of the owner
     * @param int $ownerType The type of the owner
     * @return array
     */
    public static function findAllByOwner($ownerId, $ownerType)
    {
        return static::find()
            ->where(['owner_id' => $ownerId, 'owner_type' => $ownerType])
            ->orderBy('position ASC')
            ->all();
    }

    /**
     * Find one by owner
     *
     * @param int $ownerId The id of the owner
     * @param int $ownerType The type of the owner
     * @return File|null
     */
    public static function findOneByOwner($ownerId, $ownerType)
    {
        return static::find()
            ->where(['owner_id' => $ownerId, 'owner_type' => $ownerType])
            ->one();
    }
}