jobapis / jobs-to-mail

app/Notifications/JobsCollected.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
      * Get the notification's delivery channels.
      *
      * @param  mixed  $notifiable
-     * @return array
+     * @return string[]
      */
     public function via($notifiable)
     {

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

@@ -1,8 +1,8 @@
 <?php namespace JobApis\JobsToMail\Notifications;
 
 use Illuminate\Bus\Queueable;
-use Illuminate\Notifications\Notification;
 use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Notifications\Notification;
 use Illuminate\Queue\SerializesModels;
 use JobApis\JobsToMail\Models\Search;
 use JobApis\JobsToMail\Notifications\Messages\JobMailMessage;

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
      */
     public function via($notifiable)
     {
-        return ['mail', 'database'];
+        return [ 'mail', 'database' ];
     }
 
     /**
@@ -72,8 +72,8 @@ 
         $message = new JobMailMessage();
 
         // Add user and search ID to view data
-        $message->viewData['user_id'] = $notifiable->id;
-        $message->viewData['search_id'] = $this->search->id;
+        $message->viewData[ 'user_id' ] = $notifiable->id;
+        $message->viewData[ 'search_id' ] = $this->search->id;
 
         // Update the subject
         $message->subject(count($this->jobs).' new jobs found especially for you');
@@ -95,7 +95,7 @@ 
         // Add a link to download the collection
         $message->action(
             'View More',
-            url('/notifications/' . $this->id)
+            url('/notifications/'.$this->id)
         );
 
         return $message;

app/Http/Controllers/NotificationsController.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\Jobs\Notifications\GenerateCsv;
 use JobApis\JobsToMail\Jobs\Notifications\GetNotificationById;
 

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
         $notification = $this->dispatchNow(new GetNotificationById($id));
 
         if ($notification) {
-            return view('notifications.single', ['notification' => $notification]);
+            return view('notifications.single', [ 'notification' => $notification ]);
         }
 
         $request->session()->flash('alert-danger', 'This job alert has expired.');
@@ -34,6 +34,6 @@ 
     {
         $path = $this->dispatchNow(new GenerateCsv($id));
 
-        return response()->download($path, null, ['Content-Type: text/csv']);
+        return response()->download($path, null, [ 'Content-Type: text/csv' ]);
     }
 }

app/Jobs/Notifications/SearchAndNotifyUser.php

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

@@ -1,15 +1,15 @@
 <?php namespace JobApis\JobsToMail\Jobs\Notifications;
 
 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\Filters\RecruiterFilter;
 use JobApis\JobsToMail\Models\Recruiter;
 use JobApis\JobsToMail\Models\Search;
 use JobApis\JobsToMail\Notifications\JobsCollected;
+use JobApis\Jobs\Client\JobsMulti;
 
 class SearchAndNotifyUser implements ShouldQueue
 {

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

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);

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');