Fix missing tab entries for module names with space (#3019)

2 files changed, 12 insertions, 0 deletions

diff --git a/runners/cli/src/main/kotlin/org/jetbrains/dokka/SourceSetArgumentsParser.kt b/runners/cli/src/main/kotlin/org/jetbrains/dokka/SourceSetArgumentsParser.kt
index 49e8f2ae..120bd296 100644
--- a/runners/cli/src/main/kotlin/org/jetbrains/dokka/SourceSetArgumentsParser.kt
+++ b/runners/cli/src/main/kotlin/org/jetbrains/dokka/SourceSetArgumentsParser.kt
@@ -7,6 +7,12 @@ import kotlinx.cli.delimiter
 
 internal fun parseSourceSet(moduleName: String, args: Array<String>): DokkaConfiguration.DokkaSourceSet {
 
+    if (moduleName.contains(',')) {
+        // To figure out why this is needed and if it is still relevant, see the comment here:
+        // https://github.com/Kotlin/dokka/issues/3011#issuecomment-1568620493
+        throw IllegalArgumentException("Module name cannot contain commas as it is used internally as a delimiter.")
+    }
+
     val parser = ArgParser("sourceSet", prefixStyle = ArgParser.OptionPrefixStyle.JVM)
 
     val sourceSetName by parser.option(
diff --git a/runners/maven-plugin/src/main/kotlin/DokkaMojo.kt b/runners/maven-plugin/src/main/kotlin/DokkaMojo.kt
index c82f1b59..64293332 100644
--- a/runners/maven-plugin/src/main/kotlin/DokkaMojo.kt
+++ b/runners/maven-plugin/src/main/kotlin/DokkaMojo.kt
@@ -353,6 +353,12 @@ abstract class AbstractDokkaMojo(private val defaultDokkaPlugins: List<Dependenc
             }
         }
 
+        if (moduleName.contains(',')) {
+            // To figure out why this is needed and if it is still relevant, see the comment here:
+            // https://github.com/Kotlin/dokka/issues/3011#issuecomment-1568620493
+            throw IllegalArgumentException("Module name cannot contain commas as it is used internally as a delimiter.")
+        }
+
         fun defaultLinks(config: DokkaSourceSetImpl): Set<ExternalDocumentationLinkImpl> {
             val links = mutableSetOf<ExternalDocumentationLinkImpl>()
             if (!config.noJdkLink)