Skip to content

Commit 3299b34

Browse files
Merge pull request #71 from mskilab-org/fix/minor-bugs
Fix/minor bugs
2 parents ca2a258 + c538502 commit 3299b34

File tree

3 files changed

+22
-4
lines changed

3 files changed

+22
-4
lines changed

src/components/filteredEventsListPanel/columnBuilders.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ function computeRangeBounds(records, dataIndex) {
162162
* @param {Object} rendererProps - Additional props to pass to the renderer, plus t() function
163163
* @returns {Object} Complete Ant Design column config
164164
*/
165-
export function buildColumnConfig(columnDef, records, rendererProps = {}) {
165+
export function buildColumnConfig(columnDef, records, rendererProps = {}, filteredValue = null) {
166166
const {
167167
id,
168168
title,
@@ -237,6 +237,10 @@ export function buildColumnConfig(columnDef, records, rendererProps = {}) {
237237
columnConfig.filterSearch = true;
238238
}
239239
}
240+
// Apply controlled filteredValue if provided (enables programmatic filter reset)
241+
if (filteredValue !== null) {
242+
columnConfig.filteredValue = filteredValue;
243+
}
240244
}
241245

242246
// Add sorter if enabled
@@ -262,7 +266,8 @@ export function buildColumnsFromSettings(
262266
settingsColumns = [],
263267
datasetColumns = [],
264268
records = [],
265-
rendererProps = {}
269+
rendererProps = {},
270+
filterValues = {}
266271
) {
267272
// Merge settings and dataset columns, dataset takes precedence
268273
const allColumns = [...settingsColumns];
@@ -281,6 +286,6 @@ export function buildColumnsFromSettings(
281286

282287
// Build column configs
283288
return allColumns.map((columnDef) =>
284-
buildColumnConfig(columnDef, records, rendererProps)
289+
buildColumnConfig(columnDef, records, rendererProps, filterValues[columnDef.id] ?? null)
285290
);
286291
}

src/components/filteredEventsListPanel/index.js

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,6 +318,16 @@ class FilteredEventsListPanel extends Component {
318318
let records =
319319
(eventType === "all" ? filteredEvents : recordsHash.get(eventType)) || [];
320320

321+
// Build filter values object for controlled filter state
322+
const filterValues = {
323+
gene: this.state.geneFilters,
324+
tier: this.state.tierFilters,
325+
type: this.state.typeFilters,
326+
role: this.state.roleFilters,
327+
effect: this.state.effectFilters,
328+
variant: this.state.variantFilters,
329+
};
330+
321331
// Build columns from settings.json and dataset configuration
322332
const columns = buildColumnsFromSettings(
323333
data?.filteredEventsColumns || [],
@@ -327,7 +337,8 @@ class FilteredEventsListPanel extends Component {
327337
t,
328338
selectFilteredEvent,
329339
getTierTooltipContent: this.getTierTooltipContent,
330-
}
340+
},
341+
filterValues
331342
);
332343

333344
return (

src/components/reportButtonsPanel/index.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,8 @@ const mapDispatchToProps = (dispatch) => ({
199199
const mapStateToProps = (state) => ({
200200
loading: state.PopulationStatistics.loading,
201201
id: state.CaseReport.id,
202+
CaseReport: state.CaseReport,
203+
Interpretations: state.Interpretations,
202204
mergedEvents: require("../../redux/interpretations/selectors").selectMergedEvents(state),
203205
});
204206

0 commit comments

Comments
 (0)