CreatePersonalDataExportJob::__construct() - Code Metrics - spatie/laravel-personal-data-export - Measure and Improve Code Quality continuously with Scrutinizer

CreatePersonalDataExportJob::__construct() A
last analyzed 2020-08-06 10:02 UTC

↳ Parent: CreatePersonalDataExportJob

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

namespace Spatie\PersonalDataExport\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Filesystem\Filesystem;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Support\Facades\Mail;
use Illuminate\Support\Facades\Storage;
use Spatie\PersonalDataExport\Events\PersonalDataExportCreated;
use Spatie\PersonalDataExport\Events\PersonalDataSelected;
use Spatie\PersonalDataExport\Exceptions\InvalidUser;
use Spatie\PersonalDataExport\ExportsPersonalData;
use Spatie\PersonalDataExport\PersonalDataSelection;
use Spatie\PersonalDataExport\Zip;
use Spatie\TemporaryDirectory\TemporaryDirectory;

class CreatePersonalDataExportJob implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    /** @var \Spatie\PersonalDataExport\ExportsPersonalData */
    protected $user;

    public function __construct(ExportsPersonalData $user)
    {
        $this->ensureValidUser($user);

        $this->user = $user;

        $this->queue = config('personal-data-export.job.queue');
        
        $this->connection = config('personal-data-export.job.connection');
    }

    public function handle()
    {
        $temporaryDirectory = (new TemporaryDirectory())->create();

        $personalDataSelection = $this->selectPersonalData($temporaryDirectory);

        event(new PersonalDataSelected($personalDataSelection, $this->user));

        $zipFilename = $this->zipPersonalData($personalDataSelection, $this->getDisk(), $temporaryDirectory);

        $temporaryDirectory->delete();

        event(new PersonalDataExportCreated($zipFilename, $this->user));

        $this->mailZip($zipFilename);
    }

    protected function selectPersonalData(TemporaryDirectory $temporaryDirectory): PersonalDataSelection
    {
        $personalData = (new PersonalDataSelection($temporaryDirectory))->forUser($this->user);

        $this->user->selectPersonalData($personalData);

        return $personalData;
    }

    protected function zipPersonalData(
        PersonalDataSelection $personalData,
        Filesystem $filesystem,
        TemporaryDirectory $temporaryDirectory
    ): string {
        $zip = Zip::createForPersonalData($personalData, $temporaryDirectory);

        $zipFilename = pathinfo($zip->path(), PATHINFO_BASENAME);

        $filesystem->writeStream($zipFilename, fopen($zip->path(), 'r'));

        return $zipFilename;
    }

    public function getDisk(): Filesystem
    {
        return Storage::disk(config('personal-data-export.disk'));
    }

    protected function mailZip(string $zipFilename)
    {
        $mailableClass = config('personal-data-export.mailable');

        Mail::to($this->user)->send(new $mailableClass($zipFilename));
    }

    protected function ensureValidUser(ExportsPersonalData $user)
    {
        if (is_null($user->email ?? null)) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            throw InvalidUser::doesNotHaveAnEmailProperty($user);
        }
    }
}


1			<?php
2
3			namespace Spatie\PersonalDataExport\Jobs;
4
5			use Illuminate\Bus\Queueable;
6			use Illuminate\Foundation\Bus\Dispatchable;
7			use Illuminate\Queue\InteractsWithQueue;
8			use Illuminate\Queue\SerializesModels;
9			use Illuminate\Contracts\Filesystem\Filesystem;
10			use Illuminate\Contracts\Queue\ShouldQueue;
11			use Illuminate\Support\Facades\Mail;
12			use Illuminate\Support\Facades\Storage;
13			use Spatie\PersonalDataExport\Events\PersonalDataExportCreated;
14			use Spatie\PersonalDataExport\Events\PersonalDataSelected;
15			use Spatie\PersonalDataExport\Exceptions\InvalidUser;
16			use Spatie\PersonalDataExport\ExportsPersonalData;
17			use Spatie\PersonalDataExport\PersonalDataSelection;
18			use Spatie\PersonalDataExport\Zip;
19			use Spatie\TemporaryDirectory\TemporaryDirectory;
20
21			class CreatePersonalDataExportJob implements ShouldQueue
22			{
23			use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
24
25			/** @var \Spatie\PersonalDataExport\ExportsPersonalData */
26			protected $user;
27
28			public function __construct(ExportsPersonalData $user)
29			{
30			$this->ensureValidUser($user);
31
32			$this->user = $user;
33
34			$this->queue = config('personal-data-export.job.queue');
35
36			$this->connection = config('personal-data-export.job.connection');
37			}
38
39			public function handle()
40			{
41			$temporaryDirectory = (new TemporaryDirectory())->create();
42
43			$personalDataSelection = $this->selectPersonalData($temporaryDirectory);
44
45			event(new PersonalDataSelected($personalDataSelection, $this->user));
46
47			$zipFilename = $this->zipPersonalData($personalDataSelection, $this->getDisk(), $temporaryDirectory);
48
49			$temporaryDirectory->delete();
50
51			event(new PersonalDataExportCreated($zipFilename, $this->user));
52
53			$this->mailZip($zipFilename);
54			}
55
56			protected function selectPersonalData(TemporaryDirectory $temporaryDirectory): PersonalDataSelection
57			{
58			$personalData = (new PersonalDataSelection($temporaryDirectory))->forUser($this->user);
59
60			$this->user->selectPersonalData($personalData);
61
62			return $personalData;
63			}
64
65			protected function zipPersonalData(
66			PersonalDataSelection $personalData,
67			Filesystem $filesystem,
68			TemporaryDirectory $temporaryDirectory
69			): string {
70			$zip = Zip::createForPersonalData($personalData, $temporaryDirectory);
71
72			$zipFilename = pathinfo($zip->path(), PATHINFO_BASENAME);
73
74			$filesystem->writeStream($zipFilename, fopen($zip->path(), 'r'));
75
76			return $zipFilename;
77			}
78
79			public function getDisk(): Filesystem
80			{
81			return Storage::disk(config('personal-data-export.disk'));
82			}
83
84			protected function mailZip(string $zipFilename)
85			{
86			$mailableClass = config('personal-data-export.mailable');
87
88			Mail::to($this->user)->send(new $mailableClass($zipFilename));
89			}
90
91			protected function ensureValidUser(ExportsPersonalData $user)
92			{
93			if (is_null($user->email ?? null)) {
			0 ignored issues – show Bug introduced 2019-03-10 16:00 UTC by Report Bug Copy Issue Report Accessing `email` on the interface `Spatie\PersonalDataExport\ExportsPersonalData` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
94			throw InvalidUser::doesNotHaveAnEmailProperty($user);
95			}
96			}
97			}
98

spatie / laravel-personal-data-export

CreatePersonalDataExportJob::__construct() A last analyzed 2020-08-06 10:02 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

CreatePersonalDataExportJob::__construct() A
last analyzed 2020-08-06 10:02 UTC