Sunnysideup\CMSNiceties\Traits\CMSNicetiesTraitForCMSLinks

Complexity

Total Complexity

14

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes	13
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	CMSEditLinkField()	0	18	4
A	CMSListLink()	0	11	2
A	CMSAddLink()	0	3	1
A	myModelAdminController()	0	10	2
A	CMSEditLink()	0	6	2
A	CMSEditLinkLimited()	0	3	1
A	sanitisedClassName()	0	5	2

<?php

namespace Sunnysideup\CMSNiceties\Traits;

use SilverStripe\Admin\ModelAdmin;
// use SilverStripe\Forms\GridField\GridFieldArchiveAction;
use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\Control\Controller;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Forms\HTMLReadonlyField;
use Sunnysideup\CmsEditLinkField\Api\CMSEditLinkAPI;

The type Sunnysideup\CmsEditLinkField\Api\CMSEditLinkAPI was not found. Maybe you did not declare it correctly or list all dependencies?

trait CMSNicetiesTraitForCMSLinks
{
    public function CMSEditLink(): string
    {
        if ($this instanceof SiteTree) {
            return parent::CMSEditLink();
        }
        return CMSEditLinkAPI::find_edit_link_for_object($this);
    }

    public function CMSEditLinkLimited(): string
    {
        return CMSEditLinkAPI::find_edit_link_for_object($this);
    }

    public function CMSEditLinkField(string $relName, string $name = ''): HTMLReadonlyField
    {
        $obj = $this->{$relName}();
        if (!$name) {
            $nameOptions = $this->fieldLabels();

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

            $name = $nameOptions[$relName] ?? $nameOptions[$relName . 'ID'] ?? 'error';
        }

        if ($obj && $obj->exists()) {
            $value = '<a href="' . $obj->CMSEditLink() . '">' . $obj->getTitle() . '</a>';
        } else {
            $value = '<em>(none)</em>';
        }

        return HTMLReadonlyField::create(
            $relName . 'Link',
            $name,
            $value
        );
    }

    public function CMSAddLink(): string
    {
        return CMSEditLinkAPI::find_add_link_for_object($this->ClassName);
    }

    public function CMSListLink(): string
    {
        $controller = $this->myModelAdminController();
        if ($controller) {
            return Controller::join_links(
                $controller->Link(),
                $this->sanitisedClassName()
            );
        }

        return '404-cms-list-link-not-found';
    }

    /**
     * @return null|ModelAdmin
     */
    protected function myModelAdminController()
    {
        $modelAdminClassName = $this->Config()->get('primary_model_admin_class');

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

        $obj = null;
        if ($modelAdminClassName) {
            /** @var null|ModelAdmin $obj */
            $obj = Injector::inst()->get($modelAdminClassName);
        }

        return $obj;
    }

    /**
     * Sanitise a model class' name for inclusion in a link.
     */
    protected function sanitisedClassName(): string
    {
        $className = (string) $this->hasMethod('classNameForModelAdmin') ? $this->classNameForModelAdmin() : $this->ClassName;

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

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

        return str_replace('\\', '-', $className);
    }
}