diecoding\flysystem\actions\FileAction

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	22	5
A	getFilesystem()	0	3	1

<?php

namespace diecoding\flysystem\actions;

use DateTimeImmutable;
use Yii;
use yii\base\Action;
use yii\helpers\Json;
use yii\web\NotFoundHttpException;

/**
 * FileAction for handle LocalComponent.
 *
 * To use FileAction, you need to do the following steps:
 *
 * First, declare an action of FileAction type in the `actions()` method of your `SiteController`
 * class (or whatever controller you prefer), like the following:
 *
 * ```php
 * public function actions()
 * {
 *     return [
 *         'file' => [
 *             'class' => \diecoding\flysystem\actions\FileAction::class,
 *             'component' => 'fs',
 *         ],
 *     ];
 * }
 * ```
 * 
 * @link      https://sugengsulistiyawan.my.id/
 * @author    Sugeng Sulistiyawan <sugeng.sulistiyawan@gmail.com>
 * @copyright Copyright (c) 2023
 */
class FileAction extends Action
{
    /**
     * @var string filesystem config component (fs)
     */
    public $component = 'fs';

    /**
     * @return \diecoding\flysystem\AbstractComponent
     */
    public function getFilesystem()
    {
        return Yii::$app->get($this->component);

The expression return Yii::app->get($this->component) also could return the type mixed which is incompatible with the documented return type diecoding\flysystem\AbstractComponent.

    }

    /**
     * Runs the action.
     *
     * @param string|null $data
     * @return mixed result content
     * @throws NotFoundHttpException if data not valid
     */
    public function run($data = null)
    {
        try {
            $params = Json::decode($this->filesystem->decrypt($data));

The property filesystem does not exist on diecoding\flysystem\actions\FileAction. Since you implemented __get, consider adding a @property annotation.

            $now = (int) (new DateTimeImmutable())->getTimestamp();
            $expires = (int) $params['expires'];

            if (!$this->getFilesystem()->fileExists($params['path']) || ($expires > 0 && $expires < $now)) {
                throw new NotFoundHttpException(Yii::t('yii', 'Page not found.'));
            }

            $content = $this->getFilesystem()->read($params['path']);
            $mimeType = $this->getFilesystem()->mimeType($params['path']);
            $attachmentName = (string) pathinfo($params['path'], PATHINFO_BASENAME);
        } catch (\Throwable $th) {
            throw new NotFoundHttpException(Yii::t('yii', 'Page not found.'));
        }

        return Yii::$app->getResponse()->sendContentAsFile($content, $attachmentName, [
            'mimeType' => $mimeType,
            'inline' => true,
        ]);
    }
}