newicon / neon

neon/core/form/fields/Time.php

Spacing +5 added lines, -5 removed lines

@@ -55,9 +55,9 @@ 
 	{
 		$bits = explode(':', $timeString);
 		return [
-			'hh' => Arr::get($bits,0, '00'),
-			'mm' => Arr::get($bits,1, '00'),
-			'ss' => Arr::get($bits,2, '00')
+			'hh' => Arr::get($bits, 0, '00'),
+			'mm' => Arr::get($bits, 1, '00'),
+			'ss' => Arr::get($bits, 2, '00')
 		];
 	}
 
@@ -107,7 +107,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		// don't show seconds if they are not to be set
 		return neon()->formatter->asTime($this->getValue());
@@ -125,7 +125,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		$searchData = ($searchData === null) ? $this->getValue() : $searchData;
 		if (!empty($searchData)) {

neon/core/form/fields/Real.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		$value = $this->getValue();
 		if ($this->decimal === 0)

Braces +9 added lines, -6 removed lines

@@ -36,8 +36,9 @@ 
 	 */
 	public function setValue($value)
 	{
-		if (is_numeric($value))
-			return parent::setValue(round($value, $this->decimal));
+		if (is_numeric($value)) {
+					return parent::setValue(round($value, $this->decimal));
+		}
 		parent::setvalue(null);
 	}
 
@@ -47,8 +48,9 @@ 
 	public function getValue()
 	{
 		// prevent returning 0 when casting a null value to a (real) number
-		if (!is_numeric($this->_value))
-			return null;
+		if (!is_numeric($this->_value)) {
+					return null;
+		}
 		// (real) has been deprecated >= PHP.4 - use float instead
 		return (float) $this->_value;
 	}
@@ -59,8 +61,9 @@ 
 	public function getValueDisplay($context='')
 	{
 		$value = $this->getValue();
-		if ($this->decimal === 0)
-			return neon()->formatter->asInteger($value);
+		if ($this->decimal === 0) {
+					return neon()->formatter->asInteger($value);
+		}
 		return neon()->formatter->asDecimal($value, $this->decimal);
 	}
 

neon/core/form/fields/SwitchButton.php

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		return $this->value ? $this->trueLabel : $this->falseLabel;
 	}

neon/core/form/fields/Checkbox.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		return $this->getData() ? $this->trueLabel : $this->falseLabel;
 	}

neon/core/form/fields/Email.php

Braces +3 added lines, -2 removed lines

@@ -29,8 +29,9 @@
 	{
 		parent::init();
 		// set up default email validator
-		if (!$this->hasValidator(\neon\core\validators\EmailValidator::class))
-			$this->addValidator('email');
+		if (!$this->hasValidator(\neon\core\validators\EmailValidator::class)) {
+					$this->addValidator('email');
+		}
 	}
 
 	/**

neon/core/form/fields/Field.php

Braces +24 added lines, -16 removed lines

@@ -205,8 +205,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
@@ -280,8 +281,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;
 	}
 
@@ -307,8 +309,9 @@ 
 	 */
 	public function getDataKey()
 	{
-		if ($this->_dataKey === null)
-			return $this->getName();
+		if ($this->_dataKey === null) {
+					return $this->getName();
+		}
 		return $this->_dataKey;
 	}
 
@@ -345,8 +348,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;
 	}
 
@@ -388,8 +392,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);
@@ -404,8 +409,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);
@@ -536,8 +542,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);
@@ -598,8 +605,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();

Spacing +9 added lines, -9 removed lines

@@ -194,7 +194,7 @@ 
 	 * @param array $config
 	 * @throws 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)) {
@@ -519,7 +519,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) : '';
 	}
 
 	/**
@@ -601,7 +601,7 @@ 
 	public function validate()
 	{
 		$validators = $this->getValidators();
-		if (count($validators)==0)
+		if (count($validators) == 0)
 			return true;
 
 		$isEmpty = empty($this->getValue());
@@ -630,10 +630,10 @@ 
 					$validator->validateAttribute($form, $attribute);
 				} catch (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());
 				}
 			}
 		}
@@ -761,7 +761,7 @@ 
 	 * Get a displayable representation of the fields value to be output to html
 	 * @return string
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		return Html::encode($this->getValue());
 	}
@@ -781,7 +781,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
@@ -840,7 +840,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);
 	}

neon/core/form/fields/File.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		if (empty($this->value))
 			return Html::encodeEntities($this->value);

Braces +3 added lines, -2 removed lines

@@ -99,8 +99,9 @@
 	 */
 	public function getValueDisplay($context='')
 	{
-		if (empty($this->value))
-			return Html::encodeEntities($this->value);
+		if (empty($this->value)) {
+					return Html::encodeEntities($this->value);
+		}
 		return Html::a(neon()->firefly->getFileName($this->value), neon()->firefly->getUrl($this->value), ['target' => '_blank']);
 	}
 

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		return neon()->formatter->asDate($this->getValue());
 	}