App\Mail\EmptyFile::__construct()

Complexity

Conditions	5
Paths	16

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function __construct($file)
    {

        $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: ' .$file['subject'].' Empty ' . $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') : env('APP_URL').'/bulk-upload/'
        ];
        $this->viewData = [
            'name'=>$this->user->first_name, "body" => "No data Found in ". $file['filename']. ' Please re-upload the file with data',
            'link' =>  \App::environment('local') || \App::environment('stage')   ?  env('APP_URL') : env('APP_URL').'/bulk-upload/'
        ];