App\Exports\UsersExport

Complexity

Total Complexity

5

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	sheets()	0	3	1
A	startRow()	0	2	1
A	collection()	0	9	1
A	registerEvents()	0	10	1
A	__construct()	0	4	1

<?php

namespace App\Exports;

use App\Models\Security_user;
use Maatwebsite\Excel\Concerns\FromCollection;
use Maatwebsite\Excel\Concerns\WithHeadings;
use Maatwebsite\Excel\Concerns\Exportable;
use App\Models\Import_mapping;
use Maatwebsite\Excel\Concerns\WithEvents;
use Maatwebsite\Excel\Concerns\WithStartRow;
use Maatwebsite\Excel\Events\BeforeWriting;
use Maatwebsite\Excel\Events\BeforeExport;
use Illuminate\Support\Facades\Auth;
use Maatwebsite\Excel\Excel;
use Maatwebsite\Excel\Files\LocalTemporaryFile;
use Maatwebsite\Excel\Concerns\WithMultipleSheets;
use Maatwebsite\Excel\Concerns\RegistersEventListeners;
use Maatwebsite\Excel\Events\BeforeSheet;

class UsersExport implements FromCollection, WithEvents, WithStartRow, WithMultipleSheets {

    /**
     * @return \Illuminate\Support\Collection
     */
    use Exportable,

The trait Maatwebsite\Excel\Concerns\Exportable requires some properties which are not provided by App\Exports\UsersExport: $disk, $diskOptions, $fileName, $filePath, $writerType, $headers

        RegistersEventListeners;

    public $_file_name = 'users.xls';

    public function __construct($class) {

        $this->class = $class;

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

        $this->_file_name = time() . '_' . Auth::user()->id . '_' . 'student_upload.xls';

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    public function registerEvents(): array {
        return [
        BeforeSheet::class => function(BeforeSheet $event){

The parameter $event is not used and could be removed.

//          dd($event->crea)  
        },
            BeforeWriting::class => function(BeforeWriting $event) {
                $file_path = 'app/censusNo_className_sis_students_bulk_upload_v1.xlsx';
                $event->writer->reopen(new LocalTemporaryFile(storage_path($file_path)), Excel::XLSX);
                $event->writer->getDelegate()->setActiveSheetIndex(1);
                return $event->getWriter()->getDelegate()->getActiveSheet();
            }
        ];
    }

    public function sheets(): array {
        return [
            1 => $this
        ];
    }

    public function startRow(): int {
        return 3;
    }

    public function collection() {

        $class = $this->class;
        return Security_user::select('openemis_no', 'first_name', 'gender_id', 'date_of_birth', 'address', 'birthplace_area_id')
                        ->with(['class', 'special_needs'])
                        ->whereHas('class', function ($query) use ($class) {
                            $query->where('institution_class_id', '=', $class);
                        })
                        ->get();
    }

}