SilverStripe\CKANRegistry\Model\ResourceFilter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	15	1
A	forTemplate()	0	8	2

<?php

namespace SilverStripe\CKANRegistry\Model;

use InvalidArgumentException;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Forms\FormField;
use SilverStripe\Forms\HiddenField;
use SilverStripe\Forms\ListboxField;
use SilverStripe\Forms\TextField;
use SilverStripe\i18n\i18n;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\ValidationResult;

class ResourceFilter extends DataObject
{
    private static $filter_types = [

The private property $filter_types is not used, and could be removed.

        TextField::class => 'Text',
        DropdownField::class => 'Select one from list',
    ];

    private static $table_name = 'CKANFilter';

The private property $table_name is not used, and could be removed.

    private static $db = [

The private property $db is not used, and could be removed.

        'Name' => 'Varchar',
        'Type' => 'Varchar',
        'AllFields' => 'Boolean',
        'TypeOptions' => 'Text',
    ];

    private static $has_one = [

The private property $has_one is not used, and could be removed.

        'FilterFor' => Resource::class,
    ];

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $typeTitle = $fields->dataFieldByName('Type')->Title();
        $fields->replaceField('Type', DropdownField::create(
            'Type',
            $typeTitle,
            $this->config()->get('filter_types')
        ));
        $fields->replaceField('TypeOptions', HiddenField::create('TypeOptions'));
        $fields->push(
            ListboxField::create('Fields', 'Columns', $this->FilterFor()->Fields()->map('Name', 'ReadableName'))

The method FilterFor() does not exist on SilverStripe\CKANRegistry\Model\ResourceFilter. Since you implemented __call, consider adding a @method annotation.

        );
        $fields->removeByName('FilterForID');
        return $fields;
    }

    public function forTemplate()
    {
        $options = json_decode($this->TypeOptions, true);

The property TypeOptions does not exist on SilverStripe\CKANRegistry\Model\ResourceFilter. Since you implemented __get, consider adding a @property annotation.

        $field = Injector::inst()->createWithArgs($this->Type, $options);

The property Type does not exist on SilverStripe\CKANRegistry\Model\ResourceFilter. Since you implemented __get, consider adding a @property annotation.

        if ($field instanceof FormField) {
            throw new InvalidArgumentException("$this->Type is not a FormField");
        }
        return $field;
    }
}