VideoSlide::getCMSFields() - Code Metrics - Inspection of "REFACTOR Slide models" - dynamic/silverstripe-flexslider - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#198)

by Nic

created 2019-12-27 21:16 UTC

VideoSlide::getCMSFields() A

↳ Parent: VideoSlide

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	9
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	14
rs	9.9666

<?php

namespace Dynamic\FlexSlider\Model;

use SilverStripe\AssetAdmin\Forms\UploadField;
use SilverStripe\Assets\File;
use SilverStripe\Forms\FieldList;

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


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


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


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

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

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

        'Video',
    ];

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $fields) {
            $fields->addFieldToTab(
                'Root.Main',
                UploadField::create('Video')
                    ->setTitle('Video')
                    ->setDescription('mp4 format')
                    ->setAllowedExtensions(['mp4']),
                'Content'
            );
        });

        return parent::getCMSFields();
    }
}


1			<?php
2
3			namespace Dynamic\FlexSlider\Model;
4
5			use SilverStripe\AssetAdmin\Forms\UploadField;
6			use SilverStripe\Assets\File;
7			use SilverStripe\Forms\FieldList;
8
9			/**
10			* Class VideoSlide
11			* @package Dynamic\FlexSlider\Model
12			*/
13			class VideoSlide extends Slide
14			{
15			/**
16			* @var string
17			*/
18			private static $singular_name = 'Video Slide';
			0 ignored issues – show introduced 2019-12-27 20:53 UTC by Report Bug Copy Issue Report The private property `$singular_name` is not used, and could be removed. Loading history...
19
20			/**
21			* @var string
22			*/
23			private static $plural_name = 'Video Slides';
			0 ignored issues – show introduced 2019-12-27 20:53 UTC by Report Bug Copy Issue Report The private property `$plural_name` is not used, and could be removed. Loading history...
24
25			/**
26			* @var string
27			*/
28			private static $table_name = 'VideoSlide';
			0 ignored issues – show introduced 2019-12-27 20:53 UTC by Report Bug Copy Issue Report The private property `$table_name` is not used, and could be removed. Loading history...
29
30			/**
31			* @var array
32			*/
33			private static $has_one = [
			0 ignored issues – show introduced 2019-12-27 20:53 UTC by Report Bug Copy Issue Report The private property `$has_one` is not used, and could be removed. Loading history...
34			'Video' => File::class,
35			];
36
37			/**
38			* @var array
39			*/
40			private static $owns = [
			0 ignored issues – show introduced 2019-12-27 20:53 UTC by Report Bug Copy Issue Report The private property `$owns` is not used, and could be removed. Loading history...
41			'Video',
42			];
43
44			/**
45			* @return FieldList
46			*/
47			public function getCMSFields()
48			{
49			$this->beforeUpdateCMSFields(function (FieldList $fields) {
50			$fields->addFieldToTab(
51			'Root.Main',
52			UploadField::create('Video')
53			->setTitle('Video')
54			->setDescription('mp4 format')
55			->setAllowedExtensions(['mp4']),
56			'Content'
57			);
58			});
59
60			return parent::getCMSFields();
61			}
62			}
63

dynamic / silverstripe-flexslider

Pull Request — master (#198)

VideoSlide::getCMSFields() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like