Fix: Do not attempt to return something from methods that do not return anything

[localheinz]

This pull request

• stops attempting to return something from methods that do not return anything

[valorin]

I'm going to leave this one open for a bit and think about it.

I've fixed the second return in the main branch, it's completely redundant (it was an artifact of extracting the method in PhpStorm I think), but I actually prefer the

if (condition) {
    return doSomething;
}

style much more than breaking out the return into a separate line. I find it adds unnecessary visual noise. But I do understand the argument that it's returning something where there is nothing to return.