GCTC-NTGC / TalentCloud

app/Models/Criteria.php

Indentation +2 added lines, -2 removed lines

@@ -28,8 +28,8 @@
  * @property string $level_description The localized description of the skill level (accounts for skill type).
  *
  *  Localized Properties:
-  * @property string $description
-  * @property string $specificity
+ * @property string $description
+ * @property string $specificity
  */
 class Criteria extends BaseModel
 {

app/Models/RatingGuideAnswer.php

Braces +2 added lines, -1 removed lines

@@ -36,7 +36,8 @@
      */
     public function setCriterionIdAttribute($value): void
     {
-        if (empty($value)) { // will check for empty string, null values
+        if (empty($value)) {
+// will check for empty string, null values
             $this->attributes['criterion_id'] = null;
         } else {
             $this->attributes['criterion_id'] = $value;

app/Models/User.php

Indentation +7 added lines, -7 removed lines

@@ -173,10 +173,10 @@ 
     }
 
     /**
-    * Check if the user has the specified role.
-    * @param string $role This may be either 'applicant', 'manager' or 'admin'.
-    * @return boolean
-    */
+     * Check if the user has the specified role.
+     * @param string $role This may be either 'applicant', 'manager' or 'admin'.
+     * @return boolean
+     */
     public function hasRole($role)
     {
         switch ($role) {
@@ -195,8 +195,8 @@ 
      * Set this user to the specified role.
      *
      * @param string $role Must be either 'applicant', 'manager' or 'admin.
-    * @return void
-    */
+     * @return void
+     */
     public function setRole(string $role): void
     {
         $this->user_role()->associate(UserRole::where('name', $role)->firstOrFail());
@@ -231,7 +231,7 @@ 
      * Returns a user's full name.
      *
      * @return string
-    */
+     */
     public function getFullNameAttribute(): string
     {
         return $this->first_name . ' ' . $this->last_name;

app/Models/JobPoster.php

Spacing +1 added lines, -1 removed lines

@@ -373,7 +373,7 @@
      */
     public function submitted_applications() // phpcs:ignore
     {
-        return $this->hasMany(\App\Models\JobApplication::class)->whereDoesntHave('application_status', function ($query): void {
+        return $this->hasMany(\App\Models\JobApplication::class)->whereDoesntHave('application_status', function($query): void {
             $query->where('name', 'draft');
         });
     }

app/Policies/ApplicantPolicy.php

Spacing +3 added lines, -3 removed lines

@@ -23,12 +23,12 @@ 
         $user_id = $user->id;
         return JobPoster::whereHas(
             'manager',
-            function ($q) use ($user_id) {
+            function($q) use ($user_id) {
                 $q->where('user_id', $user_id);
             }
         )->whereHas(
             'submitted_applications',
-            function ($q) use ($applicant_id) {
+            function($q) use ($applicant_id) {
                     $q->where('applicant_id', $applicant_id);
             }
         )->get()->isNotEmpty();
@@ -43,7 +43,7 @@ 
      */
     public function view(User $user, Applicant $applicant)
     {
-        $authApplicant =  $user->isApplicant() &&
+        $authApplicant = $user->isApplicant() &&
             $applicant->user->is($user);
         $authManager = $user->isManager() && $this->ownsJobApplicantAppliedTo($user, $applicant);
         return $authApplicant || $authManager;

Braces +2 added lines, -2 removed lines

@@ -23,12 +23,12 @@
         $user_id = $user->id;
         return JobPoster::whereHas(
             'manager',
-            function ($q) use ($user_id) {
+            function ($q) use ($user_id){
                 $q->where('user_id', $user_id);
             }
         )->whereHas(
             'submitted_applications',
-            function ($q) use ($applicant_id) {
+            function ($q) use ($applicant_id){
                     $q->where('applicant_id', $applicant_id);
             }
         )->get()->isNotEmpty();

app/Policies/BasePolicy.php

Spacing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     public function before($user, $ability)
     {
         if ($user->isAdmin()) {
-            $userText = '{id='.$user->id.'}';
-            Log::notice('User '.$userText.' has bypassed policy as an Admin');
+            $userText = '{id=' . $user->id . '}';
+            Log::notice('User ' . $userText . ' has bypassed policy as an Admin');
             return true;
         }
     }

app/Policies/ApplicationPolicy.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         $authManager = ($user->isManager() &&
             $jobApplication->job_poster->manager->user->is($user));
 
-        return $authApplicant||$authManager;
+        return $authApplicant || $authManager;
     }
 
     /**

database/factories/JobPosterFactory.php

Spacing +8 added lines, -8 removed lines

@@ -17,7 +17,7 @@ 
 
 $faker_fr = Faker\Factory::create('fr');
 
-$factory->define(JobPoster::class, function (Faker\Generator $faker) use ($faker_fr) {
+$factory->define(JobPoster::class, function(Faker\Generator $faker) use ($faker_fr) {
     $closeDate = $faker->dateTimeBetween('now', '1 months')->format('Y-m-d');
     $openDate = $faker->dateTimeBetween('-1 months', 'now')->format('Y-m-d');
     $startDate = $faker->dateTimeBetween('1 months', '2 months')->format('Y-m-d');
@@ -60,7 +60,7 @@ 
         'security_clearance_id' => SecurityClearance::inRandomOrder()->first()->id,
         'language_requirement_id' => LanguageRequirement::inRandomOrder()->first()->id,
         'remote_work_allowed' => $faker->boolean(50),
-        'manager_id' => function () {
+        'manager_id' => function(){
             return factory(Manager::class)->create()->id;
         },
         'team_size' => $faker->numberBetween(5, 30),
@@ -118,7 +118,7 @@ 
     ];
 });
 
-$factory->afterCreating(JobPoster::class, function ($jp) : void {
+$factory->afterCreating(JobPoster::class, function($jp) : void {
     $jp->criteria()->saveMany(factory(Criteria::class, 5)->make([
         'job_poster_id' => $jp->id
     ]));
@@ -133,7 +133,7 @@ 
 $factory->state(
     JobPoster::class,
     'byUpgradedManager',
-    ['manager_id' => function () {
+    ['manager_id' => function(){
             return factory(Manager::class)->state('upgraded')->create()->id;
     }]
 );
@@ -141,7 +141,7 @@ 
 $factory->state(
     JobPoster::class,
     'published',
-    function (Faker\Generator $faker) {
+    function(Faker\Generator $faker){
         return [
             'published' => true,
             'published_at' => $faker->dateTimeBetween('-1 months', '-3 weeks')
@@ -152,7 +152,7 @@ 
 $factory->state(
     JobPoster::class,
     'closed',
-    function (Faker\Generator $faker) {
+    function(Faker\Generator $faker){
         return [
             'published' => true,
             'published_at' => $faker->dateTimeBetween('-1 months', '-3 weeks'),
@@ -164,7 +164,7 @@ 
 $factory->state(
     JobPoster::class,
     'draft',
-    function (Faker\Generator $faker) {
+    function(Faker\Generator $faker){
         return [
             'published' => false,
             'open_date_time' => ptDayStartToUtcTime($faker->dateTimeBetween('5 days', '10 days')->format('Y-m-d')),
@@ -178,7 +178,7 @@ 
 $factory->state(
     JobPoster::class,
     'review_requested',
-    function (Faker\Generator $faker) {
+    function(Faker\Generator $faker){
         return [
             'published' => false,
             'open_date_time' => ptDayStartToUtcTime($faker->dateTimeBetween('5 days', '10 days')->format('Y-m-d')),

Braces +7 added lines, -7 removed lines

@@ -17,7 +17,7 @@ 
 
 $faker_fr = Faker\Factory::create('fr');
 
-$factory->define(JobPoster::class, function (Faker\Generator $faker) use ($faker_fr) {
+$factory->define(JobPoster::class, function (Faker\Generator $faker) use ($faker_fr){
     $closeDate = $faker->dateTimeBetween('now', '1 months')->format('Y-m-d');
     $openDate = $faker->dateTimeBetween('-1 months', 'now')->format('Y-m-d');
     $startDate = $faker->dateTimeBetween('1 months', '2 months')->format('Y-m-d');
@@ -60,7 +60,7 @@ 
         'security_clearance_id' => SecurityClearance::inRandomOrder()->first()->id,
         'language_requirement_id' => LanguageRequirement::inRandomOrder()->first()->id,
         'remote_work_allowed' => $faker->boolean(50),
-        'manager_id' => function () {
+        'manager_id' => function (){
             return factory(Manager::class)->create()->id;
         },
         'team_size' => $faker->numberBetween(5, 30),
@@ -133,7 +133,7 @@ 
 $factory->state(
     JobPoster::class,
     'byUpgradedManager',
-    ['manager_id' => function () {
+    ['manager_id' => function (){
             return factory(Manager::class)->state('upgraded')->create()->id;
     }]
 );
@@ -141,7 +141,7 @@ 
 $factory->state(
     JobPoster::class,
     'published',
-    function (Faker\Generator $faker) {
+    function (Faker\Generator $faker){
         return [
             'published' => true,
             'published_at' => $faker->dateTimeBetween('-1 months', '-3 weeks')
@@ -152,7 +152,7 @@ 
 $factory->state(
     JobPoster::class,
     'closed',
-    function (Faker\Generator $faker) {
+    function (Faker\Generator $faker){
         return [
             'published' => true,
             'published_at' => $faker->dateTimeBetween('-1 months', '-3 weeks'),
@@ -164,7 +164,7 @@ 
 $factory->state(
     JobPoster::class,
     'draft',
-    function (Faker\Generator $faker) {
+    function (Faker\Generator $faker){
         return [
             'published' => false,
             'open_date_time' => ptDayStartToUtcTime($faker->dateTimeBetween('5 days', '10 days')->format('Y-m-d')),
@@ -178,7 +178,7 @@ 
 $factory->state(
     JobPoster::class,
     'review_requested',
-    function (Faker\Generator $faker) {
+    function (Faker\Generator $faker){
         return [
             'published' => false,
             'open_date_time' => ptDayStartToUtcTime($faker->dateTimeBetween('5 days', '10 days')->format('Y-m-d')),

database/factories/RatingGuideQuestionFactory.php

Spacing +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
 use App\Models\RatingGuideQuestion;
 use App\Models\JobPoster;
 
-$factory->define(RatingGuideQuestion::class, function (Faker\Generator $faker) {
+$factory->define(RatingGuideQuestion::class, function(Faker\Generator $faker){
     return [
-        'job_poster_id' => function () {
+        'job_poster_id' => function(){
             return factory(JobPoster::class)->create()->id;
         },
         'assessment_type_id' => AssessmentType::inRandomOrder()->first()->id,

Braces +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
 use App\Models\RatingGuideQuestion;
 use App\Models\JobPoster;
 
-$factory->define(RatingGuideQuestion::class, function (Faker\Generator $faker) {
+$factory->define(RatingGuideQuestion::class, function (Faker\Generator $faker){
     return [
-        'job_poster_id' => function () {
+        'job_poster_id' => function (){
             return factory(JobPoster::class)->create()->id;
         },
         'assessment_type_id' => AssessmentType::inRandomOrder()->first()->id,