LeKoala\EmailTemplates\Extensions\EmailSubsiteExtension

Complexity

Total Complexity

18

Size/Duplication

Total Lines	104
Duplicated Lines	0 %

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	onBeforeWrite()	0	7	3
A	canCreate()	0	3	1
A	isMainDataObject()	0	6	2
A	canView()	0	3	1
A	cacheKeyComponent()	0	3	1
B	augmentSQL()	0	35	8
A	canDelete()	0	3	1
A	canEdit()	0	3	1

<?php

namespace LeKoala\EmailTemplates\Extensions;

use LeKoala\EmailTemplates\Helpers\SubsiteHelper;
use LeKoala\EmailTemplates\Models\Emailing;
use LeKoala\EmailTemplates\Models\EmailTemplate;

The type LeKoala\EmailTemplates\Models\EmailTemplate was not found. Maybe you did not declare it correctly or list all dependencies?

use LeKoala\EmailTemplates\Models\SentEmail;
use SilverStripe\ORM\DataQuery;
use SilverStripe\ORM\DataExtension;
use SilverStripe\Security\Permission;
use SilverStripe\ORM\Queries\SQLSelect;
use SilverStripe\Security\Member;

/**
 * Add subsites support
 *
 * @property-read EmailTemplate|SentEmail|Emailing $owner
 * @author lekoala
 */
class EmailSubsiteExtension extends DataExtension

The class SilverStripe\ORM\DataExtension has been deprecated: 5.3.0 Subclass SilverStripe\Core\Extension\Extension instead

{

    private static $has_one = [

The private property $has_one is not used, and could be removed.

        'Subsite' => 'Subsite',
    ];

    public function isMainDataObject()
    {
        if ($this->owner->SubsiteID == 0) {

It seems like you are loosely comparing $this->owner->SubsiteID of type mixed|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\Emailing. Since you implemented __get, consider adding a @property annotation.

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\SentEmail. Since you implemented __get, consider adding a @property annotation.

            return true;
        }
        return false;
    }

    /**
     * Update any requests to limit the results to the current site
     */
    public function augmentSQL(SQLSelect $query, DataQuery $dataQuery = null)
    {
        if (SubsiteHelper::subsiteFilterDisabled()) {
            return;
        }
        if ($dataQuery && $dataQuery->getQueryParam('Subsite.filter') === false) {

The condition $dataQuery->getQueryParam('Subsite.filter') === false is always false.

            return;
        }

        // If you're querying by ID, don't filter
        if ($query->filtersOnID()) {
            return;
        }

        // Don't run on delete queries, since they are always tied to a specific ID.
        // if ($query->getDelete()) {
        //     return;
        // }

        // If we match on a subsite, don't filter twice
        $regexp = '/^(.*\.)?("|`)?SubsiteID("|`)?\s?=/';
        foreach ($query->getWhereParameterised($parameters) as $predicate) {
            if (preg_match($regexp, $predicate)) {
                return;
            }
        }

        $subsiteID = (int) SubsiteHelper::currentSubsiteID();

        $froms = $query->getFrom();
        $froms = array_keys($froms);
        $tableName = array_shift($froms);

        if ($subsiteID) {
            $query->addWhere("\"$tableName\".\"SubsiteID\" IN ($subsiteID)");
        }
    }

    public function onBeforeWrite()
    {
        parent::onBeforeWrite();

        // Assign to current subsite when created
        if (!$this->owner->ID && !$this->owner->SubsiteID) {

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\Emailing. Since you implemented __get, consider adding a @property annotation.

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\SentEmail. Since you implemented __get, consider adding a @property annotation.

            $this->owner->SubsiteID = SubsiteHelper::currentSubsiteID();

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\SentEmail. Since you implemented __set, consider adding a @property annotation.

The property SubsiteID does not exist on LeKoala\EmailTemplates\Models\Emailing. Since you implemented __set, consider adding a @property annotation.

        }
    }

    public function canView($member = null)
    {
        return $this->canEdit($member);
    }

    /**
     * @param Member $member
     * @return boolean|null
     */
    public function canEdit($member = null)
    {
        return Permission::check('CMS_ACCESS', 'any', $member);
    }

    /**
     * @param Member $member
     * @return boolean|null
     */
    public function canCreate($member = null)
    {
        return Permission::check('CMS_ACCESS', 'any', $member);
    }

    /**
     * @param Member $member
     * @return boolean|null
     */
    public function canDelete($member = null)
    {
        return Permission::check('CMS_ACCESS', 'any', $member);
    }

    /**
     * Return a piece of text to keep DataObject cache keys appropriately specific
     */
    public function cacheKeyComponent()
    {
        return 'subsite-' . SubsiteHelper::currentSubsiteID();
    }
}