Skip to content

Commit 6970a74

Browse files
author
GuustMetz
committed
feat: move namesFilterModelFilter to fileringmodel
1 parent 25244f4 commit 6970a74

2 files changed

Lines changed: 14 additions & 19 deletions

File tree

lib/public/views/QcFlagTypes/ActiveColumns/qcFlagTypesActiveColumns.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ export const qcFlagTypesActiveColumns = {
3030
name: {
3131
name: 'Name',
3232
visible: true,
33-
filter: ({ namesFilterModel }) => textFilter(
34-
namesFilterModel,
33+
filter: ({ filteringModel }) => textFilter(
34+
filteringModel.get('names'),
3535
{ class: 'w-75 mt1', placeholder: 'e.g. BadPID, ...' },
3636
),
3737
classes: 'f6',

lib/public/views/QcFlagTypes/Overview/QcFlagTypesOverviewModel.js

Lines changed: 12 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import { TextTokensFilterModel } from '../../../components/Filters/common/filter
1515
import { OverviewPageModel } from '../../../models/OverviewModel.js';
1616
import { SelectionModel } from '../../../components/common/selection/SelectionModel.js';
1717
import { buildUrl } from '/js/src/index.js';
18-
import { FilterModel } from '../../../components/Filters/common/FilterModel.js';
18+
import { FilteringModel } from '../../../components/Filters/common/FilteringModel.js';
1919

2020
/**
2121
* QcFlagTypesOverviewModel
@@ -27,17 +27,21 @@ export class QcFlagTypesOverviewModel extends OverviewPageModel {
2727
constructor() {
2828
super();
2929

30-
this._filteringModel = new FilterModel({});
30+
this._filteringModel = new FilteringModel({
31+
names: new TextTokensFilterModel(),
32+
});
3133

32-
this._namesFilterModel = new TextTokensFilterModel();
33-
this._registerFilter(this._namesFilterModel);
3434
this._methodsFilterModel = new TextTokensFilterModel();
3535
this._registerFilter(this._methodsFilterModel);
3636
this._isBadFilterModel =
3737
new SelectionModel({ availableOptions: [{ label: 'Bad', value: true }, { label: 'Not Bad', value: false }] });
3838
this._registerFilter(this._isBadFilterModel);
3939

40-
this._filteringModel.observe(() => this._applyFilters());
40+
this._filteringModel.observe(() => {
41+
this._pagination.silentlySetCurrentPage(1);
42+
this.load();
43+
});
44+
4145
this._filteringModel.visualChange$.bubbleTo(this);
4246
}
4347

@@ -48,7 +52,7 @@ export class QcFlagTypesOverviewModel extends OverviewPageModel {
4852
const params = {};
4953
if (this.isAnyFilterActive()) {
5054
params.filter = {
51-
names: this._namesFilterModel.normalized,
55+
names: this._filteringModel.get("names").normalized,
5256
methods: this._methodsFilterModel.normalized,
5357
bad: this._isBadFilterModel.selected.length === 2
5458
? undefined
@@ -68,15 +72,6 @@ export class QcFlagTypesOverviewModel extends OverviewPageModel {
6872
return this._filteringModel;
6973
}
7074

71-
/**
72-
* Get names filter model
73-
*
74-
* @return {TextTokensFilterModel} names filter model
75-
*/
76-
get namesFilterModel() {
77-
return this._namesFilterModel;
78-
}
79-
8075
/**
8176
* Get methods filter model
8277
*
@@ -116,7 +111,7 @@ export class QcFlagTypesOverviewModel extends OverviewPageModel {
116111
* @return {boolean} true if any filter is active
117112
*/
118113
isAnyFilterActive() {
119-
return !this._namesFilterModel.isEmpty || !this._methodsFilterModel.isEmpty || this._isBadFilterModel.selected.length;
114+
return this._filteringModel.isAnyFilterActive() || !this._methodsFilterModel.isEmpty || this._isBadFilterModel.selected.length;
120115
}
121116

122117
/**
@@ -126,7 +121,7 @@ export class QcFlagTypesOverviewModel extends OverviewPageModel {
126121
*/
127122
reset() {
128123
this._methodsFilterModel.reset();
129-
this._namesFilterModel.reset();
124+
this._filteringModel.reset();
130125
this._isBadFilterModel.reset();
131126
super.reset();
132127
}

0 commit comments

Comments
 (0)