Fixed Model::all() method to work like regular Laravel and no longer operate on the eloquent query builder

Author: Smef

src/Database/Eloquent/FMEloquentBuilder.php

@@ -55,23 +55,6 @@ public function setModel($model, $ids = [])
         return $this;
     }
 
-    /**
-     * @return Collection
-     * @throws FileMakerDataApiException
-     */
-    public function all()
-    {
-        // Set the limit if the user hasn't already set one
-        // We're setting a high limit since the data API limits to 100 records by default
-        // It seems to break if we add any more 0s than 1000000000000000000
-        // Hopefully nobody is really going to hit this limit.... hopefully....
-        if ($this->query->limit == null or $this->query->limit == 0) {
-            $this->query->limit(1000000000000000000);
-        }
-
-        return $this->get();
-    }
-
 
     /**
      * Determine if any rows exist for the current query.

src/Database/Eloquent/FMModel.php

@@ -83,6 +83,13 @@ public function __construct(array $attributes = [])
         parent::__construct($attributes);
     }
 
+    public static function all($columns = ['*'])
+    {
+        return static::query()->limit(1000000000000000000)->get(
+            is_array($columns) ? $columns : func_get_args()
+        );
+    }
+
     /**
      * Create a model object from the returned FileMaker data
      *