Fusonic\OpenGraph\Consumer

Complexity

Total Complexity

17

Size/Duplication

Total Lines	134
Duplicated Lines	13.43 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

94.74%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	loadUrl()	0	7	1
A	loadHtml()	0	13	3
C	extractOpenGraphData()	18	65	12

<?php

namespace Fusonic\OpenGraph;

use Fusonic\Linq\Linq;
use Fusonic\OpenGraph\Objects\ObjectBase;
use Fusonic\OpenGraph\Objects\Website;
use GuzzleHttp\Adapter\AdapterInterface;
use GuzzleHttp\Client;
use Symfony\Component\DomCrawler\Crawler;

/**
 * Consumer that extracts Open Graph data from either a URL or a HTML string.
 */
class Consumer
{
    private $client;

    /**
     * When enabled, crawler will read content of title and meta description if no
     * Open Graph data is provided by target page.
     *
     * @var bool
     */
    public $useFallbackMode = false;

    /**
     * When enabled, crawler will throw exceptions for some crawling errors like unexpected
     * Open Graph elements.
     *
     * @var bool
     */
    public $debug = false;

    /**
     * @param   AdapterInterface $adapter Guzzle adapter to use for making HTTP requests.
     * @param   array            $config  Optional Guzzle config overrides.
     */
    public function __construct(AdapterInterface $adapter = null, array $config = [])
    {
        $config = array_replace_recursive(['adapter' => $adapter], $config);

        $this->client = new Client($config);
    }

    /**
     * Fetches HTML content from the given URL and then crawls it for Open Graph data.
     *
     * @param   string  $url            URL to be crawled.
     *
     * @return  Website
     */
    public function loadUrl($url)
    {
        // Fetch HTTP content using Guzzle
        $response = $this->client->get($url);

        return $this->loadHtml($response->getBody()->__toString(), $url);
    }

    /**
     * Crawls the given HTML string for OpenGraph data.
     *
     * @param   string  $html           HTML string, usually whole content of crawled web resource.
     * @param   string  $fallbackUrl    URL to use when fallback mode is enabled.
     *
     * @return  ObjectBase
     */
    public function loadHtml($html, $fallbackUrl = null)
    {
        // Extract all data that can be found
        $page = $this->extractOpenGraphData($html);

        // Use the user's URL as fallback
        if ($this->useFallbackMode && $page->url === null) {
            $page->url = $fallbackUrl;
        }

        // Return result
        return $page;
    }

    private function extractOpenGraphData($content)
    {
        $crawler = new Crawler($content);

        $properties = [];
        foreach(['name', 'property'] as $t)

Expected 1 space after FOREACH keyword; 0 found

        {
            // Get all meta-tags starting with "og:"
            $ogMetaTags = $crawler->filter("meta[{$t}^='og:']");
            // Create clean property array
            $props = Linq::from($ogMetaTags)
                ->select(
                    function (\DOMElement $tag) use ($t) {
                        $name = strtolower(trim($tag->getAttribute($t)));
                        $value = trim($tag->getAttribute("content"));
                        return new Property($name, $value);
                    }
                )
                ->toArray();
            $properties = array_merge($properties, $props);
          

Blank line found at end of control structure

        }
            
        // Create new object of the correct type
        $typeProperty = Linq::from($properties)
            ->firstOrNull(
                function (Property $property) {
                    return $property->key === Property::TYPE;
                }
            );
        switch ($typeProperty !== null ? $typeProperty->value : null) {
            default:

default: $object = new \Fusonic\OpenGraph\Objects\Website(); break; does not seem to be reachable.

                $object = new Website();
                break;
        }

        // Assign all properties to the object
        $object->assignProperties($properties, $this->debug);

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

        // Fallback for url
        if ($this->useFallbackMode && !$object->url) {

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

This code seems to be duplicated across your project.

            $urlElement = $crawler->filter("link[rel='canonical']")->first();
            if ($urlElement->count()) {
                $object->url = trim($urlElement->attr("href"));

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

            }
        }

        // Fallback for title
        if ($this->useFallbackMode && !$object->title) {

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

This code seems to be duplicated across your project.

            $titleElement = $crawler->filter("title")->first();
            if ($titleElement) {
                $object->title = trim($titleElement->text());

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

            }
        }

        // Fallback for description
        if ($this->useFallbackMode && !$object->description) {

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

This code seems to be duplicated across your project.

            $descriptionElement = $crawler->filter("meta[property='description']")->first();
            if ($descriptionElement) {
                $object->description = trim($descriptionElement->attr("content"));

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

            }
        }

        return $object;

The variable $object seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

    }
}