jobapis / jobs-to-mail

app/Jobs/Notifications/GenerateCsv.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
         return $this->createCsv($csv, $jobs, $this->id.'.csv');
     }
 
-    private function createCsv(Writer $csv, array $items = [], $filename = null)
+    private function createCsv(Writer $csv, array $items = [ ], $filename = null)
     {
         // Make sure line endings are detected.
         if (!ini_get("auto_detect_line_endings")) {
@@ -71,7 +71,7 @@ 
         }
 
         // Add header rows
-        $csv->insertOne(array_keys($items[0]));
+        $csv->insertOne(array_keys($items[ 0 ]));
 
         // Add each item as a new line to the CSV
         $csv->insertAll($items);

app/Repositories/Contracts/SearchRepositoryInterface.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 interface SearchRepositoryInterface
 {
-    public function create($userId = null, $data = []);
+    public function create($userId = null, $data = [ ]);
 
     public function delete($id = null);
 

routes/web.php

Spacing +6 added lines, -6 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 
 // Homepage
-Route::get('/', function () {
+Route::get('/', function() {
     return view('users.index');
 });
 
 // Terms page
-Route::get('/terms', function () {
+Route::get('/terms', function() {
     return view('static.terms');
 });
 
@@ -22,7 +22,7 @@ 
 // Current User Account page
 Route::get('/searches', 'SearchesController@index')->middleware('auth');
 
-Route::group(['prefix' => 'auth'], function () {
+Route::group([ 'prefix' => 'auth' ], function() {
     // Submit login form (part 1 of login)
     Route::post('/login', 'AuthController@login');
 
@@ -33,7 +33,7 @@ 
     Route::get('/confirm/{token}', 'AuthController@confirm');
 });
 
-Route::group(['prefix' => 'users'], function () {
+Route::group([ 'prefix' => 'users' ], function() {
     // Create new user
     Route::post('/', 'UsersController@create');
 
@@ -44,12 +44,12 @@ 
     Route::get('/{userId}/searches', 'SearchesController@index');
 });
 
-Route::group(['prefix' => 'searches'], function () {
+Route::group([ 'prefix' => 'searches' ], function() {
     // Unsubscribe by ID
     Route::get('/{searchId}/unsubscribe', 'SearchesController@unsubscribe');
 });
 
-Route::group(['prefix' => 'notifications'], function () {
+Route::group([ 'prefix' => 'notifications' ], function() {
     // Get a single notification with jobs
     Route::get('/{id}', 'NotificationsController@single');
 

app/Filters/RecruiterFilter.php

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

@@ -1,8 +1,8 @@
 <?php namespace JobApis\JobsToMail\Filters;
 
-use JobApis\Jobs\Client\Job;
 use JobApis\JobsToMail\Models\Recruiter;
 use JobApis\JobsToMail\Models\Search;
+use JobApis\Jobs\Client\Job;
 
 class RecruiterFilter
 {

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      */
     public function filter(array $jobs, Search $search)
     {
-        return array_filter($jobs, function (Job $job) use ($search) {
+        return array_filter($jobs, function(Job $job) use ($search) {
             // Make sure this job has a company
             if (isset($job->company) && $job->company) {
                 // See if this company is not a recruiter

app/Notifications/Messages/JobMailMessage.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
 
 class JobMailMessage extends MailMessage
 {
-    public $jobListings = [];
+    public $jobListings = [ ];
     public $advertisement = null;
 
     /**
@@ -17,7 +17,7 @@ 
      */
     public function listing(Job $job)
     {
-        $this->jobListings[] = [
+        $this->jobListings[ ] = [
             'link' => $job->getUrl(),
             'title' => $this->getTitle($job->getTitle()),
             'company' => $this->getCompany($job->getCompanyName(), $job->getIndustry()),

app/Http/Controllers/UsersController.php

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

@@ -1,9 +1,9 @@
 <?php namespace JobApis\JobsToMail\Http\Controllers;
 
 use Illuminate\Foundation\Bus\DispatchesJobs;
+use Illuminate\Foundation\Validation\ValidatesRequests;
 use Illuminate\Http\Request;
 use Illuminate\Routing\Controller as BaseController;
-use Illuminate\Foundation\Validation\ValidatesRequests;
 use JobApis\JobsToMail\Http\Requests\CreateUser;
 use JobApis\JobsToMail\Jobs\Users\CreateUserAndSearch;
 use JobApis\JobsToMail\Jobs\Users\Delete;

config/jobboards.php

Spacing +19 added lines, -20 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,20 +15,19 @@ 
 /**
  * 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' => [],
-    'Github' => [],
-    'Govt' => [],
-    'Ieee' => [],
-    'Jobinventory' => [],
-    'Monster' => [],
-    'Stackoverflow' => [],
+    'Careercast' => [ ],
+    'Github' => [ ],
+    'Govt' => [ ],
+    'Ieee' => [ ],
+    'Jobinventory' => [ ],
+    'Monster' => [ ],
+    'Stackoverflow' => [ ],
 ];
 
 /**
@@ -36,7 +35,7 @@ 
  * http://developer.careerbuilder.com/
  */
 if (env("CAREERBUILDER_KEY")) {
-    $jobboards['Careerbuilder'] = [
+    $jobboards[ 'Careerbuilder' ] = [
         'DeveloperKey' => env("CAREERBUILDER_KEY"),
     ];
 }
@@ -45,7 +44,7 @@ 
  * http://www.careerjet.com/partners/
  */
 if (env("CAREERJET_KEY")) {
-    $jobboards['Careerjet'] = [
+    $jobboards[ 'Careerjet' ] = [
         'affid' => env("CAREERJET_KEY"),
     ];
 }
@@ -54,7 +53,7 @@ 
  * http://www.indeed.com/publisher
  */
 if (env("INDEED_KEY")) {
-    $jobboards['Indeed'] = [
+    $jobboards[ 'Indeed' ] = [
         'publisher' => env("INDEED_KEY"),
         'userip' => $currentIp,
         'useragent' => $userAgent,
@@ -65,7 +64,7 @@ 
  * https://developer.usajobs.gov/Search-API/Overview
  */
 if (env("USAJOBS_KEY")) {
-    $jobboards['Usajobs'] = [
+    $jobboards[ 'Usajobs' ] = [
         'AuthorizationKey' => env("USAJOBS_KEY"),
     ];
 }
@@ -74,7 +73,7 @@ 
  * http://www.juju.com/publisher/spec/
  */
 if (env("JUJU_KEY")) {
-    $jobboards['Juju'] = [
+    $jobboards[ 'Juju' ] = [
         'partnerid' => env("JUJU_KEY"),
         'ipaddress' => $currentIp,
         'useragent' => $userAgent,
@@ -86,7 +85,7 @@ 
  * https://www.ziprecruiter.com/publishers
  */
 if (env("ZIPRECRUITER_KEY")) {
-    $jobboards['Ziprecruiter'] = [
+    $jobboards[ 'Ziprecruiter' ] = [
         'api_key' => env("ZIPRECRUITER_KEY"),
     ];
 }
@@ -95,7 +94,7 @@ 
  * https://www2.jobs2careers.com/advertiser.php
  */
 if (env("J2C_ID") && env("J2C_PASS")) {
-    $jobboards['J2c'] = [
+    $jobboards[ 'J2c' ] = [
         'id' => env("J2C_ID"),
         'pass' => env("J2C_PASS"),
     ];

database/migrations/2017_12_30_150223_add_max_searches_to_users.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
      */
     public function up()
     {
-        Schema::table('users', function (Blueprint $table) {
+        Schema::table('users', function(Blueprint $table) {
             $table->integer('max_searches')->default(config('app.max_searches'))->nullable();
         });
     }
@@ -25,7 +25,7 @@ 
      */
     public function down()
     {
-        Schema::table('users', function (Blueprint $table) {
+        Schema::table('users', function(Blueprint $table) {
             $table->dropColumn('max_searches');
         });
     }

database/factories/NotificationFactory.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 $notificationClass = \JobApis\JobsToMail\Models\CustomDatabaseNotification::class;
 $dataJson = json_decode('[{"url":"http://example.com/","name":"Technical Workforce Recruiter","query":"DevOps","title":"Technical Workforce Recruiter","skills":null,"source":"Careerjet","company":"AE Business Solutions","endDate":null,"industry":null,"location":"Madison, WI","sourceId":null,"startDate":null,"workHours":null,"baseSalary":null,"datePosted":"2017-12-22","description":" to Have:   Bachelors degree or higher.  Experience recruiting candidates with technical skill sets such as Cloud Architects","jobBenefits":null,"jobLocation":null,"alternateName":null,"maximumSalary":null,"minimumSalary":null,"employmentType":null,"qualifications":null,"salaryCurrency":null,"javascriptAction":null,"responsibilities":null,"hiringOrganization":{"url":null,"logo":null,"name":"AE Business Solutions","email":null,"address":null,"telephone":null,"description":null,"alternateName":null},"javascriptFunction":null,"specialCommitments":null,"occupationalCategory":null,"educationRequirements":null,"incentiveCompensation":null,"experienceRequirements":null}]');
 
-$factory->define($notificationClass, function (Faker\Generator $faker) use ($dataJson) {
+$factory->define($notificationClass, function(Faker\Generator $faker) use ($dataJson) {
     return [
         'id' => $faker->uuid(),
         'type' => 'JobApis\JobsToMail\Notifications\JobsCollected',