moe-lk / bulk-upload

app/Imports/UsersImport.php

Spacing +17 added lines, -17 removed lines

@@ -75,7 +75,7 @@ 
     public function registerEvents(): array
     {
         return [
-            BeforeSheet::class => function (BeforeSheet $event) {
+            BeforeSheet::class => function(BeforeSheet $event) {
                 $this->sheetNames[] = $event->getSheet()->getTitle();
                 $this->worksheet = $event->getSheet();
                 $this->validateClass();
@@ -87,7 +87,7 @@ 
                     throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
                 }
             },
-            BeforeImport::class => function (BeforeImport $event) {
+            BeforeImport::class => function(BeforeImport $event) {
                 $this->highestRow = ($event->getReader()->getDelegate()->getActiveSheet()->getHighestDataRow('C'));
                 if ($this->highestRow < 3) {
                     $error = \Illuminate\Validation\ValidationException::withMessages([]);
@@ -113,7 +113,7 @@ 
 
             if (!empty($institutionClass)) {
                 $row = $this->setGender($row);
-                $studentInfo = Security_user::createOrUpdateStudentProfile($row,'create',$this->file);  
+                $studentInfo = Security_user::createOrUpdateStudentProfile($row, 'create', $this->file);  
                 $academicPeriod = Academic_period::where('id', '=', $institutionClass->academic_period_id)->first();
                 $institutionGrade = Institution_class_grade::where('institution_class_id', '=', $institutionClass->id)->first();
                 $assignee_id = $institutionClass->staff_id ? $institutionClass->staff_id : $this->file['security_user_id'];
@@ -126,25 +126,25 @@ 
                     'institution_class' => $institutionClass
                 ];
 
-                Institution_student_admission::createAdmission($studentInfo->id,$row,$params,$this->file);
-                Institution_student::createOrUpdate($studentInfo->id,$row,$params,$this->file);
-                $student = Institution_class_student::createOrUpdate($studentInfo->id,$params,$this->file);
-                User_special_need::createOrUpdate($student->student_id,$row,$this->file);
-                User_body_mass::createOrUpdate($student->student_id,$row,$this->file);
+                Institution_student_admission::createAdmission($studentInfo->id, $row, $params, $this->file);
+                Institution_student::createOrUpdate($studentInfo->id, $row, $params, $this->file);
+                $student = Institution_class_student::createOrUpdate($studentInfo->id, $params, $this->file);
+                User_special_need::createOrUpdate($student->student_id, $row, $this->file);
+                User_body_mass::createOrUpdate($student->student_id, $row, $this->file);
 
-                $this->createOrUpdateGuardian($row,$student,'father');
-                $this->createOrUpdateGuardian($row,$student,'mother');
-                $this->createOrUpdateGuardian($row,$student,'guardian');
+                $this->createOrUpdateGuardian($row, $student, 'father');
+                $this->createOrUpdateGuardian($row, $student, 'mother');
+                $this->createOrUpdateGuardian($row, $student, 'guardian');
                 
                 $studentInfo['student_id'] = $studentInfo->id;
                 Institution_student::updateStudentArea($studentInfo->toArray());
 
-                $this->insertOrUpdateSubjects($row,$student,$institution);
+                $this->insertOrUpdateSubjects($row, $student, $institution);
 
                 $totalStudents = Institution_class_student::getStudentsCount($this->file['institution_class_id']);
                 if ($totalStudents['total'] > $institutionClass->no_of_students) {
                     $error = \Illuminate\Validation\ValidationException::withMessages([]);
-                    $failure = new Failure(3, 'rows', [3 => 'Class student count exceeded! Max number of students is ' . $institutionClass->no_of_students], [null]);
+                    $failure = new Failure(3, 'rows', [3 => 'Class student count exceeded! Max number of students is '.$institutionClass->no_of_students], [null]);
                     $failures = [0 => $failure];
                     throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
                     Log::info('email-sent', [$this->file]);
@@ -153,7 +153,7 @@ 
                 $institutionClass = new Institution_class();
                 $institutionClass->updateClassCount($this->file);
             }
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
             $failures = $e->failures();
             throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
@@ -168,7 +168,7 @@ 
             '*.full_name' => 'required|regex:/^[a-zA-Z .]*$/u|max:256',
             '*.preferred_name' => 'nullable|regex:/^[a-zA-Z .]*$/u|max:90',
             '*.gender_mf' => 'required|in:M,F',
-            '*.date_of_birth_yyyy_mm_dd' => 'date|required|admission_age:' . $this->file['institution_class_id'],
+            '*.date_of_birth_yyyy_mm_dd' => 'date|required|admission_age:'.$this->file['institution_class_id'],
             '*.address' => 'nullable',
             '*.birth_registrar_office_as_in_birth_certificate' => 'nullable|exists:area_administratives,name|required_if:identity_type,BC|birth_place',
             '*.birth_divisional_secretariat' => 'nullable|exists:area_administratives,name|required_with:birth_registrar_office_as_in_birth_certificate',
@@ -177,8 +177,8 @@ 
             '*.identity_number' => 'nullable|identity:identity_type|required_with:*.identity_type',
             '*.education_grade' => 'required',
             '*.option_*' => 'nullable|exists:education_subjects,name',
-            '*.bmi_height' => 'bail|required_with:*.bmi_weight|bmi:' . $this->file['institution_class_id'],
-            '*.bmi_weight' => 'bail|required_with:*.bmi_height|bmi:' . $this->file['institution_class_id'],
+            '*.bmi_height' => 'bail|required_with:*.bmi_weight|bmi:'.$this->file['institution_class_id'],
+            '*.bmi_weight' => 'bail|required_with:*.bmi_height|bmi:'.$this->file['institution_class_id'],
             '*.bmi_date_yyyy_mm_dd' => 'bail|required_with:*.bmi_height|date', //bmi:'. $this->file['institution_class_id'].'
             '*.bmi_academic_period' => 'bail|required_with:*.bmi_height|exists:academic_periods,name',
             '*.admission_no' => 'required|max:12|min:4|regex:/^[A-Za-z0-9\/]+$/',

app/Models/Institution_student.php

Spacing +34 added lines, -34 removed lines

@@ -71,7 +71,7 @@ 
     public static function boot()
     {
         parent::boot();
-        self::creating(function ($model) {
+        self::creating(function($model) {
             $model->id = (string) Uuid::generate(4);
             $model->created = now();
         });
@@ -129,9 +129,9 @@ 
                 'institution_students.admission_id'
             )
             ->where('institution_students.institution_id', $institutionGrade['institution_id'])
-            ->where('institution_students.student_status_id',1)
+            ->where('institution_students.student_status_id', 1)
             ->where('institution_students.education_grade_id', $institutionGrade['education_grade_id'])
-            ->where('institution_students.deleted_at',null)
+            ->where('institution_students.deleted_at', null)
             ->where('institution_students.academic_period_id', $academicPeriod->id)->get()->toArray();
     }
 
@@ -144,7 +144,7 @@ 
      */
     public static function createExaminationData($student, $admissionInfo)
     {
-        $student['sp_center'] = gettype((int)$student['sp_center']) == 'integer' ?  $student['sp_center'] : 0;
+        $student['sp_center'] = gettype((int) $student['sp_center']) == 'integer' ? $student['sp_center'] : 0;
         try {
             self::create([
                 'id' => (string) Uuid::generate(4),
@@ -169,7 +169,7 @@ 
                 'created' => now(),
                 'created_user_id' => 1
             ]);
-        } catch (\Throwable $th) {
+        }catch (\Throwable $th) {
             Log::error($th);
         }
     }
@@ -183,7 +183,7 @@ 
      */
     public static function updateExaminationData($student, $admissionInfo)
     {
-        $student['sp_center'] = gettype((int)$student['sp_center']) == 'integer' ?  $student['sp_center'] : 0;
+        $student['sp_center'] = gettype((int) $student['sp_center']) == 'integer' ? $student['sp_center'] : 0;
         try {
             self::where([
                 'student_id' => $student['student_id'],
@@ -204,55 +204,55 @@ 
                     'modified_user_id' => 1
                 ]
             );
-        } catch (\Throwable $th) {
+        }catch (\Throwable $th) {
             Log::error($th);
         }
     }
 
-    public static function updateStudentArea(array $student){
-        $father = Student_guardian::where('student_id',$student['student_id'])
-        ->join('security_users as sg','guardian_id', 'sg.id')
-        ->where('guardian_relation_id',1)
+    public static function updateStudentArea(array $student) {
+        $father = Student_guardian::where('student_id', $student['student_id'])
+        ->join('security_users as sg', 'guardian_id', 'sg.id')
+        ->where('guardian_relation_id', 1)
         ->get()->first();
 
-        $mother = Student_guardian::where('student_id',$student['student_id'])
-        ->join('security_users as sg','guardian_id', 'sg.id')
-        ->where('guardian_relation_id',2)
+        $mother = Student_guardian::where('student_id', $student['student_id'])
+        ->join('security_users as sg', 'guardian_id', 'sg.id')
+        ->where('guardian_relation_id', 2)
         ->get()->first();
 
-        $guardian = Student_guardian::where('student_id',$student['student_id'])
-        ->join('security_users as sg','guardian_id', 'sg.id')
-        ->where('guardian_relation_id',3)
+        $guardian = Student_guardian::where('student_id', $student['student_id'])
+        ->join('security_users as sg', 'guardian_id', 'sg.id')
+        ->where('guardian_relation_id', 3)
         ->get()->first();
     
-        if(!is_null($father) && is_null($mother) && is_null($guardian)){
-            Security_user::where('id',$student['student_id'])
+        if (!is_null($father) && is_null($mother) && is_null($guardian)) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $father->address_area_id]);
-        }elseif(!is_null($mother)  && (is_null($father) && is_null($guardian))){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($mother) && (is_null($father) && is_null($guardian))) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($guardian) && is_null($father) && is_null($mother)){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($guardian) && is_null($father) && is_null($mother)) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
-        }elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id ==  $mother->address_area_id)){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($mother) && !is_null($father) && ($father->address_area_id == $mother->address_area_id)) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id !==  $mother->address_area_id) && !is_null($guardian)){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($mother) && !is_null($father) && ($father->address_area_id !== $mother->address_area_id) && !is_null($guardian)) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
-        }elseif(!is_null($father) && $father->address == $student['address']){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($father) && $father->address == $student['address']) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $father->address_area_id]);
-        }elseif(!is_null($mother) && $mother->address == $student['address']){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($mother) && $mother->address == $student['address']) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($guardian) && $guardian->address == $student['address']){
-            Security_user::where('id',$student['student_id'])
+        }elseif (!is_null($guardian) && $guardian->address == $student['address']) {
+            Security_user::where('id', $student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
         }
     }
 
-    public static function createOrUpdate($studentId,$row,$params,$file){
+    public static function createOrUpdate($studentId, $row, $params, $file) {
         self::create([
             'student_status_id' => 1,
             'student_id' => $studentId,

Braces +7 added lines, -7 removed lines

@@ -228,25 +228,25 @@
         if(!is_null($father) && is_null($mother) && is_null($guardian)){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $father->address_area_id]);
-        }elseif(!is_null($mother)  && (is_null($father) && is_null($guardian))){
+        } elseif(!is_null($mother)  && (is_null($father) && is_null($guardian))){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($guardian) && is_null($father) && is_null($mother)){
+        } elseif(!is_null($guardian) && is_null($father) && is_null($mother)){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
-        }elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id ==  $mother->address_area_id)){
+        } elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id ==  $mother->address_area_id)){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id !==  $mother->address_area_id) && !is_null($guardian)){
+        } elseif(!is_null($mother)  && !is_null($father) && ($father->address_area_id !==  $mother->address_area_id) && !is_null($guardian)){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
-        }elseif(!is_null($father) && $father->address == $student['address']){
+        } elseif(!is_null($father) && $father->address == $student['address']){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $father->address_area_id]);
-        }elseif(!is_null($mother) && $mother->address == $student['address']){
+        } elseif(!is_null($mother) && $mother->address == $student['address']){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $mother->address_area_id]);
-        }elseif(!is_null($guardian) && $guardian->address == $student['address']){
+        } elseif(!is_null($guardian) && $guardian->address == $student['address']){
             Security_user::where('id',$student['student_id'])
             ->update(['address_area_id' => $guardian->address_area_id]);
         }

app/Providers/ValidatorExtended.php

Spacing +34 added lines, -34 removed lines

@@ -70,17 +70,17 @@ 
             if (empty($value)) {
                 return false;
             } elseif ($gradeEntity !== null) {
-                $admissionAge = (($gradeEntity->admission_age) * 12) - 1;
+                $admissionAge = (($gradeEntity->admission_age)*12) - 1;
                 $to = $academicPeriod->start_date;
                 $diff_in_months = $to->diffInMonths($value);
                 $ageOfStudent = $diff_in_months;
                 $enrolmentMaximumAge = $admissionAge + 120;
                 return ($ageOfStudent <= $enrolmentMaximumAge) && ($ageOfStudent >= $admissionAge);
-            } else {
+            }else {
                 return false;
             }
-        } else {
-            $this->_custom_messages['admission_age'] = 'given' . $attribute . 'Not found';
+        }else {
+            $this->_custom_messages['admission_age'] = 'given'.$attribute.'Not found';
             $this->_set_custom_stuff();
             return false;
         }
@@ -91,16 +91,16 @@ 
 
         if (is_numeric($value)) {
             if ($value < 10) {
-                $this->_custom_messages['bmi'] =  $attribute . ' is must greater than 10';
+                $this->_custom_messages['bmi'] = $attribute.' is must greater than 10';
                 $this->_set_custom_stuff();
                 return false;
             } elseif ($value > 250) {
-                $this->_custom_messages['bmi'] =  $attribute . ' is must smaller than 250';
+                $this->_custom_messages['bmi'] = $attribute.' is must smaller than 250';
                 $this->_set_custom_stuff();
                 return false;
             }
-        } else {
-            $this->_custom_messages['bmi'] =  $attribute . ' is must a valid numeric';
+        }else {
+            $this->_custom_messages['bmi'] = $attribute.' is must a valid numeric';
             $this->_set_custom_stuff();
             return false;
         }
@@ -109,18 +109,18 @@ 
 
     protected function validateBmi($attribute, $value, $parameters)
     {
-        $bmiGrades =  ['G1', 'G4', 'G7', 'G10'];
+        $bmiGrades = ['G1', 'G4', 'G7', 'G10'];
         $institutionGrade = Institution_class_grade::where('institution_class_id', '=', $parameters[0])
             ->join('education_grades', 'institution_class_grades.education_grade_id', 'education_grades.id')
             ->first();
-        $educationGrade =  Education_grade::where('id', '=', $institutionGrade->education_grade_id)->first();
+        $educationGrade = Education_grade::where('id', '=', $institutionGrade->education_grade_id)->first();
         if (in_array($institutionGrade->code, $bmiGrades)) {
             if (!empty($value)) {
                 if (($attribute == 'bmi_height') || ('bmi_weight')) {
                     return $this->validateHW($attribute, $value);
                 }
-            } else {
-                $this->_custom_messages['bmi'] =  $attribute . ' is required for ' . $educationGrade->name;
+            }else {
+                $this->_custom_messages['bmi'] = $attribute.' is required for '.$educationGrade->name;
                 $this->_set_custom_stuff();
                 return false;
             }
@@ -128,7 +128,7 @@ 
             if (($attribute == 'bmi_height') || ('bmi_weight')) {
                 return $this->validateHW($attribute, $value);
             }
-        } else {
+        }else {
             return true;
         }
     }
@@ -137,19 +137,19 @@ 
     {
         foreach ($validator->getData() as $data) {
             if ($data['identity_type'] == 'BC' && key_exists('birth_divisional_secretariat', $data)) {
-                $BirthDivision = Area_administrative::where('name', '=',  '%' . $data['birth_divisional_secretariat'] . '%')->where('area_administrative_level_id', '=', 5); //
+                $BirthDivision = Area_administrative::where('name', '=', '%'.$data['birth_divisional_secretariat'].'%')->where('area_administrative_level_id', '=', 5); //
                 if ($BirthDivision->count() > 0) {
-                    $BirthArea = Area_administrative::where('name', '=', '%' . $value . '%') //$data['birth_registrar_office_as_in_birth_certificate']
+                    $BirthArea = Area_administrative::where('name', '=', '%'.$value.'%') //$data['birth_registrar_office_as_in_birth_certificate']
                         ->where('parent_id', '=', $BirthDivision->first()->id)->count();
-                    return $BirthArea  > 0;
+                    return $BirthArea > 0;
                 } elseif (key_exists('birth_divisional_secretariat', $data) && (!key_exists('birth_registrar_office_as_in_birth_certificate', $data))) {
                     $this->_custom_messages['birth_place'] = 'birth_registrar_office_as_in_birth_certificate required with BC';
                     $this->_set_custom_stuff();
                     return false;
-                } else {
+                }else {
                     return true;
                 }
-            } else {
+            }else {
                 return true;
             }
         }
@@ -157,16 +157,16 @@ 
 
     protected function validateIsStudentInClass($attribute, $value, $perameters, $validator)
     {
-        $student =  Security_user::where('openemis_no', '=', $value);
+        $student = Security_user::where('openemis_no', '=', $value);
         if ($student->count() > 0) {
             $student = $student->first()->toArray();
-            $check =  Institution_class_student::where('student_id', '=', $student['id'])->where('institution_class_id', '=', $perameters[0])->count();
+            $check = Institution_class_student::where('student_id', '=', $student['id'])->where('institution_class_id', '=', $perameters[0])->count();
             if ($check == 1) {
                 return true;
-            } else {
+            }else {
                 return false;
             }
-        } else {
+        }else {
             return false;
         }
     }
@@ -177,7 +177,7 @@ 
     {
         $valid = true;
         foreach ($validator->getData() as $data) {
-            switch($data[$perameters[0]]){
+            switch ($data[$perameters[0]]) {
                 case 'BC':
                     $valid = preg_match('/^([0-9]{3,5})/i', $value);
                     break;
@@ -190,10 +190,10 @@ 
         }
         
         if ($valid == 0) {
-            $this->_custom_messages['identity'] = $attribute . ' is not valid  Please check the NIC number';
+            $this->_custom_messages['identity'] = $attribute.' is not valid  Please check the NIC number';
             $this->_set_custom_stuff();
             return false;
-        } else {
+        }else {
             return true;
         }
     }
@@ -201,7 +201,7 @@ 
     protected function validateUserUnique($attribute, $value, $perameters, $validator)
     {
         foreach ($validator->getData() as $data) {
-            $identityType = Identity_type::where('national_code', 'like', '%' . $data['identity_type'] . '%')->first();
+            $identityType = Identity_type::where('national_code', 'like', '%'.$data['identity_type'].'%')->first();
             if ($identityType !== null && ($value !== null)) {
                 if ($identityType->national_code === 'BC') {
                     return $this->checkUnique($value, $data, $identityType);
@@ -217,14 +217,14 @@ 
     protected function validateIsBc($attribute, $value, $perameters, $validator)
     {
         foreach ($validator->getData() as $data) {
-            $identityType = Identity_type::where('national_code', 'like', '%' . $data['identity_type'] . '%')->first();
+            $identityType = Identity_type::where('national_code', 'like', '%'.$data['identity_type'].'%')->first();
             if (($identityType !== null) && ($identityType !== "")) {
                 if (($identityType->national_code) === 'BC') {
                     return (strlen((string) $data['identity_number']) < 7);
-                } else {
+                }else {
                     return true;
                 }
-            } else {
+            }else {
                 return true;
             }
         }
@@ -234,24 +234,24 @@ 
     {
         $isUnique = Security_user::where('identity_number', '=', $value)->where('identity_type_id', '=', $identityType->id);
         if ($isUnique->count() > 0) {
-            $this->_custom_messages['user_unique'] = 'The identity number already in use. User ID is : ' . $isUnique->first()->openemis_no;
+            $this->_custom_messages['user_unique'] = 'The identity number already in use. User ID is : '.$isUnique->first()->openemis_no;
             $this->_set_custom_stuff();
             return false;
-        } else {
+        }else {
             return true;
         }
     }
 
     protected function IsBc($data, $value)
     {
-        $identityType = Identity_type::where('national_code', 'like', '%' . $data['identity_type'] . '%')->first();
+        $identityType = Identity_type::where('national_code', 'like', '%'.$data['identity_type'].'%')->first();
         if ($identityType !== null) {
             if (($identityType->national_code) === 'BC' && strlen((string) $value) < 8) {
                 return false;
-            } else {
+            }else {
                 return true;
             }
-        } else {
+        }else {
             return true;
         }
     }

app/Console/Commands/ExaminationStudentSeed.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
         $output->writeln('###########################################------Inserting file records------###########################################');
        
-        ExaminationStudentsController::callOnClick($this->argument('year'),$this->argument('grade'));
+        ExaminationStudentsController::callOnClick($this->argument('year'), $this->argument('grade'));
         $output->writeln('###########################################------Finished inserting file records------###########################################');
     }
 }

app/Console/Commands/ExaminationsUpdateCensusNo.php

Spacing +2 added lines, -2 removed lines

@@ -42,9 +42,9 @@
         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
         $output->writeln('###########################################------Inserting file records------###########################################');
         $this->examinationController = new ExaminationStudentsController(2019, 'G4');
-        $students =  Examination_student::all()->toArray();
+        $students = Examination_student::all()->toArray();
         $func = $this->examinationController;
-        array_walk($students,array($func,'updateCensusNo'));
+        array_walk($students, array($func, 'updateCensusNo'));
 
                 
     }

app/Http/Controllers/ExaminationStudentsController.php

Indentation +4 added lines, -4 removed lines

@@ -192,7 +192,7 @@ 
                 break;
             case 'empty';
                 $students = Examination_student::
-                     whereNull('nsid')
+                        whereNull('nsid')
                     ->where('grade', $this->grade)
                     ->where('year', $this->year)
                     ->offset($offset)
@@ -323,13 +323,13 @@ 
                 $sis_student['student_id'] =  $student['id' ];
 
                 $student = $this->setIsTakingExam($student);
-                 if (count($institutionClass) == 1) {
+                    if (count($institutionClass) == 1) {
                     $admissionInfo['instituion_class'] = $institutionClass[0];
-                     Institution_student::createExaminationData($student, $admissionInfo);
+                        Institution_student::createExaminationData($student, $admissionInfo);
                     Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_class_student::createExaminationData($student, $admissionInfo);
                 } else {
-                     Institution_student_admission::createExaminationData($student, $admissionInfo);
+                        Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_student::createExaminationData($student, $admissionInfo);
                 }
                 

Spacing +39 added lines, -39 removed lines

@@ -30,7 +30,7 @@ 
         $this->grade = $grade;
         $this->student = new Security_user();
         $this->examination_student = new Examination_student();
-        $this->academic_period =  Academic_period::where('code', '=', $this->year)->first();
+        $this->academic_period = Academic_period::where('code', '=', $this->year)->first();
         $this->education_grade = Education_grade::where('code', '=', $this->grade)->first();
         $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();
     }
@@ -71,10 +71,10 @@ 
                     );
                     Session::flash('message', 'File upload successfully!');
                     // Redirect to index
-                } else {
+                }else {
                     Session::flash('message', 'File too large. File must be less than 20MB.');
                 }
-            } else {
+            }else {
                 Session::flash('message', 'Invalid File Extension.');
             }
         }
@@ -96,7 +96,7 @@ 
             $import->import($excelFile, 'local', \Maatwebsite\Excel\Excel::CSV);
             if ($import->failures()->count() > 0) {
                 $errors = $import->failures();
-                $columns =  [
+                $columns = [
                     'remarks',
                     'st_no',
                     'stu_no',
@@ -117,10 +117,10 @@ 
                 Storage::put($file, implode(',', $columns));
 
                 foreach ($errors as $error) {
-                    Storage::append($file, implode(':', $error->errors()) . ',' . implode(',', $error->values()));
+                    Storage::append($file, implode(':', $error->errors()).','.implode(',', $error->values()));
                 }
             }
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
         }
     }
 
@@ -143,7 +143,7 @@ 
                 $Institution = $Institution[0];
                 if (count($student) == 1) {
                     $student = $student[0];
-                    if (((int)$Institution['id']) !=  ((int)$student['institution_id'])) {
+                    if (((int) $Institution['id']) != ((int) $student['institution_id'])) {
                         $studentClass = Institution_class_student::where('student_id', $student['student_id'])
                             ->first();
                         Institution_class_student::where('student_id', $student['student_id'])->delete();
@@ -153,12 +153,12 @@ 
                         if (!is_null($studentClass)) {
                             $class->updateClassCount($studentClass->toArray());
                         }
-                        $output->writeln('updated student info:' . $data['nsid']);
+                        $output->writeln('updated student info:'.$data['nsid']);
                     }
-                } else {
+                }else {
                     Institution_student::where('institution_id', '<>', $Institution['id'])->where('student_id', $student[0]['student_id'])
                         ->delete();
-                    $output->writeln('updated student info:' . $Institution['id'] . '==' . $Institution['id']);
+                    $output->writeln('updated student info:'.$Institution['id'].'=='.$Institution['id']);
                 }
             }
         }
@@ -174,7 +174,7 @@ 
         $students = [];
         switch ($mode) {
             case 'duplicate':
-                $students =  DB::table('examination_students as es')
+                $students = DB::table('examination_students as es')
                     ->select(DB::raw('count(*) as total'), 'e2.*')
                     ->where('grade', $this->grade)
                     ->where('year', $this->year)
@@ -186,7 +186,7 @@ 
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining duplicate');
+                $this->output->writeln(count($students).'students remaining duplicate');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
@@ -199,7 +199,7 @@ 
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining empty');
+                $this->output->writeln(count($students).'students remaining empty');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
@@ -209,7 +209,7 @@ 
                     ->where('year', $this->year)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining with wrong NSID');
+                $this->output->writeln(count($students).'students remaining with wrong NSID');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
@@ -220,7 +220,7 @@ 
                     ->count();
                 $all = Examination_student::select('nsid')
                     ->count();
-                $this->output->writeln($all . 'Total Unique nsid are: ' . $count);
+                $this->output->writeln($all.'Total Unique nsid are: '.$count);
                 break;
             default:
                 $students = Examination_student::offset($offset)
@@ -229,7 +229,7 @@ 
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining empty');
+                $this->output->writeln(count($students).'students remaining empty');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
         }
@@ -277,7 +277,7 @@ 
      */
     public function clone($student)
     {
-        $student = (array)json_decode(json_encode($student));
+        $student = (array) json_decode(json_encode($student));
         //get student matching with same dob and gender
 
         $matchedStudent = $this->getMatchingStudents($student);
@@ -296,13 +296,13 @@ 
                     'academic_period_id' => $this->academic_period->id
                 ]
             )
-            ->where( 'education_grade_id','in', [$this->student->education_garde_id])
+            ->where('education_grade_id', 'in', [$this->student->education_garde_id])
             ->join('institution_class_grades', 'institution_classes.id', 'institution_class_grades.institution_class_id')->get()->toArray();
 
 
-            if(!is_null($this->student->education_garde_id)){
+            if (!is_null($this->student->education_garde_id)) {
                 $this->education_grade = Education_grade::where('id', '=', $this->student->education_garde_id)->first();
-            }else{
+            }else {
                 $this->student->education_garde_id = $this->education_grade->id;
             }
             
@@ -320,7 +320,7 @@ 
 
                 //TODO implement insert student to admission table
                 $student['id'] = $sis_student['id'];
-                $sis_student['student_id'] =  $student['id' ];
+                $sis_student['student_id'] = $student['id'];
 
                 $student = $this->setIsTakingExam($student);
                  if (count($institutionClass) == 1) {
@@ -328,14 +328,14 @@ 
                      Institution_student::createExaminationData($student, $admissionInfo);
                     Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_class_student::createExaminationData($student, $admissionInfo);
-                } else {
+                }else {
                      Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_student::createExaminationData($student, $admissionInfo);
                 }
                 
                 $this->updateStudentId($student, $sis_student);
                 // update the matched student's data    
-            } else {
+            }else {
                 $student = $this->setIsTakingExam($student);
                 $studentData = $this->student->updateExaminationStudent($student, $matchedStudent);
                 $matchedStudent = array_merge((array) $student, $matchedStudent);
@@ -343,9 +343,9 @@ 
                 Institution_student::updateExaminationData($studentData, $admissionInfo);
                 $this->updateStudentId($student, $studentData);
             }
-        } else {
+        }else {
 
-            $this->output->writeln('Student ' . $student['st_no'] . ' not imorted' . $student['f_name']);
+            $this->output->writeln('Student '.$student['st_no'].' not imorted'.$student['f_name']);
         }
     }
 
@@ -362,20 +362,20 @@ 
         /**
          */
         $sis_student = $this->student->getMatches($student);
-        $doe_students =  Examination_student::where('gender', $student['gender'])
+        $doe_students = Examination_student::where('gender', $student['gender'])
             ->where('b_date', $student['b_date'])
             ->where('schoolid', $student['schoolid'])
-            ->where('year',$this->year)
-            ->where('grade',$this->grade)
+            ->where('year', $this->year)
+            ->where('grade', $this->grade)
             ->count();
         $count = $this->student->getStudentCount($student);
 
         $studentData = [];
-        $sis_users  = (array) json_decode(json_encode($sis_student), true);
+        $sis_users = (array) json_decode(json_encode($sis_student), true);
         // if the same gender same DOE has more than one 
         if (($doe_students > 1) || ($count > 1)) {
             $studentData = $this->searchSimilarName($student, $sis_users, false);
-        } else {
+        }else {
             $studentData = $this->searchSimilarName($student, $sis_users);
         }
         return $studentData;
@@ -425,7 +425,7 @@ 
         }
 
         if (count($matches) > 1) {
-            $highest =  $this->searchSimilarName($student, $sis_students, false);
+            $highest = $this->searchSimilarName($student, $sis_students, false);
         }
 
         return $highest;
@@ -441,7 +441,7 @@ 
     public function updateStudentId($student, $sis_student)
     {
         try {
-            $student['nsid'] =  $sis_student['openemis_no'];
+            $student['nsid'] = $sis_student['openemis_no'];
             // add new NSID to the examinations data set
             unset($student['id']);
             unset($student['taking_g5_exam']);
@@ -449,11 +449,11 @@ 
             unset($student['taking_ol_exam']);
             unset($student['taking_git_exam']);
             unset($student['total']);
-            $students['updated_at'] =  now();
+            $students['updated_at'] = now();
             $this->examination_student->where('st_no', $student['st_no'])->update($student);
             unset($student['st_no']);
-            $this->output->writeln('Updated  to NSID' . $sis_student['openemis_no']);
-        } catch (\Exception $th) {
+            $this->output->writeln('Updated  to NSID'.$sis_student['openemis_no']);
+        }catch (\Exception $th) {
             $this->output->writeln('error');
             Log::error($th);
         }
@@ -468,9 +468,9 @@ 
     {
         $adminUser = Security_user::where('username', 'admin')->first();
         try {
-            (new ExaminationStudentsExport($this->year,$this->grade))->store('examination/student_data_with_nsid.' . time() . '.csv');
+            (new ExaminationStudentsExport($this->year, $this->grade))->store('examination/student_data_with_nsid.'.time().'.csv');
             (new ExportReady($adminUser));
-        } catch (\Throwable $th) {
+        }catch (\Throwable $th) {
             //throw $th;
         }
         return back()->withSuccess('Export started!');
@@ -479,14 +479,14 @@ 
     public function downloadErrors()
     {
 
-        $file_path = storage_path() . '/app/examination/errors.csv';
+        $file_path = storage_path().'/app/examination/errors.csv';
         return Response::download($file_path);
     }
 
     public function downloadProcessedFile()
     {
 
-        $file_path = storage_path() . '/app/examination/student_data_with_nsid.1628499466.csv'; //hard coded file name
+        $file_path = storage_path().'/app/examination/student_data_with_nsid.1628499466.csv'; //hard coded file name
         return Response::download($file_path);
     }
 }

Braces +1 added lines, -1 removed lines

@@ -302,7 +302,7 @@
 
             if(!is_null($this->student->education_garde_id)){
                 $this->education_grade = Education_grade::where('id', '=', $this->student->education_garde_id)->first();
-            }else{
+            } else{
                 $this->student->education_garde_id = $this->education_grade->id;
             }
             

app/Models/Security_user.php

Spacing +24 added lines, -24 removed lines

@@ -204,7 +204,7 @@ 
             $studentData['id'] = $id;
             $this->uniqueUserId->updateOrInsertRecord($studentData);
             return $studentData;
-        } catch (\Exception $th) {
+        }catch (\Exception $th) {
             Log::error($th->getMessage());
             // in case of duplication of the Unique ID this will recursive till success.
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();
@@ -225,7 +225,7 @@ 
         $this->uniqueUserId = new Unique_user_id();
         $this->uniqueUId = new UniqueUid();
         // regenerate unique id if it's not available
-        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'], 9)) ?  $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
+        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'], 9)) ? $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
 
         $studentData = [
             'username' => str_replace('-', '', $uniqueId),
@@ -243,11 +243,11 @@ 
             $studentData['id'] = $sis_student['student_id'];
             $this->uniqueUserId->updateOrInsertRecord($studentData);
             return $studentData;
-        } catch (\Exception $th) {
+        }catch (\Exception $th) {
             Log::error($th);
             // in case of duplication of the Unique ID this will recursive.
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();
-            $this->updateExaminationStudent($student,$sis_student);
+            $this->updateExaminationStudent($student, $sis_student);
         }
     }
 
@@ -255,9 +255,9 @@ 
     {
         try {
             $uniqueUid = new UniqueUid();
-            $BirthArea = Area_administrative::where('name', 'like', '%' . $row['birth_registrar_office_as_in_birth_certificate'] . '%')->first();
-            $nationalityId = Nationality::where('name', 'like', '%' . $row['nationality'] . '%')->first();
-            $identityType = Identity_type::where('national_code', 'like', '%' . $row['identity_type'] . '%')->first();
+            $BirthArea = Area_administrative::where('name', 'like', '%'.$row['birth_registrar_office_as_in_birth_certificate'].'%')->first();
+            $nationalityId = Nationality::where('name', 'like', '%'.$row['nationality'].'%')->first();
+            $identityType = Identity_type::where('national_code', 'like', '%'.$row['identity_type'].'%')->first();
             
             $date = $row['date_of_birth_yyyy_mm_dd'];
             $identityType = $identityType !== null ? $identityType->id : null;
@@ -289,7 +289,7 @@ 
                     ]);
                 case 'update':
                     if (!is_null($row['student_id'])) {
-                        $studentInfo =  Security_user::where('openemis_no',trim($row['student_id']))->first();
+                        $studentInfo = Security_user::where('openemis_no', trim($row['student_id']))->first();
                         self::query()->where('openemis_no', $row['student_id'])
                             ->update([
                                 'first_name' => $row['full_name'] ? $row['full_name'] : $studentInfo['first_name'], // here we save full name in the column of first name. re reduce breaks of the system.
@@ -310,7 +310,7 @@ 
                     }
                     break;
             }
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             dd($e);
         }
     }
@@ -319,48 +319,48 @@ 
     {
         try {
             $uniqueUid = new UniqueUid();
-            $AddressArea = Area_administrative::where('name', 'like', '%' . $row[$prefix . 's_address_area'] . '%')->first();
-            $nationalityId = Nationality::where('name', 'like', '%' . $row[$prefix . 's_nationality'] . '%')->first();
-            $identityType = Identity_type::where('national_code', 'like', '%' . $row[$prefix . 's_identity_type'] . '%')->first();
+            $AddressArea = Area_administrative::where('name', 'like', '%'.$row[$prefix.'s_address_area'].'%')->first();
+            $nationalityId = Nationality::where('name', 'like', '%'.$row[$prefix.'s_nationality'].'%')->first();
+            $identityType = Identity_type::where('national_code', 'like', '%'.$row[$prefix.'s_identity_type'].'%')->first();
             $openemisNo = $uniqueUid::getUniqueAlphanumeric();
 
             $identityType = ($identityType !== null) ? $identityType->id : null;
             $nationalityId = $nationalityId !== null ? $nationalityId->id : null;
 
             $guardian = null;
-            if (!empty($row[$prefix . 's_identity_number'])) {
+            if (!empty($row[$prefix.'s_identity_number'])) {
                 $guardian = Security_user::where('identity_type_id', '=', $nationalityId)
-                    ->where('identity_number', '=', $row[$prefix . 's_identity_number'])->first();
+                    ->where('identity_number', '=', $row[$prefix.'s_identity_number'])->first();
             }
 
             if (is_null($guardian)) {
                 $guardian = self::create([
                     'username' => str_replace('-', '', $openemisNo),
                     'openemis_no' => $openemisNo,
-                    'first_name' => $row[$prefix . 's_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
-                    'last_name' => genNameWithInitials($row[$prefix . 's_full_name']),
+                    'first_name' => $row[$prefix.'s_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
+                    'last_name' => genNameWithInitials($row[$prefix.'s_full_name']),
                     'gender_id' => 1,
-                    'date_of_birth' => $row[$prefix . 's_date_of_birth_yyyy_mm_dd'],
-                    'address' => $row[$prefix . 's_address'],
+                    'date_of_birth' => $row[$prefix.'s_date_of_birth_yyyy_mm_dd'],
+                    'address' => $row[$prefix.'s_address'],
                     'address_area_id' => $AddressArea ? $AddressArea->id : null,
                     'nationality_id' => $nationalityId,
                     'identity_type_id' => $identityType,
-                    'identity_number' => $row[$prefix . 's_identity_number'],
+                    'identity_number' => $row[$prefix.'s_identity_number'],
                     'is_guardian' => 1,
                     'created_user_id' => $file['security_user_id']
                 ]);
 
 
                 $guardian['guardian_relation_id'] = 1;
-                if (array_key_exists($prefix . 's_phone', $row)) {
-                    $father['contact'] = $row[$prefix . 's_phone'];
+                if (array_key_exists($prefix.'s_phone', $row)) {
+                    $father['contact'] = $row[$prefix.'s_phone'];
                     User_contact::createOrUpdate($father, $file['security_user_id']);
                 }
-            } else {
-                Security_user::where('id', $guardian->id)->update(['address_area_id' => $AddressArea ? $AddressArea->id : null,]);
+            }else {
+                Security_user::where('id', $guardian->id)->update(['address_area_id' => $AddressArea ? $AddressArea->id : null, ]);
             }
             return $guardian;
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             Log::error($e->getMessage(), [$e]);
         }
     }

app/Console/Commands/RollbackStudentFromInstitutes.php

Spacing +4 added lines, -4 removed lines

@@ -65,21 +65,21 @@
             */
             if (!empty($institution_grades)) {
 
-                $this->info('Fetched ' . count($institution_grades) . ' records');
+                $this->info('Fetched '.count($institution_grades).' records');
 
                 foreach ($institution_grades as $institute_grade) {
                     if ($institute_grade->promoted == "2019") {
                         $this->info('Promoted year have already set to 2019');
-                    } else {
+                    }else {
                         $this->info('Updating record =======================================');
                         DB::update("update institution_grades set promoted ='2019' where institution_id = ?", [$this->argument('institution')]);
                         $this->info('Record updated ========================================');
                     }
                 }
-            } else {
+            }else {
                 $this->info('No results!');
             }
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             return $e->getMessage();
         }
     }

app/Console/Commands/RunAddStudentsToInstitutions.php

Indentation +26 added lines, -26 removed lines

@@ -84,32 +84,32 @@
             try{
                 $output->writeln($approvedstudent['student_id']."Updated");
                 Institution_student::create([
-                   'student_status_id' => 1,
-                   'student_id' => $approvedstudent['student_id'],
-                   'education_grade_id' => $approvedstudent['education_grade_id'],
-                   'academic_period_id' => $approvedstudent['academic_period_id'],
-                   'start_date' => $approvedstudent['start_date'],
-                   'start_year' => \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['start_date'])->year , // $approvedstudent['start_date']->format('Y'),
-                   'end_date' => $approvedstudent['end_date'],
-                   'end_year' =>  \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['end_date'])->year , //$approvedstudent['end_date']->format('Y'),
-                   'institution_id' => $approvedstudent['institution_id'],
-                   'admission_id' => $approvedstudent['admission_id'],
-                   'created_user_id' => $approvedstudent['created_user_id'],
-               ]);
-               if(!is_null($approvedstudent['institution_class_id'])){
-                   Institution_class_student::create([
-                       'student_id' => $approvedstudent['student_id'],
-                       'institution_class_id' => $approvedstudent['institution_class_id'],
-                       'education_grade_id' =>  $approvedstudent['education_grade_id'],
-                       'academic_period_id' => $approvedstudent['academic_period_id'],
-                       'institution_id' =>$approvedstudent['institution_id'],
-                       'student_status_id' => 1,
-                       'created_user_id' => $approvedstudent['created_user_id'],
-                   ]);
-               }
-           }catch (\Exception $e){
-              echo $e->getMessage();
-           }
+                    'student_status_id' => 1,
+                    'student_id' => $approvedstudent['student_id'],
+                    'education_grade_id' => $approvedstudent['education_grade_id'],
+                    'academic_period_id' => $approvedstudent['academic_period_id'],
+                    'start_date' => $approvedstudent['start_date'],
+                    'start_year' => \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['start_date'])->year , // $approvedstudent['start_date']->format('Y'),
+                    'end_date' => $approvedstudent['end_date'],
+                    'end_year' =>  \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['end_date'])->year , //$approvedstudent['end_date']->format('Y'),
+                    'institution_id' => $approvedstudent['institution_id'],
+                    'admission_id' => $approvedstudent['admission_id'],
+                    'created_user_id' => $approvedstudent['created_user_id'],
+                ]);
+                if(!is_null($approvedstudent['institution_class_id'])){
+                    Institution_class_student::create([
+                        'student_id' => $approvedstudent['student_id'],
+                        'institution_class_id' => $approvedstudent['institution_class_id'],
+                        'education_grade_id' =>  $approvedstudent['education_grade_id'],
+                        'academic_period_id' => $approvedstudent['academic_period_id'],
+                        'institution_id' =>$approvedstudent['institution_id'],
+                        'student_status_id' => 1,
+                        'created_user_id' => $approvedstudent['created_user_id'],
+                    ]);
+                }
+            }catch (\Exception $e){
+                echo $e->getMessage();
+            }
         }
         $output->writeln('
         ####################################################

Spacing +14 added lines, -14 removed lines

@@ -50,14 +50,14 @@ 
             'id' => $this->argument('institution')
         ])->first();
 
-        if(!is_null($institution)){
+        if (!is_null($institution)) {
             DB::enableQueryLog();
             try {
-                $this->info('adding missing students to the institution ' . $institution->name);
+                $this->info('adding missing students to the institution '.$institution->name);
                 $approvedstudent = Institution_student_admission::select('institution_student_admission.*')
                                     ->leftJoin('institution_students', 'institution_student_admission.student_id', '=', 'institution_students.student_id')
-                                    ->whereIn('status_id',[121,122,123,124])
-                                    ->where('institution_student_admission.institution_id',$institution->id)->get()->toArray();
+                                    ->whereIn('status_id', [121, 122, 123, 124])
+                                    ->where('institution_student_admission.institution_id', $institution->id)->get()->toArray();
     
                 $approvedstudent = array_chunk($approvedstudent, 50);
                 
@@ -69,19 +69,19 @@ 
         }
     }
 
-    protected function addStudents($approvedstudent){
-        array_walk($approvedstudent,array($this,'addStudent')); 
+    protected function addStudents($approvedstudent) {
+        array_walk($approvedstudent, array($this, 'addStudent')); 
     }
 
-    protected function addStudent($approvedstudent){
+    protected function addStudent($approvedstudent) {
         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
         Log::info($approvedstudent);
 //         sleep(1);
-        if((Institution_student::isDuplicated($approvedstudent) == 0)){
+        if ((Institution_student::isDuplicated($approvedstudent) == 0)) {
             dd($approvedstudent);
             $this->count += 1;
-            $this->student = $approvedstudent ;
-            try{
+            $this->student = $approvedstudent;
+            try {
                 $output->writeln($approvedstudent['student_id']."Updated");
                 Institution_student::create([
                    'student_status_id' => 1,
@@ -89,14 +89,14 @@ 
                    'education_grade_id' => $approvedstudent['education_grade_id'],
                    'academic_period_id' => $approvedstudent['academic_period_id'],
                    'start_date' => $approvedstudent['start_date'],
-                   'start_year' => \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['start_date'])->year , // $approvedstudent['start_date']->format('Y'),
+                   'start_year' => \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['start_date'])->year, // $approvedstudent['start_date']->format('Y'),
                    'end_date' => $approvedstudent['end_date'],
-                   'end_year' =>  \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['end_date'])->year , //$approvedstudent['end_date']->format('Y'),
+                   'end_year' =>  \Carbon\Carbon::createFromFormat('Y-m-d H:i:s', $approvedstudent['end_date'])->year, //$approvedstudent['end_date']->format('Y'),
                    'institution_id' => $approvedstudent['institution_id'],
                    'admission_id' => $approvedstudent['admission_id'],
                    'created_user_id' => $approvedstudent['created_user_id'],
                ]);
-               if(!is_null($approvedstudent['institution_class_id'])){
+               if (!is_null($approvedstudent['institution_class_id'])) {
                    Institution_class_student::create([
                        'student_id' => $approvedstudent['student_id'],
                        'institution_class_id' => $approvedstudent['institution_class_id'],
@@ -107,7 +107,7 @@ 
                        'created_user_id' => $approvedstudent['created_user_id'],
                    ]);
                }
-           }catch (\Exception $e){
+           }catch (\Exception $e) {
               echo $e->getMessage();
            }
         }

Braces +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
                 
                 array_walk($approvedstudent, array($this, 'addStudents'));
 
-            }catch (\Exception $e) {
+            } catch (\Exception $e) {
                 Log::error($e);
             }       
         }
@@ -107,7 +107,7 @@ 
                        'created_user_id' => $approvedstudent['created_user_id'],
                    ]);
                }
-           }catch (\Exception $e){
+           } catch (\Exception $e){
               echo $e->getMessage();
            }
         }