Filter empty version on single version query

SmartsquareGmbH · Sep 25, 2021 · 185e20a · 185e20a
1 parent c7f5c48
commit 185e20a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/src/strategy/organization-strategy.js b/src/strategy/organization-strategy.js
@@ -60,7 +60,9 @@ module.exports = class OrganizationStrategy extends Strategy {
 
     // Normalize results of each query so that we get a `versions` array for each package in either case.
     if (this.version) {
-      return result.organization.packages.nodes.map((it) => ({ ...it, versions: [it.version] }))
+      return result.organization.packages.nodes
+        .filter((it) => !!it.version)
+        .map((it) => ({ ...it, versions: [it.version] }))
     } else {
       return result.organization.packages.nodes.map((it) => ({ ...it, versions: it.versions.nodes }))
     }

diff --git a/src/strategy/repo-strategy.js b/src/strategy/repo-strategy.js
@@ -64,7 +64,9 @@ module.exports = class RepoStrategy extends Strategy {
 
     // Normalize results of each query so that we get a `versions` array for each package in either case.
     if (this.version) {
-      return result.repository.packages.nodes.map((it) => ({ ...it, versions: [it.version] }))
+      return result.repository.packages.nodes
+        .filter((it) => !!it.version)
+        .map((it) => ({ ...it, versions: [it.version] }))
     } else {
       return result.repository.packages.nodes.map((it) => ({ ...it, versions: it.versions.nodes }))
     }

diff --git a/src/strategy/user-strategy.js b/src/strategy/user-strategy.js
@@ -60,7 +60,7 @@ module.exports = class UserStrategy extends Strategy {
 
     // Normalize results of each query so that we get a `versions` array for each package in either case.
     if (this.version) {
-      return result.user.packages.nodes.map((it) => ({ ...it, versions: [it.version] }))
+      return result.user.packages.nodes.filter((it) => !!it.version).map((it) => ({ ...it, versions: [it.version] }))
     } else {
       return result.user.packages.nodes.map((it) => ({ ...it, versions: it.versions.nodes }))
     }