Handle wrongly implemented `DokkaMultiModuleFileLayout`

1 files changed, 15 insertions, 3 deletions

diff --git a/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/DokkaMultiModuleFileLayout.kt b/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/DokkaMultiModuleFileLayout.kt
index bb48ba2a..e3b82e7c 100644
--- a/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/DokkaMultiModuleFileLayout.kt
+++ b/runners/gradle-plugin/src/main/kotlin/org/jetbrains/dokka/gradle/DokkaMultiModuleFileLayout.kt
@@ -1,5 +1,6 @@
 package org.jetbrains.dokka.gradle
 
+import org.jetbrains.dokka.DokkaException
 import java.io.File
 
 /**
@@ -59,14 +60,25 @@ internal fun DokkaMultiModuleTask.copyChildOutputDirectory(child: AbstractDokkaT
     val targetChildOutputDirectory = project.file(fileLayout.targetChildOutputDirectory(this, child))
     val sourceChildOutputDirectory = child.outputDirectory.getSafe()
 
-    if (!sourceChildOutputDirectory.exists()) {
+    /* Pointing to the same directory -> No copy necessary */
+    if (sourceChildOutputDirectory.absoluteFile == targetChildOutputDirectory.absoluteFile) {
         return
     }
 
-    if (sourceChildOutputDirectory.absoluteFile == targetChildOutputDirectory.absoluteFile) {
+    /* Cannot target *inside* the original folder */
+    if (targetChildOutputDirectory.absoluteFile.startsWith(sourceChildOutputDirectory.absoluteFile)) {
+        throw DokkaException(
+            "Cannot re-locate output directory into itself.\n" +
+                    "sourceChildOutputDirectory=${sourceChildOutputDirectory.path}\n" +
+                    "targetChildOutputDirectory=${targetChildOutputDirectory.path}"
+        )
+    }
+
+    /* Source output directory is empty -> No copy necessary */
+    if (!sourceChildOutputDirectory.exists()) {
         return
     }
 
-    sourceChildOutputDirectory.copyRecursively(targetChildOutputDirectory, overwrite = false)
+    sourceChildOutputDirectory.copyRecursively(targetChildOutputDirectory, overwrite = true)
 }