Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bracket errors in GetDeliveryForecastForLineItems #238

Closed
Code-Working opened this issue Feb 14, 2017 · 2 comments
Closed

Bracket errors in GetDeliveryForecastForLineItems #238

Code-Working opened this issue Feb 14, 2017 · 2 comments
Assignees
Labels

Comments

@Code-Working
Copy link

Hello,

I've found an error in GetDeliveryForecastForLineItems examples in both versions 201611 and 201702 in line 42/43:

$forecast = $forecastService->getDeliveryForecastByIds(
    [$lineItemId1, $lineItemId2), new DeliveryForecastOptions()];

I guess this should rather be:

$forecast = $forecastService->getDeliveryForecastByIds(
    [$lineItemId1, $lineItemId2], new DeliveryForecastOptions());

Regards
Code-Working

@vtsao vtsao self-assigned this Feb 14, 2017
@vtsao vtsao added the bug label Feb 14, 2017
@vtsao
Copy link
Contributor

vtsao commented Feb 14, 2017

Thanks for reporting this, will fix it soon.

@fiboknacky
Copy link
Member

This is fixed with v25.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants