Apply eslint rules

AliceO2Group · Sep 13, 2023 · 6e65e4a · 6e65e4a
1 parent fc59160
commit 6e65e4a
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 9 deletions.
diff --git a/app/public/components/filter/activeFilters.js b/app/public/components/filter/activeFilters.js
@@ -17,16 +17,16 @@ import { h } from '/js/src/index.js';
 /**
  * Component responsible for displaying active filters
  * @param {FilterModel} model model repsonsible for filter management for the current model
- * @returns {vnode}
+ * @returns {vnode} active filters chips
  */
 export default function activeFilters(model) {
     const onClearAll = () => {
         model.reset();
-    }
+    };
 
     const onClearFilter = (filter) => {
         model.removeFilter(filter.field, filter.value, filter.type);
-    }
+    };
 
     return [
         h('.flex-wrap.justify-between.items-center',

diff --git a/app/public/model/filtering/FilterModel.js b/app/public/model/filtering/FilterModel.js
@@ -39,6 +39,7 @@ export default class FilterModel extends Observable {
             case 'to': {
                 this._activeFilters[field][type] = [];
                 this._activeFilters[field][type].push(value);
+                break;
             }
             default: break;
         }
@@ -52,7 +53,7 @@ export default class FilterModel extends Observable {
         if (!this._activeFilters.hasOwnProperty(field)) {
             this._activeFilters[field] = {
                 [type]: [],
-            }
+            };
         }
         if (!this._activeFilters[field].hasOwnProperty(type)) {
             this.resetFilterValue(field, type);
@@ -108,10 +109,10 @@ export default class FilterModel extends Observable {
         let filterPhrase = '';
         for (const [field, typeValues] of Object.entries(this._activeFilters)) {
             for (const [type, values] of Object.entries(typeValues)) {
-                values.forEach((value) => filterPhrase += `${filterPhrase.length ? '&' : ''}filter[${field}]${this.filterTypesMapping(type, value)}`);
+                values.forEach((value) =>
+                    filterPhrase += `${filterPhrase.length ? '&' : ''}filter[${field}]${this.filterTypesMapping(type, value)}`);
             }
         }
-        console.log(filterPhrase);
         return filterPhrase;
     }
 }
diff --git a/app/public/views/periods/PeriodsModel.js b/app/public/views/periods/PeriodsModel.js
@@ -52,7 +52,7 @@ export default class PeriodsModel extends Observable {
         this._filtering.observe(() => {
             this.fetchCurrentPagePeriods();
             this.notify();
-        })
+        });
 
         this._currentPagePeriods = RemoteData.notAsked();
         this._allPeriods = RemoteData.notAsked();
@@ -76,7 +76,6 @@ export default class PeriodsModel extends Observable {
         this._allPeriods = RemoteData.notAsked();
 
         const endpoint = `/api/periods/?${encodeURI(this._filtering.buildFilterPhrase())}`;
-        console.log(endpoint);
         try {
             const { items, totalCount } = await getRemoteDataSlice(endpoint);
             this._allPeriods = RemoteData.success([...items]);

diff --git a/app/public/views/periods/overview/periodsContent.js b/app/public/views/periods/overview/periodsContent.js
@@ -16,7 +16,6 @@ import { RCT } from '../../../config.js';
 import title from '../../../components/table/title.js';
 import dataActionButtons, { dataActions } from '../../../components/buttons/dataActionButtons.js';
 import { anyFiltersActive } from '../../../utils/filtering/filterUtils.js';
-import obsoleteActiveFilters from '../../userView/data/table/filtering/obsoleteActiveFilters.js';
 import { noDataFound, noMatchingData } from '../../../components/messagePanel/messages.js';
 import periodsTableHeader from '../table/periodsTableHeader.js';
 import periodsTableRow from '../table/periodsTableRow.js';