Skip to content

Commit 05461dd

Browse files
author
GuustMetz
committed
feat: move namesFilterModel to the filteringModel
1 parent 0e5d567 commit 05461dd

2 files changed

Lines changed: 12 additions & 23 deletions

File tree

lib/public/views/lhcPeriods/ActiveColumns/lhcPeriodsActiveColumns.js

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

lib/public/views/lhcPeriods/Overview/LhcPeriodsOverviewModel.js

Lines changed: 10 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
* or submit itself to any jurisdiction.
1212
*/
1313

14+
import { FilteringModel } from '../../../components/Filters/common/FilteringModel.js';
1415
import { TextTokensFilterModel } from '../../../components/Filters/common/filters/TextTokensFilterModel.js';
1516
import { OverviewPageModel } from '../../../models/OverviewModel.js';
1617
import { buildUrl } from '/js/src/index.js';
@@ -27,16 +28,16 @@ export class LhcPeriodsOverviewModel extends OverviewPageModel {
2728
constructor() {
2829
super();
2930

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

3235
this._filteringModel.visualChange$.bubbleTo(this);
3336
this._filteringModel.observe(() => {
3437
this._pagination.silentlySetCurrentPage(1);
3538
this.load();
3639
});
3740

38-
this._namesFilterModel = new TextTokensFilterModel();
39-
this._registerFilter(this._namesFilterModel);
4041
this._yearsFilterModel = new TextTokensFilterModel();
4142
this._registerFilter(this._yearsFilterModel);
4243
this._pdpBeamTypesFilterModel = new TextTokensFilterModel();
@@ -47,15 +48,11 @@ export class LhcPeriodsOverviewModel extends OverviewPageModel {
4748
* @inheritdoc
4849
*/
4950
getRootEndpoint() {
50-
const params = {
51-
filter: {
52-
names: this._namesFilterModel.normalized,
53-
years: this._yearsFilterModel.normalized,
54-
pdpBeamTypes: this._pdpBeamTypesFilterModel.normalized,
55-
},
56-
};
51+
const filter = this._filteringModel.normalized;
52+
filter.years = this._yearsFilterModel.normalized;
53+
filter.pdpBeamTypes = this._pdpBeamTypesFilterModel.normalized;
5754

58-
return buildUrl('/api/lhcPeriodsStatistics', params);
55+
return buildUrl('/api/lhcPeriodsStatistics', { filter });
5956
}
6057

6158
/**
@@ -83,14 +80,6 @@ export class LhcPeriodsOverviewModel extends OverviewPageModel {
8380
});
8481
}
8582

86-
/**
87-
* Returns lhc periods names filter model
88-
* @return {TextTokensFilterModel} lhc periods names filter model
89-
*/
90-
get namesFilterModel() {
91-
return this._namesFilterModel;
92-
}
93-
9483
/**
9584
* Returns lhc periods years filter model
9685
* @return {TextTokensFilterModel} lhc periods years filter model
@@ -114,7 +103,7 @@ export class LhcPeriodsOverviewModel extends OverviewPageModel {
114103
*/
115104
reset() {
116105
super.reset();
117-
this._namesFilterModel.reset();
106+
this._filteringModel.reset();
118107
this._yearsFilterModel.reset();
119108
this._pdpBeamTypesFilterModel.reset();
120109
}
@@ -138,6 +127,6 @@ export class LhcPeriodsOverviewModel extends OverviewPageModel {
138127
* @return {boolean} true if any filter is active
139128
*/
140129
isAnyFilterActive() {
141-
return !this._namesFilterModel.isEmpty || !this._yearsFilterModel.isEmpty || !this._pdpBeamTypesFilterModel.isEmpty;
130+
return this._filteringModel.isAnyFilterActive() || !this._yearsFilterModel.isEmpty || !this._pdpBeamTypesFilterModel.isEmpty;
142131
}
143132
}

0 commit comments

Comments
 (0)