Skip to content

Commit fcaca24

Browse files
committed
handle empty variants explicitly
1 parent fafe49d commit fcaca24

1 file changed

Lines changed: 12 additions & 5 deletions

File tree

website/src/components/views/wasap/useWasapPageData.ts

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,11 @@ async function fetchCollectionModeData(
156156
queryString: string;
157157
}[] = [];
158158

159+
const invalidVariants: {
160+
name: string;
161+
error: string;
162+
}[] = [];
163+
159164
for (const variant of collection.variants) {
160165
let queryString: string;
161166
switch (variant.query.type) {
@@ -168,6 +173,13 @@ async function fetchCollectionModeData(
168173
break;
169174
}
170175
}
176+
if (queryString === '') {
177+
invalidVariants.push({
178+
name: variant.name,
179+
error: 'Variant is empty.',
180+
});
181+
continue;
182+
}
171183
variantData.push({
172184
name: variant.name,
173185
queryString: queryString,
@@ -187,11 +199,6 @@ async function fetchCollectionModeData(
187199
coverageQuery: string;
188200
}[] = [];
189201

190-
const invalidVariants: {
191-
name: string;
192-
error: string;
193-
}[] = [];
194-
195202
variantData.forEach(({ name, queryString }, index) => {
196203
const parseResult = parseResults[index];
197204

0 commit comments

Comments
 (0)