Sunnysideup\VersionPruner\PruningTemplates\UserChanged

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getTitle()	0	3	1
A	markOlderItemsWithoutAuthor()	0	10	1
A	getDescription()	0	3	1
A	setKeepVersions()	0	5	1
A	run()	0	4	1

<?php

namespace Sunnysideup\VersionPruner\PruningTemplates;

use SilverStripe\ORM\DB;
use Sunnysideup\VersionPruner\PruningTemplatesTemplate;

class UserChanged extends PruningTemplatesTemplate
{
    protected $keepVersions = 3;

    public function setKeepVersions(int $keepVersions): self
    {
        $this->keepVersions = $keepVersions;

        return $this;
    }

    public function getTitle(): string
    {
        return 'Prune automated saves';
    }

    public function getDescription(): string
    {
        return 'Delete versions that are not edited by a logged-in user.';
    }

    public function run(?bool $verbose = false)
    {
        DB::query('SELECT * FROM SiteTree_Versions WHERE AuthorID > 0 AND RecordID = ' . $this->object->ID);
        $this->markOlderItemsWithoutAuthor();
    }

    /**
     * these can be deleted.
     */
    protected function markOlderItemsWithoutAuthor()
    {
        $filter['"AuthorID" = ?'] = 0;

$filter was never initialized. Although not strictly required by PHP, it is generally a good practice to add $filter = array(); before regardless.

        $query = $this->getBaseQuery(['AuthorID'])
            ->addWhere($this->normaliseWhere($filter))
            ->setLimit($this->normaliseLimit(), $this->normaliseOffset($this->keepVersions))
        ;
        $this->toDelete[$this->getUniqueKey()] += $this->addVersionNumberToArray(
            $this->toDelete[$this->getUniqueKey()],
            $query->execute()
        );
    }
}