App\Mail\StudentCountExceeded

Complexity

Total Complexity

6

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	7	1
A	__construct()	0	22	5

<?php

namespace App\Mail;

use App\Models\User;
use App\Models\Institution_class;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Mail\Mailer as MailerContract;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Queue\ShouldQueue;

class StudentCountExceeded extends Mailable
{
    use Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by App\Mail\StudentCountExceeded: $id, $relations, $class, $keyBy

    /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct($file)
    {
//        $this->subject = 'SIS Bulk upload: Student count exceeded' . date('Y:m:d H:i:s');
//        $this->from = env('MAIL_USERNAME');
//        $this->to = [$user->first_name, $user->email];
//        $this->viewData = [
//            'name'=>$user->first_name, "body" => "The class you tried to import data is exceeded the student count limit.Please check the class / increase the student limit"
//        ];

        $institution = Institution_class::find($file['institution_class_id']);

        $this->user = User::find($file['security_user_id']);

The property user does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->subject = 'SIS Bulk Upload: Upload Failed '.$institution->institution->code.': '. $institution->name . date('Y:m:d H:i:s');

The property code does not seem to exist on App\Models\Institution. Are you sure there is no database migration missing?

The property name does not seem to exist on App\Models\Institution_class. Are you sure there is no database migration missing?

        $this->from_address = env('MAIL_FROM_ADDRESS');

The property from_address does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

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

The property from_name does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->with = [

The property with does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            'name' => $this->user->first_name,
            'link' =>   \App::environment('local') || \App::environment('stage')    ?  env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']
        ];
        $this->viewData = [
            'name'=>$this->user->first_name, "body" => "The class you tried to import data is exceeded the student count limit.Please check the class / increase the student limit",
            'link' =>   \App::environment('local') || \App::environment('stage')    ?  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)

The property email does not seem to exist on App\Models\User. Are you sure there is no database migration missing?

            ->subject($this->subject)
            ->with($this->with);
    }

}