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/backends/DeploymentBackend.php

Doc Comments +5 added lines

@@ -10,6 +10,7 @@ 
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
 	 * @param type $leaveMaintenancePage
+	 * @return void
 	 */
 	public function deploy(DNEnvironment $environment, $sha, DeploynautLogFile $log, DNProject $project, $leaveMaintenancePage = false);
 
@@ -19,6 +20,7 @@ 
 	 *
 	 * @param DNDataTransfer $dataTransfer
 	 * @param DeploynautLogFile $log
+	 * @return void
 	 */
 	public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log);
 
@@ -28,6 +30,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function enableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 
@@ -37,6 +40,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 
@@ -46,6 +50,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function ping(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 

code/backends/PackageCache.php

Doc Comments +1 added lines

@@ -13,6 +13,7 @@
 	 * @param string $sha The SHA of the commit to be deployed
 	 * @param string $repositoryDir The directory where the repository resides
 	 * @param DeploynautLogFile $log The log to write status output to, including package-generation commands
+	 * @return string
 	 */
 	public function getPackageFilename(
 		PackageGenerator $generator,

code/model/PipelineData.php

Doc Comments +1 added lines

@@ -13,6 +13,7 @@
 	 * Log message
 	 *
 	 * @param string $message The message to log
+	 * @return void
 	 */
 	public function log($message);
 }