Wrong getter name and type for boolean properties in Javadoc

[ltennstedt]

Describe the bug
Even if a property's return value is of a primitive type, the boxed wrapper type will be shown in Javadoc generated by Dokka, and in case of boolean the getter name in Javadoc is prefixed mistakenly with get (getIsDyadic()).

Expected behaviour
The primitve type and the correct name should be shown in Javadoc.

To Reproduce
https://github.com/ltennstedt/finnmath

Dokka configuration

dokkaJavadoc {
    dokkaSourceSets {
        configureEach {
            jdkVersion.set(8)
            noJdkLink.set(true)
            noStdlibLink.set(true)
        }
    }
}

Installation

• Operating system: Ubuntu 20.04 LTS
• Build tool: Gradle v7.1
• Dokka version: 1.5.0

Are you willing to provide a PR?
Maybe with your help.

[MarcinAman]

The issue is here quite different:
when you use a kotlin property from java you use it via getX or isX depending on the type of the variable. Here the bug is that we blindly use getX instead of is convention. Also in this case property in kotlin already is prefixed with is so it should be left "as is".

If you would like to fix it bug probably could be fixed somewhere here:

dokka/plugins/kotlin-as-java/src/main/kotlin/converters/KotlinToJavaConverter.kt

Line 128 in 93da477

val name = "get" + name.capitalize()

[ltennstedt]

@MarcinAman I'm going to take a closer look at this this weekend and if I'm successful, I'll make a PR.

[IgnatBeresnev]

I had a look at this recently - you'd think the bug would in KotlinToJavaConverter, but no. I found it here:

dokka/plugins/base/src/main/kotlin/translators/descriptors/DefaultDescriptorToDocumentableTranslator.kt

Line 620 in 62f2919

val modifier = if (isGetter) "get" else "set"

but I guess the converter also needs to be checked and corrected

[IgnatBeresnev]

Fixed in #2356, should be included in the nearest release (1.6.20)