nystudio107\seomatic\models\SitemapTemplate::render()

Complexity

Conditions	19
Paths	158

Size

Total Lines	92
Code Lines	57

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	380

Importance

Changes	1
Bugs	1	Features	0

<?php
    public function render(array $params = []): string
    {
        $groupId = $params['groupId'];
        $type = $params['type'];
        $handle = $params['handle'];
        $siteId = $params['siteId'];
        $page = $params['page'] ?? 0;

        $request = Craft::$app->getRequest();
        $metaBundle = Seomatic::$plugin->metaBundles->getMetaBundleBySourceHandle($type, $handle, $siteId);
        // If it doesn't exist, throw a 404
        if ($metaBundle === null) {
            if ($request->isCpRequest || $request->isConsoleRequest) {
                return '';
            }
            throw new NotFoundHttpException(Craft::t('seomatic', 'Page not found.'));
        }
        // Check to see if robots is `none` or `no index`
        $robotsEnabled = true;
        if (!empty($metaBundle->metaGlobalVars->robots)) {
            $robotsEnabled = $metaBundle->metaGlobalVars->robots !== 'none' &&
                $metaBundle->metaGlobalVars->robots !== 'noindex';
        }
        $sitemapUrls = $metaBundle->metaSitemapVars->sitemapUrls;
        if (Seomatic::$plugin->sitemaps->anyEntryTypeHasSitemapUrls($metaBundle)) {
            $robotsEnabled = true;
            $sitemapUrls = true;
        }
        if ($sitemapUrls && !SiteHelper::siteEnabledWithUrls($siteId)) {
            $sitemapUrls = false;
        }
        // If it's disabled, just throw a 404
        if (!$sitemapUrls || !$robotsEnabled) {
            if ($request->isCpRequest || $request->isConsoleRequest) {
                return '';
            }
            throw new NotFoundHttpException(Craft::t('seomatic', 'Page not found.'));
        }

        $cache = Craft::$app->getCache();
        $pageCacheSuffix = 's' . (int)$metaBundle->metaSitemapVars->sitemapPageSize . 'p' . $page;

        $uniqueKey = $groupId . $type . $handle . $siteId . $pageCacheSuffix;
        $cacheKey = self::CACHE_KEY . $uniqueKey;
        $result = $cache->get($cacheKey);

        // If the sitemap isn't cached, render it immediately
        if ($result === false) {
            $sitemap = Sitemap::generateSitemap([
                'groupId' => $groupId,
                'type' => $type,
                'handle' => $handle,
                'siteId' => $siteId,
                'page' => $page,
            ]);

            if ($sitemap) {
                $dependency = new TagDependency([
                    'tags' => [
                        self::GLOBAL_SITEMAP_CACHE_TAG,
                        self::SITEMAP_CACHE_TAG . $handle . $siteId,
                        self::SITEMAP_CACHE_TAG . $handle . $siteId . $pageCacheSuffix,
                    ],
                ]);

                $cacheDuration = Seomatic::$devMode
                    ? Seomatic::DEVMODE_CACHE_DURATION
                    : null;

                $result = $cache->set($cacheKey, $sitemap, $cacheDuration, $dependency);

                // Output some info if this is a console app
                if (Craft::$app instanceof ConsoleApplication) {
                    echo 'Sitemap cache result: ' . print_r($result, true) . ' for cache key: ' . $cacheKey . PHP_EOL;

Are you sure print_r($result, true) of type string|true can be used in concatenation?

                }

                // If the FastCGI Cache Bust plugin is installed, clear its caches too
                /** @var FastcgiCacheBust $plugin */
                $plugin = Craft::$app->getPlugins()->getPlugin('fastcgi-cache-bust');
                if ($plugin !== null) {
                    $plugin->cache->clearAll();
                }

                return $sitemap;
            }
        } else {
            if (Craft::$app instanceof ConsoleApplication) {
                echo 'Found in cache' . PHP_EOL;
            }
        }

        return $result;
    }