silverstripe / deploynaut

code/model/jobs/DNCreateEnvironment.php

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
 	 * @return string
 	 */
 	public function LogLink() {
-		return $this->Link() . '/log';
+		return $this->Link().'/log';
 	}
 
 	/**
@@ -190,7 +190,7 @@ 
 			if($env instanceof EnvironmentCreateBackend) {
 				return $env;
 			} else {
-				throw new Exception("Invalid backend: " . $data['EnvironmentType']);
+				throw new Exception("Invalid backend: ".$data['EnvironmentType']);
 			}
 		}
 		return null;

code/control/DeploynautLogFile.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
 	 * @return string
 	 */
 	public function getRawFilePath() {
-		return $this->basePath . '/' . $this->logFile;
+		return $this->basePath.'/'.$this->logFile;
 	}
 
 	/**
@@ -47,7 +47,7 @@ 
 	 * @return string
 	 */
 	public function getSanitisedLogFilePath() {
-		return $this->basePath . '/' . strtolower(FileNameFilter::create()->filter($this->logFile));
+		return $this->basePath.'/'.strtolower(FileNameFilter::create()->filter($this->logFile));
 	}
 
 	/**
@@ -77,7 +77,7 @@ 
 		// Make sure we write into the old path for existing logs. New logs use the sanitised file path instead.
 		$path = file_exists($this->getRawFilePath()) ? $this->getRawFilePath() : $this->getSanitisedLogFilePath();
 
-		error_log('['.date('Y-m-d H:i:s').'] ' . $message .PHP_EOL, 3, $path);
+		error_log('['.date('Y-m-d H:i:s').'] '.$message.PHP_EOL, 3, $path);
 		@chmod($path, 0666);
 	}
 

code/control/CheckPipelineStatus.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 
 		printf(
 			"%s Checking status of %d pipelines... ",
-			"[" . date("Y-m-d H:i:s") . "]",
+			"[".date("Y-m-d H:i:s")."]",
 			$runningPipelines->count()
 		);
 
@@ -32,6 +32,6 @@ 
 			$pipeline->checkPipelineStatus();
 		}
 
-		echo "done!" . PHP_EOL;
+		echo "done!".PHP_EOL;
 	}
 }

code/control/DeployForm.php

Spacing +8 added lines, -8 removed lines

@@ -41,7 +41,7 @@ 
 		if(!preg_match('/^[a-f0-9]{40}$/', $sha)) {
 			$this->validationError(
 				$field,
-				"Invalid release SHA: " . Convert::raw2xml($sha),
+				"Invalid release SHA: ".Convert::raw2xml($sha),
 				"error"
 			);
 			return false;
@@ -165,7 +165,7 @@ 
 
 		// Generate actions allowed for this user
 		$actions = new FieldList(
-			FormAction::create('startPipeline', "Begin the release process on " . $environment->Name)
+			FormAction::create('startPipeline', "Begin the release process on ".$environment->Name)
 				->addExtraClass('btn btn-primary')
 				->setAttribute('onclick', "return confirm('This will begin a release pipeline. Continue?');")
 		);
@@ -175,9 +175,9 @@ 
 					->addExtraClass('btn btn-info')
 					->setAttribute(
 						'onclick',
-						"return confirm('This will begin a release pipeline, but with the following exclusions:\\n" .
-						" - No messages will be sent\\n" .
-						" - No capistrano actions will be invoked\\n" .
+						"return confirm('This will begin a release pipeline, but with the following exclusions:\\n".
+						" - No messages will be sent\\n".
+						" - No capistrano actions will be invoked\\n".
 						" - No deployments or snapshots will be created.');"
 					)
 			);
@@ -188,7 +188,7 @@ 
 					->addExtraClass('btn btn-warning')
 					->setAttribute(
 						'onclick',
-						"return confirm('This will start a direct deployment, bypassing the pipeline " .
+						"return confirm('This will start a direct deployment, bypassing the pipeline ".
 						"process in place.\\n\\nAre you sure this is necessary?');"
 					)
 			);
@@ -215,7 +215,7 @@ 
 				substr($sha, 0, 8),
 				$branch->LastUpdated()->TimeDiff()
 			);
-			$branches[$sha . '-' . $name] = $branchValue;
+			$branches[$sha.'-'.$name] = $branchValue;
 		}
 
 		// Tags
@@ -228,7 +228,7 @@ 
 				substr($sha, 0, 8),
 				$branch->LastUpdated()->TimeDiff()
 			);
-			$tags[$sha . '-' . $tag] = $tagValue;
+			$tags[$sha.'-'.$tag] = $tagValue;
 		}
 		$tags = array_reverse($tags);
 

code/form/DataArchiveFileField.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 		// Hack: loadIntoFile assumes paths relative to assets,
 		//  otherwise it creates the whole structure *within* that folder
 		$absolutePath = $dataArchive->generateFilepath($dataTransfer);
-		$relativePath = preg_replace('#^' . preg_quote(ASSETS_PATH) . '/#', '', $absolutePath);
+		$relativePath = preg_replace('#^'.preg_quote(ASSETS_PATH).'/#', '', $absolutePath);
 		$this->upload->loadIntoFile($_FILES[$this->name], $file, $relativePath);
 		if($this->upload->isError()) {
 			return false;
@@ -45,7 +45,7 @@ 
 		$file = $this->upload->getFile();
 		if($this->relationAutoSetting) {
 			// save to record
-			$record->{$this->name . 'ID'} = $file->ID;
+			$record->{$this->name.'ID'} = $file->ID;
 		}
 
 		return $this;

code/queuereport/ResqueQueue.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 	protected function getJobs() {
 		$jobs = new ArrayList();
 
-		$resqueJobs = Resque::redis()->lrange('queue:' . $this->Name, 0, -1);
+		$resqueJobs = Resque::redis()->lrange('queue:'.$this->Name, 0, -1);
 		if(!$resqueJobs) {
 			$jobs->push(new ResqueJob(array(
 				'Name' => 'null',
@@ -61,8 +61,8 @@ 
 			$stdClass = json_decode($job);
 			if($stdClass->class === 'CapistranoDeploy' || $stdClass->class === 'DeployJob') {
 				$value = $stdClass->args[0]->projectName
-					. ':' . $stdClass->args[0]->environment
-					. ' - ' . $stdClass->args[0]->sha;
+					. ':'.$stdClass->args[0]->environment
+					. ' - '.$stdClass->args[0]->sha;
 				$jobs->push(new ResqueJob(array(
 					'Name' => $stdClass->class,
 					'Value' => $value,

code/backends/SimplePackageGenerator.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
 	 * Generate the package
 	 */
 	public function generatePackage($sha, $baseDir, $outputFilename, DeploynautLogFile $log) {
-		$tempPath = TEMP_FOLDER . "/" . str_replace(".tar.gz", "", basename($outputFilename));
+		$tempPath = TEMP_FOLDER."/".str_replace(".tar.gz", "", basename($outputFilename));
 		if(!file_exists($tempPath)) {
 			mkdir($tempPath);
 		}
@@ -44,17 +44,17 @@ 
 		// Execute these in sequence until there's a failure
 		$processes = array(
 			// Export the relevant SHA into a temp folder
-			new Process("git archive $sha | tar -x -C " . $escapedTempPath, $baseDir),
+			new Process("git archive $sha | tar -x -C ".$escapedTempPath, $baseDir),
 			// Run build script
 			new Process($this->buildScript, $tempPath, null, null, 3600),
 			// Compress the result
-			new Process("tar -czf " . $escapedOutputFile . " " . $escapedTempDir, dirname($tempPath)),
+			new Process("tar -czf ".$escapedOutputFile." ".$escapedTempDir, dirname($tempPath)),
 		);
 
 		// Call at the end, regardless of success or failure
 		$cleanup = array(
 			// Delete the temporary staging folder
-			new Process("rm -rf " . $escapedTempPath),
+			new Process("rm -rf ".$escapedTempPath),
 		);
 
 		try {

code/backends/PackageGenerator.php

Spacing +2 added lines, -2 removed lines

@@ -51,12 +51,12 @@
 	public function getPackageFilename($identifier, $sha, $repositoryDir, DeploynautLogFile $log) {
 		// Fetch through the cache
 		if($this->cache) {
-			$identifier .= '-' . get_class($this) . '-' . $this->getIdentifier();
+			$identifier .= '-'.get_class($this).'-'.$this->getIdentifier();
  			return $this->cache->getPackageFilename($this, $identifier, $sha, $repositoryDir, $log);
 
  		// Default, cacheless implementation
  		} else {
- 			$filename = TEMP_FOLDER . '/' . $sha . '.tar.gz';
+ 			$filename = TEMP_FOLDER.'/'.$sha.'.tar.gz';
  			if($this->generatePackage($sha, $repositoryDir, $filename, $log)) {
  				return $filename;
  			}

code/backends/DemoDeploymentBackend.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
 		$file = sprintf('%s/%s.deploy-history.txt', DEPLOYNAUT_LOG_PATH, $environment->getFullName());
 		$CLI_file = escapeshellarg($file);
-		$CLI_line = escapeshellarg(date('Y-m-d H:i:s') . " => $sha");
+		$CLI_line = escapeshellarg(date('Y-m-d H:i:s')." => $sha");
 
 		// Put maintenance page up
 		$this->enableMaintenance($environment, $log, $project);