nystudio107 / craft-seomatic

src/models/SitemapIndexTemplate.php

Spacing +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
      */
     public function getFilename(int $groupId): string
     {
-        return 'sitemaps-' . $groupId . '-sitemap.xml';
+        return 'sitemaps-'.$groupId.'-sitemap.xml';
     }
 
     /**
@@ -150,7 +150,7 @@ 
             ],
         ]);
 
-        return $cache->getOrSet(self::CACHE_KEY . $groupId . '.' . $siteId, function () use ($groupSiteIds, $siteId) {
+        return $cache->getOrSet(self::CACHE_KEY.$groupId.'.'.$siteId, function() use ($groupSiteIds, $siteId) {
             Craft::info(
                 'Sitemap index cache miss',
                 __METHOD__

src/models/MetaSiteVars.php

Spacing +2 added lines, -2 removed lines

@@ -225,10 +225,10 @@
         }
         // Make sure these are strings
         if (!empty($this->facebookProfileId)) {
-            $this->facebookProfileId = (string)$this->facebookProfileId;
+            $this->facebookProfileId = (string) $this->facebookProfileId;
         }
         if (!empty($this->facebookAppId)) {
-            $this->facebookAppId = (string)$this->facebookAppId;
+            $this->facebookAppId = (string) $this->facebookAppId;
         }
         // Identity
         if ($this->identity !== null && is_array($this->identity)) {

src/models/metatag/TwitterTitleTag.php

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
                     $suffix = '';
                     break;
             }
-            $lengthAdjust = mb_strlen($prefix . $suffix);
+            $lengthAdjust = mb_strlen($prefix.$suffix);
             // Truncate the twitter:title tag content
             $truncLen = Seomatic::$settings->maxTitleLength - $lengthAdjust;
             if ($truncLen < 0) {
@@ -138,7 +138,7 @@ 
                     '…'
                 );
             }
-            $data['content'] = $prefix . $data['content'] . $suffix;
+            $data['content'] = $prefix.$data['content'].$suffix;
         }
 
         return $shouldRender;

src/models/metatag/OgTitleTag.php

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
                     $suffix = '';
                     break;
             }
-            $lengthAdjust = mb_strlen($prefix . $suffix);
+            $lengthAdjust = mb_strlen($prefix.$suffix);
             // Truncate the og:title tag content
             $truncLen = Seomatic::$settings->maxTitleLength - $lengthAdjust;
             if ($truncLen < 0) {
@@ -138,7 +138,7 @@ 
                     '…'
                 );
             }
-            $data['content'] = $prefix . $data['content'] . $suffix;
+            $data['content'] = $prefix.$data['content'].$suffix;
         }
 
         return $shouldRender;

src/models/MetaTitle.php

Spacing +5 added lines, -5 removed lines

@@ -143,16 +143,16 @@ 
                     break;
             }
             // Handle the case of empty titles
-            if ($prefix === (' ' . $separator . ' ')) {
+            if ($prefix === (' '.$separator.' ')) {
                 $prefix = '';
             }
-            if ($suffix === (' ' . $separator)) {
+            if ($suffix === (' '.$separator)) {
                 $suffix = '';
             }
             // Remove potential double spaces
             $prefix = preg_replace('/\s+/', ' ', $prefix);
             $suffix = preg_replace('/\s+/', ' ', $suffix);
-            $lengthAdjust = mb_strlen($prefix . $suffix);
+            $lengthAdjust = mb_strlen($prefix.$suffix);
             // Parse the data
             $scenario = $this->scenario;
             $this->setScenario('render');
@@ -171,7 +171,7 @@ 
                         '…'
                     );
                 }
-                $data = $prefix . $data . $suffix;
+                $data = $prefix.$data.$suffix;
             } else {
                 // If no title is provided, just use the site name
                 $data = MetaValueHelper::parseString($this->siteName);
@@ -180,7 +180,7 @@ 
             $data = trim($data);
             // devMode
             if (Seomatic::$devMode) {
-                $data = Seomatic::$settings->devModeTitlePrefix . $data;
+                $data = Seomatic::$settings->devModeTitlePrefix.$data;
             }
         }
 

src/services/MetaBundles.php

Spacing +16 added lines, -16 removed lines

@@ -178,7 +178,7 @@ 
      * @param MetaBundle $metaBundle
      * @param bool $forceUpdate
      */
-    public function syncBundleWithConfig(MetaBundle &$metaBundle, bool $forceUpdate = false)
+    public function syncBundleWithConfig(MetaBundle & $metaBundle, bool $forceUpdate = false)
     {
         $prevMetaBundle = $metaBundle;
         $config = [];
@@ -259,8 +259,8 @@ 
      */
     public function updateMetaBundle(MetaBundle $metaBundle, int $siteId)
     {
-        $metaBundle->sourceName = (string)$metaBundle->sourceName;
-        $metaBundle->sourceTemplate = (string)$metaBundle->sourceTemplate;
+        $metaBundle->sourceName = (string) $metaBundle->sourceName;
+        $metaBundle->sourceTemplate = (string) $metaBundle->sourceTemplate;
         // Make sure it validates
         if ($metaBundle->validate(null, true)) {
             // Save it out to a record
@@ -270,7 +270,7 @@ 
                 'sourceSiteId' => $siteId,
             ];
             if ($metaBundle->typeId !== null) {
-                $metaBundle->typeId = (int)$metaBundle->typeId;
+                $metaBundle->typeId = (int) $metaBundle->typeId;
             }
             if (!empty($metaBundle->typeId)) {
                 $params['typeId'] = $metaBundle->typeId;
@@ -286,7 +286,7 @@ 
             // @TODO remove this hack that doesn't allow environment-transformed settings to be saved in a meta bundle with a proper system to address it
             // The issue was that the containers were getting saved to the db with a hard-coded setting in them, because they'd
             // been set that way by the environment, whereas to be changeable via the GUI, it needs to be set to {seomatic.meta.robots}
-            $robotsTag = $metaBundle->metaContainers[MetaTagContainer::CONTAINER_TYPE . TagService::GENERAL_HANDLE]->data['robots'] ?? null;
+            $robotsTag = $metaBundle->metaContainers[MetaTagContainer::CONTAINER_TYPE.TagService::GENERAL_HANDLE]->data['robots'] ?? null;
             if (!empty($robotsTag)) {
                 $robotsTag->content = $robotsTag->environment['live']['content'] ?? '{{ seomatic.meta.robots }}';
             }
@@ -368,7 +368,7 @@ 
                 $metaBundleDefaults = ArrayHelper::merge(
                     $seoElement::metaBundleConfig($sourceModel),
                     [
-                        'sourceTemplate' => (string)$siteSetting->template,
+                        'sourceTemplate' => (string) $siteSetting->template,
                         'sourceSiteId' => $siteSetting->siteId,
                         'sourceAltSiteSettings' => $siteSettingsArray,
                         'sourceDateUpdated' => $dateUpdated,
@@ -416,7 +416,7 @@ 
     public function getMetaBundleBySourceHandle(string $sourceBundleType, string $sourceHandle, int $sourceSiteId, $typeId = null)
     {
         $metaBundle = null;
-        $typeId = (int)$typeId;
+        $typeId = (int) $typeId;
         // See if we have the meta bundle cached
         if (!empty($this->metaBundlesBySourceHandle[$sourceBundleType][$sourceHandle][$sourceSiteId][$typeId])) {
             $id = $this->metaBundlesBySourceHandle[$sourceBundleType][$sourceHandle][$sourceSiteId][$typeId];
@@ -503,7 +503,7 @@ 
                             /** @var Section|CategoryGroup|ProductType $sourceModel */
                             $sourceModel = $seoElement::sourceModelFromId($sourceId);
                             if ($sourceModel !== null) {
-                                $metaBundle->sourceName = (string)$sourceModel->name;
+                                $metaBundle->sourceName = (string) $sourceModel->name;
                                 $metaBundle->sourceHandle = $sourceModel->handle;
                             }
                         }
@@ -544,7 +544,7 @@ 
     public function getMetaBundleBySourceId(string $sourceBundleType, int $sourceId, int $sourceSiteId, $typeId = null)
     {
         $metaBundle = null;
-        $typeId = (int)$typeId;
+        $typeId = (int) $typeId;
         // See if we have the meta bundle cached
         if (!empty($this->metaBundlesBySourceId[$sourceBundleType][$sourceId][$sourceSiteId][$typeId])) {
             $id = $this->metaBundlesBySourceId[$sourceBundleType][$sourceId][$sourceSiteId][$typeId];
@@ -739,7 +739,7 @@ 
                 ->from(['{{%seomatic_metabundles}}'])
                 ->where(['=', 'sourceBundleType', $seoElement::META_BUNDLE_TYPE]);
 
-            if ((int)$sourceSiteId !== 0) {
+            if ((int) $sourceSiteId !== 0) {
                 $subQuery->andWhere(['sourceSiteId' => $sourceSiteId]);
             }
             if ($filter !== '') {
@@ -776,7 +776,7 @@ 
     public function getContentMetaBundleForElement(Element $element)
     {
         $source = $this->getMetaSourceFromElement($element);
-        $key = implode(".", $source) . '.' . $element->siteId;
+        $key = implode(".", $source).'.'.$element->siteId;
 
         if (empty($this->elementContentMetaBundles[$key])) {
             $this->elementContentMetaBundles[$key] = $this->getMetaBundleBySourceId($source[1], $source[0], $element->siteId, $source[4]);
@@ -798,10 +798,10 @@ 
         $seoSettingsField = Craft::$app->getFields()->getFieldByHandle($fieldHandle);
         if ($seoSettingsField) {
             $seoSettingsEnabledFields = array_flip(array_merge(
-                (array)$seoSettingsField->generalEnabledFields,
-                (array)$seoSettingsField->twitterEnabledFields,
-                (array)$seoSettingsField->facebookEnabledFields,
-                (array)$seoSettingsField->sitemapEnabledFields
+                (array) $seoSettingsField->generalEnabledFields,
+                (array) $seoSettingsField->twitterEnabledFields,
+                (array) $seoSettingsField->facebookEnabledFields,
+                (array) $seoSettingsField->sitemapEnabledFields
             ));
             // Always include some fields, as they are calculated even if not explicitly included
             $seoSettingsEnabledFields = array_merge(
@@ -829,7 +829,7 @@ 
 
 
             // Handle the mainEntityOfPage
-            if (!in_array('mainEntityOfPage', (array)$seoSettingsField->generalEnabledFields, false)) {
+            if (!in_array('mainEntityOfPage', (array) $seoSettingsField->generalEnabledFields, false)) {
                 $metaBundle->metaGlobalVars->mainEntityOfPage = '';
             }
             // metaSiteVars

src/fields/SeoSettings.php

Spacing +10 added lines, -10 removed lines

@@ -160,7 +160,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getContentColumnType(): array|string
+    public function getContentColumnType(): array | string
     {
         return Schema::TYPE_TEXT;
     }
@@ -231,7 +231,7 @@ 
             }
             // Handle the mainEntityOfPage
             $mainEntity = '';
-            if (in_array('mainEntityOfPage', (array)$this->generalEnabledFields, false) &&
+            if (in_array('mainEntityOfPage', (array) $this->generalEnabledFields, false) &&
                 !empty($config['metaBundleSettings'])) {
                 $mainEntity = SchemaHelper::getSpecificEntityType($config['metaBundleSettings'], true);
             }
@@ -261,7 +261,7 @@ 
                 $value = StringHelper::encodeMb4($value);
             }
             if (is_array($value)) {
-                array_walk_recursive($value, function (&$arrayValue, $arrayKey) {
+                array_walk_recursive($value, function(&$arrayValue, $arrayKey) {
                     if ($arrayValue !== null && is_string($arrayValue)) {
                         $arrayValue = StringHelper::encodeMb4($arrayValue);
                     }
@@ -397,7 +397,7 @@ 
         if ($element !== null && $element->uri !== null) {
             $siteId = $element->siteId;
             $uri = $element->uri;
-            $cacheKey = self::CACHE_KEY . $uri . $siteId . $this->elementDisplayPreviewType;
+            $cacheKey = self::CACHE_KEY.$uri.$siteId.$this->elementDisplayPreviewType;
             $metaBundleSourceType = Seomatic::$plugin->seoElements->getMetaBundleTypeFromElement($element);
             $seoElement = Seomatic::$plugin->seoElements->getSeoElementByMetaBundleType($metaBundleSourceType);
             $metaBundleSourceType = SeoEntry::getMetaBundleType();
@@ -408,15 +408,15 @@ 
             $dependency = new TagDependency([
                 'tags' => [
                     MetaContainers::GLOBAL_METACONTAINER_CACHE_TAG,
-                    MetaContainers::METACONTAINER_CACHE_TAG . $metaBundleSourceId . $metaBundleSourceType . $siteId,
-                    MetaContainers::METACONTAINER_CACHE_TAG . $uri . $siteId,
+                    MetaContainers::METACONTAINER_CACHE_TAG.$metaBundleSourceId.$metaBundleSourceType.$siteId,
+                    MetaContainers::METACONTAINER_CACHE_TAG.$uri.$siteId,
                 ],
             ]);
             $cache = Craft::$app->getCache();
             $cacheDuration = null;
             $html = $cache->getOrSet(
-                self::CACHE_KEY . $cacheKey,
-                function () use ($uri, $siteId, $element) {
+                self::CACHE_KEY.$cacheKey,
+                function() use ($uri, $siteId, $element) {
                     Seomatic::$plugin->metaContainers->previewMetaContainers($uri, $siteId, true);
                     $variables = [
                         'previewTypes' => [
@@ -458,7 +458,7 @@ 
     )
     {
         $variables['textFieldSources'] = array_merge(
-            ['entryGroup' => ['optgroup' => $groupName . ' Fields'], 'title' => 'Title'],
+            ['entryGroup' => ['optgroup' => $groupName.' Fields'], 'title' => 'Title'],
             FieldHelper::fieldsOfTypeFromElement(
                 $element,
                 FieldHelper::TEXT_FIELD_CLASS_KEY,
@@ -466,7 +466,7 @@ 
             )
         );
         $variables['assetFieldSources'] = array_merge(
-            ['entryGroup' => ['optgroup' => $groupName . ' Fields']],
+            ['entryGroup' => ['optgroup' => $groupName.' Fields']],
             FieldHelper::fieldsOfTypeFromElement(
                 $element,
                 FieldHelper::ASSET_FIELD_CLASS_KEY,

src/helpers/Sitemap.php

Spacing +19 added lines, -19 removed lines

@@ -79,7 +79,7 @@ 
 
         // Output some info if this is a console app
         if ($job && Craft::$app instanceof ConsoleApplication) {
-            echo $job->description . PHP_EOL;
+            echo $job->description.PHP_EOL;
         }
 
         $lines = [];
@@ -139,8 +139,8 @@ 
             while ($currentElement < $totalElements) {
                 $elements = $paginator->getPageResults();
                 if (Craft::$app instanceof ConsoleApplication) {
-                    echo 'Query ' . $paginator->getCurrentPage() . '/' . $paginator->getTotalPages()
-                        . ' - elements: ' . $paginator->getTotalResults()
+                    echo 'Query '.$paginator->getCurrentPage().'/'.$paginator->getTotalPages()
+                        . ' - elements: '.$paginator->getTotalResults()
                         . PHP_EOL;
                 }
                 /** @var ElementInterface $element */
@@ -152,7 +152,7 @@ 
                     }
                     // Output some info if this is a console app
                     if (Craft::$app instanceof ConsoleApplication) {
-                        echo "Processing element {$currentElement}/{$totalElements} - {$element->title}" . PHP_EOL;
+                        echo "Processing element {$currentElement}/{$totalElements} - {$element->title}".PHP_EOL;
                     }
 
                     $metaBundle->metaSitemapVars->setAttributes($stashedSitemapAttrs, false);
@@ -189,7 +189,7 @@ 
                             }
                             $canonicalUrl = UrlHelper::absoluteUrlWithProtocol($canonicalUrl);
                             if ($url !== $canonicalUrl) {
-                                Craft::info("Excluding URL: {$url} from the sitemap because it does not match the Canonical URL: {$canonicalUrl} - " . $metaBundle->metaGlobalVars->canonicalUrl . " - " . $element->uri);
+                                Craft::info("Excluding URL: {$url} from the sitemap because it does not match the Canonical URL: {$canonicalUrl} - ".$metaBundle->metaGlobalVars->canonicalUrl." - ".$element->uri);
                                 continue;
                             }
                         }
@@ -241,12 +241,12 @@ 
                                                 if ($primarySiteId === $altSourceSiteId && Seomatic::$settings->addXDefaultHrefLang) {
                                                     $lines[] = '<xhtml:link rel="alternate"'
                                                         . ' hreflang="x-default"'
-                                                        . ' href="' . Html::encode($altUrl) . '"'
+                                                        . ' href="'.Html::encode($altUrl).'"'
                                                         . ' />';
                                                 }
                                                 $lines[] = '<xhtml:link rel="alternate"'
-                                                    . ' hreflang="' . $altSiteSettings['language'] . '"'
-                                                    . ' href="' . Html::encode($altUrl) . '"'
+                                                    . ' hreflang="'.$altSiteSettings['language'].'"'
+                                                    . ' href="'.Html::encode($altUrl).'"'
                                                     . ' />';
                                             }
                                         }
@@ -346,11 +346,11 @@ 
         }
 
         $cache = Craft::$app->getCache();
-        $cacheKey = SitemapTemplate::CACHE_KEY . $groupId . $type . $handle . $siteId;
+        $cacheKey = SitemapTemplate::CACHE_KEY.$groupId.$type.$handle.$siteId;
         $dependency = new TagDependency([
             'tags' => [
                 SitemapTemplate::GLOBAL_SITEMAP_CACHE_TAG,
-                SitemapTemplate::SITEMAP_CACHE_TAG . $handle . $siteId,
+                SitemapTemplate::SITEMAP_CACHE_TAG.$handle.$siteId,
             ],
         ]);
         $lines = implode('', $lines);
@@ -363,10 +363,10 @@ 
         $result = $cache->set($cacheKey, $lines, $cacheDuration, $dependency);
         // Remove the queue job id from the cache too
         $cache->delete($queueJobCacheKey);
-        Craft::debug('Sitemap cache result: ' . print_r($result, true) . ' for cache key: ' . $cacheKey, __METHOD__);
+        Craft::debug('Sitemap cache result: '.print_r($result, true).' for cache key: '.$cacheKey, __METHOD__);
         // 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;
+            echo 'Sitemap cache result: '.print_r($result, true).' for cache key: '.$cacheKey.PHP_EOL;
         }
         // If the FastCGI Cache Bust plugin is installed, clear its caches too
         $plugin = Craft::$app->getPlugins()->getPlugin('fastcgi-cache-bust');
@@ -446,7 +446,7 @@ 
 
                         $attributes = \array_intersect_key(
                             $attributes,
-                            array_flip((array)$seoSettingsField->sitemapEnabledFields)
+                            array_flip((array) $seoSettingsField->sitemapEnabledFields)
                         );
                         $attributes = array_filter(
                             $attributes,
@@ -478,7 +478,7 @@ 
      */
     protected static function assetSitemapItem(Asset $asset, MetaBundle $metaBundle, array &$lines)
     {
-        if ((bool)$asset->enabledForSite && $asset->getUrl() !== null) {
+        if ((bool) $asset->enabledForSite && $asset->getUrl() !== null) {
             switch ($asset->kind) {
                 case 'image':
                     $lines[] = '<image:image>';
@@ -490,9 +490,9 @@ 
                         $fieldName = $row['field'] ?? '';
                         $propName = $row['property'] ?? '';
                         if (!empty($asset[$fieldName]) && !empty($propName)) {
-                            $lines[] = '<image:' . $propName . '>';
+                            $lines[] = '<image:'.$propName.'>';
                             $lines[] = Html::encode($asset[$fieldName]);
-                            $lines[] = '</image:' . $propName . '>';
+                            $lines[] = '</image:'.$propName.'>';
                         }
                     }
                     $lines[] = '</image:image>';
@@ -508,9 +508,9 @@ 
                         $fieldName = $row['field'] ?? '';
                         $propName = $row['property'] ?? '';
                         if (!empty($asset[$fieldName]) && !empty($propName)) {
-                            $lines[] = '<video:' . $propName . '>';
+                            $lines[] = '<video:'.$propName.'>';
                             $lines[] = Html::encode($asset[$fieldName]);
-                            $lines[] = '</video:' . $propName . '>';
+                            $lines[] = '</video:'.$propName.'>';
                         }
                     }
                     $lines[] = '</video:video>';
@@ -526,7 +526,7 @@ 
      */
     protected static function assetFilesSitemapLink(Asset $asset, MetaBundle $metaBundle, array &$lines)
     {
-        if ((bool)$asset->enabledForSite && $asset->getUrl() !== null) {
+        if ((bool) $asset->enabledForSite && $asset->getUrl() !== null) {
             if (\in_array($asset->kind, SitemapTemplate::FILE_TYPES, false)) {
                 $dateUpdated = $asset->dateUpdated ?? $asset->dateCreated ?? new \DateTime;
                 $lines[] = '<url>';

src/seoelements/SeoEntry.php

Spacing +7 added lines, -7 removed lines

@@ -108,7 +108,7 @@ 
         Event::on(
             Sections::class,
             Sections::EVENT_AFTER_SAVE_SECTION,
-            function (SectionEvent $event) {
+            function(SectionEvent $event) {
                 Craft::debug(
                     'Sections::EVENT_AFTER_SAVE_SECTION',
                     __METHOD__
@@ -119,7 +119,7 @@ 
         Event::on(
             Sections::class,
             Sections::EVENT_AFTER_DELETE_SECTION,
-            function (SectionEvent $event) {
+            function(SectionEvent $event) {
                 Craft::debug(
                     'Sections::EVENT_AFTER_DELETE_SECTION',
                     __METHOD__
@@ -134,7 +134,7 @@ 
             Event::on(
                 Sections::class,
                 Sections::EVENT_AFTER_SAVE_SECTION,
-                function (SectionEvent $event) {
+                function(SectionEvent $event) {
                     Craft::debug(
                         'Sections::EVENT_AFTER_SAVE_SECTION',
                         __METHOD__
@@ -157,7 +157,7 @@ 
             Event::on(
                 Sections::class,
                 Sections::EVENT_AFTER_DELETE_SECTION,
-                function (SectionEvent $event) {
+                function(SectionEvent $event) {
                     Craft::debug(
                         'Sections::EVENT_AFTER_DELETE_SECTION',
                         __METHOD__
@@ -186,7 +186,7 @@ 
         Event::on(
             Entry::class,
             Entry::EVENT_DEFINE_SIDEBAR_HTML,
-            static function (DefineHtmlEvent $event) {
+            static function(DefineHtmlEvent $event) {
                 Craft::debug(
                     'Entry::EVENT_DEFINE_SIDEBAR_HTML',
                     __METHOD__
@@ -227,7 +227,7 @@ 
             ->limit($metaBundle->metaSitemapVars->sitemapLimit);
         if ($metaBundle->sourceType === 'structure'
             && !empty($metaBundle->metaSitemapVars->structureDepth)) {
-            $query->level('<=' . $metaBundle->metaSitemapVars->structureDepth);
+            $query->level('<='.$metaBundle->metaSitemapVars->structureDepth);
         }
 
         return $query;
@@ -395,7 +395,7 @@ 
             ConfigHelper::getConfigFromFile(self::configFilePath()),
             [
                 'sourceId' => $sourceModel->id,
-                'sourceName' => (string)$sourceModel->name,
+                'sourceName' => (string) $sourceModel->name,
                 'sourceHandle' => $sourceModel->handle,
                 'sourceType' => $sourceModel->type,
             ]