silverstripe / silverstripe-externallinks

code/controllers/CMSExternalLinks.php

Braces +6 added lines, -2 removed lines

@@ -20,12 +20,14 @@ 
 
 		// Format status
 		$track = BrokenExternalPageTrackStatus::get_latest();
-		if($track) return json_encode(array(
+		if($track) {
+			return json_encode(array(
 			'TrackID' => $track->ID,
 			'Status' => $track->Status,
 			'Completed' => $track->getCompletedPages(),
 			'Total' => $track->getTotalPages()
 		));
+		}
 	}
 
 
@@ -35,7 +37,9 @@ 
 	public function start() {
 		// return if the a job is already running
 		$status = BrokenExternalPageTrackStatus::get_latest();
-		if ($status && $status->Status == 'Running') return;
+		if ($status && $status->Status == 'Running') {
+			return;
+		}
 
 		// Create a new job
 		if (class_exists('QueuedJobService')) {

code/jobs/CheckExternalLinksJob.php

Braces +3 added lines, -1 removed lines

@@ -1,6 +1,8 @@
 <?php
 
-if(!class_exists('AbstractQueuedJob')) return;
+if(!class_exists('AbstractQueuedJob')) {
+	return;
+}
 
 /**
  * A Job for running a external link check for published pages

code/model/BrokenExternalPageTrackStatus.php

Braces +3 added lines, -1 removed lines

@@ -40,8 +40,10 @@
 		$pageIDs = $this
 			->getIncompleteTracks()
 			->column('PageID');
-		if($pageIDs) return Versioned::get_by_stage('SiteTree', 'Stage')
+		if($pageIDs) {
+			return Versioned::get_by_stage('SiteTree', 'Stage')
 			->byIDs($pageIDs);
+		}
 	}
 
 	/**

code/reports/BrokenExternalLinksReport.php

Braces +3 added lines, -1 removed lines

@@ -56,7 +56,9 @@
 
 	public function sourceRecords() {
 		$track = BrokenExternalPageTrackStatus::get_latest();
-		if ($track) return $track->BrokenLinks();
+		if ($track) {
+			return $track->BrokenLinks();
+		}
 		return new ArrayList();
 	}
 

code/tasks/CheckExternalLinksTask.php

Braces +22 added lines, -7 removed lines

@@ -28,7 +28,9 @@ 
 	 * @param string $message
 	 */
 	protected function log($message) {
-		if(!$this->silent) Debug::message($message);
+		if(!$this->silent) {
+			Debug::message($message);
+		}
 	}
 
 	public function run($request) {
@@ -70,7 +72,10 @@ 
 
 		// Check link
 		$httpCode = $this->linkChecker->checkLink($href);
-		if($httpCode === null) return; // Null link means uncheckable, such as an internal link
+		if($httpCode === null) {
+			return;
+		}
+		// Null link means uncheckable, such as an internal link
 
 		// If this code is broken then mark as such
 		if($foundBroken = $this->isCodeBroken($httpCode)) {
@@ -84,7 +89,9 @@ 
 		}
 
 		// Check if we need to update CSS class, otherwise return
-		if($markedBroken == $foundBroken) return;
+		if($markedBroken == $foundBroken) {
+			return;
+		}
 		if($foundBroken) {
 			$class .= ' ss-broken';
 		} else {
@@ -101,11 +108,15 @@ 
 	 */
 	protected function isCodeBroken($httpCode) {
 		// Null represents no request attempted
-		if($httpCode === null) return false;
+		if($httpCode === null) {
+			return false;
+		}
 
 		// do we have any whitelisted codes
 		$ignoreCodes = Config::inst()->get('CheckExternalLinks', 'IgnoreCodes');
-		if(is_array($ignoreCodes) && in_array($httpCode, $ignoreCodes)) return false;
+		if(is_array($ignoreCodes) && in_array($httpCode, $ignoreCodes)) {
+			return false;
+		}
 
 		// Check if code is outside valid range
 		return $httpCode < 200 || $httpCode > 302;
@@ -123,7 +134,9 @@ 
 
 		// Calculate pages to run
 		$pageTracks = $status->getIncompleteTracks();
-		if($limit) $pageTracks = $pageTracks->limit($limit);
+		if($limit) {
+			$pageTracks = $pageTracks->limit($limit);
+		}
 
 		// Check each page
 		foreach ($pageTracks as $pageTrack) {
@@ -135,7 +148,9 @@ 
 			$page = $pageTrack->Page();
 			$this->log("Checking {$page->Title}");
 			$htmlValue = Injector::inst()->create('HTMLValue', $page->Content);
-			if (!$htmlValue->isValid()) continue;
+			if (!$htmlValue->isValid()) {
+				continue;
+			}
 
 			// Check each link
 			$links = $htmlValue->getElementsByTagName('a');

code/tasks/CurlLinkChecker.php

Braces +6 added lines, -2 removed lines

@@ -26,12 +26,16 @@
 	 */
 	public function checkLink($href) {
 		// Skip non-external links
-		if(!preg_match('/^https?[^:]*:\/\//', $href)) return null;
+		if(!preg_match('/^https?[^:]*:\/\//', $href)) {
+			return null;
+		}
 
 		// Check if we have a cached result
 		$cacheKey = md5($href);
 		$result = $this->getCache()->load($cacheKey);
-		if($result !== false) return $result;
+		if($result !== false) {
+			return $result;
+		}
 
 		// No cached result so just request
 		$handle = curl_init($href);