SilverStripe\Comments\Tests\Stubs\CommentableItem

Complexity

Total Complexity

9

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	RelativeLink()	0	4	1
A	canView()	0	4	1
B	canEdit()	0	15	5
A	Link()	0	4	1
A	AbsoluteLink()	0	4	1

<?php

namespace SilverStripe\Comments\Tests\Stubs;

use SilverStripe\Comments\Extensions\CommentsExtension;
use SilverStripe\Control\Director;
use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Member;
use SilverStripe\Security\Permission;

/**
 * @package comments
 * @subpackage tests
 */
class CommentableItem extends DataObject implements TestOnly
{
    private static $db = array(

Consider using a different property name as you override a private property of the parent class.

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

        'Title' => 'Varchar'
    );

    private static $extensions = array(

Consider using a different property name as you override a private property of the parent class.

The property $extensions is not used and could be removed.

        CommentsExtension::class
    );

    public function RelativeLink()
    {
        return 'CommentableItemController';
    }

    public function canView($member = null)
    {
        return true;
    }

    // This is needed for canModerateComments
    public function canEdit($member = null)
    {
        if ($member instanceof Member) {
            $memberID = $member->ID;
        } elseif (is_numeric($member)) {
            $memberID = $member;
        } else {
            $memberID = Member::currentUserID();
        }

        if ($memberID && Permission::checkMember($memberID, array('ADMIN', 'CMS_ACCESS_CommentAdmin'))) {

This if statement, and the following return statement can be replaced with return $memberID && \SilverStripe\Security\Permission::checkMember($memberID, array('ADMIN', 'CMS_ACCESS_CommentAdmin'));.

            return true;
        }
        return false;
    }

    public function Link()
    {
        return $this->RelativeLink();
    }

    public function AbsoluteLink()
    {
        return Director::absoluteURL($this->RelativeLink());
    }
}