WebStream\Annotation\Attributes\Template::onMethodInject()

Complexity

Conditions	21
Paths	320

Size

Total Lines	71
Code Lines	47

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	37
CRAP Score	24.3053

Importance

Changes

0

<?php
    public function onMethodInject(IAnnotatable $instance, \ReflectionMethod $method, Container $container)
    {
        $filename = array_key_exists('value', $this->injectAnnotation) ? $this->injectAnnotation['value'] : null;
        $engine = array_key_exists('engine', $this->injectAnnotation) ? $this->injectAnnotation['engine'] : "basic";
        $debug = array_key_exists('debug', $this->injectAnnotation) ? $this->injectAnnotation['debug'] : false;
        $cacheTime = array_key_exists('cacheTime', $this->injectAnnotation) ? $this->injectAnnotation['cacheTime'] : null;
        $logger = $container->logger;

The property logger does not exist on WebStream\Container\Container. Since you implemented __get, consider adding a @property annotation.

        if (PHP_OS === "WIN32" || PHP_OS === "WINNT") {
            if (preg_match("/^.*[. ]|.*[\p{Cntrl}\/:*?\"<>|].*|(?i:CON|PRN|AUX|CLOCK\$|NUL|COM[1-9]|LPT[1-9])(?:[.].+)?$/", $filename)) {
                throw new AnnotationException("Invalid string contains in @Template('$filename')");
            }
        } else {
            if (preg_match("/:|\.\.\/|\.\.\\\\/", $filename)) {
                throw new AnnotationException("Invalid string contains in @Template('$filename')");
            }
        }

        if ($filename === null) {
            $errorMsg = "Invalid argument of @Template('$filename'). There is no specification of the base template.";
            throw new AnnotationException($errorMsg);
        }

        if ($engine === "twig") {
            if (!is_bool($debug)) {
                if ($debug !== null) {
                    $errorMsg = "Invalid argument of @Template('$filename'). 'debug' attribute bool only be specified.";
                    throw new AnnotationException($errorMsg);
                }
                $debug = false;
            }
            $this->readAnnotation['filename'] = $filename;
            $this->readAnnotation['engine'] = $container->engine['twig'];

The property engine does not exist on WebStream\Container\Container. Since you implemented __get, consider adding a @property annotation.

            $this->readAnnotation['debug'] = $debug;

            if ($cacheTime !== null) {
                $logger->warn("'cacheTime' attribute is not used in Twig template.");

The method warn() does not exist on null.

            }
        } elseif ($engine === "basic") {
            if ($cacheTime !== null) {
                // 複数指定は不可
                if (is_array($cacheTime)) {
                    $errorMsg = "Invalid argument of @Template attribute 'cacheTime' should not be array.";
                    throw new AnnotationException($errorMsg);
                }
                // 数値以外は不可
                if (!preg_match("/^[1-9]{1}[0-9]{0,}$/", $cacheTime)) {
                    $errorMsg = "Invalid argument of @Template attribute 'cacheTime' should not be integer.";
                    throw new AnnotationException($errorMsg);
                }

                $cacheTime = intval($cacheTime);
                if ($cacheTime <= 0) {
                    $errorMsg = "Expire value is out of integer range: @Template(cacheTime=" . strval($cacheTime) . ")";
                    throw new AnnotationException($errorMsg);
                } elseif ($cacheTime >= PHP_INT_MAX) {
                    $logger->warn("Expire value converted the maximum of PHP Integer.");
                }

                $this->readAnnotation['cacheTime'] = $cacheTime;
            }

            $this->readAnnotation['filename'] = $filename;
            $this->readAnnotation['engine'] = $container->engine['basic'];

            if ($debug !== null) {
                $logger->warn("'debug' attribute is not used in Basic template.");
            }
        } else {
            $errorMsg = "Invalid 'engine' attribute of @Template('$filename').";
            throw new AnnotationException($errorMsg);
        }