Passed
Push — v3 ( 8a04fd...e916ab )
by Andrew
20:43 queued 14s
created
src/helpers/ImageTransform.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
             if ($width === null) {
198 198
                 $width = '';
199 199
             }
200
-            $width = (string)$width;
200
+            $width = (string) $width;
201 201
         }
202 202
 
203 203
         return $width;
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
             if ($height === null) {
230 230
                 $height = '';
231 231
             }
232
-            $height = (string)$height;
232
+            $height = (string) $height;
233 233
         }
234 234
 
235 235
         return $height;
@@ -251,12 +251,12 @@  discard block
 block discarded – undo
251 251
             if (is_array($assetIds)) {
252 252
                 foreach ($assetIds as $assetId) {
253 253
                     if (!empty($assetId)) {
254
-                        $assets[] = $elements->getElementById((int)$assetId, Asset::class, $siteId);
254
+                        $assets[] = $elements->getElementById((int) $assetId, Asset::class, $siteId);
255 255
                     }
256 256
                 }
257 257
             } else {
258 258
                 $assetId = $assetIds;
259
-                $assets[] = $elements->getElementById((int)$assetId, Asset::class, $siteId);
259
+                $assets[] = $elements->getElementById((int) $assetId, Asset::class, $siteId);
260 260
             }
261 261
         }
262 262
 
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
             return $asset->one();
293 293
         }
294 294
 
295
-        $resolvedAssetId = (int)$asset;
295
+        $resolvedAssetId = (int) $asset;
296 296
         $resolvedSiteId = $siteId ?? 0;
297 297
         if (isset(self::$cachedAssetsElements[$resolvedAssetId][$resolvedSiteId])) {
298 298
             return self::$cachedAssetsElements[$resolvedAssetId][$resolvedSiteId];
Please login to merge, or discard this patch.
src/helpers/Field.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
         $foundFields = [];
126 126
         if (!empty(self::FIELD_CLASSES[$fieldClassKey])) {
127 127
             // Cache me if you can
128
-            $memoKey = $fieldClassKey . $layout->id . ($keysOnly ? 'keys' : 'nokeys');
128
+            $memoKey = $fieldClassKey.$layout->id.($keysOnly ? 'keys' : 'nokeys');
129 129
             if (!empty(self::$fieldsOfTypeFromLayoutCache[$memoKey])) {
130 130
                 return self::$fieldsOfTypeFromLayoutCache[$memoKey];
131 131
             }
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
                 $prefix = $global->handle;
243 243
                 $fields = array_combine(
244 244
                     array_map(function($key) use ($prefix) {
245
-                        return $prefix . '.' . $key;
245
+                        return $prefix.'.'.$key;
246 246
                     }, array_keys($fields)),
247 247
                     $fields
248 248
                 );
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
         }
317 317
         if ($matrixBlockTypeModel) {
318 318
             // Cache me if you can
319
-            $memoKey = $fieldType . $matrixBlock->id . ($keysOnly ? 'keys' : 'nokeys');
319
+            $memoKey = $fieldType.$matrixBlock->id.($keysOnly ? 'keys' : 'nokeys');
320 320
             if (!empty(self::$matrixFieldsOfTypeCache[$memoKey])) {
321 321
                 return self::$matrixFieldsOfTypeCache[$memoKey];
322 322
             }
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
         $layout = $neoBlock->getFieldLayout();
356 356
         if ($layout) {
357 357
             // Cache me if you can
358
-            $memoKey = $fieldType . $neoBlock->id . ($keysOnly ? 'keys' : 'nokeys');
358
+            $memoKey = $fieldType.$neoBlock->id.($keysOnly ? 'keys' : 'nokeys');
359 359
             if (!empty(self::$neoFieldsOfTypeCache[$memoKey])) {
360 360
                 return self::$neoFieldsOfTypeCache[$memoKey];
361 361
             }
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
         }
399 399
         if ($superTableBlockTypeModel) {
400 400
             // Cache me if you can
401
-            $memoKey = $fieldType . $superTableBlock->id . ($keysOnly ? 'keys' : 'nokeys');
401
+            $memoKey = $fieldType.$superTableBlock->id.($keysOnly ? 'keys' : 'nokeys');
402 402
             if (!empty(self::$superTableFieldsOfTypeCache[$memoKey])) {
403 403
                 return self::$superTableFieldsOfTypeCache[$memoKey];
404 404
             }
Please login to merge, or discard this patch.
src/helpers/Json.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@
 block discarded – undo
55 55
         } elseif ($json === null || $json === '') {
56 56
             return null;
57 57
         }
58
-        $decode = json_decode((string)$json, $asArray, 512, JSON_BIGINT_AS_STRING);
58
+        $decode = json_decode((string) $json, $asArray, 512, JSON_BIGINT_AS_STRING);
59 59
         static::handleJsonError(json_last_error());
60 60
 
61 61
         return $decode;
Please login to merge, or discard this patch.
src/helpers/MetaValue.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
                 $metaValue = Craft::t(
295 295
                     'seomatic',
296 296
                     'Error rendering `{template}` -> {error}',
297
-                    ['template' => $metaValue, 'error' => $e->getMessage() . ' - ' . print_r($metaValue, true)]
297
+                    ['template' => $metaValue, 'error' => $e->getMessage().' - '.print_r($metaValue, true)]
298 298
                 );
299 299
                 Craft::error($metaValue, __METHOD__);
300 300
                 Craft::$app->getErrorHandler()->logException($e);
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
         // Handle being passed in an object
314 314
         if (is_object($metaValue)) {
315 315
             if ($metaValue instanceof Markup) {
316
-                return trim(html_entity_decode((string)$metaValue, ENT_NOQUOTES, 'UTF-8'));
316
+                return trim(html_entity_decode((string) $metaValue, ENT_NOQUOTES, 'UTF-8'));
317 317
             }
318 318
             if ($metaValue instanceof Asset) {
319 319
                 return $metaValue->uri;
Please login to merge, or discard this patch.
src/helpers/Text.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
         if (!empty($string)) {
74 74
             $string = HtmlPurifier::process($string, ['HTML.Allowed' => '']);
75 75
             $string = html_entity_decode($string, ENT_NOQUOTES, 'UTF-8');
76
-            $result = (string)Stringy::create($string)->truncate($length, $substring);
76
+            $result = (string) Stringy::create($string)->truncate($length, $substring);
77 77
         }
78 78
 
79 79
         return $result;
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
         if (!empty($string)) {
99 99
             $string = HtmlPurifier::process($string, ['HTML.Allowed' => '']);
100 100
             $string = html_entity_decode($string, ENT_NOQUOTES, 'UTF-8');
101
-            $result = (string)Stringy::create($string)->safeTruncate($length, $substring);
101
+            $result = (string) Stringy::create($string)->safeTruncate($length, $substring);
102 102
         }
103 103
 
104 104
         return $result;
@@ -132,9 +132,9 @@  discard block
 block discarded – undo
132 132
             $result = self::smartStripTags(Doxter::$plugin->getService()->parseMarkdown($field->getRaw()));
133 133
         } else {
134 134
             if (is_array($field)) {
135
-                $result = self::smartStripTags((string)$field[0]);
135
+                $result = self::smartStripTags((string) $field[0]);
136 136
             } else {
137
-                $result = self::smartStripTags((string)$field);
137
+                $result = self::smartStripTags((string) $field);
138 138
             }
139 139
         }
140 140
 
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
             $tags = $tags->all();
162 162
         }
163 163
         foreach ($tags as $tag) {
164
-            $result .= $tag->title . ', ';
164
+            $result .= $tag->title.', ';
165 165
         }
166 166
         $result = rtrim($result, ', ');
167 167
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
                     foreach ($fieldClasses as $fieldClassKey) {
204 204
                         if ($field instanceof $fieldClassKey) {
205 205
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
206
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
206
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
207 207
                             }
208 208
                         }
209 209
                     }
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
                     foreach ($fieldClasses as $fieldClassKey) {
246 246
                         if ($field instanceof $fieldClassKey) {
247 247
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
248
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
248
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
249 249
                             }
250 250
                         }
251 251
                     }
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
                     foreach ($fieldClasses as $fieldClassKey) {
295 295
                         if ($field instanceof $fieldClassKey) {
296 296
                             if ($field->handle === $fieldHandle || empty($fieldHandle)) {
297
-                                $result .= self::extractTextFromField($block[$field->handle]) . ' ';
297
+                                $result .= self::extractTextFromField($block[$field->handle]).' ';
298 298
                             }
299 299
                         }
300 300
                     }
@@ -393,7 +393,7 @@  discard block
 block discarded – undo
393 393
         $str = html_entity_decode($str, ENT_NOQUOTES, 'UTF-8');
394 394
         $str = rawurldecode($str);
395 395
         // Remove any linebreaks
396
-        $str = (string)preg_replace("/\r|\n/", "", $str);
396
+        $str = (string) preg_replace("/\r|\n/", "", $str);
397 397
         $str = HtmlPurifier::process($str, ['HTML.Allowed' => '']);
398 398
         $str = html_entity_decode($str, ENT_NOQUOTES, 'UTF-8');
399 399
         // Remove any embedded Twig code
@@ -473,7 +473,7 @@  discard block
 block discarded – undo
473 473
             $language = 'English';
474 474
         }
475 475
 
476
-        $className = 'PhpScience\\TextRank\\Tool\\StopWords\\' . ucfirst($language);
476
+        $className = 'PhpScience\\TextRank\\Tool\\StopWords\\'.ucfirst($language);
477 477
         if (class_exists($className)) {
478 478
             $stopWords = new $className();
479 479
         }
Please login to merge, or discard this patch.
src/debug/views/seomatic/render-tag.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,8 +75,11 @@
 block discarded – undo
75 75
                     </ul><?php endif; ?><?php endforeach; ?><?php endif; ?></div>
76 76
         </details>
77 77
     </td>
78
-<?php else: ?>
78
+<?php else {
79
+    : ?>
79 80
     <td>
80 81
         <div class="callout callout-secondary seomatic-callout">not included</div>
81 82
     </td>
82
-<?php endif; ?>
83
+<?php endif;
84
+}
85
+?>
Please login to merge, or discard this patch.
src/debug/views/seomatic/render-value.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,7 +45,10 @@
 block discarded – undo
45 45
             </div>
46 46
         </details>
47 47
     </td>
48
-<?php else: ?>
48
+<?php else {
49
+    : ?>
49 50
     <td><?= htmlspecialchars(VarDumper::dumpAsString($value), ENT_QUOTES | ENT_SUBSTITUTE,
50 51
             Yii::$app->charset, true) ?></td>
51
-<?php endif; ?>
52
+<?php endif;
53
+}
54
+?>
Please login to merge, or discard this patch.
src/services/MetaBundles.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
      * @param MetaBundle $metaBundle
194 194
      * @param bool $forceUpdate
195 195
      */
196
-    public function syncBundleWithConfig(MetaBundle &$metaBundle, bool $forceUpdate = false)
196
+    public function syncBundleWithConfig(MetaBundle & $metaBundle, bool $forceUpdate = false)
197 197
     {
198 198
         $prevMetaBundle = $metaBundle;
199 199
         $config = [];
@@ -274,8 +274,8 @@  discard block
 block discarded – undo
274 274
      */
275 275
     public function updateMetaBundle(MetaBundle $metaBundle, int $siteId)
276 276
     {
277
-        $metaBundle->sourceName = (string)$metaBundle->sourceName;
278
-        $metaBundle->sourceTemplate = (string)$metaBundle->sourceTemplate;
277
+        $metaBundle->sourceName = (string) $metaBundle->sourceName;
278
+        $metaBundle->sourceTemplate = (string) $metaBundle->sourceTemplate;
279 279
         // Make sure it validates
280 280
         if ($metaBundle->validate(null, true)) {
281 281
             // Save it out to a record
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
                 'sourceSiteId' => $siteId,
286 286
             ];
287 287
             if ($metaBundle->typeId !== null) {
288
-                $metaBundle->typeId = (int)$metaBundle->typeId;
288
+                $metaBundle->typeId = (int) $metaBundle->typeId;
289 289
             }
290 290
             if (!empty($metaBundle->typeId)) {
291 291
                 $params['typeId'] = $metaBundle->typeId;
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
             // The issue was that the containers were getting saved to the db with a hard-coded setting in them, because they'd
303 303
             // been set that way by the environment, whereas to be changeable via the GUI, it needs to be set to {seomatic.meta.robots}
304 304
             /** @var RobotsTag|null $robotsTag */
305
-            $robotsTag = $metaBundle->metaContainers[MetaTagContainer::CONTAINER_TYPE . TagService::GENERAL_HANDLE]->data['robots'] ?? null;
305
+            $robotsTag = $metaBundle->metaContainers[MetaTagContainer::CONTAINER_TYPE.TagService::GENERAL_HANDLE]->data['robots'] ?? null;
306 306
             if (!empty($robotsTag)) {
307 307
                 $robotsTag->content = $robotsTag->environment['live']['content'] ?? '{seomatic.meta.robots}';
308 308
             }
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
                 $metaBundleDefaults = ArrayHelper::merge(
381 381
                     $seoElement::metaBundleConfig($sourceModel),
382 382
                     [
383
-                        'sourceTemplate' => (string)$siteSetting->template,
383
+                        'sourceTemplate' => (string) $siteSetting->template,
384 384
                         'sourceSiteId' => $siteSetting->siteId,
385 385
                         'sourceAltSiteSettings' => $siteSettingsArray,
386 386
                         'sourceDateUpdated' => $dateUpdated,
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
     public function getMetaBundleBySourceHandle(string $sourceBundleType, string $sourceHandle, int $sourceSiteId, $typeId = null)
429 429
     {
430 430
         $metaBundle = null;
431
-        $typeId = (int)$typeId;
431
+        $typeId = (int) $typeId;
432 432
         // See if we have the meta bundle cached
433 433
         if (!empty($this->metaBundlesBySourceHandle[$sourceBundleType][$sourceHandle][$sourceSiteId][$typeId])) {
434 434
             $id = $this->metaBundlesBySourceHandle[$sourceBundleType][$sourceHandle][$sourceSiteId][$typeId];
@@ -515,7 +515,7 @@  discard block
 block discarded – undo
515 515
                             /** @var Section|CategoryGroup|ProductType $sourceModel */
516 516
                             $sourceModel = $seoElement::sourceModelFromId($sourceId);
517 517
                             if ($sourceModel !== null) {
518
-                                $metaBundle->sourceName = (string)$sourceModel->name;
518
+                                $metaBundle->sourceName = (string) $sourceModel->name;
519 519
                                 $metaBundle->sourceHandle = $sourceModel->handle;
520 520
                             }
521 521
                         }
@@ -556,7 +556,7 @@  discard block
 block discarded – undo
556 556
     public function getMetaBundleBySourceId(string $sourceBundleType, int $sourceId, ?int $sourceSiteId, $typeId = null)
557 557
     {
558 558
         $metaBundle = null;
559
-        $typeId = (int)$typeId;
559
+        $typeId = (int) $typeId;
560 560
         // See if we have the meta bundle cached
561 561
         if (!empty($this->metaBundlesBySourceId[$sourceBundleType][$sourceId][$sourceSiteId][$typeId])) {
562 562
             $id = $this->metaBundlesBySourceId[$sourceBundleType][$sourceId][$sourceSiteId][$typeId];
@@ -750,7 +750,7 @@  discard block
 block discarded – undo
750 750
                 ->from(['{{%seomatic_metabundles}}'])
751 751
                 ->where(['=', 'sourceBundleType', $seoElement::META_BUNDLE_TYPE]);
752 752
 
753
-            if ((int)$sourceSiteId !== 0) {
753
+            if ((int) $sourceSiteId !== 0) {
754 754
                 $subQuery->andWhere(['sourceSiteId' => $sourceSiteId]);
755 755
             }
756 756
             if ($filter !== '') {
@@ -787,7 +787,7 @@  discard block
 block discarded – undo
787 787
     public function getContentMetaBundleForElement(Element $element)
788 788
     {
789 789
         $source = $this->getMetaSourceFromElement($element);
790
-        $key = implode(".", $source) . '.' . $element->siteId;
790
+        $key = implode(".", $source).'.'.$element->siteId;
791 791
 
792 792
         if (empty($this->elementContentMetaBundles[$key])) {
793 793
             $this->elementContentMetaBundles[$key] = $this->getMetaBundleBySourceId($source[1], $source[0], $element->siteId, $source[4]);
Please login to merge, or discard this patch.
src/seoelements/SeoEntry.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
             ->limit($metaBundle->metaSitemapVars->sitemapLimit);
223 223
         if ($metaBundle->sourceType === 'structure'
224 224
             && !empty($metaBundle->metaSitemapVars->structureDepth)) {
225
-            $query->level($metaBundle->metaSitemapVars->structureDepth . '<=');
225
+            $query->level($metaBundle->metaSitemapVars->structureDepth.'<=');
226 226
         }
227 227
 
228 228
         return $query;
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
             ConfigHelper::getConfigFromFile(self::configFilePath()),
390 390
             [
391 391
                 'sourceId' => $sourceModel->id,
392
-                'sourceName' => (string)$sourceModel->name,
392
+                'sourceName' => (string) $sourceModel->name,
393 393
                 'sourceHandle' => $sourceModel->handle,
394 394
                 'sourceType' => $sourceModel->type,
395 395
             ]
Please login to merge, or discard this patch.