silverstripe / silverstripe-externallinks

src/Model/BrokenExternalLink.php

<?php

namespace SilverStripe\ExternalLinks\Model;

use SilverStripe\Control\HTTPResponse;
use SilverStripe\ExternalLinks\Model\BrokenExternalPageTrack;
use SilverStripe\ExternalLinks\Model\BrokenExternalPageTrackStatus;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Permission;
use SilverStripe\Security\Security;

/**
 * Represents a single link checked for a single run that is broken
 *
 * @method BrokenExternalPageTrack Track()
 * @method BrokenExternalPageTrackStatus Status()
 */
class BrokenExternalLink extends DataObject
{
    private static $table_name = 'BrokenExternalLink';

The private property $table_name is not used, and could be removed.

    private static $db = array(

The private property $db is not used, and could be removed.

        'Link' => 'Varchar(2083)', // 2083 is the maximum length of a URL in Internet Explorer.
        'HTTPCode' =>'Int'
    );

    private static $has_one = array(

The private property $has_one is not used, and could be removed.

        'Track' => BrokenExternalPageTrack::class,
        'Status' => BrokenExternalPageTrackStatus::class
    );

    private static $summary_fields = array(

The private property $summary_fields is not used, and could be removed.

        'Created' => 'Checked',
        'Link' => 'External Link',
        'HTTPCodeDescription' => 'HTTP Error Code',
        'Page.Title' => 'Page link is on'
    );

    private static $searchable_fields = array(

The private property $searchable_fields is not used, and could be removed.

        'HTTPCode' => array('title' => 'HTTP Code')
    );

    /**
     * @return SiteTree

The type SilverStripe\ExternalLinks\Model\SiteTree was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function Page()
    {
        return $this->Track()->Page();

The expression return $this->Track()->Page() returns the type SilverStripe\CMS\Model\SiteTree which is incompatible with the documented return type SilverStripe\ExternalLinks\Model\SiteTree.

    }

    public function canEdit($member = false)
    {
        return false;
    }

    public function canView($member = false)
    {
        $member = $member ? $member : Security::getCurrentUser();
        $codes = array('content-authors', 'administrators');
        return Permission::checkMember($member, $codes);
    }

    /**
     * Retrieve a human readable description of a response code
     *
     * @return string
     */
    public function getHTTPCodeDescription()
    {
        $code = $this->HTTPCode;

The property HTTPCode does not exist on SilverStripe\ExternalLinks\Model\BrokenExternalLink. Since you implemented __get, consider adding a @property annotation.

        try {
            $response = HTTPResponse::create('', $code);

'' of type string is incompatible with the type array expected by parameter $args of SilverStripe\Control\HTTPResponse::create().

            // Assume that $code = 0 means there was no response
            $description = $code ?
                $response->getStatusDescription() :
                _t(__CLASS__ . '.NOTAVAILABLE', 'Server Not Available');
        } catch (InvalidArgumentException $e) {

The type SilverStripe\ExternalLinks\Model\InvalidArgumentException was not found. Did you mean InvalidArgumentException? If so, make sure to prefix the type with \.

            $description = _t(__CLASS__ . '.UNKNOWNRESPONSE', 'Unknown Response Code');
        }

        return sprintf("%d (%s)", $code, $description);
    }
}