Updated to use collection count (#324)

* Updated to use collection count

* Apply fixes from StyleCI (#325)

Co-authored-by: Roshan Gautam <roshangautam@users.noreply.github.com>

Co-authored-by: Roshan Gautam <roshan.gautam@hotmail.com>
Co-authored-by: Roshan Gautam <roshangautam@users.noreply.github.com>

Author: Quentin Schmick

database/migrations/2019_09_25_103421_update_task_results_duration_type.php

@@ -28,7 +28,7 @@ public function down()
     }
 
     /**
-     * @param bool $toFloat
+     * @param  bool  $toFloat
      */
     private function migrateDurationValues(bool $toFloat = true)
     {

src/Console/Commands/ListSchedule.php

@@ -32,7 +32,7 @@ class ListSchedule extends Command
     /**
      * Create a new command instance.
      *
-     * @param Schedule $schedule
+     * @param  Schedule  $schedule
      * @return void
      */
     public function __construct(Schedule $schedule)

src/Contracts/TaskInterface.php

@@ -21,28 +21,30 @@ public function find($id);
 
     /**
      * Returns all tasks.
+     *
      * @return \Illuminate\Database\Eloquent\Collection
      */
     public function findAll();
 
     /**
      * Returns all active tasks.
+     *
      * @return \Illuminate\Database\Eloquent\Collection
      */
     public function findAllActive();
 
     /**
      * Creates a new task with the given data.
      *
-     * @param  array $input
+     * @param  array  $input
      * @return \Studio\Totem\Task
      */
     public function store(array $input);
 
     /**
      * Updates the given task with the given data.
      *
-     * @param  array $input
+     * @param  array  $input
      * @param  \Studio\Totem\Task  $task
      * @return \Studio\Totem\Task
      */

src/Events/Creating.php

@@ -21,7 +21,7 @@ class Creating implements ShouldBroadcast
     /**
      * Create a new event instance.
      *
-     * @param array $input
+     * @param  array  $input
      */
     public function __construct(array $input)
     {

src/Events/Deleted.php

@@ -7,8 +7,8 @@ class Deleted extends Event
     /**
      * Create a new event instance.
      *
-     * @param array $input
-     * @param Task $task
+     * @param  array  $input
+     * @param  Task  $task
      */
     public function __construct()
     {

src/Events/Executed.php

@@ -10,8 +10,8 @@ class Executed extends BroadcastingEvent
     /**
      * Executed constructor.
      *
-     * @param Task $task
-     * @param string $started
+     * @param  Task  $task
+     * @param  string  $started
      */
     public function __construct(Task $task, $started, $output)
     {

src/Events/TaskEvent.php

@@ -18,7 +18,7 @@ class TaskEvent extends Event
     /**
      * Constructor.
      *
-     * @param Task $task
+     * @param  Task  $task
      */
     public function __construct(Task $task)
     {

src/Events/Updating.php

@@ -16,8 +16,8 @@ class Updating extends BroadcastingEvent
     /**
      * Create a new event instance.
      *
-     * @param array $input
-     * @param Task $task
+     * @param  array  $input
+     * @param  Task  $task
      */
     public function __construct(array $input, Task $task)
     {

src/Http/Controllers/ActiveTasksController.php

@@ -13,7 +13,7 @@ class ActiveTasksController extends Controller
     private $tasks;
 
     /**
-     * @param TaskInterface $tasks
+     * @param  TaskInterface  $tasks
      */
     public function __construct(TaskInterface $tasks)
     {

src/Http/Controllers/ExecuteTasksController.php

@@ -13,7 +13,7 @@ class ExecuteTasksController extends Controller
     private $tasks;
 
     /**
-     * @param TaskInterface $tasks
+     * @param  TaskInterface  $tasks
      */
     public function __construct(TaskInterface $tasks)
     {

src/Http/Controllers/ExportTasksController.php

@@ -13,7 +13,8 @@ class ExportTasksController extends Controller
 
     /**
      * ExportTasksController constructor.
-     * @param TaskInterface $tasks
+     *
+     * @param  TaskInterface  $tasks
      */
     public function __construct(TaskInterface $tasks)
     {

src/Http/Controllers/ImportTasksController.php

@@ -14,7 +14,8 @@ class ImportTasksController extends Controller
 
     /**
      * ImportTasksController constructor.
-     * @param TaskInterface $tasks
+     *
+     * @param  TaskInterface  $tasks
      */
     public function __construct(TaskInterface $tasks)
     {
@@ -25,7 +26,9 @@ public function __construct(TaskInterface $tasks)
 
     /**
      * Import tasks from a json file.
-     * @param \Studio\Totem\Http\Requests\ImportRequest $request
+     *
+     * @param  \Studio\Totem\Http\Requests\ImportRequest  $request
+     *
      * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
      */
     public function index(ImportRequest $request)

src/Http/Requests/ImportRequest.php

@@ -51,6 +51,7 @@ public function messages()
      *
      * @param  array|mixed  $keys
      * @return array
+     *
      * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
      */
     public function all($keys = null)
@@ -68,6 +69,7 @@ public function all($keys = null)
      * Get the validated data from the request.
      *
      * @return array
+     *
      * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
      */
     public function validated()
@@ -84,7 +86,7 @@ public function validated()
     /**
      * * Handle a failed validation attempt.
      *
-     * @param Validator $validator
+     * @param  Validator  $validator
      */
     protected function failedValidation(Validator $validator)
     {

src/Listeners/BuildCache.php

@@ -19,7 +19,7 @@ public function handle(Event $event)
     /**
      * Rebuild Cache.
      *
-     * @param Event $event
+     * @param  Event  $event
      */
     protected function build(Event $event)
     {

src/Listeners/BustCache.php

@@ -19,7 +19,7 @@ public function handle(Event $event)
     /**
      * Clear Cache.
      *
-     * @param Event $event
+     * @param  Event  $event
      */
     protected function clear(Event $event)
     {

src/Listeners/BustCacheImmediately.php

@@ -15,7 +15,7 @@ class BustCacheImmediately
     /**
      * Create the event listener.
      *
-     * @param Container $app
+     * @param  Container  $app
      */
     public function __construct(Container $app)
     {
@@ -35,7 +35,7 @@ public function handle(Event $event)
     /**
      * Clear Cache.
      *
-     * @param Event $event
+     * @param  Event  $event
      */
     protected function clear(Event $event)
     {

src/Listeners/Listener.php

@@ -21,8 +21,8 @@ class Listener implements ShouldQueue
     /**
      * Create the event listener.
      *
-     * @param Container $app
-     * @param TaskInterface $tasks
+     * @param  Container  $app
+     * @param  TaskInterface  $tasks
      */
     public function __construct(Container $app, TaskInterface $tasks)
     {

src/Repositories/EloquentTaskRepository.php

@@ -44,7 +44,7 @@ public function builder(): Builder
     /**
      * Find a task by id.
      *
-     * @param int|Task $id
+     * @param  int|Task  $id
      * @return int|Task
      */
     public function find($id)
@@ -87,7 +87,7 @@ public function findAllActive()
     /**
      * Create a new task.
      *
-     * @param array $input
+     * @param  array  $input
      * @return bool|Task
      */
     public function store(array $input)
@@ -108,8 +108,8 @@ public function store(array $input)
     /**
      * Update the given task.
      *
-     * @param array $input
-     * @param Task $task
+     * @param  array  $input
+     * @param  Task  $task
      * @return bool|int|Task
      */
     public function update(array $input, $task)
@@ -130,7 +130,7 @@ public function update(array $input, $task)
     /**
      * Delete the given task.
      *
-     * @param int|Task $id
+     * @param  int|Task  $id
      * @return bool
      */
     public function destroy($id)

src/Task.php

@@ -208,7 +208,7 @@ public function autoCleanup()
                     Result::query()
                         ->whereIn('id', $rowsToDelete)
                         ->delete();
-                } while ($rowsToDelete > 0);
+                } while ($rowsToDelete->count() > 0);
             } else {
                 do {
                     $rowsToDelete = $this->results()
@@ -221,7 +221,7 @@ public function autoCleanup()
                     Result::query()
                         ->whereIn('id', $rowsToDelete)
                         ->delete();
-                } while ($rowsToDelete > 0);
+                } while ($rowsToDelete->count() > 0);
             }
         }
     }

src/Totem.php

@@ -22,7 +22,6 @@ class Totem
      * Determine if the given request can access the Totem dashboard.
      *
      * @param  \Illuminate\Http\Request  $request
-     *
      * @return bool
      */
     public static function check($request)
@@ -36,7 +35,6 @@ public static function check($request)
      * Set the callback that should be used to authenticate Totem users.
      *
      * @param  \Closure  $callback
-     *
      * @return static
      */
     public static function auth(Closure $callback)

src/Traits/FrontendSortable.php

@@ -7,10 +7,9 @@
 trait FrontendSortable
 {
     /**
-     * @param \Illuminate\Database\Eloquent\Builder $builder
-     * @param array                                 $sortableColumns
-     * @param array                                 $defaultSort
-     *
+     * @param  \Illuminate\Database\Eloquent\Builder  $builder
+     * @param  array  $sortableColumns
+     * @param  array  $defaultSort
      * @return \Illuminate\Database\Eloquent\Builder
      */
     public function scopeSortableBy(Builder $builder, array $sortableColumns, array $defaultSort = ['name' => 'asc']): Builder

tests/Feature/ViewDashboardTest.php

@@ -80,8 +80,8 @@ public function view_dashboard_multiple_tasks_with_multiple_results()
     }
 
     /**
-     * @param int $task_count
-     * @param int $result_count
+     * @param  int  $task_count
+     * @param  int  $result_count
      * @return mixed
      */
     private function _get_task_with_results($task_count = 1, $result_count = 1)