App\Imports\StudentImport

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	sheets()	0	5	1
A	__construct()	0	4	1
A	array()	0	2	1
A	registerEvents()	0	7	1

<?php

namespace App\Imports;

use App\Institution_student;

The type App\Institution_student was not found. Maybe you did not declare it correctly or list all dependencies?

use Maatwebsite\Excel\Concerns\WithEvents;
use Maatwebsite\Excel\Concerns\ToModel;
use Maatwebsite\Excel\Concerns\WithMultipleSheets;
use Maatwebsite\Excel\Events\BeforeSheet;

class StudentImport implements WithMultipleSheets, WithEvents
{

    public function __construct()
    {
        $this->sheetNames = [];

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

        $this->sheetData = [];

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

    }

    /**
    * @param array $row
    *
    * @return \Illuminate\Database\Eloquent\Model|null
    */

    public function sheets(): array
    {
        return [

The expression return array(0 => $this) returns the type array<integer,App\Imports\StudentImport> which is incompatible with the documented return type Illuminate\Database\Eloquent\Model|null.

            0 => $this
        ];
    }

    public function array(array $array){
        $this->sheetData[] = $array;

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

    }

    public function registerEvents(): array
    {
        // TODO: Implement registerEvents() method.

        return [
            BeforeSheet::class => function(BeforeSheet $event){
                $this->sheetNames[] = $event->getSheet()->getTitle();

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

            }
        ];

    }

}