DeleteStaticCacheJob::getTitle() - Code Metrics - Inspection of "SS4 Compatability" - silverstripe/silverstripe-staticpublishqueue - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#70)

by Daniel

created 2018-02-12 13:31 UTC

DeleteStaticCacheJob::getTitle() A

↳ Parent: DeleteStaticCacheJob

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php

namespace SilverStripe\StaticPublishQueue\Job;

use SilverStripe\StaticPublishQueue\Job;
use SilverStripe\StaticPublishQueue\Publisher;

class DeleteStaticCacheJob extends Job
{
    /**
     * @var int
     * @config
     */
    private static $chunk_size = 2000;


    /**
     * @return string
     */
    public function getTitle()
    {
        return 'Remove a set of static pages from the cache';
    }

    /**
     * Do some processing yourself!
     */
    public function process()
    {
        $chunkSize = self::config()->get('chunk_size');
        $count = 0;
        foreach ($this->jobData->URLsToProcess as $url => $priority) {
            if (++$count > $chunkSize) {
                break;
            }
            $meta = Publisher::singleton()->purgeURL($url);
            if (!empty($meta['success'])) {
                $this->jobData->ProcessedURLs[$url] = $url;
                unset($this->jobData->URLsToProcess[$url]);
            }
        }
        $this->isComplete = empty($this->jobData->URLsToProcess);
    }
}


1			<?php
2
3			namespace SilverStripe\StaticPublishQueue\Job;
4
5			use SilverStripe\StaticPublishQueue\Job;
6			use SilverStripe\StaticPublishQueue\Publisher;
7
8			class DeleteStaticCacheJob extends Job
9			{
10			/**
11			* @var int
12			* @config
13			*/
14			private static $chunk_size = 2000;
			0 ignored issues – show introduced 2018-01-16 16:52 UTC by Report Bug Copy Issue Report The private property `$chunk_size` is not used, and could be removed. Loading history...
15
16			/**
17			* @return string
18			*/
19			public function getTitle()
20			{
21			return 'Remove a set of static pages from the cache';
22			}
23
24			/**
25			* Do some processing yourself!
26			*/
27			public function process()
28			{
29			$chunkSize = self::config()->get('chunk_size');
30			$count = 0;
31			foreach ($this->jobData->URLsToProcess as $url => $priority) {
32			if (++$count > $chunkSize) {
33			break;
34			}
35			$meta = Publisher::singleton()->purgeURL($url);
36			if (!empty($meta['success'])) {
37			$this->jobData->ProcessedURLs[$url] = $url;
38			unset($this->jobData->URLsToProcess[$url]);
39			}
40			}
41			$this->isComplete = empty($this->jobData->URLsToProcess);
42			}
43			}
44

silverstripe / silverstripe-staticpublishqueue

Pull Request — master (#70)

DeleteStaticCacheJob::getTitle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like