KillFileLink::removeLinkFiles() - Code Metrics - Inspection of "Merge pull request #83 from butcherman/dev5" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev5 ( ecdb11...2a90e7 )

by Ron

created 2020-04-28 11:54 UTC

KillFileLink::removeLinkFiles() A

↳ Parent: KillFileLink

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	4.125

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	7
c	1
b	0
f	0
nc	3
nop	0
dl	0
loc	13
ccs	4
cts	8
cp	0.5
crap	4.125
rs	10

<?php

namespace App\Domains\FileLinks;

use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Auth;

use Carbon\Carbon;

use App\FileLinks;
use App\FileLinkFiles;

use App\Domains\FilesDomain;

class KillFileLink extends FilesDomain
{
    protected $linkID;

    //  Only disable the link by adjusting the expire date
    public function disableLink($linkID)
    {
        $linkData = FileLinks::findOrFail($linkID)->update([
            'expire' => Carbon::yesterday()
        ]);

        Log::info('File Link has been disabled by '.Auth::user()->full_name.'.  Link Data - ', array($linkData));
        return true;
    }

    //  Delete the file link and all attached files
    public function deleteFileLink($linkID)
    {
        $linkData = FileLinks::findOrFail($linkID);

        $this->removeLinkFiles();
        Log::notice('File Link has been deleted by '.Auth::user()->full_name.'.  Link Data - ', array($linkData));
        $linkData->delete();

        return true;
    }

    //  Remove all of the files attached to the link
    protected function removeLinkFiles()
    {
        $fileList = FileLinkFiles::where('link_id', $this->linkID);

        if(!empty($fileList))
        {
            foreach($fileList as $file)
            {
                Log::debug('File from File Link being deleted.  File Data - ', array($file));
                $fileID = $file->file_id;
                $file->delete();

                $this->deleteFile($fileID);
            }
        }
    }
}


1		<?php
2
3		namespace App\Domains\FileLinks;
4
5		use Illuminate\Support\Facades\Log;
6		use Illuminate\Support\Facades\Auth;
7
8		use Carbon\Carbon;
9
10		use App\FileLinks;
11		use App\FileLinkFiles;
12
13		use App\Domains\FilesDomain;
14
15		class KillFileLink extends FilesDomain
16		{
17		protected $linkID;
18
19		// Only disable the link by adjusting the expire date
20	2	public function disableLink($linkID)
21		{
22	2	$linkData = FileLinks::findOrFail($linkID)->update([
23	2	'expire' => Carbon::yesterday()
24		]);
25
26	2	Log::info('File Link has been disabled by '.Auth::user()->full_name.'. Link Data - ', array($linkData));
27	2	return true;
28		}
29
30		// Delete the file link and all attached files
31	2	public function deleteFileLink($linkID)
32		{
33	2	$linkData = FileLinks::findOrFail($linkID);
34
35	2	$this->removeLinkFiles();
36	2	Log::notice('File Link has been deleted by '.Auth::user()->full_name.'. Link Data - ', array($linkData));
37	2	$linkData->delete();
38
39	2	return true;
40		}
41
42		// Remove all of the files attached to the link
43	2	protected function removeLinkFiles()
44		{
45	2	$fileList = FileLinkFiles::where('link_id', $this->linkID);
46
47	2	if(!empty($fileList))
48		{
49	2	foreach($fileList as $file)
50		{
51		Log::debug('File from File Link being deleted. File Data - ', array($file));
52		$fileID = $file->file_id;
53		$file->delete();
54
55		$this->deleteFile($fileID);
56		}
57		}
58	2	}
59		}
60

butcherman / Tech_Bench

Push — dev5 ( ecdb11...2a90e7 )

KillFileLink::removeLinkFiles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like