newicon / neon

neon/core/grid/column/Button.php

Braces +6 added lines, -4 removed lines

@@ -94,10 +94,12 @@
 		if (count($this->_buttons)) {
 			$buttons = [];
 			foreach ($this->_buttons as $buttonKey => $btnOps) {
-				if (!isset($btnOps['data']))
-					$btnOps['data'] = [];
-				if ($this->getGrid()->hasIndexColumn())
-					$btnOps['data'] = ['index' => $model[$this->getGrid()->getIndexColumn()->getKey()] ];
+				if (!isset($btnOps['data'])) {
+									$btnOps['data'] = [];
+				}
+				if ($this->getGrid()->hasIndexColumn()) {
+									$btnOps['data'] = ['index' => $model[$this->getGrid()->getIndexColumn()->getKey()] ];
+				}
 				$func = 'buttonCustom';
 				if ($buttonKey === 'delete') {
 					$func = 'buttonDelete';

neon/core/helpers/Html.php

Braces +26 added lines, -18 removed lines

@@ -154,8 +154,9 @@ 
 			self::processTags($allowableTags, $tagsProcessed);
 		}
 		if (is_array($value)) {
-			foreach ($value as $k => $v)
-				$value[$k] = static::sanitise($v, $allowableTags);
+			foreach ($value as $k => $v) {
+							$value[$k] = static::sanitise($v, $allowableTags);
+			}
 		} else {
 			if (!is_null($value)) {
 				if (isset($tagsProcessed[$allowableTags])) {
@@ -189,14 +190,16 @@ 
 	{
 		profile_begin('Html::purify', 'html');
 		if (is_array($value)) {
-			foreach ($value as $k => $v)
-				$value[$k] = static::purify($v, $allowedTags);
+			foreach ($value as $k => $v) {
+							$value[$k] = static::purify($v, $allowedTags);
+			}
 		} else {
 			if (!is_null($value)) {
-				if ($allowedTags === false)
-					$value = trim(HtmlPurifier::process($value));
-				else
-					$value = trim(HtmlPurifier::process(strip_tags($value, $allowedTags)));
+				if ($allowedTags === false) {
+									$value = trim(HtmlPurifier::process($value));
+				} else {
+									$value = trim(HtmlPurifier::process(strip_tags($value, $allowedTags)));
+				}
 			}
 		}
 		profile_end('Html::purify', 'html');
@@ -214,8 +217,9 @@ 
 	public static function encodeEntities($value, $entities = ENT_QUOTES)
 	{
 		if (is_array($value)) {
-			foreach ($value as $k => $v)
-				$value[$k] = static::encodeEntities($v, $entities);
+			foreach ($value as $k => $v) {
+							$value[$k] = static::encodeEntities($v, $entities);
+			}
 		} else {
 			$value = htmlentities($value, $entities);
 		}
@@ -251,18 +255,22 @@ 
 	 */
 	public static function highlight($search, $html, $empty='')
 	{
-		if ($html === null)
-			return neon()->formatter->nullDisplay;
-		if ($html === '')
-			return $empty;
-		if (empty($search))
-			return $html;
+		if ($html === null) {
+					return neon()->formatter->nullDisplay;
+		}
+		if ($html === '') {
+					return $empty;
+		}
+		if (empty($search)) {
+					return $html;
+		}
 
 		profile_begin('Html::Highlight');
 		// escape search and html entities as otherwise the loadHTML or appendXML
 		// can blow up if there are < & > in the data
-		if (!is_array($search))
-			$search = [$search];
+		if (!is_array($search)) {
+					$search = [$search];
+		}
 		try {
 			foreach ($search as $s) {
 				// now search and rebuild the data

neon/phoebe/services/adapters/appforms/PhoebeObject.php

Braces +33 added lines, -22 removed lines

@@ -41,8 +41,9 @@ 
 	public function editObject($changes)
 	{
 		// check if there's anything to do
-		if (empty($changes) || $changes == $this->data)
-			return true;
+		if (empty($changes) || $changes == $this->data) {
+					return true;
+		}
 
 		// extract the data and save appropriately to
 		// the associated database tables ... if any
@@ -59,8 +60,9 @@ 
 	public function setDataSources($sources)
 	{
 		foreach ($sources as $key=>$value) {
-			if (empty($value))
-				unset($sources[$key]);
+			if (empty($value)) {
+							unset($sources[$key]);
+			}
 		}
 		$this->dataSourceIds = $sources;
 	}
@@ -97,8 +99,9 @@ 
 	 */
 	protected function getDataSources()
 	{
-		foreach ($this->dataSourceIds as $key=>$id)
-			$this->dataSourceObjects[$key] = $this->getDds()->getObject($id);
+		foreach ($this->dataSourceIds as $key=>$id) {
+					$this->dataSourceObjects[$key] = $this->getDds()->getObject($id);
+		}
 	}
 
 	/**
@@ -118,8 +121,9 @@ 
 				$object = array_merge($this->getAllowedFieldsForClass($object['_classType']), $object);
 				// now update the object with the data from daedalus
 				$data = $dds->getObject($object['_uuid']);
-				if ($data)
-					$object = array_replace($object, array_intersect_key($data, $object));
+				if ($data) {
+									$object = array_replace($object, array_intersect_key($data, $object));
+				}
 			}
 			// recursively update if this is in a tree structure
 			if ($isTree && is_array($object)) {
@@ -179,8 +183,9 @@ 
 				}
 			}
 		}
-		if (count($relations))
-			$this->saveLinkedObjects($relations);
+		if (count($relations)) {
+					$this->saveLinkedObjects($relations);
+		}
 	}
 
 	/**
@@ -216,8 +221,9 @@ 
 							$relations = $renderables[$subRenderable]['relations'];
 							foreach ($relations as $relation) {
 								$memberRef = $relation['memberRef'];
-								if (!isset($extracted['__relations'][$objId][$memberRef]))
-									$extracted['__relations'][$objId][$memberRef] = [];
+								if (!isset($extracted['__relations'][$objId][$memberRef])) {
+																	$extracted['__relations'][$objId][$memberRef] = [];
+								}
 								$extracted['__relations'][$objId][$memberRef] = array_merge($extracted['__relations'][$objId][$memberRef], $newObjectIds);
 							}
 						}
@@ -259,8 +265,9 @@ 
 		if (empty($fieldsForClass[$classType])) {
 			$definition = $this->getDefinition();
 			foreach ($definition['renderables'] as $r) {
-				if ($r['type'] == 'MemberComponent')
-					$fieldsForClass[$r['class_type']][$r['member_ref']] = null;
+				if ($r['type'] == 'MemberComponent') {
+									$fieldsForClass[$r['class_type']][$r['member_ref']] = null;
+				}
 			}
 		}
 		return $fieldsForClass[$classType];
@@ -322,8 +329,9 @@ 
 			if (isset($renderables[$phoebeKey])) {
 				// check the renderable exists and is a class component
 				$ren = $renderables[$phoebeKey];
-				if ($ren['type'] != 'ClassComponent')
-					continue;
+				if ($ren['type'] != 'ClassComponent') {
+									continue;
+				}
 
 				// create the object and any required data params
 				$object = [
@@ -334,8 +342,9 @@ 
 				foreach ($ren['items'] as $item) {
 					if (isset($renderables[$item])) {
 						$iRen = $renderables[$item];
-						if ($iRen['type'] != 'MemberComponent')
-							continue;
+						if ($iRen['type'] != 'MemberComponent') {
+													continue;
+						}
 						$object[$iRen['member_ref']] = null;
 					}
 				}
@@ -455,8 +464,9 @@ 
 	private $_class = null;
 	private function getClass()
 	{
-		if (!$this->_class)
-			$this->_class = neon('phoebe')->getIPhoebeType($this->phoebeType)->getClass($this->classType);
+		if (!$this->_class) {
+					$this->_class = neon('phoebe')->getIPhoebeType($this->phoebeType)->getClass($this->classType);
+		}
 		return $this->_class;
 	}
 
@@ -469,8 +479,9 @@ 
 	private $_dds = null;
 	private function getDds()
 	{
-		if (!$this->_dds)
-			$this->_dds = neon('dds')->getIDdsObjectManagement();
+		if (!$this->_dds) {
+					$this->_dds = neon('dds')->getIDdsObjectManagement();
+		}
 		return $this->_dds;
 	}