SilverStripe\ContentReview\Models\ContentReviewLog

Complexity

Total Complexity

1

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	canView()	0	4	1

<?php

namespace SilverStripe\ContentReview\Models;

use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Member;
use SilverStripe\Security\Security;

class ContentReviewLog extends DataObject
{
    /**
     * @var array
     */
    private static $db = [

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.

        "Note" => "Text",
    ];

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

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

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

        "Reviewer" => Member::class,
        "SiteTree" => SiteTree::class,
    ];

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

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

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

        "Note"           => ["title" => "Note"],
        "Created"        => ["title" => "Reviewed at"],
        "Reviewer.Title" => ["title" => "Reviewed by"]
    ];

    /**
     * @var string
     */
    private static $default_sort = "Created DESC";

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

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

    private static $table_name = 'ContentReviewLog';

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

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

    /**
     * @param mixed $member
     *
     * @return bool
     */
    public function canView($member = null)
    {
        return (bool) Security::getCurrentUser();
    }
}