silverstripe / deploynaut

code/backends/DeploymentStrategy.php

Braces +6 added lines, -2 removed lines

@@ -110,8 +110,12 @@
 		// Normalise "empty" values into dashes so comparisons are done properly.
 		// This means there is no diference between an empty string and a null
 		// but "0" is considered to be non-empty.
-		if(empty($from) && !strlen($from)) $from = '-';
-		if(empty($to) && !strlen($to)) $to = '-';
+		if(empty($from) && !strlen($from)) {
+			$from = '-';
+		}
+		if(empty($to) && !strlen($to)) {
+			$to = '-';
+		}
 
 		return $this->changes[$title] = array(
 			'from' => $from,

code/model/jobs/DNDeployment.php

Braces +6 added lines, -2 removed lines

@@ -205,12 +205,16 @@
 	 */
 	public function getFullDeployMessages() {
 		$strategy = $this->getDeploymentStrategy();
-		if ($strategy->getActionCode()!=='full') return null;
+		if ($strategy->getActionCode()!=='full') {
+			return null;
+		}
 
 		$changes = $strategy->getChangesModificationNeeded();
 		$messages = [];
 		foreach ($changes as $change => $details) {
-			if ($change==='Code version') continue;
+			if ($change==='Code version') {
+				continue;
+			}
 
 			$messages[] = [
 				'Flag' => sprintf(

code/model/StateMachineFactory.php

Braces +3 added lines, -1 removed lines

@@ -58,7 +58,9 @@
 
 	protected function addHandlers($machine) {
 		$class = get_class($machine->getObject());
-		if (empty($this->config()->handlers[$class])) return;
+		if (empty($this->config()->handlers[$class])) {
+			return;
+		}
 
 		$transitions = $this->config()->handlers[$class];
 		foreach ($transitions as $transName => $handlers) {

code/SS_Datetimezone.php

Indentation +2 added lines, -2 removed lines

@@ -6,8 +6,8 @@
  */
 class SS_Datetimezone extends SS_Datetime {
 	/**
-	* Returns the date in the raw SQL-format specific to a given timezone passed from the Member class, e.g. “2006-01-18 16:32:04”
-	*/
+	 * Returns the date in the raw SQL-format specific to a given timezone passed from the Member class, e.g. “2006-01-18 16:32:04”
+	 */
 	public function Nice() {
 		//instantiate new DateTime object based off received timestamp in the default timezone
 		$timestamp = new DateTime($this->value, new DateTimeZone(date_default_timezone_get()));

Braces +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 	* Returns the date in the raw SQL-format specific to a given timezone passed from the Member class, e.g. “2006-01-18 16:32:04”
 	*/
 	public function Format($format) {
-		if($this->value){
+		if($this->value) {
 			$date = new DateTime($this->value);
 			//if the current user has set a timezone that is not the default then use that
 			$member = Member::currentUser();

code/extensions/TimezoneMemberExtension.php

Braces +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 		return array_combine($timezones, $timezones);
 	}
 
-	public function updateCMSFields(FieldList $fields) {   
+	public function updateCMSFields(FieldList $fields) {
 		$fields->removeFieldFromTab('Root', 'Timezone');
 		$field = DropdownField::create(
 		  'Timezone',

code/api/DeploynautAPIFormatter.php

Indentation -1 removed lines

@@ -117,7 +117,6 @@
 
 	/**
 	 * Return data about a particular {@link Member} of the stack for use in API response.
-
 	 * Notes:
 	 * 1) This method returns null instead of an array if the member doesn't exists anymore
 	 * 2) 'role' can be null in the response. This is the case of an admin, or an operations