Skip to content

Commit 4790d30

Browse files
author
GuustMetz
committed
feat: replace ddflp's filter component with the common radiButton componnent
1 parent bdba964 commit 4790d30

2 files changed

Lines changed: 5 additions & 35 deletions

File tree

lib/public/views/Runs/ActiveColumns/runsActiveColumns.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ import { rawTextFilter } from '../../../components/Filters/common/filters/rawTex
4848
import { numericalComparisonFilter } from '../../../components/Filters/common/filters/numericalComparisonFilter.js';
4949
import { checkboxes } from '../../../components/Filters/common/filters/checkboxFilter.js';
5050
import { triggerValueFilter } from '../../../components/Filters/RunsFilter/triggerValueFilter.js';
51+
import radioButtonFilter from '../../../components/Filters/common/filters/radioButtonFilter.js';
5152

5253
/**
5354
* List of active columns for a generic runs table
@@ -525,7 +526,7 @@ export const runsActiveColumns = {
525526
classes: 'w-2 f6 w-wrapped',
526527
format: (boolean) => boolean ? 'On' : 'Off',
527528
exportFormat: (boolean) => boolean ? 'On' : 'Off',
528-
filter: ddflpFilter,
529+
filter: ({ filteringModel }) => radioButtonFilter(filteringModel.get('ddflp'), 'ddFlp'),
529530
},
530531
dcs: {
531532
name: 'DCS',

lib/public/views/Runs/Overview/RunsOverviewModel.js

Lines changed: 3 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ import { DataExportModel } from '../../../models/DataExportModel.js';
3636
import { runsActiveColumns as dataExportConfiguration } from '../ActiveColumns/runsActiveColumns.js';
3737
import { BeamModeFilterModel } from '../../../components/Filters/RunsFilter/BeamModeFilterModel.js';
3838
import { beamModesProvider } from '../../../services/beamModes/beamModesProvider.js';
39+
import { RadioButtonFilterModel } from '../../../components/Filters/common/RadioButtonFilterModel.js';
3940

4041
/**
4142
* Model representing handlers for runs page
@@ -90,6 +91,7 @@ export class RunsOverviewModel extends OverviewPageModel {
9091
inelasticInteractionRateAtStart: new NumericalComparisonFilterModel(),
9192
inelasticInteractionRateAtMid: new NumericalComparisonFilterModel(),
9293
inelasticInteractionRateAtEnd: new NumericalComparisonFilterModel(),
94+
ddflp: new RadioButtonFilterModel([{ label: 'ANY' }, { label: 'ON', value: true }, { label: 'OFF', value: false }]),
9395
});
9496

9597
this._filteringModel.observe(() => this._applyFilters(true));
@@ -147,8 +149,6 @@ export class RunsOverviewModel extends OverviewPageModel {
147149

148150
this._triggerValuesFilters = new Set();
149151

150-
this.ddflpFilter = '';
151-
152152
this.dcsFilter = '';
153153

154154
this.epnFilter = '';
@@ -165,7 +165,6 @@ export class RunsOverviewModel extends OverviewPageModel {
165165
isAnyFilterActive() {
166166
return this._filteringModel.isAnyFilterActive()
167167
|| this._triggerValuesFilters.size !== 0
168-
|| this.ddflpFilter !== ''
169168
|| this.dcsFilter !== ''
170169
|| this.epnFilter !== '';
171170
}
@@ -197,34 +196,6 @@ export class RunsOverviewModel extends OverviewPageModel {
197196
this._applyFilters();
198197
}
199198

200-
/**
201-
* Returns the boolean of ddflp
202-
* @return {Boolean} if ddflp is on
203-
*/
204-
getDdflpFilterOperation() {
205-
return this.ddflpFilter;
206-
}
207-
208-
/**
209-
* Sets the boolean of the filter if no new inputs were detected for 200 milliseconds
210-
* @param {boolean} operation if the ddflp is on
211-
* @return {undefined}
212-
*/
213-
setDdflpFilterOperation(operation) {
214-
this.ddflpFilter = operation;
215-
this._applyFilters();
216-
}
217-
218-
/**
219-
* Unchecks the ddflp checkbox and fetches all the runs.
220-
* @return {undefined}
221-
*
222-
*/
223-
removeDdflp() {
224-
this.ddflpFilter = '';
225-
this._applyFilters();
226-
}
227-
228199
/**
229200
* Returns the boolean of dcs
230201
* @return {Boolean} if dcs is on
@@ -291,9 +262,7 @@ export class RunsOverviewModel extends OverviewPageModel {
291262
...this._triggerValuesFilters.size !== 0 && {
292263
'filter[triggerValues]': Array.from(this._triggerValuesFilters).join(),
293264
},
294-
...(this.ddflpFilter === true || this.ddflpFilter === false) && {
295-
'filter[ddflp]': this.ddflpFilter,
296-
},
265+
'filter[ddflp]': this._filteringModel.get('ddflp').normalized,
297266
...(this.dcsFilter === true || this.dcsFilter === false) && {
298267
'filter[dcs]': this.dcsFilter,
299268
},

0 commit comments

Comments
 (0)