newicon / neon

neon/core/form/fields/DateRange.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		$searchData = ($searchData === null) ? $this->getData() : $searchData;
 		if ($searchData !== '') {
@@ -126,7 +126,7 @@ 
 	{
 		$value = parent::getValue();
 		// the result must be an array with 'from'=> and 'to'=>
-		if ( is_array($value) && isset($value['from'], $value['to']) ) {
+		if (is_array($value) && isset($value['from'], $value['to'])) {
 			return [
 				'from' => $this->_isDateValid($value['from']) ? $value['from'] : '',
 				'to' => $this->_isDateValid($value['to']) ? $value['to'] : ''

Braces +6 added lines, -4 removed lines

@@ -96,10 +96,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);
+			}
 		}
 	}
 

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;
 	}
 

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
 	 * Any filters you want to apply to a data map selection.
 	 * @var array  field=>value
 	 */
-	public $dataMapFilters=[];
+	public $dataMapFilters = [];
 
 	/**
 	 * Any fields you want to return in a data map selection if not the default.
@@ -74,7 +74,7 @@ 
 	 *   string (otherwise with a space) and prefix postfix arguments will be
 	 *   applied to it.
 	 */
-	public $dataMapFields=[];
+	public $dataMapFields = [];
 
 	/**
 	 * @inheritdoc
@@ -133,7 +133,7 @@ 
 	 * @return array
 	 * @throws \Exception
 	 */
-	public function getMapObjects($query=null, $start=0, $length=100)
+	public function getMapObjects($query = null, $start = 0, $length = 100)
 	{
 		$provider = $this->getProviderService();
 		$format = ['prefix'=>'', 'concat'=>' ', 'postfix' => ''];
@@ -165,12 +165,12 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
 		$items = $this->getItems();
 		if (is_array($this->value)) {
 			$values = array_intersect_key($items, array_flip($this->value));
-			return implode(', ',$values);
+			return implode(', ', $values);
 		} else if (isset($items[$this->value])) {
 			return $items[$this->value];
 		}
@@ -205,7 +205,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function processAsFilter(IQuery $query, $searchData=null)
+	public function processAsFilter(IQuery $query, $searchData = null)
 	{
 		$searchData = ($searchData === null) ? $this->getValue() : $searchData;
 		if ($searchData !== '') {

neon/core/form/fields/Text.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)) {

Braces +3 added lines, -2 removed lines

@@ -50,8 +50,9 @@
 	{
 		$guesser = new \Faker\Guesser\Name(faker());
 		$guess = $guesser->guessFormat($this->getName(), 150);
-		if (is_callable($guess))
-			return $guess();
+		if (is_callable($guess)) {
+					return $guess();
+		}
 		return faker()->realText(mt_rand(10, 150));
 	}
 

neon/core/form/fields/Guidance.php

Indentation +3 added lines, -3 removed lines

@@ -21,9 +21,9 @@
 	 */
 	public function getFieldHtml()
 	{
-        return Html::tag('neon-core-form-fields-guidance', [
-        	'text' => $this->getText(),
-        ]);
+		return Html::tag('neon-core-form-fields-guidance', [
+			'text' => $this->getText(),
+		]);
 	}
 
 	/**

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' => ''];
 	}
 

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/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;
 	}

Spacing +8 added lines, -8 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
@@ -44,9 +44,9 @@ 
 	 * Get a displayable representation of the fields value
 	 * @return string
 	 */
-	public function getValueDisplay($context='')
+	public function getValueDisplay($context = '')
 	{
-		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;