moe-lk / bulk-upload

app/Models/Education_programmes_next_programme.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 use Illuminate\Database\Eloquent\Model;
 
 
-class Education_programmes_next_programme extends Model  {
+class Education_programmes_next_programme extends Model {
 
     /**
      * The database table used by the model.

app/Mail/StudentImportSuccess.php

Indentation +2 added lines, -2 removed lines

@@ -45,8 +45,8 @@
     public function build()
     {
         return $this->view('emails.mail')
-             ->from($this->from_address,$this->from_name)
-             ->to($this->user->email)
+                ->from($this->from_address,$this->from_name)
+                ->to($this->user->email)
             ->subject($this->subject)
             ->with($this->with);
     }

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
         $institution = Institution_class::find($file['institution_class_id']);
 
         $this->user = User::find($file['security_user_id']);
-        $this->subject = 'SIS Bulk upload: '. $file['subject'] .$institution->institution->code.': '. $institution->name.' ' . date('Y:m:d H:i:s');
+        $this->subject = 'SIS Bulk upload: '.$file['subject'].$institution->institution->code.': '.$institution->name.' '.date('Y:m:d H:i:s');
         $this->from_address = env('MAIL_FROM_ADDRESS');
         $this->from_name = 'SIS Bulk Uploader';
         $this->with = [
@@ -45,7 +45,7 @@ 
     public function build()
     {
         return $this->view('emails.mail')
-             ->from($this->from_address,$this->from_name)
+             ->from($this->from_address, $this->from_name)
              ->to($this->user->email)
             ->subject($this->subject)
             ->with($this->with);

app/Mail/IncorrectTemplate.php

Indentation +5 added lines, -5 removed lines

@@ -11,11 +11,11 @@
 {
     use Queueable, SerializesModels;
 
-     /**
-     * Create a new message instance.
-     *
-     * @return void
-     */
+        /**
+         * Create a new message instance.
+         *
+         * @return void
+         */
     public function __construct($file)
     {
 

Spacing +4 added lines, -4 removed lines

@@ -23,16 +23,16 @@ 
 
 
         $this->user = User::find($file['security_user_id']);
-        $this->subject = 'SIS Bulk Upload: IncorrectTemplate'.$institution->institution->code.': '. $institution->name.' '. date('Y:m:d H:i:s');
+        $this->subject = 'SIS Bulk Upload: IncorrectTemplate'.$institution->institution->code.': '.$institution->name.' '.date('Y:m:d H:i:s');
         $this->from_address = env('MAIL_FROM_ADDRESS');
         $this->from_name = 'SIS Bulk Uploader';
         $this->with = [
             'name' => $this->user->first_name,
-            'link' => \App::environment('local') || \App::environment('stage')  ?  env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']
+            'link' => \App::environment('local') || \App::environment('stage') ?  env('APP_URL').'/download/'.$file['filename'] : env('APP_URL').'/bulk-upload/download/'.$file['filename']
         ];
         $this->viewData = [
             'name'=>$this->user->first_name, "body" => "The Template you used for upload having some issues, please reconfirm and re-upload with correct template",
-            'link' =>  \App::environment('local') || \App::environment('stage')   ?  env('APP_URL').'/download/' .$file['filename'] : env('APP_URL').'/bulk-upload/download/' .$file['filename']
+            'link' =>  \App::environment('local') || \App::environment('stage') ?  env('APP_URL').'/download/'.$file['filename'] : env('APP_URL').'/bulk-upload/download/'.$file['filename']
         ];
     }
 
@@ -44,7 +44,7 @@ 
     public function build()
     {
         return $this->view('emails.mail')
-            ->from($this->from_address,$this->from_name)
+            ->from($this->from_address, $this->from_name)
             ->to($this->user->email)
             ->subject($this->subject)
             ->with($this->with);

app/helpers.php

Indentation +2 added lines, -2 removed lines

@@ -23,9 +23,9 @@
 }
 
 function clean($string) {
-   $string = str_replace(' ', '-', $string); // Replaces all spaces with hyphens.
+    $string = str_replace(' ', '-', $string); // Replaces all spaces with hyphens.
 
-   return preg_replace('/[^A-Za-z0-9\-]/', '', $string); // Removes special chars.
+    return preg_replace('/[^A-Za-z0-9\-]/', '', $string); // Removes special chars.
 }
 
 

Spacing +50 added lines, -50 removed lines

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

Braces +10 added lines, -5 removed lines

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

app/Libraries/ShaHash/SHAHasher.php

Spacing +3 added lines, -3 removed lines

@@ -13,13 +13,13 @@ 
 
     public function check($value, $hashedValue, array $options = [])
     {
-        return password_verify ($value , $hashedValue);
+        return password_verify($value, $hashedValue);
     }
 
 
     public function make($value, array $options = [])
     {
-        return password_hash($value,PASSWORD_DEFAULT,$options);
+        return password_hash($value, PASSWORD_DEFAULT, $options);
     }
 
 
@@ -27,7 +27,7 @@ 
 
     public function needsRehash($hashedValue, array $options = [])
     {
-        return password_needs_rehash($hashedValue,$options);
+        return password_needs_rehash($hashedValue, $options);
     }
 
 

app/Models/Examination_student.php

Spacing +2 added lines, -2 removed lines

@@ -52,9 +52,9 @@
         $value = self::where('st_no', $data['st_no'])->get();
         if (count($value) > 0) {
             self::where('st_no', $data['st_no'])->update($data);
-        } else {
+        }else {
             self::insert($data);
         }
-        $output->writeln('Student name: ' . ($data['f_name']) . ' has been inserted to the database');
+        $output->writeln('Student name: '.($data['f_name']).' has been inserted to the database');
     }
 }

app/Http/Middleware/classTeacher.php

Spacing +4 added lines, -4 removed lines

@@ -15,13 +15,13 @@
      */
     public function handle($request, Closure $next)
     {
-        if($request->user()->super_admin == 1){
+        if ($request->user()->super_admin == 1) {
             return $next($request);
-        }elseif ($request->user() && (!($request->user()->permissions->isEmpty()))  && $request->user()->permissions[0]->roles &&  $request->user()->permissions[0]->roles->code === 'HOMEROOM_TEACHER') {
+        }elseif ($request->user() && (!($request->user()->permissions->isEmpty())) && $request->user()->permissions[0]->roles && $request->user()->permissions[0]->roles->code === 'HOMEROOM_TEACHER') {
             return $next($request);
-        }elseif($request->user() && (!($request->user()->principal->isEmpty()))  && $request->user()->principal[0]->roles &&  $request->user()->principal[0]->roles->code === 'PRINCIPAL'){
+        }elseif ($request->user() && (!($request->user()->principal->isEmpty())) && $request->user()->principal[0]->roles && $request->user()->principal[0]->roles->code === 'PRINCIPAL') {
             return $next($request);
-        }elseif($request->user() && (!($request->user()->zonal_cordinator->isEmpty()))  && $request->user()->zonal_cordinator[0]->roles &&  $request->user()->zonal_cordinator[0]->roles->code === 'PRINCIPAL'){
+        }elseif ($request->user() && (!($request->user()->zonal_cordinator->isEmpty())) && $request->user()->zonal_cordinator[0]->roles && $request->user()->zonal_cordinator[0]->roles->code === 'PRINCIPAL') {
             return $next($request);
         }
         return redirect('/login')->with('status', 'Your dont have access for upload data. Please get assign your to the class and try');

Braces +3 added lines, -3 removed lines

@@ -17,11 +17,11 @@
     {
         if($request->user()->super_admin == 1){
             return $next($request);
-        }elseif ($request->user() && (!($request->user()->permissions->isEmpty()))  && $request->user()->permissions[0]->roles &&  $request->user()->permissions[0]->roles->code === 'HOMEROOM_TEACHER') {
+        } elseif ($request->user() && (!($request->user()->permissions->isEmpty()))  && $request->user()->permissions[0]->roles &&  $request->user()->permissions[0]->roles->code === 'HOMEROOM_TEACHER') {
             return $next($request);
-        }elseif($request->user() && (!($request->user()->principal->isEmpty()))  && $request->user()->principal[0]->roles &&  $request->user()->principal[0]->roles->code === 'PRINCIPAL'){
+        } elseif($request->user() && (!($request->user()->principal->isEmpty()))  && $request->user()->principal[0]->roles &&  $request->user()->principal[0]->roles->code === 'PRINCIPAL'){
             return $next($request);
-        }elseif($request->user() && (!($request->user()->zonal_cordinator->isEmpty()))  && $request->user()->zonal_cordinator[0]->roles &&  $request->user()->zonal_cordinator[0]->roles->code === 'PRINCIPAL'){
+        } elseif($request->user() && (!($request->user()->zonal_cordinator->isEmpty()))  && $request->user()->zonal_cordinator[0]->roles &&  $request->user()->zonal_cordinator[0]->roles->code === 'PRINCIPAL'){
             return $next($request);
         }
         return redirect('/login')->with('status', 'Your dont have access for upload data. Please get assign your to the class and try');

app/Http/Controllers/ImportExport.php

Indentation +8 added lines, -8 removed lines

@@ -29,9 +29,9 @@ 
 
     }
 
-     /**
-    * @return \Illuminate\Support\Collection
-    */
+        /**
+         * @return \Illuminate\Support\Collection
+         */
     public function importExportView()
     {
         if(Auth::user()->super_admin ){
@@ -45,11 +45,11 @@ 
 
    
     /**
-    * @return \Illuminate\Support\Collection
-    */
+     * @return \Illuminate\Support\Collection
+     */
     public function export(Request $request) 
     {
-         $request->validate([
+            $request->validate([
                 'class' => 'required'
             ]);
         return Excel::download(new UsersExport($request->input('class')), 'users.xlsx');
@@ -58,8 +58,8 @@ 
 
 
     /**
-    * @return \Illuminate\Support\Collection
-    */
+     * @return \Illuminate\Support\Collection
+     */
     public function import(Request $request)
     {
 

Spacing +6 added lines, -6 removed lines

@@ -34,11 +34,11 @@ 
     */
     public function importExportView()
     {
-        if(Auth::user()->super_admin ){
+        if (Auth::user()->super_admin) {
             return view('uploadcsv');
-        }else{
-            $classes = (!Auth::user()->permissions->isEmpty())  ?  Auth::user()->permissions[0]->staff_class : Auth::user()->principal[0]->security_group_institution->institution_classes;
-            return view('importExport')->with('classes',$classes);
+        }else {
+            $classes = (!Auth::user()->permissions->isEmpty()) ?  Auth::user()->permissions[0]->staff_class : Auth::user()->principal[0]->security_group_institution->institution_classes;
+            return view('importExport')->with('classes', $classes);
         }
     }
 
@@ -73,9 +73,9 @@ 
 
 
             $import = new UsersImport();
-            try{
+            try {
                 $files = Storage::disk('sis-bulk-data-files')->allFiles();
-                Excel::import($import,request()->file('import_file'));
+                Excel::import($import, request()->file('import_file'));
             }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
                 $failures = $e->failures();
 

Braces +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
     {
         if(Auth::user()->super_admin ){
             return view('uploadcsv');
-        }else{
+        } else{
             $classes = (!Auth::user()->permissions->isEmpty())  ?  Auth::user()->permissions[0]->staff_class : Auth::user()->principal[0]->security_group_institution->institution_classes;
             return view('importExport')->with('classes',$classes);
         }
@@ -76,7 +76,7 @@ 
             try{
                 $files = Storage::disk('sis-bulk-data-files')->allFiles();
                 Excel::import($import,request()->file('import_file'));
-            }catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
+            } catch (\Maatwebsite\Excel\Validators\ValidationException $e) {
                 $failures = $e->failures();
 
                 foreach ($failures as $failure) {

app/Http/Controllers/FileController.php

Braces +4 added lines, -7 removed lines

@@ -99,7 +99,7 @@ 
             DB::table('uploads')
                 ->where('id', $id)
                 ->update(['is_processed' => 0]);
-        }elseif ($action == 200) {
+        } elseif ($action == 200) {
             DB::table('uploads')
                 ->where('id', $id)
                 ->update(['is_processed' => 4]);
@@ -116,8 +116,7 @@ 
             return Response::download($file_path, Auth::user()->openemis_no.'_'.$filename.$version, [
                 'Content-Length: '. filesize($file_path)
             ]);
-        }
-        else
+        } else
         {
             return response()->view('errors.404');
         }
@@ -136,8 +135,7 @@ 
             return Response::download($file_path, $filename, [
                 'Content-Length: '. filesize($file_path)
             ]);
-        }
-        else
+        } else
         {
             abort(404, 'We did not found an error file.');
         }
@@ -151,8 +149,7 @@ 
             return Response::download($file_path, $filename, [
                 'Content-Length: '. filesize($file_path)
             ]);
-        }
-        else
+        } else
         {
 
             abort(404, 'We did not found an error file.');

Spacing +14 added lines, -14 removed lines

@@ -33,7 +33,7 @@ 
      * @param Request $request
      * @return \Illuminate\Http\JsonResponse
      */
-    public function upload(Request $request){
+    public function upload(Request $request) {
 
 
 
@@ -75,7 +75,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,
@@ -83,7 +83,7 @@ 
         );
 
         $upload = new Upload;
-        $upload->fileName =$fileName;
+        $upload->fileName = $fileName;
         $upload->model = 'Student';
         $upload->node = 'None';
         $upload->institution_class_id = $class->id;
@@ -94,8 +94,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]);
@@ -107,14 +107,14 @@ 
     }
 
 
-    public function downloadTemplate(){
+    public function downloadTemplate() {
         $filename = 'censusNo_className_sis_students_bulk_upload';
         $version = '2007_V2.0.2_20201211.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
@@ -128,13 +128,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
@@ -144,12 +144,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