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

DemographicSearchClasses namespace causes autoloader problems. #7

Closed
neovdr opened this issue Jun 24, 2014 · 5 comments
Closed

DemographicSearchClasses namespace causes autoloader problems. #7

neovdr opened this issue Jun 24, 2014 · 5 comments

Comments

@neovdr
Copy link
Contributor

neovdr commented Jun 24, 2014

The DemographicsSearchClasses.php file in src/FacebookAds/Object/Search/ has the namespace:

namespace FacebookAds\Targeting;

This causes problems for the autoloader and on execution of code that uses it you get the error "Fatal error: Class 'FacebookAds\Targeting\DemographicSearchClasses' not found in...".

However when I change the namespace in DemographicSearchClasses to match the folder layout like so: namespace FacebookAds\Object\Search; the problems go away!

Is this a bug or is it something that I am doing wrong?

@paulbain
Copy link
Contributor

It's a bug, I can fix it.

@neovdr
Copy link
Contributor Author

neovdr commented Jun 24, 2014

Thank you Paul!

@paulbain
Copy link
Contributor

Should be good to go with the release 0.9.3

@neovdr
Copy link
Contributor Author

neovdr commented Jun 24, 2014

Excellent! It works when I put in composer.json 'dev-master' as the version. Packagist doesn't seem to have the 0.9.3 release. Maybe the auto-update on packagist is not enabled??

@paulbain
Copy link
Contributor

Yes we haven't got webhooks setup yet. We will hopefully soon.

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