nystudio107 / craft-seomatic

src/debug/panels/SeomaticPanel.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
 
         Event::on(MetaContainers::class,
             MetaContainers::EVENT_METABUNDLE_DEBUG_DATA,
-            function (MetaBundleDebugDataEvent $e) {
+            function(MetaBundleDebugDataEvent $e) {
                 if ($e->metaBundle) {
                     $this->metaBundles[$e->metaBundleCategory] = $e->metaBundle;
                 }
@@ -67,7 +67,7 @@ 
      */
     public function getSummary(): string
     {
-        return Craft::$app->getView()->render($this->viewPath . 'summary', ['panel' => $this]);
+        return Craft::$app->getView()->render($this->viewPath.'summary', ['panel' => $this]);
     }
 
     /**
@@ -75,7 +75,7 @@ 
      */
     public function getDetail(): string
     {
-        return Craft::$app->getView()->render($this->viewPath . 'detail', ['panel' => $this]);
+        return Craft::$app->getView()->render($this->viewPath.'detail', ['panel' => $this]);
     }
 
     /**

src/debug/views/seomatic/detail.php

Spacing +1 added lines, -1 removed lines

@@ -164,7 +164,7 @@
                     $content .= $this->render('rendered-tags', [
                         'values' => [
                             'renderedTags' => $panel->data[$metaBundleCategory]['renderedTags'][$metaContainerName] ?? [],
-                            'id' => $sectionName . '-' . $metaBundleCategory . '-' . $metaContainerName . '-rendered-tags',
+                            'id' => $sectionName.'-'.$metaBundleCategory.'-'.$metaContainerName.'-rendered-tags',
                             'view' => $this,
                             'language' => $editorLanguage,
                         ]

src/models/MetaTag.php

Spacing +4 added lines, -4 removed lines

@@ -79,7 +79,7 @@ 
         $className = MetaTag::class;
         if ($tagType) {
             // Potentially load a sub-type of MetaTag
-            $tagClassName = 'nystudio107\\seomatic\\models\\metatag\\' . ucfirst($tagType) . 'Tag';
+            $tagClassName = 'nystudio107\\seomatic\\models\\metatag\\'.ucfirst($tagType).'Tag';
             /** @var $model MetaTag */
             if (class_exists($tagClassName)) {
                 $className = $tagClassName;
@@ -155,10 +155,10 @@ 
                 if (Seomatic::$devMode) {
                     $error = Craft::t(
                         'seomatic',
-                        '{tagtype} tag `{key}` did not render because it is missing attributes. ' . print_r($data, true),
+                        '{tagtype} tag `{key}` did not render because it is missing attributes. '.print_r($data, true),
                         ['tagtype' => 'Meta', 'key' => $this->key]
                     );
-                    Craft::info('WARNING - ' . $error, __METHOD__);
+                    Craft::info('WARNING - '.$error, __METHOD__);
                 }
                 $shouldRender = false;
             }
@@ -182,7 +182,7 @@ 
         foreach ($configs as $config) {
             if ($this->prepForRender($config)) {
                 ksort($config);
-                $html .= Html::tag('meta', '', $config) . $linebreak;
+                $html .= Html::tag('meta', '', $config).$linebreak;
             }
         }
 

src/models/MetaLink.php

Spacing +3 added lines, -3 removed lines

@@ -85,7 +85,7 @@ 
         $className = MetaLink::class;
         if ($tagType) {
             // Potentially load a sub-type of MetaTag
-            $tagClassName = 'nystudio107\\seomatic\\models\\metalink\\' . ucfirst($tagType) . 'Link';
+            $tagClassName = 'nystudio107\\seomatic\\models\\metalink\\'.ucfirst($tagType).'Link';
             /** @var $model MetaLink */
             if (class_exists($tagClassName)) {
                 $className = $tagClassName;
@@ -182,7 +182,7 @@ 
                         '{tagtype} tag `{key}` did not render because it is missing attributes.',
                         ['tagtype' => 'Link', 'key' => $this->key]
                     );
-                    Craft::info('WARNING - ' . $error, __METHOD__);
+                    Craft::info('WARNING - '.$error, __METHOD__);
                 }
                 $shouldRender = false;
             }
@@ -206,7 +206,7 @@ 
         foreach ($configs as $config) {
             if ($this->prepForRender($config)) {
                 ksort($config);
-                $html .= Html::tag('link', '', $config) . $linebreak;
+                $html .= Html::tag('link', '', $config).$linebreak;
             }
         }
 

src/models/MetaJsonLd.php

Spacing +6 added lines, -6 removed lines

@@ -119,7 +119,7 @@ 
     public static function create($schemaType, array $config = []): MetaJsonLd
     {
         // Try the passed in $schemaType
-        $className = self::SCHEMA_NAMESPACE_PREFIX . $schemaType;
+        $className = self::SCHEMA_NAMESPACE_PREFIX.$schemaType;
         /** @var $model MetaJsonLd */
         if (class_exists($className)) {
             self::cleanProperties($className, $config);
@@ -127,7 +127,7 @@ 
             return new $className($config);
         }
         // Try the prefixed $schemaType
-        $className = self::SCHEMA_NAMESPACE_PREFIX . self::SCHEMA_NAME_PREFIX . $schemaType;
+        $className = self::SCHEMA_NAMESPACE_PREFIX.self::SCHEMA_NAME_PREFIX.$schemaType;
         /** @var $model MetaJsonLd */
         if (class_exists($className)) {
             self::cleanProperties($className, $config);
@@ -385,7 +385,7 @@ 
             foreach ($dataToValidate as $key => $data) {
                 /** @var array $expectedTypes */
                 foreach ($expectedTypes as $expectedType) {
-                    $className = 'nystudio107\\seomatic\\models\\jsonld\\' . $expectedType;
+                    $className = 'nystudio107\\seomatic\\models\\jsonld\\'.$expectedType;
                     switch ($expectedType) {
                         // Text always validates
                         case 'Text':
@@ -425,7 +425,7 @@ 
                         case 'Date':
                             $validator = new DateValidator;
                             $validator->type = DateValidator::TYPE_DATE;
-                            $validator->format = 'php:' . DateTime::ATOM;
+                            $validator->format = 'php:'.DateTime::ATOM;
                             if ($validator->validate($data, $error)) {
                                 $validated = true;
                             }
@@ -471,7 +471,7 @@ 
                                 $validated = true;
                             }
                             if (is_string($data)) {
-                                $targetClass = 'nystudio107\\seomatic\\models\\jsonld\\' . $data;
+                                $targetClass = 'nystudio107\\seomatic\\models\\jsonld\\'.$data;
                                 if (class_exists($targetClass)) {
                                     $validated = true;
                                 }
@@ -480,7 +480,7 @@ 
                     }
                 }
                 if (!$validated) {
-                    $this->addError($attribute, 'Must be one of these types: ' . implode(', ', $expectedTypes));
+                    $this->addError($attribute, 'Must be one of these types: '.implode(', ', $expectedTypes));
                 }
             }
         }

src/models/MetaJsonLdContainer.php

Spacing +4 added lines, -4 removed lines

@@ -51,16 +51,16 @@
     public function includeMetaData($dependency)
     {
         Craft::beginProfile('MetaJsonLdContainer::includeMetaData', __METHOD__);
-        $uniqueKey = $this->handle . $dependency->tags[3] . '-v2';
+        $uniqueKey = $this->handle.$dependency->tags[3].'-v2';
         $cache = Craft::$app->getCache();
         if ($this->clearCache) {
             TagDependency::invalidate($cache, $dependency->tags[3]);
         }
         [$jsonLd, $attrs] = $cache->getOrSet(
-            self::CONTAINER_TYPE . $uniqueKey,
-            function () use ($uniqueKey) {
+            self::CONTAINER_TYPE.$uniqueKey,
+            function() use ($uniqueKey) {
                 Craft::info(
-                    self::CONTAINER_TYPE . ' cache miss: ' . $uniqueKey,
+                    self::CONTAINER_TYPE.' cache miss: '.$uniqueKey,
                     __METHOD__
                 );
 

src/models/MetaScriptContainer.php

Spacing +5 added lines, -5 removed lines

@@ -55,16 +55,16 @@ 
     public function includeMetaData($dependency)
     {
         Craft::beginProfile('MetaScriptContainer::includeMetaData', __METHOD__);
-        $uniqueKey = $this->handle . $dependency->tags[3] . $this->dataLayerHash();
+        $uniqueKey = $this->handle.$dependency->tags[3].$this->dataLayerHash();
         $cache = Craft::$app->getCache();
         if ($this->clearCache) {
             TagDependency::invalidate($cache, $dependency->tags[3]);
         }
         $tagData = $cache->getOrSet(
-            self::CONTAINER_TYPE . $uniqueKey,
-            function () use ($uniqueKey) {
+            self::CONTAINER_TYPE.$uniqueKey,
+            function() use ($uniqueKey) {
                 Craft::info(
-                    self::CONTAINER_TYPE . ' cache miss: ' . $uniqueKey,
+                    self::CONTAINER_TYPE.' cache miss: '.$uniqueKey,
                     __METHOD__
                 );
 
@@ -140,7 +140,7 @@ 
                     'jQuery(window).load()',
                 ];
                 $position = $positionNames[$config['position'] - 1] ?? 'unknown';
-                $html .= "<!-- Position: {$position} -->" . PHP_EOL;
+                $html .= "<!-- Position: {$position} -->".PHP_EOL;
             }
             if ($bodyJs || !$params['renderScriptTags']) {
                 $html .= $config['js'];

src/services/Sitemaps.php

Spacing +16 added lines, -16 removed lines

@@ -42,11 +42,11 @@ 
     // Constants
     // =========================================================================
 
-    const SEOMATIC_SITEMAPINDEX_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapIndexTemplate::TEMPLATE_TYPE;
+    const SEOMATIC_SITEMAPINDEX_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapIndexTemplate::TEMPLATE_TYPE;
 
-    const SEOMATIC_SITEMAP_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapTemplate::TEMPLATE_TYPE;
+    const SEOMATIC_SITEMAP_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapTemplate::TEMPLATE_TYPE;
 
-    const SEOMATIC_SITEMAPCUSTOM_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapCustomTemplate::TEMPLATE_TYPE;
+    const SEOMATIC_SITEMAPCUSTOM_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapCustomTemplate::TEMPLATE_TYPE;
 
     const SEARCH_ENGINE_SUBMISSION_URLS = [
     ];
@@ -90,7 +90,7 @@ 
             Event::on(
                 UrlManager::class,
                 UrlManager::EVENT_REGISTER_SITE_URL_RULES,
-                function (RegisterUrlRulesEvent $event) {
+                function(RegisterUrlRulesEvent $event) {
                     Craft::debug(
                         'UrlManager::EVENT_REGISTER_SITE_URL_RULES',
                         __METHOD__
@@ -243,19 +243,19 @@ 
                         $siteId = $groupSiteIds[0];
                         $sitemapIndexUrl = $this->sitemapIndexUrlForSiteId($siteId);
                         if (!empty($sitemapIndexUrl)) {
-                            $submissionUrl = $url . urlencode($sitemapIndexUrl);
+                            $submissionUrl = $url.urlencode($sitemapIndexUrl);
                             // create new guzzle client
                             $guzzleClient = Craft::createGuzzleClient(['timeout' => 5, 'connect_timeout' => 5]);
                             // Submit the sitemap index to each search engine
                             try {
                                 $guzzleClient->post($submissionUrl);
                                 Craft::info(
-                                    'Sitemap index submitted to: ' . $submissionUrl,
+                                    'Sitemap index submitted to: '.$submissionUrl,
                                     __METHOD__
                                 );
                             } catch (\Exception $e) {
                                 Craft::error(
-                                    'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
+                                    'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
                                     __METHOD__
                                 );
                             }
@@ -315,19 +315,19 @@ 
             foreach ($searchEngineUrls as &$url) {
                 $sitemapUrl = $this->sitemapUrlForBundle($sourceBundleType, $sourceHandle, $sourceSiteId);
                 if (!empty($sitemapUrl)) {
-                    $submissionUrl = $url . urlencode($sitemapUrl);
+                    $submissionUrl = $url.urlencode($sitemapUrl);
                     // create new guzzle client
                     $guzzleClient = Craft::createGuzzleClient(['timeout' => 5, 'connect_timeout' => 5]);
                     // Submit the sitemap index to each search engine
                     try {
                         $guzzleClient->post($submissionUrl);
                         Craft::info(
-                            'Sitemap index submitted to: ' . $submissionUrl,
+                            'Sitemap index submitted to: '.$submissionUrl,
                             __METHOD__
                         );
                     } catch (\Exception $e) {
                         Craft::error(
-                            'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
+                            'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
                             __METHOD__
                         );
                     }
@@ -393,19 +393,19 @@ 
             foreach ($searchEngineUrls as &$url) {
                 $sitemapUrl = $this->sitemapCustomUrlForSiteId($siteId);
                 if (!empty($sitemapUrl)) {
-                    $submissionUrl = $url . urlencode($sitemapUrl);
+                    $submissionUrl = $url.urlencode($sitemapUrl);
                     // create new guzzle client
                     $guzzleClient = Craft::createGuzzleClient(['timeout' => 5, 'connect_timeout' => 5]);
                     // Submit the sitemap index to each search engine
                     try {
                         $guzzleClient->post($submissionUrl);
                         Craft::info(
-                            'Sitemap Custom submitted to: ' . $submissionUrl,
+                            'Sitemap Custom submitted to: '.$submissionUrl,
                             __METHOD__
                         );
                     } catch (\Exception $e) {
                         Craft::error(
-                            'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
+                            'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
                             __METHOD__
                         );
                     }
@@ -483,7 +483,7 @@ 
         }
 
         foreach ($sites as $site) {
-            $result .= 'sitemap: ' . $this->sitemapIndexUrlForSiteId($site->id) . PHP_EOL;
+            $result .= 'sitemap: '.$this->sitemapIndexUrlForSiteId($site->id).PHP_EOL;
         }
 
         return rtrim($result, PHP_EOL);
@@ -515,9 +515,9 @@ 
         // Since we want a stale-while-revalidate pattern, only invalidate the cache if we're asked to
         if ($invalidateCache) {
             $cache = Craft::$app->getCache();
-            TagDependency::invalidate($cache, SitemapTemplate::SITEMAP_CACHE_TAG . $handle . $siteId);
+            TagDependency::invalidate($cache, SitemapTemplate::SITEMAP_CACHE_TAG.$handle.$siteId);
             Craft::info(
-                'Sitemap cache cleared: ' . $handle,
+                'Sitemap cache cleared: '.$handle,
                 __METHOD__
             );
         }

src/helpers/UrlHelper.php

Spacing +13 added lines, -13 removed lines

@@ -44,12 +44,12 @@ 
             $siteUrl = MetaValue::parseString($siteUrl);
             // Extract out just the path part
             $parts = self::decomposeUrl($path);
-            $path = $parts['path'] . $parts['suffix'];
-            $url = rtrim($siteUrl, '/') . '/' . ltrim($path, '/');
+            $path = $parts['path'].$parts['suffix'];
+            $url = rtrim($siteUrl, '/').'/'.ltrim($path, '/');
             // Handle trailing slashes properly for generated URLs
             $generalConfig = Craft::$app->getConfig()->getGeneral();
             if ($generalConfig->addTrailingSlashesToUrls && !preg_match('/\.[^\/]+$/', $url)) {
-                $url = rtrim($url, '/') . '/';
+                $url = rtrim($url, '/').'/';
             }
             if (!$generalConfig->addTrailingSlashesToUrls) {
                 $url = rtrim($url, '/');
@@ -126,7 +126,7 @@ 
         // Handle trailing slashes properly for generated URLs
         $generalConfig = Craft::$app->getConfig()->getGeneral();
         if ($generalConfig->addTrailingSlashesToUrls && !preg_match('/\.[^\/]+$/', $url)) {
-            $url = rtrim($url, '/') . '/';
+            $url = rtrim($url, '/').'/';
         }
         if (!$generalConfig->addTrailingSlashesToUrls) {
             $url = rtrim($url, '/');
@@ -146,13 +146,13 @@ 
         $urlParts = parse_url($url);
         $encodedUrl = "";
         if (isset($urlParts['scheme'])) {
-            $encodedUrl .= $urlParts['scheme'] . '://';
+            $encodedUrl .= $urlParts['scheme'].'://';
         }
         if (isset($urlParts['host'])) {
             $encodedUrl .= $urlParts['host'];
         }
         if (isset($urlParts['port'])) {
-            $encodedUrl .= ':' . $urlParts['port'];
+            $encodedUrl .= ':'.$urlParts['port'];
         }
         if (isset($urlParts['path'])) {
             $encodedUrl .= $urlParts['path'];
@@ -162,15 +162,15 @@ 
             foreach ($query as $j => $value) {
                 $value = explode('=', $value, 2);
                 if (count($value) === 2) {
-                    $query[$j] = urlencode($value[0]) . '=' . urlencode($value[1]);
+                    $query[$j] = urlencode($value[0]).'='.urlencode($value[1]);
                 } else {
                     $query[$j] = urlencode($value[0]);
                 }
             }
-            $encodedUrl .= '?' . implode('&', $query);
+            $encodedUrl .= '?'.implode('&', $query);
         }
         if (isset($urlParts['fragment'])) {
-            $encodedUrl .= '#' . $urlParts['fragment'];
+            $encodedUrl .= '#'.$urlParts['fragment'];
         }
 
         return $encodedUrl;
@@ -210,7 +210,7 @@ 
             if ($siteId !== null) {
                 $site = $sites->getSiteById($siteId, true);
                 if (!$site) {
-                    throw new Exception('Invalid site ID: ' . $siteId);
+                    throw new Exception('Invalid site ID: '.$siteId);
                 }
             }
 
@@ -236,11 +236,11 @@ 
 
         if (filter_var($pathOrUrl, FILTER_VALIDATE_URL)) {
             $url_parts = parse_url($pathOrUrl);
-            $result['prefix'] = $url_parts['scheme'] . '://' . $url_parts['host'];
+            $result['prefix'] = $url_parts['scheme'].'://'.$url_parts['host'];
             $result['path'] = $url_parts['path'] ?? '';
             $result['suffix'] = '';
-            $result['suffix'] .= empty($url_parts['query']) ? '' : '?' . $url_parts['query'];
-            $result['suffix'] .= empty($url_parts['fragment']) ? '' : '#' . $url_parts['fragment'];
+            $result['suffix'] .= empty($url_parts['query']) ? '' : '?'.$url_parts['query'];
+            $result['suffix'] .= empty($url_parts['fragment']) ? '' : '#'.$url_parts['fragment'];
         } else {
             $result['prefix'] = '';
             $result['path'] = $pathOrUrl;