NovaExportConfiguration\Nova\Actions\CustomFileExports

Complexity

Total Complexity

12

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	name()	0	3	1
B	handle()	0	51	8
A	__construct()	0	3	1
A	exportsList()	0	5	1
A	fields()	0	9	1

<?php

namespace NovaExportConfiguration\Nova\Actions;

use Illuminate\Bus\Queueable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Str;
use Laravel\Nova\Actions\Action;
use Laravel\Nova\Fields\ActionFields;
use Laravel\Nova\Fields\Select;
use Laravel\Nova\Http\Requests\NovaRequest;
use Maatwebsite\Excel\Facades\Excel;
use Maatwebsite\LaravelNovaExcel\Concerns\WithDisk;
use Maatwebsite\LaravelNovaExcel\Interactions\AskForFilename;
use Maatwebsite\LaravelNovaExcel\Interactions\AskForWriterType;
use NovaExportConfiguration\Export\CustomExport;
use NovaExportConfiguration\Models\ExportStoredFile;
use NovaExportConfiguration\NovaExportConfig;

class CustomFileExports extends Action
{
    use InteractsWithQueue, Queueable;
    use AskForFilename,
        AskForWriterType,
        WithDisk,
        WithQueue;

    public $standalone = true;

    public $showOnIndex = true;

    public $showInline = false;

    public $showOnDetail = false;

    protected $actionFields = [];

    protected array $exportsList = [];

    public function __construct(array $exportsList = [])
    {
        $this->exportsList = $exportsList;
    }

    public function exportsList(array $exportsList = []): static
    {
        $this->exportsList = $exportsList;

        return $this;
    }


    public function name()
    {
        return __('Custom Exports');
    }

    public function handle(ActionFields $fields, Collection $models)

The parameter $models is not used and could be removed.

    {
        /** @var CustomExport $exportable */
        $exportable = NovaExportConfig::customExportsByKey($fields->get('export'));
        if (!$exportable) {

$exportable is of type NovaExportConfiguration\Export\CustomExport, thus it always evaluated to true.

            return Action::danger(__('Exportable config not found'));
        }

        $writerType = $fields->get('writer_type');
        $type       = 'custom-export';
        $name       = $fields->get('filename', $exportable::name()) . '.' . Str::lower($writerType ?: 'xlsx');
        $filename   = date('Y/m/d/') . Str::uuid() . '.' . Str::lower($writerType ?: 'xlsx');
        $disk       = $this->getDisk()?: $exportable::diskName();

        $response = Excel::store(
            $exportable,
            $filename,
            $disk,
            $writerType
        );

        if (false === $response) {
            return Action::danger(__('Resource could not be exported.'));

It seems like __('Resource could not be exported.') can also be of type array; however, parameter $message of Laravel\Nova\Actions\Action::danger() does only seem to accept string, maybe add an additional type check?

        }

        $dbExport       = new ExportStoredFile();
        $dbExport->type = $type;

The property type does not seem to exist on NovaExportConfiguration\Models\ExportStoredFile. Are you sure there is no database migration missing?

        $dbExport->disk = $disk;

The property disk does not seem to exist on NovaExportConfiguration\Models\ExportStoredFile. Are you sure there is no database migration missing?

        $dbExport->path = $filename;

The property path does not seem to exist on NovaExportConfiguration\Models\ExportStoredFile. Are you sure there is no database migration missing?

        $dbExport->name = $name;

The property name does not seem to exist on NovaExportConfiguration\Models\ExportStoredFile. Are you sure there is no database migration missing?

        if ($user = Auth::user()) {
            $dbExport->meta->toMorph('author', $user);

The property meta does not seem to exist on NovaExportConfiguration\Models\ExportStoredFile. Are you sure there is no database migration missing?

        }

        $exportable->setFileModelData(serialize($dbExport));

        if ($queueName = $this->getQueue($exportable::queueName())) {
            $exportable->queue(
                $filename,
                $disk,
                $writerType
            )->allOnQueue($queueName);
        } else {
            $exportable->store(
                $filename,
                $disk,
                $writerType
            );
        }

        return Action::message(__('Data exported to file.'));

It seems like __('Data exported to file.') can also be of type array; however, parameter $message of Laravel\Nova\Actions\Action::message() does only seem to accept string, maybe add an additional type check?

    }

    public function fields(NovaRequest $request)
    {

        return array_merge([
            Select::make('Export', 'export')
                ->options($this->exportsList)
                ->required()
                ->displayUsingLabels(),
        ], $this->actionFields);
    }
}