[5.4] Refactors if structure

[regularlabs]

Refactors the if structure in Parser::parameters by using an early return (continue in foreach).
This eliminates the use of the else keyword.

[deleugpn]

Is continue really considered early return approach to replace else? It seems like swapping 6 for half a dozen.

[regularlabs]

It has the same logic/reasoning behind it.
You 'exit' the code because what is following it is not applicable anymore.

Compare these 2 approaches.
Loop with an early continue:

foreach ($items as $item) {
    if (! passSomeCheck() ) {
        // do something

        continue;
    }

    // do something else
}

Code extracted to a separate function with an early return:
(This is a better approach when you want to reuse code and/or have more code to do inside the loop)

foreach ($items as $item) {
    doSomething($item)
}

function doSomething($item) {
    if (! passSomeCheck() ) {
        // do something

        return;
    }

    // do something else
}

[taylorotwell]

Unless it fixes a bug I don't typically merge these kinds of changes.