Test Setup Failed
Push — master ( d7cd8c...4050cf )
by Mohamed
18:09 queued 11:08
created
app/Imports/UsersImport.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
     public function registerEvents(): array
76 76
     {
77 77
         return [
78
-            BeforeSheet::class => function (BeforeSheet $event) {
78
+            BeforeSheet::class => function(BeforeSheet $event) {
79 79
                 $this->sheetNames[] = $event->getSheet()->getTitle();
80 80
                 $this->worksheet = $event->getSheet();
81 81
                 $this->validateClass();
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
                     throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
88 88
                 }
89 89
             },
90
-            BeforeImport::class => function (BeforeImport $event) {
90
+            BeforeImport::class => function(BeforeImport $event) {
91 91
                 $this->highestRow = ($event->getReader()->getDelegate()->getActiveSheet()->getHighestDataRow('C'));
92 92
                 if ($this->highestRow < 3) {
93 93
                     $error = \Illuminate\Validation\ValidationException::withMessages([]);
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
                 $mandatorySubject = Institution_class_subject::getMandetorySubjects($this->file['institution_class_id']);
115 115
                 // dd($mandatorySubject);
116 116
                 $subjects = getMatchingKeys($row);
117
-                $genderId  = null;
117
+                $genderId = null;
118 118
                 switch ($row['gender_mf']) {
119 119
                     case 'M':
120 120
                         $genderId = $row['gender_mf'] = 1;
@@ -122,15 +122,15 @@  discard block
 block discarded – undo
122 122
                         $this->maleStudentsCount += 1;
123 123
                         break;
124 124
                     case 'F':
125
-                        $genderId =  $row['gender_mf'] = 2;
125
+                        $genderId = $row['gender_mf'] = 2;
126 126
                         $genderId = 2;
127 127
                         $this->femaleStudentsCount += 1;
128 128
                         break;
129 129
                 }
130 130
 
131
-                $BirthArea = Area_administrative::where('name', 'like', '%' . $row['birth_registrar_office_as_in_birth_certificate'] . '%')->first();
132
-                $nationalityId = Nationality::where('name', 'like', '%' . $row['nationality'] . '%')->first();
133
-                $identityType = Identity_type::where('national_code', 'like', '%' . $row['identity_type'] . '%')->first();
131
+                $BirthArea = Area_administrative::where('name', 'like', '%'.$row['birth_registrar_office_as_in_birth_certificate'].'%')->first();
132
+                $nationalityId = Nationality::where('name', 'like', '%'.$row['nationality'].'%')->first();
133
+                $identityType = Identity_type::where('national_code', 'like', '%'.$row['identity_type'].'%')->first();
134 134
                 $academicPeriod = Academic_period::where('name', '=', $row['academic_period'])->first();
135 135
 
136 136
 
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
                     if (array_key_exists('preferred_name', $row)) {
151 151
                     $preferred_name = $row['preferred_name'];
152 152
                 }
153
-                $student =  Security_user::create([
153
+                $student = Security_user::create([
154 154
                     'username' => str_replace('-', '', $openemisStudent),
155 155
                     'openemis_no' => $openemisStudent,
156 156
                     'first_name' => $row['full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
@@ -240,9 +240,9 @@  discard block
 block discarded – undo
240 240
                 if (!empty($row['bmi_weight']) && !empty($row['bmi_weight']) && !empty($row['bmi_date_yyyy_mm_dd'])) {
241 241
                     try {
242 242
                         // convert Meeter to CM
243
-                        $hight = $row['bmi_height'] / 100;
243
+                        $hight = $row['bmi_height']/100;
244 244
                         //calculate BMI
245
-                        $bodyMass = ($row['bmi_weight']) / pow($hight, 2);
245
+                        $bodyMass = ($row['bmi_weight'])/pow($hight, 2);
246 246
 
247 247
                         $bmiAcademic = Academic_period::where('name', '=', $row['bmi_academic_period'])->first();
248 248
 
@@ -256,17 +256,17 @@  discard block
 block discarded – undo
256 256
                             'security_user_id' => $student->student_id,
257 257
                             'created_user_id' => $this->file['security_user_id']
258 258
                         ]);
259
-                    } catch (\Throwable $th) {
260
-                        \Log::error('User_body_mass:' . $th->getMessage());
259
+                    }catch (\Throwable $th) {
260
+                        \Log::error('User_body_mass:'.$th->getMessage());
261 261
                     }
262 262
                 }
263 263
 
264 264
 
265 265
                 if (!empty($row['fathers_full_name']) && ($row['fathers_date_of_birth_yyyy_mm_dd'] !== null)) {
266 266
 
267
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['fathers_address_area'] . '%')->first();
268
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['fathers_nationality'] . '%')->first();
269
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['fathers_identity_type'] . '%')->first();
267
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['fathers_address_area'].'%')->first();
268
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['fathers_nationality'].'%')->first();
269
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['fathers_identity_type'].'%')->first();
270 270
                     $openemisFather = $this->uniqueUid::getUniqueAlphanumeric();
271 271
 
272 272
                     $identityType = ($identityType !== null) ? $identityType->id : null;
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
                             User_contact::createOrUpdate($father, $this->file['security_user_id']);
305 305
                         }
306 306
                         Student_guardian::createStudentGuardian($student, $father, $this->file['security_user_id']);
307
-                    } else {
307
+                    }else {
308 308
                         Security_user::where('id', '=', $father->id)
309 309
                             ->update(['is_guardian' => 1]);
310 310
                         $father['guardian_relation_id'] = 1;
@@ -317,9 +317,9 @@  discard block
 block discarded – undo
317 317
                 }
318 318
 
319 319
                 if (!empty($row['mothers_full_name']) && ($row['mothers_date_of_birth_yyyy_mm_dd'] !== null)) {
320
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['mothers_address_area'] . '%')->first();
321
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['mothers_nationality'] . '%')->first();
322
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['mothers_identity_type'] . '%')->first();
320
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['mothers_address_area'].'%')->first();
321
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['mothers_nationality'].'%')->first();
322
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['mothers_identity_type'].'%')->first();
323 323
                     $openemisMother = $this->uniqueUid::getUniqueAlphanumeric();
324 324
 
325 325
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
                             User_contact::createOrUpdate($mother, $this->file['security_user_id']);
356 356
                         }
357 357
                         Student_guardian::createStudentGuardian($student, $mother, $this->file['security_user_id']);
358
-                    } else {
358
+                    }else {
359 359
                         Security_user::where('id', '=', $mother->id)
360 360
                             ->update(['is_guardian' => 1]);
361 361
                         $mother['guardian_relation_id'] = 2;
@@ -370,9 +370,9 @@  discard block
 block discarded – undo
370 370
 
371 371
                 if (!empty($row['guardians_full_name']) && ($row['guardians_date_of_birth_yyyy_mm_dd'] !== null)) {
372 372
                     $genderId = $row['guardians_gender_mf'] == 'M' ? 1 : 2;
373
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['guardians_address_area'] . '%')->first();
374
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['guardians_nationality'] . '%')->first();
375
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['guardians_identity_type'] . '%')->first();
373
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['guardians_address_area'].'%')->first();
374
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['guardians_nationality'].'%')->first();
375
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['guardians_identity_type'].'%')->first();
376 376
                     $openemisGuardian = $this->uniqueUid::getUniqueAlphanumeric();
377 377
 
378 378
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -409,7 +409,7 @@  discard block
 block discarded – undo
409 409
                             User_contact::createOrUpdate($guardian, $this->file['security_user_id']);
410 410
                         }
411 411
                         Student_guardian::createStudentGuardian($student, $guardian, $this->file['security_user_id']);
412
-                    } else {
412
+                    }else {
413 413
                         Security_user::where('id', '=', $guardian->id)
414 414
                             ->update(['is_guardian' => 1]);
415 415
                         $guardian['guardian_relation_id'] = 3;
@@ -441,7 +441,7 @@  discard block
 block discarded – undo
441 441
 
442 442
                 if ($totalStudents['total'] > $institutionClass->no_of_students) {
443 443
                     $error = \Illuminate\Validation\ValidationException::withMessages([]);
444
-                    $failure = new Failure(3, 'rows', [3 => 'Class student count exceeded! Max number of students is ' . $institutionClass->no_of_students], [null]);
444
+                    $failure = new Failure(3, 'rows', [3 => 'Class student count exceeded! Max number of students is '.$institutionClass->no_of_students], [null]);
445 445
                     $failures = [0 => $failure];
446 446
                     throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
447 447
                     Log::info('email-sent', [$this->file]);
@@ -453,7 +453,7 @@  discard block
 block discarded – undo
453 453
                         'total_female_students' => $totalStudents['total_female_students']
454 454
                     ]);
455 455
             }
456
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
456
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
457 457
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
458 458
             //            $failure = new Failure(3, 'remark', [3 => ], [null]);
459 459
             $failures = $e->failures();
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
             '*.full_name' => 'required|regex:/^[\pL\s\-]+$/u|max:256',
470 470
             '*.preferred_name' => 'required|regex:/^[\pL\s\-]+$/u|max:90',
471 471
             '*.gender_mf' => 'required|in:M,F',
472
-            '*.date_of_birth_yyyy_mm_dd' => 'date|required|admission_age:' . $this->file['institution_class_id'],
472
+            '*.date_of_birth_yyyy_mm_dd' => 'date|required|admission_age:'.$this->file['institution_class_id'],
473 473
             '*.address' => 'nullable',
474 474
             '*.birth_registrar_office_as_in_birth_certificate' => 'nullable|exists:area_administratives,name|required_if:identity_type,BC|birth_place',
475 475
             '*.birth_divisional_secretariat' => 'nullable|exists:area_administratives,name|required_with:birth_registrar_office_as_in_birth_certificate',
@@ -479,8 +479,8 @@  discard block
 block discarded – undo
479 479
             '*.academic_period' => 'required|exists:academic_periods,name',
480 480
             '*.education_grade' => 'required',
481 481
             '*.option_*' => 'nullable|exists:education_subjects,name',
482
-            '*.bmi_height' => 'bail|required_with:*.bmi_weight|bmi:' . $this->file['institution_class_id'],
483
-            '*.bmi_weight' => 'bail|required_with:*.bmi_height|bmi:' . $this->file['institution_class_id'],
482
+            '*.bmi_height' => 'bail|required_with:*.bmi_weight|bmi:'.$this->file['institution_class_id'],
483
+            '*.bmi_weight' => 'bail|required_with:*.bmi_height|bmi:'.$this->file['institution_class_id'],
484 484
             '*.bmi_date_yyyy_mm_dd' => 'bail|required_with:*.bmi_height|date', //bmi:'. $this->file['institution_class_id'].'
485 485
             '*.bmi_academic_period' => 'bail|required_with:*.bmi_height|exists:academic_periods,name',
486 486
             '*.admission_no' => 'required|max:12|min:4|regex:/^[A-Za-z0-9\/]+$/',
Please login to merge, or discard this patch.
app/Imports/StudentUpdate.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 use Maatwebsite\Excel\Concerns\WithMultipleSheets;
56 56
 use Maatwebsite\Excel\Concerns\RegistersEventListeners;
57 57
 
58
-class StudentUpdate extends Import implements  ToModel, WithStartRow, WithHeadingRow, WithMultipleSheets, WithEvents, WithMapping, WithLimit, WithBatchInserts, WithValidation , SkipsOnFailure , SkipsOnError{
58
+class StudentUpdate extends Import implements  ToModel, WithStartRow, WithHeadingRow, WithMultipleSheets, WithEvents, WithMapping, WithLimit, WithBatchInserts, WithValidation, SkipsOnFailure, SkipsOnError{
59 59
 
60 60
     use Importable,
61 61
         RegistersEventListeners,
@@ -110,9 +110,9 @@  discard block
 block discarded – undo
110 110
                         break;
111 111
                 }
112 112
 
113
-                $BirthArea = Area_administrative::where('name', 'like', '%' . $row['birth_registrar_office_as_in_birth_certificate'] . '%')->first();
114
-                $nationalityId = Nationality::where('name', 'like', '%' . $row['nationality'] . '%')->first();
115
-                $identityType = Identity_type::where('national_code', 'like', '%' . $row['identity_type'] . '%')->first();
113
+                $BirthArea = Area_administrative::where('name', 'like', '%'.$row['birth_registrar_office_as_in_birth_certificate'].'%')->first();
114
+                $nationalityId = Nationality::where('name', 'like', '%'.$row['nationality'].'%')->first();
115
+                $identityType = Identity_type::where('national_code', 'like', '%'.$row['identity_type'].'%')->first();
116 116
                 $academicPeriod = Academic_period::where('name', '=', $institutionClass->academic_period_id)->first();
117 117
 
118 118
 
@@ -152,10 +152,10 @@  discard block
 block discarded – undo
152 152
 
153 153
                 $student = Institution_class_student::where('student_id', '=', $studentInfo->id)->first();
154 154
 
155
-                if(!empty($row['admission_no']) && !empty($academicPeriod)){
156
-                    Institution_student::where('student_id','=',$studentInfo->id)
157
-                    ->where('institution_id','=', $institution)
158
-                    ->where('academic_period_id','=',$academicPeriod->id)
155
+                if (!empty($row['admission_no']) && !empty($academicPeriod)) {
156
+                    Institution_student::where('student_id', '=', $studentInfo->id)
157
+                    ->where('institution_id', '=', $institution)
158
+                    ->where('academic_period_id', '=', $academicPeriod->id)
159 159
                     ->update(['admission_id'=> $row['admission_no']]);
160 160
                 }
161 161
                 
@@ -181,17 +181,17 @@  discard block
 block discarded – undo
181 181
                 if (!empty($row['bmi_height']) && (!empty($row['bmi_weight']))) {
182 182
 
183 183
                     // convert Meeter to CM
184
-                    $hight = $row['bmi_height'] / 100;
184
+                    $hight = $row['bmi_height']/100;
185 185
 
186 186
                     //calculate BMI
187
-                    $bodyMass = ($row['bmi_weight']) / pow($hight, 2);
187
+                    $bodyMass = ($row['bmi_weight'])/pow($hight, 2);
188 188
 
189 189
                     $bmiAcademic = Academic_period::where('name', '=', $row['bmi_academic_period'])->first();
190
-                    $count = User_body_mass::where('academic_period_id' ,'=',$bmiAcademic->id )
191
-                            ->where('security_user_id','=',$student->student_id)->count();
190
+                    $count = User_body_mass::where('academic_period_id', '=', $bmiAcademic->id)
191
+                            ->where('security_user_id', '=', $student->student_id)->count();
192 192
 
193 193
                     \Log::debug('User_body_mass');
194
-                    if(!($count > 0)){
194
+                    if (!($count > 0)) {
195 195
                         User_body_mass::create([
196 196
                             'height' => $row['bmi_height'],
197 197
                             'weight' => $row['bmi_weight'],
@@ -205,10 +205,10 @@  discard block
 block discarded – undo
205 205
                 }
206 206
 
207 207
                 if (!empty($row['fathers_full_name']) && ($row['fathers_date_of_birth_yyyy_mm_dd'] !== null)) {
208
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['fathers_address_area'] . '%')->first();
209
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['fathers_nationality'] . '%')->first();
210
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['fathers_identity_type'] . '%')->first();
211
-                    $openemisFather =  $this->uniqueUid::getUniqueAlphanumeric();
208
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['fathers_address_area'].'%')->first();
209
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['fathers_nationality'].'%')->first();
210
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['fathers_identity_type'].'%')->first();
211
+                    $openemisFather = $this->uniqueUid::getUniqueAlphanumeric();
212 212
 
213 213
                     $identityType = ($identityType !== null) ? $identityType->id : null;
214 214
                     $nationalityId = $nationalityId !== null ? $nationalityId->id : null;
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
 
223 223
                     if ($father === null) {
224 224
                         $data = [
225
-                            'username' => str_replace('-','',$openemisFather),
225
+                            'username' => str_replace('-', '', $openemisFather),
226 226
                             'openemis_no' => $openemisFather,
227 227
                             'first_name' => $row['fathers_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
228 228
                             'last_name' => genNameWithInitials($row['fathers_full_name']),
@@ -241,25 +241,25 @@  discard block
 block discarded – undo
241 241
                         $father['guardian_relation_id'] = 1;
242 242
                         if (array_key_exists('fathers_phone', $row)) {
243 243
                             $father['contact'] = $row['fathers_phone'];
244
-                            User_contact::createOrUpdate($father,$this->file['security_user_id']);
244
+                            User_contact::createOrUpdate($father, $this->file['security_user_id']);
245 245
                         }
246 246
                         Student_guardian::createStudentGuardian($student, $father, $this->file['security_user_id']);
247
-                    } else {
247
+                    }else {
248 248
                         Security_user::where('id', '=', $father->id)
249 249
                                 ->update(['is_guardian' => 1]);
250 250
                         $father['guardian_relation_id'] = 1;
251 251
                         if (array_key_exists('fathers_phone', $row)) {
252 252
                             $father['contact'] = $row['fathers_phone'];
253
-                            User_contact::createOrUpdate($father,$this->file['security_user_id']);
253
+                            User_contact::createOrUpdate($father, $this->file['security_user_id']);
254 254
                         }
255 255
                         Student_guardian::createStudentGuardian($student, $father, $this->file['security_user_id']);
256 256
                     }
257 257
                 }
258 258
 
259 259
                 if (!empty($row['mothers_full_name']) && ($row['mothers_date_of_birth_yyyy_mm_dd'] !== null)) {
260
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['mothers_address_area'] . '%')->first();
261
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['mothers_nationality'] . '%')->first();
262
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['mothers_identity_type'] . '%')->first();
260
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['mothers_address_area'].'%')->first();
261
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['mothers_nationality'].'%')->first();
262
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['mothers_identity_type'].'%')->first();
263 263
                     $openemisMother = $this->uniqueUid::getUniqueAlphanumeric();
264 264
 
265 265
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
 
275 275
                     if ($mother === null) {
276 276
                         $mother = Security_user::create([
277
-                                    'username' => str_replace('-','',$openemisMother),
277
+                                    'username' => str_replace('-', '', $openemisMother),
278 278
                                     'openemis_no' => $openemisMother,
279 279
                                     'first_name' => $row['mothers_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
280 280
                                     'last_name' => genNameWithInitials($row['mothers_full_name']),
@@ -292,16 +292,16 @@  discard block
 block discarded – undo
292 292
                         $mother['guardian_relation_id'] = 2;
293 293
                         if (array_key_exists('mothers_phone', $row)) {
294 294
                             $mother['contact'] = $row['mothers_phone'];
295
-                            User_contact::createOrUpdate($mother,$this->file['security_user_id']);
295
+                            User_contact::createOrUpdate($mother, $this->file['security_user_id']);
296 296
                         }   
297 297
                         Student_guardian::createStudentGuardian($student, $mother, $this->file['security_user_id']);
298
-                    } else {
298
+                    }else {
299 299
                         Security_user::where('id', '=', $mother->id)
300 300
                                 ->update(['is_guardian' => 1]);
301 301
                         $mother['guardian_relation_id'] = 2;
302 302
                         if (array_key_exists('mothers_phone', $row)) {
303 303
                             $mother['contact'] = $row['mothers_phone'];
304
-                            User_contact::createOrUpdate($mother,$this->file['security_user_id']);
304
+                            User_contact::createOrUpdate($mother, $this->file['security_user_id']);
305 305
                         }
306 306
                         Student_guardian::createStudentGuardian($student, $mother, $this->file['security_user_id']);
307 307
                     }
@@ -310,9 +310,9 @@  discard block
 block discarded – undo
310 310
 
311 311
                 if (!empty($row['guardians_full_name']) && ($row['guardians_date_of_birth_yyyy_mm_dd'] !== null)) {
312 312
                     $genderId = $row['guardians_gender_mf'] == 'M' ? 1 : 2;
313
-                    $AddressArea = Area_administrative::where('name', 'like', '%' . $row['guardians_address_area'] . '%')->first();
314
-                    $nationalityId = Nationality::where('name', 'like', '%' . $row['guardians_nationality'] . '%')->first();
315
-                    $identityType = Identity_type::where('national_code', 'like', '%' . $row['guardians_identity_type'] . '%')->first();
313
+                    $AddressArea = Area_administrative::where('name', 'like', '%'.$row['guardians_address_area'].'%')->first();
314
+                    $nationalityId = Nationality::where('name', 'like', '%'.$row['guardians_nationality'].'%')->first();
315
+                    $identityType = Identity_type::where('national_code', 'like', '%'.$row['guardians_identity_type'].'%')->first();
316 316
                     $openemisGuardian = $this->uniqueUid::getUniqueAlphanumeric();
317 317
 
318 318
                     $identityType = $identityType !== null ? $identityType->id : null;
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
327 327
 
328 328
                     if ($guardian === null) {
329 329
                         $guardian = Security_user::create([
330
-                                    'username' => str_replace('-','',$openemisGuardian),
330
+                                    'username' => str_replace('-', '', $openemisGuardian),
331 331
                                     'openemis_no' => $openemisGuardian,
332 332
                                     'first_name' => $row['guardians_full_name'], // here we save full name in the column of first name. re reduce breaks of the system.
333 333
                                     'last_name' => genNameWithInitials($row['guardians_full_name']),
@@ -346,22 +346,22 @@  discard block
 block discarded – undo
346 346
                         $guardian['guardian_relation_id'] = 3;
347 347
                         if (array_key_exists('guardians_phone', $row)) {
348 348
                             $guardian['contact'] = $row['guardians_phone'];
349
-                            User_contact::createOrUpdate($guardian,$this->file['security_user_id']);
349
+                            User_contact::createOrUpdate($guardian, $this->file['security_user_id']);
350 350
                         }  
351 351
                         Student_guardian::createStudentGuardian($student, $guardian, $this->file['security_user_id']);
352
-                    } else {
352
+                    }else {
353 353
                         Security_user::where('id', '=', $guardian->id)
354 354
                                 ->update(['is_guardian' => 1]);
355 355
                         $guardian['guardian_relation_id'] = 3;
356 356
                         if (array_key_exists('guardians_phone', $row)) {
357 357
                             $guardian['contact'] = $row['guardians_phone'];
358
-                            User_contact::createOrUpdate($guardian,$this->file['security_user_id']);
358
+                            User_contact::createOrUpdate($guardian, $this->file['security_user_id']);
359 359
                         } 
360 360
                         Student_guardian::createStudentGuardian($student, $guardian, $this->file['security_user_id']);
361 361
                     }
362 362
                 }
363 363
 
364
-                $optionalSubjects =  Institution_class_subject::getStudentOptionalSubject($subjects, $student, $row, $institution);
364
+                $optionalSubjects = Institution_class_subject::getStudentOptionalSubject($subjects, $student, $row, $institution);
365 365
 
366 366
                 $allSubjects = array_merge_recursive($optionalSubjects, $mandatorySubject);
367 367
                 // $stundetSubjects = $this->getStudentSubjects($student);
@@ -370,10 +370,10 @@  discard block
 block discarded – undo
370 370
                 if (!empty($allSubjects)) {
371 371
                     $allSubjects = unique_multidim_array($allSubjects, 'institution_subject_id');
372 372
                     $this->student = $student;
373
-                    $allSubjects = array_map(array($this,'setStudentSubjects'),$allSubjects);
373
+                    $allSubjects = array_map(array($this, 'setStudentSubjects'), $allSubjects);
374 374
                     // $allSubjects = array_unique($allSubjects,SORT_REGULAR);
375 375
                     $allSubjects = unique_multidim_array($allSubjects, 'education_subject_id');
376
-                    array_walk($allSubjects,array($this,'insertSubject'));
376
+                    array_walk($allSubjects, array($this, 'insertSubject'));
377 377
                     array_walk($allSubjects, array($this, 'updateSubjectCount'));
378 378
                 }
379 379
 
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
                             'total_male_students' => $totalStudents['total_male_students'],
387 387
                             'total_female_students' => $totalStudents['total_female_students']]);
388 388
             }
389
-        } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
389
+        }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
390 390
             $error = \Illuminate\Validation\ValidationException::withMessages([]);
391 391
             $failures = $e->failures();
392 392
             throw new \Maatwebsite\Excel\Validators\ValidationException($error, $failures);
Please login to merge, or discard this patch.