IncorrectTemplate::build() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/moe-lk..." - moe-lk/bulk-upload - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 93675a...ba3342 )

by Mohamed

created 2020-06-12 04:34 UTC

IncorrectTemplate::build() A

↳ Parent: IncorrectTemplate

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	5
nc	1
nop	0
dl	0
loc	7
rs	10
c	0
b	0
f	0

<?php

namespace App\Mail;

use Illuminate\Bus\Queueable;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Queue\ShouldQueue;

class IncorrectTemplate extends Mailable
{
    use Queueable, SerializesModels;


     /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct($file)
    {

        $institution = Institution_class::find($file['institution_class_id']);
filter:
    dependency_paths: ["lib/*"]


        $this->user = User::find($file['security_user_id']);
filter:
    dependency_paths: ["lib/*"]
        $this->subject = 'SIS Bulk Upload: IncorrectTemplate'.$institution->institution->code.': '. $institution->name.' '. date('Y:m:d H:i:s');
        $this->from_address = env('MAIL_FROM_ADDRESS');

        $this->from_name = 'SIS Bulk Uploader';

        $this->with = [

            'name' => $this->user->first_name,
            'link' =>  env('APP_ENV','local') == 'stage' | 'local' | 'prod' ?  env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']

        ];
        $this->viewData = [
            'name'=>$this->user->first_name, "body" => "The Template you used for upload having some issues, please reconfirm and re-upload with correct template",
            'link' =>  env('APP_ENV','local') == 'stage' | 'local' | 'prod' ?  env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']

        ];
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {
        return $this->view('emails.mail')
            ->from($this->from_address,$this->from_name)
            ->to($this->user->email)
            ->subject($this->subject)
            ->with($this->with);
    }
}


1			<?php
2
3			namespace App\Mail;
4
5			use Illuminate\Bus\Queueable;
6			use Illuminate\Mail\Mailable;
7			use Illuminate\Queue\SerializesModels;
8			use Illuminate\Contracts\Queue\ShouldQueue;
9
10			class IncorrectTemplate extends Mailable
11			{
12			use Queueable, SerializesModels;
			0 ignored issues – show introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The trait `Illuminate\Queue\SerializesModels` requires some properties which are not provided by `App\Mail\IncorrectTemplate`: `$id`, `$relations`, `$class`, `$keyBy` Loading history...
13
14			/**
15			* Create a new message instance.
16			*
17			* @return void
18			*/
19			public function __construct($file)
20			{
21
22			$institution = Institution_class::find($file['institution_class_id']);
			0 ignored issues – show Bug introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The type `App\Mail\Institution_class` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
23
24
25			$this->user = User::find($file['security_user_id']);
			0 ignored issues – show Bug Best Practice introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The property `user` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history... Bug introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The type `App\Mail\User` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
26			$this->subject = 'SIS Bulk Upload: IncorrectTemplate'.$institution->institution->code.': '. $institution->name.' '. date('Y:m:d H:i:s');
27			$this->from_address = env('MAIL_FROM_ADDRESS');
			0 ignored issues – show Bug Best Practice introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The property `from_address` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
28			$this->from_name = 'SIS Bulk Uploader';
			0 ignored issues – show Bug Best Practice introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The property `from_name` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
29			$this->with = [
			0 ignored issues – show Bug Best Practice introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report The property `with` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
30			'name' => $this->user->first_name,
31			'link' => env('APP_ENV','local') == 'stage' \| 'local' \| 'prod' ? env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']
			0 ignored issues – show Bug introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report Are you sure you want to use the bitwise `\|` or did you mean `\|\|`? Loading history...
32			];
33			$this->viewData = [
34			'name'=>$this->user->first_name, "body" => "The Template you used for upload having some issues, please reconfirm and re-upload with correct template",
35			'link' => env('APP_ENV','local') == 'stage' \| 'local' \| 'prod' ? env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']
			0 ignored issues – show Bug introduced 2020-06-12 04:48 UTC by Report Bug Copy Issue Report Are you sure you want to use the bitwise `\|` or did you mean `\|\|`? Loading history...
36			];
37			}
38
39			/**
40			* Build the message.
41			*
42			* @return $this
43			*/
44			public function build()
45			{
46			return $this->view('emails.mail')
47			->from($this->from_address,$this->from_name)
48			->to($this->user->email)
49			->subject($this->subject)
50			->with($this->with);
51			}
52			}
53

moe-lk / bulk-upload

Push — master ( 93675a...ba3342 )

IncorrectTemplate::build() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like