CWP\Core\Control\CwpCanonicalURLMiddleware

Complexity

Total Complexity

3

Size/Duplication

Total Lines	11
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	isEnabled()	0	9	3

<?php

namespace CWP\Core\Control;

use SilverStripe\Control\Director;
use SilverStripe\Control\Middleware\CanonicalURLMiddleware;

/**
 * @internal Used to override CanonicalURLMiddleware to prevent canonical URL causing a redirect on CLI unless
 * explicitly enabled. See https://github.com/silverstripe/silverstripe-framework/pull/8158
 * Note, it is very likely that this class will deprecated after CWP 2.0.
 */
class CwpCanonicalURLMiddleware extends CanonicalURLMiddleware
{
    protected function isEnabled()
    {
        $enabledEnvs = $this->getEnabledEnvs();

        // If CLI, EnabledEnvs must contain CLI
        if (Director::is_cli() && !in_array('cli', $enabledEnvs)) {

It seems like $enabledEnvs can also be of type boolean; however, parameter $haystack of in_array() does only seem to accept array, maybe add an additional type check?

            return false;
        }
        return parent::isEnabled();
    }
}