SilverStripe\Subsites\Extensions\ErrorPageSubsite

Complexity

Total Complexity

7

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	updateErrorFilename()	0	37	7

<?php

namespace SilverStripe\Subsites\Extensions;

use SilverStripe\Assets\FileNameFilter;
use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\Core\Config\Config;
use SilverStripe\ORM\DataExtension;
use SilverStripe\ORM\DataObject;
use SilverStripe\Subsites\Model\Subsite;

class ErrorPageSubsite extends DataExtension
{
    /**
     * Alter file path to generated a static (static) error page file to handle error page template
     * on different sub-sites
     *
     * @see ErrorPage::get_error_filename()
     *
     * FIXME since {@link Subsite::currentSubsite()} partly relies on Session, viewing other sub-site (including
     * main site) between opening ErrorPage in the CMS and publish ErrorPage causes static error page to get
     * generated incorrectly.
     *
     * @param string $name
     * @param int $statusCode
     */
    public function updateErrorFilename(&$name, &$statusCode)
    {
        $static_filepath = Config::inst()->get($this->owner->ClassName, 'static_filepath');
        $subdomainPart = '';

        // Try to get current subsite from session
        $subsite = Subsite::currentSubsite();

        // since this function is called from Page class before the controller is created, we have
        // to get subsite from domain instead
        if (!$subsite) {

$subsite is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

            $subsiteID = Subsite::getSubsiteIDForDomain();
            if ($subsiteID != 0) {
                $subsite = DataObject::get_by_id(Subsite::class, $subsiteID);
            } else {
                $subsite = null;
            }
        }

        if ($subsite) {

$subsite is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

            $subdomain = $subsite->domain();
            $subdomainPart = "-{$subdomain}";
        }

        // @todo implement Translatable namespace
        if (singleton(SiteTree::class)->hasExtension('Translatable')
            && $locale

The variable $locale seems to be never defined.

            && $locale != Translatable::default_locale()

The type SilverStripe\Subsites\Extensions\Translatable was not found. Maybe you did not declare it correctly or list all dependencies?

        ) {
            $fileName = "error-{$statusCode}-{$locale}{$subdomainPart}.html";
        } else {
            $fileName=  "error-{$statusCode}{$subdomainPart}.html";
        }

        $fileName = FileNameFilter::create()->filter($fileName);

        $name = implode('/', [$static_filepath, $fileName]);
    }
}