RegistryPageTestContact

Complexity

Total Complexity

1

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

1 Method

Rating	Name	Duplication	Size	Complexity
A	getSearchFields()	0	6	1

<?php
class RegistryPageTestContact extends DataObject implements RegistryDataInterface {

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

As per PSR2, the opening brace for this class should be on a new line.

	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)'
	);

	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.

		'FirstName' => 'First name',
		'Surname' => 'Surname'
	);

	public function getSearchFields() {
		return new FieldList(
			new TextField('FirstName', 'First name'),
			new TextField('Surname', 'Surname')
		);
	}

}

According to PSR2, the closing brace of classes should be placed on the next line directly after the body.