schulzefelix / laravel-stat-search-analytics

src/Api/StatBulk.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace SchulzeFelix\Stat\Api;
 
 use Carbon\Carbon;
-use Illuminate\Support\Collection;
-use Illuminate\Support\Facades\Cache;
 use SchulzeFelix\Stat\Exceptions\ApiException;
 use SchulzeFelix\Stat\Objects\StatBulkJob;
 use SchulzeFelix\Stat\Objects\StatKeyword;

Spacing +37 added lines, -37 removed lines

@@ -25,19 +25,19 @@ 
 
         $bulkJobs = collect();
 
-        if($response['resultsreturned'] == 0) {
+        if ($response['resultsreturned'] == 0) {
             return $bulkJobs;
         }
 
-        if($response['resultsreturned'] == 1) {
+        if ($response['resultsreturned'] == 1) {
             $bulkJobs->push($response['Result']);
         }
 
-        if($response['resultsreturned'] > 1) {
+        if ($response['resultsreturned'] > 1) {
             $bulkJobs = collect($response['Result']);
         }
 
-        return $bulkJobs->transform(function ($job, $key) {
+        return $bulkJobs->transform(function($job, $key) {
 
             return new StatBulkJob([
                 'id' => $job['Id'],
@@ -64,7 +64,7 @@ 
      * @param bool $crawledKeywordsOnly This parameter causes the API to only include output for keywords that were crawled on the date parameter provided.
      * @return int
      */
-    public function ranks(Carbon $date, array $sites = null, $rankType = 'highest', $engines = null, bool $currentlyTrackedOnly = false , bool $crawledKeywordsOnly = false)
+    public function ranks(Carbon $date, array $sites = null, $rankType = 'highest', $engines = null, bool $currentlyTrackedOnly = false, bool $crawledKeywordsOnly = false)
     {
         $this->validateBulkDate($date);
 
@@ -73,15 +73,15 @@ 
         $arguments['currently_tracked_only'] = $currentlyTrackedOnly;
         $arguments['crawled_keywords_only'] = $crawledKeywordsOnly;
 
-        if( ! is_null($sites) && count($sites) > 0){
-            $arguments['site_id'] = implode(',', $sites);;
+        if (!is_null($sites) && count($sites) > 0) {
+            $arguments['site_id'] = implode(',', $sites); ;
         }
-        if( ! is_null($engines) && count($engines) > 0){
+        if (!is_null($engines) && count($engines) > 0) {
             $arguments['engines'] = implode(',', $engines);
         }
         $response = $this->performQuery('bulk/ranks', $arguments);
 
-        return (int) $response['Result']['Id'];
+        return (int)$response['Result']['Id'];
     }
 
     public function status($bulkJobID)
@@ -95,9 +95,9 @@ 
         $jobStatus->date = $response['Result']['Date'];
 
         $jobStatus->sites = collect();
-        if(isset($response['Result']['SiteId'])){
-            $jobStatus->sites = collect( explode(',', $response['Result']['SiteId']))
-                                ->transform(function ($site, $key) {
+        if (isset($response['Result']['SiteId'])) {
+            $jobStatus->sites = collect(explode(',', $response['Result']['SiteId']))
+                                ->transform(function($site, $key) {
                                     return (int)$site;
                                 });
         }
@@ -116,7 +116,7 @@ 
     {
         $response = $this->performQuery('bulk/delete', ['id' => $bulkJobID]);
 
-        return (int) $response['Result']['Id'];
+        return (int)$response['Result']['Id'];
     }
 
     public function siteRankingDistributions($date)
@@ -124,7 +124,7 @@ 
         $this->validateBulkDate($date);
 
         $response = $this->performQuery('bulk/site_ranking_distributions', ['date' => $date->toDateString()]);
-        return (int) $response['Result']['Id'];
+        return (int)$response['Result']['Id'];
     }
 
     public function tagRankingDistributions($date)
@@ -132,14 +132,14 @@ 
         $this->validateBulkDate($date);
 
         $response = $this->performQuery('bulk/tag_ranking_distributions', ['date' => $date->toDateString()]);
-        return (int) $response['Result']['Id'];
+        return (int)$response['Result']['Id'];
     }
 
     public function get($bulkJobID)
     {
         $bulkStatus = $this->status($bulkJobID);
 
-        if($bulkStatus['status'] != 'Completed') {
+        if ($bulkStatus['status'] != 'Completed') {
             throw ApiException::resultError('Bulk Job is not completed. Current status: ' . $bulkJobID['status'] . '.');
         }
 
@@ -164,7 +164,7 @@ 
     {
         $projects = $this->getCollection($bulkStream['Project']);
 
-        $projects->transform(function ($project, $key) {
+        $projects->transform(function($project, $key) {
             return $this->transformProject($project);
         });
 
@@ -179,16 +179,16 @@ 
         $transformedProject->total_sites = $project['TotalSites'];
         $transformedProject->created_at = $project['CreatedAt'];
 
-        if( $project['TotalSites'] == 0) {
+        if ($project['TotalSites'] == 0) {
             $transformedProject->sites = collect();
         }
-        if( $project['TotalSites'] == 1) {
+        if ($project['TotalSites'] == 1) {
             $transformedProject->sites = collect([$project['Site']]);
         }
-        if( $project['TotalSites'] > 1) {
+        if ($project['TotalSites'] > 1) {
             $transformedProject->sites = collect($project['Site']);
         }
-        $transformedProject->sites->transform(function ($site, $key) {
+        $transformedProject->sites->transform(function($site, $key) {
             return $this->transformSite($site);
         });
 
@@ -204,19 +204,19 @@ 
         $transformedSite->total_keywords = $site['TotalKeywords'];
         $transformedSite->created_at = $site['CreatedAt'];
 
-        if(array_key_exists('Keyword', $site)){
+        if (array_key_exists('Keyword', $site)) {
 
-            $transformedSite->keywords = collect($site['Keyword'])->transform(function ($keyword, $key) {
+            $transformedSite->keywords = collect($site['Keyword'])->transform(function($keyword, $key) {
                 return $this->transformKeyword($keyword);
             });
         }
 
-        if(array_key_exists('RankDistribution', $site)){
+        if (array_key_exists('RankDistribution', $site)) {
             $transformedSite->rank_distribution = $this->transformRankDistribution($site['RankDistribution']);
         }
 
-        if(array_key_exists('Tag', $site)){
-            $transformedSite->tags = $this->getCollection($site['Tag'])->transform(function ($tag, $key) {
+        if (array_key_exists('Tag', $site)) {
+            $transformedSite->tags = $this->getCollection($site['Tag'])->transform(function($tag, $key) {
                 return $this->transformTag($tag);
             });
         }
@@ -235,16 +235,16 @@ 
         $modifiedKeyword->keyword_device = $keyword['KeywordDevice'];
         $modifiedKeyword->keyword_categories = $keyword['KeywordCategories'];
 
-        if(is_null($keyword['KeywordTags'])) {
+        if (is_null($keyword['KeywordTags'])) {
             $modifiedKeyword->keyword_tags = collect();
         } else {
             $modifiedKeyword->keyword_tags = collect(explode(',', $keyword['KeywordTags']));
         }
 
-        if( is_null($keyword['KeywordStats']) ) {
+        if (is_null($keyword['KeywordStats'])) {
             $modifiedKeyword->keyword_stats = null;
         } else {
-            $localTrends = collect($keyword['KeywordStats']['LocalSearchTrendsByMonth'])->map(function ($searchVolume, $month){
+            $localTrends = collect($keyword['KeywordStats']['LocalSearchTrendsByMonth'])->map(function($searchVolume, $month) {
                 return new StatLocalSearchTrend([
                     'month' => strtolower($month),
                     'search_volume' => ($searchVolume == '-') ? null : $searchVolume,
@@ -267,15 +267,15 @@ 
             'type' => $keyword['Ranking']['type']
         ]);
 
-        if(array_key_exists('Google', $keyword['Ranking'])){
+        if (array_key_exists('Google', $keyword['Ranking'])) {
             $modifiedKeyword->ranking->google = $this->analyzeRanking($keyword['Ranking']['Google'], $keyword['Ranking']['type']);
         }
 
-        if(array_key_exists('Yahoo', $keyword['Ranking'])){
+        if (array_key_exists('Yahoo', $keyword['Ranking'])) {
             $modifiedKeyword->ranking->yahoo = $this->analyzeRanking($keyword['Ranking']['Yahoo'], $keyword['Ranking']['type']);
         }
 
-        if(array_key_exists('Bing', $keyword['Ranking'])){
+        if (array_key_exists('Bing', $keyword['Ranking'])) {
             $modifiedKeyword->ranking->bing = $this->analyzeRanking($keyword['Ranking']['Bing'], $keyword['Ranking']['type']);
         }
 
@@ -284,17 +284,17 @@ 
 
     private function analyzeRanking($rankingForEngine, $rankingType)
     {
-        if($rankingType == 'highest') {
+        if ($rankingType == 'highest') {
             return $this->transformRanking($rankingForEngine);
         }
 
-        if(is_null($rankingForEngine['Result'])){
+        if (is_null($rankingForEngine['Result'])) {
             return null;
         }
 
         $rankings = $this->getCollection($rankingForEngine['Result'], 'Rank');
 
-        $rankings->transform(function($ranking, $key){
+        $rankings->transform(function($ranking, $key) {
             return $this->transformRanking($ranking);
         });
 
@@ -306,7 +306,7 @@ 
     {
         $transformedRanking = new StatKeywordEngineRanking();
         $transformedRanking->rank = $ranking['Rank'];
-        if(array_key_exists('BaseRank', $ranking)){
+        if (array_key_exists('BaseRank', $ranking)) {
             $transformedRanking->base_rank = $ranking['BaseRank'];
         }
         $transformedRanking->url = $ranking['Url'];
@@ -320,7 +320,7 @@ 
         $modifiedTag->id = $tag['Id'];
         $modifiedTag->tag = $tag['Tag'];
 
-        if(isset($tag['RankDistribution'])){
+        if (isset($tag['RankDistribution'])) {
             $modifiedTag->rank_distribution = $this->transformRankDistribution($tag['RankDistribution']);
         } else {
             $modifiedTag->rank_distribution = null;

src/Api/StatKeywords.php

Doc Comments +2 added lines, -2 removed lines

@@ -122,7 +122,7 @@ 
 
     /**
      * @param $keyword
-     * @return mixed
+     * @return StatKeyword
      */
     protected function transformCreatedKeyword($keyword) {
 
@@ -139,7 +139,7 @@ 
 
     /**
      * @param $keyword
-     * @return mixed
+     * @return StatKeyword
      */
     protected function transformListedKeyword($keyword) {
         $modifiedKeyword = new StatKeyword();

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace SchulzeFelix\Stat\Api;
 
-use Carbon\Carbon;
 use Illuminate\Support\Collection;
 use SchulzeFelix\Stat\Objects\StatKeyword;
 use SchulzeFelix\Stat\Objects\StatKeywordEngineRanking;

Spacing +17 added lines, -17 removed lines

@@ -22,14 +22,14 @@ 
         $keywords = collect();
 
         do {
-            $response = $this->performQuery('keywords/list', ['site_id' => $siteID, 'start' => $start, 'results' => 5000 ]);
+            $response = $this->performQuery('keywords/list', ['site_id' => $siteID, 'start' => $start, 'results' => 5000]);
             $start += 5000;
 
-            if($response['totalresults'] == 0) {
+            if ($response['totalresults'] == 0) {
                 break;
             }
 
-            if(isset($response['Result']['Id'])) {
+            if (isset($response['Result']['Id'])) {
                 $keywords->push($response['Result']);
             }
 
@@ -42,7 +42,7 @@ 
         } while ($response['resultsreturned'] < $response['totalresults']);
 
 
-        $keywords = $keywords->transform(function ($keyword, $key) {
+        $keywords = $keywords->transform(function($keyword, $key) {
                 return $this->transformListedKeyword($keyword);
         });
 
@@ -65,30 +65,30 @@ 
         $arguments['market'] = $market;
         $arguments['device'] = $device;
         $arguments['type'] = 'regular';
-        $arguments['keyword'] = implode(',', array_map(function ($el) {
+        $arguments['keyword'] = implode(',', array_map(function($el) {
                                     return str_replace(',', '\,', $el);
                                 }, $keywords));
 
-        if( ! is_null($tags) && count($tags) > 0)
+        if (!is_null($tags) && count($tags) > 0)
             $arguments['tag'] = implode(',', $tags);
 
-        if( ! is_null($location) && $location != '')
+        if (!is_null($location) && $location != '')
             $arguments['location'] = $location;
 
         $response = $this->performQuery('keywords/create', $arguments);
 
         $keywords = collect();
 
-        if($response['resultsreturned'] == 0) {
+        if ($response['resultsreturned'] == 0) {
             return $keywords;
         }
-        if($response['resultsreturned'] == 1) {
+        if ($response['resultsreturned'] == 1) {
             $keywords->push($response['Result']);
         } else {
             $keywords = collect($response['Result']);
         }
 
-        return $keywords->transform(function ($keyword, $key) {
+        return $keywords->transform(function($keyword, $key) {
             return $this->transformCreatedKeyword($keyword);
         });
 
@@ -100,7 +100,7 @@ 
      */
     public function delete($id)
     {
-        if(!is_array($id)) {
+        if (!is_array($id)) {
             $id = [$id];
         }
 
@@ -108,12 +108,12 @@ 
 
         $response = $this->performQuery('keywords/delete', ['id' => $ids]);
 
-        if(isset($response['Result']['Id'])){
+        if (isset($response['Result']['Id'])) {
             return collect($response['Result']['Id']);
         }
 
 
-        return collect($response['Result'])->transform(function ($keywordID, $key) {
+        return collect($response['Result'])->transform(function($keywordID, $key) {
             return $keywordID['Id'];
         });
 
@@ -150,17 +150,17 @@ 
         $modifiedKeyword->keyword_device = $keyword['KeywordDevice'];
 
 
-        if($keyword['KeywordTags'] == 'none') {
+        if ($keyword['KeywordTags'] == 'none') {
             $modifiedKeyword->keyword_tags = collect();
         } else {
             $modifiedKeyword->keyword_tags = collect(explode(',', $keyword['KeywordTags']));
         }
 
-        if( is_null($keyword['KeywordStats']) ) {
+        if (is_null($keyword['KeywordStats'])) {
             $modifiedKeyword->keyword_stats = null;
         } else {
 
-            $localTrends = collect($keyword['KeywordStats']['LocalSearchTrendsByMonth'])->map(function ($searchVolume, $month){
+            $localTrends = collect($keyword['KeywordStats']['LocalSearchTrendsByMonth'])->map(function($searchVolume, $month) {
                 return new StatLocalSearchTrend([
                     'month' => strtolower($month),
                     'search_volume' => ($searchVolume == '-') ? null : $searchVolume,
@@ -177,7 +177,7 @@ 
 
         }
 
-        if( is_null($keyword['KeywordRanking']) ) {
+        if (is_null($keyword['KeywordRanking'])) {
             $modifiedKeyword->keyword_ranking = null;
         } else {
 

Braces +6 added lines, -4 removed lines

@@ -69,11 +69,13 @@
                                     return str_replace(',', '\,', $el);
                                 }, $keywords));
 
-        if( ! is_null($tags) && count($tags) > 0)
-            $arguments['tag'] = implode(',', $tags);
+        if( ! is_null($tags) && count($tags) > 0) {
+                    $arguments['tag'] = implode(',', $tags);
+        }
 
-        if( ! is_null($location) && $location != '')
-            $arguments['location'] = $location;
+        if( ! is_null($location) && $location != '') {
+                    $arguments['location'] = $location;
+        }
 
         $response = $this->performQuery('keywords/create', $arguments);
 

src/Api/StatProjects.php

Unused Use Statements -2 removed lines

@@ -2,9 +2,7 @@
 
 namespace SchulzeFelix\Stat\Api;
 
-use Carbon\Carbon;
 use Illuminate\Support\Collection;
-use Illuminate\Support\Facades\Cache;
 use SchulzeFelix\Stat\Objects\StatProject;
 use SchulzeFelix\Stat\Stat;
 

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     {
         $response = $this->performQuery('projects/list');
 
-        $projects = collect($response['Result'])->map(function ($project, $key) {
+        $projects = collect($response['Result'])->map(function($project, $key) {
             return new StatProject([
                 'id' => $project['Id'],
                 'name' => $project['Name'],

src/Api/StatSubAccounts.php

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace SchulzeFelix\Stat\Api;
 
-use Carbon\Carbon;
-use Carbon\CarbonInterval;
 use Illuminate\Support\Collection;
 use SchulzeFelix\Stat\Objects\StatSubAccount;
 

Spacing +3 added lines, -3 removed lines

@@ -19,17 +19,17 @@
 
         $subaccounts = collect();
 
-        if( ! isset($response['User']) ){
+        if (!isset($response['User'])) {
             return $subaccounts;
         }
 
-        if(isset($response['User']['Id'])) {
+        if (isset($response['User']['Id'])) {
             $subaccounts->push($response['User']);
         } else {
             $subaccounts = collect($response['User']);
         }
 
-        $subaccounts->transform(function ($item, $key) {
+        $subaccounts->transform(function($item, $key) {
 
             return new StatSubAccount([
                 'id' => $item['Id'],

src/Api/StatTags.php

Spacing +3 added lines, -3 removed lines

@@ -21,9 +21,9 @@ 
             return collect();
         }
 
-        $tags = collect($response['Result'])->transform(function ($item, $key) {
+        $tags = collect($response['Result'])->transform(function($item, $key) {
 
-            if($item['Keywords'] == 'none') {
+            if ($item['Keywords'] == 'none') {
                 $item['Keywords'] = collect();
             } else {
                 $item['Keywords'] = collect($item['Keywords']['Id']);
@@ -58,7 +58,7 @@ 
             $rankDistribution = collect([$response['RankDistribution']]);
         }
 
-        $rankDistribution->transform(function ($distribution, $key) {
+        $rankDistribution->transform(function($distribution, $key) {
             return $this->transformRankDistribution($distribution);
         });
 

src/Api/StatSites.php

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
         } while ($response['resultsreturned'] < $response['totalresults']);
 
 
-        $sites->transform(function ($site, $key) {
+        $sites->transform(function($site, $key) {
             return new StatSite([
                 'id' => $site['Id'],
                 'project_id' => $site['ProjectId'],
@@ -67,7 +67,7 @@ 
             $sites = collect($response['Result']);
         }
 
-        $sites->transform(function ($site, $key) use($projectID) {
+        $sites->transform(function($site, $key) use($projectID) {
             return new StatSite([
                 'id' => $site['Id'],
                 'project_id' => $projectID,
@@ -103,7 +103,7 @@ 
             $rankDistribution = collect([$response['RankDistribution']]);
         }
 
-        $rankDistribution->transform(function ($distribution, $key) {
+        $rankDistribution->transform(function($distribution, $key) {
             return $this->transformRankDistribution($distribution);
         });
 
@@ -163,6 +163,6 @@ 
     {
         $response = $this->performQuery('sites/delete', ['id' => $siteID]);
 
-        return (int) $response['Result']['Id'];
+        return (int)$response['Result']['Id'];
     }
 }

src/Api/BaseStat.php

Spacing +8 added lines, -8 removed lines

@@ -37,7 +37,7 @@ 
         {
             $response = $this->statClient->performQuery($method, $parameters);
         }
-        catch(ClientException $e)
+        catch (ClientException $e)
         {
             $xml = simplexml_load_string($e->getResponse()->getBody()->getContents());
             throw ApiException::requestException($xml->__toString());
@@ -59,8 +59,8 @@ 
     protected function checkMaximumDateRange(Carbon $fromDate, Carbon $toDate, $maxDays = 31)
     {
 
-        if($fromDate->diffInDays($toDate) > $maxDays) {
-            throw ApiException::resultError('The maximum date range between from_date and to_date is '.$maxDays.' days.');
+        if ($fromDate->diffInDays($toDate) > $maxDays) {
+            throw ApiException::resultError('The maximum date range between from_date and to_date is ' . $maxDays . ' days.');
         }
     }
 
@@ -71,7 +71,7 @@ 
      */
     protected function getCollection($element, $identifier = 'Id')
     {
-        if(isset($element[$identifier])){
+        if (isset($element[$identifier])) {
             $collection = collect([$element]);
         } else {
             $collection = collect($element);
@@ -89,7 +89,7 @@ 
         $rankDistribution = new StatRankDistribution();
         $rankDistribution->date = $distribution['date'];
 
-        if(array_key_exists('Google', $distribution)){
+        if (array_key_exists('Google', $distribution)) {
             $rankDistribution->google = new StatEngineRankDistribution([
                 'one' => $distribution['Google']['One'],
                 'two' => $distribution['Google']['Two'],
@@ -106,7 +106,7 @@ 
             ]);
         }
 
-        if(array_key_exists('GoogleBaseRank', $distribution)){
+        if (array_key_exists('GoogleBaseRank', $distribution)) {
             $rankDistribution->google_base_rank = new StatEngineRankDistribution([
                 'one' => $distribution['GoogleBaseRank']['One'],
                 'two' => $distribution['GoogleBaseRank']['Two'],
@@ -123,7 +123,7 @@ 
             ]);
         }
 
-        if(array_key_exists('Yahoo', $distribution)){
+        if (array_key_exists('Yahoo', $distribution)) {
             $rankDistribution->yahoo = new StatEngineRankDistribution([
                 'one' => $distribution['Yahoo']['One'],
                 'two' => $distribution['Yahoo']['Two'],
@@ -140,7 +140,7 @@ 
             ]);
         }
 
-        if(array_key_exists('Bing', $distribution)){
+        if (array_key_exists('Bing', $distribution)) {
             $rankDistribution->bing = new StatEngineRankDistribution([
                 'one' => $distribution['Bing']['One'],
                 'two' => $distribution['Bing']['Two'],

Braces +1 added lines, -2 removed lines

@@ -36,8 +36,7 @@
         try
         {
             $response = $this->statClient->performQuery($method, $parameters);
-        }
-        catch(ClientException $e)
+        } catch(ClientException $e)
         {
             $xml = simplexml_load_string($e->getResponse()->getBody()->getContents());
             throw ApiException::requestException($xml->__toString());

src/Api/StatRankings.php

Spacing +4 added lines, -4 removed lines

@@ -21,15 +21,15 @@ 
             $response = $this->performQuery('rankings/list', ['keyword_id' => $keywordID, 'from_date' => $fromDate->toDateString(), 'to_date' => $toDate->toDateString(), 'start' => 0]);
             $start += 30;
 
-            if($response['totalresults'] == 0) {
+            if ($response['totalresults'] == 0) {
                 break;
             }
 
-            if($response['totalresults'] == 1) {
+            if ($response['totalresults'] == 1) {
                 $rankings->push($response['Result']);
             }
 
-            if($response['totalresults'] > 1) {
+            if ($response['totalresults'] > 1) {
                 $rankings = $rankings->merge($response['Result']);
             }
 
@@ -41,7 +41,7 @@ 
         } while ($response['resultsreturned'] < $response['totalresults']);
 
 
-        $rankings = $rankings->transform(function ($ranking, $key) {
+        $rankings = $rankings->transform(function($ranking, $key) {
 
             return new StatKeywordRanking([
                 'date' => $ranking['date'],

src/Api/StatSerps.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     {
         $response = $this->performQuery('serps/show', ['keyword_id' => $keywordID, 'engine' => $engine, 'date' => $date->toDateString()]);
 
-        return collect($response['Result'])->transform(function ($ranking, $key) {
+        return collect($response['Result'])->transform(function($ranking, $key) {
 
             return new StatSerpItem([
                 'result_type' => $ranking['ResultType'],