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

Complexity

Conditions	5
Paths	5

Size

Total Lines	17
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    protected function saveFiles($model)
    {
        if (! in_array(StorageTrait::class, class_uses($model))) {
            throw new InvalidArgumentException('The model you used does not use the Storage trait');
        }

        try {
            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);
        } catch (\Exception $e) {
            Log::error($e->getMessage());
        }