Test Failed
Push — master ( c1ba7a...cb4298 )
by Russell
04:09
created
src/Task/StaticSiteRewriteLinksTask.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 
171 171
         $task = $this;
172 172
         // Callback for URL rewriter, called from StaticSiteLinkRewriter and passed through $callback($url)
173
-        $rewriter = StaticSiteLinkRewriter::create(function ($url) use ($pageLookup, $fileLookup, $task) {
173
+        $rewriter = StaticSiteLinkRewriter::create(function($url) use ($pageLookup, $fileLookup, $task) {
174 174
             $origUrl = $url;
175 175
             $anchor = '';
176 176
 
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
             if ($siteTreeObject = $pageLookup->find('StaticSiteURL', $pageMapKey)) {
209 209
                 $output = '[sitetree_link,id=' . $siteTreeObject->ID . ']';
210 210
                 $task->printMessage("\tFound: SiteTree ID#" . $siteTreeObject->ID, null, $output);
211
-                $anchorPattern = "<[\w]+\s+(name|id)=('|\")?". $anchor ."('|\")?";
211
+                $anchorPattern = "<[\w]+\s+(name|id)=('|\")?" . $anchor . "('|\")?";
212 212
 
213 213
                 if (strlen($anchor) && preg_match("#$anchorPattern#mi", $siteTreeObject->Content)) {
214 214
                     $output = "#$anchor";
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
         $this->printMessage("Amended $changedFields content fields for {$pages->count()} pages and {$files->count()} files processed.");
290 290
 
291 291
         $msgNextSteps = " - Not all links will get fixed. It's recommended to also run a 3rd party link-checker over your imported content.";
292
-        $msgSeeReport = " - Check the CMS \"".singleton(FailedURLRewriteReport::class)->title()."\" report for a summary of failed link-rewrites.";
292
+        $msgSeeReport = " - Check the CMS \"" . singleton(FailedURLRewriteReport::class)->title() . "\" report for a summary of failed link-rewrites.";
293 293
 
294 294
         $this->printMessage("Tips:");
295 295
         $this->printMessage("{$newLine}$msgNextSteps");
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
             return;
314 314
         }
315 315
         $url = ($url ? '(' . $url . ') ' : '');
316
-        $level = ($level ? '[' . $level .'] ' : '');
316
+        $level = ($level ? '[' . $level . '] ' : '');
317 317
         if (Director::is_cli()) {
318 318
             echo "$level$message$url" . PHP_EOL;
319 319
         } else {
@@ -346,7 +346,7 @@  discard block
 block discarded – undo
346 346
                 'ContainedInID' => $failure['ContainedInID']
347 347
             ));
348 348
 
349
-            if ($failureExists->count() >0) {
349
+            if ($failureExists->count() > 0) {
350 350
                 continue;
351 351
             }
352 352
 
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
     public function linkIsThirdParty($link)
423 423
     {
424 424
         $link = trim($link);
425
-        return (bool)preg_match("#^http(s)?://#", $link);
425
+        return (bool) preg_match("#^http(s)?://#", $link);
426 426
     }
427 427
 
428 428
     /**
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
         $nonHTTPSchemes = implode('|', self::$non_http_uri_schemes);
437 437
         $badScheme = preg_match("#^($nonHTTPSchemes):#", $link);
438 438
         $alreadyImported = $this->linkIsAlreadyRewritten($link);
439
-        return (bool)($badScheme || $alreadyImported);
439
+        return (bool) ($badScheme || $alreadyImported);
440 440
     }
441 441
 
442 442
     /**
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
447 447
      */
448 448
     public function linkIsNotImported($link)
449 449
     {
450
-        return (bool)(stristr($link, 'sitetree') === false && stristr($link, 'assets') === false);
450
+        return (bool) (stristr($link, 'sitetree') === false && stristr($link, 'assets') === false);
451 451
     }
452 452
 
453 453
     /**
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
      */
459 459
     public function linkIsAlreadyRewritten($link)
460 460
     {
461
-        return (bool)(stristr($link, 'sitetree') !== false || stristr($link, 'assets') !== false);
461
+        return (bool) (stristr($link, 'sitetree') !== false || stristr($link, 'assets') !== false);
462 462
     }
463 463
 
464 464
     /**
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
      */
470 470
     public function linkIsJunk($link)
471 471
     {
472
-        return (bool)preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
472
+        return (bool) preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
473 473
     }
474 474
 
475 475
     /**
@@ -592,13 +592,13 @@  discard block
 block discarded – undo
592 592
 
593 593
         // List the content sources to prompt user for selection
594 594
         if ($contentSources = StaticSiteContentSource::get()) {
595
-            $this->printMessage($newLine.'Available content-sources:'.$newLine);
595
+            $this->printMessage($newLine . 'Available content-sources:' . $newLine);
596 596
             foreach ($contentSources as $i => $contentSource) {
597 597
                 $this->printMessage("\tdev/tasks/" . __CLASS__ . ' SourceID=' . $contentSource->ID . ' ImportID=<number>');
598 598
             }
599 599
             echo $newLine;
600 600
             if (Director::is_cli()) {
601
-                $this->printMessage('Available command line options: '.$newLine);
601
+                $this->printMessage('Available command line options: ' . $newLine);
602 602
                 $this->printMessage("\tSourceID=<number> \t\tThe ID of the original crawl.");
603 603
                 $this->printMessage("\tImportID=<number> \t\tThe ID of the import to use.");
604 604
                 $this->printMessage("\tSHOW=pages \tPrint the contents of the pages map.");
Please login to merge, or discard this patch.
src/Model/StaticSiteContentSource.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -217,7 +217,7 @@
 block discarded – undo
217 217
                 $crawlButtonText = _t('StaticSiteContentSource.RECRAWL_SITE', 'Re-Crawl');
218 218
                 break;
219 219
             default:
220
-                throw new \LogicException("Invalid getSpiderStatus() value '".$this->urlList()->getSpiderStatus().";");
220
+                throw new \LogicException("Invalid getSpiderStatus() value '" . $this->urlList()->getSpiderStatus() . ";");
221 221
         }
222 222
 
223 223
         $crawlButton = FormAction::create('crawlsite', $crawlButtonText)
Please login to merge, or discard this patch.