moe-lk / bulk-upload

app/Http/Middleware/Authenticate.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      */
     protected function redirectTo($request)
     {
-        if (! $request->expectsJson()) {
+        if (!$request->expectsJson()) {
             return route('login');
         }
     }

app/Http/Controllers/FileController.php

Spacing +14 added lines, -14 removed lines

@@ -34,7 +34,7 @@ 
      * @param Request $request
      * @return \Illuminate\Http\JsonResponse
      */
-    public function upload(Request $request){
+    public function upload(Request $request) {
 
 
 
@@ -76,7 +76,7 @@ 
         $institution = auth()->user()->permissions->isEmpty() ? auth()->user()->principal[0]->institution_group[0]->institution->code : auth()->user()->permissions[0]->institution_staff->institution->code;
 
 
-        $fileName = time().'_'.$institution.'_'.str_replace(' ','_', clean($class->name)).'_'.auth()->user()->openemis_no.'_student_bulk_data.xlsx';
+        $fileName = time().'_'.$institution.'_'.str_replace(' ', '_', clean($class->name)).'_'.auth()->user()->openemis_no.'_student_bulk_data.xlsx';
         Storage::disk('local')->putFileAs(
             'sis-bulk-data-files/',
             $uploadFile,
@@ -84,7 +84,7 @@ 
         );
 
         $upload = new Upload;
-        $upload->fileName =$fileName;
+        $upload->fileName = $fileName;
         $upload->model = 'Student';
         $upload->node = 'None';
         $upload->institution_class_id = $class->id;
@@ -95,8 +95,8 @@ 
         return redirect('/')->withSuccess('The file is uploaded, we will process and let you know by your email');
     }
 
-    public function updateQueueWithUnprocessedFiles($id, $action){
-        if($action == 100){
+    public function updateQueueWithUnprocessedFiles($id, $action) {
+        if ($action == 100) {
             DB::table('uploads')
                 ->where('id', $id)
                 ->update(['is_processed' => 0]);
@@ -108,14 +108,14 @@ 
     }
 
 
-    public function downloadTemplate(){
+    public function downloadTemplate() {
         $filename = 'censusNo_className_sis_students_bulk_upload';
         $version = '2007_V1.7_20200510.xlsx';
-        $file_path = storage_path() .'/app/public/'. $filename.'_'.$version;;
+        $file_path = storage_path().'/app/public/'.$filename.'_'.$version; ;
         if (file_exists($file_path))
         {
             return Response::download($file_path, Auth::user()->openemis_no.'_'.$filename.$version, [
-                'Content-Length: '. filesize($file_path)
+                'Content-Length: '.filesize($file_path)
             ]);
         }
         else
@@ -129,13 +129,13 @@ 
      * @param $filename
      * @return Processed excel file with error
      */
-    public function downloadErrorFile($filename){
+    public function downloadErrorFile($filename) {
 
-        $file_path = storage_path().'/app/sis-bulk-data-files/processed/'. $filename;
+        $file_path = storage_path().'/app/sis-bulk-data-files/processed/'.$filename;
         if (file_exists($file_path))
         {
             return Response::download($file_path, $filename, [
-                'Content-Length: '. filesize($file_path)
+                'Content-Length: '.filesize($file_path)
             ]);
         }
         else
@@ -145,12 +145,12 @@ 
     }
 
 
-    public function downloadFile($filename){
-        $file_path = storage_path().'/app/sis-bulk-data-files/'. $filename;
+    public function downloadFile($filename) {
+        $file_path = storage_path().'/app/sis-bulk-data-files/'.$filename;
         if (file_exists($file_path))
         {
             return Response::download($file_path, $filename, [
-                'Content-Length: '. filesize($file_path)
+                'Content-Length: '.filesize($file_path)
             ]);
         }
         else

app/Http/Controllers/Auth/LoginController.php

Spacing +3 added lines, -3 removed lines

@@ -54,10 +54,10 @@ 
     /**
      * @return string
      */
-    public function findUsername(){
+    public function findUsername() {
         $login = request()->input('username');
 
-        $fieldType  = filter_var($login,FILTER_VALIDATE_EMAIL) ? 'email' : 'username';
+        $fieldType = filter_var($login, FILTER_VALIDATE_EMAIL) ? 'email' : 'username';
 
         request()->merge([$fieldType => $login]);
 
@@ -67,7 +67,7 @@ 
     /**
      * @return string
      */
-    public function username(){
+    public function username() {
         return $this->username;
     }
 }

app/Permissions/HasPermissionsTrait.php

Spacing +3 added lines, -3 removed lines

@@ -7,12 +7,12 @@
 
 trait HasPermissionsTrait
 {
-    public function roles(){
-        return $this->belongsToMany(Security_group_user::class,'security_group_users');
+    public function roles() {
+        return $this->belongsToMany(Security_group_user::class, 'security_group_users');
     }
 
 
-    public function hasRole( ... $roles ) {
+    public function hasRole(... $roles) {
         foreach ($roles as $role) {
             if ($this->roles->contains('code', $role)) {
                 return true;

app/Imports/Import.php

Spacing +38 added lines, -38 removed lines

@@ -84,10 +84,10 @@ 
         $highestColumn = $this->worksheet->getHighestDataColumn(3);
         $higestRow = 0;
         for ($row = $this->startRow(); $row <= $this->highestRow; $row++) {
-            $rowData = $this->worksheet->rangeToArray('A' . $row . ':' . $highestColumn . $row, NULL, TRUE, FALSE);
+            $rowData = $this->worksheet->rangeToArray('A'.$row.':'.$highestColumn.$row, NULL, TRUE, FALSE);
             if (isEmptyRow(reset($rowData))) {
                 continue;
-            } else {
+            }else {
                 $higestRow += 1;
             }
         }
@@ -96,7 +96,7 @@ 
 
     public function validateColumns($column) {
         $columns = Config::get('excel.columns');
-        if ( ($column !== "") && (!in_array($column,$columns))) {
+        if (($column !== "") && (!in_array($column, $columns))) {
 
             $this->isValidSheet = false;
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
@@ -111,16 +111,16 @@ 
         $highestColumn = $this->worksheet->getHighestDataColumn(3);
         $higestRow = 1;
         for ($row = $this->startRow(); $row <= $this->highestRow; $row++) {
-            $rowData = $this->worksheet->rangeToArray('A' . $row . ':' . $highestColumn . $row, NULL, TRUE, FALSE);
+            $rowData = $this->worksheet->rangeToArray('A'.$row.':'.$highestColumn.$row, NULL, TRUE, FALSE);
             if (isEmptyRow(reset($rowData))) {
                 continue;
-            } else {
+            }else {
                 $higestRow += 1;
             }
         }
         if ($higestRow == 0) {
             exit;
-        } else {
+        }else {
             return $higestRow;
         }
     }
@@ -135,22 +135,22 @@ 
     }
 
 
-    protected function formateDate($row,$column,$format = 'Y-m-d'){
+    protected function formateDate($row, $column, $format = 'Y-m-d') {
         try {
-            if(!empty($row[$column]) && ($row[$column] !== null)){
-                switch (gettype($row[$column])){
+            if (!empty($row[$column]) && ($row[$column] !== null)) {
+                switch (gettype($row[$column])) {
                     case 'string':
                         $row[$column] = preg_replace('/[^A-Za-z0-9\-]/', '-', $row[$column]);
                         $row[$column] = date($format, strtotime($row[$column])); //date($row[$column]);
-                        $row[$column] =  \Carbon\Carbon::createFromFormat($format, $row[$column]);
+                        $row[$column] = \Carbon\Carbon::createFromFormat($format, $row[$column]);
                         break;
                     case 'double';
-                        $row[$column] =  \PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row[$column]);
+                        $row[$column] = \PhpOffice\PhpSpreadsheet\Shared\Date::excelToDateTimeObject($row[$column]);
                         break;
                 }
             }
             return $row;
-        }catch (Exception $e){
+        }catch (Exception $e) {
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
             $failure = new Failure(3, 'remark', [0 => 'Template is not valid for upload, use the template given in the system'], [null]);
             $failures = [0 => $failure];
@@ -160,32 +160,32 @@ 
     }
 
 
-    protected function mapFields($row){
+    protected function mapFields($row) {
 
         $keys = array_keys($row);
 
-        array_walk($keys,array($this,'validateColumns'));
-            $row = $this->formateDate($row,'date_of_birth_yyyy_mm_dd');
-            $row = $this->formateDate($row,'bmi_date_yyyy_mm_dd');
-            $row = $this->formateDate($row,'start_date_yyyy_mm_dd');
-            $row = $this->formateDate($row,'fathers_date_of_birth_yyyy_mm_dd');
-            $row = $this->formateDate($row,'mothers_date_of_birth_yyyy_mm_dd');
-            $row = $this->formateDate($row,'guardians_date_of_birth_yyyy_mm_dd');
+        array_walk($keys, array($this, 'validateColumns'));
+            $row = $this->formateDate($row, 'date_of_birth_yyyy_mm_dd');
+            $row = $this->formateDate($row, 'bmi_date_yyyy_mm_dd');
+            $row = $this->formateDate($row, 'start_date_yyyy_mm_dd');
+            $row = $this->formateDate($row, 'fathers_date_of_birth_yyyy_mm_dd');
+            $row = $this->formateDate($row, 'mothers_date_of_birth_yyyy_mm_dd');
+            $row = $this->formateDate($row, 'guardians_date_of_birth_yyyy_mm_dd');
             $columns = Config::get('excel.columns');
-            if(!$this->template) {
+            if (!$this->template) {
                 array_walk($columns, array($this, 'checkKeys'), $row);
                 $this->template = true;
             }
-            $row['admission_no'] =  str_pad($row['admission_no'], 4, '0', STR_PAD_LEFT);
+            $row['admission_no'] = str_pad($row['admission_no'], 4, '0', STR_PAD_LEFT);
             if ($row['identity_type'] == 'BC' && (!empty($row['birth_divisional_secretariat'])) && ($row['identity_number'] !== null) && $row['date_of_birth_yyyy_mm_dd'] !== null) {
-                $row['identity_number'] =  str_pad($row['identity_number'], 4, '0', STR_PAD_LEFT);
+                $row['identity_number'] = str_pad($row['identity_number'], 4, '0', STR_PAD_LEFT);
                 // dd(($row['date_of_birth_yyyy_mm_dd']));
-                $BirthDivision = Area_administrative::where('name', 'like', '%' . $row['birth_divisional_secretariat'] . '%')->where('area_administrative_level_id', '=', 5)->first();
+                $BirthDivision = Area_administrative::where('name', 'like', '%'.$row['birth_divisional_secretariat'].'%')->where('area_administrative_level_id', '=', 5)->first();
                 if ($BirthDivision !== null) {
-                    $BirthArea = Area_administrative::where('name', 'like', '%' . $row['birth_registrar_office_as_in_birth_certificate'] . '%')
+                    $BirthArea = Area_administrative::where('name', 'like', '%'.$row['birth_registrar_office_as_in_birth_certificate'].'%')
                         ->where('parent_id', '=', $BirthDivision->id)->first();
                     if ($BirthArea !== null) {
-                        $row['identity_number'] = $BirthArea->id . '' . $row['identity_number'] . '' . substr($row['date_of_birth_yyyy_mm_dd']->format("yy"), -2) . '' . $row['date_of_birth_yyyy_mm_dd']->format("m");
+                        $row['identity_number'] = $BirthArea->id.''.$row['identity_number'].''.substr($row['date_of_birth_yyyy_mm_dd']->format("yy"), -2).''.$row['date_of_birth_yyyy_mm_dd']->format("m");
                     }
                 }
             }
@@ -193,12 +193,12 @@ 
         return $row;
     }
 
-    protected function checkKeys($key,$count,$row){
-       if(array_key_exists($key,$row)){
+    protected function checkKeys($key, $count, $row) {
+       if (array_key_exists($key, $row)) {
            return true;
-       }else{
+       }else {
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
-            $failure = new Failure($count, 'remark', [0 => 'Template is not valid for upload, use the template given in the system'. $key ,' Is missing form the template'], [null]);
+            $failure = new Failure($count, 'remark', [0 => 'Template is not valid for upload, use the template given in the system'.$key, ' Is missing form the template'], [null]);
             $failures = [0 => $failure];
             new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
         };
@@ -217,7 +217,7 @@ 
     public function map($row): array {
         try {
          $row = $this->mapFields($row);
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
                 $error = \Illuminate\Validation\ValidationException::withMessages([]);
                 $failure = new Failure(3, 'remark', [0 => 'Template is not valid for upload, use the template given in the system'], [null]);
                 $failures = [0 => $failure];
@@ -239,19 +239,19 @@ 
         if ($exceededStudents == true) {
             try {
                 $error = \Illuminate\Validation\ValidationException::withMessages([]);
-                $failure = new Failure(3, 'remark', [3 => 'Class student count exceeded! Max number of students is' . $institutionClass->no_of_students], [null]);
+                $failure = new Failure(3, 'remark', [3 => 'Class student count exceeded! Max number of students is'.$institutionClass->no_of_students], [null]);
                 $failures = [0 => $failure];
                 throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
                 Log::info('email-sent', [$this->file]);
-            } catch (Exception $e) {
+            }catch (Exception $e) {
                 Log::info('email-sending-failed', [$e]);
             }
-        } else {
+        }else {
             return true;
         }
     }
 
-    public function getNode(){
+    public function getNode() {
         return $this->file['node'];
     }
 
@@ -305,7 +305,7 @@ 
     /**
      *
      */
-    protected function setStudentSubjects($subject){
+    protected function setStudentSubjects($subject) {
         return [
             'id' => (string) Uuid::generate(4),
             'student_id' => $this->student->student_id,
@@ -321,8 +321,8 @@ 
         ];
     }
 
-    protected function insertSubject($subject){
-        if(!Institution_subject_student::isDuplicated($subject)){
+    protected function insertSubject($subject) {
+        if (!Institution_subject_student::isDuplicated($subject)) {
             Institution_subject_student::updateOrInsert($subject);
         }
     }

app/Imports/StudentImport.php

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

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

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

app/Providers/ValidationExtensionServiceProvider.php

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