WilderAmorim\StructuredData\Schema

Complexity

Total Complexity

11

Size/Duplication

Total Lines	135
Duplicated Lines	0 %

Importance

Changes	5
Bugs	1	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	imageObject()	0	10	1
A	organization()	0	13	1
A	person()	0	9	1
A	json()	0	3	1
A	companyUrl()	0	3	1
A	companyName()	0	3	1
A	setDate()	0	16	4

<?php


namespace WilderAmorim\StructuredData;

/**
 * Class Schema
 * @package WilderAmorim\StructuredData
 */
abstract class Schema
{
    /** @var */
    protected $person;

    /** @var */
    protected $organization;

    /** @var */
    protected $imageObject;

    /** @var string */
    protected $companyName;

    /**  @var string */
    protected $companyUrl;

    /** @var array */
    protected $companySameAs;

    /**
     * @see http://schema.org
     */
    const CONTEXT = 'http://schema.org';

    /**
     * @see https://schema.org/WebPage
     */
    const TYPE = 'WebPage';


    /**
     * Schema constructor.
     */
    public function __construct()
    {
    }

    /**
     * @return Schema
     */
    protected function companyUrl(): string
    {
        return $this->companyUrl;

The expression return $this->companyUrl returns the type string which is incompatible with the documented return type WilderAmorim\StructuredData\Schema.

    }

    /**
     * @return Schema
     */
    protected function companyName(): string
    {
        return $this->companyName;

The expression return $this->companyName returns the type string which is incompatible with the documented return type WilderAmorim\StructuredData\Schema.

    }

    /**
     * @see https://schema.org/Person
     *
     * @param string $name
     * @param string $image
     * @param array $sameAs
     * @return Schema
     */
    protected function person(string $name, string $image, ?array $sameAs = null): Schema
    {
        $this->person = (object)[
            '@type' => 'Person',
            'name' => $name,
            'image' => $image,
            'sameAs' => $sameAs
        ];
        return $this;
    }

    /**
     * @see https://schema.org/Organization
     */
    protected function organization(string $logo, string $image): Schema
    {
        $logo = $this->imageObject($logo, null, null);
        $image = $this->imageObject($image, null, null);
        $this->organization = (object)[
            '@type' => 'Organization',
            'name' => $this->companyName,
            'url' => $this->companyUrl,
            'logo' => $logo,
            'image' => $image,
            'sameAs' => ($this->companySameAs ?? null)
        ];
        return $this;
    }

    /**
     * @see https://schema.org/ImageObject
     */
    protected function imageObject(string $url, ?int $width = 1280, ?int $height = 720): array
    {
        $this->imageObject = [
            '@type' => 'ImageObject',
            'url' => $url,
            'width' => $width,
            'height' => $height,
            'caption' => $this->companyName
        ];
        return $this->imageObject;
    }

    /**
     * @param array $schema
     * @return string
     */
    protected function json(array $schema): string
    {
        return json_encode($schema, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
    }

    /**
     * @param string $date
     * @return string
     */
    protected function setDate(string $date): string
    {
        $date = (strpos($date, ' ') ? explode(' ', $date)[0] : $date);

        $date = str_replace('/', '-', $date);
        $date = explode('-', $date);

        if (strlen($date[0]) == 4) {
            $date = "{$date[0]}-{$date[1]}-$date[2]";
        }

        if (strlen($date[2]) == 4) {
            $date = "{$date[2]}-{$date[1]}-$date[0]";
        }

        return $date;

The expression return $date could return the type string[] which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}