Add missing return in resolveShippingRates

[GordonLesti]

In the function resolveShippingRates is a missing return statement after selectShippingMethodAction. Without this statement it can happen, that selectShippingMethodAction is called second time.
This leads to the point that selectShippingMethodAction is called with the parameter true and that for example shipping.js returns undefined - undefined as carrier title and method title of the shipping method.

This behavior can be reproduced with

• more than one available shipping method so that ratesData.length is bigger than 1
• select a shipping method in the checkout so that window.checkoutConfig.selectedShippingMethod exists
• go back to cart
• the shipping method in cart-totals will be rendered twice and the second time with Shipping (undefined - undefined)

Edit

Since commit e72482c9fd7eeaac29f9710b6c39c9eafe5ed0b0 the code

if (!availableRate && window.checkoutConfig.selectedShippingMethod) {
    availableRate = true;
    selectShippingMethodAction(window.checkoutConfig.selectedShippingMethod);
}

has changed to

if (!availableRate && window.checkoutConfig.selectedShippingMethod) {
    availableRate = window.checkoutConfig.selectedShippingMethod;
    selectShippingMethodAction(window.checkoutConfig.selectedShippingMethod);
}

and the undefined - undefined problem doesn't occur anymore, but without the return statement the function selectShippingMethodAction is called twice with the same argument and the shipping method is rendered twice without any reason.

[vrann]

@GordonLesti please update the code with the latest mainline

[GordonLesti]

@vrann code is updated with latest mainline.

[vrann]

@GordonLesti successfully merged, thank you!

[GordonLesti]

@vrann thank you