ilateral\SilverStripe\Carousel\Model\CarouselSlide

Complexity

Total Complexity

17

Size/Duplication

Total Lines	171
Duplicated Lines	0 %

Importance

Changes	5
Bugs	1	Features	2

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	20	1
A	canEdit()	0	5	3
A	canDelete()	0	5	3
A	getThumbnail()	0	6	2
A	getRenderedImage()	0	11	2
A	canCreate()	0	5	3
A	canView()	0	5	3

<?php

namespace ilateral\SilverStripe\Carousel\Model;

use SilverStripe\Assets\Image;
use gorriecoe\Link\Models\Link;
use SilverStripe\ORM\DataObject;
use gorriecoe\LinkField\LinkField;
use SilverStripe\CMS\Model\SiteTree;

/**
 * Representation of a slide object that can be extended to add extra
 * data (such as links, additional content, etc)
 * 
 * @author i-lateral (http://www.i-lateral.com)
 * @package carousel
 */
class CarouselSlide extends DataObject
{

    private static $table_name = 'CarouselSlide';

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

    /**
     * DB Columns
     * 
     * @var array
     * @config
     */
    private static $db = [

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

        'Title'     => 'Varchar(99)',
        'Sort'      => 'Int'
    ];

    /**
     * Has One relations
     * 
     * @var array
     * @config
     */
    private static $has_one = [

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

        'Parent'    => SiteTree::class,
        'Image'     => Image::class,
        'Link'		=> Link::class
    ];

    /**
     * Ownership of relations
     *
     * @var array
     */
    private static $owns = [

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

        'Image',
        'Link'
    ];

    /**
     * Default casting for functions to templates
     * 
     * @var array
     * @config
     */
    private static $casting = array(

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

        'Thumbnail' => 'Varchar'
    );

    /**
     * Summary columns/fields for this object
     * 
     * @var array
     * @config
     */
    private static $summary_fields = array(

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

        'Thumbnail' => 'Image',
        'Title'     => 'Title',
        'Link.Title'=> 'Link'
    );

    /**
     * Default sorting of this object
     * 
     * @var string
     * @config
     */
    private static $default_sort = "Sort ASC";

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

    /**
     * Default image profile to use
     *
     * @var string
     * @config
     */
    private static $default_proile = 'ShortCarousel';
    
    /**
     * Get fully rendered image for template
     *
     * @return HTMLText

The type ilateral\SilverStripe\Carousel\Model\HTMLText was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public function getRenderedImage()
    {
        $parent = $this->Parent(); 

The method Parent() does not exist on ilateral\SilverStripe\Carousel\Model\CarouselSlide. Since you implemented __call, consider adding a @method annotation.

        $profile = $parent->CarouselProfile;
        if ($profile) {
            return $this->Image->{$profile}();

The property Image does not exist on ilateral\SilverStripe\Carousel\Model\CarouselSlide. Since you implemented __get, consider adding a @property annotation.

        } else {
            $profile = $this->config()->default_proile;
        }

        return $this->Image->{$profile}();
    }

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->removeByName([
            'ParentID',
            'Sort',
            'LinkID'
        ]);

		$fields->addFieldToTab(
			'Root.Main', 
			LinkField::create(
                'Link',
                $this->fieldLabel('Link'),
                $this
            )
		);

        return $fields;
    }

    public function getThumbnail()
    {
        if($this->Image()) {

The method Image() does not exist on ilateral\SilverStripe\Carousel\Model\CarouselSlide. Since you implemented __call, consider adding a @method annotation.

            return $this->Image()->CMSThumbnail();
        } else {
            return '(No Image)';
        }
    }
    
    /**
     * Check parent permissions
     *
     * @return Boolean
     */
    public function canView($member = null) {
        $extended = $this->extend('canView', $member);
        if($extended && $extended !== null) return $extended;

        return $this->Parent()->canView($member);
    }

    /**
     * Anyone can create a carousel slide
     *
     * @return Boolean
     */
    public function canCreate($member = null, $context = []) {
        $extended = $this->extend('canCreate', $member, $context);
        if($extended && $extended !== null) return $extended;

        return true;
    }

    /**
     * Check parent permissions
     *
     * @return Boolean
     */
    public function canEdit($member = null) {
        $extended = $this->extend('canEdit', $member);
        if($extended && $extended !== null) return $extended;

        return $this->Parent()->canEdit($member);
    }

    /**
     * Check parent permissions
     *
     * @return Boolean
     */
    public function canDelete($member = null) {
        $extended = $this->extend('canDelete', $member);
        if($extended && $extended !== null) return $extended;

        return $this->Parent()->canEdit($member);
    }
}