phptek / silverstripe-exodus

src/Tool/StaticSiteContentExtractor.php

Spacing +2 added lines, -2 removed lines

@@ -216,7 +216,7 @@ 
             if (!trim($excludeSelector)) {
                 continue;
             }
-            $element = $this->phpQuery[$parentSelector.' '.$excludeSelector];
+            $element = $this->phpQuery[$parentSelector . ' ' . $excludeSelector];
             if ($element) {
                 $remove = $element->htmlOuter();
                 $content = str_replace($remove, '', $content);
@@ -330,7 +330,7 @@ 
         // Run request
         curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
         // See: http://forums.devshed.com/php-development-5/curlopt-timeout-option-for-curl-calls-isn-t-being-obeyed-605642.html
-        curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, $timeout);		// No. seconds to wait while trying to connect.
+        curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, $timeout); // No. seconds to wait while trying to connect.
 
         // Deal to files, write to them directly and then return
         if ($this->mimeProcessor->isOfFileOrImage($this->mime)) {

src/Tool/StaticSiteMimeProcessor.php

Spacing +4 added lines, -4 removed lines

@@ -175,8 +175,8 @@ 
         $mimeCategories = File::config()->get('app_categories');
         list($ext, $mime) = [strtolower($ext), strtolower($mime)];
 
-        $notAuthoratative = !isset($httpMimeTypes[$ext]);					// We've found ourselves a weird extension
-        $notMatch = (!$notAuthoratative && $httpMimeTypes[$ext] !== $mime);	// No match found for passed extension in our ext=>mime mapping from config
+        $notAuthoratative = !isset($httpMimeTypes[$ext]); // We've found ourselves a weird extension
+        $notMatch = (!$notAuthoratative && $httpMimeTypes[$ext] !== $mime); // No match found for passed extension in our ext=>mime mapping from config
 
         if ($notAuthoratative || $notMatch) {
             if (!$fix) {
@@ -187,8 +187,8 @@ 
             $coreExts = array_merge($mimeCategories['document'], $mimeCategories['image']);
             foreach ($coreExts as $coreExt) {
                 // Make sure we check the correct category so we don't find a match for ms-excel in the image \File category (.cel) !!
-                $isFile = in_array($coreExt, $mimeCategories['document']) && singleton(__CLASS__)->isOfFile($mime);		// dirty
-                $isImge = in_array($coreExt, $mimeCategories['image']) && singleton(__CLASS__)->isOfImage($mime);	// more dirt
+                $isFile = in_array($coreExt, $mimeCategories['document']) && singleton(__CLASS__)->isOfFile($mime); // dirty
+                $isImge = in_array($coreExt, $mimeCategories['image']) && singleton(__CLASS__)->isOfImage($mime); // more dirt
 
                 if (($isFile || $isImge) && stristr($mime, $coreExt) !== false) {
                     // "Manually" force "jpg" as the file-suffix to be returned

src/Tool/StaticSiteUtils.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
 
         if ($logFile && is_writable($logFile) || !file_exists($logFile) && is_writable(dirname($logFile))) {
             $message = $message . ($filename ? ' ' . $filename : '') . ($mime ? ' (' . $mime . ')' : '');
-            error_log($message. PHP_EOL, 3, $logFile);
+            error_log($message . PHP_EOL, 3, $logFile);
         }
     }
 
@@ -47,7 +47,7 @@ 
         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/Model/StaticSiteImportDataObject.php

Spacing +2 added lines, -2 removed lines

@@ -100,11 +100,11 @@
         $relatedFailedRewriteObjects = DataObject::get(FailedURLRewriteObject::class)->filter('ImportID', $this->ID);
         $relatedFailedRewriteSummaries = DataObject::get(FailedURLRewriteSummary::class)->filter('ImportID', $this->ID);
 
-        $relatedFailedRewriteObjects->each(function ($item) {
+        $relatedFailedRewriteObjects->each(function($item) {
             $item->delete();
         });
 
-        $relatedFailedRewriteSummaries->each(function ($item) {
+        $relatedFailedRewriteSummaries->each(function($item) {
             $item->delete();
         });
     }

src/Model/StaticSiteContentSourceImportSchema.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
         "Order" => DBInt::class,
         "AppliesTo" => DBVarchar::class,
         "MimeTypes" => DBText::class,
-        "Notes" => DBText::class,	// Purely informational. Not used in imports.
+        "Notes" => DBText::class, // Purely informational. Not used in imports.
     ];
 
     /**
@@ -96,7 +96,7 @@ 
      */
     public function getTitle(): string
     {
-        return $this->DataType . ' (' .$this->AppliesTo . ')';
+        return $this->DataType . ' (' . $this->AppliesTo . ')';
     }
 
     /**

src/Report/FailedURLRewriteReport.php

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
         return [
             'Title' => [
                 'title' => 'Imported page',
-                'formatting' => function ($value, $item) {
+                'formatting' => function($value, $item) {
                     return sprintf(
                         '<a href="admin/pages/edit/show/%s">%s</a>',
                         $item->ContainedInID,

src/Extension/StaticSiteExternalContentAdminExtension.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
         }
 
         if ($imports) {
-            $imports->each(function ($item) {
+            $imports->each(function($item) {
                 $item->delete();
             });
             $messageType = 'good';

src/Extension/StaticSiteDataExtension.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     {
         parent::onAfterDelete();
         if ($failedRewriteObjects = DataObject::get(FailedURLRewriteObject::class)->filter('ContainedInID', $this->owner->ID)) {
-            $failedRewriteObjects->each(function ($item) {
+            $failedRewriteObjects->each(function($item) {
                 $item->delete();
             });
         }

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");
@@ -313,7 +313,7 @@ 
             return;
         }
         $url = ($url ? '(' . $url . ') ' : '');
-        $level = ($level ? '[' . $level .'] ' : '');
+        $level = ($level ? '[' . $level . '] ' : '');
         if (Director::is_cli()) {
             echo "$level$message$url" . PHP_EOL;
         } else {
@@ -346,7 +346,7 @@ 
                 'ContainedInID' => $failure['ContainedInID']
             ));
 
-            if ($failureExists->count() >0) {
+            if ($failureExists->count() > 0) {
                 continue;
             }
 
@@ -422,7 +422,7 @@ 
     public function linkIsThirdParty($link)
     {
         $link = trim($link);
-        return (bool)preg_match("#^http(s)?://#", $link);
+        return (bool) preg_match("#^http(s)?://#", $link);
     }
 
     /**
@@ -436,7 +436,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);
     }
 
     /**
@@ -447,7 +447,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);
     }
 
     /**
@@ -458,7 +458,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);
     }
 
     /**
@@ -469,7 +469,7 @@ 
      */
     public function linkIsJunk($link)
     {
-        return (bool)preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
+        return (bool) preg_match("#^[^\[\/a-zA-Z\d].+#", $link);
     }
 
     /**
@@ -592,13 +592,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.");