newicon / neon

neon/core/form/fields/Field.php

Spacing +8 added lines, -8 removed lines

@@ -190,7 +190,7 @@ 
 	 * @param array $config
 	 * @throws \neon\core\form\exceptions\InvalidNameFormat
 	 */
-	public function __construct($name=[], array $config=[])
+	public function __construct($name = [], array $config = [])
 	{
 		// The $name field can also be a configuration array
 		if (is_array($name)) {
@@ -515,7 +515,7 @@ 
 	 */
 	public function getLabelHtml()
 	{
-		return $this->getLabel() ? Html::activeLabel($this->getForm(), $this->getName(), $this->labelAttributes ) : '';
+		return $this->getLabel() ? Html::activeLabel($this->getForm(), $this->getName(), $this->labelAttributes) : '';
 	}
 
 	/**
@@ -597,7 +597,7 @@ 
 	public function validate()
 	{
 		$validators = $this->getValidators();
-		if (count($validators)==0)
+		if (count($validators) == 0)
 			return true;
 
 		$isEmpty = empty($this->getValue());
@@ -625,10 +625,10 @@ 
 					$validator->validateAttribute($form, $attribute);
 				} catch (\yii\base\UnknownPropertyException $e) {
 					throw new UnknownPropertyException(
-						"You are using a validator " . get_class($validator) . " that has a when clause or needs to use validateAttribute. "
+						"You are using a validator ".get_class($validator)." that has a when clause or needs to use validateAttribute. "
 						. "This requires you to have <br><br>"
-						. "public function get" . ucfirst($attribute) . "<br>{ <br>&nbsp;&nbsp;return \$this->getField(&quot;$attribute&quot;)->getValue()); <br>}"
-						. "<br><br>on the form. You may also need the equivalent set" . ucfirst($attribute) . " method. <br>Error was " . $e->getMessage());
+						. "public function get".ucfirst($attribute)."<br>{ <br>&nbsp;&nbsp;return \$this->getField(&quot;$attribute&quot;)->getValue()); <br>}"
+						. "<br><br>on the form. You may also need the equivalent set".ucfirst($attribute)." method. <br>Error was ".$e->getMessage());
 				}
 			}
 		}
@@ -776,7 +776,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		$searchData = ($searchData === null) ? $this->getValue() : $searchData;
 		// Note: we can not use empty here
@@ -821,7 +821,7 @@ 
 	 * @param array|null $ddsMembers if not specified it will look up the members of the classType
 	 * @deprecated
 	 */
-	public function ddsSaveDefinition($classType=null, $ddsMembers=null)
+	public function ddsSaveDefinition($classType = null, $ddsMembers = null)
 	{
 		Deprecated::ddsSaveDefinitionField($this, $classType, $ddsMembers);
 	}

Braces +27 added lines, -18 removed lines

@@ -204,8 +204,9 @@ 
 		// By this point $config should be the required minimum config
 		// i.e. has a `name` property
 		// -----
-		if (isset($config['name']))
-			$this->setName($config['name']);
+		if (isset($config['name'])) {
+					$this->setName($config['name']);
+		}
 
 		// Configure the field - the name property must be set by this point.
 		// as many configuration options such as adding Validators and attaching
@@ -279,8 +280,9 @@ 
 	 */
 	public function getComponentName()
 	{
-		if ($this->_componentName === null)
-			$this->_componentName = str_replace('\\', '-', get_class($this));
+		if ($this->_componentName === null) {
+					$this->_componentName = str_replace('\\', '-', get_class($this));
+		}
 		return $this->_componentName;
 	}
 
@@ -306,8 +308,9 @@ 
 	 */
 	public function getDataKey()
 	{
-		if ($this->_dataKey === null)
-			return $this->getName();
+		if ($this->_dataKey === null) {
+					return $this->getName();
+		}
 		return $this->_dataKey;
 	}
 
@@ -344,8 +347,9 @@ 
 	public function getValue()
 	{
 		// return sanitised data only
-		if ($this->_sanitisedValue === null)
-			$this->_sanitisedValue = $this->sanitiseInput($this->getUnsafeValue());
+		if ($this->_sanitisedValue === null) {
+					$this->_sanitisedValue = $this->sanitiseInput($this->getUnsafeValue());
+		}
 		return $this->_sanitisedValue;
 	}
 
@@ -387,8 +391,9 @@ 
 	 */
 	protected function sanitiseInput($value)
 	{
-		if (empty($value))
-			return $value;
+		if (empty($value)) {
+					return $value;
+		}
 
 		profile_begin('form sanitise');
 		$sanitise = Html::sanitise($value, $this->allowableTags);
@@ -403,8 +408,9 @@ 
 	 */
 	protected function purifyInput($value)
 	{
-		if (empty($value))
-			return $value;
+		if (empty($value)) {
+					return $value;
+		}
 
 		profile_begin('form purify');
 		$sanitise = Html::purify($value, $this->allowableTags);
@@ -535,8 +541,9 @@ 
 	public function getErrorHtml()
 	{
 		$error = $this->getFirstError();
-		if ($error === '')
-			return '';
+		if ($error === '') {
+					return '';
+		}
 		$options = $this->errorOptions;
 		$tag = Arr::remove($options, 'tag', 'div');
 		$encode = Arr::remove($options, 'encode', true);
@@ -597,8 +604,9 @@ 
 	public function validate()
 	{
 		$validators = $this->getValidators();
-		if (count($validators)==0)
-			return true;
+		if (count($validators)==0) {
+					return true;
+		}
 
 		$isEmpty = empty($this->getValue());
 		$hasErrors = $this->hasError();
@@ -615,8 +623,9 @@ 
 			// ok - now apply the validator
 			try {
 				if ($validator->when == null || call_user_func($validator->when, $form, $attribute)) {
-					if (!$validator->validate($unsafeValue, $error))
-						$this->addError($error);
+					if (!$validator->validate($unsafeValue, $error)) {
+											$this->addError($error);
+					}
 				}
 			} catch (NotSupportedException $e) {
 				try {

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/el/DateTimeRange.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		$searchData = ($searchData === null) ? $this->getValue() : $searchData;
 		if ($searchData !== '') {

Braces +9 added lines, -6 removed lines

@@ -66,10 +66,12 @@ 
 		if ($searchData !== '') {
 			$dateFrom = Arr::get($searchData, 'from');
 			$dateTo   = Arr::get($searchData, 'to');
-			if ($dateFrom)
-				$query->where($this->getDataKey(), '>=', $dateFrom);
-			if ($dateTo)
-				$query->where($this->getDataKey(), '<=', $dateTo);
+			if ($dateFrom) {
+							$query->where($this->getDataKey(), '>=', $dateFrom);
+			}
+			if ($dateTo) {
+							$query->where($this->getDataKey(), '<=', $dateTo);
+			}
 		}
 	}
 
@@ -87,8 +89,9 @@ 
 	public function getValue()
 	{
 		$value = parent::getValue();
-		if (is_array($value))
-			return $value;
+		if (is_array($value)) {
+					return $value;
+		}
 		return ['from' => '', 'to' => ''];
 	}
 }
\ No newline at end of file

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

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 	 */
 	public function getProperties()
 	{
-		return array_merge(parent::getProperties(), ['decimal', 'height', 'vertical', 'min', 'max'] );
+		return array_merge(parent::getProperties(), ['decimal', 'height', 'vertical', 'min', 'max']);
 	}
 	/**
 	 * @inheritdoc

neon/core/form/fields/DisplayText.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		// no filters can be applied here - so we override and do nothing.
 	}

neon/core/form/fields/Uuid64.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	 */
 	public function getValueDisplay()
 	{
-		return '<code>' . $this->getValue() . '</code>';
+		return '<code>'.$this->getValue().'</code>';
 	}
 
 	/**

neon/core/form/fields/UrlLink.php

Spacing +7 added lines, -7 removed lines

@@ -21,7 +21,7 @@ 
 	/**
 	 * set whether or not this is an absolute or relative URL
 	 */
-	public $absolute=true;
+	public $absolute = true;
 
 	/**
 	 * Set some allowable tags for single text fields
@@ -46,7 +46,7 @@ 
 	 */
 	public function getValueDisplay()
 	{
-		return '<a href="'.$this->getValue().'" target="_blank" noopener noreferrer>'.$this->getValue().'</a>' ;
+		return '<a href="'.$this->getValue().'" target="_blank" noopener noreferrer>'.$this->getValue().'</a>';
 	}
 
 	/**
@@ -59,16 +59,16 @@ 
 		if ($this->_sanitisedValue === null) {
 			$value = $this->getUnsafeValue();
 			// disallow any quotes in the URL to prevent a class of attacks
-			if (strpos($value, "'") === false && strpos($value,'"') === false) {
+			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.'">';
 				$safeValue = $this->purifyInput($link);
 				if (strpos($safeValue, 'href')) {
-					$exploded = explode('"',$safeValue);
+					$exploded = explode('"', $safeValue);
 					$this->_sanitisedValue = $exploded[1];
 				}
 			} else
@@ -83,7 +83,7 @@ 
 	public function getProperties()
 	{
 		// convert the database values to booleans
-		$this->absolute = (bool)$this->absolute;
+		$this->absolute = (bool) $this->absolute;
 		$properties = parent::getProperties();
 		$properties[] = 'absolute';
 		return $properties;

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/FileBrowser.php

Spacing +3 added lines, -3 removed lines

@@ -40,8 +40,8 @@ 
 	 */
 	public function getFieldHtml()
 	{
-		return "<div id=\"{$this->getId()}\">" .
-			"<firefly-form-file-browser value=\"{$this->value}\" name=\"{$this->getInputName()}\"></firefly-form-file-browser>" .
+		return "<div id=\"{$this->getId()}\">".
+			"<firefly-form-file-browser value=\"{$this->value}\" name=\"{$this->getInputName()}\"></firefly-form-file-browser>".
 		"</div>";
 	}
 
@@ -60,7 +60,7 @@ 
 	public function getData()
 	{
 		$name = $this->getInputName();
-		if ( neon()->request->hasFile($name) ) {
+		if (neon()->request->hasFile($name)) {
 			$this->value = neon()->request->file($name)->save();
 		}
 		return $this->value;

neon/core/form/fields/ChoiceMultiple.php

Spacing +2 added lines, -2 removed lines

@@ -75,11 +75,11 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		if (isset($searchData[0])) {
 			$keys = explode(',', $searchData[0]);
-			foreach($keys as $key) {
+			foreach ($keys as $key) {
 				if (!empty($key) && $key !== 'null') {
 					$query->where($this->getDataKey(), 'like', $key);
 				}

Braces +6 added lines, -4 removed lines

@@ -41,8 +41,9 @@ 
 	public function setValueFromDb($value)
 	{
 		$items = [];
-		foreach ($value as $item)
-			$items[] = $item['key'];
+		foreach ($value as $item) {
+					$items[] = $item['key'];
+		}
 		return parent::setValueFromDb($items);
 	}
 
@@ -55,8 +56,9 @@ 
 	public function getSelectedItems()
 	{
 		$values = $this->getValue();
-		if (!is_array($values))
-			$values = [$values];
+		if (!is_array($values)) {
+					$values = [$values];
+		}
 		return Arr::only($this->getItems(), $values);
 	}