CMSEditLinkField

Complexity

Total Complexity

7

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	21	6
A	setNameAppendix()	0	4	1

<?php


class CMSEditLinkField extends ReadonlyField

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    /**
     * what are we linking to?
     * @var DataObject
     */
    protected $linkedObject = null;

    /**
     * appendix for field name
     * @var String
     */
    protected $nameAppendix = '_CMSEditLink';


    /**
     *
     * @param string $name                 e.g. MyLinkedObjectID
     * @param string $title                e.g. My Fancy Title
     * @param DataObject $linkedObject     e.g. $this->MyLinkedObjectID()
     * @param string $methodOrVariable     (OPTIONAL) - e.g. MyFullTitle
     */
    public function __construct($name, $title, $linkedObject, $methodOrVariable = 'getTitle')
    {
        $name .= $this->nameAppendix;
        if ($linkedObject && $linkedObject->exists() && $linkedObject->hasMethod('CMSEditLink')) {
            $this->linkedObject = $linkedObject;
            if($this->linkedObject->hasMethod($methodOrVariable)) {
                $description = $this->linkedObject->$methodOrVariable();
            } elseif(isset($this->linkedObject->$methodOrVariable)) {
                $description = $this->linkedObject->$methodOrVariable;
            } else {
                $description = 'ERROR!';
                user_error($methodOrVariable.' does not exist on '.$this->linkedObject.' (as method or variable)');
            }
            $content = '<p class="cms-edit-link"><a href="'.$this->linkedObject->CMSEditLink().'">'.Convert::raw2xml($description).'</a></p>';
            $this->dontEscape = true;

            return parent::__construct($name, $title, $content);

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

        } else {
            return parent::__construct($name, $title);

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

        }
    }

    /**
     *
     *
     * @param string $s
     */
    function setNameAppendix($s)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->nameAppendix = $s;
    }


}