silverstripe / silverstripe-externallinks

src/Model/BrokenExternalPageTrack.php

<?php

namespace SilverStripe\ExternalLinks\Model;

use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\ExternalLinks\Model\BrokenExternalPageTrackStatus;
use SilverStripe\ExternalLinks\Model\BrokenExternalLink;
use SilverStripe\Versioned\Versioned;
use SilverStripe\ORM\DataObject;

/**
 * Represents a track for a single page
 */
class BrokenExternalPageTrack extends DataObject
{
    private static $table_name = 'BrokenExternalPageTrack';

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.

        'Processed' => 'Boolean'
    );

    private static $has_one = array(

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

        'Page' => SiteTree::class,
        'Status' => BrokenExternalPageTrackStatus::class
    );

    private static $has_many = array(

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

        'BrokenLinks' => BrokenExternalLink::class
    );

    /**
     * @return SiteTree
     */
    public function Page()
    {
        return Versioned::get_by_stage(SiteTree::class, 'Stage')
            ->byID($this->PageID);

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

    }
}