jobapis / jobs-to-mail

app/Models/Search.php

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
     {
         parent::boot();
 
-        static::creating(function ($model) {
+        static::creating(function($model) {
             $model->{$model->getKeyName()} = Uuid::uuid4();
         });
     }
@@ -55,7 +55,7 @@ 
      */
     public function scopeActive($query)
     {
-        return $query->whereHas('user', function ($query) {
+        return $query->whereHas('user', function($query) {
             return $query->confirmed();
         });
     }
@@ -67,7 +67,7 @@ 
      */
     public function scopeWhereUserEmail($query, $email = null)
     {
-        return $query->whereHas('user', function ($query) use ($email) {
+        return $query->whereHas('user', function($query) use ($email) {
             return $query->where('email', $email);
         });
     }
@@ -79,7 +79,7 @@ 
      */
     public function scopeWhereUserId($query, $id = null)
     {
-        return $query->whereHas('user', function ($query) use ($id) {
+        return $query->whereHas('user', function($query) use ($id) {
             return $query->where('id', $id);
         });
     }

routes/web.php

Spacing +3 added lines, -3 removed lines

@@ -2,11 +2,11 @@ 
 
 Route::get('/', 'UsersController@index');
 
-Route::get('/terms', function () {
+Route::get('/terms', function() {
     return view('static.terms');
 });
 
-Route::group(['prefix' => 'users'], function () {
+Route::group([ 'prefix' => 'users' ], function() {
 
     // Create new user
     Route::post('/', 'UsersController@create');
@@ -21,7 +21,7 @@ 
     Route::get('/{userId}/searches', 'UsersController@searches');
 });
 
-Route::group(['prefix' => 'searches'], function () {
+Route::group([ 'prefix' => 'searches' ], function() {
 
     // Unsubscribe by ID
     Route::get('/{searchId}/unsubscribe', 'SearchesController@unsubscribe');

app/Filters/CollectionFilter.php

Doc Comments -1 removed lines

@@ -100,7 +100,6 @@
     /**
      * Logs all the errors attached to a collection
      *
-     * @param array $jobsByProvider
      *
      * @return void
      */

Spacing +4 added lines, -4 removed lines

@@ -12,16 +12,16 @@
      *
      * @return array
      */
-    public function getJobsFromCollections($collectionsArray = [], $max = 50)
+    public function getJobsFromCollections($collectionsArray = [ ], $max = 50)
     {
-        $jobs = [];
+        $jobs = [ ];
         array_walk_recursive(
             $collectionsArray,
-            function (Collection $collection) use (&$jobs, $max) {
+            function(Collection $collection) use (&$jobs, $max) {
                 $this->logErrorsFromCollection($collection);
                 $jobListings = array_slice($collection->all(), 0, $max);
                 foreach ($jobListings as $jobListing) {
-                    $jobs[] = $jobListing;
+                    $jobs[ ] = $jobListing;
                 }
             }
         );

app/Jobs/SearchAndNotifyUser.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -1,13 +1,13 @@
 <?php namespace JobApis\JobsToMail\Jobs;
 
 use Illuminate\Bus\Queueable;
-use Illuminate\Queue\SerializesModels;
-use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Queue\InteractsWithQueue;
+use Illuminate\Queue\SerializesModels;
 use Illuminate\Support\Facades\Log;
-use JobApis\Jobs\Client\JobsMulti;
 use JobApis\JobsToMail\Models\User;
 use JobApis\JobsToMail\Notifications\JobsCollected;
+use JobApis\Jobs\Client\JobsMulti;
 
 class CollectJobsForUser implements ShouldQueue
 {

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     /**
      * @var array Jobs collected from providers.
      */
-    protected $jobs = [];
+    protected $jobs = [ ];
 
     /**
      * @var Recruiter recruiter model

database/migrations/2016_10_22_134445_recruiters.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 
-use Illuminate\Support\Facades\Schema;
-use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
 
 class CreateTokensTable extends Migration
 {

Spacing +3 added lines, -3 removed lines

@@ -13,14 +13,14 @@ 
      */
     public function up()
     {
-        Schema::create('recruiters', function (Blueprint $table) {
+        Schema::create('recruiters', function(Blueprint $table) {
             $table->uuid('id')->primary();
             $table->string('name')->unique();
             $table->string('url')->nullable();
             $table->timestamps();
         });
 
-        Schema::table('searches', function (Blueprint $table) {
+        Schema::table('searches', function(Blueprint $table) {
             $table->boolean('no_recruiters')->default(false);
         });
     }
@@ -32,7 +32,7 @@ 
      */
     public function down()
     {
-        Schema::table('searches', function (Blueprint $table) {
+        Schema::table('searches', function(Blueprint $table) {
             $table->dropColumn('no_recruiters');
         });
 

database/seeds/TestingDatabaseSeeder.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,10 +2,9 @@
 
 use Faker\Factory as Faker;
 use Illuminate\Database\Seeder;
-use JobApis\JobsToMail\Models\User;
-use JobApis\JobsToMail\Models\Recruiter;
 use JobApis\JobsToMail\Models\Search;
 use JobApis\JobsToMail\Models\Token;
+use JobApis\JobsToMail\Models\User;
 
 class TestingDatabaseSeeder extends Seeder
 {

app/Models/Recruiter.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
     {
         parent::boot();
 
-        static::creating(function ($model) {
+        static::creating(function($model) {
             $model->{$model->getKeyName()} = Uuid::uuid4();
         });
     }
@@ -46,6 +46,6 @@ 
     public function scopeWhereNameLike($query, $name = null)
     {
         $where = "to_tsvector('english', name) @@ plainto_tsquery('english', ?)";
-        return $query->whereRaw($where, [$name]);
+        return $query->whereRaw($where, [ $name ]);
     }
 }

app/Filters/RecruiterFilter.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         // Make sure this search wants to filter recruiters
         if ($search->no_recruiters === true) {
-            return array_filter($jobs, function ($job) {
+            return array_filter($jobs, function($job) {
                 // Make sure this job has a company
                 if (isset($job->company)) {
                     // Make sure this company is not a recruiter

app/Filters/JobFilter.php

Spacing +3 added lines, -3 removed lines

@@ -9,14 +9,14 @@
      *
      * @return array
      */
-    public function sort($jobs = [], $maxAge = 14, $maxJobs = 50)
+    public function sort($jobs = [ ], $maxAge = 14, $maxJobs = 50)
     {
         // Sort by date
-        usort($jobs, function ($item1, $item2) {
+        usort($jobs, function($item1, $item2) {
             return $item2->datePosted <=> $item1->datePosted;
         });
         // Filter any older than max age
-        $jobs = array_filter($jobs, function ($job) use ($maxAge) {
+        $jobs = array_filter($jobs, function($job) use ($maxAge) {
             return $job->datePosted > new \DateTime($maxAge.' days ago');
         });
         // Truncate to the max number of results