A modern programming language that makes developers happier.

kotlinlang

<@U5UU34LPK>
```
val method = result.jsonObject?.get("result")?.asString
            ?: throwInsertionError(result)
        return when (method) {
```

Hm... I'd rewrite it as
```
private fun ResultType.jsonResultAsString() = jsonObject?.get("result")?.asString
           ?: throwInsertionError(result)
//...
return when (val method = result.jsonResultAsString()) {
```
Where `val method =` part is only needed if you're using `method` in branches' bodies

The inspection you mention has been useful to me on numerous occasions, so it is use-case dependent. Sometimes the refactoring I suggested above is not useful or possible, so, yeah, on those occasions the inspection is not welcome. But in general if I see that, it is usually and indication that relevant code can be polished further.

and depending on what happens in the `when (method)`, IntelliJ may be suggesting that it all get rolled into one `when` statement…

This will be fixed in 1.3.40: <https://youtrack.jetbrains.com/issue/KT-30457>