silverstripe / silverstripe-externallinks

src/Jobs/CheckExternalLinksJob.php

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 17 and the first side effect is on line 10.

namespace SilverStripe\ExternalLinks\Jobs;

use Symbiote\QueuedJobs\Services\AbstractQueuedJob;

The type Symbiote\QueuedJobs\Services\AbstractQueuedJob was not found. Maybe you did not declare it correctly or list all dependencies?

use Symbiote\QueuedJobs\Services\QueuedJob;

The type Symbiote\QueuedJobs\Services\QueuedJob was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\ExternalLinks\Tasks\CheckExternalLinksTask;

if (!class_exists(AbstractQueuedJob::class)) {
    return;
}

/**
 * A Job for running a external link check for published pages
 *
 */
class CheckExternalLinksJob extends AbstractQueuedJob implements QueuedJob
{

    public function getTitle()
    {
        return _t(__CLASS__ . '.TITLE', 'Checking for external broken links');
    }

    public function getJobType()
    {
        return QueuedJob::QUEUED;
    }

    public function getSignature()
    {
        return md5(get_class($this));
    }

    /**
     * Check an individual page
     */
    public function process()
    {
        $task = CheckExternalLinksTask::create();
        $track = $task->runLinksCheck(1);
        $this->currentStep = $track->CompletedPages;

The property currentStep does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->totalSteps = $track->TotalPages;

The property totalSteps does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->isComplete = $track->Status === 'Completed';

The property isComplete does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }
}