nystudio107 / craft-seomatic

src/models/SitemapTemplate.php

Spacing +77 added lines, -77 removed lines

@@ -63,7 +63,7 @@ 
      *
      * @return null|SitemapTemplate
      */
-    public static function create(array $config = [])
+    public static function create(array $config = [ ])
     {
         $defaults = [
             'path'       => 'sitemaps/<groupId:\d+>/<type:[-\w\.*]+>/<handle:[-\w\.*]+>/<siteId:\d+>/<file:[-\w\.*]+>',
@@ -110,32 +110,32 @@ 
     /**
      * @inheritdoc
      */
-    public function render($params = []): string
+    public function render($params = [ ]): string
     {
         $cache = Craft::$app->getCache();
-        $groupId = $params['groupId'];
+        $groupId = $params[ 'groupId' ];
         /** @var SiteGroup $siteGroup */
         $siteGroup = Craft::$app->getSites()->getGroupById($groupId);
         $groupSiteIds = $siteGroup->getSiteIds();
-        $type = $params['type'];
-        $handle = $params['handle'];
-        $siteId = $params['siteId'];
+        $type = $params[ 'type' ];
+        $handle = $params[ 'handle' ];
+        $siteId = $params[ 'siteId' ];
         $duration = Seomatic::$devMode ? $this::DEVMODE_SITEMAP_CACHE_DURATION : $this::SITEMAP_CACHE_DURATION;
         $dependency = new TagDependency([
             'tags' => [
                 $this::GLOBAL_SITEMAP_CACHE_TAG,
-                $this::SITEMAP_CACHE_TAG.$handle.$siteId,
+                $this::SITEMAP_CACHE_TAG . $handle . $siteId,
             ],
         ]);
 
-        return $cache->getOrSet($this::CACHE_KEY.$groupId.$handle.$siteId, function () use ($type, $handle, $siteId, $groupSiteIds) {
+        return $cache->getOrSet($this::CACHE_KEY . $groupId . $handle . $siteId, function() use ($type, $handle, $siteId, $groupSiteIds) {
             Craft::info(
-                'Sitemap cache miss: '.$handle.'/'.$siteId,
+                'Sitemap cache miss: ' . $handle . '/' . $siteId,
                 __METHOD__
             );
-            $lines = [];
+            $lines = [ ];
             // Sitemap index XML header and opening tag
-            $lines[] = '<?xml version="1.0" encoding="UTF-8"?>';
+            $lines[ ] = '<?xml version="1.0" encoding="UTF-8"?>';
             // One sitemap entry for each element
             $metaBundle = Seomatic::$plugin->metaBundles->getMetaBundleBySourceHandle($type, $handle, $siteId);
             $multiSite = count($metaBundle->sourceAltSiteSettings) > 1;
@@ -150,7 +150,7 @@ 
                     $urlsetLine .= ' xmlns:xhtml="http://www.w3.org/1999/xhtml"';
                 }
                 $urlsetLine .= '>';
-                $lines[] = $urlsetLine;
+                $lines[ ] = $urlsetLine;
                 // Handle each element type separately
                 switch ($metaBundle->sourceBundleType) {
                     case MetaBundles::SECTION_META_BUNDLE:
@@ -176,25 +176,25 @@ 
                     $path = ($element->uri === '__home__') ? '' : $element->uri;
                     $url = UrlHelper::siteUrl($path, null, null, $metaBundle->sourceSiteId);
                     $dateUpdated = $element->dateUpdated ?? $element->dateCreated ?? new \DateTime;
-                    $lines[] = '  <url>';
+                    $lines[ ] = '  <url>';
                     // Standard sitemap key/values
-                    $lines[] = '    <loc>';
-                    $lines[] = '      '.$url;
-                    $lines[] = '    </loc>';
-                    $lines[] = '    <lastmod>';
-                    $lines[] = '      '.$dateUpdated->format(\DateTime::W3C);
-                    $lines[] = '    </lastmod>';
-                    $lines[] = '    <changefreq>';
-                    $lines[] = '      '.$metaBundle->metaSitemapVars->sitemapChangeFreq;
-                    $lines[] = '    </changefreq>';
-                    $lines[] = '    <priority>';
-                    $lines[] = '      '.$metaBundle->metaSitemapVars->sitemapPriority;
-                    $lines[] = '    </priority>';
+                    $lines[ ] = '    <loc>';
+                    $lines[ ] = '      ' . $url;
+                    $lines[ ] = '    </loc>';
+                    $lines[ ] = '    <lastmod>';
+                    $lines[ ] = '      ' . $dateUpdated->format(\DateTime::W3C);
+                    $lines[ ] = '    </lastmod>';
+                    $lines[ ] = '    <changefreq>';
+                    $lines[ ] = '      ' . $metaBundle->metaSitemapVars->sitemapChangeFreq;
+                    $lines[ ] = '    </changefreq>';
+                    $lines[ ] = '    <priority>';
+                    $lines[ ] = '      ' . $metaBundle->metaSitemapVars->sitemapPriority;
+                    $lines[ ] = '    </priority>';
                     // Handle alternate URLs if this is multi-site
                     if ($multiSite && $metaBundle->metaSitemapVars->sitemapAltLinks) {
                         /** @var  $altSiteSettings */
                         foreach ($metaBundle->sourceAltSiteSettings as $altSiteSettings) {
-                            if (in_array($altSiteSettings['siteId'], $groupSiteIds)) {
+                            if (in_array($altSiteSettings[ 'siteId' ], $groupSiteIds)) {
                                 $altElement = null;
                                 // Handle each element type separately
                                 switch ($metaBundle->sourceBundleType) {
@@ -202,7 +202,7 @@ 
                                         $altElement = Entry::find()
                                             ->section($metaBundle->sourceHandle)
                                             ->id($element->id)
-                                            ->siteId($altSiteSettings['siteId'])
+                                            ->siteId($altSiteSettings[ 'siteId' ])
                                             ->enabledForSite(true)
                                             ->limit(1)
                                             ->one();
@@ -211,7 +211,7 @@ 
                                     case MetaBundles::CATEGORYGROUP_META_BUNDLE:
                                         $altElement = Category::find()
                                             ->id($element->id)
-                                            ->siteId($altSiteSettings['siteId'])
+                                            ->siteId($altSiteSettings[ 'siteId' ])
                                             ->limit(1)
                                             ->enabledForSite(true)
                                             ->one();
@@ -219,9 +219,9 @@ 
                                     // @todo: handle Commerce products
                                 }
                                 if ($altElement) {
-                                    $lines[] = '    <xhtml:link rel="alternate"'
-                                        .' hreflang="'.$altSiteSettings['language'].'"'
-                                        .' href="'.$altElement->url.'"'
+                                    $lines[ ] = '    <xhtml:link rel="alternate"'
+                                        .' hreflang="' . $altSiteSettings[ 'language' ] . '"'
+                                        .' href="' . $altElement->url . '"'
                                         .' />';
                                 }
                             }
@@ -236,7 +236,7 @@ 
                             true
                         );
                         foreach ($assetFields as $assetField) {
-                            $assets = $element[$assetField]->all();
+                            $assets = $element[ $assetField ]->all();
                             foreach ($assets as $asset) {
                                 $this->assetSitemapItem($asset, $metaBundle, $lines);
                             }
@@ -248,19 +248,19 @@ 
                             true
                         );
                         foreach ($matrixFields as $matrixField) {
-                            $matrixBlocks = $element[$matrixField]->all();
+                            $matrixBlocks = $element[ $matrixField ]->all();
                             foreach ($matrixBlocks as $matrixBlock) {
                                 $assetFields = FieldHelper::matrixFieldsOfType($matrixBlock, AssetsField::class);
                                 foreach ($assetFields as $assetField) {
                                     /** @var MatrixBlock $matrixBlock */
-                                    foreach ($matrixBlock[$assetField]->all() as $asset) {
+                                    foreach ($matrixBlock[ $assetField ]->all() as $asset) {
                                         $this->assetSitemapItem($asset, $metaBundle, $lines);
                                     }
                                 }
                             }
                         }
                     }
-                    $lines[] = '  </url>';
+                    $lines[ ] = '  </url>';
                     // Include links to any known file types in the assets fields
                     if ($metaBundle->metaSitemapVars->sitemapFiles) {
                         // Regular Assets fields
@@ -270,7 +270,7 @@ 
                             true
                         );
                         foreach ($assetFields as $assetField) {
-                            $assets = $element[$assetField]->all();
+                            $assets = $element[ $assetField ]->all();
                             foreach ($assets as $asset) {
                                 $this->assetFilesSitemapLink($asset, $metaBundle, $lines);
                             }
@@ -282,12 +282,12 @@ 
                             true
                         );
                         foreach ($matrixFields as $matrixField) {
-                            $matrixBlocks = $element[$matrixField]->all();
+                            $matrixBlocks = $element[ $matrixField ]->all();
                             /** @var MatrixBlock $matrixBlock */
                             foreach ($matrixBlocks as $matrixBlock) {
                                 $assetFields = FieldHelper::matrixFieldsOfType($matrixBlock, AssetsField::class);
                                 foreach ($assetFields as $assetField) {
-                                    foreach ($matrixBlock[$assetField]->all() as $asset) {
+                                    foreach ($matrixBlock[ $assetField ]->all() as $asset) {
                                         $this->assetFilesSitemapLink($asset, $metaBundle, $lines);
                                     }
                                 }
@@ -296,7 +296,7 @@ 
                     }
                 }
                 // Sitemap index closing tag
-                $lines[] = '</urlset>';
+                $lines[ ] = '</urlset>';
             }
 
             return implode("\r\n", $lines);
@@ -312,9 +312,9 @@ 
     public function invalidateCache(string $handle, int $siteId)
     {
         $cache = Craft::$app->getCache();
-        TagDependency::invalidate($cache, $this::SITEMAP_CACHE_TAG.$handle.$siteId);
+        TagDependency::invalidate($cache, $this::SITEMAP_CACHE_TAG . $handle . $siteId);
         Craft::info(
-            'Sitemap cache cleared: '.$handle,
+            'Sitemap cache cleared: ' . $handle,
             __METHOD__
         );
     }
@@ -329,39 +329,39 @@ 
         if ($asset->enabledForSite) {
             switch ($asset->kind) {
                 case 'image':
-                    $lines[] = '    <image:image>';
-                    $lines[] = '      <image:loc>';
-                    $lines[] = '        '.$asset->getUrl();
-                    $lines[] = '      </image:loc>';
+                    $lines[ ] = '    <image:image>';
+                    $lines[ ] = '      <image:loc>';
+                    $lines[ ] = '        ' . $asset->getUrl();
+                    $lines[ ] = '      </image:loc>';
                     // Handle the dynamic field => property mappings
                     foreach ($metaBundle->metaSitemapVars->sitemapImageFieldMap as $row) {
-                        $fieldName = $row['field'] ?? '';
-                        $propName = $row['property'] ?? '';
-                        if (!empty($asset[$fieldName]) && !empty($propName)) {
-                            $lines[] = '      <image:'.$propName.'>';
-                            $lines[] = '        '.$asset[$fieldName];
-                            $lines[] = '      </image:'.$propName.'>';
+                        $fieldName = $row[ 'field' ] ?? '';
+                        $propName = $row[ 'property' ] ?? '';
+                        if (!empty($asset[ $fieldName ]) && !empty($propName)) {
+                            $lines[ ] = '      <image:' . $propName . '>';
+                            $lines[ ] = '        ' . $asset[ $fieldName ];
+                            $lines[ ] = '      </image:' . $propName . '>';
                         }
                     }
-                    $lines[] = '    </image:image>';
+                    $lines[ ] = '    </image:image>';
                     break;
 
                 case 'video':
-                    $lines[] = '    <video:video>';
-                    $lines[] = '      <video:content_loc>';
-                    $lines[] = '        '.$asset->getUrl();
-                    $lines[] = '      </video:content_loc>';
+                    $lines[ ] = '    <video:video>';
+                    $lines[ ] = '      <video:content_loc>';
+                    $lines[ ] = '        ' . $asset->getUrl();
+                    $lines[ ] = '      </video:content_loc>';
                     // Handle the dynamic field => property mappings
                     foreach ($metaBundle->metaSitemapVars->sitemapVideoFieldMap as $row) {
-                        $fieldName = $row['field'] ?? '';
-                        $propName = $row['property'] ?? '';
-                        if (!empty($asset[$fieldName]) && !empty($propName)) {
-                            $lines[] = '      <video:'.$propName.'>';
-                            $lines[] = '        '.$asset[$fieldName];
-                            $lines[] = '      </video:'.$propName.'>';
+                        $fieldName = $row[ 'field' ] ?? '';
+                        $propName = $row[ 'property' ] ?? '';
+                        if (!empty($asset[ $fieldName ]) && !empty($propName)) {
+                            $lines[ ] = '      <video:' . $propName . '>';
+                            $lines[ ] = '        ' . $asset[ $fieldName ];
+                            $lines[ ] = '      </video:' . $propName . '>';
                         }
                     }
-                    $lines[] = '    </video:video>';
+                    $lines[ ] = '    </video:video>';
                     break;
             }
         }
@@ -377,20 +377,20 @@ 
         if ($asset->enabledForSite) {
             if (in_array($asset->kind, $this::FILE_TYPES)) {
                 $dateUpdated = $asset->dateUpdated ?? $asset->dateCreated ?? new \DateTime;
-                $lines[] = '  <url>';
-                $lines[] = '    <loc>';
-                $lines[] = '      '.$asset->getUrl();
-                $lines[] = '    </loc>';
-                $lines[] = '    <lastmod>';
-                $lines[] = '      '.$dateUpdated->format(\DateTime::W3C);
-                $lines[] = '    </lastmod>';
-                $lines[] = '    <changefreq>';
-                $lines[] = '      '.$metaBundle->metaSitemapVars->sitemapChangeFreq;
-                $lines[] = '    </changefreq>';
-                $lines[] = '    <priority>';
-                $lines[] = '      '.$metaBundle->metaSitemapVars->sitemapPriority;
-                $lines[] = '    </priority>';
-                $lines[] = '  </url>';
+                $lines[ ] = '  <url>';
+                $lines[ ] = '    <loc>';
+                $lines[ ] = '      ' . $asset->getUrl();
+                $lines[ ] = '    </loc>';
+                $lines[ ] = '    <lastmod>';
+                $lines[ ] = '      ' . $dateUpdated->format(\DateTime::W3C);
+                $lines[ ] = '    </lastmod>';
+                $lines[ ] = '    <changefreq>';
+                $lines[ ] = '      ' . $metaBundle->metaSitemapVars->sitemapChangeFreq;
+                $lines[ ] = '    </changefreq>';
+                $lines[ ] = '    <priority>';
+                $lines[ ] = '      ' . $metaBundle->metaSitemapVars->sitemapPriority;
+                $lines[ ] = '    </priority>';
+                $lines[ ] = '  </url>';
             }
         }
     }