SurajAdsul\AppMarket\PlayStoreInfo

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	lookup()	0	27	3

<?php

namespace SurajAdsul\AppMarket;

class PlayStoreInfo extends AbstractStoreInfo
{
    private static $lookupUrl = 'https://play.google.com/store/apps/details?id=%s';

    /**
     * @return array
     */
    protected function lookup()
    {
        $url = sprintf(self::$lookupUrl, $this->storeId);

        $client = new CrawlerClient();
        $crawler = $client->request('GET', $url);

        $this->name = $crawler->filter('[itemprop="name"]')->text();

        $crawler->filter('div.score-container meta')->each(function ($node) {
            switch ($node->attr('itemprop')) {
                case 'ratingValue':
                    $this->ratingStars = round($node->attr('content'));

The property $ratingStars was declared of type integer, but round($node->attr('content')) is of type double. Maybe add a type cast?

                    break;
                case 'ratingCount':
                    $this->ratingCount = $node->attr('content');
            }
        });

//        $this->contentRating = $crawler->filter('[itemprop="contentRating"]')->text();

62% of this comment could be valid code. Did you maybe forget this after debugging?

        $iconUrl = $crawler->filter('[itemprop="image"]')->attr('src');
        $this->icon = $this->addScheme($iconUrl);

        $crawler->filter('[alt="Screenshot Image"]')->each(function ($node) {
            $this->screenshots[] = $this->addScheme($node->attr('src'));
        });
    }
}