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

Reservation of ReachFrequencyPrediction not working. #92

Closed
monitotxi opened this issue Jun 9, 2015 · 2 comments
Closed

Reservation of ReachFrequencyPrediction not working. #92

monitotxi opened this issue Jun 9, 2015 · 2 comments

Comments

@monitotxi
Copy link

The method ReachFrequencyPrediction::reserve() throws an exception 'Array to string conversion' on line 116.

$response->getContent() returns an associative array with the ID of the prediction, but it's typecasted to a string.

@pruno
Copy link
Contributor

pruno commented Jun 9, 2015

@monitotxi, can you provide a code to reproduce this?

@monitotxi
Copy link
Author

@pruno For an existing prediction (with an existing API instance):

use FacebookAds\Object\ReachFrequencyPrediction;

$prediction_id = 'XXX';
$adaccount_id = 'XXX';
$prediction = new ReachFrequencyPrediction($prediction_id, $adaccount_id);
$prediction->reserve(); //throws the exception

@pruno pruno closed this as completed in c987167 Jun 10, 2015
pruno pushed a commit that referenced this issue Jul 10, 2015
Summary: fix #92 - fix #92

Test Plan:
  (new ReachFrequencyPrediction(<PREDICTION_ID>, <ACCOUNT_ID>))->reserve();
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants