PhpTek\Exodus\Model\StaticSiteContentSourceImportRule::getCMSFields()

Complexity

Conditions	3
Paths	2

Size

Total Lines	47
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

<?php
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $dataType = $this->Schema()->DataType;

The method Schema() does not exist on PhpTek\Exodus\Model\StaticSiteContentSourceImportRule. Since you implemented __call, consider adding a @method annotation.

        if ($dataType) {
            $fieldList = singleton(DataObjectSchema::class)
                ->fieldSpecs($dataType, DataObjectSchema::DB_ONLY);
            $fieldList = array_combine(array_keys($fieldList), array_keys($fieldList));
            foreach (
                array_merge(
                    array_keys(DataObject::config()->get('fixed_fields')),
                    array_filter($fieldList, function ($item) {
                        return preg_match('#(^Static|ID$)#i', $item);
                    }),
                    ['Version']
                ) as $exclusion
            ) {
                unset($fieldList[$exclusion]);
            }

            natsort($fieldList);

            $fieldNameField = DropdownField::create("FieldName", 'Target Field', $fieldList)
                ->setEmptyString("(choose)")
                ->setDescription('Remote content matched by the CSS selector below is written to this field.');
            $fields->insertBefore($fieldNameField, 'CSSSelector');

'CSSSelector' of type string is incompatible with the type SilverStripe\Forms\FormField expected by parameter $item of SilverStripe\Forms\FieldList::insertBefore().

            $fields->dataFieldByName('CSSSelector')
                ->setDescription('A valid CSS selector whose content is written to the "Target Field" above.')
                ->setAttribute('style', 'width: 300px;');
            $fields->dataFieldByName('ExcludeCSSSelector')
                ->setDescription('A list of valid CSS selectors whose content'
                . ' should be ignored. This is useful for fine-tuning what is returned in an import.'
                . ' Separate multiple exclusions with a newline.');
            $fields->dataFieldByName('OuterHTML')
                ->setDescription('Use outer HTML (Fetches parent element and content and that of its children)');
            $fields->dataFieldByName('PlainText')
                ->setDescription('Convert to plain text (Removes markup)');
        } else {
            $fields->replaceField('FieldName', $fieldName = ReadonlyField::create("FieldName", "Field Name"));
            $fieldName->setDescription('Save this rule before being able to add a field name');
        }

        $fields->dataFieldByName('Attribute')->setDescription('Add the element attribute where the desired text'
        . ' can be found (such as "alt" or "title") if not found as the selected element\'s text itself.');

        return $fields;

The expression return $fields returns the type SilverStripe\Forms\FieldList which is incompatible with the documented return type PhpTek\Exodus\Model\FieldList.

    }