MembersOnlyPage_Controller

Complexity

Total Complexity

1

Size/Duplication

Total Lines	8
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	init()	0	5	1

<?php

File has mixed line endings; this may cause incorrect results

class MembersOnlyPage extends Page

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

{
    private static $add_action = 'Members Only Page';

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

    private static $icon = 'mysite/images/treeicons/MembersOnlyPage';

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

    private static $default_parent = 'MembersOnlyPage';

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

    private static $allowed_children = array("MembersOnlyPage");

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

    private static $group_code = "intranet-users";
    public static function set_group_code($v)
    {
        self::$group_code = $v;
    }
    public static function get_group_code()
    {
        return self::$group_code;
    }

    private static $group_name = "intranet users";
    public static function set_group_name($v)
    {
        self::$group_name = $v;
    }
    public static function get_group_name()
    {
        return self::$group_name;
    }

    private static $permission_code = "INTRANET_USERS";
    public static function set_permission_code($v)
    {
        self::$permission_code = $v;
    }
    public static function get_permission_code()
    {
        return self::$permission_code;
    }

    private static $defaults = array(

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

        "ProvideComments" => 1,
        "ShowInSearch" => 0
    );


    public function getCMSFields()

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

    {
        $fields = parent::getCMSFields();
        return $fields;
    }

    public function canView($member = null)
    {
        if ($member = Member::currentUser()) {
            if ($member->inGroup("ADMIN") || Permission::checkMember($member, self::$permission_code)) {
                return true;
            }
        }
        return false;
    }

    public function getShowInMenus()
    {
        return $this->canView();
    }

    public function ShowInMenus()
    {
        return $this->canView();
    }

    public function getShowInSearch()
    {
        return $this->canView();
    }

    public function ShowInSearch()
    {
        return $this->canView();
    }

    public function requireDefaultRecords()
    {
        parent::requireDefaultRecords();
        $intranetGroup = Group::get()->filter(array("Code" => $this->Config()->get("group_code")))->first();
        if ($intranetGroup && $intranetGroup->exists()) {

This if statement is empty and can be removed.

            //do nothing
        } else {
            $intranetGroup = new Group();
            DB::alteration_message($this->Config()->get("group_name").' group created', "created");
        }
        if ($intranetGroup) {
            $intranetGroup->Code = $this->Config()->get("group_code");
            $intranetGroup->Title = $this->Config()->get("group_name");
            $intranetGroup->write();
            Permission::grant($intranetGroup->ID, $this->Config()->get("permission_code"));
            if (DB::query("

				SELECT *

				FROM Permission

				WHERE \"GroupID\" = '".$intranetGroup->ID."'

					AND \"Code\" LIKE '".$this->Config()->get("permission_code")."'")->numRecords() == 0
            ) {
                Permission::grant($intranetGroup->ID, $this->Config()->get("permission_code"));
                DB::alteration_message($this->Config()->get("group_name").' permissions granted', "created");
            }
        }
    }
}

class MembersOnlyPage_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();
        Requirements::themedCSS("MembersOnlyPage", "membersonlypages");
    }
}