SingleBase64Image::uploadRepeatableFile() - Code Metrics - Inspection of "refactoring" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — crud-uploads ( f5e93a...469ece )

by Pedro

created 2023-03-31 17:46 UTC

SingleBase64Image::uploadRepeatableFile() A

↳ Parent: SingleBase64Image

Complexity

Conditions	5
Paths	8

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	13
nc	8
nop	2
dl	0
loc	24
rs	9.5222
c	0
b	0
f	0

<?php

namespace Backpack\CRUD\app\Library\Uploaders;

use Backpack\CRUD\app\Library\CrudPanel\CrudPanelFacade as CRUD;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Str;

class SingleBase64Image extends Uploader
{
    public function uploadFile(Model $entry, $value = null)
    {
        $value = $value ?? CRUD::getRequest()->get($this->getName());
        $value = $value ?? CRUD::/** @scrutinizer ignore-call */ getRequest()->get($this->getName());
        $previousImage = $entry->getOriginal($this->getName());

        if (! $value && $previousImage) {
            Storage::disk($this->getDisk())->delete($previousImage);

            return null;
        }

        if (Str::startsWith($value, 'data:image')) {
            if ($previousImage) {
                Storage::disk($this->getDisk())->delete($previousImage);
            }

            $base64Image = Str::after($value, ';base64,');
            $finalPath = $this->getPath().$this->getFileName($value);

            Storage::disk($this->getDisk())->put($finalPath, base64_decode($base64Image));

            return $finalPath;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
        }

        return $previousImage;

    }

    public function uploadRepeatableFile(Model $entry, $value = null)
    {
        $previousImages = $this->getPreviousRepeatableValues($entry);

        foreach ($value as $row => $rowValue) {
            if ($rowValue) {
                if (Str::startsWith($rowValue, 'data:image')) {
                    $base64Image = Str::after($rowValue, ';base64,');
                    $finalPath = $this->getPath().$this->getFileName($rowValue);
                    Storage::disk($this->getDisk())->put($finalPath, base64_decode($base64Image));
                    $value[$row] = $previousImages[] = $finalPath;

                    continue;
                }
            }
        }

        $imagesToDelete = array_diff($previousImages, $value);

        foreach ($imagesToDelete as $image) {
            Storage::disk($this->getDisk())->delete($image);
        }

        return $value;
    }
}


Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Push — crud-uploads ( f5e93a...469ece )

SingleBase64Image::uploadRepeatableFile() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like