SilverStripe\ExternalLinks\Jobs\CheckExternalLinksJob

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getJobType()	0	3	1
A	getTitle()	0	3	1
A	getSignature()	0	3	1
A	process()	0	7	1

<?php

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.

    }
}