jobapis / jobs-to-mail

app/Http/Controllers/UsersController.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     /**
      * Unsubscribe user account
      *
-     * @return string Json of all users
+     * @return \Illuminate\Http\RedirectResponse Json of all users
      */
     public function unsubscribe(Request $request, $userId)
     {

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     {
         $data = $request->only(array_keys($request->rules()));
 
-        if($user = $this->users->create($data)) {
+        if ($user = $this->users->create($data)) {
             $request->session()->flash('alert-success', 'A confirmation email has been sent. Once confirmed, you will start receiving job listings within 24 hours.');
         } else {
             $request->session()->flash('alert-warning', 'Something went wrong and your job search was not created. Please try again or file an issue on Github.');

app/Jobs/CollectJobsForUser.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Contracts\Queue\ShouldQueue;
 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;

Spacing +4 added lines, -4 removed lines

@@ -53,17 +53,17 @@
         }
     }
 
-    protected function sortJobs($collectionsArray = [])
+    protected function sortJobs($collectionsArray = [ ])
     {
-        $jobs = [];
+        $jobs = [ ];
         // Convert the array of collections to one large array
         foreach ($collectionsArray as $collection) {
             foreach (array_slice($collection->all(), 0, 10) as $jobListing) {
-                $jobs[] = $jobListing;
+                $jobs[ ] = $jobListing;
             }
         }
         // Order by date posted, desc
-        usort($jobs, function ($item1, $item2) {
+        usort($jobs, function($item1, $item2) {
             return $item2->datePosted <=> $item1->datePosted;
         });
         return $jobs;

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

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
      *
      * @return void
      */
-    public function __construct($jobs = [])
+    public function __construct($jobs = [ ])
     {
         $this->jobs = $jobs;
     }
@@ -33,7 +33,7 @@ 
      */
     public function via($notifiable)
     {
-        return ['mail'];
+        return [ 'mail' ];
     }
 
     /**
@@ -46,7 +46,7 @@ 
     {
         $count = count($this->jobs);
         $message = new JobMailMessage();
-        $message->viewData['user_id'] = $notifiable->id;
+        $message->viewData[ 'user_id' ] = $notifiable->id;
         $message->subject($count.' job listings found especially for you')
             ->greeting('Hello,')
             ->line('We found the following jobs that we think you\'ll be interested in based on your search:');

app/Notifications/Messages/JobMailMessage.php

Doc Comments -1 removed lines

@@ -12,7 +12,6 @@
     /**
      * Add a Job listing to the notification
      *
-     * @param  \Illuminate\Notifications\Action|string  $line
      * @return $this
      */
     public function listing(Job $job)

Unused Use Statements -2 removed lines

@@ -1,9 +1,7 @@
 <?php namespace JobApis\JobsToMail\Notifications\Messages;
 
-use Illuminate\Contracts\Logging\Log;
 use Illuminate\Notifications\Messages\MailMessage;
 use JobApis\Jobs\Client\Job;
-use League\Flysystem\Exception;
 
 class JobMailMessage extends MailMessage
 {

Spacing +3 added lines, -3 removed lines

@@ -7,7 +7,7 @@ 
 
 class JobMailMessage extends MailMessage
 {
-    public $jobListings = [];
+    public $jobListings = [ ];
 
     /**
      * Add a Job listing to the notification
@@ -26,7 +26,7 @@ 
             $line .= " in {$job->getLocation()}";
         }
         $line .= ".";
-        $this->jobListings[] = [
+        $this->jobListings[ ] = [
             'link' => $job->getUrl(),
             'text' => $line,
         ];
@@ -43,7 +43,7 @@ 
         return array_merge(
             $this->toArray(),
             $this->viewData,
-            ['jobListings' => $this->jobListings]
+            [ 'jobListings' => $this->jobListings ]
         );
     }
 }

app/Notifications/TokenGenerated.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)
     {

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function via($notifiable)
     {
-        return ['mail'];
+        return [ 'mail' ];
     }
 
     /**
@@ -46,7 +46,7 @@ 
     {
         $url = config('app.url').'users/confirm/'.$this->token;
         $message = new MailMessage;
-        $message->viewData['user_id'] = $notifiable->id;
+        $message->viewData[ 'user_id' ] = $notifiable->id;
         return $message
             ->subject('Confirm your email address to start receiving jobs')
             ->greeting('Thank you for joining JobsToMail.com.')

app/Repositories/UserRepository.php

Doc Comments -1 removed lines

@@ -20,7 +20,6 @@
     /**
      * UserRepository constructor.
      *
-     * @param User $model
      */
     public function __construct(User $users, Token $tokens)
     {

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
     {
         $tokenObject = $this->getUnexpiredConfirmationToken($token);
         if ($tokenObject) {
-            if ($this->update($tokenObject->user_id, ['confirmed_at' => Carbon::now()])) {
+            if ($this->update($tokenObject->user_id, [ 'confirmed_at' => Carbon::now() ])) {
                 return true;
             }
         }
@@ -53,7 +53,7 @@ 
      *
      * @return \JobApis\JobsToMail\Models\User
      */
-    public function create($data = [])
+    public function create($data = [ ])
     {
         // Create the user
         if ($user = $this->users->create($data)) {
@@ -73,7 +73,7 @@ 
      *
      * @return \JobApis\JobsToMail\Models\User
      */
-    public function getById($id = null, $options = [])
+    public function getById($id = null, $options = [ ])
     {
         return $this->users->where('id', $id)->first();
     }
@@ -96,7 +96,7 @@ 
      *
      * @return boolean
      */
-    public function update($id = null, $data = [])
+    public function update($id = null, $data = [ ])
     {
         return $this->users->where('id', $id)->update($data);
     }

app/Models/Token.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     {
         parent::boot();
 
-        static::creating(function ($model) {
+        static::creating(function($model) {
             // Generate a secure random token
             $model->{$model->getKeyName()} = bin2hex(openssl_random_pseudo_bytes(16));
         });

app/Models/User.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     {
         parent::boot();
 
-        static::creating(function ($model) {
+        static::creating(function($model) {
             $model->{$model->getKeyName()} = Uuid::uuid4();
         });
     }

app/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         Route::group([
             'middleware' => 'web',
             'namespace' => $this->namespace,
-        ], function ($router) {
+        ], function($router) {
             require base_path('routes/web.php');
         });
     }
@@ -70,7 +70,7 @@ 
             'middleware' => 'api',
             'namespace' => $this->namespace,
             'prefix' => 'api',
-        ], function ($router) {
+        ], function($router) {
             require base_path('routes/api.php');
         });
     }