silverstripe / sspak

tests/DataExtractor/CsvTableWriterTest.php

Spacing +6 added lines, -6 removed lines

@@ -13,14 +13,14 @@ 
 		$csv = new CsvTableWriter('/tmp/output.csv');
 
 		$csv->start(['Col1', 'Col2', 'Col3']);
-		$csv->writeRecord([ 'Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three' ]);
-		$csv->writeRecord([ 'Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?" ]);
+		$csv->writeRecord(['Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three']);
+		$csv->writeRecord(['Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?"]);
 		$csv->finish();
 
 		$csvContent = file_get_contents('/tmp/output.csv');
 		unlink('/tmp/output.csv');
 
-		$fixture = file_get_contents(__DIR__ . '/fixture/input.csv');
+		$fixture = file_get_contents(__DIR__.'/fixture/input.csv');
 
 		$this->assertEquals($fixture, $csvContent);
 	}
@@ -33,14 +33,14 @@ 
 
 		$csv = new CsvTableWriter('/tmp/output.csv');
 
-		$csv->writeRecord([ 'Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three' ]);
-		$csv->writeRecord([ 'Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?" ]);
+		$csv->writeRecord(['Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three']);
+		$csv->writeRecord(['Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?"]);
 		$csv->finish();
 
 		$csvContent = file_get_contents('/tmp/output.csv');
 		unlink('/tmp/output.csv');
 
-		$fixture = file_get_contents(__DIR__ . '/fixture/input.csv');
+		$fixture = file_get_contents(__DIR__.'/fixture/input.csv');
 
 		$this->assertEquals($fixture, $csvContent);
 	}

tests/SmokeTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-define('PACKAGE_ROOT' , dirname(__DIR__).'/');
-require_once(PACKAGE_ROOT . 'src/SSPak.php');
+define('PACKAGE_ROOT', dirname(__DIR__).'/');
+require_once(PACKAGE_ROOT.'src/SSPak.php');
 
 /**
  * Confirm that the compile binary executes

src/SSPakFile.php

Spacing +17 added lines, -17 removed lines

@@ -8,16 +8,16 @@ 
 
 	function __construct($path, $executor, $pharAlias = 'sspak.phar') {
 		parent::__construct($path, $executor);
-		if(!$this->isLocal()) throw new LogicException("Can't manipulate remote .sspak.phar files, only remote webroots.");
+		if (!$this->isLocal()) throw new LogicException("Can't manipulate remote .sspak.phar files, only remote webroots.");
 
 		$this->pharAlias = $pharAlias;
 		$this->pharPath = $path;
 
 		// Executable Phar version
-		if(substr($path,-5) === '.phar') {
+		if (substr($path, -5) === '.phar') {
 			$this->phar = new Phar($path, FilesystemIterator::CURRENT_AS_FILEINFO | FilesystemIterator::KEY_AS_FILENAME,
 				$this->pharAlias);
-			if(!file_exists($this->path)) $this->makeExecutable();
+			if (!file_exists($this->path)) $this->makeExecutable();
 
 		// Non-executable Tar version
 		} else {
@@ -34,11 +34,11 @@ 
 	 * Add the SSPak executable information into this SSPak file
 	 */
 	function makeExecutable() {
-		if(ini_get('phar.readonly')) {
+		if (ini_get('phar.readonly')) {
 			throw new Exception("Please set phar.readonly to false in your php.ini.");
 		}
 
-		passthru("composer install -d " . escapeshellarg(PACKAGE_ROOT) . " --no-dev");
+		passthru("composer install -d ".escapeshellarg(PACKAGE_ROOT)." --no-dev");
 
 		$root = PACKAGE_ROOT;
 		$srcRoots = [
@@ -47,11 +47,11 @@ 
 		];
 
 		// Add the bin file, but strip of the #! exec header.
-		$this->phar['bin/sspak'] = preg_replace("/^#!\/usr\/bin\/env php\n/", '', file_get_contents($root . "bin/sspak"));
+		$this->phar['bin/sspak'] = preg_replace("/^#!\/usr\/bin\/env php\n/", '', file_get_contents($root."bin/sspak"));
 
-		foreach($srcRoots as $srcRoot) {
-			foreach(new RecursiveIteratorIterator(new RecursiveDirectoryIterator($root . $srcRoot)) as $fileObj) {
-				if($fileObj->isFile()) {
+		foreach ($srcRoots as $srcRoot) {
+			foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($root.$srcRoot)) as $fileObj) {
+				if ($fileObj->isFile()) {
 					$file = $fileObj->getRealPath();
 
 					$relativeFile = str_replace($root, '', $file);
@@ -74,7 +74,7 @@ 
 		$this->phar->setStub($stub);
 		chmod($this->path, 0775);
 
-		passthru("composer install -d " . escapeshellarg(PACKAGE_ROOT));
+		passthru("composer install -d ".escapeshellarg(PACKAGE_ROOT));
 	}
 
 	/**
@@ -103,8 +103,8 @@ 
 		// Non-executable Phars can't have content streamed into them
 		// This means that we need to create a temp file, which is a pain, if that file happens to be a 3GB
 		// asset dump. :-/
-		if($this->phar instanceof PharData) {
-			$tmpFile = '/tmp/sspak-content-' .rand(100000,999999);
+		if ($this->phar instanceof PharData) {
+			$tmpFile = '/tmp/sspak-content-'.rand(100000, 999999);
 			$process->exec(array('outputFile' => $tmpFile));
 			$this->phar->addFile($tmpFile, $filename);
 			unlink($tmpFile);
@@ -123,7 +123,7 @@ 
 	 * @return Stream context
 	 */
 	function writeStreamForFile($filename) {
-		return fopen('phar://' . $this->pharAlias . '/' . $filename, 'w');
+		return fopen('phar://'.$this->pharAlias.'/'.$filename, 'w');
 	}
 
 	/**
@@ -134,7 +134,7 @@ 
 	function readStreamForFile($filename) {
 		// Note: using pharAlias here doesn't work on Debian Wheezy (nor on Windows for that matter).
 		//return fopen('phar://' . $this->pharAlias . '/' . $filename, 'r');
-		return fopen('phar://' . $this->pharPath . '/' . $filename, 'r');
+		return fopen('phar://'.$this->pharPath.'/'.$filename, 'r');
 	}
 
 	/**
@@ -153,10 +153,10 @@ 
 	function gitRemoteDetails() {
 		$content = $this->content('git-remote');
 		$details = array();
-		foreach(explode("\n", trim($content)) as $line) {
-			if(!$line) continue;
+		foreach (explode("\n", trim($content)) as $line) {
+			if (!$line) continue;
 
-			if(preg_match('/^([^ ]+) *= *(.*)$/', $line, $matches)) {
+			if (preg_match('/^([^ ]+) *= *(.*)$/', $line, $matches)) {
 				$details[$matches[1]] = $matches[2];
 			} else {
 				throw new Exception("Bad line '$line'");