Skip to content

Commit 1bd16ac

Browse files
authored
only order variable tree entities alphabetically (#1324)
1 parent 57c08f5 commit 1bd16ac

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

packages/libs/eda/src/lib/core/components/variableSelectors/VariableTree.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ import { ClearSelectionButton } from './VariableTreeDropdown';
1414
import { VariableLinkConfig } from '../VariableLink';
1515
import { useHistory } from 'react-router';
1616
import { useMemo } from 'react';
17+
import { orderBy } from 'lodash';
1718

1819
export interface VariableTreeProps {
1920
starredVariables?: VariableDescriptor[];
@@ -44,7 +45,7 @@ export default function VariableTree({
4445
scope,
4546
asDropdown,
4647
}: VariableTreeProps) {
47-
const entities = useStudyEntities();
48+
const entities = orderBy(useStudyEntities(), 'displayName');
4849
const valuesMap = useValuesMap(entities);
4950
const flattenedFields = useFlattenedFields(entities, scope);
5051
const fieldsByTerm = useFlattenFieldsByTerm(flattenedFields);

packages/libs/eda/src/lib/core/utils/study-metadata.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,7 @@ import {
1313
import { preorder } from '@veupathdb/wdk-client/lib/Utils/TreeUtils';
1414

1515
export function entityTreeToArray(rootEntity: StudyEntity): StudyEntity[] {
16-
return Seq.from(preorder(rootEntity, (e) => e.children ?? []))
17-
.orderBy((node) => node.displayName)
18-
.toArray();
16+
return Array.from(preorder(rootEntity, (e) => e.children ?? []));
1917
}
2018

2119
export interface EntityAndVariable {

0 commit comments

Comments
 (0)