Passed
Push — develop ( 03731b...fd148c )
by Andrew
16:09
created
src/services/Link.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
      */
76 76
     public function add($metaItem, string $handle = self::GENERAL_HANDLE)
77 77
     {
78
-        $key = MetaLinkContainer::CONTAINER_TYPE . $handle;
78
+        $key = MetaLinkContainer::CONTAINER_TYPE.$handle;
79 79
         Seomatic::$plugin->metaContainers->addToMetaContainer($metaItem, $key);
80 80
 
81 81
         /** @var MetaLink $metaItem */
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
      */
100 100
     public function container(string $handle = self::GENERAL_HANDLE)
101 101
     {
102
-        $key = MetaLinkContainer::CONTAINER_TYPE . $handle;
102
+        $key = MetaLinkContainer::CONTAINER_TYPE.$handle;
103 103
 
104 104
         return Seomatic::$plugin->metaContainers->getMetaContainer($key);
105 105
     }
Please login to merge, or discard this patch.
src/services/Sitemaps.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -42,11 +42,11 @@  discard block
 block discarded – undo
42 42
     // Constants
43 43
     // =========================================================================
44 44
 
45
-    const SEOMATIC_SITEMAPINDEX_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapIndexTemplate::TEMPLATE_TYPE;
45
+    const SEOMATIC_SITEMAPINDEX_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapIndexTemplate::TEMPLATE_TYPE;
46 46
 
47
-    const SEOMATIC_SITEMAP_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapTemplate::TEMPLATE_TYPE;
47
+    const SEOMATIC_SITEMAP_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapTemplate::TEMPLATE_TYPE;
48 48
 
49
-    const SEOMATIC_SITEMAPCUSTOM_CONTAINER = Seomatic::SEOMATIC_HANDLE . SitemapCustomTemplate::TEMPLATE_TYPE;
49
+    const SEOMATIC_SITEMAPCUSTOM_CONTAINER = Seomatic::SEOMATIC_HANDLE.SitemapCustomTemplate::TEMPLATE_TYPE;
50 50
 
51 51
     const SEARCH_ENGINE_SUBMISSION_URLS = [
52 52
         'google' => 'https://www.google.com/ping?sitemap=',
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
             Event::on(
92 92
                 UrlManager::class,
93 93
                 UrlManager::EVENT_REGISTER_SITE_URL_RULES,
94
-                function (RegisterUrlRulesEvent $event) {
94
+                function(RegisterUrlRulesEvent $event) {
95 95
                     Craft::debug(
96 96
                         'UrlManager::EVENT_REGISTER_SITE_URL_RULES',
97 97
                         __METHOD__
@@ -244,19 +244,19 @@  discard block
 block discarded – undo
244 244
                         $siteId = $groupSiteIds[0];
245 245
                         $sitemapIndexUrl = $this->sitemapIndexUrlForSiteId($siteId);
246 246
                         if (!empty($sitemapIndexUrl)) {
247
-                            $submissionUrl = $url . urlencode($sitemapIndexUrl);
247
+                            $submissionUrl = $url.urlencode($sitemapIndexUrl);
248 248
                             // create new guzzle client
249 249
                             $guzzleClient = Craft::createGuzzleClient(['timeout' => 120, 'connect_timeout' => 120]);
250 250
                             // Submit the sitemap index to each search engine
251 251
                             try {
252 252
                                 $guzzleClient->post($submissionUrl);
253 253
                                 Craft::info(
254
-                                    'Sitemap index submitted to: ' . $submissionUrl,
254
+                                    'Sitemap index submitted to: '.$submissionUrl,
255 255
                                     __METHOD__
256 256
                                 );
257 257
                             } catch (\Exception $e) {
258 258
                                 Craft::error(
259
-                                    'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
259
+                                    'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
260 260
                                     __METHOD__
261 261
                                 );
262 262
                             }
@@ -316,19 +316,19 @@  discard block
 block discarded – undo
316 316
             foreach ($searchEngineUrls as &$url) {
317 317
                 $sitemapUrl = $this->sitemapUrlForBundle($sourceBundleType, $sourceHandle, $sourceSiteId);
318 318
                 if (!empty($sitemapUrl)) {
319
-                    $submissionUrl = $url . urlencode($sitemapUrl);
319
+                    $submissionUrl = $url.urlencode($sitemapUrl);
320 320
                     // create new guzzle client
321 321
                     $guzzleClient = Craft::createGuzzleClient(['timeout' => 120, 'connect_timeout' => 120]);
322 322
                     // Submit the sitemap index to each search engine
323 323
                     try {
324 324
                         $guzzleClient->post($submissionUrl);
325 325
                         Craft::info(
326
-                            'Sitemap index submitted to: ' . $submissionUrl,
326
+                            'Sitemap index submitted to: '.$submissionUrl,
327 327
                             __METHOD__
328 328
                         );
329 329
                     } catch (\Exception $e) {
330 330
                         Craft::error(
331
-                            'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
331
+                            'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
332 332
                             __METHOD__
333 333
                         );
334 334
                     }
@@ -394,19 +394,19 @@  discard block
 block discarded – undo
394 394
             foreach ($searchEngineUrls as &$url) {
395 395
                 $sitemapUrl = $this->sitemapCustomUrlForSiteId($siteId);
396 396
                 if (!empty($sitemapUrl)) {
397
-                    $submissionUrl = $url . urlencode($sitemapUrl);
397
+                    $submissionUrl = $url.urlencode($sitemapUrl);
398 398
                     // create new guzzle client
399 399
                     $guzzleClient = Craft::createGuzzleClient(['timeout' => 120, 'connect_timeout' => 120]);
400 400
                     // Submit the sitemap index to each search engine
401 401
                     try {
402 402
                         $guzzleClient->post($submissionUrl);
403 403
                         Craft::info(
404
-                            'Sitemap Custom submitted to: ' . $submissionUrl,
404
+                            'Sitemap Custom submitted to: '.$submissionUrl,
405 405
                             __METHOD__
406 406
                         );
407 407
                     } catch (\Exception $e) {
408 408
                         Craft::error(
409
-                            'Error submitting sitemap index to: ' . $submissionUrl . ' - ' . $e->getMessage(),
409
+                            'Error submitting sitemap index to: '.$submissionUrl.' - '.$e->getMessage(),
410 410
                             __METHOD__
411 411
                         );
412 412
                     }
@@ -484,7 +484,7 @@  discard block
 block discarded – undo
484 484
         }
485 485
 
486 486
         foreach ($sites as $site) {
487
-            $result .= 'sitemap: ' . $this->sitemapIndexUrlForSiteId($site->id) . PHP_EOL;
487
+            $result .= 'sitemap: '.$this->sitemapIndexUrlForSiteId($site->id).PHP_EOL;
488 488
         }
489 489
 
490 490
         return rtrim($result, PHP_EOL);
@@ -516,9 +516,9 @@  discard block
 block discarded – undo
516 516
         // Since we want a stale-while-revalidate pattern, only invalidate the cache if we're asked to
517 517
         if ($invalidateCache) {
518 518
             $cache = Craft::$app->getCache();
519
-            TagDependency::invalidate($cache, SitemapTemplate::SITEMAP_CACHE_TAG . $handle . $siteId);
519
+            TagDependency::invalidate($cache, SitemapTemplate::SITEMAP_CACHE_TAG.$handle.$siteId);
520 520
             Craft::info(
521
-                'Sitemap cache cleared: ' . $handle,
521
+                'Sitemap cache cleared: '.$handle,
522 522
                 __METHOD__
523 523
             );
524 524
         }
Please login to merge, or discard this patch.
src/seoelements/SeoProduct.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         Event::on(
105 105
             ProductTypes::class,
106 106
             ProductTypes::EVENT_AFTER_SAVE_PRODUCTTYPE,
107
-            function (ProductTypeEvent $event) {
107
+            function(ProductTypeEvent $event) {
108 108
                 Craft::debug(
109 109
                     'ProductTypes::EVENT_AFTER_SAVE_PRODUCTTYPE',
110 110
                     __METHOD__
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
             Event::on(
120 120
                 ProductTypes::class,
121 121
                 ProductTypes::EVENT_AFTER_SAVE_PRODUCTTYPE,
122
-                static function (ProductTypeEvent $event) {
122
+                static function(ProductTypeEvent $event) {
123 123
                     Craft::debug(
124 124
                         'ProductTypes::EVENT_AFTER_SAVE_PRODUCTTYPE',
125 125
                         __METHOD__
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
             // Commerce Product Types sidebar
176 176
             $commerce = CommercePlugin::getInstance();
177 177
             if ($commerce !== null) {
178
-                Seomatic::$view->hook('cp.commerce.product.edit.details', static function (&$context) {
178
+                Seomatic::$view->hook('cp.commerce.product.edit.details', static function(&$context) {
179 179
                     $html = '';
180 180
                     Seomatic::$view->registerAssetBundle(SeomaticAsset::class);
181 181
                     /** @var  $product Product */
@@ -382,7 +382,7 @@  discard block
 block discarded – undo
382 382
             ConfigHelper::getConfigFromFile(self::configFilePath()),
383 383
             [
384 384
                 'sourceId' => $sourceModel->id,
385
-                'sourceName' => (string)$sourceModel->name,
385
+                'sourceName' => (string) $sourceModel->name,
386 386
                 'sourceHandle' => $sourceModel->handle,
387 387
             ]
388 388
         );
Please login to merge, or discard this patch.
src/console/controllers/SitemapController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
      */
75 75
     public function actionGenerate()
76 76
     {
77
-        echo 'Generating sitemap' . PHP_EOL;
77
+        echo 'Generating sitemap'.PHP_EOL;
78 78
         if ($this->siteId !== null) {
79 79
             $siteIds[] = $this->siteId;
80 80
         } else {
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
                         ]);
117 117
                     }
118 118
 
119
-                    echo '---' . PHP_EOL;
119
+                    echo '---'.PHP_EOL;
120 120
                 }
121 121
             }
122 122
         }
Please login to merge, or discard this patch.
src/models/SitemapTemplate.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -148,9 +148,9 @@  discard block
 block discarded – undo
148 148
         }
149 149
 
150 150
         $cache = Craft::$app->getCache();
151
-        $uniqueKey = $groupId . $type . $handle . $siteId;
152
-        $cacheKey = self::CACHE_KEY . $uniqueKey;
153
-        $queueJobCacheKey = self::QUEUE_JOB_CACHE_KEY . $uniqueKey;
151
+        $uniqueKey = $groupId.$type.$handle.$siteId;
152
+        $cacheKey = self::CACHE_KEY.$uniqueKey;
153
+        $queueJobCacheKey = self::QUEUE_JOB_CACHE_KEY.$uniqueKey;
154 154
         $result = $cache->get($cacheKey);
155 155
         // If the sitemap isn't cached, start a job to create it
156 156
         // Even if it is cached, if $throwException === false we should regenerate it, as it is part of
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
                 $dependency = new TagDependency([
190 190
                     'tags' => [
191 191
                         self::GLOBAL_SITEMAP_CACHE_TAG,
192
-                        self::CACHE_KEY . $uniqueKey,
192
+                        self::CACHE_KEY.$uniqueKey,
193 193
                     ],
194 194
                 ]);
195 195
                 $cache->set($queueJobCacheKey, $jobId, $cacheDuration, $dependency);
@@ -221,11 +221,11 @@  discard block
 block discarded – undo
221 221
                 // Return an empty XML document
222 222
                 $lines[] = '<?xml version="1.0" encoding="UTF-8"?>';
223 223
                 $lines[] = '<?xml-stylesheet type="text/xsl" href="sitemap-empty.xsl"?>';
224
-                $lines[] = '<!-- ' . Craft::t('seomatic', 'This sitemap has not been generated yet.') . ' -->';
225
-                $lines[] = '<!-- ' . Craft::t('seomatic', 'If you are seeing this in local dev or an') . ' -->';
226
-                $lines[] = '<!-- ' . Craft::t('seomatic', 'environment with `devMode` on, caches only') . ' -->';
227
-                $lines[] = '<!-- ' . Craft::t('seomatic', 'last for 30 seconds in local dev, so it is') . ' -->';
228
-                $lines[] = '<!-- ' . Craft::t('seomatic', 'normal for the sitemap to not be cached.') . ' -->';
224
+                $lines[] = '<!-- '.Craft::t('seomatic', 'This sitemap has not been generated yet.').' -->';
225
+                $lines[] = '<!-- '.Craft::t('seomatic', 'If you are seeing this in local dev or an').' -->';
226
+                $lines[] = '<!-- '.Craft::t('seomatic', 'environment with `devMode` on, caches only').' -->';
227
+                $lines[] = '<!-- '.Craft::t('seomatic', 'last for 30 seconds in local dev, so it is').' -->';
228
+                $lines[] = '<!-- '.Craft::t('seomatic', 'normal for the sitemap to not be cached.').' -->';
229 229
                 $lines[] = '<urlset>';
230 230
                 $lines[] = '</urlset>';
231 231
             }
@@ -246,9 +246,9 @@  discard block
 block discarded – undo
246 246
     public function invalidateCache(string $handle, int $siteId)
247 247
     {
248 248
         $cache = Craft::$app->getCache();
249
-        TagDependency::invalidate($cache, self::SITEMAP_CACHE_TAG . $handle . $siteId);
249
+        TagDependency::invalidate($cache, self::SITEMAP_CACHE_TAG.$handle.$siteId);
250 250
         Craft::info(
251
-            'Sitemap cache cleared: ' . $handle,
251
+            'Sitemap cache cleared: '.$handle,
252 252
             __METHOD__
253 253
         );
254 254
     }
Please login to merge, or discard this patch.
src/models/metalink/CanonicalLink.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@
 block discarded – undo
100 100
                 // Craft adds the `tokenParam` URL param back in via UrlHelper, strip it again
101 101
                 if (Seomatic::$plugin->metaContainers->paginationPage === '1') {
102 102
                     $token = Craft::$app->getConfig()->getGeneral()->tokenParam;
103
-                    $url = preg_replace('~([?&])' . $token . '=[^&]*~', '$1', $url);
103
+                    $url = preg_replace('~([?&])'.$token.'=[^&]*~', '$1', $url);
104 104
                     $url = rtrim($url, '?&');
105 105
                     $url = str_replace(['&&', '?&'], ['&', '?'], $url);
106 106
                     $data['href'] = $url;
Please login to merge, or discard this patch.
src/models/MetaScript.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -265,7 +265,7 @@
 block discarded – undo
265 265
         }
266 266
 
267 267
         if (empty($html) && !empty($this->templatePath)) {
268
-            $html = '/* ' . $this->name . Craft::t('seomatic', ' script did not render') . ' */' . PHP_EOL;
268
+            $html = '/* '.$this->name.Craft::t('seomatic', ' script did not render').' */'.PHP_EOL;
269 269
         }
270 270
 
271 271
         return $html;
Please login to merge, or discard this patch.
src/seomatic-config/categorymeta/TitleContainer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,11 +19,11 @@
 block discarded – undo
19 19
  */
20 20
 
21 21
 return [
22
-    MetaTitleContainer::CONTAINER_TYPE . TitleService::GENERAL_HANDLE => [
22
+    MetaTitleContainer::CONTAINER_TYPE.TitleService::GENERAL_HANDLE => [
23 23
         'name' => 'General',
24 24
         'description' => 'Meta Title Tag',
25 25
         'handle' => TitleService::GENERAL_HANDLE,
26
-        'class' => (string)MetaTitleContainer::class,
26
+        'class' => (string) MetaTitleContainer::class,
27 27
         'include' => true,
28 28
         'dependencies' => [
29 29
         ],
Please login to merge, or discard this patch.
src/seomatic-config/categorymeta/JsonLdContainer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,11 +19,11 @@
 block discarded – undo
19 19
  */
20 20
 
21 21
 return [
22
-    MetaJsonLdContainer::CONTAINER_TYPE . JsonLdService::GENERAL_HANDLE => [
22
+    MetaJsonLdContainer::CONTAINER_TYPE.JsonLdService::GENERAL_HANDLE => [
23 23
         'name' => 'General',
24 24
         'description' => 'JsonLd Tags',
25 25
         'handle' => JsonLdService::GENERAL_HANDLE,
26
-        'class' => (string)MetaJsonLdContainer::class,
26
+        'class' => (string) MetaJsonLdContainer::class,
27 27
         'include' => true,
28 28
         'dependencies' => [
29 29
         ],
Please login to merge, or discard this patch.