Dynamic\Elements\Flexslider\Elements\ElementSlideshow

Complexity

Total Complexity

5

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Test Coverage

Coverage

84%

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	provideBlockSchema()	0	5	1
A	fieldLabels()	0	7	1
A	getSummary()	0	9	1
A	getType()	0	3	1
A	getCMSFields()	0	12	1

<?php

namespace Dynamic\Elements\Flexslider\Elements;

use DNADesign\Elemental\Models\BaseElement;
use Dynamic\FlexSlider\Model\SlideImage;
use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\FieldType\DBField;

/**
 * Class ElementSlideshow
 * @package Dynamic\Elements\Flexslider\Elements
 *
 * @property string Content
 */
class ElementSlideshow extends BaseElement
{
    /**
     * @var string
     */
    private static $icon = 'font-icon-block-carousel';

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

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

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

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

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

        'Content' => 'HTMLText',
    ];

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

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

        'Slides',
    ];

    /**
     * @var bool
     */
    private static $inline_editable = false;

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

    /**
     * @var string
     */
    private static $slide_tab_title = 'Main';

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

    /**
     * @param bool $includerelations
     * @return array
     */
    public function fieldLabels($includerelations = true)
    {
        $labels = parent::fieldLabels($includerelations);

        $labels['Content'] = _t(__CLASS__.'.ContentLabel', 'Description');

        return $labels;
    }

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $fields) {
            $fields->dataFieldByName('Content')
                ->setRows(5)
                ->setDescription(_t(
                    __CLASS__ . '.ContentDescription',
                    'optional. Add introductory copy to your slideshow.'
                ));
        });

        return parent::getCMSFields();
    }

    /**
     * @return \SilverStripe\ORM\FieldType\DBHTMLText
     */
    public function getSummary()
    {
        $count = $this->Slides()->count();

The method Slides() does not exist on Dynamic\Elements\Flexslider\Elements\ElementSlideshow. Since you implemented __call, consider adding a @method annotation.

        $label = _t(
            SlideImage::class . '.PLURALS',
            '{count} Slide|{count} Slides',
            [ 'count' => $count ]
        );
        return DBField::create_field('HTMLText', $label)->Summary(20);
    }

    /**
     * @return array
     */
    protected function provideBlockSchema()
    {
        $blockSchema = parent::provideBlockSchema();
        $blockSchema['content'] = $this->getSummary();
        return $blockSchema;
    }

    /**
     * @return string
     */
    public function getType()
    {
        return _t(__CLASS__.'.BlockType', 'Slideshow');
    }
}