Merge pull request #2841 from EnesSacid-Buker/master

Collect function in models

app/Abstracts/Model.php

@@ -102,12 +102,22 @@ abstract class Model extends Eloquent implements Ownable
     {
         $request = request();
 
+        /**
+         * Modules that use the sort parameter in CRUD operations cause an error,
+         * so this sort parameter set back to old value after the query is executed.
+         * 
+         * for Custom Fields module
+         */
+        $request_sort = $request->get('sort');
+
         $query->usingSearchString()->sortable($sort);
 
         if ($request->expectsJson() && $request->isNotApi()) {
             return $query->get();
         }
 
+        $request->merge(['sort' => $request_sort]);
+        $request->offsetUnset('direction');
         $limit = (int) $request->get('limit', setting('default.list_limit', '25'));
 
         return $query->paginate($limit);

app/Models/Auth/User.php

@@ -190,11 +190,23 @@ class User extends Authenticatable implements HasLocalePreference
     public function scopeCollect($query, $sort = 'name')
     {
         $request = request();
-
         $search = $request->get('search');
+
+        /**
+         * Modules that use the sort parameter in CRUD operations cause an error,
+         * so this sort parameter set back to old value after the query is executed.
+         * 
+         * for Custom Fields module
+         */
+        $request_sort = $request->get('sort');
+
+        $query->usingSearchString($search)->sortable($sort);
+
+        $request->merge(['sort' => $request_sort]);
+        $request->offsetUnset('direction');
         $limit = (int) $request->get('limit', setting('default.list_limit', '25'));
 
-        return $query->usingSearchString($search)->sortable($sort)->paginate($limit);
+        return $query->paginate($limit);
     }
 
     /**