SilverStripe\VersionedAdmin\Extensions\BlockArchiveExtension

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getArchiveField()	0	25	3
A	isArchiveFieldEnabled()	0	3	1
A	getArchiveFieldClass()	0	3	1

<?php

namespace SilverStripe\VersionedAdmin\Extensions;

use DNADesign\Elemental\Models\BaseElement;

The type DNADesign\Elemental\Models\BaseElement was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Forms\GridField\GridFieldDataColumns;
use SilverStripe\ORM\DataExtension;
use SilverStripe\Security\Member;
use SilverStripe\VersionedAdmin\ArchiveAdmin;
use SilverStripe\VersionedAdmin\ArchiveViewProvider;

/**
 * Adds a archive view for Elemental blocks
 */
class BlockArchiveExtension extends DataExtension implements ArchiveViewProvider
{
    /**
     * @inheritDoc
    */
    public function getArchiveFieldClass()
    {
        return BaseElement::class;

The expression return DNADesign\Elemental\Models\BaseElement::class returns the type string which is incompatible with the return type mandated by SilverStripe\VersionedAdmin\ArchiveViewProvider::getArchiveFieldClass() of array.

    }

    /**
     * @inheritDoc
    */
    public function getArchiveField()
    {
        $listField = ArchiveAdmin::createArchiveGridField('Blocks', BaseElement::class);

        $listColumns = $listField->getConfig()->getComponentByType(GridFieldDataColumns::class);
        $listColumns->setDisplayFields([

The method setDisplayFields() does not exist on SilverStripe\Forms\GridField\GridFieldComponent. It seems like you code against a sub-type of SilverStripe\Forms\GridField\GridFieldComponent such as SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\GridField\GridFieldDataColumns or SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\Tests\GridField\GridField_URLHandlerTest\TestComponent or SilverStripe\Forms\GridField\GridFieldDetailForm.

            'Title' => BaseElement::singleton()->fieldLabel('Title'),
            'Type' => _t(ArchiveAdmin::class . '.COLUMN_TYPE', 'Type'),
            'LastEdited.Ago' => _t(ArchiveAdmin::class . '.COLUMN_DATEARCHIVED', 'Date Archived'),
            'Breadcrumbs' => _t(ArchiveAdmin::class . '.COLUMN_ORIGIN', 'Origin'),
            'AuthorID' => _t(ArchiveAdmin::class . '.COLUMN_ARCHIVEDBY', 'Archived By')
        ]);
        $listColumns->setFieldFormatting([

The method setFieldFormatting() does not exist on SilverStripe\Forms\GridField\GridFieldComponent. It seems like you code against a sub-type of SilverStripe\Forms\GridField\GridFieldComponent such as SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\GridField\GridFieldDataColumns or SilverStripe\Forms\GridField\GridFieldPrintButton or SilverStripe\Forms\Tests\GridField\GridField_URLHandlerTest\TestComponent or SilverStripe\Forms\GridField\GridFieldDetailForm.

            'Breadcrumbs' => function ($val, $item) {
                $parent = $item->Page;

                return $parent ? $parent->Breadcrumbs() : null;
            },
            'AuthorID' => function ($val, $item) {

The parameter $item is not used and could be removed.

                $member = Member::get_by_id($val);
                return $member ? $member->Name : null;

$member is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

            },
        ]);

        return $listField;
    }

    /**
     * @inheritDoc
    */
    public function isArchiveFieldEnabled()
    {
        return true;
    }
}