Tweaks to clean up code and returned data type

Author: Michael Deck

src/Database/Eloquent/FMEloquentBuilder.php

@@ -23,12 +23,12 @@ class FMEloquentBuilder extends Builder
     public function get($columns = ['*'])
     {
         $records = $this->toBase()->get();
-        $models = $this->model->createModelsFromRecordSet(collect($records));
+        $models = $this->model->createModelsFromRecordSet($records);
 
         // If we actually found models we will also eager load any relationships that
         // have been specified as needing to be eager loaded, which will solve the
         // n+1 query issue for the developers to avoid running a lot of queries.
-        if (count($models) > 0) {
+        if ($models->isNotEmpty()) {
             $models = $this->eagerLoadRelations($models->all());
         } else {
             $models = $models->all();

src/Database/Query/FMBaseBuilder.php

@@ -410,7 +410,7 @@ public function layoutResponse($name): FMBaseBuilder
      */
     public function get($columns = ['*'])
     {
-        $records = Arr::get($this->getData(), 'response.data', []);
+        $records = collect(Arr::get($this->getData(), 'response.data', []));
 
         // filter to only requested columns
         if ($columns !== ['*']) {
@@ -583,6 +583,7 @@ public function min($column, $direction = self::ASCEND)
         $this->orderBy($column, $direction);
         $result = $this->first();
         $min = $result['fieldData'][$this->getMappedFieldName($column)];
+
         return $min;
     }