From 6d32225954280ff92b600865b56f6257110cd4bc Mon Sep 17 00:00:00 2001 From: igoroctaviano Date: Fri, 7 Aug 2020 11:25:25 -0300 Subject: [PATCH] ohif-318: add comment --- platform/viewer/src/routes/WorkList/WorkList.jsx | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/platform/viewer/src/routes/WorkList/WorkList.jsx b/platform/viewer/src/routes/WorkList/WorkList.jsx index fc8fa1223..14484b10f 100644 --- a/platform/viewer/src/routes/WorkList/WorkList.jsx +++ b/platform/viewer/src/routes/WorkList/WorkList.jsx @@ -45,7 +45,12 @@ function WorkList({ history, data: studies, isLoadingData, dataSource }) { const debouncedFilterValues = useDebounce(filterValues, 200); const { resultsPerPage, pageNumber, sortBy, sortDirection } = filterValues; - let dynamicFilterValues = {}; + /* + * The default sort value keep the filters synchronized with runtime conditional sorting + * Only applied if no other sorting is specified and there are less than 101 studies + */ + let defaultSortValues = {}; + const sortedStudies = studies // TOOD: Move sort to DataSourceWrapper? // TODO: MOTIVATION, this is triggered on every render, even if list/sort does not change @@ -55,7 +60,7 @@ function WorkList({ history, data: studies, isLoadingData, dataSource }) { if (noSortApplied && studies.length < 101) { const ascendingSortModifier = -1; - dynamicFilterValues = { + defaultSortValues = { sortBy: 'studyDate', sortDirection: 'ascending', }; @@ -357,7 +362,7 @@ function WorkList({ history, data: studies, isLoadingData, dataSource }) { setFilterValues(defaultFilterValues)} isFiltering={isFiltering(filterValues, defaultFilterValues)}