SilverStripe\Blog\Model\BlogObject

Complexity

Total Complexity

23

Size/Duplication

Total Lines	221
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	BlogPosts()	0	8	1
A	getCMSFields()	0	15	1
B	validate()	0	19	5
A	getLink()	0	8	1
A	canView()	0	10	2
A	canCreate()	0	12	2
A	canDelete()	0	10	2
A	canEdit()	0	10	2
A	onBeforeWrite()	0	7	2
A	generateURLSegment()	0	17	3
A	getDuplicatesByField()	0	16	2
	getListUrlSegment()	0	1	?
	getDuplicateError()	0	1	?

<?php

namespace SilverStripe\Blog\Model;

use SilverStripe\Control\Controller;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\Tab;
use SilverStripe\Forms\TabSet;
use SilverStripe\Forms\TextField;
use SilverStripe\ORM\DataList;
use SilverStripe\Security\Permission;
use SilverStripe\View\Parsers\URLSegmentFilter;

/**
 * An object shared by BlogTag and BlogCategory.
 *
 * @package silverstripe
 * @subpackage blog
 */
trait BlogObject
{
    /**
     * @return DataList
     */
    public function BlogPosts()
    {
        $blogPosts = parent::BlogPosts();

        $this->extend('updateGetBlogPosts', $blogPosts);

It seems like extend() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $blogPosts;
    }

    /**
     * {@inheritdoc}
     */
    public function getCMSFields()
    {
        $fields = TabSet::create(
            'Root',
            Tab::create(
                'Main',
                TextField::create('Title', _t(self::class . '.Title', 'Title'))
            )
        );

        $fields = FieldList::create($fields);
        $this->extend('updateCMSFields', $fields);

It seems like extend() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $fields;
    }

    /**
     * {@inheritdoc}
     * @return ValidationResult
     */
    public function validate()
    {
        /** @var ValidationResult $validation */
        $validation = parent::validate();
        if (!$validation->isValid()) {
            return $validation;
        }

        $blog = $this->Blog();

It seems like Blog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (!$blog || !$blog->exists()) {
            return $validation;
        }

        if ($this->getDuplicatesByField('Title')->count() > 0) {
            $validation->addError($this->getDuplicateError(), self::DUPLICATE_EXCEPTION);
        }

        return $validation;
    }

    /**
     * Returns a relative link to this category.
     *
     * @return string
     */
    public function getLink()
    {
        return Controller::join_links(
            $this->Blog()->Link(),

It seems like Blog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->getListUrlSegment(),
            $this->URLSegment

The property URLSegment does not exist. Did you maybe forget to declare it?

        );
    }

    /**
     * Inherits from the parent blog or can be overwritten using a DataExtension.
     *
     * @param null|Member $member
     *
     * @return bool
     */
    public function canView($member = null)
    {
        $extended = $this->extendedCan(__FUNCTION__, $member);

It seems like extendedCan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($extended !== null) {
            return $extended;
        }

        return $this->Blog()->canView($member);

It seems like Blog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritdoc}
     */
    public function canCreate($member = null, $context = array())

The parameter $context is not used and could be removed.

    {
        $extended = $this->extendedCan(__FUNCTION__, $member);

It seems like extendedCan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($extended !== null) {
            return $extended;
        }

        $permission = Blog::config()->grant_user_permission;

        return Permission::checkMember($member, $permission);
    }

    /**
     * Inherits from the parent blog or can be overwritten using a DataExtension.
     *
     * @param null|Member $member
     *
     * @return bool
     */
    public function canDelete($member = null)
    {
        $extended = $this->extendedCan(__FUNCTION__, $member);

It seems like extendedCan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($extended !== null) {
            return $extended;
        }

        return $this->Blog()->canDelete($member);

It seems like Blog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Inherits from the parent blog or can be overwritten using a DataExtension.
     *
     * @param null|Member $member
     *
     * @return bool
     */
    public function canEdit($member = null)
    {
        $extended = $this->extendedCan(__FUNCTION__, $member);

It seems like extendedCan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($extended !== null) {
            return $extended;
        }

        return $this->Blog()->canEdit($member);

It seems like Blog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * {@inheritdoc}
     */
    protected function onBeforeWrite()
    {
        parent::onBeforeWrite();
        if (empty($this->URLSegment)) {
            return $this->generateURLSegment();
        }
    }

    /**
     * Generates a unique URLSegment from the title.
     *
     * @param int $increment
     *
     * @return string
     */
    public function generateURLSegment($increment = 0)
    {
        $increment = (int) $increment;
        $filter = URLSegmentFilter::create();

        $this->URLSegment = $filter->filter($this->Title);

The property Title does not exist. Did you maybe forget to declare it?

        if ($increment > 0) {
            $this->URLSegment .= '-' . $increment;
        }

        if ($this->getDuplicatesByField('URLSegment')->count() > 0) {
            $this->generateURLSegment($increment + 1);
        }

        return $this->URLSegment;
    }

    /**
     * Looks for objects o the same type and the same value by the given Field
     *
     * @param  string $field E.g. URLSegment or Title
     * @return DataList
     */
    protected function getDuplicatesByField($field)
    {
        $duplicates = DataList::create(self::class)
            ->filter(
                [
                    $field   => $this->$field,
                    'BlogID' => (int) $this->BlogID

The property BlogID does not exist. Did you maybe forget to declare it?

                ]
            );

        if ($this->ID) {
            $duplicates = $duplicates->exclude('ID', $this->ID);

The property ID does not exist. Did you maybe forget to declare it?

        }

        return $duplicates;
    }

    /**
     * This returns the url segment for the listing page.
     * eg. 'categories' in /my-blog/categories/category-url
     *
     * This is not editable at the moment, but a method is being used incase we want
     * to make it editable in the future. We can use this method to provide logic
     * without replacing multiple areas of the code base. We're also not being opinionated
     * about how the segment should be obtained at the moment and allowing for the
     * implementation to decide.
     *
     * @return string
     */
    abstract protected function getListUrlSegment();

    /**
     * Returns an error message for this object when it tries to write a duplicate.
     *
     * @return string
     */
    abstract protected function getDuplicateError();
}