Correct an inconsistency with default values and leave a comment of i…

…ntentions
Kotlin · Dec 23, 2021 · 8e6079e · 8e6079e
1 parent bfdcf5d
commit 8e6079e
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/...ase/src/main/kotlin/transformers/documentables/DocumentableVisibilityFilterTransformer.kt b/...ase/src/main/kotlin/transformers/documentables/DocumentableVisibilityFilterTransformer.kt
@@ -40,12 +40,10 @@ class DocumentableVisibilityFilterTransformer(val context: DokkaContext) : PreMe
                 else -> globalOptions.documentedVisibilities to globalOptions.includeNonPublic
             }
 
+            // if `documentedVisibilities` is explicitly overridden by the user (i.e. not default value by reference),
+            // deprecated `includeNonPublic` should not be taken into account, so that only one setting prevails
             val isDocumentedVisibilitiesOverridden = documentedVisibilities !== DokkaDefaults.documentedVisibilities
-            return if (isDocumentedVisibilitiesOverridden) {
-                documentedVisibilities.contains(visibility)
-            } else {
-                visibility == DokkaConfiguration.Visibility.PUBLIC || includeNonPublic
-            }
+            return documentedVisibilities.contains(visibility) || (!isDocumentedVisibilitiesOverridden && includeNonPublic)
         }
 
         fun processModule(original: DModule) =