academico-sis / academico

app/Services/MailerliteService.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $subscriberGroups = $this->subscribersApi->getGroups($email); // returns array of group objects subscriber belongs to
 
         foreach ($subscriberGroups as $group) {
-            $groupId = $array = json_decode(json_encode($group),true);
+            $groupId = $array = json_decode(json_encode($group), true);
             $this->groupsApi->removeSubscriber($groupId['id'], $email); // returns empty response
         }
 

app/Console/Kernel.php

Spacing +7 added lines, -7 removed lines

@@ -34,12 +34,12 @@ 
      */
     protected function schedule(Schedule $schedule)
     {
-        $schedule->call(function () {
+        $schedule->call(function() {
             Log::info('Sending attendance reminders');
             $this->remindPendingAttendance();
         })->dailyAt('08:15');
 
-        $schedule->call(function () {
+        $schedule->call(function() {
             Log::info('Checking default periods');
 
             // when we finish the current period; remove the manual override to automatically fallbackto the next one
@@ -57,14 +57,14 @@ 
         })->dailyAt('00:00');
 
         if (config('settings.partnership_alerts')) {
-            $schedule->call(function () {
+            $schedule->call(function() {
                 Log::info('Checking expired partnerships');
 
                 // if one of the partnerships is expiring soon, send an email
-                $partners = Partner::where(function ($query) {
+                $partners = Partner::where(function($query) {
                     $query->whereNotNull('expired_on')->where('expired_on', '<', Carbon::now()->addDays(28));
                 })
-                    ->where(function ($query) {
+                    ->where(function($query) {
                         $query->whereNull('last_alert_sent_at')->orWhere('last_alert_sent_at', '>', Carbon::now()->subDays(28))->get();
                     });
 
@@ -75,14 +75,14 @@ 
         }
 
         if (config('settings.external_courses_report')) {
-            $schedule->call(function () {
+            $schedule->call(function() {
                 Log::info('Sending external courses reports');
                 event(new ExternalCoursesReportEvent());
             })->dailyAt('02:10');
         }
 
         if (config('settings.monthly_report')) {
-            $schedule->call(function () {
+            $schedule->call(function() {
                 Log::info('Sending monthly hours report');
                 event(new MonthlyReportEvent());
             })->monthlyOn(20);

app/Console/Commands/migrateGrades.php

Spacing +5 added lines, -5 removed lines

@@ -45,14 +45,14 @@ 
             $course = Course::find($entry->course_id);
             $matchFound = false;
 
-            echo "\nMigrating grades from course " . $course->id;
+            echo "\nMigrating grades from course ".$course->id;
 
             // collect all grade types associated to this course.
             $courseGradeTypes = DB::table('course_grade_type')->where('course_id', $entry->course_id)->pluck('grade_type_id');
             // if this configuration matches an evaluation type, use it, otherwise create it
             foreach (EvaluationType::has('gradeTypes')->get() as $evaluationType) {
                 if ($courseGradeTypes->diff($evaluationType->gradeTypes->pluck('id'))->count() == 0) {
-                    echo "match with eval type " . $evaluationType->id;
+                    echo "match with eval type ".$evaluationType->id;
                     $course->update(['evaluation_type_id' => $evaluationType->id]);
                     $matchFound = true;
                 }
@@ -61,7 +61,7 @@ 
             // if no match has been found
             if (!$matchFound) {
                 echo "no match; creating new eval type";
-                $evalType = EvaluationType::create(['name' => 'cours ' . $entry->course_id]);
+                $evalType = EvaluationType::create(['name' => 'cours '.$entry->course_id]);
                 $evalType->gradeTypes()->sync($courseGradeTypes);
             }
 
@@ -74,7 +74,7 @@ 
             $course = Course::find($entry->course_id);
             $matchFound = false;
 
-            echo "\nMigrating skills from course " . $course->id;
+            echo "\nMigrating skills from course ".$course->id;
 
             // collect all grade types associated to this course.
             $courseSkills = DB::table('course_skill')->where('course_id', $entry->course_id)->pluck('skill_id');
@@ -90,7 +90,7 @@ 
             // if no match has been found
             if (!$matchFound) {
                 $levelName = isset($course->level) ? $course->level->name : $course->id; 
-                $evalType = EvaluationType::create(['name' => 'Niveau ' . $levelName]);
+                $evalType = EvaluationType::create(['name' => 'Niveau '.$levelName]);
                 $evalType->skills()->sync($courseSkills);
             }
         }

app/Console/Commands/importData.php

Spacing +2 added lines, -2 removed lines

@@ -37,13 +37,13 @@
             $period = Period::firstWhere('name', $record['year']);
 
             // Retrieve or create the student
-            $email = $record['email'] !== "" ? $record['email'] : $this->stripAccents($record['firstname']) . "." . $this->stripAccents($record['name']) . "@academico.afsantiago.es";
+            $email = $record['email'] !== "" ? $record['email'] : $this->stripAccents($record['firstname']).".".$this->stripAccents($record['name'])."@academico.afsantiago.es";
 
             if (User::where('email', $email)->count() > 0) {
                 $user = User::where('email', $email)->first();
                 if (($user->firstname !== trim($record['firstname']) || $user->lastname !== trim($record['name'])))
                 {
-                    $email = $this->stripAccents($record['firstname']) . "." . $this->stripAccents($record['name']) . "@academico.afsantiago.es";
+                    $email = $this->stripAccents($record['firstname']).".".$this->stripAccents($record['name'])."@academico.afsantiago.es";
                 }
             }
 

app/Models/Fee.php

Spacing +2 added lines, -2 removed lines

@@ -73,10 +73,10 @@
     {
         if (config('app.currency_position') === 'before')
         {
-            return config('app.currency_symbol') . " ". $this->price;
+            return config('app.currency_symbol')." ".$this->price;
         }
 
-        return $this->price . " " . config('app.currency_symbol');
+        return $this->price." ".config('app.currency_symbol');
     }
 
     public function getTypeAttribute()

app/Http/Controllers/Admin/ScheduledPaymentCrudController.php

Spacing +12 added lines, -12 removed lines

@@ -30,7 +30,7 @@ 
     public function setup()
     {
         CRUD::setModel(\App\Models\ScheduledPayment::class);
-        CRUD::setRoute(config('backpack.base.route_prefix') . '/scheduled-payment');
+        CRUD::setRoute(config('backpack.base.route_prefix').'/scheduled-payment');
         CRUD::setEntityNameStrings(__('scheduled payment'), __('scheduled payments'));
         CRUD::enableExportButtons();
     }
@@ -49,9 +49,9 @@ 
             'attribute' => 'lastname',
             'label' => __('Last Name'),
             'type' => 'relationship',
-            'searchLogic' => function ($query, $column, $searchTerm) {
-                $query->orWhereHas('enrollment', function ($q) use ($searchTerm) {
-                    $q->whereHas('user', function ($q) use ($searchTerm) {
+            'searchLogic' => function($query, $column, $searchTerm) {
+                $query->orWhereHas('enrollment', function($q) use ($searchTerm) {
+                    $q->whereHas('user', function($q) use ($searchTerm) {
                         $q->where('lastname', 'like', '%'.$searchTerm.'%');
                     });
                 });
@@ -64,9 +64,9 @@ 
             'attribute' => 'firstname',
             'label' => __('First Name'),
             'type' => 'relationship',
-            'searchLogic' => function ($query, $column, $searchTerm) {
-                $query->orWhereHas('enrollment', function ($q) use ($searchTerm) {
-                    $q->whereHas('user', function ($q) use ($searchTerm) {
+            'searchLogic' => function($query, $column, $searchTerm) {
+                $query->orWhereHas('enrollment', function($q) use ($searchTerm) {
+                    $q->whereHas('user', function($q) use ($searchTerm) {
                         $q->where('firstname', 'like', '%'.$searchTerm.'%');
                     });
                 });
@@ -80,9 +80,9 @@ 
             'attribute' => 'email',
             'label' => __('Email'),
             'type' => 'relationship',
-            'searchLogic' => function ($query, $column, $searchTerm) {
-                $query->orWhereHas('enrollment', function ($q) use ($searchTerm) {
-                    $q->whereHas('user', function ($q) use ($searchTerm) {
+            'searchLogic' => function($query, $column, $searchTerm) {
+                $query->orWhereHas('enrollment', function($q) use ($searchTerm) {
+                    $q->whereHas('user', function($q) use ($searchTerm) {
                         $q->where('email', 'like', '%'.$searchTerm.'%');
                     });
                 });
@@ -112,13 +112,13 @@ 
             'name' => 'status_id',
             'type' => 'select2',
             'label'=> __('Status'),
-        ], function () {
+        ], function() {
             return [
                 1 => __('Pending'),
                 2 => __('Paid'),
             ];
         },
-            function ($value) { // if the filter is active
+            function($value) { // if the filter is active
                 CRUD::addClause('status', $value);
             });
     }

app/Models/ScheduledPayment.php

Spacing +4 added lines, -4 removed lines

@@ -62,7 +62,7 @@ 
 
     public function scopeStatus(Builder $query, $status)
     {
-        return match ($status) {
+        return match($status) {
             "2" => $query->where('status', 2)->orWhereHas('invoices'),
             "1" => $query->where('status', 1)->orWhereDoesntHave('invoices'),
             default => $query,
@@ -111,10 +111,10 @@ 
     {
         if (config('app.currency_position') === 'before')
         {
-            return config('app.currency_symbol') . " ". $this->value;
+            return config('app.currency_symbol')." ".$this->value;
         }
 
-        return $this->value . " " . config('app.currency_symbol');
+        return $this->value." ".config('app.currency_symbol');
     }
 
     function getDateForHumansAttribute()
@@ -140,7 +140,7 @@ 
 
     public function identifiableAttribute()
     {
-        return $this->date . " (" . $this->value_with_currency . ")";
+        return $this->date." (".$this->value_with_currency.")";
     }
 
     public function getStatusTypeNameAttribute()

app/Models/Course.php

Spacing +11 added lines, -11 removed lines

@@ -309,7 +309,7 @@ 
      */
     public function skills()
     {
-        return $this->evaluationType?->skills()->orderBy('order');
+        return $this->evaluationType ? ->skills()->orderBy('order');
     }
 
     public function books()
@@ -334,7 +334,7 @@ 
      */
     public function getPendingAttendanceAttribute()
     {
-        $events = Event::where(function ($query) {
+        $events = Event::where(function($query) {
             $query->where('course_id', $this->id);
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
@@ -471,7 +471,7 @@ 
             foreach ($courseTimes as $courseTime) {
                 $initial = $daysInitials[$courseTime->day];
 
-                if (! isset($parsedCourseTimes[$initial])) {
+                if (!isset($parsedCourseTimes[$initial])) {
                     $parsedCourseTimes[$initial] = [];
                 }
 
@@ -507,12 +507,12 @@ 
         }
 
         // handle for case Mon - Friday 9:00am - 5:00pm but on the Wed 10:00am - 2:00pm, so will split up like so [Mon - Tue 9:00am - 5:00pm | Wed 10:00am -2:00pm | Thur - Friday 9:00am - 5:00pm]
-        $result = collect($parsed)->map(function ($value) {
+        $result = collect($parsed)->map(function($value) {
             $value = collect($value);
             $day = $value->first()['day'];
             $times = $value->first()['time'];
             if ($value->count() > 1) {
-                $day .= ' - ' . $value->last()['day'];
+                $day .= ' - '.$value->last()['day'];
             }
 
             return "$day $times";
@@ -548,7 +548,7 @@ 
     public function getCourseTeacherNameAttribute()
     {
         if ($this->teacher_id) {
-            return $this->teacher?->name;
+            return $this->teacher ? ->name;
         } else {
             return '-';
         }
@@ -561,7 +561,7 @@ 
 
     public function getDescriptionAttribute()
     {
-        return '[' . $this->course_period_name . '] - ' . $this->name;
+        return '['.$this->course_period_name.'] - '.$this->name;
     }
 
     public function getChildrenCountAttribute()
@@ -582,7 +582,7 @@ 
 
     public function getAcceptsNewStudentsAttribute(): bool
     {
-        if (! $this->spots || $this->spots == 0) {
+        if (!$this->spots || $this->spots == 0) {
             return true;
         }
 
@@ -605,7 +605,7 @@ 
 
     public function eventsWithExpectedAttendance()
     {
-        return $this->events()->where(function ($query) {
+        return $this->events()->where(function($query) {
             $query->where('exempt_attendance', '!=', true);
             $query->where('exempt_attendance', '!=', 1);
             $query->orWhereNull('exempt_attendance');
@@ -634,10 +634,10 @@ 
     {
         if (config('app.currency_position') === 'before')
         {
-            return config('app.currency_symbol') . " ". $this->price;
+            return config('app.currency_symbol')." ".$this->price;
         }
 
-        return $this->price . " " . config('app.currency_symbol');
+        return $this->price." ".config('app.currency_symbol');
     }