ohif-318: add comment
This commit is contained in:
parent
c627280982
commit
6d32225954
@ -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 }) {
|
||||
<StudyListFilter
|
||||
numOfStudies={numOfStudies}
|
||||
filtersMeta={filtersMeta}
|
||||
filterValues={{ ...filterValues, ...dynamicFilterValues }}
|
||||
filterValues={{ ...filterValues, ...defaultSortValues }}
|
||||
onChange={setFilterValues}
|
||||
clearFilters={() => setFilterValues(defaultFilterValues)}
|
||||
isFiltering={isFiltering(filterValues, defaultFilterValues)}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user