SilverstripeColumnsPageControllerExtension

Complexity

Total Complexity

25

Size/Duplication

Total Lines	132
Duplicated Lines	9.09 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	HasFullWidthContent()	12	12	4
B	HasSideBar()	0	19	6
B	NumberOfColumns()	0	24	5
A	RecentlyUpdated()	0	18	3
A	InThisSection()	0	10	3
A	AlsoSee()	0	14	4

<?php

/**
 *@author nicolaas [at] sunnysideup.co.nz
 *
 *
 **/

class SilverstripeColumnsPageControllerExtension extends Extension

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

{

    /**
     * @return bool
     */
    function HasFullWidthContent()

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

This method seems to be duplicated in your project.

    {
        if($this->owner->hasMethod('HasFullWidthContentOverloaded')) {
            $v = $this->owner->HasFullWidthContentOverloaded();
            if($v !== null) {
                return $v;
            }
        }
        if($this->owner->owner->getFullWidthContent()) {
            return true;
        }
    }

    /**
     * @return bool
     */
    function HasSideBar()

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

    {
        if($this->owner->hasMethod('HasSideBarOverloaded')) {
            $v = $this->owner->HasSideBarOverloaded();
            if($v !== null) {
                return $v;
            }
        }
        if(
            (
                $this->owner->UseDefaultSideBarContent() &&
                strlen($this->owner->getMyDefaultSidebarContent()) > 17
            )
            ||
            $this->owner->MySidebarImage()
        ) {
            return true;
        }
    }


    /**
     * @param boolean $asClassName
     *
     * @return string | int
     */
    function NumberOfColumns($asClassName = true)

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

    {
        if($this->owner->hasMethod('NumberOfColumnsOverloaded')) {
            return $this->owner->NumberOfColumnsOverloaded();
            if($v !== null) {

if ($v !== null) { return $v; } does not seem to be reachable.

                return $v;
            }
        }
        $count = 1;
        if($this->owner->HasSideBar()) {
            $count++;
        }
        if($asClassName) {
            $array = array(
                1 => 'one',
                2 => 'two',
                3 => 'three'
            );
            return $array[$count];
        }
        else {
            return $count;
        }
    }


    /**
     * returns a data list of items that have been edited last - up to one day ago.
     * This ensures that we do not show stuff we have just fixed up...
     * @return DataList
     */
    function RecentlyUpdated($limit = 5)

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

    {
        if($this->owner->hasMethod('RecentlyUpdatedOverloaded')) {
            $v = $this->owner->RecentlyUpdatedOverloaded();
            if($v !== null) {
                return $v;
            }
        }
        return Page::get()
            ->filter(
                array(
                    'ShowInSearch' => true,
                    'LastEdited:LessThan' => date('Y-m-d h:i:s', time() - 86400)
                )
            )
            ->sort(array('LastEdited' => 'DESC'))
            ->limit($limit);
    }

    /**
     * @return null | DataList
     */
    function InThisSection()

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

    {
        if($this->owner->hasMethod('InThisSectionOverloaded')) {
            $v = $this->owner->InThisSectionOverloaded();
            if($v !== null) {
                return $v;
            }
        }
        return Page::get()->filter(array('ParentID' => $this->owner->ID, 'ShowInMenus' => 1));
    }

    /**
     * Siblings
     * @return null | DataList
     */
    function AlsoSee()

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

    {
        if($this->owner->hasMethod('AlsoSeeOverloaded')) {
            $v = $this->owner->AlsoSeeOverloaded();
            if($v !== null) {
                return $v;
            }
        }
        if($this->owner->ParentID) {
            return Page::get()
                ->filter(array('ParentID' => $this->owner->ParentID, 'ShowInMenus' => 1))
                ->exclude(array('ID' => $this->owner->ID));
        }
    }

}