FilesDomain::saveFile() - Code Metrics - Inspection of "Updated New Logo Administration" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev5a ( 9eb7a1...6555e1 )

by Ron

created 2020-06-06 18:32 UTC

FilesDomain::saveFile() A

↳ Parent: FilesDomain

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	1
eloc	5
nc	1
nop	1
dl	0
loc	8
ccs	6
cts	6
cp	1
crap	1
rs	10
c	2
b	0
f	0

<?php

namespace App\Domains;

use Illuminate\Http\UploadedFile;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Storage;

class FilesDomain
{
    protected $path, $disk;

    public function __construct($path = null, $disk = 'local')
    {
        $this->path = $path === null ? config('filesystems.paths.default') : $path;
        $this->disk = $disk;
    }

    public function saveFile(UploadedFile $file)
    {
        $fileName = $this->cleanFileName($file->getClientOriginalName());
        $fileName = $this->checkForDuplicate($fileName);
        $file->storeAs($this->path, $fileName, $this->disk);

        Log::debug('New File stored.  Details - ', ['disk' => $this->disk, 'path' => $this->path, 'filename' => $fileName]);
        return $fileName;
    }

    //  Sanitize the filename to remove any illegal characters
    protected function cleanFileName($name)
    {
        return preg_replace("([^\w\s\d\-_~,;\[\]\(\).])", '', $name);
    }

    //  Check to see if the file already exists, if so append filename as needed
    protected function checkForDuplicate($name)
    {
        if(Storage::disk($this->disk)->exists($this->path.DIRECTORY_SEPARATOR.$name))
        {
            $parts = pathinfo($name);
            $ext   = isset($parts['extension']) ? ('.'.$parts['extension']) : '';
            $base = $parts['filename'];
             $number = 0;

            do
            {
                $name = $base.'('.++$number.')'.$ext;
            } while(Storage::disk($this->disk)->exists($this->path.DIRECTORY_SEPARATOR.$name));
        }

        return $name;
    }
}


1		<?php
2
3		namespace App\Domains;
4
5		use Illuminate\Http\UploadedFile;
6		use Illuminate\Support\Facades\Log;
7		use Illuminate\Support\Facades\Storage;
8
9		class FilesDomain
10		{
11		protected $path, $disk;
12
13	6	public function __construct($path = null, $disk = 'local')
14		{
15	6	$this->path = $path === null ? config('filesystems.paths.default') : $path;
16	6	$this->disk = $disk;
17	6	}
18
19	6	public function saveFile(UploadedFile $file)
20		{
21	6	$fileName = $this->cleanFileName($file->getClientOriginalName());
22	6	$fileName = $this->checkForDuplicate($fileName);
23	6	$file->storeAs($this->path, $fileName, $this->disk);
24
25	6	Log::debug('New File stored. Details - ', ['disk' => $this->disk, 'path' => $this->path, 'filename' => $fileName]);
26	6	return $fileName;
27		}
28
29		// Sanitize the filename to remove any illegal characters
30	6	protected function cleanFileName($name)
31		{
32	6	return preg_replace("([^\w\s\d\-_~,;\[\]\(\).])", '', $name);
33		}
34
35		// Check to see if the file already exists, if so append filename as needed
36	6	protected function checkForDuplicate($name)
37		{
38	6	if(Storage::disk($this->disk)->exists($this->path.DIRECTORY_SEPARATOR.$name))
39		{
40	2	$parts = pathinfo($name);
41	2	$ext = isset($parts['extension']) ? ('.'.$parts['extension']) : '';
42	2	$base = $parts['filename'];
43	2	$number = 0;
44
45		do
46		{
47	2	$name = $base.'('.++$number.')'.$ext;
48	2	} while(Storage::disk($this->disk)->exists($this->path.DIRECTORY_SEPARATOR.$name));
49		}
50
51	6	return $name;
52		}
53		}
54

butcherman / Tech_Bench

Push — dev5a ( 9eb7a1...6555e1 )

FilesDomain::saveFile() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like