silverstripe / deploynaut

code/model/jobs/DNDeployment.php

Spacing +4 added lines, -4 removed lines

@@ -201,12 +201,12 @@ 
 	 */
 	public function getFullDeployMessages() {
 		$strategy = $this->getDeploymentStrategy();
-		if ($strategy->getActionCode()!=='full') return null;
+		if($strategy->getActionCode() !== 'full') return null;
 
 		$changes = $strategy->getChangesModificationNeeded();
 		$messages = [];
-		foreach ($changes as $change => $details) {
-			if ($change==='Code version') continue;
+		foreach($changes as $change => $details) {
+			if($change === 'Code version') continue;
 
 			$messages[] = [
 				'Flag' => sprintf(
@@ -217,7 +217,7 @@ 
 			];
 		}
 
-		if (empty($messages)) {
+		if(empty($messages)) {
 			$messages[] = [
 				'Flag' => '',
 				'Text' => '<i>Environment changes have been made.</i>'

Braces +9 added lines, -3 removed lines

@@ -201,12 +201,16 @@ 
 	 */
 	public function getFullDeployMessages() {
 		$strategy = $this->getDeploymentStrategy();
-		if ($strategy->getActionCode()!=='full') return null;
+		if ($strategy->getActionCode()!=='full') {
+			return null;
+		}
 
 		$changes = $strategy->getChangesModificationNeeded();
 		$messages = [];
 		foreach ($changes as $change => $details) {
-			if ($change==='Code version') continue;
+			if ($change==='Code version') {
+				continue;
+			}
 
 			$messages[] = [
 				'Flag' => sprintf(
@@ -260,7 +264,9 @@ 
 		$list = new ArrayList();
 		$strategy = $this->getDeploymentStrategy();
 		foreach($strategy->getChanges() as $name => $change) {
-			if(empty($change['to'])) continue;
+			if(empty($change['to'])) {
+				continue;
+			}
 
 			$list->push(new ArrayData([
 				'Name' => $name,

code/model/steps/SmokeTestPipelineStep.php

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
 		curl_setopt($ch, CURLOPT_URL, $test['URL']);
 
 		// Allow individual tests to override number of attempts
-		$attempts = (int) $this->getConfigSetting('Attempts');
+		$attempts = (int)$this->getConfigSetting('Attempts');
 		if(!empty($test['Attempts'])) {
 			$attempts = $test['Attempts'];
 		}

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
 	 */
 	public function startApproval() {
 		$this->Status = 'Started';
-		$this->log("Starting {$this->Title}...");
+		$this->log("starting {$this->Title}...");
 		// Determine if we should use delayed notification
 		$recipientGroup = 'all';
 		if($this->getConfigSetting('RecipientsDelay')) {

code/model/steps/UserConfirmationStep.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
 	 */
 	public function startApproval() {
 		$this->Status = 'Started';
-		$this->log("Starting {$this->Title}...");
+		$this->log("starting {$this->Title}...");
 		// Determine if we should use delayed notification
 		$recipientGroup = 'all';
 		if($this->getConfigSetting('RecipientsDelay')) {

code/tasks/DNMigrate1_1to1_2.php

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * Will convert the Filename to the version 1.2 of deploynaut where it
- * more or less is exactly is the same as the Name but with .rb
- */
+	 * Will convert the Filename to the version 1.2 of deploynaut where it
+	 * more or less is exactly is the same as the Name but with .rb
+	 */
 class DNMigrate1_1to1_2 extends BuildTask {
 	/**
 	 * @param SS_HTTPRequest $request

code/tasks/FilesystemSyncTask.php

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 /**
- * Surface Filesystem::sync() as a dev task in case CMS has not been included.
- */
+	 * Surface Filesystem::sync() as a dev task in case CMS has not been included.
+	 */
 class CMSFilesystemSyncTask extends BuildTask {
 	public function run($request = null) {
 		Filesystem::sync();

code/model/steps/TriggerDeployStep.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
 	 */
 	public function startApproval() {
 		$this->Status = 'Started';
-		$this->log("Starting {$this->Title}...");
+		$this->log("starting {$this->Title}...");
 		// Determine if we should use delayed notification
 		$recipientGroup = 'all';
 		if($this->getConfigSetting('RecipientsDelay')) {

code/model/jobs/DNPing.php

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * This class will queue a ping job and also proxy to the log file of that output
- *
- * @property string $ResqueToken
- *
- * @method DNEnvironment Environment()
- * @property int EnvironmentID
- * @method Member Deployer()
- * @property int DeployerID
- */
+	 * This class will queue a ping job and also proxy to the log file of that output
+	 *
+	 * @property string $ResqueToken
+	 *
+	 * @method DNEnvironment Environment()
+	 * @property int EnvironmentID
+	 * @method Member Deployer()
+	 * @property int DeployerID
+	 */
 class DNPing extends DataObject {
 
 	/**