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

Complexity

Conditions	11
Paths	37

Size

Total Lines	100
Code Lines	68

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	132

Importance

Changes	1
Bugs	0	Features	0

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

        $metaBundle = Seomatic::$plugin->metaBundles->getMetaBundleBySourceHandle($type, $handle, $siteId);
        // If it doesn't exist, throw a 404
        if ($metaBundle === null ) {
            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 it's disabled, just throw a 404
        if (!$sitemapUrls || !$robotsEnabled) {
            throw new NotFoundHttpException(Craft::t('seomatic', 'Page not found.'));
        }

        $cache = Craft::$app->getCache();
        $uniqueKey = $groupId.$type.$handle.$siteId;
        $cacheKey = self::CACHE_KEY.$uniqueKey;
        $queueJobCacheKey = self::QUEUE_JOB_CACHE_KEY.$uniqueKey;
        $result = $cache->get($cacheKey);
        // If the sitemap isn't cached, start a job to create it
        if ($result === false) {
            $queue = Craft::$app->getQueue();
            // If there's an existing queue job, release it so queue jobs don't stack
            $existingJobId = $cache->get($queueJobCacheKey);
            // Make sure the queue uses the Craft web interface
            if ($existingJobId && $queue instanceof QueueInterface) {
                $queue = Craft::$app->getQueue();
                $queue->release($existingJobId);
                $cache->delete($queueJobCacheKey);
            }
            // Push a new queue job
            $jobId = $queue->push(new GenerateSitemap([
                'groupId' => $groupId,
                'type' => $type,
                'handle' => $handle,
                'siteId' => $siteId,
                'queueJobCacheKey' => $queueJobCacheKey,
            ]));
            // Stash the queue job id in the cache for future reference
            $cacheDuration = 3600;
            $dependency = new TagDependency([
                'tags' => [
                    self::GLOBAL_SITEMAP_CACHE_TAG,
                    self::CACHE_KEY.$uniqueKey,
                ],
            ]);
            $cache->set($queueJobCacheKey, $jobId, $cacheDuration, $dependency);
            Craft::debug(
                Craft::t(
                    'seomatic',
                    'Started GenerateSitemap queue job id: {jobId} with cache key {cacheKey}',
                    [
                        'jobId' => $jobId,
                        'cacheKey' => $cacheKey,
                    ]
                ),
                __METHOD__
            );
            // Try to run the queue immediately
            QueueHelper::run();
            // Try it again now
            $result = $cache->get($cacheKey);
            if ($result !== false) {
                return $result;
            }
            // Return a 503 Service Unavailable an a Retry-After so bots will try back later
            $response = Craft::$app->getResponse();
            $response->setStatusCode(503);
            $response->headers->add('Retry-After', 60);
            $response->headers->add('Cache-Control', 'no-cache, no-store');
            // Return an empty XML document
            $lines[] = '<?xml version="1.0" encoding="UTF-8"?>';

$lines was never initialized. Although not strictly required by PHP, it is generally a good practice to add $lines = array(); before regardless.

            $lines[] = '<?xml-stylesheet type="text/xsl" href="sitemap-empty.xsl"?>';
            $lines[] = '<!-- ' . Craft::t('seomatic', 'This sitemap has not been generated yet.') . ' -->';
            $lines[] = '<!-- ' . Craft::t('seomatic', 'If you are seeing this in local dev or an') . ' -->';
            $lines[] = '<!-- ' . Craft::t('seomatic', 'environment with `devMode` on, caches only') . ' -->';
            $lines[] = '<!-- ' . Craft::t('seomatic', 'last for 30 seconds in local dev, so it is') . ' -->';
            $lines[] = '<!-- ' . Craft::t('seomatic', 'normal for the sitemap to not be cached.') . ' -->';
            $lines[] = '<urlset>';
            $lines[] = '</urlset>';
            $lines = implode("\r\n", $lines);

            return $lines;
        }

        return $result;
    }