jobapis / jobs-to-mail

config/jobboards.php

Spacing +14 added lines, -15 removed lines

@@ -3,10 +3,10 @@ 
 /**
  * Makes a best guess at the server's IP defaulting to localhost.
  */
-if (isset($_SERVER['REMOTE_ADDR'])) {
-    $currentIp = $_SERVER['REMOTE_ADDR'];
-} elseif (isset($_SERVER["HTTP_X_FORWARDED_FOR"])) {
-    $ipAddresses = explode(',', $_SERVER['HTTP_X_FORWARDED_FOR']);
+if (isset($_SERVER[ 'REMOTE_ADDR' ])) {
+    $currentIp = $_SERVER[ 'REMOTE_ADDR' ];
+} elseif (isset($_SERVER[ "HTTP_X_FORWARDED_FOR" ])) {
+    $ipAddresses = explode(',', $_SERVER[ 'HTTP_X_FORWARDED_FOR' ]);
     $currentIp = trim(end($ipAddresses));
 } else {
     $currentIp = '127.0.0.1';
@@ -15,17 +15,16 @@ 
 /**
  * Makes a best guess at the user agent making this request
  */
-$userAgent = isset($_SERVER['HTTP_USER_AGENT']) ? $_SERVER['HTTP_USER_AGENT'] :
-    'Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2228.0 Safari/537.36';
+$userAgent = isset($_SERVER[ 'HTTP_USER_AGENT' ]) ? $_SERVER[ 'HTTP_USER_AGENT' ] : 'Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2228.0 Safari/537.36';
 
 /**
  * Default job boards that require no API key permissions
  */
 $jobboards = [
-    'Careercast' => [],
-    'Dice' => [],
-    'Github' => [],
-    'Govt' => [],
+    'Careercast' => [ ],
+    'Dice' => [ ],
+    'Github' => [ ],
+    'Govt' => [ ],
 ];
 
 /**
@@ -33,7 +32,7 @@ 
  * http://developer.careerbuilder.com/
  */
 if (env("CAREERBUILDER_KEY")) {
-    $jobboards['Careerbuilder'] = [
+    $jobboards[ 'Careerbuilder' ] = [
         'DeveloperKey' => env("CAREERBUILDER_KEY"),
     ];
 }
@@ -42,7 +41,7 @@ 
  * http://www.indeed.com/publisher
  */
 if (env("INDEED_KEY")) {
-    $jobboards['Indeed'] = [
+    $jobboards[ 'Indeed' ] = [
         'publisher' => env("INDEED_KEY"),
         'userip' => $currentIp,
         'useragent' => $userAgent,
@@ -53,7 +52,7 @@ 
  * https://developer.usajobs.gov/Search-API/Overview
  */
 if (env("USAJOBS_KEY")) {
-    $jobboards['Usajobs'] = [
+    $jobboards[ 'Usajobs' ] = [
         'AuthorizationKey' => env("USAJOBS_KEY"),
     ];
 }
@@ -62,7 +61,7 @@ 
  * http://www.juju.com/publisher/spec/
  */
 if (env("JUJU_KEY")) {
-    $jobboards['Juju'] = [
+    $jobboards[ 'Juju' ] = [
         'partnerid' => env("JUJU_KEY"),
         'ipaddress' => $currentIp,
         'useragent' => $userAgent,
@@ -74,7 +73,7 @@ 
  * https://www.ziprecruiter.com/publishers
  */
 if (env("ZIPRECRUITER_KEY")) {
-    $jobboards['Ziprecruiter'] = [
+    $jobboards[ 'Ziprecruiter' ] = [
         'api_key' => env("ZIPRECRUITER_KEY"),
     ];
 }

database/factories/TokenFactory.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 |--------------------------------------------------------------------------
 */
 
-$factory->define(\JobApis\JobsToMail\Models\Token::class, function (Faker\Generator $faker) {
+$factory->define(\JobApis\JobsToMail\Models\Token::class, function(Faker\Generator $faker) {
     return [
         'user_id' => null,
         'type' => 'confirm',

database/factories/UserFactory.php

Spacing +3 added lines, -3 removed lines

@@ -6,7 +6,7 @@ 
 |--------------------------------------------------------------------------
 */
 
-$factory->define(\JobApis\JobsToMail\Models\User::class, function (Faker\Generator $faker) {
+$factory->define(\JobApis\JobsToMail\Models\User::class, function(Faker\Generator $faker) {
     return [
         'email' => $faker->safeEmail(),
         'keyword' => $faker->word(),
@@ -14,12 +14,12 @@ 
         'confirmed_at' => null,
     ];
 });
-$factory->state(\JobApis\JobsToMail\Models\User::class, 'active', function (Faker\Generator $faker) {
+$factory->state(\JobApis\JobsToMail\Models\User::class, 'active', function(Faker\Generator $faker) {
     return [
         'confirmed_at' => $faker->dateTimeThisYear(),
     ];
 });
-$factory->state(\JobApis\JobsToMail\Models\User::class, 'deleted', function (Faker\Generator $faker) {
+$factory->state(\JobApis\JobsToMail\Models\User::class, 'deleted', function(Faker\Generator $faker) {
     return [
         'deleted_at' => $faker->dateTimeThisYear(),
     ];

app/Console/Commands/EmailJobs.php

Unused Use Statements -3 removed lines

@@ -2,11 +2,8 @@
 
 use Illuminate\Console\Command;
 use Illuminate\Foundation\Bus\DispatchesJobs;
-use JobApis\JobsToMail\Jobs\CollectJobsForUser;
 use JobApis\JobsToMail\Jobs\SearchAndNotifyUser;
 use JobApis\JobsToMail\Repositories\Contracts\SearchRepositoryInterface;
-use JobApis\JobsToMail\Repositories\SearchRepository;
-use JobApis\JobsToMail\Repositories\UserRepository;
 
 class EmailJobs extends Command
 {

app/Jobs/SearchAndNotifyUser.php

Doc Comments -1 removed lines

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

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

@@ -1,14 +1,14 @@
 <?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\Collection;
-use JobApis\Jobs\Client\JobsMulti;
 use JobApis\JobsToMail\Models\User;
 use JobApis\JobsToMail\Notifications\JobsCollected;
+use JobApis\Jobs\Client\Collection;
+use JobApis\Jobs\Client\JobsMulti;
 
 class CollectJobsForUser implements ShouldQueue
 {

Spacing +7 added lines, -7 removed lines

@@ -77,12 +77,12 @@ 
      *
      * @return array
      */
-    protected function getJobsFromCollections($collectionsArray = [])
+    protected function getJobsFromCollections($collectionsArray = [ ])
     {
-        $jobs = [];
+        $jobs = [ ];
         array_walk_recursive(
             $collectionsArray,
-            function (Collection $collection) use (&$jobs) {
+            function(Collection $collection) use (&$jobs) {
                 $this->logErrorsFromCollection($collection);
                 $jobListings = array_slice(
                     $collection->all(),
@@ -90,7 +90,7 @@ 
                     self::MAX_JOBS_FROM_PROVIDER
                 );
                 foreach ($jobListings as $jobListing) {
-                    $jobs[] = $jobListing;
+                    $jobs[ ] = $jobListing;
                 }
             }
         );
@@ -120,14 +120,14 @@ 
      *
      * @return array
      */
-    protected function sortJobs($jobs = [])
+    protected function sortJobs($jobs = [ ])
     {
         // 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) {
+        $jobs = array_filter($jobs, function($job) {
             return $job->datePosted > new \DateTime(self::MAX_DAYS_OLD.' days ago');
         });
         // Truncate to the max number of results

app/Repositories/SearchRepository.php

Doc Comments -1 removed lines

@@ -12,7 +12,6 @@
     /**
      * SearchRepository constructor.
      *
-     * @param Search $model
      */
     public function __construct(Search $searches)
     {

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
      *
      * @return Search
      */
-    public function create($userId = null, $data = [])
+    public function create($userId = null, $data = [ ])
     {
         return $this->searches->create(
-            array_merge(['user_id' => $userId], $data)
+            array_merge([ 'user_id' => $userId ], $data)
         );
     }
 

database/migrations/2016_10_14_215937_create_searches_table.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 +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('searches', function (Blueprint $table) {
+        Schema::create('searches', function(Blueprint $table) {
             $table->uuid('id');
             $table->primary('id');
             $table->uuid('user_id')->unsigned();

app/Jobs/CreateUserAndSearch.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     /**
      * Create a new job instance.
      */
-    public function __construct($data = [])
+    public function __construct($data = [ ])
     {
         $this->data = $data;
     }

app/Repositories/Contracts/UserRepositoryInterface.php

Spacing +4 added lines, -4 removed lines

@@ -4,11 +4,11 @@
 {
     public function confirm($token = null);
 
-    public function create($data = []);
+    public function create($data = [ ]);
 
-    public function firstOrCreate($data = []);
+    public function firstOrCreate($data = [ ]);
 
-    public function getById($id = null, $options = []);
+    public function getById($id = null, $options = [ ]);
 
-    public function update($id = null, $data = []);
+    public function update($id = null, $data = [ ]);
 }