moe-lk / bulk-upload

app/Models/Institution_student.php

Spacing +31 added lines, -31 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();
         });
@@ -142,7 +142,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),
@@ -167,7 +167,7 @@ 
                 'created' => now(),
                 'created_user_id' => 1
             ]);
-        } catch (\Throwable $th) {
+        }catch (\Throwable $th) {
             Log::error($th);
         }
     }
@@ -181,7 +181,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'],
@@ -202,50 +202,50 @@ 
                     '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' => $guardian->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]);
         }
     }

Braces +7 added lines, -7 removed lines

@@ -226,25 +226,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' => $guardian->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/Console/Commands/UpdateClassEntriyWithZeroId.php

Indentation +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
 
     public function process($student)
     {
-       try{
+        try{
         $wrongStudentsClass = Institution_class_student::where('institution_id', $student['institution_id'])
             ->whereRaw('institution_class_id not in (select id from institution_classes where institution_id ='.$student['institution_id'].' )')
             ->orWhere('institution_class_id', 0)
@@ -142,8 +142,8 @@ 
                 array_walk($institutionClassStudent, array($this->class, 'updateClassCount'));
             }
         }
-       }catch(\Exception $e){
+        }catch(\Exception $e){
             dd($e);
-       }
+        }
     }
 }

Spacing +10 added lines, -10 removed lines

@@ -51,10 +51,10 @@ 
 
         if ($this->argument('code') !== 'All') {
             $institutions = Institution::where('code', $this->argument('code'))->get()->toArray();
-            processParallel(array($this,'processInstitution'),$institutions,$this->argument('max'));
-        } else {
+            processParallel(array($this, 'processInstitution'), $institutions, $this->argument('max'));
+        }else {
             $institutions = Institution::where('institution_status_id', 1)->get()->toArray();
-            processParallel(array($this,'processInstitution'),$institutions,$this->argument('max'));
+            processParallel(array($this, 'processInstitution'), $institutions, $this->argument('max'));
         }
     }
 
@@ -65,16 +65,16 @@ 
             ->where('institutions.id', $institution['id'])
             ->get()->toArray();
         if (count($students) > 0) {
-            array_walk($students,array($this, 'process'));
-            $this->output->writeln("institution :" .$institution['code']. ' cleaned');
-        } else {
-            $this->output->writeln("all records are cleaned at  :".$institution['code'] );
+            array_walk($students, array($this, 'process'));
+            $this->output->writeln("institution :".$institution['code'].' cleaned');
+        }else {
+            $this->output->writeln("all records are cleaned at  :".$institution['code']);
         }
     }
 
     public function process($student)
     {
-       try{
+       try {
         $wrongStudentsClass = Institution_class_student::where('institution_id', $student['institution_id'])
             ->whereRaw('institution_class_id not in (select id from institution_classes where institution_id ='.$student['institution_id'].' )')
             ->orWhere('institution_class_id', 0)
@@ -88,7 +88,7 @@ 
 
             array_walk($wrongStudentsClass, array($this->class, 'updateClassCount'));
 
-            $institutionClass =  Institution_class::getGradeClasses($student['education_grade_id'], $student['institution_id']);
+            $institutionClass = Institution_class::getGradeClasses($student['education_grade_id'], $student['institution_id']);
 
             if (count($institutionClass) == 1) {
                 $start_date = new Carbon($student['start_date']);
@@ -142,7 +142,7 @@ 
                 array_walk($institutionClassStudent, array($this->class, 'updateClassCount'));
             }
         }
-       }catch(\Exception $e){
+       }catch (\Exception $e) {
             dd($e);
        }
     }

Braces +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
                 array_walk($institutionClassStudent, array($this->class, 'updateClassCount'));
             }
         }
-       }catch(\Exception $e){
+       } catch(\Exception $e){
             dd($e);
        }
     }

app/Console/Commands/RemoveDuplicatedGuardians.php

Indentation +5 added lines, -5 removed lines

@@ -52,16 +52,16 @@
     }
 
     public function delete(){
-       try{
-           DB::statement("UPDATE  student_guardians t1
+        try{
+            DB::statement("UPDATE  student_guardians t1
            INNER JOIN student_guardians t2 
                set t1.deleted_at=now() 
            WHERE 
                t1.id > t2.id AND
                t1.student_id = t2.student_id AND
                t1.guardian_id = t2.guardian_id");
-       }catch(\Exception $e){
-           dd($e);
-       }
+        }catch(\Exception $e){
+            dd($e);
+        }
     }
 }

Spacing +6 added lines, -6 removed lines

@@ -45,14 +45,14 @@ 
             $this->output->writeln('############### Starting delete Duplication ################');
             Student_guardian::withTrashed()->restore();
             $this->delete();
-            $this->end_time  = microtime(TRUE);
-            $this->output->writeln('The cook took ' . ($this->end_time - $this->start_time) . ' seconds to complete');
-        } catch (\Throwable $th) {
+            $this->end_time = microtime(TRUE);
+            $this->output->writeln('The cook took '.($this->end_time - $this->start_time).' seconds to complete');
+        }catch (\Throwable $th) {
         }
     }
 
-    public function delete(){
-       try{
+    public function delete() {
+       try {
            DB::statement("UPDATE  student_guardians t1
            INNER JOIN student_guardians t2 
                set t1.deleted_at=now() 
@@ -60,7 +60,7 @@ 
                t1.id > t2.id AND
                t1.student_id = t2.student_id AND
                t1.guardian_id = t2.guardian_id");
-       }catch(\Exception $e){
+       }catch (\Exception $e) {
            dd($e);
        }
     }

Braces +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
                t1.id > t2.id AND
                t1.student_id = t2.student_id AND
                t1.guardian_id = t2.guardian_id");
-       }catch(\Exception $e){
+       } catch(\Exception $e){
            dd($e);
        }
     }

app/Console/Commands/CloneConfigData.php

Braces +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         if(count($shift) > 0){
             // processParallel($function,$shift, $this->argument('max'),$params);
             array_walk($shift,$function,$params);
-        }else{
+        } else{
             $this->output->writeln('Nothing to clone');
         }
         $this->end_time = microtime(TRUE);

Spacing +6 added lines, -6 removed lines

@@ -54,8 +54,8 @@ 
         $academicPeriod = $this->academic_period->getAcademicPeriod($year);
         $mode = $this->argument('mode') == 'AL' ? true : false; 
         $previousAcademicPeriodYear = $academicPeriod->order;
-        $previousAcademicPeriod = Academic_period::where('order',$previousAcademicPeriodYear+1)->first();
-        $shift = $this->shifts->getShiftsToClone($previousAcademicPeriod->code,$this->argument(('max')),$mode);
+        $previousAcademicPeriod = Academic_period::where('order', $previousAcademicPeriodYear + 1)->first();
+        $shift = $this->shifts->getShiftsToClone($previousAcademicPeriod->code, $this->argument(('max')), $mode);
         $params = [
             'year' => $year,
             'academic_period' => $academicPeriod,
@@ -64,17 +64,17 @@ 
         ];
 
         $function = array($this->clone, 'process');
-        if(count($shift) > 0){
+        if (count($shift) > 0) {
             // processParallel($function,$shift, $this->argument('max'),$params);
-            array_walk($shift,$function,$params);
-        }else{
+            array_walk($shift, $function, $params);
+        }else {
             $this->output->writeln('Nothing to clone');
         }
         $this->end_time = microtime(TRUE);
 
 
         $this->output->writeln('$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$');
-        $this->output->writeln('The cook took ' . ($this->end_time - $this->start_time) . ' seconds to complete');
+        $this->output->writeln('The cook took '.($this->end_time - $this->start_time).' seconds to complete');
         $this->output->writeln('$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$');
     }  
 }

app/Console/Commands/MapStudentArea.php

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      */
     public function __construct()
     {
-        $this->output =  new \Symfony\Component\Console\Output\ConsoleOutput();
+        $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();
         parent::__construct();
     }
 
@@ -41,13 +41,13 @@ 
      */
     public function handle()
     {
-        $students = Security_user::where('is_student',true)->get()->toArray();
-        array_walk($students,array($this,'process'));
+        $students = Security_user::where('is_student', true)->get()->toArray();
+        array_walk($students, array($this, 'process'));
     }
 
-    public function process($student){
+    public function process($student) {
         $student['student_id'] = $student['id'];
         Institution_student::updateStudentArea($student);
-        $this->output->writeln('area updated for student:'. $student['openemis_no']) ;
+        $this->output->writeln('area updated for student:'.$student['openemis_no']);
     }
 }

app/Console/Commands/cleanConfig.php

Spacing +3 added lines, -3 removed lines

@@ -48,16 +48,16 @@
         $year = $this->argument('year');
         $academicPeriod = $this->academic_period->getAcademicPeriod($year);
         $previousAcademicPeriodYear = $academicPeriod->order;
-        $previousAcademicPeriod = Academic_period::where('order',$previousAcademicPeriodYear+1)->first();
+        $previousAcademicPeriod = Academic_period::where('order', $previousAcademicPeriodYear + 1)->first();
 
         $params = [
             'academic_period' => $academicPeriod,
             'previous_academic_period' => $previousAcademicPeriod
         ];
 
-        if($year == '2019' || $year == '2018/19'){
+        if ($year == '2019' || $year == '2018/19') {
             die('Academic Year 2019 or earlier can`t be deleted');
-        }else{
+        }else {
             $this->clone->cleanConfig($params);
         }
     }

Braces +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
         if($year == '2019' || $year == '2018/19'){
             die('Academic Year 2019 or earlier can`t be deleted');
-        }else{
+        } else{
             $this->clone->cleanConfig($params);
         }
     }

app/Http/Controllers/CloneController.php

Indentation +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public function array_walk($shift, $count, $params)
     {
         try{
-           DB::beginTransaction();
+            DB::beginTransaction();
             array_walk($shift, array($this, 'process'), $params);
             DB::commit();
         }catch(\Exception $e){

Spacing +35 added lines, -35 removed lines

@@ -31,21 +31,21 @@ 
         $this->academic_period = new Academic_period();
         $this->institution_classes = new Institution_class();
         $this->institution_class_subjects = new Institution_class_subject();
-        $this->institution_subjects =  new Institution_subject();
+        $this->institution_subjects = new Institution_subject();
         $this->institution_grades = new Institution_grade();
-        $this->education_grade_subjects =  new Education_grades_subject();
+        $this->education_grade_subjects = new Education_grades_subject();
         $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();
     }
 
 
     public function array_walk($shift, $count, $params)
     {
-        try{
+        try {
            DB::beginTransaction();
             array_walk($shift, array($this, 'process'), $params);
             DB::commit();
-        }catch(\Exception $e){
-            $this->output->writeln('Terminating ' . $shift['institution_id']);
+        }catch (\Exception $e) {
+            $this->output->writeln('Terminating '.$shift['institution_id']);
             DB::rollBack();
         }
     }
@@ -59,24 +59,24 @@ 
         $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();
+        $classIds = $this->institution_classes->select('id')->where(['academic_period_id' => $academicPeriod->id])->get()->toArray();
         $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);
+        } while ($deleted > 0);
 
         do {
-            $deleted =  $this->institution_subjects->where('academic_period_id', $academicPeriod->id)->limit(10000)->delete();
+            $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");
+        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'];
@@ -94,14 +94,14 @@ 
 
 
         if ($mode) {
-            $institutionClasses = $this->institution_classes->getShiftClasses($shift,  $mode);
+            $institutionClasses = $this->institution_classes->getShiftClasses($shift, $mode);
             try {
                 array_walk($institutionClasses, array($this, 'updateInstitutionClasses'), $params);
-                $this->output->writeln('updating from ' . $shift['institution_id']);
-            } catch (\Exception $e) {
+                $this->output->writeln('updating from '.$shift['institution_id']);
+            }catch (\Exception $e) {
                 Log::error($e->getMessage(), [$e]);
             }
-        } else {
+        }else {
             $institutionSubjects = $this->institution_grades->getGradeSubjects($shift['institution_id']);
             try {
                 if ($data['created']) {
@@ -112,32 +112,32 @@ 
                         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']);
+                            $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 {
+                        $this->output->writeln('no classes found '.$shift['institution_id']);
                     }
-                } else {
+                }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']);
+                        $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) {
+            }catch (\Exception $e) {
                 Log::error($e->getMessage(), [$e]);
-                $this->output->writeln('Terminating ' . $shift['institution_id']);
+                $this->output->writeln('Terminating '.$shift['institution_id']);
                 DB::rollBack();
             }
         }
@@ -156,7 +156,7 @@ 
             $subject['created'] = now();
             $subject['created_user_id'] = 1;
             $this->institution_subjects->create($subject);
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             DB::rollBack();
             Log::error($e->getMessage(), [$e]);
         }
@@ -185,7 +185,7 @@ 
                         'academic_period_id' => $params['academic_period_id']
                     ]);
             }
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             DB::rollBack();
             Log::error($e->getMessage(), [$e]);
         }
@@ -215,8 +215,8 @@ 
             $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);
+            $this->output->writeln('Create class:'.$class['name']);
+            $class = Institution_class::create($class);
             $institutionClassGrdaeObj['institution_class_id'] = $class->id;
             $institutionClassGrdaeObj['education_grade_id'] = $educationGrdae;
             $institutionClassGrdaeObj['created_user_id'] = $class['created_user_id'];
@@ -228,7 +228,7 @@ 
                 ->toArray();
             $params['class'] = $class;
             $this->insertInstitutionClassSubjects($institutionSubjects, $class);
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             DB::rollBack();
             Log::error($e->getMessage(), [$e]);
         }
@@ -239,8 +239,8 @@ 
         if (!empty($subjects)) {
             try {
                 array_walk($subjects, array($this, 'insertClassSubjects'), $class);
-                $this->output->writeln('updating subjects ' . $class->name);
-            } catch (\Exception $e) {
+                $this->output->writeln('updating subjects '.$class->name);
+            }catch (\Exception $e) {
                 DB::rollBack();
                 Log::error($e->getMessage(), [$e]);
             }
@@ -261,7 +261,7 @@ 
             if (!$this->institution_class_subjects->isDuplicated($subjectobj)) {
                 $this->institution_class_subjects->create($subjectobj);
             }
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             DB::rollBack();
             Log::error($e->getMessage(), [$e]);
         }
@@ -309,19 +309,19 @@ 
                 unset($shift['id']);
                 unset($shift['created']);
                 unset($shift['modified']);
-                $shift = $this->shifts->create((array)$shift);
+                $shift = $this->shifts->create((array) $shift);
                 $data = [
                     'shift_id' => $shift->id,
                     'created' => true
                 ];
-            } else {
+            }else {
                 $data = [
                     'shift_id' => $exist->id,
                     'created' => false
                 ];
             };
             return $data;
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             DB::rollBack();
         }
     }

Braces +1 added lines, -1 removed lines

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

app/Console/Commands/CallPromotionCommand.php

Spacing +7 added lines, -7 removed lines

@@ -48,21 +48,21 @@
         $mode = $this->argument('mode');
         $academicPeriod = $this->academic_period->getAcademicPeriod($year);
         $previousAcademicPeriodYear = $academicPeriod->order;
-        $previousAcademicPeriod = Academic_period::where('order',$previousAcademicPeriodYear+1)->first();
-        $institutions = $this->instituion_grade->getInstitutionGradeList($previousAcademicPeriod->code,$limit,$mode);
+        $previousAcademicPeriod = Academic_period::where('order', $previousAcademicPeriodYear + 1)->first();
+        $institutions = $this->instituion_grade->getInstitutionGradeList($previousAcademicPeriod->code, $limit, $mode);
         $params = [
             'year' => $year,
             'mode'=> $mode
         ];
-        if(in_array($mode,['AL','1-5','SP','6-11'])){
-            array_walk($institutions,array($this,'callPromotion'),$params);
-        }else{
+        if (in_array($mode, ['AL', '1-5', 'SP', '6-11'])) {
+            array_walk($institutions, array($this, 'callPromotion'), $params);
+        }else {
             die('The give mode not support');
         }
        
     }
 
-    protected function callPromotion($institution,$count,$params){
-        $this->call('promote:students',['year' => $params['year'],'institution' => $institution['code'],'mode' => $params['mode'] ]);
+    protected function callPromotion($institution, $count, $params) {
+        $this->call('promote:students', ['year' => $params['year'], 'institution' => $institution['code'], 'mode' => $params['mode']]);
     }
 }

Braces +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         ];
         if(in_array($mode,['AL','1-5','SP','6-11'])){
             array_walk($institutions,array($this,'callPromotion'),$params);
-        }else{
+        } else{
             die('The give mode not support');
         }
        

app/Console/Commands/PromoteStudents.php

Spacing +3 added lines, -3 removed lines

@@ -65,15 +65,15 @@
         $institution = $this->argument('institution');
         $academicPeriod = $this->academic_period->getAcademicPeriod($year);
         $previousAcademicPeriodYear = $academicPeriod->order;
-        $previousAcademicPeriod = Academic_period::where('order',$previousAcademicPeriodYear+1)->first();
+        $previousAcademicPeriod = Academic_period::where('order', $previousAcademicPeriodYear + 1)->first();
         $mode = $this->argument('mode');
-        $institutionGrade = $this->instituion_grade->getInstitutionGradeToPromoted($previousAcademicPeriod->code,$institution,$mode);
+        $institutionGrade = $this->instituion_grade->getInstitutionGradeToPromoted($previousAcademicPeriod->code, $institution, $mode);
         $output->writeln('Start promoting:'.$institution);
         $params = [
             'academicPeriod' => $academicPeriod,
             'previousAcademicPeriod' => $previousAcademicPeriod
         ];
-        (new BulkPromotion())->callback($institutionGrade,$params);
+        (new BulkPromotion())->callback($institutionGrade, $params);
         $output->writeln('Finished promoting:'.$institution);
     }
 }