Test Setup Failed
Pull Request — master (#651)
by Mohamed
08:18
created
app/Mail/EmptyFile.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -25,16 +25,16 @@  discard block
 block discarded – undo
25 25
 
26 26
 
27 27
         $this->user = User::find($file['security_user_id']);
28
-        $this->subject = 'SIS Bulk Upload: ' .$file['subject'].' Empty ' . $institution->institution->code.': '. $institution->name.' '. date('Y:m:d H:i:s');
28
+        $this->subject = 'SIS Bulk Upload: '.$file['subject'].' Empty '.$institution->institution->code.': '.$institution->name.' '.date('Y:m:d H:i:s');
29 29
         $this->from_address = env('MAIL_FROM_ADDRESS');
30 30
         $this->from_name = 'SIS Bulk Uploader';
31 31
         $this->with = [
32 32
             'name' => $this->user->first_name,
33
-            'link' => \App::environment('local') || \App::environment('stage')   ?  env('APP_URL') : env('APP_URL').'/bulk-upload/'
33
+            'link' => \App::environment('local') || \App::environment('stage') ?  env('APP_URL') : env('APP_URL').'/bulk-upload/'
34 34
         ];
35 35
         $this->viewData = [
36
-            'name'=>$this->user->first_name, "body" => "No data Found in ". $file['filename']. ' Please re-upload the file with data',
37
-            'link' =>  \App::environment('local') || \App::environment('stage')   ?  env('APP_URL') : env('APP_URL').'/bulk-upload/'
36
+            'name'=>$this->user->first_name, "body" => "No data Found in ".$file['filename'].' Please re-upload the file with data',
37
+            'link' =>  \App::environment('local') || \App::environment('stage') ?  env('APP_URL') : env('APP_URL').'/bulk-upload/'
38 38
         ];
39 39
     }
40 40
 
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
     public function build()
47 47
     {
48 48
         return $this->view('emails.mail')
49
-            ->from($this->from_address,$this->from_name)
49
+            ->from($this->from_address, $this->from_name)
50 50
             ->to($this->user->email)
51 51
             ->subject($this->subject)
52 52
             ->with($this->with);
Please login to merge, or discard this patch.
app/Models/DashboardViews.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
             DbSchema::dropIfExists('students_count_view_table');
37 37
             DB::statement('CREATE TABLE students_count_view_table AS  (select * from students_count_view)');
38 38
             $output->writeln('creat : students_count_view');
39
-        } catch (\Throwable $th) {
39
+        }catch (\Throwable $th) {
40 40
             $output->writeln($th->getMessage());
41 41
         }
42 42
     }
@@ -103,17 +103,17 @@  discard block
 block discarded – undo
103 103
                 ->leftJoin("education_grades as eg", "eg.id", "i.id")
104 104
                 ->leftJoin("institution_class_students as ics", "ist.student_id", "ics.student_id")
105 105
                 ->leftJoin("institution_classes  as ic", "ic.id", "ics.institution_class_id")
106
-                ->leftJoin("student_guardians  as sgf", function ($join) {
106
+                ->leftJoin("student_guardians  as sgf", function($join) {
107 107
                     $join->on("sgf.student_id", "stu.id");
108 108
                     $join->where("sgf.guardian_relation_id", 1);
109 109
                 })
110 110
                 ->leftJoin("security_users  as sufu", "sgf.guardian_id", "sufu.id")
111
-                ->leftJoin("student_guardians  as sgm", function ($join) {
111
+                ->leftJoin("student_guardians  as sgm", function($join) {
112 112
                     $join->on("sgm.student_id", "stu.id");
113 113
                     $join->where("sgm.guardian_relation_id", 2);
114 114
                 })
115 115
                 ->leftJoin("security_users  as sumu", "sgm.guardian_id", "sumu.id")
116
-                ->leftJoin("student_guardians  as sg", function ($join) {
116
+                ->leftJoin("student_guardians  as sg", function($join) {
117 117
                     $join->on("sg.student_id", "stu.id");
118 118
                     $join->where("sg.guardian_relation_id", 3);
119 119
                 })
@@ -132,27 +132,27 @@  discard block
 block discarded – undo
132 132
                 ->leftJoin("academic_periods as acps", "acps.id", "ist.academic_period_id")
133 133
                 ->leftJoin("institution_subject_students as iss", "stu.id", "iss.student_id")
134 134
                 ->leftJoin("education_subjects as edus", "edus.id", "iss.education_subject_id")
135
-                ->leftJoin("nationalities as natif", function ($join) {
135
+                ->leftJoin("nationalities as natif", function($join) {
136 136
                     $join->on("sufu.nationality_id", "natif.id");
137 137
                     $join->where("sgf.guardian_relation_id", 1);
138 138
                 })
139
-                ->leftJoin("nationalities as natim", function ($join) {
139
+                ->leftJoin("nationalities as natim", function($join) {
140 140
                     $join->on("sumu.nationality_id", "natim.id");
141 141
                     $join->where("sgm.guardian_relation_id", 2);
142 142
                 })
143
-                ->leftJoin("nationalities as natig", function ($join) {
143
+                ->leftJoin("nationalities as natig", function($join) {
144 144
                     $join->on("sugu.nationality_id", "natig.id");
145 145
                     $join->where("sg.guardian_relation_id", 3);
146 146
                 })
147
-                ->leftJoin("identity_types as idtf", function ($join) {
147
+                ->leftJoin("identity_types as idtf", function($join) {
148 148
                     $join->on("sufu.nationality_id", "idtf.id");
149 149
                     $join->where("sgf.guardian_relation_id", 1);
150 150
                 })
151
-                ->leftJoin("identity_types as idtm", function ($join) {
151
+                ->leftJoin("identity_types as idtm", function($join) {
152 152
                     $join->on("sumu.nationality_id", "idtm.id");
153 153
                     $join->where("sgm.guardian_relation_id", 2);
154 154
                 })
155
-                ->leftJoin("identity_types as idtg", function ($join) {
155
+                ->leftJoin("identity_types as idtg", function($join) {
156 156
                     $join->on("sugu.nationality_id", "idtg.id");
157 157
                     $join->where("sg.guardian_relation_id", 3);
158 158
                 })
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
             DB::statement('CREATE INDEX user_institution ON students_list_view_table (institution_id);');
167 167
             $output->writeln('created : students_list_view_table');
168 168
             $output->writeln('created : students_list_view');
169
-        } catch (\Throwable $th) {
169
+        }catch (\Throwable $th) {
170 170
             $output->writeln($th->getMessage());
171 171
         }
172 172
     }
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
             Schema::createOrReplaceView('upload_list_view', $query);
227 227
             Schema::disableForeignKeyConstraints('upload_list_view');
228 228
             $output->writeln('created : upload_list_view');
229
-        } catch (\Throwable $th) {
229
+        }catch (\Throwable $th) {
230 230
             $output->writeln($th->getMessage());
231 231
         }
232 232
     }
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
             Schema::dropIfExists("upload_count_view");
262 262
             Schema::createOrReplaceView("upload_count_view", $query);
263 263
             $output->writeln('created : upload_count_view');
264
-        } catch (\Throwable $th) {
264
+        }catch (\Throwable $th) {
265 265
             $output->writeln($th->getMessage());
266 266
         }
267 267
     }
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
             Schema::dropIfExists("institution_info_view");
289 289
             Schema::createOrReplaceView("institution_info_view", $query);
290 290
             $output->writeln('created : institution_info_view');
291
-        } catch (\Throwable $th) {
291
+        }catch (\Throwable $th) {
292 292
             $output->writeln($th->getMessage());
293 293
         }
294 294
     }
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
             Schema::dropIfExists("students_count_by_grade_view");
327 327
             Schema::createOrReplaceView('students_count_by_grade_view', $query);
328 328
             $output->writeln('created : students_count_by_grade_view');
329
-        } catch (\Throwable $th) {
329
+        }catch (\Throwable $th) {
330 330
             $output->writeln($th->getMessage());
331 331
         }
332 332
     }
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
                     "ist.created"
352 352
                 )
353 353
                 ->join("institutions as i", "i.id", "ist.institution_id")
354
-                ->join('user_body_masses as ubm', function ($join) {
354
+                ->join('user_body_masses as ubm', function($join) {
355 355
                     $join->on('ubm.security_user_id', 'ist.student_id');
356 356
                     $join->where('ubm.academic_period_id', 'ist.academic_period_id');
357 357
                 })
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
             Schema::dropIfExists("students_count_by_bmi_view");
360 360
             Schema::createOrReplaceView("students_count_by_bmi_view", $query);
361 361
             $output->writeln('created : students_count_by_bmi_view');
362
-        } catch (\Throwable $th) {
362
+        }catch (\Throwable $th) {
363 363
             $output->writeln($th->getMessage());
364 364
         }
365 365
     }
@@ -392,7 +392,7 @@  discard block
 block discarded – undo
392 392
             Schema::dropIfExists("student_count_by_class_view");
393 393
             Schema::createOrReplaceView('student_count_by_class_view', $query);
394 394
             $output->writeln('created : student_count_by_class_view');
395
-        } catch (\Throwable $th) {
395
+        }catch (\Throwable $th) {
396 396
             $output->writeln($th->getMessage());
397 397
         }
398 398
     }
Please login to merge, or discard this patch.
app/Models/Institution_student_admission.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,13 +67,13 @@
 block discarded – undo
67 67
                 'institution_id' => $admissionInfo['instituion']->id,
68 68
                 'academic_period_id' => $admissionInfo['academic_period']->id,
69 69
                 'education_grade_id' => $admissionInfo['education_grade']->id,
70
-                'institution_class_id' => (($admissionInfo['instituion_class']  != []) && (count($admissionInfo['instituion_class'])==1)) ? $admissionInfo['instituion_class']['id'] : null,
70
+                'institution_class_id' => (($admissionInfo['instituion_class'] != []) && (count($admissionInfo['instituion_class']) == 1)) ? $admissionInfo['instituion_class']['id'] : null,
71 71
                 'comment' => 'Imported From Examination Data',
72 72
                 'updated_from' => 'doe',
73 73
                 'created_user_id' => 1
74 74
             ];
75 75
             self::create($data);
76
-        } catch (\Throwable $th) {
76
+        }catch (\Throwable $th) {
77 77
             Log::error($th);
78 78
         }
79 79
     }
Please login to merge, or discard this patch.
app/Notifications/ExportReady.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
      */
21 21
     public function __construct($user)
22 22
     {
23
-        $this->user  = $user;
23
+        $this->user = $user;
24 24
     }
25 25
 
26 26
     /**
Please login to merge, or discard this patch.
app/Console/Commands/UpdateStudentsCount.php 3 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -27,11 +27,11 @@
 block discarded – undo
27 27
      */
28 28
     protected $description = 'Update the students count male/female';
29 29
 
30
-      /**
31
-     * Create a new command instance.
32
-     *
33
-     * @return void
34
-     */
30
+        /**
31
+         * Create a new command instance.
32
+         *
33
+         * @return void
34
+         */
35 35
     public function __construct()
36 36
     {
37 37
         parent::__construct();
Please login to merge, or discard this patch.
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -48,36 +48,36 @@  discard block
 block discarded – undo
48 48
     public function handle()
49 49
     {
50 50
         $this->output->writeln('#########################################');
51
-        if($this->argument('entity') == 'class'){
51
+        if ($this->argument('entity') == 'class') {
52 52
             $classes = Institution_class::get()->toArray();
53
-            $func = array($this,'updateCount');
54
-            array_walk($subjects,array($this,'process'));
55
-            $this->output->writeln('start updating:'. count($classes));
56
-        }elseif($this->argument('entity') == 'subject'){
53
+            $func = array($this, 'updateCount');
54
+            array_walk($subjects, array($this, 'process'));
55
+            $this->output->writeln('start updating:'.count($classes));
56
+        }elseif ($this->argument('entity') == 'subject') {
57 57
             $subjects = Institution_subject::get()->toArray(); 
58
-            $subjects = array_chunk($subjects,10000);
59
-            $this->output->writeln('start updating:'. count($subjects));
60
-            array_walk($subjects,array($this,'process'));
58
+            $subjects = array_chunk($subjects, 10000);
59
+            $this->output->writeln('start updating:'.count($subjects));
60
+            array_walk($subjects, array($this, 'process'));
61 61
             $this->output->writeln('#########################################');
62 62
         }
63 63
        
64 64
     }
65 65
 
66
-    public function process($data){
67
-        if($this->argument('entity') == 'class'){
68
-            $func = array($this,'updateCount');
69
-            processParallel($func,$data,$this->argument('max'));
70
-            $this->output->writeln('start updating calss count:'. count($data));
71
-        }elseif($this->argument('entity') == 'subject'){
72
-            $this->output->writeln('start updating subject count:'. count($data));
73
-            $func_subject = array($this,'updateSubjectCount');
74
-            processParallel($func_subject,$data,$this->argument('max'));
66
+    public function process($data) {
67
+        if ($this->argument('entity') == 'class') {
68
+            $func = array($this, 'updateCount');
69
+            processParallel($func, $data, $this->argument('max'));
70
+            $this->output->writeln('start updating calss count:'.count($data));
71
+        }elseif ($this->argument('entity') == 'subject') {
72
+            $this->output->writeln('start updating subject count:'.count($data));
73
+            $func_subject = array($this, 'updateSubjectCount');
74
+            processParallel($func_subject, $data, $this->argument('max'));
75 75
         }
76 76
     }
77 77
 
78
-    public function updateCount($class){
79
-        $this->output->writeln('updating class:'. $class['id']);
80
-        $totalStudents =  Institution_class_student::getStudentsCount($class['id']);
78
+    public function updateCount($class) {
79
+        $this->output->writeln('updating class:'.$class['id']);
80
+        $totalStudents = Institution_class_student::getStudentsCount($class['id']);
81 81
         Institution_class::where('id', '=', $class['id'])
82 82
         ->update([
83 83
             'total_male_students' => $totalStudents['total_male_students'],
@@ -85,8 +85,8 @@  discard block
 block discarded – undo
85 85
         ]);
86 86
     }
87 87
 
88
-    public function updateSubjectCount($subject){
89
-        $this->output->writeln('updating subject:'. $subject['id']);
88
+    public function updateSubjectCount($subject) {
89
+        $this->output->writeln('updating subject:'.$subject['id']);
90 90
         $totalStudents = Institution_subject_student::getStudentsCount($subject['id']);
91 91
         Institution_subject::where(['id' => $subject['id']])
92 92
             ->update([
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
             $func = array($this,'updateCount');
54 54
             array_walk($subjects,array($this,'process'));
55 55
             $this->output->writeln('start updating:'. count($classes));
56
-        }elseif($this->argument('entity') == 'subject'){
56
+        } elseif($this->argument('entity') == 'subject'){
57 57
             $subjects = Institution_subject::get()->toArray(); 
58 58
             $subjects = array_chunk($subjects,10000);
59 59
             $this->output->writeln('start updating:'. count($subjects));
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
             $func = array($this,'updateCount');
69 69
             processParallel($func,$data,$this->argument('max'));
70 70
             $this->output->writeln('start updating calss count:'. count($data));
71
-        }elseif($this->argument('entity') == 'subject'){
71
+        } elseif($this->argument('entity') == 'subject'){
72 72
             $this->output->writeln('start updating subject count:'. count($data));
73 73
             $func_subject = array($this,'updateSubjectCount');
74 74
             processParallel($func_subject,$data,$this->argument('max'));
Please login to merge, or discard this patch.
app/Console/Commands/UpdateViews.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
      */
40 40
     public function handle()
41 41
     {
42
-       $views = new DashboardViewsController();
43
-       $views->callback();
42
+        $views = new DashboardViewsController();
43
+        $views->callback();
44 44
     }
45 45
 }
Please login to merge, or discard this patch.
app/Models/Security_user.php 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -152,12 +152,12 @@
 block discarded – undo
152 152
             ->get()->toArray();
153 153
     }
154 154
 
155
-     /**
156
-     * First level search for students
157
-     *
158
-     * @param array $student
159
-     * @return array
160
-     */
155
+        /**
156
+         * First level search for students
157
+         *
158
+         * @param array $student
159
+         * @return array
160
+         */
161 161
     public function getStudentCount($student)
162 162
     {
163 163
         return $this
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -149,9 +149,9 @@  discard block
 block discarded – undo
149 149
     public function getMatches($student)
150 150
     {
151 151
         return $this
152
-        ->where('gender_id',$student['gender'] + 1)
153
-        ->where('institutions.code',$student['schoolid'])
154
-        ->where('date_of_birth',$student['b_date'])
152
+        ->where('gender_id', $student['gender'] + 1)
153
+        ->where('institutions.code', $student['schoolid'])
154
+        ->where('date_of_birth', $student['b_date'])
155 155
             ->join('institution_students', 'security_users.id', 'institution_students.student_id')
156 156
             ->join('institutions', 'institution_students.institution_id', 'institutions.id')
157 157
             ->get()->toArray();
@@ -166,9 +166,9 @@  discard block
 block discarded – undo
166 166
     public function getStudentCount($student)
167 167
     {
168 168
         return $this
169
-        ->where('gender_id',$student['gender'] + 1)
170
-        ->where('institutions.code',$student['schoolid'])
171
-        ->where('date_of_birth',$student['b_date'])
169
+        ->where('gender_id', $student['gender'] + 1)
170
+        ->where('institutions.code', $student['schoolid'])
171
+        ->where('date_of_birth', $student['b_date'])
172 172
             ->join('institution_students', 'security_users.id', 'institution_students.student_id')
173 173
             ->join('institutions', 'institution_students.institution_id', 'institutions.id')
174 174
             ->count();
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
             $studentData['id'] = $id;
203 203
             $this->uniqueUserId->updateOrInsertRecord($studentData);
204 204
             return $studentData;
205
-        } catch (\Exception $th) {
205
+        }catch (\Exception $th) {
206 206
             Log::error($th->getMessage());
207 207
             // in case of duplication of the Unique ID this will recursive.
208 208
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
         $this->uniqueUserId = new Unique_user_id();
224 224
         $this->uniqueUId = new UniqueUid();
225 225
         // regenerate unique id if it's not available
226
-        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'],9)) ?  $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
226
+        $uniqueId = ($this->uniqueUId::isValidUniqueId($sis_student['openemis_no'], 9)) ? $sis_student['openemis_no'] : $this->uniqueUId::getUniqueAlphanumeric();
227 227
 
228 228
         $studentData = [
229 229
             'username' => str_replace('-', '', $uniqueId),
@@ -237,11 +237,11 @@  discard block
 block discarded – undo
237 237
         ];
238 238
 
239 239
         try {
240
-            self::where( 'id'  , $sis_student['student_id'])->update($studentData);
240
+            self::where('id', $sis_student['student_id'])->update($studentData);
241 241
             $studentData['id'] = $sis_student['student_id'];
242 242
             $this->uniqueUserId->updateOrInsertRecord($studentData);
243 243
             return $studentData;
244
-        } catch (\Exception $th) {
244
+        }catch (\Exception $th) {
245 245
             Log::error($th);
246 246
             // in case of duplication of the Unique ID this will recursive.
247 247
             $sis_student['openemis_no'] = $this->uniqueUId::getUniqueAlphanumeric();
Please login to merge, or discard this patch.
app/Console/Commands/ExaminationStudentMigrate.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
             $output->writeln('###########################################------starting export------###########################################');
48 48
             $this->examinationController->export();
49 49
             $output->writeln('###########################################------Finished inserting file records------###########################################');
50
-        } else {
50
+        }else {
51 51
             $this->examinationController->doMatch($this->argument('offset'), $this->argument('limit'), $this->argument(('mode')));
52 52
             $this->examinationController->export();
53 53
         }
Please login to merge, or discard this patch.
app/Exports/ExaminationStudentsExport.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,8 +37,8 @@
 block discarded – undo
37 37
     }
38 38
     
39 39
     /**
40
-    * @return \Illuminate\Support\Collection
41
-    */
40
+     * @return \Illuminate\Support\Collection
41
+     */
42 42
     public function query()
43 43
     {
44 44
         return Examination_student::query()->whereNotNull('nsid')
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
 use Maatwebsite\Excel\Concerns\WithHeadings;
11 11
 use Carbon\Carbon;
12 12
 
13
-class ExaminationStudentsExport implements FromQuery , WithHeadings 
13
+class ExaminationStudentsExport implements FromQuery, WithHeadings 
14 14
 {
15 15
 
16 16
     use Exportable;
@@ -44,6 +44,6 @@  discard block
 block discarded – undo
44 44
     {
45 45
         return Examination_student::query()->whereNotNull('nsid')
46 46
         ->distinct('nsid')
47
-        ->whereDate('updated_at','>=',Carbon::today());
47
+        ->whereDate('updated_at', '>=', Carbon::today());
48 48
     }
49 49
 }
Please login to merge, or discard this patch.