jobapis / jobs-to-mail

app/Jobs/CollectJobsForUser.php

Doc Comments -1 removed lines

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

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

database/seeds/TestingDatabaseSeeder.php

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

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

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(),
     ];