SilverStripe\DocumentConverter\SettingsField

Complexity

Total Complexity

3

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	70	2
A	getInnerField()	0	3	1

<?php

namespace SilverStripe\DocumentConverter;

use InvalidArgumentException;
use SilverStripe\Assets\Folder;
use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\CompositeField;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\HeaderField;
use SilverStripe\Forms\TreeDropdownField;
use SilverStripe\View\Requirements;

/**
 * Provides a document import capability through the use of an external service.
 * Includes several options fields, which are bundled together with an UploadField
 * into a CompositeField.
 */
class SettingsField extends CompositeField
{
    /**
     * Reference to the inner upload field (ImportField).
     */
    private $innerField = null;

    /**
     * Augments a simple CompositeField with uploader and import options.
     *
     * @param $children FieldSet/array Any additional children.
     */

The doc comment FieldSet/array at position 0 could not be parsed: Unknown type name 'FieldSet/array' at position 0 in FieldSet/array.

    public function __construct($children = null)
    {
        if ($children) {
            $class = get_class();
            throw new InvalidArgumentException(
                "${class}::__construct does not accept extra parameters."
            );
        }

        // Add JS specific to this field.
        Requirements::javascript('silverstripe/documentconverter: javascript/DocumentConversionField.js');

        $fields = FieldList::create([
            HeaderField::create(
                'FileWarningHeader',

'FileWarningHeader' of type string is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

                _t(
                    __CLASS__ . '.FileWarningHeader',
                    'Warning: import will remove all content and subpages of this page.'
                ),
                4

4 of type integer is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

            ),
            $splitHeader = DropdownField::create(
                'DocumentConversionSettings-SplitHeader',
                _t(
                    __CLASS__ . '.SplitHeader',
                    'Split document into pages'
                ),
                [
                    0 => _t(__CLASS__ . '.No', 'no'),
                    1 => _t(__CLASS__ . '.EachH1', 'for each heading 1'),
                    2 => _t(__CLASS__ . '.EachH2', 'for each heading 2')
                ]
            ),
            $keepSource = CheckboxField::create(
                'DocumentConversionSettings-KeepSource',
                _t(
                    __CLASS__ . '.KeepSource',
                    'Keep the original document. Adds a link to it on TOC, if enabled.'
                )
            ),
            $chosenFolderID = TreeDropdownField::create(
                'DocumentConversionSettings-ChosenFolderID',
                _t(__CLASS__ . '.ChooseFolder', 'Choose a folder to save this file'),
                Folder::class
            ),
            $includeTOC = CheckboxField::create(
                'DocumentConversionSettings-IncludeTOC',
                _t(__CLASS__ . '.IncludeTOC', 'Replace this page with a Table of Contents.')
            ),
            $publishPages = CheckboxField::create(
                'DocumentConversionSettings-PublishPages',
                _t(
                    __CLASS__ . '.publishPages',
                    'Publish modified pages (not recommended unless you are sure about the conversion outcome)'
                )
            ),
            $this->innerField = ImportField::create(
                'ImportedFromFile',
                _t(__CLASS__ . '.ImportedFromFile', 'Import content from a word document')
            )
        ]);

        // Prevent the warning popup that appears when navigating away from the page.
        $splitHeader->addExtraClass('no-change-track');
        $keepSource->addExtraClass('no-change-track');
        $chosenFolderID->addExtraClass('no-change-track');
        $includeTOC->addExtraClass('no-change-track');
        $publishPages->addExtraClass('no-change-track');

        return parent::__construct($fields);

Are you sure the usage of parent::__construct($fields) targeting SilverStripe\Forms\CompositeField::__construct() seems to always return null.

    }

    public function getInnerField()
    {
        return $this->innerField;
    }
}