moe-lk / bulk-upload

app/Models/Institution_subject_student.php

Spacing +11 added lines, -11 removed lines

@@ -7,7 +7,7 @@ 
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
 
-class Institution_subject_student extends Model  {
+class Institution_subject_student extends Model {
 
 
     use SoftDeletes;
@@ -56,7 +56,7 @@ 
     public static function boot()
     {
         parent::boot();
-        self::creating(function ($model) {
+        self::creating(function($model) {
             $model->id = (string) Uuid::generate(4);
         });
 
@@ -68,31 +68,31 @@ 
      *
      *
      */
-    public static function  isDuplicated($inputs){
+    public static function  isDuplicated($inputs) {
 
-       $exists = self::where('student_id','=',$inputs['student_id'])
-           ->where('institution_subject_id','=',$inputs['institution_subject_id'])
-           ->where('education_subject_id','=',$inputs['education_subject_id'])->count();
+       $exists = self::where('student_id', '=', $inputs['student_id'])
+           ->where('institution_subject_id', '=', $inputs['institution_subject_id'])
+           ->where('education_subject_id', '=', $inputs['education_subject_id'])->count();
 
 
         return $exists ? true :false;
     }
 
     
-     public function student(){
-        return $this->belongsTo('App\Models\Security_user','student_id');
+     public function student() {
+        return $this->belongsTo('App\Models\Security_user', 'student_id');
     }
     
-    public static function getStudentsCount($institution_subject_id){
+    public static function getStudentsCount($institution_subject_id) {
          $total_male_students = self::with(['student' => function($query) {
                         $query->where('student.gender_id', '=', 1);
-                    }])->whereHas('student', function ($query) {
+                    }])->whereHas('student', function($query) {
                     $query->where('gender_id', '=', 1);
                 })->where('institution_subject_id', '=', $institution_subject_id)->count();
 
         $total_female_students = self::with(['student' => function($query) {
                         $query->where('student.gender_id', '=', 2);
-                    }])->whereHas('student', function ($query) {
+                    }])->whereHas('student', function($query) {
                     $query->where('gender_id', '=', 2);
                 })->where('institution_subject_id', '=', $institution_subject_id)->count();
 

app/Imports/StudentUpdate.php

Spacing +39 added lines, -39 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,
@@ -108,9 +108,9 @@ 
                         break;
                 }
 
-                $BirthArea = Area_administrative::where('name', 'like', '%' . $row['birth_registrar_office_as_in_birth_certificate'] . '%')->first();
-                $nationalityId = Nationality::where('name', 'like', '%' . $row['nationality'] . '%')->first();
-                $identityType = Identity_type::where('national_code', 'like', '%' . $row['identity_type'] . '%')->first();
+                $BirthArea = Area_administrative::where('name', 'like', '%'.$row['birth_registrar_office_as_in_birth_certificate'].'%')->first();
+                $nationalityId = Nationality::where('name', 'like', '%'.$row['nationality'].'%')->first();
+                $identityType = Identity_type::where('national_code', 'like', '%'.$row['identity_type'].'%')->first();
                 $academicPeriod = Academic_period::where('name', '=', $row['academic_period'])->first();
 
 
@@ -144,10 +144,10 @@ 
 
                 $student = Institution_class_student::where('student_id', '=', $studentInfo->id)->first();
 
-                if(!empty($row['admission_no']) && !empty($academicPeriod)){
-                    Institution_student::where('student_id','=',$studentInfo->id)
-                    ->where('institution_id','=', $institution)
-                    ->where('academic_period_id','=',$academicPeriod->id)
+                if (!empty($row['admission_no']) && !empty($academicPeriod)) {
+                    Institution_student::where('student_id', '=', $studentInfo->id)
+                    ->where('institution_id', '=', $institution)
+                    ->where('academic_period_id', '=', $academicPeriod->id)
                     ->update(['admission_id'=> $row['admission_no']]);
                 }
                 
@@ -173,17 +173,17 @@ 
                 if (!empty($row['bmi_height']) && (!empty($row['bmi_weight']))) {
 
                     // convert Meeter to CM
-                    $hight = $row['bmi_height'] / 100;
+                    $hight = $row['bmi_height']/100;
 
                     //calculate BMI
-                    $bodyMass = ($row['bmi_weight']) / pow($hight, 2);
+                    $bodyMass = ($row['bmi_weight'])/pow($hight, 2);
 
                     $bmiAcademic = Academic_period::where('name', '=', $row['bmi_academic_period'])->first();
-                    $count = User_body_mass::where('academic_period_id' ,'=',$bmiAcademic->id )
-                            ->where('security_user_id','=',$student->student_id)->count();
+                    $count = User_body_mass::where('academic_period_id', '=', $bmiAcademic->id)
+                            ->where('security_user_id', '=', $student->student_id)->count();
 
                     \Log::debug('User_body_mass');
-                    if(!($count > 0)){
+                    if (!($count > 0)) {
                         User_body_mass::create([
                             'height' => $row['bmi_height'],
                             'weight' => $row['bmi_weight'],
@@ -197,10 +197,10 @@ 
                 }
 
                 if (!empty($row['fathers_full_name']) && ($row['fathers_date_of_birth_yyyy_mm_dd'] !== null)) {
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['fathers_address_area'] . '%')->first();
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['fathers_nationality'] . '%')->first();
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['fathers_identity_type'] . '%')->first();
-                    $openemisFather =  $this->uniqueUid::getUniqueAlphanumeric();
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['fathers_address_area'].'%')->first();
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['fathers_nationality'].'%')->first();
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['fathers_identity_type'].'%')->first();
+                    $openemisFather = $this->uniqueUid::getUniqueAlphanumeric();
 
                     $identityType = ($identityType !== null) ? $identityType->id : null;
                     $nationalityId = $nationalityId !== null ? $nationalityId->id : null;
@@ -214,7 +214,7 @@ 
 
                     if ($father === null) {
                         $data = [
-                            'username' => str_replace('-','',$openemisFather),
+                            'username' => str_replace('-', '', $openemisFather),
                             'openemis_no' => $openemisFather,
                             'first_name' => $row['fathers_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
                             'last_name' => genNameWithInitials($row['fathers_full_name']),
@@ -233,25 +233,25 @@ 
                         $father['guardian_relation_id'] = 1;
                         if (array_key_exists('fathers_phone', $row)) {
                             $father['contact'] = $row['fathers_phone'];
-                            User_contact::createOrUpdate($father,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($father, $this->file['security_user_id']);
                         }
                         Student_guardian::createStudentGuardian($student, $father, $this->file['security_user_id']);
-                    } else {
+                    }else {
                         Security_user::where('id', '=', $father->id)
                                 ->update(['is_guardian' => 1]);
                         $father['guardian_relation_id'] = 1;
                         if (array_key_exists('fathers_phone', $row)) {
                             $father['contact'] = $row['fathers_phone'];
-                            User_contact::createOrUpdate($father,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($father, $this->file['security_user_id']);
                         }
                         Student_guardian::createStudentGuardian($student, $father, $this->file['security_user_id']);
                     }
                 }
 
                 if (!empty($row['mothers_full_name']) && ($row['mothers_date_of_birth_yyyy_mm_dd'] !== null)) {
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['mothers_address_area'] . '%')->first();
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['mothers_nationality'] . '%')->first();
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['mothers_identity_type'] . '%')->first();
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['mothers_address_area'].'%')->first();
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['mothers_nationality'].'%')->first();
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['mothers_identity_type'].'%')->first();
                     $openemisMother = $this->uniqueUid::getUniqueAlphanumeric();
 
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -266,7 +266,7 @@ 
 
                     if ($mother === null) {
                         $mother = Security_user::create([
-                                    'username' => str_replace('-','',$openemisMother),
+                                    'username' => str_replace('-', '', $openemisMother),
                                     'openemis_no' => $openemisMother,
                                     'first_name' => $row['mothers_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
                                     'last_name' => genNameWithInitials($row['mothers_full_name']),
@@ -284,16 +284,16 @@ 
                         $mother['guardian_relation_id'] = 2;
                         if (array_key_exists('mothers_phone', $row)) {
                             $mother['contact'] = $row['mothers_phone'];
-                            User_contact::createOrUpdate($mother,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($mother, $this->file['security_user_id']);
                         }   
                         Student_guardian::createStudentGuardian($student, $mother, $this->file['security_user_id']);
-                    } else {
+                    }else {
                         Security_user::where('id', '=', $mother->id)
                                 ->update(['is_guardian' => 1]);
                         $mother['guardian_relation_id'] = 2;
                         if (array_key_exists('mothers_phone', $row)) {
                             $mother['contact'] = $row['mothers_phone'];
-                            User_contact::createOrUpdate($mother,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($mother, $this->file['security_user_id']);
                         }
                         Student_guardian::createStudentGuardian($student, $mother, $this->file['security_user_id']);
                     }
@@ -302,9 +302,9 @@ 
 
                 if (!empty($row['guardians_full_name']) && ($row['guardians_date_of_birth_yyyy_mm_dd'] !== null)) {
                     $genderId = $row['guardians_gender_mf'] == 'M' ? 1 : 2;
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['guardians_address_area'] . '%')->first();
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['guardians_nationality'] . '%')->first();
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['guardians_identity_type'] . '%')->first();
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['guardians_address_area'].'%')->first();
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['guardians_nationality'].'%')->first();
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['guardians_identity_type'].'%')->first();
                     $openemisGuardian = $this->uniqueUid::getUniqueAlphanumeric();
 
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -319,7 +319,7 @@ 
 
                     if ($guardian === null) {
                         $guardian = Security_user::create([
-                                    'username' => str_replace('-','',$openemisGuardian),
+                                    'username' => str_replace('-', '', $openemisGuardian),
                                     'openemis_no' => $openemisGuardian,
                                     'first_name' => $row['guardians_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
                                     'last_name' => genNameWithInitials($row['guardians_full_name']),
@@ -338,22 +338,22 @@ 
                         $guardian['guardian_relation_id'] = 3;
                         if (array_key_exists('guardians_phone', $row)) {
                             $guardian['contact'] = $row['guardians_phone'];
-                            User_contact::createOrUpdate($guardian,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($guardian, $this->file['security_user_id']);
                         }  
                         Student_guardian::createStudentGuardian($student, $guardian, $this->file['security_user_id']);
-                    } else {
+                    }else {
                         Security_user::where('id', '=', $guardian->id)
                                 ->update(['is_guardian' => 1]);
                         $guardian['guardian_relation_id'] = 3;
                         if (array_key_exists('guardians_phone', $row)) {
                             $guardian['contact'] = $row['guardians_phone'];
-                            User_contact::createOrUpdate($guardian,$this->file['security_user_id']);
+                            User_contact::createOrUpdate($guardian, $this->file['security_user_id']);
                         } 
                         Student_guardian::createStudentGuardian($student, $guardian, $this->file['security_user_id']);
                     }
                 }
 
-                $optionalSubjects =  Institution_class_subject::getStudentOptionalSubject($subjects, $student, $row, $institution);
+                $optionalSubjects = Institution_class_subject::getStudentOptionalSubject($subjects, $student, $row, $institution);
 
                 $allSubjects = array_merge_recursive($optionalSubjects, $mandatorySubject);
                 // $stundetSubjects = $this->getStudentSubjects($student);
@@ -362,10 +362,10 @@ 
                 if (!empty($allSubjects)) {
                     $allSubjects = unique_multidim_array($allSubjects, 'institution_subject_id');
                     $this->student = $student;
-                    $allSubjects = array_map(array($this,'setStudentSubjects'),$allSubjects);
+                    $allSubjects = array_map(array($this, 'setStudentSubjects'), $allSubjects);
                     // $allSubjects = array_unique($allSubjects,SORT_REGULAR);
                     $allSubjects = unique_multidim_array($allSubjects, 'education_subject_id');
-                    array_walk($allSubjects,array($this,'insertSubject'));
+                    array_walk($allSubjects, array($this, 'insertSubject'));
                     array_walk($allSubjects, array($this, 'updateSubjectCount'));
                 }
 
@@ -378,7 +378,7 @@ 
                             '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,7 +48,7 @@ 
     return (bool) preg_match('/^[0-9a-f]{40}$/i', $str);
 }
 
-function isEmpty($value){
+function isEmpty($value) {
     return $value['institution_optional_subject'] !== null;
 }
 
@@ -57,7 +57,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;
@@ -69,10 +69,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);
@@ -80,25 +80,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;
         }
@@ -114,17 +114,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)
@@ -136,23 +136,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)
@@ -164,14 +164,14 @@ 
 }
 
 
-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;
         }
 
@@ -179,37 +179,37 @@ 
 }
 
 function isEmptyRow($row) {
-    foreach($row as $cell){
+    foreach ($row as $cell) {
         if (null !== $cell) return false;
     }
     return true;
 }
 
-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;
             }
         }

app/Models/Unique_user_id.php

Spacing +3 added lines, -3 removed lines

@@ -56,10 +56,10 @@ 
     {
        try {
             // regenerate unique id if it's not available
-        $uniqueId =  $this->uniqueUserId::isValidUniqueId($user['openemis_no'],9) ? $this->uniqueUserId::getUniqueAlphanumeric() : $user['openemis_no'];
+        $uniqueId = $this->uniqueUserId::isValidUniqueId($user['openemis_no'], 9) ? $this->uniqueUserId::getUniqueAlphanumeric() : $user['openemis_no'];
 
         //check if the user's entry exits ?
-        $exists = Unique_user_id::where('unique_id' , $uniqueId)->exists();
+        $exists = Unique_user_id::where('unique_id', $uniqueId)->exists();
 
         if (!$exists) {
             // try to feed unique user id
@@ -69,7 +69,7 @@ 
             ]);
         }
         return $user;
-       } catch (\Exception $th) {
+       }catch (\Exception $th) {
             Log::error($th->getMessage());
            $user['openemis_no'] = $this->uniqueUserId::getUniqueAlphanumeric();
            $this->updateOrInsertRecord($user);

app/Models/Security_user.php

Spacing +10 added lines, -10 removed lines

@@ -149,9 +149,9 @@ 
     public function getMatches($student)
     {
         return $this
-        ->where('gender_id',$student['gender'] + 1)
-        ->where('institutions.code',$student['schoolid'])
-        ->where('date_of_birth',$student['b_date'])
+        ->where('gender_id', $student['gender'] + 1)
+        ->where('institutions.code', $student['schoolid'])
+        ->where('date_of_birth', $student['b_date'])
             ->join('institution_students', 'security_users.id', 'institution_students.student_id')
             ->join('institutions', 'institution_students.institution_id', 'institutions.id')
             ->get()->toArray();
@@ -166,9 +166,9 @@ 
     public function getStudentCount($student)
     {
         return $this
-        ->where('gender_id',$student['gender'] + 1)
-        ->where('institutions.code',$student['schoolid'])
-        ->where('date_of_birth',$student['b_date'])
+        ->where('gender_id', $student['gender'] + 1)
+        ->where('institutions.code', $student['schoolid'])
+        ->where('date_of_birth', $student['b_date'])
             ->join('institution_students', 'security_users.id', 'institution_students.student_id')
             ->join('institutions', 'institution_students.institution_id', 'institutions.id')
             ->count();
@@ -202,7 +202,7 @@ 
             $studentData['id'] = $id;
             $this->uniqueUserId->updateOrInsertRecord($studentData);
             return $studentData;
-        } catch (\Exception $th) {
+        }catch (\Exception $th) {
             Log::error($th->getMessage());
             // in case of duplication of the Unique ID this will recursive.
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();
@@ -223,7 +223,7 @@ 
         $this->uniqueUserId = new Unique_user_id();
         $this->uniqueUId = new UniqueUid();
         // regenerate unique id if it's not available
-        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'],9)) ?  $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
+        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'], 9)) ? $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
 
         $studentData = [
             'username' => str_replace('-', '', $uniqueId),
@@ -237,11 +237,11 @@ 
         ];
 
         try {
-            self::where( 'id'  , $sis_student['student_id'])->update($studentData);
+            self::where('id', $sis_student['student_id'])->update($studentData);
             $studentData['id'] = $sis_student['student_id'];
             $this->uniqueUserId->updateOrInsertRecord($studentData);
             return $studentData;
-        } catch (\Exception $th) {
+        }catch (\Exception $th) {
             Log::error($th);
             // in case of duplication of the Unique ID this will recursive.
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();

app/Exports/ExaminationStudentsExport.php

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
 use Maatwebsite\Excel\Concerns\WithHeadings;
 use Carbon\Carbon;
 
-class ExaminationStudentsExport implements FromQuery , WithHeadings 
+class ExaminationStudentsExport implements FromQuery, WithHeadings 
 {
 
     use Exportable;
@@ -44,6 +44,6 @@ 
     {
         return Examination_student::query()->whereNotNull('nsid')
         ->distinct('nsid')
-        ->whereDate('updated_at','>=',Carbon::today());
+        ->whereDate('updated_at', '>=', Carbon::today());
     }
 }

app/Http/Controllers/ExaminationStudentsController.php

Spacing +39 added lines, -39 removed lines

@@ -30,7 +30,7 @@ 
         $this->grade = $grade;
         $this->student = new Security_user();
         $this->examination_student = new Examination_student();
-        $this->academic_period =  Academic_period::where('code', '=', $this->year)->first();
+        $this->academic_period = Academic_period::where('code', '=', $this->year)->first();
         $this->education_grade = Education_grade::where('code', '=', $this->grade)->first();
         $this->output = new \Symfony\Component\Console\Output\ConsoleOutput();
     }
@@ -72,10 +72,10 @@ 
                     );
                     Session::flash('message', 'File upload successfully!');
                     // Redirect to index
-                } else {
+                }else {
                     Session::flash('message', 'File too large. File must be less than 20MB.');
                 }
-            } else {
+            }else {
                 Session::flash('message', 'Invalid File Extension.');
             }
         }
@@ -97,7 +97,7 @@ 
             $import->import($excelFile, 'local', \Maatwebsite\Excel\Excel::CSV);
             if ($import->failures()->count() > 0) {
                 $errors = $import->failures();
-                $columns =  [
+                $columns = [
                     'remarks',
                     'st_no',
                     'stu_no',
@@ -118,10 +118,10 @@ 
                 Storage::put($file, implode(',', $columns));
 
                 foreach ($errors as $error) {
-                    Storage::append($file, implode(':', $error->errors()) . ',' . implode(',', $error->values()));
+                    Storage::append($file, implode(':', $error->errors()).','.implode(',', $error->values()));
                 }
             }
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
         }
     }
 
@@ -135,9 +135,9 @@ 
         $students = [];
         switch ($mode) {
             case 'duplicate':
-                $students =  DB::table('examination_students as es')
+                $students = DB::table('examination_students as es')
                     ->select(DB::raw('count(*) as total'), 'e2.*')
-                    ->join('examination_students as e2', 'es.nsid','e2.nsid')
+                    ->join('examination_students as e2', 'es.nsid', 'e2.nsid')
                     ->having('total', '>', 1)
                     ->groupBy('e2.st_no')
                     ->orderBy('e2.st_no')
@@ -145,18 +145,18 @@ 
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining duplicate');
+                $this->output->writeln(count($students).'students remaining duplicate');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
             case 'empty';
                 $students = Examination_student::whereNull('nsid')
-                    ->orWhere('nsid','<>','')
+                    ->orWhere('nsid', '<>', '')
                     ->offset($offset)
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining empty');
+                $this->output->writeln(count($students).'students remaining empty');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
@@ -165,7 +165,7 @@ 
                     whereRaw('CHAR_LENGTH(nsid) > 11')
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining with wrong NSID');
+                $this->output->writeln(count($students).'students remaining with wrong NSID');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
                 break;
@@ -174,14 +174,14 @@ 
                 ->count();
                 $all = Examination_student::select('nsid')
                     ->count();
-                $this->output->writeln( $all. 'Total Unique nsid are: ' .$count);
+                $this->output->writeln($all.'Total Unique nsid are: '.$count);
                 break;
             default:
                 $students = Examination_student::offset($offset)
                     ->limit($limit)
                     ->get()->toArray();
                 $students = (array) json_decode(json_encode($students));
-                $this->output->writeln(count($students) . 'students remaining empty');
+                $this->output->writeln(count($students).'students remaining empty');
                 array_walk($students, array($this, 'clone'));
                 $this->output->writeln('All are generated');
         }
@@ -227,13 +227,13 @@ 
      */
     public function clone($student)
     {
-        $student = (array)json_decode(json_encode($student));
+        $student = (array) json_decode(json_encode($student));
         //get student matching with same dob and gender
 
         $matchedStudent = $this->getMatchingStudents($student);
 
         // if the first match missing do complete insertion
-        $institution = Institution::where('code', '=', (int)$student['schoolid'])->first();
+        $institution = Institution::where('code', '=', (int) $student['schoolid'])->first();
 
         if (!is_null($institution)) {
 
@@ -260,7 +260,7 @@ 
 
                 //TODO implement insert student to admission table
                 $student['id'] = $sis_student['id'];
-                $sis_student['student_id'] =  $student['id'];
+                $sis_student['student_id'] = $student['id'];
 
                 $student = $this->setIsTakingExam($student);
                 if (count($institutionClass) == 1) {
@@ -268,13 +268,13 @@ 
                     Institution_student::createExaminationData($student, $admissionInfo);
                     Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_class_student::createExaminationData($student, $admissionInfo);
-                } else {
+                }else {
                     Institution_student_admission::createExaminationData($student, $admissionInfo);
                     Institution_student::createExaminationData($student, $admissionInfo);
                 }
                 $this->updateStudentId($student, $sis_student);
                 // update the matched student's data    
-            } else {
+            }else {
                 $student = $this->setIsTakingExam($student);
                 $studentData = $this->student->updateExaminationStudent($student, $matchedStudent);
                 $matchedStudent = array_merge((array) $student, $matchedStudent);
@@ -282,9 +282,9 @@ 
                 Institution_student::updateExaminationData($studentData, $admissionInfo);
                 $this->updateStudentId($student, $studentData);
             }
-        } else {
+        }else {
 
-            $this->output->writeln('Student ' . $student['st_no'] . ' not imorted' . $student['f_name']);
+            $this->output->writeln('Student '.$student['st_no'].' not imorted'.$student['f_name']);
         }
     }
 
@@ -301,18 +301,18 @@ 
         /**
          */
         $sis_student = $this->student->getMatches($student);
-        $doe_students =  Examination_student::where('gender',$student['gender'])
-            ->where('b_date',$student['b_date'])
-            ->where('schoolid',$student['schoolid'])
+        $doe_students = Examination_student::where('gender', $student['gender'])
+            ->where('b_date', $student['b_date'])
+            ->where('schoolid', $student['schoolid'])
             ->count();
         $count = $this->student->getStudentCount($student);
 
         $studentData = [];
-        $sis_users  = (array) json_decode(json_encode($sis_student), true);
+        $sis_users = (array) json_decode(json_encode($sis_student), true);
         // if the same gender same DOE has more than one 
-        if(($doe_students > 1) || ($count > 1)){
-            $studentData = $this->searchSimilarName($student, $sis_users,false);
-        }else{
+        if (($doe_students > 1) || ($count > 1)) {
+            $studentData = $this->searchSimilarName($student, $sis_users, false);
+        }else {
             $studentData = $this->searchSimilarName($student, $sis_users);
         }   
         return $studentData;
@@ -325,7 +325,7 @@ 
      * @param array $sis_students
      * @return array
      */
-    public function searchSimilarName($student, $sis_students,$surname_search = true)
+    public function searchSimilarName($student, $sis_students, $surname_search = true)
     {
         $highest = [];
         $minDistance = 0;
@@ -345,7 +345,7 @@ 
             }
         }
 
-        if($surname_search){
+        if ($surname_search) {
             if (empty($highest)) {
                 foreach ($sis_students as $key => $value) {
                     //search name with last name
@@ -361,8 +361,8 @@ 
             }
         }
 
-        if(count($matches)>1){
-            $highest =  $this->searchSimilarName($student, $sis_students,false);
+        if (count($matches) > 1) {
+            $highest = $this->searchSimilarName($student, $sis_students, false);
         }
 
         return $highest;
@@ -378,18 +378,18 @@ 
     public function updateStudentId($student, $sis_student)
     {
         try {
-            $student['nsid'] =  $sis_student['openemis_no'];
+            $student['nsid'] = $sis_student['openemis_no'];
             // add new NSID to the examinations data set
             unset($student['id']);
             unset($student['taking_g5_exam']);
             unset($student['taking_al_exam']);
             unset($student['taking_ol_exam']);
             unset($student['total']);
-            $students['updated_at'] =  now();
+            $students['updated_at'] = now();
             $this->examination_student->where('st_no', $student['st_no'])->update($student);
             unset($student['st_no']);
-            $this->output->writeln('Updated  to NSID' . $sis_student['openemis_no']);
-        } catch (\Exception $th) {
+            $this->output->writeln('Updated  to NSID'.$sis_student['openemis_no']);
+        }catch (\Exception $th) {
             dd($th);
             $this->output->writeln('error');
             Log::error($th);
@@ -407,7 +407,7 @@ 
         try {
             (new ExaminationStudentsExport)->store('examination/student_data_with_nsid.'.time().'.csv');
             (new ExportReady($adminUser));
-        } catch (\Throwable $th) {
+        }catch (\Throwable $th) {
             //throw $th;
             dd($th);
         }
@@ -417,13 +417,13 @@ 
     public function downloadErrors()
     {
 
-        $file_path = storage_path() . '/app/examination/errors.csv';
+        $file_path = storage_path().'/app/examination/errors.csv';
         return Response::download($file_path);
     }
 
     public function downloadProcessedFile()
     {
-        $file_path = storage_path() . '/app/examination/student_data_with_nsid.csv';
+        $file_path = storage_path().'/app/examination/student_data_with_nsid.csv';
         return Response::download($file_path);
     }
 }