Dynamic\FlexSlider\Model\VideoEmbedSlide

Complexity

Total Complexity

1

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	12	1

<?php

namespace Dynamic\FlexSlider\Model;

use Sheadawson\Linkable\Forms\EmbeddedObjectField;
use Sheadawson\Linkable\Models\EmbeddedObject;
use SilverStripe\AssetAdmin\Forms\UploadField;
use SilverStripe\Assets\File;
use SilverStripe\Forms\FieldList;

/**
 * Class VideoEmbedSlide
 * @package Dynamic\FlexSlider\Model
 */
class VideoEmbedSlide extends Slide
{
    /**
     * @var string
     */
    private static $singular_name = 'Video Embed Slide';

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

    /**
     * @var string
     */
    private static $plural_name = 'Video Embed Slides';

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

    /**
     * @var string
     */
    private static $table_name = 'VideoEmbedSlide';

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

    /**
     * @var array
     */
    private static $has_one = [

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

        'Video' => EmbeddedObject::class,
    ];

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $fields) {
            $fields->removeByName(['VideoID']);
            $fields->addFieldToTab(
                'Root.Main',
                EmbeddedObjectField::create('Video', 'Video', $this->Video()),

The method Video() does not exist on Dynamic\FlexSlider\Model\VideoEmbedSlide. Since you implemented __call, consider adding a @method annotation.

                'Content'
            );
        });

        return parent::getCMSFields();
    }
}