nystudio107 / craft-seomatic

src/helpers/MetaValue.php

Spacing +2 added lines, -2 removed lines

@@ -294,7 +294,7 @@ 
                 $metaValue = Craft::t(
                     'seomatic',
                     'Error rendering `{template}` -> {error}',
-                    ['template' => $metaValue, 'error' => $e->getMessage() . ' - ' . print_r($metaValue, true)]
+                    ['template' => $metaValue, 'error' => $e->getMessage().' - '.print_r($metaValue, true)]
                 );
                 Craft::error($metaValue, __METHOD__);
                 Craft::$app->getErrorHandler()->logException($e);
@@ -313,7 +313,7 @@ 
         // Handle being passed in an object
         if (is_object($metaValue)) {
             if ($metaValue instanceof Markup) {
-                return trim(html_entity_decode((string)$metaValue, ENT_NOQUOTES, 'UTF-8'));
+                return trim(html_entity_decode((string) $metaValue, ENT_NOQUOTES, 'UTF-8'));
             }
             if ($metaValue instanceof Asset) {
                 return $metaValue->uri;

src/helpers/Text.php

Spacing +10 added lines, -10 removed lines

@@ -73,7 +73,7 @@ 
         if (!empty($string)) {
             $string = HtmlPurifier::process($string, ['HTML.Allowed' => '']);
             $string = html_entity_decode($string, ENT_NOQUOTES, 'UTF-8');
-            $result = (string)Stringy::create($string)->truncate($length, $substring);
+            $result = (string) Stringy::create($string)->truncate($length, $substring);
         }
 
         return $result;
@@ -98,7 +98,7 @@ 
         if (!empty($string)) {
             $string = HtmlPurifier::process($string, ['HTML.Allowed' => '']);
             $string = html_entity_decode($string, ENT_NOQUOTES, 'UTF-8');
-            $result = (string)Stringy::create($string)->safeTruncate($length, $substring);
+            $result = (string) Stringy::create($string)->safeTruncate($length, $substring);
         }
 
         return $result;
@@ -132,9 +132,9 @@ 
             $result = self::smartStripTags(Doxter::$plugin->getService()->parseMarkdown($field->getRaw()));
         } else {
             if (is_array($field)) {
-                $result = self::smartStripTags((string)$field[0]);
+                $result = self::smartStripTags((string) $field[0]);
             } else {
-                $result = self::smartStripTags((string)$field);
+                $result = self::smartStripTags((string) $field);
             }
         }
 
@@ -161,7 +161,7 @@ 
             $tags = $tags->all();
         }
         foreach ($tags as $tag) {
-            $result .= $tag->title . ', ';
+            $result .= $tag->title.', ';
         }
         $result = rtrim($result, ', ');
 
@@ -203,7 +203,7 @@ 
                     foreach ($fieldClasses as $fieldClassKey) {
                         if ($field instanceof $fieldClassKey) {
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
                             }
                         }
                     }
@@ -245,7 +245,7 @@ 
                     foreach ($fieldClasses as $fieldClassKey) {
                         if ($field instanceof $fieldClassKey) {
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
                             }
                         }
                     }
@@ -294,7 +294,7 @@ 
                     foreach ($fieldClasses as $fieldClassKey) {
                         if ($field instanceof $fieldClassKey) {
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
                             }
                         }
                     }
@@ -393,7 +393,7 @@ 
         $str = html_entity_decode($str, ENT_NOQUOTES, 'UTF-8');
         $str = rawurldecode($str);
         // Remove any linebreaks
-        $str = (string)preg_replace("/\r|\n/", "", $str);
+        $str = (string) preg_replace("/\r|\n/", "", $str);
         $str = HtmlPurifier::process($str, ['HTML.Allowed' => '']);
         $str = html_entity_decode($str, ENT_NOQUOTES, 'UTF-8');
         // Remove any embedded Twig code
@@ -473,7 +473,7 @@ 
             $language = 'English';
         }
 
-        $className = 'PhpScience\\TextRank\\Tool\\StopWords\\' . ucfirst($language);
+        $className = 'PhpScience\\TextRank\\Tool\\StopWords\\'.ucfirst($language);
         if (class_exists($className)) {
             $stopWords = new $className();
         }

src/debug/views/seomatic/render-tag.php

Braces +5 added lines, -2 removed lines

@@ -75,8 +75,11 @@
                     </ul><?php endif; ?><?php endforeach; ?><?php endif; ?></div>
         </details>
     </td>
-<?php else: ?>
+<?php else {
+    : ?>
     <td>
         <div class="callout callout-secondary seomatic-callout">not included</div>
     </td>
-<?php endif; ?>
+<?php endif;
+}
+?>

src/debug/views/seomatic/render-value.php

Braces +5 added lines, -2 removed lines

@@ -45,7 +45,10 @@
             </div>
         </details>
     </td>
-<?php else: ?>
+<?php else {
+    : ?>
     <td><?= htmlspecialchars(VarDumper::dumpAsString($value), ENT_QUOTES | ENT_SUBSTITUTE,
             Yii::$app->charset, true) ?></td>
-<?php endif; ?>
+<?php endif;
+}
+?>

src/seoelements/SeoEvent.php

Spacing +3 added lines, -3 removed lines

@@ -241,8 +241,8 @@ 
         $query
             ->setCalendar($metaBundle->sourceHandle)
             ->setLoadOccurrences(false)
-            ->siteId((int)$metaBundle->sourceSiteId)
-            ->limit((int)$metaBundle->metaSitemapVars->sitemapLimit);
+            ->siteId((int) $metaBundle->sourceSiteId)
+            ->limit((int) $metaBundle->metaSitemapVars->sitemapLimit);
 
         return $query;
     }
@@ -405,7 +405,7 @@ 
             ConfigHelper::getConfigFromFile(self::configFilePath()),
             [
                 'sourceId' => $sourceModel->id,
-                'sourceName' => (string)$sourceModel->name,
+                'sourceName' => (string) $sourceModel->name,
                 'sourceHandle' => $sourceModel->handle,
             ]
         );

src/base/FluentModel.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         // Set the property
         $value = $args[0];
         if (is_object($value) && $value instanceof Markup) {
-            $value = (string)$value;
+            $value = (string) $value;
         }
         $property->setValue($this, $value);
 

src/base/MetaItem.php

Spacing +6 added lines, -6 removed lines

@@ -162,7 +162,7 @@ 
                 // Add a URL to the schema.org type if this is a MetaJsonLD object
                 if ($isMetaJsonLdModel) {
                     /** @var MetaJsonLd $this */
-                    $extraInfo = ' for http://schema.org/' . $this->type;
+                    $extraInfo = ' for http://schema.org/'.$this->type;
                 }
                 $errorMsg =
                     Craft::t('seomatic', 'Scenario: "')
@@ -173,10 +173,10 @@ 
                     . print_r($this->render(), true);
                 Craft::info($errorMsg, __METHOD__);
                 foreach ($this->errors as $param => $errors) {
-                    $errorMsg = Craft::t('seomatic', $errorLabel) . $param;
+                    $errorMsg = Craft::t('seomatic', $errorLabel).$param;
                     /** @var array $errors */
                     foreach ($errors as $error) {
-                        $errorMsg .= ' -> ' . $error;
+                        $errorMsg .= ' -> '.$error;
                         // Change the error level depending on the error message if this is a MetaJsonLD object
                         if ($isMetaJsonLdModel) {
                             if (strpos($error, 'recommended') !== false) {
@@ -189,14 +189,14 @@ 
                             }
                         }
                     }
-                    Craft::info(strtoupper($logLevel) . ' - ' . $errorMsg, __METHOD__);
+                    Craft::info(strtoupper($logLevel).' - '.$errorMsg, __METHOD__);
                     // Extra debugging info for MetaJsonLd objects
                     if ($isMetaJsonLdModel) {
                         /** @var MetaJsonLd $className */
                         $className = get_class($this);
                         if (!empty($className->schemaPropertyDescriptions[$param])) {
-                            $errorMsg = Craft::t('seomatic', $errorLabel) . $param;
-                            $errorMsg .= ' -> ' . $className->schemaPropertyDescriptions[$param];
+                            $errorMsg = Craft::t('seomatic', $errorLabel).$param;
+                            $errorMsg .= ' -> '.$className->schemaPropertyDescriptions[$param];
                             Craft::info($errorMsg, __METHOD__);
                         }
                     }

src/models/MetaSiteVars.php

Spacing +2 added lines, -2 removed lines

@@ -238,10 +238,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 (is_array($this->identity)) {

src/helpers/Schema.php

Spacing +16 added lines, -16 removed lines

@@ -100,7 +100,7 @@ 
             // Go from most specific type to least specific type
             foreach (self::SCHEMA_TYPES as $schemaType) {
                 if (!empty($settings[$schemaType]) && ($settings[$schemaType] !== 'none')) {
-                    $result = $settings[$schemaType] . self::SCHEMA_PATH_DELIMITER . $result;
+                    $result = $settings[$schemaType].self::SCHEMA_PATH_DELIMITER.$result;
                 }
             }
         }
@@ -131,7 +131,7 @@ 
             if (isset($result['schemaTypeDescription'])) {
                 $description = $result['schemaTypeDescription'];
                 $description = preg_replace("`\[\[([A-z]*)\]\]`", '[$1](https://schema.org/$1)', $description);
-                $description = Markdown::process((string)$description);
+                $description = Markdown::process((string) $description);
                 $description = str_replace(['<p>', '</p>', '\n'], ['', '', ' '], $description);
                 $result['schemaTypeDescription'] = $description;
             }
@@ -151,7 +151,7 @@ 
     {
         $result = [];
         while ($schemaType) {
-            $className = 'nystudio107\\seomatic\\models\\jsonld\\' . $schemaType;
+            $className = 'nystudio107\\seomatic\\models\\jsonld\\'.$schemaType;
             if (class_exists($className)) {
                 $classRef = new ReflectionClass($className);
                 $staticProps = $classRef->getStaticProperties();
@@ -248,15 +248,15 @@ 
         $dependency = new TagDependency([
             'tags' => [
                 self::GLOBAL_SCHEMA_CACHE_TAG,
-                self::SCHEMA_CACHE_TAG . 'schemaArray',
+                self::SCHEMA_CACHE_TAG.'schemaArray',
             ],
         ]);
         $cache = Craft::$app->getCache();
         $typesArray = $cache->getOrSet(
-            self::CACHE_KEY . 'schemaArray',
+            self::CACHE_KEY.'schemaArray',
             function() use ($path) {
                 Craft::info(
-                    'schemaArray cache miss' . $path,
+                    'schemaArray cache miss'.$path,
                     __METHOD__
                 );
                 $filePath = Craft::getAlias('@nystudio107/seomatic/resources/schema/tree.jsonld');
@@ -321,12 +321,12 @@ 
         $dependency = new TagDependency([
             'tags' => [
                 self::GLOBAL_SCHEMA_CACHE_TAG,
-                self::SCHEMA_CACHE_TAG . 'schemaTree',
+                self::SCHEMA_CACHE_TAG.'schemaTree',
             ],
         ]);
         $cache = Craft::$app->getCache();
         $typesArray = $cache->getOrSet(
-            self::CACHE_KEY . 'schemaTree',
+            self::CACHE_KEY.'schemaTree',
             function() {
                 Craft::info(
                     'schemaArray cache miss',
@@ -390,11 +390,11 @@ 
         foreach ($typesArray as $key => $value) {
             $indent = html_entity_decode(str_repeat('&nbsp;', $indentLevel));
             if (is_array($value)) {
-                $result[$key] = $indent . $key;
+                $result[$key] = $indent.$key;
                 $value = self::flattenSchemaArray($value, $indentLevel + self::MENU_INDENT_STEP);
                 $result = array_merge($result, $value);
             } else {
-                $result[$key] = $indent . $value;
+                $result[$key] = $indent.$value;
             }
         }
 
@@ -488,7 +488,7 @@ 
             $children = [];
             $name = $typesArray['name'];
             // Construct a path-based $id, excluding the top-level `Thing` schema
-            $id = $name === 'Thing' ? '' : $path . self::SCHEMA_PATH_DELIMITER . $name;
+            $id = $name === 'Thing' ? '' : $path.self::SCHEMA_PATH_DELIMITER.$name;
             $id = ltrim($id, self::SCHEMA_PATH_DELIMITER);
             // Make sure we have at most 3 specifiers in the schema path
             $parts = explode(self::SCHEMA_PATH_DELIMITER, $id);
@@ -515,12 +515,12 @@ 
             $schemaPath = explode(self::SCHEMA_PATH_DELIMITER, $id);
             // Use only the specific (last) type for now, rather than the complete path of types
             $schemaPath = [end($schemaPath)];
-            if ((bool)array_intersect($schemaPath, array_keys($googleRichSnippetTypes))) {
-                $name .= ' (' . Craft::t('seomatic', 'Google rich result') . ')';
+            if ((bool) array_intersect($schemaPath, array_keys($googleRichSnippetTypes))) {
+                $name .= ' ('.Craft::t('seomatic', 'Google rich result').')';
             }
             // Mark it as pending, if applicable
             if (isset($typesArray['pending']) && $typesArray['pending']) {
-                $name .= ' (' . Craft::t('seomatic', 'pending') . ')';
+                $name .= ' ('.Craft::t('seomatic', 'pending').')';
             }
             $result['label'] = $name;
             $result['id'] = $id;
@@ -539,12 +539,12 @@ 
         $dependency = new TagDependency([
             'tags' => [
                 self::GLOBAL_SCHEMA_CACHE_TAG,
-                self::SCHEMA_CACHE_TAG . 'googleRichSnippets',
+                self::SCHEMA_CACHE_TAG.'googleRichSnippets',
             ],
         ]);
         $cache = Craft::$app->getCache();
         return $cache->getOrSet(
-            self::CACHE_KEY . 'googleRichSnippets',
+            self::CACHE_KEY.'googleRichSnippets',
             function() {
                 Craft::info(
                     'googleRichSnippets cache miss',