newicon / neon

neon/core/form/fields/SelectDynamic.php

Braces +9 added lines, -6 removed lines

@@ -96,8 +96,9 @@ 
 	public function getItems()
 	{
 		// if we haven't or can't map a map request we can't have any values
-		if (empty($this->dataMapKey))
-			return [];
+		if (empty($this->dataMapKey)) {
+					return [];
+		}
 
 		$provider = $this->getProviderService();
 		$format = false;
@@ -119,8 +120,9 @@ 
 			$concat = isset($format['concat']) ? $format['concat'] : ' ';
 			$prefix = isset($format['prefix']) ? $format['prefix'] : '';
 			$postfix = isset($format['postfix']) ? $format['postfix'] : '';
-			foreach ($items as $k=>$i)
-				$items[$k] = $prefix.implode($concat, is_array($i) ? $i : [$i]).$postfix;
+			foreach ($items as $k=>$i) {
+							$items[$k] = $prefix.implode($concat, is_array($i) ? $i : [$i]).$postfix;
+			}
 		}
 		return $items;
 	}
@@ -184,8 +186,9 @@ 
 	public function setValueFromDb($value)
 	{
 		parent::setValueFromDb($value);
-		if (!empty($value))
-			$this->makeMapRequest($value);
+		if (!empty($value)) {
+					$this->makeMapRequest($value);
+		}
 		return $this;
 	}
 

neon/core/form/fields/el/DateRange.php

Braces +3 added lines, -2 removed lines

@@ -31,8 +31,9 @@
 	public function getValue()
 	{
 		$value = parent::getValue();
-		if (is_array($value))
-			return $value;
+		if (is_array($value)) {
+					return $value;
+		}
 		return ['from' => '', 'to' => ''];
 	}
 

neon/core/form/fields/UrlLink.php

Braces +6 added lines, -4 removed lines

@@ -61,8 +61,9 @@ 
 			// disallow any quotes in the URL to prevent a class of attacks
 			if (strpos($value, "'") === false && strpos($value,'"') === false) {
 				// prefix with the schema if not provided
-				if ($value && $this->absolute && strpos($value, 'http')===false)
-					$value = str_replace('///','//','https://'.$value);
+				if ($value && $this->absolute && strpos($value, 'http')===false) {
+									$value = str_replace('///','//','https://'.$value);
+				}
 				// now turn into a link and run through purify to strip
 				// out any dodgy shenanigans
 				$link = '<a href="'.$value.'">';
@@ -71,8 +72,9 @@ 
 					$exploded = explode('"',$safeValue);
 					$this->_sanitisedValue = $exploded[1];
 				}
-			} else
-				$this->_sanitisedValue = '';
+			} else {
+							$this->_sanitisedValue = '';
+			}
 		}
 		return $this->_sanitisedValue;
 	}

neon/core/form/fields/Wysiwyg.php

Braces +3 added lines, -2 removed lines

@@ -81,8 +81,9 @@
 		$fullConfig = array_merge_recursive($defaultConfigSets, $additionalConfigSets);
 		// convert the extraPlugins into a comma separated list as required by ckeditor
 		foreach ($fullConfig as $set=>$config) {
-			if (isset($config['extraPlugins']))
-				$fullConfig[$set]['extraPlugins'] = implode(',', $config['extraPlugins']);
+			if (isset($config['extraPlugins'])) {
+							$fullConfig[$set]['extraPlugins'] = implode(',', $config['extraPlugins']);
+			}
 		}
 		return json_encode($fullConfig);
 	}

neon/core/form/fields/filters/File.php

Braces +3 added lines, -2 removed lines

@@ -26,7 +26,8 @@
 	{
 		// do a query
 		$ids = neon()->db->query()->from('firefly_file_manager')->select('uuid')->where(['like', 'name', $searchData])->column();
-		if (!empty($ids))
-			$query->where($this->getDataKey(), '=', $ids);
+		if (!empty($ids)) {
+					$query->where($this->getDataKey(), '=', $ids);
+		}
 	}
 }
\ No newline at end of file

neon/core/form/traits/BuilderTrait.php

Braces +6 added lines, -4 removed lines

@@ -352,8 +352,9 @@ 
 	 */
 	public function makeField($field, $defaults=[])
 	{
-		if (is_array($field))
-			$field = $this->createFieldObjectFromConfig($field, $defaults);
+		if (is_array($field)) {
+					$field = $this->createFieldObjectFromConfig($field, $defaults);
+		}
 
 		if (!($field instanceof IField || $field instanceof Form)) {
 			// add this form as the parent form reference
@@ -422,8 +423,9 @@ 
 	 */
 	public function getFieldClassByAlias($alias)
 	{
-		if (!$this->fieldAliasExists($alias))
-			throw new \InvalidArgumentException("There is no alias with the key '$alias' in the Forms fieldAliases array");
+		if (!$this->fieldAliasExists($alias)) {
+					throw new \InvalidArgumentException("There is no alias with the key '$alias' in the Forms fieldAliases array");
+		}
 		return $this->fieldAliases[strtolower($alias)];
 	}
 

neon/core/form/Deprecated.php

Braces +3 added lines, -2 removed lines

@@ -180,8 +180,9 @@
 		}
 
 		// ignore if this is a deleted field
-		if ($field->deleted == 1)
-			return;
+		if ($field->deleted == 1) {
+					return;
+		}
 
 		// check if member exists
 		$memberRef = $field->getDdsName();

neon/core/form/FormRepeater.php

Braces +15 added lines, -10 removed lines

@@ -60,8 +60,9 @@ 
 	public function loadFromDb($data)
 	{
 		foreach ($data as $name => $value) {
-			if (!$this->hasField($name))
-				$this->addInstance($name);
+			if (!$this->hasField($name)) {
+							$this->addInstance($name);
+			}
 			$this->getField($name)->setValueFromDb($value);
 		}
 	}
@@ -102,8 +103,9 @@ 
 	{
 		// throw an exception - should probably cope with this scenario but set
 		// a validation to fail rather than throwing toys out of the pram
-		if (!is_array($value))
-			throw new \InvalidArgumentException('The value should be an array');
+		if (!is_array($value)) {
+					throw new \InvalidArgumentException('The value should be an array');
+		}
 
 		foreach($value as $position => $values) {
 			if (!$this->hasField($position)) {
@@ -134,8 +136,9 @@ 
 	public function registerScripts($view=null, $mount=true)
 	{
 		$view = ($view === null) ? $this->getView() : $view;
-		foreach ($this->getTemplate()->getFields() as $field)
-			$field->registerScripts($view, $mount);
+		foreach ($this->getTemplate()->getFields() as $field) {
+					$field->registerScripts($view, $mount);
+		}
 	}
 
 	/**
@@ -240,16 +243,18 @@ 
 	{
 		// the root path bit can be the the name or the id.
 		$bits = explode('.', $path);
-		if (!($bits[0] === $this->name))
-			throw new \Exception("The first part of the path should be the current repeater name. Repeater form name=$this->name, id=$this->id, path=$path.");
+		if (!($bits[0] === $this->name)) {
+					throw new \Exception("The first part of the path should be the current repeater name. Repeater form name=$this->name, id=$this->id, path=$path.");
+		}
 
 		// if the repeater has an existing instance - then lets return that
 		if ($this->hasField($bits[1])) {
 			$field = $this->getField($bits[1]);
 			// fields within repeaters should always be forms - so this should never happen
 			// but just to ensure we trap the error - lets throw an exception
-			if (!$field->isForm())
-				throw new \InvalidArgumentException('FormRepeaters should only contain form instances.');
+			if (!$field->isForm()) {
+							throw new \InvalidArgumentException('FormRepeaters should only contain form instances.');
+			}
 			array_splice($bits, 0, 1);
 			return $field->getFieldByPath(implode('.', $bits));
 		}

neon/core/view/MarkdownNeon.php

Braces +6 added lines, -4 removed lines

@@ -83,8 +83,9 @@ 
 		$plugin = $block['plugin'];
 		$content = implode("\n",$block['content']);
 		$attributes = $block['attributes'];
-		if (class_exists($plugin))
-			return $plugin::widget(array_merge($attributes, ['content' => $content]));
+		if (class_exists($plugin)) {
+					return $plugin::widget(array_merge($attributes, ['content' => $content]));
+		}
 		return '<div class="alert alert-danger">unknown plugin: &quot;' .  $plugin . '&quot;</div>';
 	}
 
@@ -151,8 +152,9 @@ 
 		for ($i = $current + 1, $count = count($lines); $i < $count; $i++) {
 			$line = $lines[$i];
 			// stop consuming lines if we find the closing tag
-			if (strpos($line, $closingTag) !== false)
-				break;
+			if (strpos($line, $closingTag) !== false) {
+							break;
+			}
 			$content[] = $line;
 		}
 		return $content;