Skip to content

Commit

Permalink
Make the package compatible with last google API version (v201702)
Browse files Browse the repository at this point in the history
  • Loading branch information
Edujugon committed Apr 11, 2017
1 parent bdd23fe commit 3d9b6f5
Show file tree
Hide file tree
Showing 13 changed files with 187 additions and 180 deletions.
307 changes: 157 additions & 150 deletions composer.lock

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/GoogleAds.php
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ public function fields()
/**
* Set the google adwords service.
*
* @param \Google\AdsApi\AdWords\v201609\cm\* $service
* @param \Google\AdsApi\AdWords\v201702\cm\* $service
* @return \Edujugon\GoogleAds\Services\Service
*/
public function service($service)
Expand Down
2 changes: 1 addition & 1 deletion src/Reports/Fields.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

use Edujugon\GoogleAds\Session\AdwordsSession;
use Google\AdsApi\AdWords\AdWordsServices;
use Google\AdsApi\AdWords\v201609\cm\ReportDefinitionService;
use Google\AdsApi\AdWords\v201702\cm\ReportDefinitionService;


/**
Expand Down
2 changes: 1 addition & 1 deletion src/Reports/Format.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@


use Edujugon\GoogleAds\Exceptions\ReportFormat;
use Google\AdsApi\AdWords\Reporting\v201609\DownloadFormat;
use Google\AdsApi\AdWords\Reporting\v201702\DownloadFormat;
use ReflectionClass;

abstract class Format
Expand Down
4 changes: 2 additions & 2 deletions src/Reports/Report.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@


use Edujugon\GoogleAds\Session\AdwordsSession;
use Google\AdsApi\AdWords\Reporting\v201609\ReportDownloader;
use Google\AdsApi\AdWords\v201609\cm\ApiException;
use Google\AdsApi\AdWords\Reporting\v201702\ReportDownloader;
use Google\AdsApi\AdWords\v201702\cm\ApiException;

class Report
{
Expand Down
2 changes: 1 addition & 1 deletion src/Reports/ReportTypes.php
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?php
namespace Edujugon\GoogleAds\Reports;

use Google\AdsApi\AdWords\v201609\cm\ReportDefinitionReportType;
use Google\AdsApi\AdWords\v201702\cm\ReportDefinitionReportType;
use Edujugon\GoogleAds\Exceptions\ReportTypes as ReportException;
use ReflectionClass;

Expand Down
2 changes: 1 addition & 1 deletion src/Services/AdGroup.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@


use Google\AdsApi\AdWords\AdWordsSession;
use Google\AdsApi\AdWords\v201609\cm\AdGroupService;
use Google\AdsApi\AdWords\v201702\cm\AdGroupService;

class AdGroup extends Service
{
Expand Down
2 changes: 1 addition & 1 deletion src/Services/AdGroupAd.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@


use Google\AdsApi\AdWords\AdWordsSession;
use Google\AdsApi\AdWords\v201609\cm\AdGroupAdService;
use Google\AdsApi\AdWords\v201702\cm\AdGroupAdService;

class AdGroupAd extends Service
{
Expand Down
2 changes: 1 addition & 1 deletion src/Services/Campaign.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
namespace Edujugon\GoogleAds\Services;

use Google\AdsApi\AdWords\AdWordsSession;
use Google\AdsApi\AdWords\v201609\cm\CampaignService;
use Google\AdsApi\AdWords\v201702\cm\CampaignService;

class Campaign extends Service
{
Expand Down
3 changes: 0 additions & 3 deletions src/Services/Service.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,6 @@
use Edujugon\GoogleAds\Exceptions\Session;
use Edujugon\GoogleAds\Session\AdwordsSession;
use Google\AdsApi\AdWords\AdWordsServices;
use Google\AdsApi\AdWords\v201609\cm\AdGroupAdService;
use Google\AdsApi\AdWords\v201609\cm\AdGroupService;
use Google\AdsApi\AdWords\v201609\cm\CampaignService;

class Service
{
Expand Down
12 changes: 6 additions & 6 deletions src/Services/ServiceCollection.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,12 @@

namespace Edujugon\GoogleAds\Services;

use Google\AdsApi\AdWords\v201609\cm\AdGroupAdOperation;
use Google\AdsApi\AdWords\v201609\cm\AdGroupAdService;
use Google\AdsApi\AdWords\v201609\cm\AdGroupOperation;
use Google\AdsApi\AdWords\v201609\cm\AdGroupService;
use Google\AdsApi\AdWords\v201609\cm\CampaignOperation;
use Google\AdsApi\AdWords\v201609\cm\CampaignService;
use Google\AdsApi\AdWords\v201702\cm\AdGroupAdOperation;
use Google\AdsApi\AdWords\v201702\cm\AdGroupAdService;
use Google\AdsApi\AdWords\v201702\cm\AdGroupOperation;
use Google\AdsApi\AdWords\v201702\cm\AdGroupService;
use Google\AdsApi\AdWords\v201702\cm\CampaignOperation;
use Google\AdsApi\AdWords\v201702\cm\CampaignService;
use Illuminate\Support\Collection;

class ServiceCollection
Expand Down
4 changes: 2 additions & 2 deletions tests/GoogleAdsTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@

use Edujugon\GoogleAds\GoogleAds;
use Edujugon\GoogleAds\Reports\MyReport;
use Google\AdsApi\AdWords\v201609\cm\AdGroupService;
use Google\AdsApi\AdWords\v201609\cm\CampaignService;
use Google\AdsApi\AdWords\v201702\cm\AdGroupService;
use Google\AdsApi\AdWords\v201702\cm\CampaignService;

class GoogleAdsTest extends PHPUnit_Framework_TestCase {

Expand Down
23 changes: 13 additions & 10 deletions tests/ServicesTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
use Google\AdsApi\AdWords\v201609\cm\BudgetBudgetDeliveryMethod;
use Google\AdsApi\AdWords\v201609\cm\BudgetOperation;
use Google\AdsApi\AdWords\v201609\cm\CampaignOperation;
use Google\AdsApi\AdWords\v201609\cm\CampaignService;
use Google\AdsApi\AdWords\v201702\cm\CampaignService;
use Google\AdsApi\AdWords\v201609\cm\Money;
use Google\AdsApi\AdWords\v201609\cm\Operator;

Expand All @@ -22,7 +22,7 @@ class ServicesTest extends PHPUnit_Framework_TestCase {
public function add_service()
{
$service = (new \Edujugon\GoogleAds\Services\Service(CampaignService::class));
$this->assertInstanceOf(\Google\AdsApi\AdWords\v201609\cm\CampaignService::class,$service->getService());
$this->assertInstanceOf(\Google\AdsApi\AdWords\v201702\cm\CampaignService::class,$service->getService());
}


Expand Down Expand Up @@ -117,15 +117,18 @@ public function update_campaign_name()
$results = $campaignService->select('CampaignId','CampaignName')->get();

$campaign = $results->where('id',$this->testedCampaignId)->set('name','hello !!');
if(! $campaign->isEmpty()){

$campaign = $campaign->save();
$changed = $campaign->save();

if($campaign)
{
$this->assertInstanceOf(\Google\AdsApi\AdWords\v201609\cm\CampaignReturnValue::class,$campaign);
if($changed)
{
$this->assertInstanceOf(\Google\AdsApi\AdWords\v201702\cm\CampaignReturnValue::class,$changed);

$this->assertEquals('hello !!',$campaign->getValue()[0]->getName());
$this->assertEquals('hello !!',$changed->getValue()[0]->getName());
}
}

}

/** @test */
Expand All @@ -134,16 +137,16 @@ public function create_new_campaign()
// $campaignService = (new \Edujugon\GoogleAds\Services\Service(CampaignService::class))->getService();
//
// //Create the campaign
// $campaign = new \Google\AdsApi\AdWords\v201609\cm\Campaign();
// $campaign = new \Google\AdsApi\AdWords\v201702\cm\Campaign();
// $campaign->setName('My first campaign');
// $campaign->setStatus(\Google\AdsApi\AdWords\v201609\cm\CampaignStatus::PAUSED);
// $campaign->setStatus(\Google\AdsApi\AdWords\v201702\cm\CampaignStatus::PAUSED);
//
// $biddingStrategyConfiguration = new BiddingStrategyConfiguration();
// $biddingStrategyConfiguration->setBiddingStrategyType(BiddingStrategyType::MANUAL_CPC);
// $campaign->setBiddingStrategyConfiguration($biddingStrategyConfiguration);
//
// //Budget
// $budgetService = (new \Edujugon\GoogleAds\Services\Service(\Google\AdsApi\AdWords\v201609\cm\BudgetService::class))->getService();
// $budgetService = (new \Edujugon\GoogleAds\Services\Service(\Google\AdsApi\AdWords\v201702\cm\BudgetService::class))->getService();
// $sharedBudget = new Budget();
// $budget = new Budget();
//
Expand Down

0 comments on commit 3d9b6f5

Please sign in to comment.