Fomvasss\MediaLibraryExtension\MediaLibraryManager

Complexity

Total Complexity

20

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	manage()	0	18	5
A	add()	0	14	1
B	processMultiple()	0	30	10
A	processSingle()	0	20	4

<?php
/**
 * Created by PhpStorm.
 * User: fomvasss
 * Date: 25.11.2019
 * Time: 1:03
 */

namespace Fomvasss\MediaLibraryExtension;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\Request;

The type Illuminate\Http\Request was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Http\UploadedFile;

The type Illuminate\Http\UploadedFile was not found. Maybe you did not declare it correctly or list all dependencies?

use Spatie\MediaLibrary\Models\Media;

class MediaLibraryManager
{
    public function manage(Model $entity, Request $request)
    {
        if (method_exists($entity, 'getMediaFieldsMultiple') === false) {
            throw new \Exception("Method 'getMediaFieldsMultiple' not found in model class " . class_basename($entity));
        }

        if (method_exists($entity, 'getMediaFieldsSingle') === false) {
            throw new \Exception("Method 'getMediaFieldsSingle' not found in model class " . class_basename($entity));
        }

        // Multiple fields
        foreach ($entity->getMediaFieldsMultiple() as $field) {
            $this->processMultiple($entity, $request, $field);
        }

        // Single field
        foreach ($entity->getMediaFieldsSingle() as $field) {
            $this->processSingle($entity, $request, $field);
        }
    }

    public function add(Model $entity, UploadedFile $uploadedFile, string $collectionName): Media
    {
        $originalName = $uploadedFile->getClientOriginalName();

        $filenameGenerator = config('medialibrary-extension.filename_generator');

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

        $filename = $filenameGenerator::get($originalName);

//        $fileName = pathinfo($originalName, PATHINFO_FILENAME);
//        $fileName = \Illuminate\Support\Str::slug($fileName);
//        $fileExtension = pathinfo($originalName, PATHINFO_EXTENSION);

        return $entity->addMedia($uploadedFile)

The expression return $entity->addMedia($uploadedFile)->usingFileName($filename)->toMediaCollection($collectionName) could return the type Illuminate\Database\Eloquent\Builder which is incompatible with the type-hinted return Spatie\MediaLibrary\Models\Media. Consider adding an additional type-check to rule them out.

            ->usingFileName($filename)
            ->toMediaCollection($collectionName);
    }

    protected function processMultiple(Model $entity, Request $request, $field)
    {
        //$request->validate($entity->getMediaFieldsValidation($field));
        $validator = \Validator::make($request->only($field), $entity->getMediaFieldsValidation($field));

It seems like $entity->getMediaFieldsValidation($field) can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $rules of Illuminate\Support\Facades\Validator::make() does only seem to accept array, maybe add an additional type check?

        if ($validator->fails()) {
            return redirect()->back()->withErrors($validator)->withInput();

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        if ($request->hasFile($field)) {
            //$entity->addMultipleMediaFromRequest([$field])->each(function ($fileAdder) use ($field) {
            //    $fileAdder->toMediaCollection($field);
            //});
            foreach ($request->file($field) as $file) {
                $this->add($entity, $file, $field);
            }
        }

        $weightSuffix = config('medialibrary-extension.field_suffixes.weight', '_weight');

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

        if (($weight = $request->get($field . $weightSuffix)) && is_array($weight)) {
            foreach ($weight as $key => $value) {
                $entity->media()->where('id', $key)->update(['order_column' => $value]);
            }
        }

        $deletedSuffix = config('medialibrary-extension.field_suffixes.deleted', '_deleted');
        if (($ids = $request->get($field . $deletedSuffix)) && is_array($ids)) {
            array_map(function($id) use ($entity) {
                $id ? $entity->deleteMedia($id) : null;
            }, $ids);
        }
        //$entity->media()->whereIn('id', $ids)->delete();
    }

    protected function processSingle(Model $entity, Request $request, $field)
    {
        //$request->validate($entity->getMediaFieldsValidation($field));
        $validator = \Validator::make($request->only($field), $entity->getMediaFieldsValidation($field));

It seems like $entity->getMediaFieldsValidation($field) can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $rules of Illuminate\Support\Facades\Validator::make() does only seem to accept array, maybe add an additional type check?

        if ($validator->fails()) {
            return redirect()->back()->withErrors($validator)->withInput();

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        if ($request->hasFile($field)) {
            $entity->getMedia($field)->each(function ($e) {
                $e->delete();
            });

            $this->add($entity, $request->file($field), $field);
        }

        $deletedSuffix = config('medialibrary-extension.field_suffixes.deleted', '_deleted');

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

        if ($id = $request->get($field . $deletedSuffix)) {
            $entity->deleteMedia($id);
        }
    }
}