ImportSubscriptions::handle() - Code Metrics - NathanGeerinck/laravel-newsletter - Measure and Improve Code Quality continuously with Scrutinizer

ImportSubscriptions::handle() A
last analyzed 2018-06-08 15:13 UTC

↳ Parent: ImportSubscriptions

Complexity

Conditions	3
Paths	4

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	4
nc	4
nop	0
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0

<?php

namespace App\Jobs;

use Mail;
use App\Models\User;
use App\Mail\ListImported;
use App\Models\MailingList;
use Illuminate\Bus\Queueable;
use Illuminate\Queue\SerializesModels;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Queue\ShouldQueue;

/**
 * Class ImportSubscriptions.
 * @property User user
 * @property MailingList list
 */
class ImportSubscriptions implements ShouldQueue
{
    use InteractsWithQueue, Queueable, SerializesModels;


    /**
     * @var MailingList
     */
    protected $user;
    protected $list;
    protected $results;

    /**
     * ImportSubscriptions constructor.
     * @param User $user
     * @param MailingList $list
     * @param $results
     */
    public function __construct(User $user, MailingList $list, $results)
    {
        $this->user = $user;
        $this->list = $list;
        $this->results = $results;
    }

    public function handle()
    {
        foreach ($this->results as $result) {
            $this->list->subscriptions()->create($result);
        }

        if (env('NOTIFICATIONS') == true) {
            Mail::to($this->user)->queue(new ListImported($this->list));
        }
    }
}


1			<?php
2
3			namespace App\Jobs;
4
5			use Mail;
6			use App\Models\User;
7			use App\Mail\ListImported;
8			use App\Models\MailingList;
9			use Illuminate\Bus\Queueable;
10			use Illuminate\Queue\SerializesModels;
11			use Illuminate\Queue\InteractsWithQueue;
12			use Illuminate\Contracts\Queue\ShouldQueue;
13
14			/**
15			* Class ImportSubscriptions.
16			* @property User user
17			* @property MailingList list
18			*/
19			class ImportSubscriptions implements ShouldQueue
20			{
21			use InteractsWithQueue, Queueable, SerializesModels;
			0 ignored issues – show introduced 2018-04-10 22:25 UTC by Report Bug Copy Issue Report The trait `Illuminate\Queue\SerializesModels` requires some properties which are not provided by `App\Jobs\ImportSubscriptions`: `$id`, `$class`, `$relations` Loading history...
22
23			/**
24			* @var MailingList
25			*/
26			protected $user;
27			protected $list;
28			protected $results;
29
30			/**
31			* ImportSubscriptions constructor.
32			* @param User $user
33			* @param MailingList $list
34			* @param $results
35			*/
36			public function __construct(User $user, MailingList $list, $results)
37			{
38			$this->user = $user;
39			$this->list = $list;
40			$this->results = $results;
41			}
42
43			public function handle()
44			{
45			foreach ($this->results as $result) {
46			$this->list->subscriptions()->create($result);
47			}
48
49			if (env('NOTIFICATIONS') == true) {
50			Mail::to($this->user)->queue(new ListImported($this->list));
51			}
52			}
53			}
54

NathanGeerinck / laravel-newsletter

ImportSubscriptions::handle() A last analyzed 2018-06-08 15:13 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ImportSubscriptions::handle() A
last analyzed 2018-06-08 15:13 UTC