CopyFactoryLog

Complexity

Total Complexity

13

Size/Duplication

Total Lines	174
Duplicated Lines	18.97 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	canEdit()	0	4	1
A	canDelete()	0	4	1
A	getCMSFields()	0	39	1
A	getCopyCause()	11	11	3
A	getCopyFrom()	11	11	3
A	getCopyInto()	11	11	3
A	getFormattedShortAction()	0	4	1

<?php

/**
 * keeps track of any changes being made, fake or real
 *
 *
 */

class CopyFactoryLog extends DataObject
{

    /**
     * we can write faster to MyISAM?
     *
     */
    private static $create_table_options = array(

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

        'MySQLDatabase' => 'ENGINE=MyISAM'
    );

    private static $db = array(

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

        "StartTime" => "SS_Datetime",
        "Type" => "Enum('Unknown,Fake,Real', 'Unknown')",
        "CopyCausingClassName" => "Varchar(200)",
        "CopyCausingClassNameID" => "Int",
        "CopyFromClassNameID" => "Int",
        "CopyIntoClassName" => "Varchar(200)",
        "CopyIntoClassNameID" => "Int",
        "Action" => "Text"
    );

    private static $indexes = array(

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

        "CopyCausingClassName" => true,
        "CopyCausingClassNameID" => true,
        "CopyFromClassNameID" => true,
        "CopyIntoClassName" => true,
        "CopyIntoClassNameID" => true
    );

    private static $summary_fields = array(

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

        "StartTime",
        "Type",
        "CopyCause",
        "CopyFrom",
        "CopyInto",
        "FormattedShortAction"
    );

    private static $field_labels = array(

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

        "StartTime" => "Started",
        "Type" => "Real or Fake",
        "CopyCause" => "Started by",
        "CopyFrom" => "From",
        "CopyInto" => "Into",
        "Action" => "Description"
    );

    private static $searchable_fields = array(

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

        "Type" => "PartialMatchFilter",
        "Action" => "PartialMatchFilter"
    );

    private static $casting = array(

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

        "CopyCause" => "Varchar",
        "CopyFrom" => "Varchar",
        "CopyInto" => "Varchar",
        "FormattedShortAction" => "HTMLText"
    );

    private static $default_sort = "Created ASC";

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

    public function canEdit($member = null)
    {
        return false;
    }

    public function canDelete($member = null)
    {
        return false;
    }

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab(
            "Root.Main",
            new ReadonlyField(
                "CopyCause",
                _t("CopyFactory.COPY_CAUSE", "Copy initiated by"),
                $this->getCopyCause()
            ),
            "CopyCausingClassName"
        );
        $fields->addFieldToTab(
            "Root.Main",
            new ReadonlyField(
                "CopyFrom",
                _t("CopyFactory.COPY_FROM", "Copying from"),
                $this->getCopyFrom()
            ),
            "CopyFromClassNameID"
        );
        $fields->addFieldToTab(
            "Root.Main",
            new ReadonlyField(
                "CopyInto",
                _t("CopyFactory.COPY_INTO", "Copying into"),
                $this->getCopyInto()
            ),
            "CopyIntoClassName"
        );
        $fields->addFieldToTab(
            "Root.Main",
            new LiteralField(
                "Action",
                "<pre>$this->Action</pre>"

The property Action does not seem to exist. Did you mean allowed_actions?

            )
        );
        return $fields;
    }

    /**
     * Copy Causer Description
     * @casted
     * @return Str
     */
    public function getCopyCause()

This method seems to be duplicated in your project.

    {
        $className = $this->CopyCausingClassName;

The property CopyCausingClassName does not seem to exist. Did you mean ClassName?

        if ($className) {
            $obj = $className::get()->byID($this->CopyCausingClassNameID);

The property CopyCausingClassNameID does not seem to exist. Did you mean ClassName?

            if ($obj) {
                return CopyFactory::title_for_object($obj)." (".$className.")";
            }
        }
        return _t("CopyFactory.N_A", "n/a");
    }

    /**
     * Copy From Description
     * @casted
     * @return Str
     */
    public function getCopyFrom()

This method seems to be duplicated in your project.

    {
        $className = $this->CopyIntoClassName;

The property CopyIntoClassName does not seem to exist. Did you mean ClassName?

        if ($className) {
            $obj = $className::get()->byID($this->CopyFromClassNameID);

The property CopyFromClassNameID does not seem to exist. Did you mean ClassName?

            if ($obj) {
                return CopyFactory::title_for_object($obj)." (".$className.")";
            }
        }
        return _t("CopyFactory.N_A", "n/a");
    }

    /**
     * Copy Into Description
     * @casted
     * @return Str
     */
    public function getCopyInto()

This method seems to be duplicated in your project.

    {
        $className = $this->CopyIntoClassName;

The property CopyIntoClassName does not seem to exist. Did you mean ClassName?

        if ($className) {
            $obj = $className::get()->byID($this->CopyIntoClassNameID);

The property CopyIntoClassNameID does not seem to exist. Did you mean ClassName?

            if ($obj) {
                return CopyFactory::title_for_object($obj)." (".$className.")";
            }
        }
        return _t("CopyFactory.N_A", "n/a");
    }


    /**
     * Copy Into Description
     * @casted
     * @return Str

Should the return type not be DBField?

     */
    public function getFormattedShortAction()
    {
        return DBField::create_field("HTMLText", str_replace("\n", "<br />", trim(substr($this->Action, 0, 799))));

The property Action does not seem to exist. Did you mean allowed_actions?

    }
}