Add default affiliation

rickstaa · Mar 22, 2022 · 81ff6d6 · 81ff6d6
1 parent 3b68abe
commit 81ff6d6
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/fetchers/stats-fetcher.js b/src/fetchers/stats-fetcher.js
@@ -121,7 +121,12 @@ async function fetchStats(
     rank: { level: "C", score: 0 },
   };
 
-  let res = await retryer(fetcher, { login: username, ownerAffiliations});
+  // Set default value for ownerAffiliations in GraphQL query won't work because
+  // parseArray() will always return an empty array even nothing was specified
+  // and GraphQL would consider that empty arr as a valid value. Nothing will be
+  // queried in that case as no affiliation is presented.
+  ownerAffiliations = ownerAffiliations.length > 0 ? ownerAffiliations : ["OWNER"];
+  let res = await retryer(fetcher, { login: username, ownerAffiliations });
 
   if (res.data.errors) {
     logger.error(res.data.errors);

diff --git a/src/fetchers/top-languages-fetcher.js b/src/fetchers/top-languages-fetcher.js
@@ -47,6 +47,11 @@ const fetcher = (variables, token) => {
 async function fetchTopLanguages(username, ownerAffiliations, exclude_repo = []) {
   if (!username) throw new MissingParamError(["username"]);
 
+  // Set default value for ownerAffiliations in GraphQL query won't work because
+  // parseArray() will always return an empty array even nothing was specified
+  // and GraphQL would consider that empty arr as a valid value. Nothing will be
+  // queried in that case as no affiliation is presented.
+  ownerAffiliations = ownerAffiliations.length > 0 ? ownerAffiliations : ["OWNER"];
   const res = await retryer(fetcher, { login: username, ownerAffiliations });
 
   if (res.data.errors) {