BristolSU\ControlDB\Export\Handler\SaveCsvHandler

Complexity

Total Complexity

1

Size/Duplication

Total Lines	13
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	save()	0	6	1

<?php

Missing file doc comment

namespace BristolSU\ControlDB\Export\Handler;

use Illuminate\Support\Facades\Storage;

class SaveCsvHandler extends Handler

Missing doc comment for class SaveCsvHandler

{
    use UsesCsv;
    
    /**

Missing short description in doc comment

Parameter $items should have a doc-comment as per coding-style.

     * @inheritDoc
     */

Missing @return tag in function comment

    protected function save(array $items)
    {
        $csv = $this->generateCsv($items, $this->config('defaultIfNull', 'N/A'));
        
        Storage::disk($this->config('disk', null))
            ->put($this->config('filename', 'export.csv'), $csv);

It seems like $csv can also be of type false; however, parameter $contents of Illuminate\Filesystem\FilesystemAdapter::put() does only seem to accept resource|string, maybe add an additional type check?

    }
}