SheaDawson\Blocks\Model\Block

Complexity

Total Complexity

68

Size/Duplication

Total Lines	477
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	25

Test Coverage

Coverage

39.78%

Importance

Changes

0

23 Methods

Rating	Name	Duplication	Size	Complexity
A	fieldLabels()	0	15	1
A	getDefaultSearchContext()	0	14	2
A	getTypeForGridfield()	0	4	1
B	getCMSFields()	0	98	6
A	forTemplate()	0	12	3
A	BlockHTML()	0	4	1
A	onAfterDelete()	0	8	2
A	onAfterDuplicate()	0	6	1
C	canView()	0	37	16
A	canEdit()	0	4	2
A	canDelete()	0	4	2
A	canCreate()	0	4	2
A	canPublish()	0	4	2
A	providePermissions()	0	21	1
A	onAfterWrite()	0	7	2
A	pagesAffectedByChanges()	0	10	2
B	UsageListAsString()	0	16	5
A	isPublished()	0	10	3
A	isPublishedNice()	0	7	1
A	isPublishedIcon()	0	10	2
B	CSSClasses()	0	14	5
A	getCurrentPage()	0	4	1
B	getController()	0	20	5

<?php

namespace SheaDawson\Blocks\Model;

use SheaDawson\Blocks\BlockManager;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Security;
use SilverStripe\Versioned\Versioned;
use SilverStripe\ORM\FieldType\DBBoolean;
use SilverStripe\ORM\FieldType\DBHTMLText;
use SilverStripe\ORM\ValidationException;
use SilverStripe\ORM\DB;
use SilverStripe\Core\ClassInfo;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\View\Requirements;
use SilverStripe\View\SSViewer;
use SilverStripe\Control\Controller;
use SilverStripe\Security\PermissionProvider;
use SilverStripe\Security\Permission;
use SilverStripe\Security\Group;
use SilverStripe\Security\Member;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Forms\OptionsetField;
use SilverStripe\Forms\ListboxField;
use SilverStripe\Forms\Tab;

/**
 * Block
 * Subclass this basic Block with your more interesting ones.
 *
 * @author Shea Dawson <shea@silverstripe.com.au>
 */
class Block extends DataObject implements PermissionProvider
{

	private static $table_name = 'Block';

Consider using a different property name as you override a private property of the parent class.

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

    /**
     * @var array
     */
    private static $db = [

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.

        'Title' => 'Varchar(255)',
        'CanViewType' => "Enum('Anyone, LoggedInUsers, OnlyTheseUsers', 'Anyone')",
        'ExtraCSSClasses' => 'Varchar'
    ];

    /**
     * @var array
     */
    private static $many_many = [

Consider using a different property name as you override a private property of the parent class.

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

        "ViewerGroups" => Group::class,
    ];

    /**
     * @var array
     */
    private static $belongs_many_many = [

Consider using a different property name as you override a private property of the parent class.

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

        "Pages" => SiteTree::class,
        "BlockSets" => BlockSet::class,
    ];

    private static $summary_fields = [

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.

        'singular_name',
        'Title',
        'isPublishedField',
        'UsageListAsString',
    ];

    private static $searchable_fields = [

Consider using a different property name as you override a private property of the parent class.

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

        'Title',
        'ClassName',
    ];

    public function fieldLabels($includerelations = true)
    {
        $labels =  parent::fieldLabels($includerelations);

        $labels = array_merge($labels, [
            'singular_name' => _t('Block.BlockType', 'Block Type'),
            'Title' => _t('Block.Title', 'Title'),
            'isPublishedField' => _t('Block.IsPublishedField', 'Published'),
            'UsageListAsString' => _t('Block.UsageListAsString', 'Used on'),
            'ExtraCSSClasses' => _t('Block.ExtraCSSClasses', 'Extra CSS Classes'),
            'ClassName' => _t('Block.BlockType', 'Block Type'),
        ]);

        return $labels;
    }

    public function getDefaultSearchContext()
    {
        $context = parent::getDefaultSearchContext();

        $results = $this->blockManager->getBlockClasses();
        if (sizeof($results) > 1) {
            $classfield = new DropdownField('ClassName', _t('Block.BlockType', 'Block Type'));
            $classfield->setSource($results);
            $classfield->setEmptyString(_t('Block.Any', '(any)'));
            $context->addField($classfield);
        }

        return $context;
    }

    /**
     * @var array
     */
    private static $default_sort = ['Title' => 'ASC'];

Consider using a different property name as you override a private property of the parent class.

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

    /**
     * @var array
     */
    private static $dependencies = [

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

        'blockManager' => '%$blockManager',
    ];

    /**
     * @var BlockManager
     */
    public $blockManager;

    /**
     * @var BlockController
     */
    protected $controller;

    /**
     * @return mixed
     */
    public function getTypeForGridfield()
    {
        return $this->singular_name();
    }

    public function getCMSFields()
    {
        $self = $this;
        $this->beforeUpdateCMSFields(function($fields) use($self) {
            /** @var FieldList $fields */
            Requirements::add_i18n_javascript(BLOCKS_DIR . '/javascript/lang');

            // this line is a temporary patch until I can work out why this dependency isn't being
            // loaded in some cases...
            if (!$self->blockManager) {
                $self->blockManager = singleton("SheaDawson\\Blocks\\BlockManager");
            }

            // ClassNmae - block type/class field
            $classes = $self->blockManager->getBlockClasses();
            $fields->addFieldToTab('Root.Main', DropdownField::create('ClassName', _t('Block.BlockType', 'Block Type'), $classes)->addExtraClass('block-type'), 'Title');

            // BlockArea - display areas field if on page edit controller
            if (Controller::curr()->class == 'CMSPageEditController') {
                $currentPage = Controller::curr()->currentPage();
                $areas = $self->blockManager->getAreasForPageType($currentPage->ClassName);
                $fields->addFieldToTab(
                    'Root.Main',
                    $blockAreaField = DropdownField::create('ManyMany[BlockArea]', _t('Block.BlockArea', 'Block Area'), $areas),
                    'ClassName'
                );



                if (count($areas) > 1) {
                    $blockAreaField->setEmptyString('(Select one)');
                }

                if (BlockManager::config()->get('block_area_preview')) {
                    $blockAreaField->setRightTitle($currentPage->areasPreviewButton());
                }
            }

            $fields->removeFieldFromTab('Root', 'BlockSets');
            $fields->removeFieldFromTab('Root', 'Pages');

            // legacy fields, will be removed in later release
            $fields->removeByName('Weight');
            $fields->removeByName('Area');
            $fields->removeByName('Published');

            if ($self->blockManager->getUseExtraCSSClasses()) {
                $fields->addFieldToTab('Root.Main', $fields->dataFieldByName('ExtraCSSClasses'), 'Title');
            } else {
                $fields->removeByName('ExtraCSSClasses');
            }

            // Viewer groups
            $fields->removeFieldFromTab('Root', 'ViewerGroups');
            $groupsMap = Group::get()->map('ID', 'Breadcrumbs')->toArray();
            asort($groupsMap);
            $viewersOptionsField = new OptionsetField(
                'CanViewType',
                _t('SiteTree.ACCESSHEADER', 'Who can view this page?')
            );
            $viewerGroupsField = ListboxField::create('ViewerGroups', _t('SiteTree.VIEWERGROUPS', 'Viewer Groups'))
                ->setSource($groupsMap)
                ->setAttribute(
                    'data-placeholder',
                    _t('SiteTree.GroupPlaceholder', 'Click to select group')
                );
            $viewersOptionsSource = [];
            $viewersOptionsSource['Anyone'] = _t('SiteTree.ACCESSANYONE', 'Anyone');
            $viewersOptionsSource['LoggedInUsers'] = _t('SiteTree.ACCESSLOGGEDIN', 'Logged-in users');
            $viewersOptionsSource['OnlyTheseUsers'] = _t('SiteTree.ACCESSONLYTHESE', 'Only these people (choose from list)');
            $viewersOptionsField->setSource($viewersOptionsSource)->setValue('Anyone');

            $fields->addFieldToTab('Root', new Tab('ViewerGroups', _t('Block.ViewerGroups', 'Viewer Groups')));
            $fields->addFieldsToTab('Root.ViewerGroups', [
                $viewersOptionsField,
                $viewerGroupsField,
            ]);

            // Disabled for now, until we can list ALL pages this block is applied to (inc via sets)
            // As otherwise it could be misleading
            // Show a GridField (list only) with pages which this block is used on
            // $fields->removeFieldFromTab('Root.Pages', 'Pages');

73% of this comment could be valid code. Did you maybe forget this after debugging?

            // $fields->addFieldsToTab('Root.Pages',

72% of this comment could be valid code. Did you maybe forget this after debugging?

            //         new GridField(
            //                 'Pages',
            //                 'Used on pages',
            //                 $self->Pages(),

72% of this comment could be valid code. Did you maybe forget this after debugging?

            //                 $gconf = GridFieldConfig_Base::create()));

54% of this comment could be valid code. Did you maybe forget this after debugging?

            // enhance gridfield with edit links to pages if GFEditSiteTreeItemButtons is available
            // a GFRecordEditor (default) combined with BetterButtons already gives the possibility to
            // edit versioned records (Pages), but STbutton loads them in their own interface instead
            // of GFdetailform
            // if(class_exists('GridFieldEditSiteTreeItemButton')) {

70% of this comment could be valid code. Did you maybe forget this after debugging?

            //     $gconf->addComponent(new GridFieldEditSiteTreeItemButton());

67% of this comment could be valid code. Did you maybe forget this after debugging?

            // }
        });
        return parent::getCMSFields();
    }

    /**
     * Renders this block with appropriate templates
     * looks for templates that match BlockClassName_AreaName
     * falls back to BlockClassName.
     *
     * @return string
     **/
    public function forTemplate()
    {
        if ($this->BlockArea) {

The property BlockArea does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

            $template = [$this->class.'_'.$this->BlockArea];

The property class does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

The property BlockArea does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

            if (SSViewer::hasTemplate($template)) {
                return $this->renderWith($template);
            }
        }

        return $this->renderWith($this->ClassName, $this->getController());
    }

    /**
     * @return string
     */
    public function BlockHTML()
    {
        return $this->forTemplate();
    }

    /*
     * Checks if deletion was an actual deletion, not just unpublish
     * If so, remove relations
     */
    public function onAfterDelete()
    {
        parent::onAfterDelete();
        if (Versioned::get_stage() == 'Stage') {
            $this->Pages()->removeAll();

The method Pages does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

            $this->BlockSets()->removeAll();

The method BlockSets does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        }
    }

    /**
     * Remove relations onAfterDuplicate.
     */
    public function onAfterDuplicate()
    {
        // remove relations to pages & blocksets duplicated from the original item
        $this->Pages()->removeAll();

The method Pages does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        $this->BlockSets()->removeAll();

The method BlockSets does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

    }

    /*
     * Base permissions
     */
    public function canView($member = null)
    {
        if (!$member || !(is_a($member, 'Member')) || is_numeric($member)) {
            $member = Security::getCurrentUser()->ID;
        }

        // admin override
        if ($member && Permission::checkMember($member, ['ADMIN', 'SITETREE_VIEW_ALL'])) {
            return true;
        }

        // Standard mechanism for accepting permission changes from extensions
        $extended = $this->extendedCan('canView', $member);
        if ($extended !== null) {
            return $extended;
        }

        // check for empty spec
        if (!$this->CanViewType || $this->CanViewType == 'Anyone') {

The property CanViewType does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

            return true;
        }

        // check for any logged-in users
        if ($this->CanViewType == 'LoggedInUsers' && $member) {

The property CanViewType does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

            return true;
        }

        // check for specific groups
        if ($member && is_numeric($member)) {
            $member = Member::get()->byID($member);
        }
        if ($this->CanViewType == 'OnlyTheseUsers' && $member && $member->inGroups($this->ViewerGroups())) {

The property CanViewType does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

The method ViewerGroups does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

            return true;
        }

        return false;
    }

    public function canEdit($member = null)
    {
        return Permission::check('ADMIN', 'any', $member) || Permission::check('BLOCK_EDIT', 'any', $member);
    }

    public function canDelete($member = null)
    {
        return Permission::check('ADMIN', 'any', $member) || Permission::check('BLOCK_DELETE', 'any', $member);
    }

    public function canCreate($member = null, $context = [])
    {
        return Permission::check('ADMIN', 'any', $member) || Permission::check('BLOCK_CREATE', 'any', $member);
    }

    public function canPublish($member = null)
    {
        return Permission::check('ADMIN', 'any', $member) || Permission::check('BLOCK_PUBLISH', 'any', $member);
    }

    public function providePermissions()
    {
        return [
            'BLOCK_EDIT' => [
                'name' => _t('Block.EditBlock', 'Edit a Block'),
                'category' => _t('Block.PermissionCategory', 'Blocks'),
            ],
            'BLOCK_DELETE' => [
                'name' => _t('Block.DeleteBlock', 'Delete a Block'),
                'category' => _t('Block.PermissionCategory', 'Blocks'),
            ],
            'BLOCK_CREATE' => [
                'name' => _t('Block.CreateBlock', 'Create a Block'),
                'category' => _t('Block.PermissionCategory', 'Blocks'),
            ],
            'BLOCK_PUBLISH' => [
                'name' => _t('Block.PublishBlock', 'Publish a Block'),
                'category' => _t('Block.PermissionCategory', 'Blocks'),
            ],
        ];
    }

    public function onAfterWrite()
    {
        parent::onAfterWrite();
        if ($this->hasExtension('FilesystemPublisher')) {
            $this->republish($this);

The method republish does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        }
    }

    /**
     * Get a list of URL's to republish when this block changes
     * if using StaticPublisher module.
     */
    public function pagesAffectedByChanges()
    {
        $pages = $this->Pages();

The method Pages does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        $urls = [];
        foreach ($pages as $page) {
            $urls[] = $page->Link();
        }

        return $urls;
    }

    /*
     * Get a list of Page and Blockset titles this block is used on
     */
    public function UsageListAsString()
    {
        $pages = implode(', ', $this->Pages()->column('URLSegment'));

The method Pages does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        $sets = implode(', ', $this->BlockSets()->column('Title'));

The method BlockSets does not exist on object<SheaDawson\Blocks\Model\Block>? Since you implemented __call, maybe consider adding a @method annotation.

        $_t_pages = _t('Block.PagesAsString', 'Pages: {pages}', '', ['pages' => $pages]);
        $_t_sets = _t('Block.BlockSetsAsString', 'Block Sets: {sets}', '', ['sets' => $sets]);
        if ($pages && $sets) {
            return "$_t_pages<br />$_t_sets";
        }
        if ($pages) {
            return $_t_pages;
        }
        if ($sets) {
            return $_t_sets;
        }
    }

    /**
     * Check if this block has been published.
     *
     * @return bool True if this page has been published.
     */
    public function isPublished()
    {
        if (!$this->exists()) {
            return false;
        }

        return (DB::query("SELECT \"ID\" FROM \"Block_Live\" WHERE \"ID\" = $this->ID")->value())
            ? 1
            : 0;
    }

    /**
     * Check if this block has been published.
     *
     * @return bool True if this page has been published.
     */
    public function isPublishedNice()
    {
        $field = DBBoolean::create('isPublished');
        $field->setValue($this->isPublished());

        return $field->Nice();
    }

    /**
     * @return DBHTMLText
     */
    public function isPublishedIcon()
    {
        $obj = DBHTMLText::create();
        if ($this->isPublished()) {

The expression $this->isPublished() of type false|integer is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

            $obj->setValue('<img src="' . FRAMEWORK_ADMIN_DIR . '/images/alert-good.gif" />');
        } else {
            $obj->setValue('<img src="' . FRAMEWORK_ADMIN_DIR . '/images/alert-bad.gif" />');
        }
        return $obj;
    }

    /**
     * CSS Classes to apply to block element in template.
     *
     * @return string $classes
     */
    public function CSSClasses($stopAtClass = 'DataObject')
    {
        $classes = strtolower(parent::CSSClasses($stopAtClass));

        if (!empty($classes) && ($prefix = $this->blockManager->getPrefixDefaultCSSClasses())) {
            $classes = $prefix.str_replace(' ', " {$prefix}", $classes);
        }

        if ($this->blockManager->getUseExtraCSSClasses()) {
            $classes = $this->ExtraCSSClasses ? $classes." $this->ExtraCSSClasses" : $classes;

The property ExtraCSSClasses does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

        }

        return $classes;
    }

    /**
     * Access current page scope from Block templates with $CurrentPage
     *
     * @return Controller
     */
    public function getCurrentPage()
    {
        return Controller::curr();
    }

    /**
     * @throws Exception
     *
     * @return BlockController
     */
    public function getController()
    {
        if ($this->controller) {
            return $this->controller;
        }
        foreach (array_reverse(ClassInfo::ancestry($this->class)) as $blockClass) {

The property class does not exist on object<SheaDawson\Blocks\Model\Block>. Since you implemented __get, maybe consider adding a @property annotation.

            $controllerClass = "{$blockClass}Controller";
            if (class_exists($controllerClass)) {
                break;
            }
        }
        if (!class_exists($controllerClass)) {

The variable $controllerClass does not seem to be defined for all execution paths leading up to this point.

            throw new ValidationException("Could not find controller class for $this->classname");

The property classname does not seem to exist. Did you mean ClassName?

        }

        $this->controller = Injector::inst()->create($controllerClass, $this);
        $this->controller->init();

        return $this->controller;
    }
}