GCTC-NTGC / TalentCloud

app/Http/Controllers/Admin/DepartmentCrudController.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
         // Custom strings to display within the backpack UI.
         $this->crud->setEntityNameStrings('department', 'departments');
 
-        $this->crud->operation(['create', 'update'], function () {
+        $this->crud->operation(['create', 'update'], function(){
             $this->crud->addField([
                 'name' => 'name',
                 'type' => 'text',
@@ -76,8 +76,8 @@ 
             'label' => 'Name',
             'orderable' => true,
             'limit' => 70,
-            'orderLogic' => function ($query, $column, $columnDirection) use ($locale) {
-                return $query->orderBy('name->' . $locale, $columnDirection)->select('*');
+            'orderLogic' => function($query, $column, $columnDirection) use ($locale) {
+                return $query->orderBy('name->'.$locale, $columnDirection)->select('*');
             }
         ]);
 

app/Http/Controllers/ApplicantProfileController.php

Spacing +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
         $validator->validate($request->all());
 
         foreach ($questions as $question) {
-            $answerName = $this->answerFormInputName . '.' . $question->id;
+            $answerName = $this->answerFormInputName.'.'.$question->id;
             if ($request->has($answerName)) {
                 $answer = ApplicantProfileAnswer::where(
                     [
@@ -142,7 +142,7 @@ 
                 )->first();
                 if ($answer == null) {
                     $answer = new ApplicantProfileAnswer();
-                    $answer->applicant_id =$applicant->id;
+                    $answer->applicant_id = $applicant->id;
                     $answer->applicant_profile_question_id = $question->id;
                 }
                 $answer->answer = $request->input($answerName);

app/Services/Validation/Rules/TwitterHandleRule.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function passes($attribute, $value)
     {
-        return preg_match('/' . self::PATTERN . '/', $value);
+        return preg_match('/'.self::PATTERN.'/', $value);
     }
 
     /**

routes/web.php

Spacing +15 added lines, -15 removed lines

@@ -18,10 +18,10 @@ 
         'prefix' => LaravelLocalization::setLocale(),
         'middleware' => ['localeSessionRedirect', 'localizationRedirect', 'localeViewPath']
     ],
-    function () : void {
+    function() : void {
         /** ADD ALL LOCALIZED ROUTES INSIDE THIS GROUP **/
 
-        Route::group(['prefix' => 'demo'], function () : void {
+        Route::group(['prefix' => 'demo'], function() : void {
 
             /* Temporary Blog Index */
             Route::view('blog', 'common/blog-index')->middleware('localOnly')->name('blog');
@@ -51,7 +51,7 @@ 
         });
 
 
-        Route::group(['prefix' => config('app.applicant_prefix')], function () : void {
+        Route::group(['prefix' => config('app.applicant_prefix')], function() : void {
 
             /* Home */
             Route::get('/', 'HomepageController@applicant')->name('home');
@@ -64,7 +64,7 @@ 
                 ->name('jobs.show');
 
             /* Require being logged in */
-            Route::middleware(['auth'])->group(function () : void {
+            Route::middleware(['auth'])->group(function() : void {
                 /* Managers */
                 Route::get('managers/{manager}', 'ManagerProfileController@show')
                     ->middleware('can:view,manager')
@@ -72,7 +72,7 @@ 
             });
 
             /* Require being logged in as applicant */
-            Route::middleware(['auth', 'role:applicant'])->group(function () : void {
+            Route::middleware(['auth', 'role:applicant'])->group(function() : void {
 
             // Application permissions are handled within the controller instead of with middleware
                 /* Applications */
@@ -207,9 +207,9 @@ 
 
         Route::group([
             'prefix' => config('app.manager_prefix'),
-        ], function (): void {
+        ], function(): void {
 
-            Route::middleware(['finishManagerRegistration'])->group(function (): void {
+            Route::middleware(['finishManagerRegistration'])->group(function(): void {
 
                 /* Home */
                 Route::get('/', 'HomepageController@manager')->name('manager.home');
@@ -225,7 +225,7 @@ 
                     'ManagerProfileController@faq'
                 )->name('manager.faq.section');
 
-                Route::middleware(['auth', 'role:manager'])->group(function (): void {
+                Route::middleware(['auth', 'role:manager'])->group(function(): void {
 
                     Route::get('profile', 'ManagerProfileController@editAuthenticated')->name('manager.profile');
 
@@ -322,7 +322,7 @@ 
             });
 
             // These routes must be excluded from the finishManagerRegistration middleware to avoid an infinite loop of redirects
-            Route::middleware(['auth', 'role:manager'])->group(function (): void {
+            Route::middleware(['auth', 'role:manager'])->group(function(): void {
                 Route::get('first-visit', 'Auth\FirstVisitController@showFirstVisitManagerForm')
                     ->name('manager.first_visit');
                 Route::post('finish_registration', 'Auth\FirstVisitController@finishManagerRegistration')
@@ -348,7 +348,7 @@ 
         /* AJAX calls =============================================================== */
 
         /* Require being logged in */
-        Route::middleware(['auth'])->group(function () : void {
+        Route::middleware(['auth'])->group(function() : void {
 
             Route::delete('courses/{course}', 'CourseController@destroy')
                 ->middleware('can:delete,course')
@@ -414,7 +414,7 @@ 
                 'prefix' => 'admin',
                 'middleware' => ['auth', 'role:admin']
             ],
-            function (): void {
+            function(): void {
                 /* Edit Job */
                 Route::get('jobs/{jobPoster}/edit', 'JobController@edit')
                     ->where('jobPoster', '[0-9]+')
@@ -435,7 +435,7 @@ 
         'prefix' => 'admin',
         'middleware' => ['auth', 'role:admin']
     ],
-    function (): void {
+    function(): void {
         // This page is non-localized, because the middleware that redirects to localized pages changes POSTs to GETs and messes up the request.
         Route::post('jobs/create/as-manager/{manager}', 'JobController@createAsManager')
             ->middleware('can:create,App\Models\JobPoster')
@@ -449,7 +449,7 @@ 
 
 
 /** API routes - currently using same default http auth, but not localized */
-Route::group(['prefix' => 'api'], function (): void {
+Route::group(['prefix' => 'api'], function(): void {
     // Protected by a gate in the controller, instead of policy middleware
     Route::get('jobs/{jobPoster}/assessment-plan', 'AssessmentPlanController@getForJob');
     // Public, not protected by policy or gate
@@ -494,7 +494,7 @@ 
         ->name('api.jobs.submit');
     Route::resource('jobs', 'Api\JobController')->only([
         'show', 'store', 'update'
-    ])->names([ // Specify custom names because default names collied with existing routes.
+    ])->names([// Specify custom names because default names collied with existing routes.
         'show' => 'api.jobs.show',
         'store' => 'api.jobs.store',
         'update' => 'api.jobs.update'
@@ -502,7 +502,7 @@ 
 
     Route::resource('managers', 'Api\ManagerController')->only([
         'show', 'update'
-    ])->names([ // Specify custom names because default names collied with existing routes
+    ])->names([// Specify custom names because default names collied with existing routes
         'show' => 'api.managers.show',
         'update' => 'api.managers.update'
     ]);

app/Http/Controllers/Admin/JobPosterCrudController.php

Spacing +9 added lines, -9 removed lines

@@ -54,11 +54,11 @@ 
             'name' => 'title',
             'type' => 'text',
             'label' => 'Title',
-            'searchLogic' => function ($query, $column, $searchTerm) use ($locale) : void {
-                $query->orWhere('title->' . $locale, 'ilike', "%$searchTerm%");
+            'searchLogic' => function($query, $column, $searchTerm) use ($locale) : void {
+                $query->orWhere('title->'.$locale, 'ilike', "%$searchTerm%");
             },
-            'orderLogic' => function ($query, $column, $columnDirection) use ($locale) {
-                return $query->orderBy('title->' . $locale, $columnDirection)->select('*');
+            'orderLogic' => function($query, $column, $columnDirection) use ($locale) {
+                return $query->orderBy('title->'.$locale, $columnDirection)->select('*');
             }
         ]);
         $this->crud->addColumn([
@@ -77,8 +77,8 @@ 
             'type' => 'closure',
             'label' => 'Manager',
             'orderable' => false,
-            'function' => function ($entry) {
-                return '<a href="' . route('manager.profile.edit', $entry->manager->user->id) . '" target="_blank">' . $entry->manager->user->full_name . '</a>';
+            'function' => function($entry){
+                return '<a href="'.route('manager.profile.edit', $entry->manager->user->id).'" target="_blank">'.$entry->manager->user->full_name.'</a>';
             }
         ]);
         $this->crud->addColumn([
@@ -98,10 +98,10 @@ 
             'name' => 'departments',
             'type' => 'select2_multiple',
             'label' => 'Departments'
-        ], function () {
+        ], function(){
             return Department::all()->pluck('name', 'id')->toArray();
-        }, function ($values) {
-            $this->crud->addClause('WhereHas', 'department', function ($query) use ($values) {
+        }, function($values){
+            $this->crud->addClause('WhereHas', 'department', function($query) use ($values) {
                 foreach (json_decode($values) as $key => $value) {
                     if ($key === 0) {
                         $query->where('id', $value);