moe-lk / bulk-upload

app/Models/Institution_student.php

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]);
         }

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' => $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]);
         }
     }
 
-    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,

app/Providers/ValidatorExtended.php

Spacing +35 added lines, -35 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]{11,12})/i', $value);
                     break;
@@ -190,11 +190,11 @@ 
         }
         
         if ($valid == 0) {
-            dd($data[$perameters[0]],$valid,$value);
-            $this->_custom_messages['identity'] = $attribute . ' is not valid  Please check the NIC number';
+            dd($data[$perameters[0]], $valid, $value);
+            $this->_custom_messages['identity'] = $attribute.' is not valid  Please check the NIC number';
             $this->_set_custom_stuff();
             return false;
-        } else {
+        }else {
             return true;
         }
     }
@@ -202,7 +202,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);
@@ -218,14 +218,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;
             }
         }
@@ -235,24 +235,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/ImportStudents.php

Spacing +55 added lines, -55 removed lines

@@ -47,7 +47,7 @@ 
      */
     public function __construct()
     {
-        $this->output =  new \Symfony\Component\Console\Output\ConsoleOutput();
+        $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();
         parent::__construct();
     }
 
@@ -69,18 +69,18 @@ 
                         $this->process($files);
                         unset($files);
                         exit();
-                    } else {
+                    }else {
                         $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                         $this->output->writeln('No files found,Waiting for files');
                         exit();
                     }
-                } catch (Exception $e) {
+                }catch (Exception $e) {
                     $output = new \Symfony\Component\Console\Output\ConsoleOutput();
                     $this->output->writeln($e);
                     sleep(300);
                     $this->handle();
                 }
-            } else {
+            }else {
                 exit();
             }
         }
@@ -94,7 +94,7 @@ 
         $files[0]['node'] = $node;
         $this->processSheet($files[0]);
         $now = Carbon::now()->tz('Asia/Colombo');
-        $this->output->writeln('=============== Time taken to batch ' . $now->diffInMinutes($time));
+        $this->output->writeln('=============== Time taken to batch '.$now->diffInMinutes($time));
     }
 
     protected function getTerminatedFiles()
@@ -137,10 +137,10 @@ 
             ->join('contact_types', 'user_contacts.contact_type_id', '=', 'contact_types.id')
        
             ;
-            if(env('APP_ENV') == 'stage'){
+            if (env('APP_ENV') == 'stage') {
                 $query->where('contact_types.contact_option_id', '=', 5)
                 ->where('contact_types.name', '=', 'TestEmail');
-            }else{
+            }else {
                 $query->where('contact_types.contact_option_id', '!=', 5);
             }
             
@@ -170,13 +170,13 @@ 
     public function processSuccessEmail($file, $user, $subject)
     {
         $file['subject'] = $subject;
-        $this->output->writeln('Processing the file: ' . $file['filename']);
+        $this->output->writeln('Processing the file: '.$file['filename']);
         try {
             Mail::to($user->email)->send(new StudentImportSuccess($file));
             DB::table('uploads')
                 ->where('id', $file['id'])
                 ->update(['is_processed' => 1, 'is_email_sent' => 1, 'updated_at' => now()]);
-        } catch (\Exception $ex) {
+        }catch (\Exception $ex) {
             $this->output->writeln($ex->getMessage());
             DB::table('uploads')
                 ->where('id', $file['id'])
@@ -192,7 +192,7 @@ 
             DB::table('uploads')
                 ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'is_email_sent' => 1, 'updated_at' => now()]);
-        } catch (\Exception $ex) {
+        }catch (\Exception $ex) {
             $this->output->writeln($ex->getMessage());
             DB::table('uploads')
                 ->where('id', $file['id'])
@@ -208,7 +208,7 @@ 
             DB::table('uploads')
                 ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'is_email_sent' => 1, 'updated_at' => now()]);
-        } catch (\Exception $ex) {
+        }catch (\Exception $ex) {
             $this->output->writeln($ex->getMessage());
             DB::table('uploads')
                 ->where('id', $file['id'])
@@ -221,9 +221,9 @@ 
         $node = $this->argument('node');
         if ($node == $file['node']) {
             $output = new \Symfony\Component\Console\Output\ConsoleOutput();
-            $this->output->writeln('Processing from:' . $node);
+            $this->output->writeln('Processing from:'.$node);
             return true;
-        } else {
+        }else {
             exit;
             return false;
         }
@@ -235,20 +235,20 @@ 
         $user = User::find($file['security_user_id']);
         $this->checkNode($file);
         $this->output->writeln('##########################################################################################################################');
-        $this->output->writeln('Processing the file: ' . $file['filename']);
+        $this->output->writeln('Processing the file: '.$file['filename']);
         if ($this->checkTime()) {
             try {
                 $this->import($file, 1, 'C');
                 sleep(10);
                 $this->import($file, 2, 'B');
-            } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+            }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
                 $this->output->writeln($e->getMessage());
                 try {
                     Mail::to($user->email)->send(new IncorrectTemplate($file));
                     DB::table('uploads')
                         ->where('id', $file['id'])
                         ->update(['is_processed' => 2, 'is_email_sent' => 1, 'updated_at' => now()]);
-                } catch (\Exception $ex) {
+                }catch (\Exception $ex) {
                     $this->output->writeln($e->getMessage());
                     $this->handle();
                     DB::table('uploads')
@@ -256,15 +256,15 @@ 
                         ->update(['is_processed' => 2, 'is_email_sent' => 2, 'updated_at' => now()]);
                 }
             }
-        } else {
+        }else {
             exit();
         }
     }
 
     protected function getType($file)
     {
-        $file =  storage_path() . '/app/sis-bulk-data-files/' . $file;
-        $inputFileType =  \PhpOffice\PhpSpreadsheet\IOFactory::identify($file);
+        $file = storage_path().'/app/sis-bulk-data-files/'.$file;
+        $inputFileType = \PhpOffice\PhpSpreadsheet\IOFactory::identify($file);
         return $inputFileType;
     }
 
@@ -322,7 +322,7 @@ 
             ini_set('memory_limit', '2048M');
             $this->getFileSize($file);
             $user = User::find($file['security_user_id']);
-            $excelFile = '/sis-bulk-data-files/' . $file['filename'];
+            $excelFile = '/sis-bulk-data-files/'.$file['filename'];
             $this->highestRow = $this->getHighestRow($file, $sheet, $column);
             switch ($sheet) {
                 case 1;
@@ -341,14 +341,14 @@ 
                                 ->update(['insert' => 3, 'updated_at' => now()]);
                             $this->processFailedEmail($file, $user, 'Fresh Student Data Upload:Partial Success ');
                             $this->stdOut('Insert Students', $this->highestRow);
-                        } else {
+                        }else {
                             DB::table('uploads')
                             ->where('id', $file['id'])
                             ->update(['insert' => 1, 'updated_at' => now()]);
                             $this->processSuccessEmail($file, $user, 'Fresh Student Data Upload:Success ');
                             $this->stdOut('Insert Students', $this->highestRow);
                         }
-                    } else if (($this->getSheetName($file, 'Insert Students')) && $this->highestRow > 0) {
+                    }else if (($this->getSheetName($file, 'Insert Students')) && $this->highestRow > 0) {
                         DB::table('uploads')
                             ->where('id', $file['id'])
                             ->update(['is_processed' => 2]);
@@ -367,14 +367,14 @@ 
                                 ->update(['update' => 3, 'is_processed' => 1, 'updated_at' => now()]);
                             $this->processFailedEmail($file, $user, 'Existing Student Data Update:Partial Success ');
                             $this->stdOut('Update Students', $this->highestRow);
-                        } else {
+                        }else {
                             DB::table('uploads')
                             ->where('id', $file['id'])
                             ->update(['update' => 1, 'is_processed' => 1, 'updated_at' => now()]);
                             $this->processSuccessEmail($file, $user, 'Existing Student Data Update:Success ');
                             $this->stdOut('Update Students', $this->highestRow);
                         }
-                    } else if (($this->getSheetName($file, 'Update Students')) && $this->highestRow == 0) {
+                    }else if (($this->getSheetName($file, 'Update Students')) && $this->highestRow == 0) {
                         DB::table('uploads')
                             ->where('id', $file['id'])
                             ->update(['is_processed' => 2, 'updated_at' => now()]);
@@ -382,7 +382,7 @@ 
                     }
                     break;
             }
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
             $this->output->writeln($e->getMessage());
             if ($sheet == 1) {
                 self::writeErrors($e, $file, 'Insert Students');
@@ -390,7 +390,7 @@ 
                     ->where('id', $file['id'])
                     ->update(['insert' => 2, 'updated_at' => now()]);
                 $this->processFailedEmail($file, $user, 'Fresh Student Data Upload:Failed');
-            } else if ($sheet == 2) {
+            }else if ($sheet == 2) {
                 self::writeErrors($e, $file, 'Update Students');
                 DB::table('uploads')
                     ->where('id', $file['id'])
@@ -398,7 +398,7 @@ 
                 $this->processFailedEmail($file, $user, 'Existing Student Data Update:Failed');
             }
             DB::table('uploads')
-                ->where('id',  $file['id'])
+                ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'updated_at' => now()]);
         }
     }
@@ -416,14 +416,14 @@ 
 
     protected function  getFileSize($file)
     {
-        $excelFile =  '/sis-bulk-data-files/' . $file['filename'];
+        $excelFile = '/sis-bulk-data-files/'.$file['filename'];
         $size = Storage::disk('local')->size($excelFile);
         $user = User::find($file['security_user_id']);
         if ($size > 0) {
             return true;
-        } else {
+        }else {
             DB::table('uploads')
-                ->where('id',  $file['id'])
+                ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'updated_at' => now()]);
             $this->stdOut('No valid data found :Please re-upload the file', 0);
             $this->processEmptyEmail($file, $user, 'No valid data found :Please re-upload the file');
@@ -433,21 +433,21 @@ 
     protected function setReader($file)
     {
         try {
-            $excelFile =  '/sis-bulk-data-files/processed/' . $file['filename'];
+            $excelFile = '/sis-bulk-data-files/processed/'.$file['filename'];
             $exists = Storage::disk('local')->exists($excelFile);
             if (!$exists) {
 
-                $excelFile =  "/sis-bulk-data-files/" . $file['filename'];
+                $excelFile = "/sis-bulk-data-files/".$file['filename'];
             }
-            $excelFile = storage_path() . "/app" . $excelFile;
+            $excelFile = storage_path()."/app".$excelFile;
             $reader = \PhpOffice\PhpSpreadsheet\IOFactory::createReader($this->getType($file['filename']));
-            $objPHPExcel =  $reader->load($excelFile);
+            $objPHPExcel = $reader->load($excelFile);
             return $objPHPExcel;
-        } catch (Exception $e) {
+        }catch (Exception $e) {
             $this->output->writeln($e->getMessage());
             $user = User::find($file['security_user_id']);
             DB::table('uploads')
-                ->where('id',  $file['id'])
+                ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'updated_at' => now()]);
             $this->stdOut('No valid data found :Please re-upload the file', 0);
             $this->processEmptyEmail($file, $user, 'No valid data found :Please re-upload the file');
@@ -458,12 +458,12 @@ 
     {
         try {;
             $objPHPExcel = $this->setReader($file);
-            return $objPHPExcel->getSheetByName($sheet)  !== null;
-        } catch (Exception $e) {
+            return $objPHPExcel->getSheetByName($sheet) !== null;
+        }catch (Exception $e) {
             $this->output->writeln($e->getMessage());
             $user = User::find($file['security_user_id']);
             DB::table('uploads')
-                ->where('id',  $file['id'])
+                ->where('id', $file['id'])
                 ->update(['is_processed' => 2, 'updated_at' => now()]);
             $this->stdOut('No valid data found :Please re-upload the file', 0);
             $this->processEmptyEmail($file, $user, 'No valid data found :Please re-upload the file');
@@ -475,15 +475,15 @@ 
         try {
             $reader = $this->setReader($file);
             $highestRowCount = 0;
-            $sheet =  $reader->getSheet($sheet);
+            $sheet = $reader->getSheet($sheet);
             $highestRow = $sheet->getHighestDataRow($column); 
-            for ($row = 3; $row <= $highestRow; $row++){ 
-                $rowData = $sheet->rangeToArray('A' . $row . ':' . $column . $row,NULL,TRUE,FALSE);
-                if(isEmptyRow(reset($rowData))) { continue; } // skip empty row
+            for ($row = 3; $row <= $highestRow; $row++) { 
+                $rowData = $sheet->rangeToArray('A'.$row.':'.$column.$row, NULL, TRUE, FALSE);
+                if (isEmptyRow(reset($rowData))) { continue; } // skip empty row
                 $highestRowCount += 1;
             }
             return  $highestRowCount;
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             $this->output->writeln($e->getMessage());
             $user = User::find($file['security_user_id']);
             DB::beginTransaction();
@@ -498,10 +498,10 @@ 
 
     protected function stdOut($title, $rows)
     {
-        $this->output->writeln($title . ' Process completed at . ' . ' ' . now());
+        $this->output->writeln($title.' Process completed at . '.' '.now());
         $now = Carbon::now()->tz('Asia/Colombo');
-        $this->output->writeln('Total Processed lines: ' . $rows);
-        $this->output->writeln('Time taken to process           : ' .   $now->diffInSeconds($this->startTime) . ' Seconds');
+        $this->output->writeln('Total Processed lines: '.$rows);
+        $this->output->writeln('Time taken to process           : '.$now->diffInSeconds($this->startTime).' Seconds');
         $this->output->writeln('--------------------------------------------------------------------------------------------------------------------------');
     }
 
@@ -513,7 +513,7 @@ 
         $sheet = $reader->getActiveSheet();
         if (!in_array($row, $params['rows'])) {
             $output = new \Symfony\Component\Console\Output\ConsoleOutput();
-            $this->output->writeln(' removing row . ' . ' ' . $row);
+            $this->output->writeln(' removing row . '.' '.$row);
             $reader->getActiveSheet()->getCellCollection()->removeRow($row);
         }
     }
@@ -536,7 +536,7 @@ 
             if (count($failures) > 0) {
                 $rows = array_map('rows', $failures);
                 $rows = array_unique($rows);
-                $rowIndex =   range(3, $this->highestRow + 2);
+                $rowIndex = range(3, $this->highestRow + 2);
                 $params = [
                     'rows' => $rows,
                     'reader' => $reader
@@ -545,17 +545,17 @@ 
                 array_walk($rowIndex, array($this, 'removeRows'), $params);
                 $objWriter = $this->getSheetWriter($file, $reader);
                 Storage::disk('local')->makeDirectory('sis-bulk-data-files/processed');
-                $objWriter->save(storage_path() . '/app/sis-bulk-data-files/processed/' . $file['filename']);
+                $objWriter->save(storage_path().'/app/sis-bulk-data-files/processed/'.$file['filename']);
                 $now = Carbon::now()->tz('Asia/Colombo');
-                $this->output->writeln($reader->getActiveSheet()->getTitle() . ' Process completed at . ' . ' ' . now());
-                $this->output->writeln('memory usage for the processes : ' . (memory_get_usage() - $baseMemory));
-                $this->output->writeln('Time taken to process           : ' .   $now->diffInSeconds($this->startTime) . ' Seconds');
-                $this->output->writeln(' errors reported               : ' . count($failures));
+                $this->output->writeln($reader->getActiveSheet()->getTitle().' Process completed at . '.' '.now());
+                $this->output->writeln('memory usage for the processes : '.(memory_get_usage() - $baseMemory));
+                $this->output->writeln('Time taken to process           : '.$now->diffInSeconds($this->startTime).' Seconds');
+                $this->output->writeln(' errors reported               : '.count($failures));
                 $this->output->writeln('--------------------------------------------------------------------------------------------------------------------------');
                 unset($objWriter);
                 unset($failures);
             }
-        } catch (Eception $e) {
+        }catch (Eception $e) {
             $this->output->writeln($e->getMessage());
             $user = User::find($file['security_user_id']);
             DB::beginTransaction();

app/Imports/StudentUpdate.php

Spacing +12 added lines, -12 removed lines

@@ -55,7 +55,7 @@ 
 use Maatwebsite\Excel\Concerns\WithMultipleSheets;
 use Maatwebsite\Excel\Concerns\RegistersEventListeners;
 
-class StudentUpdate extends Import implements  ToModel, WithStartRow, WithHeadingRow, WithMultipleSheets, WithEvents, WithMapping, WithLimit, WithBatchInserts, WithValidation , SkipsOnFailure , SkipsOnError{
+class StudentUpdate extends Import implements  ToModel, WithStartRow, WithHeadingRow, WithMultipleSheets, WithEvents, WithMapping, WithLimit, WithBatchInserts, WithValidation, SkipsOnFailure, SkipsOnError{
 
     use Importable,
         RegistersEventListeners,
@@ -89,30 +89,30 @@ 
             $institution = $institutionClass->institution_id;
             if (!empty($institutionClass)) {
                 $this->setGender($row);
-                $studentInfo = Security_user::createOrUpdateStudentProfile($row,'update',$this->file);
+                $studentInfo = Security_user::createOrUpdateStudentProfile($row, 'update', $this->file);
                 $student = Institution_class_student::where('student_id', '=', $studentInfo->id)->first();
-                if(!empty($row['admission_no'])){
-                    Institution_student::where('student_id','=',$studentInfo->id)
-                    ->where('institution_id','=', $institution)
+                if (!empty($row['admission_no'])) {
+                    Institution_student::where('student_id', '=', $studentInfo->id)
+                    ->where('institution_id', '=', $institution)
                     ->update(['admission_id'=> $row['admission_no']]);
                 }
-                User_special_need::createOrUpdate($studentInfo->id,$row,$this->file);
-                User_body_mass::createOrUpdate($studentInfo->id,$row,$this->file);
+                User_special_need::createOrUpdate($studentInfo->id, $row, $this->file);
+                User_body_mass::createOrUpdate($studentInfo->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']);
                 Institution_class::where('id', '=', $institutionClass->id)
                         ->update([
                             'total_male_students' => $totalStudents['total_male_students'],
                             'total_female_students' => $totalStudents['total_female_students']]);
             }
-        } 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);

app/helpers.php

Spacing +50 added lines, -50 removed lines

@@ -1,22 +1,22 @@ 
 <?php
 
-function get_l_name($name){
+function get_l_name($name) {
     $name = trim($name);
-    $last_name = (strpos($name,' ') === false) ? '' : preg_replace('#.*\s([\w-]*)$#', '$1', $name);
+    $last_name = (strpos($name, ' ') === false) ? '' : preg_replace('#.*\s([\w-]*)$#', '$1', $name);
     return $last_name;
 }
 
 // Gen name with initials with help of fullname
-function genNameWithInitials($fullname = null){
+function genNameWithInitials($fullname = null) {
     $names = explode(' ', $fullname);
-    $length  = count($names);
+    $length = count($names);
     $Initials = '';
-    if($length > 1){
-        for ($i = 0; ($length-1) > $i; $i++) {
-            $Initials = $Initials . '' . mb_substr($names[$i], 0, 1, "UTF-8");
+    if ($length > 1) {
+        for ($i = 0; ($length - 1) > $i; $i++) {
+            $Initials = $Initials.''.mb_substr($names[$i], 0, 1, "UTF-8");
         }
-        $nameWithInitials = $Initials . ' ' . $names[$length - 1];
-    }else{
+        $nameWithInitials = $Initials.' '.$names[$length - 1];
+    }else {
         $nameWithInitials = $fullname;
     }
     return $nameWithInitials;
@@ -35,10 +35,10 @@ 
 }
 
 
-function getMatchingKeys($array){
+function getMatchingKeys($array) {
     $keys = [];
-    foreach ($array as $key => $value){
-        if(strstr($key , 'option'))
+    foreach ($array as $key => $value) {
+        if (strstr($key, 'option'))
             $keys[] = $key;
     }
     return $keys;
@@ -48,12 +48,12 @@ 
     return (bool) preg_match('/^[0-9a-f]{40}$/i', $str);
 }
 
-function isEmpty($value){
+function isEmpty($value) {
     return $value['institution_optional_subject'] !== null;
 }
 
 function isEmptyRow($row) {
-    foreach($row as $cell){
+    foreach ($row as $cell) {
         if (null !== $cell) return false;
     }
     return true;
@@ -64,7 +64,7 @@ 
     $i = 0;
     $key_array = array();
 
-    foreach($array as $val) {
+    foreach ($array as $val) {
         if (!in_array($val[$key], $key_array)) {
             $key_array[$i] = $val[$key];
             $temp_array[$i] = $val;
@@ -76,10 +76,10 @@ 
 
 
 
-function merge_two_arrays($array1,$array2) {
+function merge_two_arrays($array1, $array2) {
 
     $data = array();
-    $arrayAB = array_merge($array1,$array2);
+    $arrayAB = array_merge($array1, $array2);
 
     foreach ($arrayAB as $value) {
         dd($arrayAB);
@@ -87,25 +87,25 @@ 
         if (!isset($data[$id])) {
             $data[$id] = array();
         }
-        $data[$id] = array_merge($data[$id],$value);
+        $data[$id] = array_merge($data[$id], $value);
     }
     return $data;
 }
 
-function array_value_recursive($key, array $arr){
+function array_value_recursive($key, array $arr) {
     $val = array();
     array_walk_recursive($arr, function($v, $k) use($key, &$val){
-        if($k == $key) array_push($val, $v);
+        if ($k == $key) array_push($val, $v);
     });
     return count($val) > 1 ? $val : array_pop($val);
 }
 
 
-function merge_error_by_row($errors,$key){
+function merge_error_by_row($errors, $key) {
     $temp_array = array();
     $i = 0;
 
-    foreach($errors as $keys => $val) {
+    foreach ($errors as $keys => $val) {
         if (!in_array($val[$key], $temp_array)) {
             $temp_array[$keys]['errors'][] = $val;
         }
@@ -121,17 +121,17 @@ 
  * bind error messages to the excel file
  */
 
-function append_errors_to_excel($error, $count, $reader){
+function append_errors_to_excel($error, $count, $reader) {
     $active_sheet = $reader->getActiveSheet();
     $prev_value = $active_sheet->getCell('A'.$error['row'])->getValue();
-    $active_sheet->setCellValue('A'. ($error['row']) ,  $prev_value.','.implode(',',$error['errors']));
-    $active_sheet->getStyle('A'. ($error['row']))->getAlignment()->setWrapText(true);
+    $active_sheet->setCellValue('A'.($error['row']), $prev_value.','.implode(',', $error['errors']));
+    $active_sheet->getStyle('A'.($error['row']))->getAlignment()->setWrapText(true);
     $columns = Illuminate\Support\Facades\Config::get('excel.columns');
 
-    $column = array_keys($columns,$error['attribute']);
-    if(!empty($column)){
-        $column = $column[0]+1;
-        $selectedCells = $active_sheet->setSelectedCellByColumnAndRow($column,$error['row']);
+    $column = array_keys($columns, $error['attribute']);
+    if (!empty($column)) {
+        $column = $column[0] + 1;
+        $selectedCells = $active_sheet->setSelectedCellByColumnAndRow($column, $error['row']);
         $active_cell = ($selectedCells->getActiveCell());
 
         $active_sheet->getStyle($active_cell)
@@ -143,23 +143,23 @@ 
         }
 }
 
-function rows($error){
+function rows($error) {
     return $error['row'];
 }
 
-function rowIndex($row){
+function rowIndex($row) {
     return $row->getRowIndex();
 }
 
-function removeRows($row,$param){
-    if(in_array($row,$param['rows'])){
+function removeRows($row, $param) {
+    if (in_array($row, $param['rows'])) {
         $param['reader']->getActiveSheet()->removeRow($row);
     }
 }
 
-function colorizeCell($column,$error,$active_sheet){
-    $column = array_keys($column,$error['attribute']);
-    $selectedCells = $active_sheet->setSelectedCellByColumnAndRow($column,$error['row']);
+function colorizeCell($column, $error, $active_sheet) {
+    $column = array_keys($column, $error['attribute']);
+    $selectedCells = $active_sheet->setSelectedCellByColumnAndRow($column, $error['row']);
     $active_cell = ($selectedCells->getActiveCell());
 
     $active_sheet->getStyle($active_cell)
@@ -171,45 +171,45 @@ 
 }
 
 
-function errors_unique_array($item,$key){
+function errors_unique_array($item, $key) {
 
-        $search = array_filter($item,function ($data) use ($item){
-            return isset($data['row']) &&  ($data['row']  == $item->row());
+        $search = array_filter($item, function($data) use ($item){
+            return isset($data['row']) && ($data['row'] == $item->row());
         });
 
-        if($search){
-            array_push($search[0]['errors'],implode(',',$item->errors()));
+        if ($search) {
+            array_push($search[0]['errors'], implode(',', $item->errors()));
             $errors = $search;
         }
 
         return $errors;
 }
 
-function sig_handler($signo){
+function sig_handler($signo) {
     global $child;
-    switch($signo){
+    switch ($signo) {
         case 'SIFCLD':
     }
 
 }
-function processParallel($func ,array $arr, $procs = 4,$params =[])
+function processParallel($func, array $arr, $procs = 4, $params = [])
     {
         // Break array up into $procs chunks.
-        $chunks   = array_chunk($arr, ceil((count($arr) / $procs)));
+        $chunks   = array_chunk($arr, ceil((count($arr)/$procs)));
         $pid      = -1;
         $children = array();
         foreach ($chunks as $items) {
             $pid = pcntl_fork();
             if ($pid === -1) {
                 die('could not fork');
-            } else if ($pid === 0) {
+            }else if ($pid === 0) {
                 // We are the child process. Pass a chunk of items to process.
-                echo('['.getmypid().']This Process executed at'.date("F d, Y h:i:s A")."\n") ;
-                array_walk($items, $func,$params);
+                echo('['.getmypid().']This Process executed at'.date("F d, Y h:i:s A")."\n");
+                array_walk($items, $func, $params);
                 exit(0);
-            } else {
+            }else {
                 // We are the parent.
-                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");
                 $children[] = $pid;
             }
         }

Braces +10 added lines, -5 removed lines

@@ -16,7 +16,7 @@ 
             $Initials = $Initials . '' . mb_substr($names[$i], 0, 1, "UTF-8");
         }
         $nameWithInitials = $Initials . ' ' . $names[$length - 1];
-    }else{
+    } else{
         $nameWithInitials = $fullname;
     }
     return $nameWithInitials;
@@ -38,8 +38,9 @@ 
 function getMatchingKeys($array){
     $keys = [];
     foreach ($array as $key => $value){
-        if(strstr($key , 'option'))
-            $keys[] = $key;
+        if(strstr($key , 'option')) {
+                    $keys[] = $key;
+        }
     }
     return $keys;
 }
@@ -54,7 +55,9 @@ 
 
 function isEmptyRow($row) {
     foreach($row as $cell){
-        if (null !== $cell) return false;
+        if (null !== $cell) {
+            return false;
+        }
     }
     return true;
 }
@@ -95,7 +98,9 @@ 
 function array_value_recursive($key, array $arr){
     $val = array();
     array_walk_recursive($arr, function($v, $k) use($key, &$val){
-        if($k == $key) array_push($val, $v);
+        if($k == $key) {
+            array_push($val, $v);
+        }
     });
     return count($val) > 1 ? $val : array_pop($val);
 }