phptek / silverstripe-exodus

src/Tool/StaticSiteUtils.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 
         if ($logFile && is_writable($logFile)) {
             $message = $message . ($filename ? ' ' . $filename : '') . ($mime ? ' (' . $mime . ')' : '');
-            error_log($message. PHP_EOL, 3, $logFile);
+            error_log($message . PHP_EOL, 3, $logFile);
         }
     }
 
@@ -43,12 +43,12 @@ 
      * @param StaticSiteCrawler $crawler (Warning: Pass by reference)
      * @return array Returns an array of the config options in a format consumable by curl.
      */
-    public function defineProxyOpts(bool $set, StaticSiteCrawler &$crawler = null): array
+    public function defineProxyOpts(bool $set, StaticSiteCrawler & $crawler = null): array
     {
         if ($set && is_bool($set) && $set !== false) {
             $proxyOpts = StaticSiteContentExtractor::config()->get('curl_opts_proxy');
 
-            if (!$proxyOpts || !is_array($proxyOpts) || !count($proxyOpts) >0) {
+            if (!$proxyOpts || !is_array($proxyOpts) || !count($proxyOpts) > 0) {
                 return [];
             }
 

src/Task/StaticSiteRewriteLinksTask.php

Spacing +12 added lines, -12 removed lines

@@ -170,7 +170,7 @@ 
 
         $task = $this;
         // Callback for URL rewriter, called from StaticSiteLinkRewriter and passed through $callback($url)
-        $rewriter = StaticSiteLinkRewriter::create(function ($url) use ($pageLookup, $fileLookup, $task) {
+        $rewriter = StaticSiteLinkRewriter::create(function($url) use ($pageLookup, $fileLookup, $task) {
             $origUrl = $url;
             $anchor = '';
 
@@ -208,7 +208,7 @@ 
             if ($siteTreeObject = $pageLookup->find('StaticSiteURL', $pageMapKey)) {
                 $output = '[sitetree_link,id=' . $siteTreeObject->ID . ']';
                 $task->printMessage("\tFound: SiteTree ID#" . $siteTreeObject->ID, null, $output);
-                $anchorPattern = "<[\w]+\s+(name|id)=('|\")?". $anchor ."('|\")?";
+                $anchorPattern = "<[\w]+\s+(name|id)=('|\")?" . $anchor . "('|\")?";
 
                 if (strlen($anchor) && preg_match("#$anchorPattern#mi", $siteTreeObject->Content)) {
                     $output = "#$anchor";
@@ -289,7 +289,7 @@ 
         $this->printMessage("Amended $changedFields content fields for {$pages->count()} pages and {$files->count()} files processed.");
 
         $msgNextSteps = " - Not all links will get fixed. It's recommended to also run a 3rd party link-checker over your imported content.";
-        $msgSeeReport = " - Check the CMS \"".singleton(FailedURLRewriteReport::class)->title()."\" report for a summary of failed link-rewrites.";
+        $msgSeeReport = " - Check the CMS \"" . singleton(FailedURLRewriteReport::class)->title() . "\" report for a summary of failed link-rewrites.";
 
         $this->printMessage("Tips:");
         $this->printMessage("{$newLine}$msgNextSteps");
@@ -315,7 +315,7 @@ 
         }
 
         $url = ($url ? '(' . $url . ') ' : '');
-        $level = ($level ? '[' . $level .'] ' : '');
+        $level = ($level ? '[' . $level . '] ' : '');
 
         echo "$level$message$url" . PHP_EOL;
     }
@@ -345,7 +345,7 @@ 
                 'ContainedInID' => $failure['ContainedInID']
             ));
 
-            if ($failureExists->count() >0) {
+            if ($failureExists->count() > 0) {
                 continue;
             }
 
@@ -421,7 +421,7 @@ 
     public function linkIsThirdParty($link)
     {
         $link = trim($link);
-        return (bool)preg_match("#^http(s)?://#", $link);
+        return (bool) preg_match("#^http(s)?://#", $link);
     }
 
     /**
@@ -435,7 +435,7 @@ 
         $nonHTTPSchemes = implode('|', self::$non_http_uri_schemes);
         $badScheme = preg_match("#^($nonHTTPSchemes):#", $link);
         $alreadyImported = $this->linkIsAlreadyRewritten($link);
-        return (bool)($badScheme || $alreadyImported);
+        return (bool) ($badScheme || $alreadyImported);
     }
 
     /**
@@ -446,7 +446,7 @@ 
      */
     public function linkIsNotImported($link)
     {
-        return (bool)(stristr($link, 'sitetree') === false && stristr($link, 'assets') === false);
+        return (bool) (stristr($link, 'sitetree') === false && stristr($link, 'assets') === false);
     }
 
     /**
@@ -457,7 +457,7 @@ 
      */
     public function linkIsAlreadyRewritten($link)
     {
-        return (bool)(stristr($link, 'sitetree') !== false || stristr($link, 'assets') !== false);
+        return (bool) (stristr($link, 'sitetree') !== false || stristr($link, 'assets') !== false);
     }
 
     /**
@@ -468,7 +468,7 @@ 
      */
     public function linkIsJunk($link)
     {
-        return (bool)preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
+        return (bool) preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
     }
 
     /**
@@ -591,13 +591,13 @@ 
 
         // List the content sources to prompt user for selection
         if ($contentSources = StaticSiteContentSource::get()) {
-            $this->printMessage($newLine.'Available content-sources:'.$newLine);
+            $this->printMessage($newLine . 'Available content-sources:' . $newLine);
             foreach ($contentSources as $i => $contentSource) {
                 $this->printMessage("\tdev/tasks/" . __CLASS__ . ' SourceID=' . $contentSource->ID . ' ImportID=<number>');
             }
             echo $newLine;
             if (Director::is_cli()) {
-                $this->printMessage('Available command line options: '.$newLine);
+                $this->printMessage('Available command line options: ' . $newLine);
                 $this->printMessage("\tSourceID=<number> \t\tThe ID of the original crawl.");
                 $this->printMessage("\tImportID=<number> \t\tThe ID of the import to use.");
                 $this->printMessage("\tSHOW=pages \tPrint the contents of the pages map.");