move the sort 'filter' to fetchlogs, since it doesn't actually filter anything

Author: Guust

lib/public/views/Logs/Overview/LogsOverviewModel.js

@@ -80,13 +80,18 @@ export class LogsOverviewModel extends Observable {
      */
     async fetchLogs() {
         const keepExisting = this._pagination.currentPage > 1 && this._pagination.isInfiniteScrollEnabled;
+        const sortOn = this._overviewSortModel.appliedOn;
+        const sortDirection = this._overviewSortModel.appliedDirection;
 
         if (!keepExisting) {
             this._logs = RemoteData.loading();
             this.notify();
         }
 
         const params = {
+            ...sortOn && sortDirection && {
+                [`sort[${sortOn}]`]: sortDirection,
+            },
             ...this._getFilterQueryParams(),
             'page[offset]': this._pagination.firstItemOffset,
             'page[limit]': this._pagination.itemsPerPage,
@@ -189,8 +194,6 @@ export class LogsOverviewModel extends Observable {
      * @private
      */
     _getFilterQueryParams() {
-        const sortOn = this._overviewSortModel.appliedOn;
-        const sortDirection = this._overviewSortModel.appliedDirection;
         const titleFilter = this._filteringModel.get('titleFilter');
         const contentFilter = this._filteringModel.get('contentFilter');
         const authorFilter = this._filteringModel.get('authorFilter');
@@ -230,9 +233,6 @@ export class LogsOverviewModel extends Observable {
                 'filter[lhcFills][values]': lhcFills.normalized,
                 'filter[lhcFills][operation]': this.lhcFillFilterOperation.toLowerCase(),
             },
-            ...sortOn && sortDirection && {
-                [`sort[${sortOn}]`]: sortDirection,
-            },
         };
     }
 }