SiteMapPage

Complexity

Total Complexity

2

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	canCreate()	0	4	1
A	getCMSFields()	0	6	1

<?php


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


class SiteMapPage extends Page

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

{
    private static $db = array(

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

        "ShowAllPages" => "Boolean"
    );

    private static $description = "Sitemap Page: shows all the pages of a site in a tree format";

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

    private static $add_action = 'Site Map Page';

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

    private static $icon = 'sitemappage/images/treeicons/SiteMapPage';

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

    public function canCreate($member = null)
    {
        return ! SiteMapPage::get()->count();
    }

    public function getCMSFields()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab("Root.Display", new CheckboxField("ShowAllPages", "Show all pages from the start"));
        return $fields;
    }
}

class SiteMapPage_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.

{
    public function init()
    {
        parent::init();
        if (!$this->ShowAllPages) {
            Requirements::javascript(THIRDPARTY_DIR . '/jquery/jquery.js');
            Requirements::javascript('sitemappage/javascript/SiteMapPage.js');
            Requirements::themedCSS('SiteMapPage', "sitemappage");
        } else {
            Requirements::themedCSS('SiteMapPageOpen', "sitemappage");
        }
    }

    public function LevelOneSiteMapPages()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $homePage = SiteTree::get()->filter(array("ParentID" => 0))->first();
        return $homePage->SiteMapPages($noParent = true);
    }
}