refactor(composer): Simply use getValue() for consistency

If values are expected to exist in a map, the code base usually simply uses `getValue()`. Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · Jul 23, 2024 · bfacb16 · bfacb16
1 parent 1df5d29
commit bfacb16
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/plugins/package-managers/composer/src/main/kotlin/Composer.kt b/plugins/package-managers/composer/src/main/kotlin/Composer.kt
@@ -20,7 +20,6 @@
 package org.ossreviewtoolkit.plugins.packagemanagers.composer
 
 import java.io.File
-import java.io.IOException
 
 import org.apache.logging.log4j.kotlin.logger
 
@@ -157,9 +156,6 @@ class Composer(
         getScopeDependencies(scopeName).filterNot { packageName ->
             packageName.isPlatformDependency() || packageName in virtualPackages // Virtual packages have no metadata.
         }.mapNotNullTo(mutableSetOf()) { packageName ->
-            val childInfo = packageInfoForName[packageName]
-                ?: throw IOException("Could not find package info for $packageName")
-
             if (packageName in dependencyBranch) {
                 logger.debug {
                     "Not adding circular dependency '$packageName' to the tree, it is already on this branch of the " +
@@ -169,6 +165,8 @@ class Composer(
                 return@mapNotNullTo null
             }
 
+            val childInfo = packageInfoForName.getValue(packageName)
+
             PackageReference(
                 id = childInfo.toId(),
                 dependencies = childInfo.toPackageReferences(