academico-sis / academico

app/Http/Controllers/InvoiceController.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
 
         // persist the products
         foreach ($request->products as $f => $product) {
-            $productType = match ($product['type']) {
+            $productType = match($product['type']) {
                 'enrollment' => Enrollment::class,
                 'scheduledPayment' => Enrollment::class,
                 'fee' => Fee::class,
@@ -155,7 +155,7 @@ 
             Payment::create([
                 'responsable_id' => backpack_user()->id,
                 'invoice_id' => $invoice->id,
-                'payment_method' => isset($payment['method']) ? $payment['method'] : null ,
+                'payment_method' => isset($payment['method']) ? $payment['method'] : null,
                 'value' => $payment['value'],
                 'date' => isset($payment['date']) ? Carbon::parse($payment['date']) : Carbon::now(),
             ]);

app/Http/Controllers/LeadStatusController.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         // if the sync with external mailing system is enabled
         if (config('mailing-system.external_mailing_enabled') == true) {
 
-            match ($request->input('status')) {
+            match($request->input('status')) {
                 1 => $listId = config('mailing-system.mailerlite.activeStudentsListId'),
                 2, 3 => $listId = config('mailing-system.mailerlite.inactiveStudentsListId'),
                 default => abort(422, 'List ID not found'),

app/Http/Controllers/HRController.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         $report_end_date = $request->report_end_date ? Carbon::parse($request->report_end_date) : Carbon::parse($period->end);
 
         // if we are dealing with a complete period, add theoretical volumes
-        if (! $request->report_start_date && ! $request->report_end_date) {
+        if (!$request->report_start_date && !$request->report_end_date) {
             // ensure the report end date is not before the end date to avoid inconsistent results.
             $report_end_date = $report_start_date->max($report_end_date);
 

app/Events/LeadStatusUpdatedEvent.php

Spacing +2 added lines, -3 removed lines

@@ -17,10 +17,9 @@
     public $firstname;
     public $listId;
 
-    public function __construct(Student|Contact $user, $listId = null)
+    public function __construct(Student | Contact $user, $listId = null)
     {
-        $listId = $listId ? $listId :
-        $this->email = $user->email;
+        $listId = $listId ? $listId : $this->email = $user->email;
         $this->name = $user->firstname;
         $this->lastname = $user->lastname;
         $this->listId = $listId;

app/Mail/ExternalCourseReport.php

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
 
     public function build()
     {
-        return $this->subject('Facturation ' . $this->data['partner_name'])->view('emails.external_courses_report');
+        return $this->subject('Facturation '.$this->data['partner_name'])->view('emails.external_courses_report');
     }
 }

app/Listeners/SendExternalCoursesReport.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         // foreach partner with an alert set for this day of month
         foreach (Partner::where('send_report_on', Carbon::now()->day)->get() as $partner) {
             // get all courses
-            $courses = $partner->courses()->whereHas('events', function (Builder $query) use ($period_start, $period_end) {
+            $courses = $partner->courses()->whereHas('events', function(Builder $query) use ($period_start, $period_end) {
                 $query->where('start', '>=', $period_start->toDateString())->where('end', '<=', $period_end->toDateString());
             })
             ->with('events')->withCount('events')

app/Providers/TelescopeServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 
         $this->hideSensitiveRequestDetails();
 
-        Telescope::filter(function (IncomingEntry $entry) {
+        Telescope::filter(function(IncomingEntry $entry) {
             if ($this->app->isLocal()) {
                 return true;
             }
@@ -61,7 +61,7 @@ 
      */
     protected function gate()
     {
-        Gate::define('viewTelescope', function ($user = null) {
+        Gate::define('viewTelescope', function($user = null) {
             return backpack_user() && backpack_user()->email === config('app.admin_email');
         });
     }

app/Services/ApolearnService.php

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
     public function authenticate() : string
     {
         Log::info('launching API');
-        $response = Http::post(config('lms.apolearn.url') . '/auth.gettoken', [
+        $response = Http::post(config('lms.apolearn.url').'/auth.gettoken', [
               'api_key' => $this->apiKey,
               'username' => config('lms.apolearn.username'),
               'password' => config('lms.apolearn.password'),
@@ -37,9 +37,9 @@ 
 
     public function createUser(User $user, ?string $password = null) : void
     {
-        Log::info('checking if user exists for local ID ' . $user->id);
+        Log::info('checking if user exists for local ID '.$user->id);
         // first check if the user already exists (email)
-        $response = Http::get(config('lms.apolearn.url') . "/users/getbyemail/$user->email", [
+        $response = Http::get(config('lms.apolearn.url')."/users/getbyemail/$user->email", [
             'auth_token' => $this->token,
             'api_key' => $this->apiKey,
         ]);
@@ -63,7 +63,7 @@ 
                 $data = Arr::add($data, 'password', $password);
             }
 
-            $response = Http::post(config('lms.apolearn.url') . '/users', $data);
+            $response = Http::post(config('lms.apolearn.url').'/users', $data);
 
             if ($this->actionSucceeded($response)) {
                 $user->update(['lms_id' => $response->json()['result']['id']]);
@@ -91,7 +91,7 @@ 
                 $data = Arr::add($data, 'password', $password);
             }
 
-            $response = Http::put(config('lms.apolearn.url') . '/users/' . $user->lms_id, $data);
+            $response = Http::put(config('lms.apolearn.url').'/users/'.$user->lms_id, $data);
         }
     }
 
@@ -102,8 +102,8 @@ 
             abort(422, 'This course already exists on the remote platform');
         }
 
-        Log::info('pushing local course ' . $course->id . ' to API');
-        $response = Http::post(config('lms.apolearn.url') . '/classrooms', [
+        Log::info('pushing local course '.$course->id.' to API');
+        $response = Http::post(config('lms.apolearn.url').'/classrooms', [
             "name" => $course->name,
             "shortname" => $course->shortname,
             "description" => $course->description,
@@ -121,7 +121,7 @@ 
         $course->update(['lms_id' => $courseId]);
 
         // assign an admin to the new class
-        $response = Http::post(config('lms.apolearn.url') . "/classrooms/addadmin/$courseId", [
+        $response = Http::post(config('lms.apolearn.url')."/classrooms/addadmin/$courseId", [
             'user_id' => config('lms.apolearn.admin_user_id'),
             'auth_token' => $this->token,
             'api_key' => $this->apiKey,
@@ -137,8 +137,8 @@ 
             $this->createCourse($course);
         }
 
-        Log::info('updating course with locale ID' . $course->id);
-        $response = Http::put(config('lms.apolearn.url') . "/classrooms/$course->lms_id", [
+        Log::info('updating course with locale ID'.$course->id);
+        $response = Http::put(config('lms.apolearn.url')."/classrooms/$course->lms_id", [
             "name" => $course->name,
             "shortname" => $course->shortname,
             "description" => $course->description,
@@ -153,14 +153,14 @@ 
 
         Log::info('updating the course teacher');
         // ensure the teacher is up to date
-        $response = Http::get(config('lms.apolearn.url') . "/classrooms/teachers/$course->lms_id", [
+        $response = Http::get(config('lms.apolearn.url')."/classrooms/teachers/$course->lms_id", [
             'auth_token' => $this->token,
             'api_key' => $this->apiKey,
         ]);
 
         if ($this->actionSucceeded($response)) {
             $teachers = collect($response->json()['result']['users']);
-            Log::info('found these teachers IDs on LMS:' . implode(', ', $teachers->pluck('id')->toArray()));
+            Log::info('found these teachers IDs on LMS:'.implode(', ', $teachers->pluck('id')->toArray()));
 
             // if the course has no teacher, stop
             if (!$course->teacher) {
@@ -171,9 +171,9 @@ 
             } else {
                 // check if remote course teachers are still valid
                 foreach ($teachers as $teacher) {
-                    Log::info('comparing ' . $teacher['id'] . ' and ' . $course->teacher->user->lms_id);
+                    Log::info('comparing '.$teacher['id'].' and '.$course->teacher->user->lms_id);
                     if ($teacher['id'] !== $course->teacher->user->lms_id) {
-                        Log::info('Removing teacher ' . $teacher['id'] . ' from course');
+                        Log::info('Removing teacher '.$teacher['id'].' from course');
                         $this->removeTeacher($course->lms_id, $teacher['id']);
                     }
                 }
@@ -199,7 +199,7 @@ 
             $this->createUser($student->user);
         }
 
-        $response = Http::post(config('lms.apolearn.url') . "/classrooms/addstudent/$courseId", [
+        $response = Http::post(config('lms.apolearn.url')."/classrooms/addstudent/$courseId", [
             'user_id' => $student->user->lms_id,
             'auth_token' => $this->token,
             'api_key' => $this->apiKey,
@@ -219,7 +219,7 @@ 
         if ($course->teacher_id)
         {
             // if the teacher doesn't exist on LMS, create them
-            if (! $course->teacher->user->lms_id) {
+            if (!$course->teacher->user->lms_id) {
                 Log::info('creating user now');
                 $this->createUser($course->teacher->user);
             }
@@ -236,7 +236,7 @@ 
 
     protected function removeTeacher($courseId, $teacherId): void
     {
-        Log::info('Removing teacher ' . $teacherId . ' from course ' . $courseId);
+        Log::info('Removing teacher '.$teacherId.' from course '.$courseId);
         $response = Http::put(config('lms.apolearn.url')."/classrooms/removeteacher/$courseId", [
             'user_id' => $teacherId,
             'auth_token' => $this->token,
@@ -249,7 +249,7 @@ 
 
     public function removeStudent($courseId, $userId): void
     {
-        Log::info('removing user id ' . $userId . ' from course ' . $courseId);
+        Log::info('removing user id '.$userId.' from course '.$courseId);
         $response = Http::put(config('lms.apolearn.url')."/classrooms/removestudent/$courseId", [
             'user_id' => $userId,
             'auth_token' => $this->token,

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
         }