Do not use textual 'null' for dependencies without group (#725)

ben-manes · Feb 20, 2023 · 724f61c · 724f61c
1 parent fae2043
commit 724f61c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/...-plugin/src/main/kotlin/com/github/benmanes/gradle/versions/reporter/PlainTextReporter.kt b/...-plugin/src/main/kotlin/com/github/benmanes/gradle/versions/reporter/PlainTextReporter.kt
@@ -191,7 +191,7 @@ class PlainTextReporter(
   companion object {
     /** Returns the dependency key as a stringified label. */
     private fun label(dependency: Dependency): String {
-      return "${dependency.group}:${dependency.name}"
+      return "${dependency.group.orEmpty()}:${dependency.name}"
     }
   }
 }
diff --git a/...e-versions-plugin/src/main/kotlin/com/github/benmanes/gradle/versions/updates/Resolver.kt b/...e-versions-plugin/src/main/kotlin/com/github/benmanes/gradle/versions/updates/Resolver.kt
@@ -69,7 +69,7 @@ class Resolver(
       val resolvedCoordinate = Coordinate.from(dependency.module.id)
       val originalCoordinate = coordinates[resolvedCoordinate.key]
       val coord = originalCoordinate ?: resolvedCoordinate
-      if (originalCoordinate == null && resolvedCoordinate.groupId != "null") {
+      if (originalCoordinate == null && resolvedCoordinate.groupId != "none") {
         project.logger.info("Skipping hidden dependency: $resolvedCoordinate")
       } else {
         val projectUrl = getProjectUrl(dependency.module.id)
@@ -167,7 +167,7 @@ class Resolver(
     }
 
     // Format the query with an optional classifier and extension
-    var query = "${dependency.group}:${dependency.name}:$version"
+    var query = "${dependency.group.orEmpty()}:${dependency.name}:$version"
     if (dependency.artifacts.isNotEmpty()) {
       dependency.artifacts.firstOrNull()?.classifier?.let { classifier ->
         query += ":$classifier"
@@ -191,7 +191,7 @@ class Resolver(
     // If no version was specified then use "none" to pass it through.
     val version = if (dependency.version == null) "none" else "+"
     val nonTransitiveDependency =
-      project.dependencies.create("${dependency.group}:${dependency.name}:$version") as ModuleDependency
+      project.dependencies.create("${dependency.group.orEmpty()}:${dependency.name}:$version") as ModuleDependency
     nonTransitiveDependency.isTransitive = false
     return nonTransitiveDependency
   }
@@ -378,7 +378,7 @@ class Resolver(
             } else {
               val parent = getParentFromPom(file)
               if (parent != null &&
-                "${parent.group}:${parent.name}" != "org.sonatype.oss:oss-parent"
+                "${parent.group.orEmpty()}:${parent.name}" != "org.sonatype.oss:oss-parent"
               ) {
                 url = getProjectUrl(parent)
                 if (!url.isNullOrEmpty()) {

diff --git a/...thub/benmanes/gradle/versions/updates/resolutionstrategy/ComponentSelectionWithCurrent.kt b/...thub/benmanes/gradle/versions/updates/resolutionstrategy/ComponentSelectionWithCurrent.kt
@@ -10,7 +10,7 @@ class ComponentSelectionWithCurrent(
   override fun toString(): String {
     return """\
 ComponentSelectionWithCurrent{
-    group="${candidate.group}",
+    group="${candidate.group.orEmpty()}",
     module="${candidate.module}",
     version="${candidate.version}",
     currentVersion="$currentVersion",