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

Braces +3 added lines, -4 removed lines

@@ -4,18 +4,17 @@
  *
  * Adds customisable timezones to the nice method on {@link SS_Datetime}.
  */
-class SS_Datetimezone extends SS_Datetime
-{
+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”
     */
     public function Nice() {
-        if($timestamp = $this){
+        if($timestamp = $this) {
             //instantiate new DateTime object based off received timestamp in the default timezone
             $timestamp = new DateTime($timestamp, new DateTimeZone(date_default_timezone_get()));
             //if a user is logged in, has set a timezone that is in the allowed list and 
             //that timezone is NOT the default one then convert the timestamp to use the selected timezone
-            if(Member::currentUserID() && isset(Member::CurrentUser()->Timezone) && in_array(Member::CurrentUser()->Timezone, timezone_identifiers_list()) && Member::CurrentUser()->Timezone != date_default_timezone_get()){
+            if(Member::currentUserID() && isset(Member::CurrentUser()->Timezone) && in_array(Member::CurrentUser()->Timezone, timezone_identifiers_list()) && Member::CurrentUser()->Timezone != date_default_timezone_get()) {
                 $timestamp->setTimezone(new DateTimeZone(Member::CurrentUser()->Timezone));    
             }
         }

code/extensions/TimezoneMemberExtension.php

Braces +2 added lines, -4 removed lines

@@ -4,8 +4,7 @@ 
  *
  * Adds all available timezones as an optional field to SilverStripe {@link Member}.
  */
-class TimezoneMemberExtension extends DataExtension
-{
+class TimezoneMemberExtension extends DataExtension {
     private static $db = [
         'Timezone' => 'Varchar(255)',
     ];
@@ -15,8 +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',