silverstripe / deploynaut

code/model/DNDataArchive.php

Spacing +3 added lines, -3 removed lines

@@ -254,7 +254,7 @@ 
 	 * @return boolean true if $member can upload archives linked to this environment, false if they can't.
 	 */
 	public function canMoveTo($targetEnv, $member = null) {
-		if ($this->Environment()->Project()->ID!=$targetEnv->Project()->ID) {
+		if($this->Environment()->Project()->ID != $targetEnv->Project()->ID) {
 			// We don't permit moving snapshots between projects at this stage.
 			return false;
 		}
@@ -266,7 +266,7 @@ 
 		if(Permission::checkMember($member, 'ADMIN')) return true;
 
 		// Checks if the user can actually access the archive.
-		if (!$this->canDownload($member)) return false;
+		if(!$this->canDownload($member)) return false;
 
 		// Hooks into ArchiveUploaders permission to prevent proliferation of permission checkboxes.
 		// Bypasses the quota check - we don't need to check for it as long as we move the snapshot within the project.
@@ -284,7 +284,7 @@ 
 		$archive = $this;
 		$envs = $this->Environment()->Project()->DNEnvironmentList()
 			->filterByCallback(function($item) use ($archive) {
-				return $archive->EnvironmentID!=$item->ID && $archive->canMoveTo($item);
+				return $archive->EnvironmentID != $item->ID && $archive->canMoveTo($item);
 			});
 
 		return $envs;

Braces +19 added lines, -6 removed lines

@@ -207,7 +207,9 @@ 
 	 */
 	public function canRestore($member = null) {
 		$memberID = $member ? $member->ID : Member::currentUserID();
-		if(!$memberID) return false;
+		if(!$memberID) {
+			return false;
+		}
 
 		$key = $memberID . '-' . $this->EnvironmentID;
 		if(!isset(self::$_cache_can_restore[$key])) {
@@ -226,7 +228,9 @@ 
 	 */
 	public function canDownload($member = null) {
 		$memberID = $member ? $member->ID : Member::currentUserID();
-		if(!$memberID) return false;
+		if(!$memberID) {
+			return false;
+		}
 
 		$key = $memberID . '-' . $this->EnvironmentID;
 		if(!isset(self::$_cache_can_download[$key])) {
@@ -259,14 +263,23 @@ 
 			return false;
 		}
 
-		if(!$member) $member = Member::currentUser();
-		if(!$member) return false; // Must be logged in to check permissions
+		if(!$member) {
+			$member = Member::currentUser();
+		}
+		if(!$member) {
+			return false;
+		}
+		// Must be logged in to check permissions
 
 		// Admin can always move.
-		if(Permission::checkMember($member, 'ADMIN')) return true;
+		if(Permission::checkMember($member, 'ADMIN')) {
+			return true;
+		}
 
 		// Checks if the user can actually access the archive.
-		if (!$this->canDownload($member)) return false;
+		if (!$this->canDownload($member)) {
+			return false;
+		}
 
 		// Hooks into ArchiveUploaders permission to prevent proliferation of permission checkboxes.
 		// Bypasses the quota check - we don't need to check for it as long as we move the snapshot within the project.

code/model/DNFilteredCommits.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 	}
 
 	public function getCommits() {
-		if (!isset($this->env)) {
+		if(!isset($this->env)) {
 			$this->environmentNotSetException();
 		}
 		$successfulCommits =
@@ -54,7 +54,7 @@ 
 	}
 
 	public function getLatestCommit() {
-		if (!isset($this->env)) {
+		if(!isset($this->env)) {
 			$this->environmentNotSetException();
 		}
 		$commits = $this->getCommits();

code/model/GitonomyCache.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 	public static $cache = array();
 
 	public static function getIncludingBranches($commit) {
-		$cacheKey = 'getIncludingBranches-' .$commit->getRepository()->getPath() . '-' . $commit->gethash();
+		$cacheKey = 'getIncludingBranches-' . $commit->getRepository()->getPath() . '-' . $commit->gethash();
 		if(!isset(self::$cache[$cacheKey])) {
 			self::$cache[$cacheKey] = $commit->getIncludingBranches();
 		}

code/model/jobs/DNPing.php

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 /**
- * This class will queue a ping job and also proxy to the log file of that output
- */
+	 * This class will queue a ping job and also proxy to the log file of that output
+	 */
 class DNPing extends DataObject {
 
 	/**

code/model/steps/DeploymentPipelineStep.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 						return false;
 				}
 			case 'Queued':
-				if ($this->Pipeline()->SkipSnapshot) {
+				if($this->Pipeline()->SkipSnapshot) {
 					return $this->startDeploy();
 				} else {
 					return $this->createSnapshot();
@@ -84,7 +84,7 @@ 
 
 		// Skip deployment for dry run
 		if($this->Pipeline()->DryRun) {
-			$this->log("[Skipped] Create DNDeployment for SHA ".$pipeline->SHA);
+			$this->log("[Skipped] Create DNDeployment for SHA " . $pipeline->SHA);
 			$this->write();
 			return true;
 		}

code/model/steps/EmergencyRollbackStep.php

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
 		}
 
 		// check if we have timed out
-		if ($this->isTimedOut()) {
+		if($this->isTimedOut()) {
 			$this->log(sprintf(_t('EmergencyRollbackStep.ROLLBACKTIMEOUT',
 				"{$this->Title} is older than %s seconds and has timed out"),
 				$this->MaxDuration));
@@ -85,7 +85,7 @@ 
 			);
 		}
 
-		if ($this->Status == 'Queued') {
+		if($this->Status == 'Queued') {
 			$this->start();
 		}
 
@@ -96,7 +96,7 @@ 
 		$this->write();
 
 		// Rollback itself is handled by the Pipeline object. This step will be marked as failed.
-		if ($this->Pipeline()->isRunning()) {
+		if($this->Pipeline()->isRunning()) {
 			$this->Pipeline()->markFailed();
 			return true;
 		} else {
@@ -127,7 +127,7 @@ 
 	 */
 	public function beginRollbackWindow() {
 		$this->Status = 'Started';
-		if (!$this->Started) $this->Started = SS_Datetime::now()->Rfc2822();
+		if(!$this->Started) $this->Started = SS_Datetime::now()->Rfc2822();
 		$this->log(_t('EmergencyRollbackStep.BEGINROLLBACKWINDOW',
 			"{$this->Title} is beginning a rollback window..."));
 		$this->write();

Braces +6 added lines, -2 removed lines

@@ -127,7 +127,9 @@ 
 	 */
 	public function beginRollbackWindow() {
 		$this->Status = 'Started';
-		if (!$this->Started) $this->Started = SS_Datetime::now()->Rfc2822();
+		if (!$this->Started) {
+			$this->Started = SS_Datetime::now()->Rfc2822();
+		}
 		$this->log(_t('EmergencyRollbackStep.BEGINROLLBACKWINDOW',
 			"{$this->Title} is beginning a rollback window..."));
 		$this->write();
@@ -138,7 +140,9 @@ 
 
 	public function getRunningDescription() {
 		// Don't show options if this step has already been confirmed
-		if($this->RolledBack) return;
+		if($this->RolledBack) {
+			return;
+		}
 
 		return _t('EmergencyRollbackStep.RUNNINGDESCRIPTION',
 			'You may now roll back to the previous version, if needed.');

code/model/steps/LongRunningPipelineStep.php

Braces +3 added lines, -1 removed lines

@@ -41,7 +41,9 @@
 		if($this->Started) {
 			$started = intval($this->dbObject('Started')->Format('U'));
 			$now = intval(SS_Datetime::now()->Format('U'));
-			if($started && $now) return $now - $started;
+			if($started && $now) {
+				return $now - $started;
+			}
 		}
 		return 0;
 	}

code/model/steps/RollbackStep.php

Spacing +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
 	 * @return boolean
 	 */
 	protected function doRestoreDB() {
-		if ($this->Pipeline()->SkipSnapshot) return false;
+		if($this->Pipeline()->SkipSnapshot) return false;
 
 		return $this->getConfigSetting('RestoreDB') && $this->Pipeline()->PreviousSnapshot();
 	}

Braces +3 added lines, -1 removed lines

@@ -219,7 +219,9 @@
 	 * @return boolean
 	 */
 	protected function doRestoreDB() {
-		if ($this->Pipeline()->SkipSnapshot) return false;
+		if ($this->Pipeline()->SkipSnapshot) {
+			return false;
+		}
 
 		return $this->getConfigSetting('RestoreDB') && $this->Pipeline()->PreviousSnapshot();
 	}

code/tasks/DNMigrate1_1to1_2.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 			foreach($environments as $environment) {
 				$newFilename = basename($environment->Filename);
 				if($environment->Filename != $newFilename) {
-					echo 'Migrating "'.$environment->Filename.'" to '.$newFilename.PHP_EOL;
+					echo 'Migrating "' . $environment->Filename . '" to ' . $newFilename . PHP_EOL;
 					$environment->Filename = $newFilename;
 					$environment->write();
 				}
@@ -26,7 +26,7 @@ 
 		foreach($projects as $project) {
 			if(!$project->projectFolderExists()) {
 				$warnings = true;
-				echo 'Project '.$project->Name.' don\'t have a cap project folder'.PHP_EOL;
+				echo 'Project ' . $project->Name . ' don\'t have a cap project folder' . PHP_EOL;
 			}
 		}
 		if($warnings) {