FAQFolder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	8	1
A	getPortletTitle()	0	3	1
A	getPortletImage()	0	9	2
A	getPortletCaption()	0	3	1

<?php
/**
 * Defines the FAQFolder page type - initial code created by ss generator
 */
class FAQFolder extends Page implements RenderableAsPortlet {

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

	static $allowed_children = array( 'FAQFolder', 'FAQ' );

The visibility should be declared for property $allowed_children.

	static $has_one = array(

The visibility should be declared for property $has_one.

		'MainImage' => 'Image'
	);

	function getCMSFields() {

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

		$fields = parent::getCMSFields();

		$fields->addFieldToTab( 'Root.Content.Image', new ImageField( 'MainImage' ) );


		return $fields;
	}


	public function getPortletTitle() {
    return $this->Title;
  }  


  // FIXME - make this more efficient
  public function getPortletImage() {
  	if ($this->MainImageID) {
    return DataObject::get_by_id('Image', $this->MainImageID);

  	} else {
  		return null;
  	}
    
  }
  
  
 
  public function getPortletCaption() {
    return '';
  }

}

class FAQFolder_Controller extends Page_Controller {

PSR1 recommends that each class should be in its own file to aid autoloaders.

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

}

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.