App\Http\Controllers\CloneController

Complexity

Total Complexity

37

Size/Duplication

Total Lines	309
Duplicated Lines	0 %

Importance

Changes	7
Bugs	2	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	updateInstitutionClasses()	0	29	3
A	insertInstitutionClassSubjects()	0	9	3
A	insertClassSubjects()	0	16	3
A	insertInstitutionClasses()	0	40	3
A	insertInstitutionSubjects()	0	10	2
A	array_walk()	0	9	2
C	process()	0	65	11
A	__construct()	0	11	1
A	cleanConfig()	0	22	3
A	updateShifts()	0	28	3
A	generateNewClass()	0	12	3

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\Academic_period;
use App\Models\Institution_class;
use App\Models\Institution_shift;
use Illuminate\Support\Facades\DB;
use App\Models\Institution_subject;
use Illuminate\Support\Facades\Log;
use App\Http\Controllers\Controller;
use App\Models\Institution_class_grade;
use App\Models\Education_grades_subject;
use App\Models\Institution_class_student;
use App\Models\Institution_class_subject;
use App\Models\Institution_grade;
use App\Models\Institution_student;

class CloneController extends Controller
{
    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()

    {
        $this->shifts = new Institution_shift();

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

        $this->academic_period = new Academic_period();

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

        $this->institution_classes = new Institution_class();

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

        $this->institution_class_subjects = new Institution_class_subject();

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

        $this->institution_subjects =  new Institution_subject();

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

        $this->institution_grades = new Institution_grade();

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

        $this->education_grade_subjects =  new Education_grades_subject();

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

        $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();

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

    }


    public function array_walk($shift, $count, $params)

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

    {
        try{
           DB::beginTransaction();
            array_walk($shift, array($this, 'process'), $params);
            DB::commit();
        }catch(\Exception $e){
            $this->output->writeln('Terminating ' . $shift['institution_id']);
            DB::rollBack();
        }
    }

    public function cleanConfig($params)
    {
        $academicPeriod = $params['academic_period'];
        $this->shifts->where(['academic_period_id' => $academicPeriod->id])->delete();
        $this->output->writeln('cleaned shifts');

        $this->shifts->where(['cloned' => $academicPeriod->code])->update(['cloned' => $params['previous_academic_period']['code']]);
        $this->output->writeln('updated shifts');

        $classIds =  $this->institution_classes->select('id')->where(['academic_period_id' => $academicPeriod->id])->get()->toArray();

The assignment to $classIds is dead and can be removed.

        $this->institution_classes->where(['academic_period_id' => $academicPeriod->id])->delete();
        $this->output->writeln('cleaned classes');

        do {
            $deleted = $this->institution_class_subjects->whereRaw("institution_class_id not in (select id from institution_classes where academic_period_id =".$academicPeriod->id." )")->limit(10000)->delete();
            $this->output->writeln('cleaned subjects');
        }while($deleted > 0);

        do {
            $deleted =  $this->institution_subjects->where('academic_period_id', $academicPeriod->id)->limit(10000)->delete();
            $this->output->writeln('10000 institutions cleaned subjects');
        } while ($deleted > 0);
    }

    public function process($shift, $count, $params)
    {
        echo ('[' . getmypid() . ']This Process executed at' . date("F d, Y h:i:s A") . "\n");
        $year = $params['year'];
        $academicPeriod = $params['academic_period'];
        $previousAcademicPeriod = $params['previous_academic_period'];
        $mode = $params['mode'] == 'AL' ? true : false;

        $data = $this->updateShifts($year, $shift);
        $shiftId = $data['shift_id'];

        $params = [
            'previous_academic_period_id' => $previousAcademicPeriod->id,
            'academic_period_id' => $academicPeriod->id,
            'shift_id' => $data['shift_id'],
            'mode' => $mode
        ];


        if ($mode) {
            $institutionClasses = $this->institution_classes->getShiftClasses($shift,  $mode, $params);
            try {
                array_walk($institutionClasses, array($this, 'updateInstitutionClasses'), $params);
                $this->output->writeln('updating from ' . $shift['institution_id']);
            } catch (\Exception $e) {
                Log::error($e->getMessage(), [$e]);
            }
        } else {
            $institutionSubjects = $this->institution_grades->getGradeSubjects($shift['institution_id']);
            try {
                if ($data['created']) {
                    $institutionClasses = $this->institution_classes->getShiftClasses($shift, $mode);
                    array_walk($institutionSubjects, array($this, 'insertInstitutionSubjects'), $academicPeriod);
                    if (!empty($institutionClasses) && !is_null($shiftId) && !is_null($academicPeriod)) {
                        $newInstitutionClasses = $this->generateNewClass($institutionClasses, $shiftId, $academicPeriod->id);
                        try {
                            array_walk($newInstitutionClasses, array($this, 'insertInstitutionClasses'), $params);
                            $this->output->writeln('##########################################################################################################################');
                            $this->output->writeln('updating from = ' . $shift['institution_id']);
                        } catch (\Exception $e) {
                            $this->output->writeln('Terminating ' . $shift['institution_id']);
                            DB::rollBack();
                            Log::error($e->getMessage(), [$e]);
                        }
                    } else {
                        $this->output->writeln('no classes found ' . $shift['institution_id']);
                    }
                } else {
                    try {
                        $shift['id'] = $shiftId;
                        $institutionClasses = $this->institution_classes->getShiftClasses($shift, $mode);
                        array_walk($institutionClasses, array($this, 'updateInstitutionClasses'), $params);
                        $this->output->writeln('##########################################################################################################################');
                        $this->output->writeln('updating from ' . $shift['institution_id']);
                    } catch (\Exception $e) {
                        $this->output->writeln('Terminating ' . $shift['institution_id']);
                        DB::rollBack();
                        Log::error($e->getMessage(), [$e]);
                    }
                }
                DB::commit();
            } catch (\Exception $e) {
                Log::error($e->getMessage(), [$e]);
                $this->output->writeln('Terminating ' . $shift['institution_id']);
                DB::rollBack();
            }
        }
    }


    /**
     * @param $subjects
     * @param $count
     * @param $academicPeriod
     */
    public function insertInstitutionSubjects($subject, $count, $academicPeriod)
    {
        try {
            $subject['academic_period_id'] = $academicPeriod->id;
            $subject['created'] = now();
            $subject['created_user_id'] = 1;
            $this->institution_subjects->create($subject);
        } catch (\Exception $e) {
            DB::rollBack();
            Log::error($e->getMessage(), [$e]);
        }
    }

    public function updateInstitutionClasses($class, $count, $params)
    {
        try {
            if ($params['mode']) {
                Institution_class::where('id', $class['id'])
                    ->where('academic_period_id',$params['previous_academic_period_id'])
                    ->update([
                        'institution_shift_id' => $params['shift_id'],
                        'academic_period_id' => $params['academic_period_id']
                    ]);

                Institution_class_student::where('institution_class_id', $class['id'])
                    ->where('academic_period_id',$params['previous_academic_period_id'])    
                    ->update([
                        'academic_period_id' => $params['academic_period_id'],
                        'modified' => now()
                    ]);

                $educationGrade = Institution_class_grade::select('education_grade_id')->where('institution_class_id', $class['id'])->get()->toArray();

                Institution_student::whereIn('education_grade_id', $educationGrade)
                    ->where('academic_period_id',$params['previous_academic_period_id'])
                    ->update([
                        'academic_period_id' => $params['academic_period_id']
                    ]);
            }
        } catch (\Exception $e) {
            DB::rollBack();
            Log::error($e->getMessage(), [$e]);
        }
    }

    public function  insertInstitutionClasses($class, $count, $param)
    {
        try {
            $academicPeriod = $param['academic_period_id'];
            $educationGrdae = $class['education_grade_id'];

            $classId = $class['id'];
            unset($class['id']);
            $institutionSubjects = Institution_subject::query()->where('education_grade_id', $class['education_grade_id'])
                ->where('institution_id', $class['institution_id'])
                ->where('academic_period_id', $academicPeriod)->get()->toArray();
            $params = [
                'class' => $class,
                'subjects' => $institutionSubjects,
                'academic_period_id' => $academicPeriod,
                'classId' => $classId
            ];
            unset($class['education_grade_id']);
            $noOfStudents = $class['no_of_students'] == 0 ? 40 : $class['no_of_students'];
            $class['academic_period_id'] = $academicPeriod;
            $class['no_of_students'] = $noOfStudents;
            $class['created'] = now();
            $class['institution_shift_id'] = $param['shift_id'];
            // $class['created_user_id'] = 
            $this->output->writeln('Create class:' . $class['name']);
            $class =  Institution_class::create($class);
            $institutionClassGrdaeObj['institution_class_id'] = $class->id;

$institutionClassGrdaeObj was never initialized. Although not strictly required by PHP, it is generally a good practice to add $institutionClassGrdaeObj = array(); before regardless.

            $institutionClassGrdaeObj['education_grade_id'] = $educationGrdae;
            $institutionClassGrdaeObj['created_user_id'] = $class['created_user_id'];
            Institution_class_grade::create($institutionClassGrdaeObj);
            $institutionSubjects = Institution_subject::query()->where('education_grade_id', $educationGrdae)
                ->where('institution_id', $class->institution_id)

The property institution_id does not exist on App\Models\Institution_class. Did you mean institution?

                ->where('academic_period_id', $academicPeriod)
                ->get()
                ->toArray();
            $params['class'] = $class;
            $this->insertInstitutionClassSubjects($institutionSubjects, $class);
        } catch (\Exception $e) {
            DB::rollBack();
            Log::error($e->getMessage(), [$e]);
        }
    }

    public function insertInstitutionClassSubjects($subjects, $class)
    {
        if (!empty($subjects)) {
            try {
                array_walk($subjects, array($this, 'insertClassSubjects'), $class);
                $this->output->writeln('updating subjects ' . $class->name);
            } catch (\Exception $e) {
                DB::rollBack();
                Log::error($e->getMessage(), [$e]);
            }
        };
    }


    public function insertClassSubjects($subject, $count, $newClassId)
    {
        try {
            $subjectobj['status'] = 1;

$subjectobj was never initialized. Although not strictly required by PHP, it is generally a good practice to add $subjectobj = array(); before regardless.

            $subjectobj['created_user_id'] = 1;
            $subjectobj['created'] = now();

            $subjectobj['institution_class_id'] = $newClassId->id;
            $subjectobj['institution_subject_id'] = $subject['id'];

            if (!$this->institution_class_subjects->isDuplicated($subjectobj)) {
                $this->institution_class_subjects->create($subjectobj);
            }
        } catch (\Exception $e) {
            DB::rollBack();
            Log::error($e->getMessage(), [$e]);
        }
    }

    /**
     * generate new class object for new academic year
     *
     * @param $classes
     * @param $shiftId
     * @param $academicPeriod
     * @return array
     */
    public function generateNewClass($classes, $shiftId, $academicPeriod)
    {
        $newClasses = [];
        foreach ($classes as $class) {
            $noOfStudents = $class['no_of_students'] == 0 ? 40 : $class['no_of_students'];
            $class['academic_period_id'] = $academicPeriod;
            $class['no_of_students'] = $noOfStudents;
            $class['created'] = now();
            $class['institution_shift_id'] = $shiftId;
            array_push($newClasses, $class);
        }
        return $newClasses;
    }

    /**
     * update shifts
     * @param $year
     * @param $shift
     * @return mixed
     */
    public function updateShifts($year, $shift)
    {
        try {
            $academicPeriod = $this->academic_period->getAcademicPeriod($year);
            $this->shifts->where(['id' =>  $shift['id']])->update(['cloned' => $year]);
            $shift['academic_period_id'] = $academicPeriod->id;
            $exist = $this->shifts->getShift($shift);
            $data = array();

            if (is_null($exist)) {
                $shift['cloned'] = $academicPeriod->code;

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

                unset($shift['id']);
                unset($shift['created']);
                unset($shift['modified']);
                $shift = $this->shifts->create((array)$shift);
                $data = [
                    'shift_id' => $shift->id,
                    'created' => true
                ];
            } else {
                $data = [
                    'shift_id' => $exist->id,
                    'created' => false
                ];
            };
            return $data;
        } catch (\Exception $e) {
            DB::rollBack();
        }
    }
}