DiscussionObserver - Code Metrics - Inspection of "deleting discussion" - faithgen/discussions - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ec7d58...d2da01 )

by Innocent

created 2020-04-15 13:28 UTC

DiscussionObserver A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

Metric	Value
wmc	6
lcom	0
cbo	1
dl	0
loc	49
rs	10
c	0
b	0
f	0

<?php

namespace Faithgen\Discussions\Observers;

use Faithgen\Discussions\Jobs\ProcessImages;
use Faithgen\Discussions\Jobs\S3Upload;
use Faithgen\Discussions\Jobs\UploadImages;
use Faithgen\Discussions\Models\Discussion;
use FaithGen\SDK\Models\User;
use FaithGen\SDK\Traits\FileTraits;

class DiscussionObserver
{
    use FileTraits;

    private array $levels = [

        'Free'        => false,
        'Premium'     => true,
        'PremiumPlus' => true,
    ];

    /**
     * Handle the discussion "created" event.
     *
     * @param  Discussion  $discussion
     *
     * @return void
     */
    public function created(Discussion $discussion)
    {
        if ($this->levels[auth()->user()->account->level] && request()->has('images')) {
            if (is_string(request('images'))) {
                $images = json_decode(request('images'), true);
            } else {
                $images = request('images');
            }

            UploadImages::withChain([
                new ProcessImages($discussion),
                new S3Upload($discussion),
            ])->dispatch($discussion, $images);
        }
    }

    /**
     * Handle the discussion "updated" event.
     *
     * @param  Discussion  $discussion
     *
     * @return void
     */
    public function updated(Discussion $discussion)
    {
        //
    }

    /**
     * Handle the discussion "deleted" event.
     *
     * @param  Discussion  $discussion
     *
     * @return void
     */
    public function deleted(Discussion $discussion)
    {
        if ($discussion->images) {
            $this->deleteFiles($discussion);
        }
    }

    /**
     * Converts data when a discussion is being created.
     *
     * @param  Discussion  $discussion
     */
    public function creating(Discussion $discussion)
    {
        if ($user = auth('web')->user()) {
            $discussion->discussable_type = User::class;
            $discussion->discussable_id = $user->id;
        } else {
            $discussion->discussable_type = get_class(auth()->user());
            $discussion->discussable_id = auth()->user()->id;
            $discussion->approved = true;
        }
    }
}


1			<?php
2
3			namespace Faithgen\Discussions\Observers;
4
5			use Faithgen\Discussions\Jobs\ProcessImages;
6			use Faithgen\Discussions\Jobs\S3Upload;
7			use Faithgen\Discussions\Jobs\UploadImages;
8			use Faithgen\Discussions\Models\Discussion;
9			use FaithGen\SDK\Models\User;
10			use FaithGen\SDK\Traits\FileTraits;
11
12			class DiscussionObserver
13			{
14			use FileTraits;
15
16			private array $levels = [
			0 ignored issues – show Bug introduced 2020-04-15 13:32 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Syntax error, unexpected T_ARRAY, expecting T_FUNCTION or T_CONST Loading history...
17			'Free' => false,
18			'Premium' => true,
19			'PremiumPlus' => true,
20			];
21
22			/**
23			* Handle the discussion "created" event.
24			*
25			* @param Discussion $discussion
26			*
27			* @return void
28			*/
29			public function created(Discussion $discussion)
30			{
31			if ($this->levels[auth()->user()->account->level] && request()->has('images')) {
32			if (is_string(request('images'))) {
33			$images = json_decode(request('images'), true);
34			} else {
35			$images = request('images');
36			}
37
38			UploadImages::withChain([
39			new ProcessImages($discussion),
40			new S3Upload($discussion),
41			])->dispatch($discussion, $images);
42			}
43			}
44
45			/**
46			* Handle the discussion "updated" event.
47			*
48			* @param Discussion $discussion
49			*
50			* @return void
51			*/
52			public function updated(Discussion $discussion)
53			{
54			//
55			}
56
57			/**
58			* Handle the discussion "deleted" event.
59			*
60			* @param Discussion $discussion
61			*
62			* @return void
63			*/
64			public function deleted(Discussion $discussion)
65			{
66			if ($discussion->images) {
67			$this->deleteFiles($discussion);
68			}
69			}
70
71			/**
72			* Converts data when a discussion is being created.
73			*
74			* @param Discussion $discussion
75			*/
76			public function creating(Discussion $discussion)
77			{
78			if ($user = auth('web')->user()) {
79			$discussion->discussable_type = User::class;
80			$discussion->discussable_id = $user->id;
81			} else {
82			$discussion->discussable_type = get_class(auth()->user());
83			$discussion->discussable_id = auth()->user()->id;
84			$discussion->approved = true;
85			}
86			}
87			}
88

faithgen / discussions

Push — master ( ec7d58...d2da01 )

DiscussionObserver A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

Duplication Side-by-Side

Filter issues like