Ktlint: Need to wrap when statements that use an assignment if/else with a lambda

Created on 10 Sep 2020 · 2Comments · Source: pinterest/ktlint

Expected Behavior

I expect the following to pass formatting.

fun broken(letter: Letter): String? = when (letter) {
    Letter.A -> if (hasPrivilegedStatePermission) {
        "subscriberId"
    } else {
        throw SecurityException(
            "does not have privileged phone permission"
        )
    }
    Letter.B -> if (hasPrivilegedStatePermission) {
        "subscriberId"
    } else {
        null
    }
}

Observed Behavior

Fails with: Missing newline after "->" (indent). Wrapping with { .. } fixes the issue

Steps to Reproduce

enum class Letter(val value: String) {
    A("a"),
    B("b");
}
val hasPrivilegedStatePermission = true
fun broken(letter: Letter): String? = when (letter) {
    Letter.A -> if (hasPrivilegedStatePermission) {
        "subscriberId"
    } else {
        throw SecurityException(
            "does not have privileged phone permission"
        )
    }
    Letter.B -> if (hasPrivilegedStatePermission) {
        "subscriberId"
    } else {
        null
    }
}
fun fixed(letter: Letter): String? = when (letter) {
    Letter.A -> {
        if (hasPrivilegedStatePermission) {
            "subscriberId"
        } else {
            throw SecurityException(
                "does not have privileged phone permission"
            )
        }
    }
    Letter.B -> {
        if (hasPrivilegedStatePermission) {
            "subscriberId"
        } else {
            null
        }
    }
}