SunnysideUp\ShareThis\MyTwitterData

Complexity

Total Complexity

7

Size/Duplication

Total Lines	107
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	forTemplate()	0	4	1
A	Link()	0	4	1
A	canView()	0	4	1
A	canCreate()	0	4	1
A	canEdit()	0	4	1
A	canDelete()	0	4	1
A	HideNice()	0	4	1

<?php

namespace SunnysideUp\ShareThis;

use SilverStripe\Core\Config\Config;
use SilverStripe\Security\Permission;
use SilverStripe\ORM\DataObject;

/**
 * MyTwitterData
 */
class MyTwitterData extends DataObject
{
    /**
     * @var string
     */
    private static $table_name = 'MyTwitterData';

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

    /**
     * @var string
     */
    private static $username = "";

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

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

        "Date" => "Datetime",
        "TwitterID" => "Varchar(64)",
        "Title" => "HTMLText",
        "Hide" => "Boolean"
    ];

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

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

        "Date" => "Date",
        "Title" => "Title",
        "HideNice" => "Hide"
    ];

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

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

        "TwitterID" => true
    ];

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

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

        "Link" => "Varchar",
        "HideNice" => "Varchar"
    ];

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

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

    /**
     * @return string
     */
    public function forTemplate()
    {
        return $this->Title;
    }

    /**
     * @return string
     */
    public function Link()
    {
        return "https://twitter.com/".Config::inst()->get(MyTwitterData::class, "username")."/status/".$this->TwitterID;

The property TwitterID does not exist on object<SunnysideUp\ShareThis\MyTwitterData>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * @return boolean

Should the return type not be boolean|string?

     */
    public function canView($member = null)
    {
        return Permission::checkMember($member, 'SOCIAL_MEDIA');
    }

    /**
     * @return boolean
     */
    public function canCreate($member = null, $context = [])
    {
        return false;
    }

    /**
     * @return boolean

Should the return type not be boolean|string?

     */
    public function canEdit($member = null)
    {
        return Permission::checkMember($member, 'SOCIAL_MEDIA');
    }

    /**
     * @return boolean
     */
    public function canDelete($member = null)
    {
        return false;
    }

    /**
     * @return boolean
     */
    public function HideNice()
    {
        return $this->dbObject('Hide')->Nice();
    }
}