newicon / neon

neon/core/grid/DdsDataProvider.php

Braces +3 added lines, -2 removed lines

@@ -45,8 +45,9 @@
 	public function __clone()
 	{
 		parent::__clone();
-		if ($this->_queryBuilder)
-			$this->_queryBuilder = clone $this->_queryBuilder;
+		if ($this->_queryBuilder) {
+					$this->_queryBuilder = clone $this->_queryBuilder;
+		}
 	}
 
 	/**

neon/core/grid/GridRenderer.php

Braces +6 added lines, -4 removed lines

@@ -133,13 +133,15 @@
 		$pagination = $this->_grid->getDataProvider()->getPagination();
 		$filters = [];
 		foreach($this->_grid->getFilterForm()->getFields() as $field) {
-			if (!empty($field->getValue()))
-				$filters[$field->getInputName()] = $field->getValue();
+			if (!empty($field->getValue())) {
+							$filters[$field->getInputName()] = $field->getValue();
+			}
 		}
 		$queryParams = neon()->request->queryParams;
 		// remove any pre existing grid specific filters from the url - we don't want to add them twice
-		if (isset($queryParams[$this->_grid->gridName()]))
-			unset($queryParams[$this->_grid->gridName()]);
+		if (isset($queryParams[$this->_grid->gridName()])) {
+					unset($queryParams[$this->_grid->gridName()]);
+		}
 		// gross - we need to remove any pre existing filters from the query params
 		$pagination->params = $queryParams + $filters;
 

neon/core/grid/PhoebeGrid.php

Braces +3 added lines, -2 removed lines

@@ -124,8 +124,9 @@
 	 */
 	public function getDataProvider()
 	{
-		if (!$this->_dataProvider)
-			throw new \RuntimeException('You need to call phoebeLoadGrid before the grid can be used');
+		if (!$this->_dataProvider) {
+					throw new \RuntimeException('You need to call phoebeLoadGrid before the grid can be used');
+		}
 		return $this->_dataProvider;
 	}
 

neon/core/grid/query/YiiQuery.php

Braces +5 added lines, -4 removed lines

@@ -30,10 +30,11 @@
 		if (is_array($column)) {
 			return $this->addArrayOfWheres($column);
 		}
-		if ($operator==='=')
-			$this->_activeQuery->andWhere([$column => $value]);
-		else
-			$this->_activeQuery->andWhere([$operator, $column, $value]);
+		if ($operator==='=') {
+					$this->_activeQuery->andWhere([$column => $value]);
+		} else {
+					$this->_activeQuery->andWhere([$operator, $column, $value]);
+		}
 		return $this;
 	}
 

neon/core/db/ActiveRecord.php

Braces +3 added lines, -2 removed lines

@@ -147,8 +147,9 @@
 	{
 		$tableName = $this->tableName();
 		foreach($fields as $field) {
-			if (!$this->hasAttribute($field))
-				throw new \InvalidArgumentException("No '$field' exists on the '$tableName' table. $message");
+			if (!$this->hasAttribute($field)) {
+							throw new \InvalidArgumentException("No '$field' exists on the '$tableName' table. $message");
+			}
 		}
 	}
 

neon/core/db/Migrator.php

Braces +25 added lines, -19 removed lines

@@ -113,8 +113,9 @@ 
 	public function getNewMigrationsForApp($appName)
 	{
 		$app = Neon::$app->getApp($appName);
-		if ($app === null)
-			throw new \Exception('No app exists with id "'.$appName.'"');
+		if ($app === null) {
+					throw new \Exception('No app exists with id "'.$appName.'"');
+		}
 		$alias = $app->getClassAlias().'/migrations';
 		$migrationAlias = $app->getMigrationAlias().'/migrations';
 		$applied = [];
@@ -136,8 +137,9 @@ 
 	 */
 	private function getNewMigrationsInPath($pathAlias, $applied, $migrationAlias=null)
 	{
-		if ($migrationAlias === null)
-			$migrationAlias = $pathAlias;
+		if ($migrationAlias === null) {
+					$migrationAlias = $pathAlias;
+		}
 		$migrations = [];
 		$migrationPath = Neon::getAlias($pathAlias);
 		if (file_exists($migrationPath)) {
@@ -402,9 +404,7 @@ 
 				$string = $prefix.$message;
 			}
 			return Console::stdout("$string\n");
-		}
-
-		else {
+		} else {
 			return $message;
 		}
 	}
@@ -431,15 +431,17 @@ 
 		foreach ($migrations as $migration) {
 			// remove any paths to get the migration itself
 			$migrationPart = $migration;
-			if (strpos($migration,'/')!==false)
-				$migrationPart = substr($migration, strrpos($migration,'/')+1);
+			if (strpos($migration,'/')!==false) {
+							$migrationPart = substr($migration, strrpos($migration,'/')+1);
+			}
 
 			$parts = explode('_', substr($migrationPart,1)); // remove initial m
 
 			// check it was a migration and has enough parts
 			if (count($parts) == 1) {
-				if (!is_numeric($parts[0]))
-					continue;
+				if (!is_numeric($parts[0])) {
+									continue;
+				}
 				$date = $parts[0];
 				$time = '000000';
 			} else {
@@ -449,24 +451,28 @@ 
 
 			// some handwritten migrations have bad datetimes
 			// capture case where no time has been set or is actually a date
-			if (!is_numeric($time) || strlen($time) != 6)
-				$time = '000000';
+			if (!is_numeric($time) || strlen($time) != 6) {
+							$time = '000000';
+			}
 			// some datetimes are YYYYMMDD and others YYMMDD. Canonicalise these
-			if (strlen($date)==6)
-				$date = "20$date";
+			if (strlen($date)==6) {
+							$date = "20$date";
+			}
 			// turn this into 'YYYY-MM-DD HH:MM:SS' and then into a timestamp
 			$dateTime = substr($date,0,4)."-".substr($date,4,2)."-".substr($date,6,2)." ".substr($time,0,2).":".substr($time,2,2).":".substr($time,4,2);
 			$timestamp = strtotime($dateTime);
-			if ($timestamp == false)
-				throw new \Exception("Invalid datetime in migration $migration. Date(yyyymmdd)=$date and time(hhmmss)=$time. Please fix.");
+			if ($timestamp == false) {
+							throw new \Exception("Invalid datetime in migration $migration. Date(yyyymmdd)=$date and time(hhmmss)=$time. Please fix.");
+			}
 			// beware migrations with same timestamp
 			$sortableMigrations[$timestamp][] = $migration;
 		}
 		ksort($sortableMigrations);
 		$sortedMigrations = [];
 		foreach ($sortableMigrations as $mgs) {
-			foreach ($mgs as $mig)
-				$sortedMigrations[] = $mig;
+			foreach ($mgs as $mig) {
+							$sortedMigrations[] = $mig;
+			}
 		}
 		return $sortedMigrations;
 	}

neon/core/services/Formatter.php

Braces +6 added lines, -4 removed lines

@@ -42,14 +42,16 @@
 		parent::init();
 		// the main config can set this - but it can also be loaded by application settings (and possibly user setting in future)
 		$this->dateFormat = setting('settings', 'dateFormat', $this->dateFormat);
-		if ($this->dateFormat === 'custom')
-			$this->dateFormat = setting('settings', 'dateFormatCustom', $this->dateFormat);
+		if ($this->dateFormat === 'custom') {
+					$this->dateFormat = setting('settings', 'dateFormatCustom', $this->dateFormat);
+		}
 
 		$this->timeFormat = setting('settings', 'timeFormat', $this->timeFormat);
 		$this->nullDisplay = setting('settings', 'nullDisplay', $this->nullDisplay);
 		$this->datetimeFormat = setting('settings', 'datetimeFormat', $this->datetimeFormat);
-		if ($this->datetimeFormat === 'custom')
-			$this->datetimeFormat = setting('settings', 'datetimeFormatCustom', $this->datetimeFormat);
+		if ($this->datetimeFormat === 'custom') {
+					$this->datetimeFormat = setting('settings', 'datetimeFormatCustom', $this->datetimeFormat);
+		}
 	}
 
 	/**

neon/core/form/fields/File.php

Braces +3 added lines, -2 removed lines

@@ -99,8 +99,9 @@
 	 */
 	public function getValueDisplay()
 	{
-		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/Choice.php

Braces +3 added lines, -2 removed lines

@@ -105,8 +105,9 @@
 	public function getItemsDisplay()
 	{
 		$items = $this->getItems();
-		foreach ($items as $key => $value)
-			$items[$key] = '<span class="neonField_labelText">'.$value.'</span>';
+		foreach ($items as $key => $value) {
+					$items[$key] = '<span class="neonField_labelText">'.$value.'</span>';
+		}
 		return $items;
 	}