FaithGen\SDK\Traits\SavesToAmazonS3

Complexity

Total Complexity

3

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	saveFiles()	0	10	3

<?php

namespace FaithGen\SDK\Traits;

use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Str;

trait SavesToAmazonS3
{
    use FileTraits;

    protected function saveFiles($model)
    {
        foreach ($model->images as $image) {
            $imageFiles = $this->getImages($model->filesDir(), $image->name, $model->getImageDimensions());

            foreach ($imageFiles as $imageFile) {
                Storage::disk('s3')->put(Str::of($imageFile)->after('public/'), fopen($imageFile, 'r+'), 'public');

It seems like fopen($imageFile, 'r+') 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?

            }
        }
        $this->deleteFiles($model);
    }
}