Get beta version of this client library to stable.

[vtsao]

Re-purposing this as a tracking bug for getting the beta version of this client library to stable.

Blocked by:

• Add revocation methods to OAuth2Handler #68
• Service account support #77
• phpDoc is wrong everywhere #86
• Feature request: revoke access token example #115
• Report download with experimental #120
• add GetRefreshToken script for AdWords #124
• ErrorException: the 'CN_match' SSL context option is deprecated in favor of 'peer_name' #125
• Namespace library and conform to PSR-4. #138
• Add BatchJobService support to beta client library. #145

[thierrymarianne]

Hey @vtsao, we have started to revise classes in this regard : https://github.com/theodo/adwords

[jxn]

I agree on PSR-0. Note also that PSR-4 has now been approved as well: https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader.md

[onema]

👍 Some PSR-0/PSR-4 love please!

[raul782]

That would be awesome, I'm deploying apps with php 5.5, and going back to use code w/o namespaces or compatible with 5.2 is a no no, Even php officially dx http://php.net/eol.php, 5.2 three years ago.
We should push towards PSR-0/1/2 at least and re-organize classes with their proper namespaces.
Maybe start a new repo and leave this as legacy code.

[saturnism]

hey guys, have you all checked out the experimental branch w/ full namespace support?

[thierrymarianne]

Hey @saturnism, I did and I was pretty impressed by the good job. However, I had the feeling it was not totally compliant with latest autoloading standards (but I may be wrong about the latter):

see https://github.com/googleads/googleads-php-lib/blob/experimental/src/Google/Api/Ads/AdWords/v201402/AdGroupAdService.php#L31

ping @davinov

[vtsao]

Thanks, we are still working on the experimental branch and deciding the path we want to take moving forward with it.