Contact

Complexity

Total Complexity

3

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	Title()	0	4	1
A	getTitle()	0	4	1
A	getCMSFields()	0	10	1

<?php

class Contact extends DataObject

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

{
    private static $db = array(

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.

        'FirstName' => 'Varchar(50)',
        'Surname' => 'Varchar(50)',
        'Email' => 'EmailAddress',
        'BusinessName' => 'Varchar(50)',
        'Website' => 'Varchar(50)',
        'IsProfessional' => 'Boolean',
        'IsVisible' => 'Boolean',
        'IsHighlighted' => 'Boolean',
        'Reference' => 'Varchar(50)'
    );

    private static $has_one = array(

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.

        'Location' => 'ContactLocation',
    );

    private static $many_many = array(

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

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

        'Type'  => 'ContactCategory'
    );

    private static $summary_fields = array(

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.

        'Title' => 'Name',
        'BusinessName' => 'BusinessName'
    );

    private static $field_labels = array(

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

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

        'LocationID' => 'Locationnnn'
    );

    private static $searchable_fields = array(

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

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

        'Email' => 'PartialMatchFilter',
        'FirstName' => 'PartialMatchFilter',
        'Surname' => 'PartialMatchFilter',
        'BusinessName' => 'PartialMatchFilter',
        'Website' => 'PartialMatchFilter',
        'IsProfessional' => 'ExactMatchFilter',
        'IsVisible' => 'ExactMatchFilter',
        'IsHighlighted' => 'ExactMatchFilter',
        'Reference' => 'PartialMatchFilter'
    );

    /**
     * @inherited
     */
    private static $casting = array(

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

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

        'Title' => "Varchar"
    );

    /**
     * @inherited
     */
    private static $default_sort = array(

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.

        'BusinessName' => "ASC"
    );

    private static $singular_name = 'Contact';

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

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

    private static $plural_name = 'Contacts';

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

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

    /**
     * @return String
     */
    public function Title()
    {
        return $this->getTitle();
    }
    public function getTitle()
    {
        return implode(' ', array($this->FirstName, $this->Surname));

The property FirstName does not exist on object<Contact>. Since you implemented __get, maybe consider adding a @property annotation.

The property Surname does not exist on object<Contact>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function getCMSFields()
    {
        $fields = parent::getCMSfields();
        $locationField = $fields->dataFieldByName('LocationID');
        $fields->replaceField(
            'LocationID',
            DropdownField::create('LocationID', $locationField->Title(), ContactLocation::get()->map())
        );
        return $fields;
    }
}