FaithGen\SDK\Traits\SavesToAmazonS3::saveFiles()

Complexity

Conditions	3
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
    }