silverstripe / sspak

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'");

src/sspak-sniffer.php

Spacing +12 added lines, -12 removed lines

@@ -7,13 +7,13 @@ 
  */
 
 // Argument parsing
-if(empty($_SERVER['argv'][1])) {
+if (empty($_SERVER['argv'][1])) {
 	echo "Usage: {$_SERVER['argv'][0]} (site-docroot)\n";
 	exit(1);
 }
 
 $basePath = $_SERVER['argv'][1];
-if($basePath[0] != '/') $basePath = getcwd() . '/' . $basePath;
+if ($basePath[0] != '/') $basePath = getcwd().'/'.$basePath;
 
 // SilverStripe bootstrap
 define('BASE_PATH', $basePath);
@@ -23,24 +23,24 @@ 
 $_SERVER['HTTP_HOST'] = 'localhost';
 chdir(BASE_PATH);
 
-if(file_exists(BASE_PATH.'/sapphire/core/Core.php')) {
+if (file_exists(BASE_PATH.'/sapphire/core/Core.php')) {
 	//SS 2.x
-	require_once(BASE_PATH . '/sapphire/core/Core.php');
-} else if(file_exists(BASE_PATH.'/framework/core/Core.php')) {
+	require_once(BASE_PATH.'/sapphire/core/Core.php');
+} else if (file_exists(BASE_PATH.'/framework/core/Core.php')) {
 	//SS 3.x
-	require_once(BASE_PATH. '/framework/core/Core.php');
-} else if(file_exists(BASE_PATH.'/framework/src/Core/Core.php')) {
+	require_once(BASE_PATH.'/framework/core/Core.php');
+} else if (file_exists(BASE_PATH.'/framework/src/Core/Core.php')) {
 	//SS 4.x
-	require_once(BASE_PATH. '/vendor/autoload.php');
-	require_once(BASE_PATH. '/framework/src/Core/Core.php');
+	require_once(BASE_PATH.'/vendor/autoload.php');
+	require_once(BASE_PATH.'/framework/src/Core/Core.php');
 } else {
-	echo "Couldn't locate framework's Core.php. Perhaps " . BASE_PATH . " is not a SilverStripe project?\n";
+	echo "Couldn't locate framework's Core.php. Perhaps ".BASE_PATH." is not a SilverStripe project?\n";
 	exit(2);
 }
 
 $output = array();
-foreach($databaseConfig as $k => $v) {
-	$output['db_' . $k] = $v;
+foreach ($databaseConfig as $k => $v) {
+	$output['db_'.$k] = $v;
 }
 $output['assets_path'] = ASSETS_PATH;