silverstripe / deploynaut

code/api/nouns/APIProject.php

Spacing +8 added lines, -8 removed lines

@@ -15,11 +15,11 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	public function index(SS_HTTPRequest $request) {
-		if(!$this->record->canView($this->getMember())) {
+		if (!$this->record->canView($this->getMember())) {
 			return $this->message('You are not authorized to this environment', 403);
 		}
 
-		switch($request->httpMethod()) {
+		switch ($request->httpMethod()) {
 			case 'GET':
 				$response = array(
 					"name" => $this->record->Name,
@@ -29,7 +29,7 @@ 
 					"disk-quota-mb" => $this->record->DiskQuotaMB,
 					"environments" => array(),
 				);
-				foreach($this->record->DNEnvironmentList() as $environment) {
+				foreach ($this->record->DNEnvironmentList() as $environment) {
 					$response['environments'][] = array(
 						'name' => $environment->Name,
 						'href' => Director::absoluteURL($this->record->APILink($environment->Name)),
@@ -47,10 +47,10 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	public function fetch(SS_HTTPRequest $request) {
-		if(!$this->record->canView($this->getMember())) {
+		if (!$this->record->canView($this->getMember())) {
 			return $this->message('You are not authorized to do that on this environment', 403);
 		}
-		switch($request->httpMethod()) {
+		switch ($request->httpMethod()) {
 			case 'GET':
 				return $this->getFetch($this->getRequest()->param('ID'));
 			case 'POST':
@@ -66,7 +66,7 @@ 
 	 */
 	protected function getFetch($ID) {
 		$ping = DNGitFetch::get()->byID($ID);
-		if(!$ping) {
+		if (!$ping) {
 			return $this->message('Fetch not found', 404);
 		}
 		$output = array(
@@ -87,9 +87,9 @@ 
 		$fetch->write();
 		$fetch->start();
 
-		$location = Director::absoluteBaseURL() . $this->Link() . '/fetch/' . $fetch->ID;
+		$location = Director::absoluteBaseURL().$this->Link().'/fetch/'.$fetch->ID;
 		$output = array(
-			'message' => 'Fetch queued as job ' . $fetch->ResqueToken,
+			'message' => 'Fetch queued as job '.$fetch->ResqueToken,
 			'href' => $location,
 		);
 

code/api/nouns/APIEnvironment.php

Spacing +16 added lines, -16 removed lines

@@ -15,10 +15,10 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	public function index(SS_HTTPRequest $request) {
-		if(!$this->record->canView($this->getMember())) {
+		if (!$this->record->canView($this->getMember())) {
 			return $this->message('You are not authorized to view this environment', 403);
 		}
-		switch($request->httpMethod()) {
+		switch ($request->httpMethod()) {
 			case 'GET':
 				$href = Director::absoluteURL($this->record->Project()->APILink($this->record->Name));
 				return $this->getAPIResponse(array(
@@ -51,10 +51,10 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	public function ping(SS_HTTPRequest $request) {
-		if(!$this->record->canView($this->getMember())) {
+		if (!$this->record->canView($this->getMember())) {
 			return $this->message('You are not authorized to do that on this environment', 403);
 		}
-		switch($request->httpMethod()) {
+		switch ($request->httpMethod()) {
 			case 'GET':
 				return $this->getPing($this->getRequest()->param('ID'));
 			case 'POST':
@@ -71,10 +71,10 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	public function deploy(SS_HTTPRequest $request) {
-		if(!$this->record->canView($this->getMember())) {
+		if (!$this->record->canView($this->getMember())) {
 			return $this->message('You are not authorized to do that on this environment', 403);
 		}
-		switch($request->httpMethod()) {
+		switch ($request->httpMethod()) {
 			case 'GET':
 				return $this->getDeploy($this->getRequest()->param('ID'));
 			case 'POST':
@@ -106,7 +106,7 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	protected function createPing() {
-		if(!$this->record->canDeploy($this->getMember())) {
+		if (!$this->record->canDeploy($this->getMember())) {
 			return $this->message('You are not authorized to do that on this environment', 403);
 		}
 		$ping = DNPing::create();
@@ -114,9 +114,9 @@ 
 		$ping->write();
 		$ping->start();
 
-		$location = Director::absoluteBaseURL() . $this->Link() . '/ping/' . $ping->ID;
+		$location = Director::absoluteBaseURL().$this->Link().'/ping/'.$ping->ID;
 		$output = array(
-			'message' => 'Ping queued as job ' . $ping->ResqueToken,
+			'message' => 'Ping queued as job '.$ping->ResqueToken,
 			'href' => $location,
 		);
 
@@ -132,7 +132,7 @@ 
 	 */
 	protected function getPing($ID) {
 		$ping = DNPing::get()->byID($ID);
-		if(!$ping) {
+		if (!$ping) {
 			return $this->message('Ping not found', 404);
 		}
 		$output = array(
@@ -149,17 +149,17 @@ 
 	 * @return SS_HTTPResponse
 	 */
 	protected function createDeploy() {
-		if(!$this->record->canDeploy($this->getMember())) {
+		if (!$this->record->canDeploy($this->getMember())) {
 			return $this->message('You are not authorized to do that on this environment', 403);
 		}
 
 		$reqBody = $this->getRequestBody();
 
-		if($reqBody === null) {
+		if ($reqBody === null) {
 			return $this->message('the request body did not contain a valid JSON object.', 400);
 		}
 
-		if(empty($reqBody['release'])) {
+		if (empty($reqBody['release'])) {
 			return $this->message('deploy requires a {"release": "sha1"} in the body of the request.', 400);
 		}
 
@@ -168,9 +168,9 @@ 
 		));
 		$deploy = $strategy->createDeployment();
 		$deploy->start();
-		$location = Director::absoluteBaseURL() . $this->Link() . '/deploy/' . $deploy->ID;
+		$location = Director::absoluteBaseURL().$this->Link().'/deploy/'.$deploy->ID;
 		$output = array(
-			'message' => 'Deploy queued as job ' . $deploy->ResqueToken,
+			'message' => 'Deploy queued as job '.$deploy->ResqueToken,
 			'href' => $location,
 		);
 		$response = $this->getAPIResponse($output);
@@ -187,7 +187,7 @@ 
 	 */
 	protected function getDeploy($id) {
 		$deploy = DNDeployment::get()->byID($id);
-		if(!$deploy) {
+		if (!$deploy) {
 			return $this->message('Deploy not found', 404);
 		}
 		$output = array(

code/api/DeploynautAPI.php

Spacing +8 added lines, -8 removed lines

@@ -51,12 +51,12 @@ 
 			'projects' => array(),
 		);
 
-		if($request->httpMethod() != 'GET') {
+		if ($request->httpMethod() != 'GET') {
 			return $this->message('API not found', 404);
 		}
 
-		foreach(DNProject::get() as $item) {
-			if($item->canView($this->getMember())) {
+		foreach (DNProject::get() as $item) {
+			if ($item->canView($this->getMember())) {
 				$response['projects'][] = array(
 					"name" => $item->Name,
 					"href" => Director::absoluteURL($item->APILink("")),
@@ -75,7 +75,7 @@ 
 	 */
 	public function project(SS_HTTPRequest $request) {
 		$project = $this->getProject();
-		if(!$project) {
+		if (!$project) {
 			return $this->project404Response();
 		}
 		return new APIProject($this, $project);
@@ -89,12 +89,12 @@ 
 	 */
 	public function environment(SS_HTTPRequest $request) {
 		$project = $this->getProject();
-		if(!$project) {
+		if (!$project) {
 			return $this->project404Response();
 		}
 
 		$environment = $this->getEnvironment();
-		if(!$environment) {
+		if (!$environment) {
 			return $this->environment404Response();
 		}
 		return new APIEnvironment($this, $environment);
@@ -105,7 +105,7 @@ 
 	 */
 	protected function project404Response() {
 		$projectName = Convert::raw2xml($this->getRequest()->latestParam('Project'));
-		return new SS_HTTPResponse('Project "' . $projectName . '" not found.', 404);
+		return new SS_HTTPResponse('Project "'.$projectName.'" not found.', 404);
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 	 */
 	protected function environment404Response() {
 		$envName = Convert::raw2xml($this->getRequest()->latestParam('Environment'));
-		return new SS_HTTPResponse('Environment "' . $envName . '" not found.', 404);
+		return new SS_HTTPResponse('Environment "'.$envName.'" not found.', 404);
 	}
 
 	/**

code/tasks/MigrateDeploymentStatusTask.php

Spacing +2 added lines, -2 removed lines

@@ -4,8 +4,8 @@
 
 	public function run($request) {
 		$log = function($message) {
-			$message = sprintf('[%s] ', date('Y-m-d H:i:s')) . $message;
-			echo $message . PHP_EOL;
+			$message = sprintf('[%s] ', date('Y-m-d H:i:s')).$message;
+			echo $message.PHP_EOL;
 		};
 
 		if (!Director::is_cli()) {

code/model/GitonomyCache.php

Spacing +2 added lines, -2 removed lines

@@ -7,8 +7,8 @@
 	public static $cache = array();
 
 	public static function getIncludingBranches($commit) {
-		$cacheKey = 'getIncludingBranches-' . $commit->getRepository()->getPath() . '-' . $commit->gethash();
-		if(!isset(self::$cache[$cacheKey])) {
+		$cacheKey = 'getIncludingBranches-'.$commit->getRepository()->getPath().'-'.$commit->gethash();
+		if (!isset(self::$cache[$cacheKey])) {
 			self::$cache[$cacheKey] = $commit->getIncludingBranches();
 		}
 		return self::$cache[$cacheKey];

code/model/DNBranchList.php

Spacing +9 added lines, -9 removed lines

@@ -58,20 +58,20 @@ 
 
 		try {
 			$repository = new Gitonomy\Git\Repository($this->project->getLocalCVSPath());
-		} catch(Exception $e) {
+		} catch (Exception $e) {
 			return $branches;
 		}
 
-		foreach($repository->getReferences()->getBranches() as $branch) {
+		foreach ($repository->getReferences()->getBranches() as $branch) {
 			/** @var DNBranch $obj */
 			$obj = DNBranch::create($branch, $this->project, $this->data);
-			if($branch->getName() == 'master') {
+			if ($branch->getName() == 'master') {
 				$firstBranch = array($branch->getName() => $obj);
 			} else {
 				$branches[$branch->getName()] = $obj;
 			}
 		}
-		if($firstBranch) {
+		if ($firstBranch) {
 			$branches = $firstBranch + $branches;
 		}
 
@@ -86,11 +86,11 @@ 
 	 * @return string
 	 */
 	public function byName($name) {
-		if($this->loaded === false) {
+		if ($this->loaded === false) {
 			$this->getIterator();
 		}
 
-		if(isset($this->items[$name])) {
+		if (isset($this->items[$name])) {
 			return $this->items[$name];
 		}
 
@@ -104,12 +104,12 @@ 
 	 * @return ArrayIterator
 	 */
 	public function getIterator() {
-		if($this->loaded === false) {
+		if ($this->loaded === false) {
 			$this->items = $this->getReferences();
 			$this->loaded = true;
 		}
-		foreach($this->items as $i => $item) {
-			if(is_array($item)) {
+		foreach ($this->items as $i => $item) {
+			if (is_array($item)) {
 				$this->items[$i] = new ArrayData($item);
 			}
 		}

code/model/jobs/DNCreateEnvironment.php

Spacing +9 added lines, -9 removed lines

@@ -68,7 +68,7 @@ 
 	 * @return string
 	 */
 	public function LogLink() {
-		return $this->Link() . '/log';
+		return $this->Link().'/log';
 	}
 
 	/**
@@ -110,9 +110,9 @@ 
 		$status = new Resque_Job_Status($this->ResqueToken);
 		$statusCode = $status->get();
 		// The Resque job can no longer be found, fallback to the DNDeployment.Status
-		if($statusCode === false) {
+		if ($statusCode === false) {
 			// Translate from the DNDeployment.Status to the Resque job status for UI purposes
-			switch($this->Status) {
+			switch ($this->Status) {
 				case 'Finished':
 					return 'Complete';
 				case 'Started':
@@ -139,11 +139,11 @@ 
 			'projectName' => $project->Name
 		);
 
-		if(!$this->CreatorID) {
+		if (!$this->CreatorID) {
 			$this->CreatorID = Member::currentUserID();
 		}
 
-		if($this->CreatorID) {
+		if ($this->CreatorID) {
 			$creator = $this->Creator();
 			$message = sprintf(
 				'Environment creation for project %s initiated by %s (%s), with IP address %s',
@@ -171,7 +171,7 @@ 
 
 	public function createEnvironment() {
 		$backend = $this->getBackend();
-		if($backend) {
+		if ($backend) {
 			return $backend->createEnvironment($this);
 		}
 		throw new Exception("Unable to find backend.");
@@ -185,12 +185,12 @@ 
 	 */
 	public function getBackend() {
 		$data = unserialize($this->Data);
-		if(isset($data['EnvironmentType']) && class_exists($data['EnvironmentType'])) {
+		if (isset($data['EnvironmentType']) && class_exists($data['EnvironmentType'])) {
 			$env = Injector::inst()->get($data['EnvironmentType']);
-			if($env instanceof EnvironmentCreateBackend) {
+			if ($env instanceof EnvironmentCreateBackend) {
 				return $env;
 			} else {
-				throw new Exception("Invalid backend: " . $data['EnvironmentType']);
+				throw new Exception("Invalid backend: ".$data['EnvironmentType']);
 			}
 		}
 		return null;

code/model/jobs/DNDataTransfer.php

Spacing +13 added lines, -13 removed lines

@@ -112,7 +112,7 @@ 
 	}
 
 	public function getTitle() {
-		return $this->dbObject('Created')->Nice() . " (Status: {$this->Status})";
+		return $this->dbObject('Created')->Nice()." (Status: {$this->Status})";
 	}
 
 	public function Link() {
@@ -169,11 +169,11 @@ 
 			'backupBeforePush' => $this->backupBeforePush
 		);
 
-		if(!$this->AuthorID) {
+		if (!$this->AuthorID) {
 			$this->AuthorID = Member::currentUserID();
 		}
 
-		if($this->AuthorID) {
+		if ($this->AuthorID) {
 			$author = $this->Author();
 			$message = sprintf(
 				'Data transfer on %s (%s, %s) initiated by %s (%s), with IP address %s',
@@ -231,23 +231,23 @@ 
 
 	public function getDescription() {
 		$envName = $this->Environment()->getFullName();
-		if($this->Direction == 'get') {
-			if($this->Origin == 'ManualUpload') {
-				$description = 'Manual upload of ' . $this->getModeNice() . ' to ' . $envName;
-			} elseif($this->IsBackupDataTransfer()) {
-				$description = 'Automated backup of ' . $this->getModeNice() . ' from ' . $envName;
+		if ($this->Direction == 'get') {
+			if ($this->Origin == 'ManualUpload') {
+				$description = 'Manual upload of '.$this->getModeNice().' to '.$envName;
+			} elseif ($this->IsBackupDataTransfer()) {
+				$description = 'Automated backup of '.$this->getModeNice().' from '.$envName;
 			} else {
-				$description = 'Backup of ' . $this->getModeNice() . ' to ' . $envName;
+				$description = 'Backup of '.$this->getModeNice().' to '.$envName;
 			}
 		} else {
-			$description = 'Restore ' . $this->getModeNice() . ' to ' . $envName;
+			$description = 'Restore '.$this->getModeNice().' to '.$envName;
 		}
 
 		return $description;
 	}
 
 	public function getModeNice() {
-		if($this->Mode == 'all') {
+		if ($this->Mode == 'all') {
 			return 'database and assets';
 		} else {
 			return $this->Mode;
@@ -287,9 +287,9 @@ 
 		$status = new Resque_Job_Status($this->ResqueToken);
 		$statusCode = $status->get();
 		// The Resque job can no longer be found, fallback to the DNDataTransfer.Status
-		if($statusCode === false) {
+		if ($statusCode === false) {
 			// Translate from the DNDataTransfer.Status to the Resque job status for UI purposes
-			switch($this->Status) {
+			switch ($this->Status) {
 				case 'Finished':
 					return 'Complete';
 				case 'Started':

code/model/jobs/DNGitFetch.php

Spacing +4 added lines, -4 removed lines

@@ -54,11 +54,11 @@ 
 		$project = $this->Project();
 		$log = $this->log();
 
-		if(!$this->DeployerID) {
+		if (!$this->DeployerID) {
 			$this->DeployerID = Member::currentUserID();
 		}
 
-		if($this->DeployerID) {
+		if ($this->DeployerID) {
 			$deployer = $this->Deployer();
 			$message = sprintf(
 				'Update repository job for %s initiated by %s (%s)',
@@ -131,9 +131,9 @@ 
 		$status = new Resque_Job_Status($this->ResqueToken);
 		$statusCode = $status->get();
 		// The Resque job can no longer be found, fallback to the DNDeployment.Status
-		if($statusCode === false) {
+		if ($statusCode === false) {
 			// Translate from the DNDeployment.Status to the Resque job status for UI purposes
-			switch($this->Status) {
+			switch ($this->Status) {
 				case 'Finished':
 					return 'Complete';
 				case 'Started':