AdSense Host API v4.1 PHP sample code

AdSense Host API v4.1 PHP sample code

[chirags, 2012-09-19 17:12:12]

LGTM! The generated classes should now be prefixed with Google_. Let me know if
you need help regenerating the classes.

https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php
File examples/AddAdUnitToPublisher.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.p...
examples/AddAdUnitToPublisher.php:35: $adUnit = new AdUnit();
Should this be Google_AdUnit?

https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
File examples/AddCustomChannelToHost.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
examples/AddCustomChannelToHost.php:33: $customChannel = new CustomChannel();
This should be Google_CustomChannel

https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
File examples/UpdateAdUnitOnPublisher.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
examples/UpdateAdUnitOnPublisher.php:41: $contentAdsSettings = new
AdUnitContentAdsSettings();
Google_AdUnitContentAdsSettings

[sgomes_google, 2012-09-19 17:28:15]

Thanks, Chirag! Could you perhaps generate Google_AdsensehostService.php for me,
so that I can use a canonical version? The API is public now, so we can host the
file.

On 2012/09/19 17:12:12, chirags wrote:
> LGTM! The generated classes should now be prefixed with Google_. Let me know
if
> you need help regenerating the classes.
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php
> File examples/AddAdUnitToPublisher.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.p...
> examples/AddAdUnitToPublisher.php:35: $adUnit = new AdUnit();
> Should this be Google_AdUnit?
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> File examples/AddCustomChannelToHost.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> examples/AddCustomChannelToHost.php:33: $customChannel = new CustomChannel();
> This should be Google_CustomChannel
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> File examples/UpdateAdUnitOnPublisher.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> examples/UpdateAdUnitOnPublisher.php:41: $contentAdsSettings = new
> AdUnitContentAdsSettings();
> Google_AdUnitContentAdsSettings

[chirags, 2012-09-19 17:39:53]

I just published the latest version of contrib/Google_AdsenseService.php
and contrib/Google_AdsensehostService.php.

Available on TRUNK as of r494.

On Wed, Sep 19, 2012 at 10:28 AM, <sgomes@google.com> wrote:

> Thanks, Chirag! Could you perhaps generate Google_AdsensehostService.php
> for me, so that I can use a canonical version? The API is public now, so
> we can host the file.
>
>
> On 2012/09/19 17:12:12, chirags wrote:
>
>> LGTM! The generated classes should now be prefixed with Google_. Let
>>
> me know if
>
>> you need help regenerating the classes.
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
AddAdUnitToPublisher.php<https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php>
>
>> File examples/AddAdUnitToPublisher.**php (right):
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
AddAdUnitToPublisher.php#**newcode35<https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php#newcode35>
>
>> examples/AddAdUnitToPublisher.**php:35: $adUnit = new AdUnit();
>> Should this be Google_AdUnit?
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
AddCustomChannelToHost.php<https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost.php>
>
>> File examples/**AddCustomChannelToHost.php (right):
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
AddCustomChannelToHost.php#**newcode33<https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost.php#newcode33>
>
>> examples/**AddCustomChannelToHost.php:33: $customChannel = new
>>
> CustomChannel();
>
>> This should be Google_CustomChannel
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
UpdateAdUnitOnPublisher.php<https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublisher.php>
>
>> File examples/**UpdateAdUnitOnPublisher.php (right):
>>
>
>
> https://codereview.appspot.**com/6535054/diff/1/examples/**
>
UpdateAdUnitOnPublisher.php#**newcode41<https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublisher.php#newcode41>
>
>> examples/**UpdateAdUnitOnPublisher.php:**41: $contentAdsSettings = new
>> AdUnitContentAdsSettings();
>> Google_**AdUnitContentAdsSettings
>>
>
>
>
https://codereview.appspot.**com/6535054/<https://codereview.appspot.com/6535...
>

[sgomes_google, 2012-09-20 13:28:08]

Fixed everything to match new namespaces/class names.

https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php
File examples/AddAdUnitToPublisher.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.p...
examples/AddAdUnitToPublisher.php:35: $adUnit = new AdUnit();
On 2012/09/19 17:12:13, chirags wrote:
> Should this be Google_AdUnit?

Done.

https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
File examples/AddCustomChannelToHost.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
examples/AddCustomChannelToHost.php:33: $customChannel = new CustomChannel();
On 2012/09/19 17:12:13, chirags wrote:
> This should be Google_CustomChannel

Done.

https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
File examples/UpdateAdUnitOnPublisher.php (right):

https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
examples/UpdateAdUnitOnPublisher.php:41: $contentAdsSettings = new
AdUnitContentAdsSettings();
On 2012/09/19 17:12:13, chirags wrote:
> Google_AdUnitContentAdsSettings

Done.

[sgomes_google, 2012-09-25 11:13:21]

Hey Chirag,

Do the latest changes look good? I'd like to get this into the repository as
soon as possible.

Thanks!

--Sérgio

---
On 2012/09/20 13:28:08, sgomes_google wrote:
> Fixed everything to match new namespaces/class names.
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php
> File examples/AddAdUnitToPublisher.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.p...
> examples/AddAdUnitToPublisher.php:35: $adUnit = new AdUnit();
> On 2012/09/19 17:12:13, chirags wrote:
> > Should this be Google_AdUnit?
> 
> Done.
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> File examples/AddCustomChannelToHost.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> examples/AddCustomChannelToHost.php:33: $customChannel = new CustomChannel();
> On 2012/09/19 17:12:13, chirags wrote:
> > This should be Google_CustomChannel
> 
> Done.
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> File examples/UpdateAdUnitOnPublisher.php (right):
> 
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> examples/UpdateAdUnitOnPublisher.php:41: $contentAdsSettings = new
> AdUnitContentAdsSettings();
> On 2012/09/19 17:12:13, chirags wrote:
> > Google_AdUnitContentAdsSettings
> 
> Done.

[chirags, 2012-09-25 21:52:28]

LGTM. Thank you!

Also, you should have commit access to the PHP client.

On 2012/09/25 11:13:21, sgomes_google wrote:
> Hey Chirag,
> 
> Do the latest changes look good? I'd like to get this into the repository as
> soon as possible.
> 
> Thanks!
> 
> --Sérgio
> 
> ---
> On 2012/09/20 13:28:08, sgomes_google wrote:
> > Fixed everything to match new namespaces/class names.
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.php
> > File examples/AddAdUnitToPublisher.php (right):
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/AddAdUnitToPublisher.p...
> > examples/AddAdUnitToPublisher.php:35: $adUnit = new AdUnit();
> > On 2012/09/19 17:12:13, chirags wrote:
> > > Should this be Google_AdUnit?
> > 
> > Done.
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> > File examples/AddCustomChannelToHost.php (right):
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/AddCustomChannelToHost...
> > examples/AddCustomChannelToHost.php:33: $customChannel = new
CustomChannel();
> > On 2012/09/19 17:12:13, chirags wrote:
> > > This should be Google_CustomChannel
> > 
> > Done.
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> > File examples/UpdateAdUnitOnPublisher.php (right):
> > 
> >
>
https://codereview.appspot.com/6535054/diff/1/examples/UpdateAdUnitOnPublishe...
> > examples/UpdateAdUnitOnPublisher.php:41: $contentAdsSettings = new
> > AdUnitContentAdsSettings();
> > On 2012/09/19 17:12:13, chirags wrote:
> > > Google_AdUnitContentAdsSettings
> > 
> > Done.