[5.6] Use early returns in else method

[NoelDavies]

Then when method should be using early returns instead of if/else

[browner12]

how does this change anything?

[NoelDavies]

The else is useless as you're already returning in the initial if statement.

Removing the else improves readability if nothing else. Technically it should be a strict comparison, but that would probably break a lot of code that isn't strictly typed

[browner12]

i'm pretty sure both of these iterations operate identically. i see no reason to change it unless you can find some performance improvement, or precedence in other code

[robclancy]

This kind of thing has annoyed me in the past but it is usually like this in the framework and just down to code style and nothing more.