silverstripe / deploynaut

code/backends/SimplePackageGenerator.php

Spacing +4 added lines, -4 removed lines

@@ -30,7 +30,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);
 		}
@@ -42,17 +42,17 @@ 
 		// Execute these in sequence until there's a failure
 		$processes = array(
 			// Export the relevant SHA into a temp folder
-			new AbortableProcess("git archive $sha | tar -x -C " . $escapedTempPath, $baseDir),
+			new AbortableProcess("git archive $sha | tar -x -C ".$escapedTempPath, $baseDir),
 			// Run build script
 			new AbortableProcess($this->buildScript, $tempPath, null, null, 3600),
 			// Compress the result
-			new AbortableProcess("tar -czf " . $escapedOutputFile . " " . $escapedTempDir, dirname($tempPath)),
+			new AbortableProcess("tar -czf ".$escapedOutputFile." ".$escapedTempDir, dirname($tempPath)),
 		);
 
 		// Call at the end, regardless of success or failure
 		$cleanup = array(
 			// Delete the temporary staging folder
-			new AbortableProcess("rm -rf " . $escapedTempPath),
+			new AbortableProcess("rm -rf ".$escapedTempPath),
 		);
 
 		try {

code/AbortableProcess.php

Doc Comments +4 added lines

@@ -13,6 +13,10 @@
  * will still get called if we loop over ->isRunning.
  */
 class SymfonyProcessFixer extends \Symfony\Component\Process\Process {
+
+	/**
+	 * @param Closure $callback
+	 */
 	public function run($callback = null) {
 		parent::start($callback);
 		while(parent::isRunning()) {

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 			sleep(1);
 		}
 
-		if ($this->hasBeenSignaled()) {
+		if($this->hasBeenSignaled()) {
 			throw new RuntimeException(sprintf('The process has been signaled with signal "%s".', $this->getTermSignal));
 		}