Skip to content

Commit c2a0bb8

Browse files
authored
[DUOS-2869] Fix various errors in dev (#2431)
1 parent d8cbb41 commit c2a0bb8

File tree

3 files changed

+24
-25
lines changed

3 files changed

+24
-25
lines changed

src/components/SignIn.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ export const SignIn = (props) => {
2525
if (isSubscribed) {
2626
const googleClientId = await Config.getGoogleClientId();
2727
setClientId(googleClientId);
28-
if (GoogleIS.client === null) {
28+
if (window.google !== undefined && GoogleIS.client === null) {
2929
await GoogleIS.initTokenClient(googleClientId, onSuccess, onFailure);
3030
}
3131
}

src/pages/DatasetCatalog.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,16 @@ export default function DatasetCatalog(props) {
8585
const [filterToOnlySelected, setFilterToOnlySelected] = useState(false);
8686

8787

88+
const applyDatasetSort = useCallback((sortParams, datasets) => {
89+
const sortedList = datasets.sort((a, b) => {
90+
const aVal = a[sortParams.field] || findPropertyValue(a, sortParams.field);
91+
const bVal = b[sortParams.field] || findPropertyValue(b, sortParams.field);
92+
return aVal.localeCompare(bVal, 'en', {numeric: true}) * sortParams.dir;
93+
});
94+
setSort(sortParams);
95+
setDatasetList(sortedList);
96+
}, []);
97+
8898
const getDatasets = useCallback(async () => {
8999
let datasets = await DataSet.getDatasets();
90100
let localDacs = await getDacs();
@@ -172,16 +182,6 @@ export default function DatasetCatalog(props) {
172182

173183
}, [searchDulText, pageSize, selectedDatasets, datasetList, filterToOnlySelected, currentPageOnlySelected, currentPageAllDatasets, dacFilter, useCustomFilter]);
174184

175-
const applyDatasetSort = useCallback((sortParams, datasets) => {
176-
const sortedList = datasets.sort((a, b) => {
177-
const aVal = a[sortParams.field] || findPropertyValue(a, sortParams.field);
178-
const bVal = b[sortParams.field] || findPropertyValue(b, sortParams.field);
179-
return aVal.localeCompare(bVal, 'en', {numeric: true}) * sortParams.dir;
180-
});
181-
setSort(sortParams);
182-
setDatasetList(sortedList);
183-
}, []);
184-
185185
const getDacs = async () => {
186186
let dacs = await DAC.list(false);
187187
dacs = dacs.map(dac => {

src/pages/manage_dac/ManageDac.js

Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,19 @@ export const ManageDac = function ManageDac() {
3434
const [selectedDatasets, setSelectedDatasets] = useState([]);
3535

3636

37+
const reloadDacList = useCallback(async () => {
38+
setIsLoading(true);
39+
DAC.list().then(
40+
dacs => {
41+
if (userRole === CHAIR) {
42+
dacs = dacs.filter((dac) => dacIDs.includes(dac.dacId));
43+
}
44+
setDacs(dacs);
45+
setIsLoading(false);
46+
}
47+
);
48+
}, [dacIDs, userRole]);
49+
3750
useEffect(() => {
3851
Promise.all([
3952
reloadUserRole(),
@@ -55,20 +68,6 @@ export const ManageDac = function ManageDac() {
5568
}
5669
};
5770

58-
const reloadDacList = useCallback(async () => {
59-
setIsLoading(true);
60-
DAC.list().then(
61-
dacs => {
62-
if (userRole === CHAIR) {
63-
dacs = dacs.filter((dac) => dacIDs.includes(dac.dacId));
64-
}
65-
setDacs(dacs);
66-
setIsLoading(false);
67-
}
68-
);
69-
}, [dacIDs, userRole]);
70-
71-
7271
const reloadUserRole = useCallback(async () => {
7372
setIsLoading(true);
7473
const currentUser = Storage.getCurrentUser();

0 commit comments

Comments
 (0)