Skip to content

Commit f583422

Browse files
committed
w
1 parent e06a941 commit f583422

3 files changed

Lines changed: 6 additions & 24 deletions

File tree

lib/public/views/DataPasses/DataPassesOverviewModel.js

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import { SelectionFilterModel } from '../../../components/Filters/common/filters
1515
import { TextTokensFilterModel } from '../../../components/Filters/common/filters/TextTokensFilterModel.js';
1616
import { NON_PHYSICS_PRODUCTIONS_NAMES_WORDS, NonPhysicsProductionsNamesWords } from '../../../domain/enums/NonPhysicsProductionsNamesWords.js';
1717
import { OverviewPageModel } from '../../../models/OverviewModel.js';
18-
import { SortModel } from '../../components/common/table/SortModel.js';
1918

2019
/**
2120
* Data Passes Per LHC Period overview model
@@ -26,14 +25,6 @@ export class DataPassesOverviewModel extends OverviewPageModel {
2625
*/
2726
constructor() {
2827
super();
29-
this._sortModel = new SortModel();
30-
this._sortModel.observe(() => {
31-
this._pagination.silentlySetCurrentPage(1);
32-
this.load();
33-
});
34-
35-
this._sortModel.visualChange$.bubbleTo(this);
36-
3728
this._filteringModel = new FilteringModel({
3829
names: new TextTokensFilterModel(),
3930
nonPhysics: new SelectionFilterModel({
@@ -49,25 +40,19 @@ export class DataPassesOverviewModel extends OverviewPageModel {
4940
}
5041

5142
/**
52-
* Return filters and sort params of base model
43+
* Return filter params of base model
5344
*
54-
* @return {{ filter: object, sort: object }} filters and sort params
45+
* @return {object} filter
5546
*/
56-
getBaseParams() {
47+
getFilterParams() {
5748
const nonPhysicsSelectionModel = this._filteringModel.get('nonPhysics').selectionModel;
5849
const filter = {
5950
names: this._filteringModel.get('names').normalized,
6051
includeTest: nonPhysicsSelectionModel.selected.includes(NonPhysicsProductionsNamesWords.TEST),
6152
includeDebug: nonPhysicsSelectionModel.selected.includes(NonPhysicsProductionsNamesWords.DEBUG),
6253
};
6354

64-
const { appliedOn: sortOn, appliedDirection: sortDirection } = this._sortModel;
65-
const sortParams = {};
66-
if (sortOn && sortDirection) {
67-
sortParams[`sort[${sortOn}]`] = sortDirection;
68-
}
69-
70-
return { filter, sort: sortParams };
55+
return filter;
7156
}
7257

7358
/**

lib/public/views/DataPasses/PerLhcPeriodOverview/DataPassesPerLhcPeriodOverviewModel.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,12 @@ export class DataPassesPerLhcPeriodOverviewModel extends DataPassesOverviewModel
3636
* @inheritdoc
3737
*/
3838
getRootEndpoint() {
39-
const { filter, sort } = this.getBaseParams();
40-
console.log(sort)
39+
const filter = this.getFilterParams();
4140
const params = {
4241
filter: {
4342
lhcPeriodIds: [this._lhcPeriodId],
4443
...filter,
4544
},
46-
...sort,
4745
};
4846

4947
return buildUrl('/api/dataPasses', params);

lib/public/views/DataPasses/PerSimulationPassOverview/DataPassesPerSimulationPassOverviewModel.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,12 @@ export class DataPassesPerSimulationPassOverviewModel extends DataPassesOverview
5454
* @inheritdoc
5555
*/
5656
getRootEndpoint() {
57-
const { filter, sort } = this.getBaseParams();
57+
const filter = this.getFilterParams();
5858
const params = {
5959
filter: {
6060
simulationPassIds: [this._simulationPassId],
6161
...filter,
6262
},
63-
...sort,
6463
};
6564

6665
return buildUrl('/api/dataPasses', params);

0 commit comments

Comments
 (0)