moe-lk / bulk-upload

app/Imports/StudentImport.php

Indentation +4 added lines, -4 removed lines

@@ -18,10 +18,10 @@
     }
 
     /**
-    * @param array $row
-    *
-    * @return \Illuminate\Database\Eloquent\Model|null
-    */
+     * @param array $row
+     *
+     * @return \Illuminate\Database\Eloquent\Model|null
+     */
 
     public function sheets(): array
     {

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         // TODO: Implement registerEvents() method.
 
         return [
-            BeforeSheet::class => function(BeforeSheet $event){
+            BeforeSheet::class => function(BeforeSheet $event) {
                 $this->sheetNames[] = $event->getSheet()->getTitle();
 
             }

app/Providers/AppServiceProvider.php

Indentation +7 added lines, -7 removed lines

@@ -25,11 +25,11 @@
      *
      * @return void
      */
-     public function boot()
-     {
-         //
-         if((\App::environment('dev')) |  (\App::environment('stage')) | (\App::environment('prod'))) {
-             URL::forceScheme('https');
-         }
-     }
+        public function boot()
+        {
+            //
+            if((\App::environment('dev')) |  (\App::environment('stage')) | (\App::environment('prod'))) {
+                URL::forceScheme('https');
+            }
+        }
 }

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      public function boot()
      {
          //
-         if((\App::environment('dev')) |  (\App::environment('stage')) | (\App::environment('prod'))) {
+         if ((\App::environment('dev')) | (\App::environment('stage')) | (\App::environment('prod'))) {
              URL::forceScheme('https');
          }
      }

app/Providers/ValidatorExtended.php

Indentation +7 added lines, -7 removed lines

@@ -135,13 +135,13 @@
     }
     protected function validateNic($attribute, $value, $perameters, $validator){
         $valid = preg_match('/^([0-9]{9}[VX]|[0-9]{12})$/i', $value);
-       if(!$valid){
-             $this->_custom_messages['nic'] = $attribute. ' is not valid,  Please check the NIC number';
-             $this->_set_custom_stuff();
-             return false;
-       }else{
-           return true;
-       }
+        if(!$valid){
+                $this->_custom_messages['nic'] = $attribute. ' is not valid,  Please check the NIC number';
+                $this->_set_custom_stuff();
+                return false;
+        }else{
+            return true;
+        }
     }
 
     protected function validateUserUnique($attribute, $value, $perameters, $validator) {

Spacing +41 added lines, -41 removed lines

@@ -56,43 +56,43 @@ 
         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;
         }
     }
 
     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')
+        ->join('education_grades', 'institution_class_grades.education_grade_id', 'education_grades.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')){
+        $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')) {
                         $v = Validator::make([$attribute => $value], [
                             $attribute => 'number|min:10|max:200'
                             ]);
-                        if($v->fails()) {
-                            $this->_custom_messages['bmi'] =  $attribute.' is not a valid input';
+                        if ($v->fails()) {
+                            $this->_custom_messages['bmi'] = $attribute.' is not a valid input';
                             $this->_set_custom_stuff();
                             return false;
                         }
                         return true;
                     }
-                }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;
                 }
-            }else{
+            }else {
                 return true;
             }
     }
@@ -100,58 +100,58 @@ 
     protected function validateBirthPlace($attribute, $value, $perameters, $validator) {
         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); //
-                if ($BirthDivision->count() > 0 ) {
-                    $BirthArea = Area_administrative::where('name', '=', '%'. $value.'%') //$data['birth_registrar_office_as_in_birth_certificate']
+                $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']
                                     ->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;
             }
         }
     }
 
     protected function validateIsStudentInClass($attribute, $value, $perameters, $validator) {
-        $student =  Security_user::where('openemis_no', '=', $value);
-        if($student->count() > 0){
+        $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();
-            if($check == 1){
+            $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;
         }
 
     }
-    protected function validateNic($attribute, $value, $perameters, $validator){
+    protected function validateNic($attribute, $value, $perameters, $validator) {
         $valid = preg_match('/^([0-9]{9}[VX]|[0-9]{12})$/i', $value);
-       if(!$valid){
-             $this->_custom_messages['nic'] = $attribute. ' is not valid,  Please check the NIC number';
+       if (!$valid) {
+             $this->_custom_messages['nic'] = $attribute.' is not valid,  Please check the NIC number';
              $this->_set_custom_stuff();
              return false;
-       }else{
+       }else {
            return true;
        }
     }
 
     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);
+                    return $this->checkUnique($value, $data, $identityType);
                 } elseif ($identityType->national_code === 'NIC') {
-                    return $this->checkUnique($value, $data,$identityType);
+                    return $this->checkUnique($value, $data, $identityType);
                 }
             } elseif (($value == null) || $value == "") {
                 return true;
@@ -161,39 +161,39 @@ 
 
     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;
             }
         }
     }
 
-    protected function checkUnique($value, $data,$identityType) {
+    protected function checkUnique($value, $data, $identityType) {
         $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;
         }
     }

Braces +5 added lines, -5 removed lines

@@ -87,12 +87,12 @@ 
                         }
                         return true;
                     }
-                }else{
+                } else{
                     $this->_custom_messages['bmi'] =  $attribute.' is required for '. $educationGrade->name;
                     $this->_set_custom_stuff();
                     return false;
                 }
-            }else{
+            } else{
                 return true;
             }
     }
@@ -125,10 +125,10 @@ 
             $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;
         }
 
@@ -139,7 +139,7 @@ 
              $this->_custom_messages['nic'] = $attribute. ' is not valid,  Please check the NIC number';
              $this->_set_custom_stuff();
              return false;
-       }else{
+       } else{
            return true;
        }
     }

app/Providers/ValidationExtensionServiceProvider.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
     public function boot() {
         $this->app->validator->resolver( function( $translator, $data, $rules,
-                                                   $messages = array(), $customAttributes = array() ) {
+                                                    $messages = array(), $customAttributes = array() ) {
             return new ValidatorExtended( $translator, $data, $rules, $messages,
                 $customAttributes );
         } );

Spacing +4 added lines, -4 removed lines

@@ -10,10 +10,10 @@
     public function register() {}
 
     public function boot() {
-        $this->app->validator->resolver( function( $translator, $data, $rules,
-                                                   $messages = array(), $customAttributes = array() ) {
-            return new ValidatorExtended( $translator, $data, $rules, $messages,
-                $customAttributes );
+        $this->app->validator->resolver(function($translator, $data, $rules,
+                                                   $messages = array(), $customAttributes = array()) {
+            return new ValidatorExtended($translator, $data, $rules, $messages,
+                $customAttributes);
         } );
     }
 }

app/Providers/ShaHashServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 
     public function register()
     {
-        $this->app->singleton('hash',function (){
+        $this->app->singleton('hash', function() {
             return new SHAHasher($this->app);
         });
     }

app/Providers/RouteServiceProvider.php

Indentation +5 added lines, -5 removed lines

@@ -52,8 +52,8 @@ 
     protected function mapWebRoutes()
     {
         Route::middleware('web')
-             ->namespace($this->namespace)
-             ->group(base_path('routes/web.php'));
+                ->namespace($this->namespace)
+                ->group(base_path('routes/web.php'));
     }
 
     /**
@@ -66,8 +66,8 @@ 
     protected function mapApiRoutes()
     {
         Route::prefix('api')
-             ->middleware('api')
-             ->namespace($this->namespace)
-             ->group(base_path('routes/api.php'));
+                ->middleware('api')
+                ->namespace($this->namespace)
+                ->group(base_path('routes/api.php'));
     }
 }

app/Console/Commands/PromoteStudents.php

Indentation +1 added lines, -1 removed lines

@@ -212,7 +212,7 @@
                 'created_user_id' => $student['created_user_id']
             ];
             try{
-               Institution_student::where('id',(string)$student['id'])->update($studentData);
+                Institution_student::where('id',(string)$student['id'])->update($studentData);
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                 $output->writeln('----------------- '. $student['admission_id'] . ' to ' . $studentData['education_grade_id']);
 

Spacing +58 added lines, -58 removed lines

@@ -63,14 +63,14 @@ 
     {
         $year = $this->argument('year');
         $institution = $this->argument('institution');
-        $institutionGrade = $this->instituion_grade->getInstitutionGradeToPromoted($year,$institution);
+        $institutionGrade = $this->instituion_grade->getInstitutionGradeToPromoted($year, $institution);
 //        dd($institutionGrade);
-        array_walk($institutionGrade,array($this,'processGrades'),$year);
+        array_walk($institutionGrade, array($this, 'processGrades'), $year);
     }
 
-    public function processGrades($institutionGrade,$count,$year){
-        if(!empty($institutionGrade) && $this->institutions->isActive($institutionGrade['institution_id'])) {
-            $this->instituion_grade->updatePromoted($year,$institutionGrade['id']);
+    public function processGrades($institutionGrade, $count, $year) {
+        if (!empty($institutionGrade) && $this->institutions->isActive($institutionGrade['institution_id'])) {
+            $this->instituion_grade->updatePromoted($year, $institutionGrade['id']);
             $isAvailableforPromotion = 0;
             $nextGrade = $this->education_grades->getNextGrade($institutionGrade['education_grade_id']);
 
@@ -79,9 +79,9 @@ 
             }
 
             if (!empty($isAvailableforPromotion)) {
-                $this->process($institutionGrade,$nextGrade,$year,1);
-            }else{
-                $this->process($institutionGrade,$nextGrade,$year,3);
+                $this->process($institutionGrade, $nextGrade, $year, 1);
+            }else {
+                $this->process($institutionGrade, $nextGrade, $year, 3);
             }
         }
     }
@@ -97,9 +97,9 @@ 
      * @param array $parallelClasses
      * @param $status
      */
-        public function promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,$parallelClasses = [],$status){
-            $institution = Institution::where( 'id',$institutionGrade['institution_id'])->get()->first();
-            $studentListToPromote = $this->institution_students->getStudentListToPromote($institutionGrade,$academicPeriod);
+        public function promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, $parallelClasses = [], $status) {
+            $institution = Institution::where('id', $institutionGrade['institution_id'])->get()->first();
+            $studentListToPromote = $this->institution_students->getStudentListToPromote($institutionGrade, $academicPeriod);
 
             $params = [
                 $nextAcademicPeriod,
@@ -108,15 +108,15 @@ 
                 $status
             ];
 
-            try{
-                array_walk($studentListToPromote,array($this,'promote'),$params);
+            try {
+                array_walk($studentListToPromote, array($this, 'promote'), $params);
 
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                 $output->writeln('##########################################################################################################################');
-                $output->writeln('Promoting from '. $institutionGrade['name'] .' IN '.$institution->name.' No of Students: '. count($studentListToPromote));
+                $output->writeln('Promoting from '.$institutionGrade['name'].' IN '.$institution->name.' No of Students: '.count($studentListToPromote));
 
 
-                if(!empty($parallelClasses)){
+                if (!empty($parallelClasses)) {
                     $params = [
                         $nextAcademicPeriod,
                         $institutionGrade,
@@ -124,10 +124,10 @@ 
                         $parallelClasses,
                         $status
                     ];
-                    array_walk($studentListToPromote,array($this,'assingeToClasses'),$params);
-                    array_walk($parallelClasses,array($this,'updateStudentCount'));
+                    array_walk($studentListToPromote, array($this, 'assingeToClasses'), $params);
+                    array_walk($parallelClasses, array($this, 'updateStudentCount'));
                 }
-            }catch (\Exception $e){
+            }catch (\Exception $e) {
                 dd($e);
                 Log::error($e->getMessage());
             }
@@ -135,10 +135,10 @@ 
 
 
 
-        public function updateStudentCount($class){
+        public function updateStudentCount($class) {
             $studentCounts = Institution_class_student::getStudentsCount($class['id']);
             unset($studentCounts['total']);
-            Institution_class::query()->where('id',$class['id'])->update($studentCounts);
+            Institution_class::query()->where('id', $class['id'])->update($studentCounts);
         }
 
 
@@ -150,38 +150,38 @@ 
      * @param $year
      * @return int
      */
-        public function process($institutionGrade,$nextGrade,$year){
-            $academicPeriod = Academic_period::query()->where('code',$year -1)->get()->first();
-            $nextAcademicPeriod = Academic_period::query()->where('code',$year)->get()->first();
+        public function process($institutionGrade, $nextGrade, $year) {
+            $academicPeriod = Academic_period::query()->where('code', $year - 1)->get()->first();
+            $nextAcademicPeriod = Academic_period::query()->where('code', $year)->get()->first();
 
             $nextGradeObj = null;
-                    if($nextGrade !== []  && !is_null($nextGrade) ){
-                        $currentGradeObj = $this->instituion_grade->getParallelClasses($institutionGrade['id'],$institutionGrade['institution_id'],$institutionGrade['education_grade_id'],$academicPeriod->id);
-                        $nextGradeObj = $this->instituion_grade->getParallelClasses($institutionGrade['id'],$institutionGrade['institution_id'],$nextGrade->id,$nextAcademicPeriod->id);
+                    if ($nextGrade !== [] && !is_null($nextGrade)) {
+                        $currentGradeObj = $this->instituion_grade->getParallelClasses($institutionGrade['id'], $institutionGrade['institution_id'], $institutionGrade['education_grade_id'], $academicPeriod->id);
+                        $nextGradeObj = $this->instituion_grade->getParallelClasses($institutionGrade['id'], $institutionGrade['institution_id'], $nextGrade->id, $nextAcademicPeriod->id);
 
                     }
 
-            if(!is_null($nextGradeObj)){
-                if($nextGradeObj->count() == 1){
+            if (!is_null($nextGradeObj)) {
+                if ($nextGradeObj->count() == 1) {
                     // promote parallel classes
-                    $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,$nextGradeObj->toArray(),1);
+                    $this->promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, $nextGradeObj->toArray(), 1);
                     return 1;
-                }elseif (($nextGradeObj->count() > 1) && ($nextGradeObj->count() !==  $currentGradeObj->count())){
+                }elseif (($nextGradeObj->count() > 1) && ($nextGradeObj->count() !== $currentGradeObj->count())) {
                     // promote pool promotion
-                    $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],1);
+                    $this->promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, [], 1);
                     return 2;
-                }elseif(($nextGradeObj->count() > 1) && $currentGradeObj->count() == $nextGradeObj->count()){
+                }elseif (($nextGradeObj->count() > 1) && $currentGradeObj->count() == $nextGradeObj->count()) {
                     // Promote matching class name with previous class
-                    $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,$nextGradeObj->toArray(),1);
+                    $this->promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, $nextGradeObj->toArray(), 1);
                     return 1;
-                }else{
+                }else {
                     // default pool promotion
-                    $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],1);
+                    $this->promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, [], 1);
                     return 2;
                 }
-            }else{
+            }else {
                 // default pool promotion
-                $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],3);
+                $this->promotion($institutionGrade, $nextGrade, $academicPeriod, $nextAcademicPeriod, [], 3);
                 return 2;
             }
         }
@@ -194,7 +194,7 @@ 
      * @param $count
      * @param $params
      */
-        public function promote($student,$count,$params){
+        public function promote($student, $count, $params) {
 
             $academicPeriod = $params[0];
             $nextGrade = $params[2];
@@ -204,19 +204,19 @@ 
                 'education_grade_id' => $nextGrade !== null ? $nextGrade->id : $student['education_grade_id'],
                 'academic_period_id' => $academicPeriod->id,
                 'start_date' => $academicPeriod->start_date,
-                'start_year' =>$academicPeriod->start_year ,
+                'start_year' =>$academicPeriod->start_year,
                 'end_date' => $academicPeriod->end_date,
-                'end_year' =>   $academicPeriod->end_year ,
+                'end_year' =>   $academicPeriod->end_year,
                 'institution_id' => $student['institution_id'],
                 'admission_id' => $student['admission_id'],
                 'created_user_id' => $student['created_user_id']
             ];
-            try{
-               Institution_student::where('id',(string)$student['id'])->update($studentData);
+            try {
+               Institution_student::where('id', (string) $student['id'])->update($studentData);
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
-                $output->writeln('----------------- '. $student['admission_id'] . ' to ' . $studentData['education_grade_id']);
+                $output->writeln('----------------- '.$student['admission_id'].' to '.$studentData['education_grade_id']);
 
-            }catch (\Exception $e){
+            }catch (\Exception $e) {
                 dd($e);
                 Log::error($e->getMessage());
             }
@@ -232,11 +232,11 @@ 
      * @param $classes
      * @return false|int|string|null
      */
-    public function getStudentClass($student,$educationGrade,$nextGrade,$classes){
+    public function getStudentClass($student, $educationGrade, $nextGrade, $classes) {
         $studentClass = $this->institution_class_students->getStudentNewClass($student);
-        if(!is_null($studentClass)){
-            return  array_search(str_replace($educationGrade['name'],$nextGrade->name,$studentClass->name),array_column($classes,'name'));
-        }else{
+        if (!is_null($studentClass)) {
+            return  array_search(str_replace($educationGrade['name'], $nextGrade->name, $studentClass->name), array_column($classes, 'name'));
+        }else {
             return false;
         }
 
@@ -249,7 +249,7 @@ 
      * @param $count
      * @param $params
      */
-    public function assingeToClasses($student,$count,$params){
+    public function assingeToClasses($student, $count, $params) {
         $academicPeriod = $params[0];
         $educationGrade = $params[1];
         $nextGrade = $params[2];
@@ -257,15 +257,15 @@ 
         $status = $params[4];
 
 
-        $class = $this->getStudentClass($student,$educationGrade,$nextGrade,$classes);
-        if(is_numeric($class)){
+        $class = $this->getStudentClass($student, $educationGrade, $nextGrade, $classes);
+        if (is_numeric($class)) {
             $class = $classes[$class];
 
-            if(count($classes) == 1){
+            if (count($classes) == 1) {
                 $class = $classes[0];
             }
 
-            if(!is_null($class)){
+            if (!is_null($class)) {
 
                 $studentObj = [
                     'student_id' => $student['student_id'],
@@ -276,14 +276,14 @@ 
                     'student_status_id' => $status,
                     'created_user_id' => $student['created_user_id']
                 ];
-                if(!$this->institution_class_students->isDuplicated($studentObj)){
+                if (!$this->institution_class_students->isDuplicated($studentObj)) {
                     $this->institution_class_students->create($studentObj);
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
-                    $output->writeln('----------------- '. $student['student_id']. 'to ' . $class['name']);
-                }else{
-                    $this->institution_class_students->where('id',(string)$student['id'])->update($studentObj);
+                    $output->writeln('----------------- '.$student['student_id'].'to '.$class['name']);
+                }else {
+                    $this->institution_class_students->where('id', (string) $student['id'])->update($studentObj);
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
-                    $output->writeln('----------------- '. $student['student_id']. 'to ' . $class['name']);
+                    $output->writeln('----------------- '.$student['student_id'].'to '.$class['name']);
                 }
             }
         }

Braces +9 added lines, -9 removed lines

@@ -80,7 +80,7 @@ 
 
             if (!empty($isAvailableforPromotion)) {
                 $this->process($institutionGrade,$nextGrade,$year,1);
-            }else{
+            } else{
                 $this->process($institutionGrade,$nextGrade,$year,3);
             }
         }
@@ -127,7 +127,7 @@ 
                     array_walk($studentListToPromote,array($this,'assingeToClasses'),$params);
                     array_walk($parallelClasses,array($this,'updateStudentCount'));
                 }
-            }catch (\Exception $e){
+            } catch (\Exception $e){
                 dd($e);
                 Log::error($e->getMessage());
             }
@@ -166,20 +166,20 @@ 
                     // promote parallel classes
                     $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,$nextGradeObj->toArray(),1);
                     return 1;
-                }elseif (($nextGradeObj->count() > 1) && ($nextGradeObj->count() !==  $currentGradeObj->count())){
+                } elseif (($nextGradeObj->count() > 1) && ($nextGradeObj->count() !==  $currentGradeObj->count())){
                     // promote pool promotion
                     $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],1);
                     return 2;
-                }elseif(($nextGradeObj->count() > 1) && $currentGradeObj->count() == $nextGradeObj->count()){
+                } elseif(($nextGradeObj->count() > 1) && $currentGradeObj->count() == $nextGradeObj->count()){
                     // Promote matching class name with previous class
                     $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,$nextGradeObj->toArray(),1);
                     return 1;
-                }else{
+                } else{
                     // default pool promotion
                     $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],1);
                     return 2;
                 }
-            }else{
+            } else{
                 // default pool promotion
                 $this->promotion($institutionGrade,$nextGrade,$academicPeriod,$nextAcademicPeriod,[],3);
                 return 2;
@@ -216,7 +216,7 @@ 
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                 $output->writeln('----------------- '. $student['admission_id'] . ' to ' . $studentData['education_grade_id']);
 
-            }catch (\Exception $e){
+            } catch (\Exception $e){
                 dd($e);
                 Log::error($e->getMessage());
             }
@@ -236,7 +236,7 @@ 
         $studentClass = $this->institution_class_students->getStudentNewClass($student);
         if(!is_null($studentClass)){
             return  array_search(str_replace($educationGrade['name'],$nextGrade->name,$studentClass->name),array_column($classes,'name'));
-        }else{
+        } else{
             return false;
         }
 
@@ -280,7 +280,7 @@ 
                     $this->institution_class_students->create($studentObj);
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                     $output->writeln('----------------- '. $student['student_id']. 'to ' . $class['name']);
-                }else{
+                } else{
                     $this->institution_class_students->where('id',(string)$student['id'])->update($studentObj);
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                     $output->writeln('----------------- '. $student['student_id']. 'to ' . $class['name']);

app/Console/Commands/StudentsIdGen.php

Spacing +7 added lines, -7 removed lines

@@ -46,16 +46,16 @@ 
         $this->start_time = microtime(TRUE);
         ini_set('memory_limit', '2048M');
         $students = $this->students->query()
-            ->where('is_student',1)
+            ->where('is_student', 1)
             ->limit(100000)
             ->offset($this->argument('offset'))
             ->get()->toArray();
         $this->output->writeln('no of students'.count($students));
         $this->output->writeln('Update started');
-        array_walk($students,array($this,'updateNewUUID'));
+        array_walk($students, array($this, 'updateNewUUID'));
         $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('$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$');
     }
 
@@ -64,12 +64,12 @@ 
      * @param $student
      * @throws \Exception
      */
-    public function updateNewUUID($student){
-        if(!MoeUuid::isValidMoeUuid(3)){
+    public function updateNewUUID($student) {
+        if (!MoeUuid::isValidMoeUuid(3)) {
             $newId = MoeUuid::getUniqueAlphanumeric(3);
             $this->output->writeln('Updating student:'.$student['id']);
-            Security_user::query()->where('id',$student['id'])
-                ->update(['openemis_no' => $newId , 'username' => str_replace('-','',$newId)]);
+            Security_user::query()->where('id', $student['id'])
+                ->update(['openemis_no' => $newId, 'username' => str_replace('-', '', $newId)]);
         }
     }
 }

app/Console/Commands/ProcessTerminatedFiles.php

Spacing +11 added lines, -11 removed lines

@@ -45,19 +45,19 @@ 
     {
             $files = $this->getFiles();
             try {
-                if(!empty($files)){
-                    array_walk($files,array($this,'process'));
+                if (!empty($files)) {
+                    array_walk($files, array($this, 'process'));
                     unset($files);
                     exit();
 
-                }else{
+                }else {
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                     $output->writeln('No files found,Waiting for files');
                     exit();
 
                 }
 
-            }catch (Exception $e){
+            }catch (Exception $e) {
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                 $output->writeln($e);
                 sleep(300);
@@ -67,25 +67,25 @@ 
 
     }
 
-    protected function getFiles(){
+    protected function getFiles() {
         $files = Upload::where('is_processed', '=', 3)
-            ->where('is_email_sent','=',0)
+            ->where('is_email_sent', '=', 0)
             ->where('updated_at', '<=', Carbon::now()->tz('Asia/Colombo')->subHours(3))
             ->limit(50)
             ->get()->toArray();
         return $files;
     }
 
-    protected function  process($file){
+    protected function  process($file) {
         $time = Carbon::now()->tz('Asia/Colombo');
         $this->processSheet($file);
         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
         $now = Carbon::now()->tz('Asia/Colombo');
-        $output->writeln('=============== Time taken to batch ' .$now->diffInMinutes($time));
+        $output->writeln('=============== Time taken to batch '.$now->diffInMinutes($time));
 
     }
 
-    protected function processSheet($file){
+    protected function processSheet($file) {
         $this->startTime = Carbon::now()->tz('Asia/Colombo');
         $user = User::find($file['security_user_id']);
         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
@@ -95,8 +95,8 @@ 
             Mail::to($user->email)->send(new TerminatedReport($file));
             DB::table('uploads')
                 ->where('id', $file['id'])
-                ->update(['is_processed' => 3, 'is_email_sent' => 1,'updated_at' => now()]);
-        } else {
+                ->update(['is_processed' => 3, 'is_email_sent' => 1, 'updated_at' => now()]);
+        }else {
             exit();
         }
     }

Braces +2 added lines, -2 removed lines

@@ -50,14 +50,14 @@
                     unset($files);
                     exit();
 
-                }else{
+                } else{
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                     $output->writeln('No files found,Waiting for files');
                     exit();
 
                 }
 
-            }catch (Exception $e){
+            } catch (Exception $e){
                 $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                 $output->writeln($e);
                 sleep(300);