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

PHP Warning: Invalid argument supplied for foreach #527

Closed
notian opened this issue Sep 5, 2018 · 3 comments
Closed

PHP Warning: Invalid argument supplied for foreach #527

notian opened this issue Sep 5, 2018 · 3 comments
Assignees
Labels

Comments

@notian
Copy link

notian commented Sep 5, 2018

PHP Warning when getEntries returns null instead of empty array:

Can be duplicated with this example:
https://developers.google.com/adwords/api/docs/samples/php/optimization#get-a-bid-landscape-for-an-ad-group-and-criterion

PHP Warning: Invalid argument supplied for foreach() in ..../vendor/googleads/googleads-php-lib/src/Google/AdsApi/AdWords/Query/v201806/ServiceQuery.php on line 135
PHP Stack trace:

PHP 6. Google\AdsApi\Examples\AdWords\v201806\Optimization\GetKeywordBidSimulations::main() ..../Testing.php:117
PHP 7. Google\AdsApi\Examples\AdWords\v201806\Optimization\GetKeywordBidSimulations::runExample() ..../Testing.php:113
PHP 8. Google\AdsApi\AdWords\Query\v201806\ServiceQuery->hasNext() ..../Testing.php:108
PHP 9. Google\AdsApi\AdWords\Query\v201806\ServiceQuery::countLandscapePoints() ..../vendor/googleads/googleads-php-lib/src/Google/AdsApi/AdWords/Query/v201806/ServiceQuery.php:211

@thangduo
Copy link
Contributor

thangduo commented Sep 6, 2018

Thanks for reporting this bug. It will be fixed in the next release.

@thangduo thangduo closed this as completed Sep 6, 2018
@thangduo
Copy link
Contributor

thangduo commented Sep 6, 2018

Reopening the issue until next release. Once the fix goes live, this issue will be marked as closed.

@thangduo thangduo reopened this Sep 6, 2018
@fiboknacky
Copy link
Member

This is fixed in the new release.

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