silverstripe / sspak

src/Args.php

Spacing +9 added lines, -9 removed lines

@@ -11,10 +11,10 @@ 
 	public function __construct($args) {
 		array_shift($args);
 
-		foreach($args as $arg) {
-			if(preg_match('/^--([^=]+)=(.*)$/', $arg, $matches)) {
+		foreach ($args as $arg) {
+			if (preg_match('/^--([^=]+)=(.*)$/', $arg, $matches)) {
 				$this->namedArgs[$matches[1]] = $matches[2];
-			} else if(preg_match('/^--([^=]+)$/', $arg, $matches)) {
+			} else if (preg_match('/^--([^=]+)$/', $arg, $matches)) {
 				$this->namedArgs[$matches[1]] = true;
 			} else {
 				$this->unnamedArgs[] = $arg;
@@ -51,8 +51,8 @@ 
 	 * Return the sudo argument, preferring a more specific one with the given optional prefix
 	 */
 	public function sudo($optionalPrefix) {
-		if(!empty($this->namedArgs[$optionalPrefix . '-sudo'])) return $this->namedArgs[$optionalPrefix . '-sudo'];
-		else if(!empty($this->namedArgs['sudo'])) return $this->namedArgs['sudo'];
+		if (!empty($this->namedArgs[$optionalPrefix.'-sudo'])) return $this->namedArgs[$optionalPrefix.'-sudo'];
+		else if (!empty($this->namedArgs['sudo'])) return $this->namedArgs['sudo'];
 		else return null;
 	}
 
@@ -62,18 +62,18 @@ 
 	public function pakParts() {
 		// Look up which parts of the sspak are going to be saved
 		$pakParks = array();
-		foreach(array('assets','db','git-remote') as $part) {
+		foreach (array('assets', 'db', 'git-remote') as $part) {
 			$pakParts[$part] = !empty($this->namedArgs[$part]);
 		}
 
 		// Default to db and assets
-		if(!array_filter($pakParts)) $pakParts = array('db' => true, 'assets' => true, 'git-remote' => true);
+		if (!array_filter($pakParts)) $pakParts = array('db' => true, 'assets' => true, 'git-remote' => true);
 		return $pakParts;
 	}
 
 	public function requireUnnamed($items) {
-		if(sizeof($this->unnamedArgs) < sizeof($items)) {
-			echo "Usage: {$_SERVER['argv'][0]} " . $this->action . " (";
+		if (sizeof($this->unnamedArgs) < sizeof($items)) {
+			echo "Usage: {$_SERVER['argv'][0]} ".$this->action." (";
 			echo implode(") (", $items);
 			echo ")\n";
 			throw new Exception('Arguments missing.');

Braces +10 added lines, -4 removed lines

@@ -51,9 +51,13 @@ 
 	 * Return the sudo argument, preferring a more specific one with the given optional prefix
 	 */
 	public function sudo($optionalPrefix) {
-		if(!empty($this->namedArgs[$optionalPrefix . '-sudo'])) return $this->namedArgs[$optionalPrefix . '-sudo'];
-		else if(!empty($this->namedArgs['sudo'])) return $this->namedArgs['sudo'];
-		else return null;
+		if(!empty($this->namedArgs[$optionalPrefix . '-sudo'])) {
+			return $this->namedArgs[$optionalPrefix . '-sudo'];
+		} else if(!empty($this->namedArgs['sudo'])) {
+			return $this->namedArgs['sudo'];
+		} else {
+			return null;
+		}
 	}
 
 	/**
@@ -67,7 +71,9 @@ 
 		}
 
 		// Default to db and assets
-		if(!array_filter($pakParts)) $pakParts = array('db' => true, 'assets' => true, 'git-remote' => true);
+		if(!array_filter($pakParts)) {
+			$pakParts = array('db' => true, 'assets' => true, 'git-remote' => true);
+		}
 		return $pakParts;
 	}
 

tests/DataExtractor/CsvTableReaderTest.php

Spacing +4 added lines, -4 removed lines

@@ -6,18 +6,18 @@
 {
 	public function testCsvReading() {
 
-		$csv = new CsvTableReader(__DIR__ . '/fixture/input.csv');
+		$csv = new CsvTableReader(__DIR__.'/fixture/input.csv');
 		$this->assertEquals(['Col1', 'Col2', 'Col3'], $csv->getColumns());
 
 		$extractedData = [];
-		foreach($csv as $record) {
+		foreach ($csv as $record) {
 			$extractedData[] = $record;
 		}
 
 		$this->assertEquals(
 			[
-				[ 'Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three' ],
-				[ 'Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?" ]
+				['Col1' => 'One', 'Col2' => 2, 'Col3' => 'Three'],
+				['Col1' => 'Hello, Sam', 'Col2' => 5, 'Col3' => "Nice to meet you\nWhat is your name?"]
 			],
 			$extractedData
 		);

src/sspak-sniffer.php

Braces +3 added lines, -1 removed lines

@@ -13,7 +13,9 @@
 }
 
 $basePath = $_SERVER['argv'][1];
-if($basePath[0] != '/') $basePath = getcwd() . '/' . $basePath;
+if($basePath[0] != '/') {
+	$basePath = getcwd() . '/' . $basePath;
+}
 
 // SilverStripe bootstrap
 define('BASE_PATH', realpath($basePath));

Spacing +11 added lines, -11 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', realpath($basePath));
@@ -23,15 +23,15 @@ 
 $_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.'/vendor/silverstripe/framework')) {
+	require_once(BASE_PATH.'/framework/core/Core.php');
+} else if (file_exists(BASE_PATH.'/vendor/silverstripe/framework')) {
 	//SS 4.x
-	require_once(BASE_PATH. '/vendor/autoload.php');
+	require_once(BASE_PATH.'/vendor/autoload.php');
 	$kernel = new SilverStripe\Core\CoreKernel(BASE_PATH);
 	//boot the parts of the kernel to populate the DB config
 	foreach (array('bootDatabaseEnvVars', 'bootDatabaseGlobals') as $bootMethod) {
@@ -41,13 +41,13 @@ 
 	}
 	$databaseConfig = SilverStripe\ORM\DB::getConfig();
 } 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;