[resources] Fix Res class generation if the library is declared as 'api'

JetBrains · Mar 1, 2024 · b318e8d · b318e8d
1 parent f833fa9
commit b318e8d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/gradle-plugins/compose/src/main/kotlin/org/jetbrains/compose/resources/ResourcesGenerator.kt b/gradle-plugins/compose/src/main/kotlin/org/jetbrains/compose/resources/ResourcesGenerator.kt
@@ -140,12 +140,14 @@ private fun Project.configureResourceGenerator(commonComposeResourcesDir: File,
         if (ComposeProperties.alwaysGenerateResourceAccessors(project).get()) {
             true
         } else {
-            configurations
-                .getByName(commonSourceSet.implementationConfigurationName)
-                .allDependencies.any { dep ->
-                    val depStringNotation = dep.let { "${it.group}:${it.name}:${it.version}" }
-                    depStringNotation == ComposePlugin.CommonComponentsDependencies.resources
-                }
+            configurations.run {
+                //because the implementation configuration doesn't extend the api in the KGP ¯\_(ツ)_/¯
+                getByName(commonSourceSet.implementationConfigurationName).allDependencies +
+                        getByName(commonSourceSet.apiConfigurationName).allDependencies
+            }.any { dep ->
+                val depStringNotation = dep.let { "${it.group}:${it.name}:${it.version}" }
+                depStringNotation == ComposePlugin.CommonComponentsDependencies.resources
+            }
         }
     }
 

diff --git a/gradle-plugins/compose/src/test/test-projects/misc/commonResources/build.gradle.kts b/gradle-plugins/compose/src/test/test-projects/misc/commonResources/build.gradle.kts
@@ -21,7 +21,9 @@ kotlin {
             dependencies {
                 implementation(compose.runtime)
                 implementation(compose.material)
-                implementation(compose.components.resources)
+                //there is the api to check correctness of the api configuration
+                //https://github.com/JetBrains/compose-multiplatform/issues/4405
+                api(compose.components.resources)
             }
         }
     }