newicon / neon

neon/core/form/fields/Submit.php

Braces +3 added lines, -2 removed lines

@@ -52,8 +52,9 @@
 	 */
 	public function setValue($value)
 	{
-		if ($value === 'true' || $value === true)
-			$this->getForm()->submittedButton=$this->getName();
+		if ($value === 'true' || $value === true) {
+					$this->getForm()->submittedButton=$this->getName();
+		}
 		return $this;
 	}
 

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

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/user/form/fields/UserSelectorMultiple.php

Indentation +2 added lines, -2 removed lines

@@ -21,8 +21,8 @@
 	public $dataMapKey = 'users';
 
 		/**
-	 * @inheritdoc
-	 */
+		 * @inheritdoc
+		 */
 	public function getComponentDetails()
 	{
 		return [

neon/cms/form/fields/ImageCompare.php

Indentation +9 added lines, -9 removed lines

@@ -14,21 +14,21 @@ 
 
 class ImageCompare extends ImageMultiple
 {
-    /**
+	/**
 	 * @inheritdoc
 	 */
-    public $ddsDataType = 'json';
+	public $ddsDataType = 'json';
 
-    /**
+	/**
 	 * @inheritdoc
 	 */
 	public function registerScripts($view)
 	{
-        $view->registerAssetBundle(\neon\cms\assets\CmsFormAsset::class);
+		$view->registerAssetBundle(\neon\cms\assets\CmsFormAsset::class);
 		$view->registerAssetBundle(\neon\firefly\assets\BrowserAsset::class);
-    }
+	}
 
-    /**
+	/**
 	 * @inheritdoc
 	 */
 	public function getFilterField()
@@ -36,7 +36,7 @@ 
 		return ['class' => 'neon\\core\\form\\fields\\Text'];
 	}
 
-    /**
+	/**
 	 * @inheritdoc
 	 */
 	public function getComponentDetails()
@@ -44,9 +44,9 @@ 
 		return [
 			'icon' => 'fa fa-image', 'group' => 'Media', 'order' => 612, 'label' => 'Image Compare'
 		];
-    }
+	}
 
-    public function getValueDisplay($context='')
+	public function getValueDisplay($context='')
 	{
 		$json = $this->getValue();
 		if (empty($json))

neon/core/grid/GridBase.php

Braces +45 added lines, -30 removed lines

@@ -311,8 +311,9 @@ 
 			return $scope['key'];
 		}
 		// No url request scope defined check if a default is set
-		if ($this->defaultScope !== null)
-			return $this->defaultScope;
+		if ($this->defaultScope !== null) {
+					return $this->defaultScope;
+		}
 
 		// No default scope so return the first in the list as the default
 		$defaultScope = reset($this->_scopes);
@@ -441,8 +442,9 @@ 
 			foreach($this->getColumns() as $key => $column) {
 				if ($column->getFilter() && $column->getFilterField() !== false) {
 					$field = $column->getFilterField();
-					if (is_object($field))
-						$field = $field->toArray();
+					if (is_object($field)) {
+											$field = $field->toArray();
+					}
 					$field['name'] = $key;
 					$field['dataKey'] = $column->getDbField();
 					$this->_filterForm->add($field);
@@ -643,8 +645,9 @@ 
 			$formsDefinitionCache[$cacheKey] = $formDefinition;
 		}
 		$form = new \neon\core\form\Form($formsDefinitionCache[$cacheKey]);
-		if ($member === null)
-			$member = $key;
+		if ($member === null) {
+					$member = $key;
+		}
 		return $this->addColumn($key, [
 			'title' => $form->getField($member)->getLabel(),
 			'class' => 'neon\core\grid\column\Column',
@@ -706,8 +709,9 @@ 
 	 */
 	public function removeColumn($key)
 	{
-		if ($this->columnExists($key))
-			unset($this->_columns[$key]);
+		if ($this->columnExists($key)) {
+					unset($this->_columns[$key]);
+		}
 	}
 
 	/**
@@ -754,8 +758,9 @@ 
 	 */
 	public function refreshDataProvider()
 	{
-		if ($this->_dataProvider)
-			$this->_dataProvider = clone $this->getDataProvider();
+		if ($this->_dataProvider) {
+					$this->_dataProvider = clone $this->getDataProvider();
+		}
 	}
 
 	/**
@@ -780,8 +785,9 @@ 
 	 */
 	public function gridBoot()
 	{
-		if (! $this->hasData())
-			$this->_gridData = Arr::get($_REQUEST, $this->id, null);
+		if (! $this->hasData()) {
+					$this->_gridData = Arr::get($_REQUEST, $this->id, null);
+		}
 		// load the active scope
 		$this->_scope = Arr::get($this->_gridData, "scope", []);
 		// load request data into the form
@@ -818,8 +824,9 @@ 
 	public function exportCsv($page=0, $pageSize=1000, $asRows=false)
 	{
 		// check we're allowed to do this
-		if (!$this->canExportCsv)
-			return null;
+		if (!$this->canExportCsv) {
+					return null;
+		}
 
 		$this->gridBoot();
 		// Get header column names
@@ -841,8 +848,9 @@ 
 		$pagination->page=$page;
 		$pagination->pageSize = min(1000,$pageSize);
 		$models = $dataProvider->getModels();
-		if (empty($models))
-			return null;
+		if (empty($models)) {
+					return null;
+		}
 
 		$rows = [];
 		foreach($models as $index => $row) {
@@ -858,8 +866,9 @@ 
 		$dataProvider->models = null;
 
 		// Add headers columns as the first row if we are on the first page
-		if ($page==0)
-			array_unshift($rows, implode(',', $headerCols));
+		if ($page==0) {
+					array_unshift($rows, implode(',', $headerCols));
+		}
 
 		return $asRows ? $rows : implode("\n", $rows);
 	}
@@ -935,8 +944,9 @@ 
 	{
 		foreach ($this->getScopes() as $key => $scope) {
 			// counts across whole tables are very very expensive. Don't count more than once
-			if (isset($this->_scopeCounts[$key]))
-				continue;
+			if (isset($this->_scopeCounts[$key])) {
+							continue;
+			}
 			$dp = clone $this->getDataProvider();
 			// call the scope function defined by `'scope'.$key` functions in child classes
 			$this->callScope($scope, $dp->getQueryBuilder());
@@ -964,8 +974,9 @@ 
 	public function processSearch()
 	{
 		$data = $this->getGridData();
-		if (!isset($data['filter']))
-			return;
+		if (!isset($data['filter'])) {
+					return;
+		}
 		foreach ($this->getColumns() as $key => $column) {
 			// check there is a form filter field
 			// check that the filter field has request data
@@ -1049,8 +1060,9 @@ 
 	 */
 	public function getColumn($columnKey)
 	{
-		if (!isset($this->_columns[$columnKey]))
-			throw new \Exception("No column with key '$columnKey' exists in the grid");
+		if (!isset($this->_columns[$columnKey])) {
+					throw new \Exception("No column with key '$columnKey' exists in the grid");
+		}
 		return $this->_columns[$columnKey];
 	}
 
@@ -1141,8 +1153,9 @@ 
 	{
 		$columns = [];
 		foreach($this->getColumns() as $key => $column) {
-			if ($column->visible)
-				$columns[$key] = $column;
+			if ($column->visible) {
+							$columns[$key] = $column;
+			}
 		}
 		return $columns;
 	}
@@ -1187,8 +1200,9 @@ 
 	 */
 	public function getIndexColumn()
 	{
-		if (!isset($this->_columns[$this->_indexedByColumn]))
-			throw new \Exception("No index column exists with key '$this->_indexedByColumn'.");
+		if (!isset($this->_columns[$this->_indexedByColumn])) {
+					throw new \Exception("No index column exists with key '$this->_indexedByColumn'.");
+		}
 		return $this->_columns[$this->_indexedByColumn];
 	}
 
@@ -1250,8 +1264,9 @@ 
 		$plain = html_entity_decode(
 				str_replace(["\n", '"'], ['', '\''], strip_tags($content)),
 				ENT_HTML401 | ENT_QUOTES | ENT_HTML5);
-		if ($plain === $this->emptyCellDisplay)
-			$plain = '';
+		if ($plain === $this->emptyCellDisplay) {
+					$plain = '';
+		}
 		return '"'.$plain.'"';
 	}
 }

neon/core/grid/DdsDataProvider.php

Braces +6 added lines, -4 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;
+		}
 	}
 
 	/**
@@ -67,8 +68,9 @@ 
 	{
 		// prepare pagination
 		$pagination = $this->getPagination();
-		if ($pagination === false)
-			return [];
+		if ($pagination === false) {
+					return [];
+		}
 		// depending on how this grid was set up, this call may or may not have been made.
 		// The results are cached so call it here to be safe.
 		$totalCount = $this->getTotalCount();

neon/daedalus/services/ddsManager/DdsObjectManager.php

Braces +206 added lines, -142 removed lines

@@ -108,8 +108,9 @@ 
 					// set null result information about the request
 					$newData[$k] = ['start' => $r['limit']['start'], 'length' => 0, 'total' => $r['total'], 'rows' => []];
 					$sql[] = $r['sql'];
-					if ($r['totalSql'])
-						$sql[] = $r['totalSql'];
+					if ($r['totalSql']) {
+											$sql[] = $r['totalSql'];
+					}
 					$resultKey2RequestKey[$r['resultKey']] = $r['requestKey'];
 				}
 			}
@@ -170,8 +171,9 @@ 
 	 */
 	public function runSingleObjectRequest($classType, array $filters = [], array $order = [], array $limit = [], $includeDeleted = false)
 	{
-		if (empty($classType))
-			throw new \InvalidArgumentException('ClassType must be specified');
+		if (empty($classType)) {
+					throw new \InvalidArgumentException('ClassType must be specified');
+		}
 
 		// make sure that any currently queued requests from elsewhere are saved off the queue
 		$currentRequests = $this->popRequests();
@@ -254,16 +256,18 @@ 
 	public function listObjects($classType, &$total = null, $includeDeleted = false, $inFull = true, $start = 0, $length = 100, $order = null)
 	{
 		$classType = $this->canonicaliseRef($classType);
-		if (empty($order))
-			$order = ['_created' => 'DESC'];
+		if (empty($order)) {
+					$order = ['_created' => 'DESC'];
+		}
 		$order = $this->canonicaliseOrder($order);
 
 		// get the total first
 		$class = null;
 		$this->findClass($classType, $class, true);
 		$total = $class->count_total;
-		if (!$includeDeleted)
-			$total -= $class->count_deleted;
+		if (!$includeDeleted) {
+					$total -= $class->count_deleted;
+		}
 
 		// now get the objects
 		$query = (new \neon\core\db\Query)
@@ -273,8 +277,9 @@ 
 			->offset($start)
 			->limit($length);
 
-		if (!$includeDeleted)
-			$query->andWhere('[[o]].[[_deleted]] = 0');
+		if (!$includeDeleted) {
+					$query->andWhere('[[o]].[[_deleted]] = 0');
+		}
 		if ($inFull) {
 			$tableName = $this->getTableFromClassType($classType);
 			$query->innerJoin('{{' . $tableName . '}} t', '[[o]].[[_uuid]]=[[t]].[[_uuid]]');
@@ -289,8 +294,9 @@ 
 					$orderBits[] = "[[t]].$k $d";
 				}
 			}
-			if ($orderBits)
-				$orderClause = implode(', ', $orderBits);
+			if ($orderBits) {
+							$orderClause = implode(', ', $orderBits);
+			}
 			$query->orderBy($orderClause);
 		}
 
@@ -307,8 +313,9 @@ 
 		$class = null;
 		$classType = $this->canonicaliseRef($classType);
 		$this->findClass($classType, $class, true);
-		if ($class->hasChangeLog())
-			$newValues = $data;
+		if ($class->hasChangeLog()) {
+					$newValues = $data;
+		}
 		$table = $this->getTableFromClassType($classType);
 
 		// create the new object
@@ -319,8 +326,9 @@ 
 		$object->_class_type = $classType;
 		$object->_created = $now;
 		$object->_updated = $now;
-		if (!$object->save())
-			throw new \RuntimeException("Couldn't create object: " . print_r($object->errors, true));
+		if (!$object->save()) {
+					throw new \RuntimeException("Couldn't create object: " . print_r($object->errors, true));
+		}
 
 		// increment the class's object count
 		$class->count_total += 1;
@@ -344,10 +352,11 @@ 
 				}
 			}
 
-			if (count($fields))
-				$query = "INSERT INTO `$table` (`_uuid`, " . implode(', ', $fields) . ") VALUES ('{$object->_uuid}', " . implode(', ', $inserts) . ")";
-			else
-				$query = "INSERT INTO `$table` (`_uuid`) VALUES ('{$object->_uuid}')";
+			if (count($fields)) {
+							$query = "INSERT INTO `$table` (`_uuid`, " . implode(', ', $fields) . ") VALUES ('{$object->_uuid}', " . implode(', ', $inserts) . ")";
+			} else {
+							$query = "INSERT INTO `$table` (`_uuid`) VALUES ('{$object->_uuid}')";
+			}
 			$command = neon()->db->createCommand($query);
 			$command->bindValues($values);
 			$command->execute();
@@ -355,8 +364,9 @@ 
 			// if there are any links then add these to the link table
 			$this->setMemberLinks($object->_uuid, $links);
 
-			if ($class->hasChangeLog())
-				$changeLogUuid = $this->createChangeLogEntry($class, 'ADD', $object->_uuid, [], $newValues);
+			if ($class->hasChangeLog()) {
+							$changeLogUuid = $this->createChangeLogEntry($class, 'ADD', $object->_uuid, [], $newValues);
+			}
 		} catch (\Exception $e) {
 			// execution failed so clean up the object
 			$object->delete();
@@ -400,8 +410,9 @@ 
 			$objectsLinks = [];
 			$now = date('Y-m-d H:i:s');
 			foreach ($chunkData as $row) {
-				if ($hasChangeLog)
-					$newValues = $row;
+				if ($hasChangeLog) {
+									$newValues = $row;
+				}
 				// ddt table value - format row data and append the uuid
 				$this->convertFromPHPToDB($classType, $row, $links);
 				$uuid = (!empty($row['_uuid']) && $this->isUUID($row['_uuid'])) ? $row['_uuid'] : $this->generateUUID();
@@ -411,8 +422,9 @@ 
 				// object table row value
 				$objectDbRows[] = [$row['_uuid'], $classType, $now, $now];
 				$objectsLinks[$row['_uuid']] = $links;
-				if ($hasChangeLog)
-					$this->createChangeLogEntry($class, 'ADD', $uuid, [], $newValues);
+				if ($hasChangeLog) {
+									$this->createChangeLogEntry($class, 'ADD', $uuid, [], $newValues);
+				}
 			}
 			neon()->db->createCommand()->batchInsert(DdsObject::tableName(), ['_uuid', '_class_type', '_created', '_updated'], $objectDbRows)->execute();
 			neon()->db->createCommand()->batchInsert($table, $columns, $rows)->execute();
@@ -446,20 +458,23 @@ 
 	public function editObject($uuid, array $changes, &$changeLogUuid = null)
 	{
 		// check there are some changes to make
-		if (count($changes) == 0)
-			return false;
+		if (count($changes) == 0) {
+					return false;
+		}
 
 		// check that the object exists and find its class type
-		if (!$this->getObjectFromId($uuid, $object))
-			throw new \InvalidArgumentException("Couldn't edit object with id $uuid as not found");
+		if (!$this->getObjectFromId($uuid, $object)) {
+					throw new \InvalidArgumentException("Couldn't edit object with id $uuid as not found");
+		}
 
 		// see if we need to store the change log
 		$class = null;
 		$this->findClass($object['_class_type'], $class);
 
 		$object->_updated = date('Y-m-d H:i:s');
-		if (!$object->save())
-			return $object->errors;
+		if (!$object->save()) {
+					return $object->errors;
+		}
 
 		// now update any member changes
 		try {
@@ -488,8 +503,9 @@ 
 					}
 				}
 			}
-			if (count($updates) == 0)
-				return false;
+			if (count($updates) == 0) {
+							return false;
+			}
 
 			$query = "UPDATE `$table` SET " . implode(', ', $updates) . " WHERE `_uuid`='$uuid' LIMIT 1";
 			neon()->db->createCommand($query)->bindValues($values)->execute();
@@ -498,8 +514,9 @@ 
 			$this->setMemberLinks($object->_uuid, $links);
 
 			// finally note the changes in the change log
-			if ($class && $class->hasChangeLog())
-				$changeLogUuid = $this->createChangeLogEntry($class, 'EDIT', $uuid, $originalValues, $newValues);
+			if ($class && $class->hasChangeLog()) {
+							$changeLogUuid = $this->createChangeLogEntry($class, 'EDIT', $uuid, $originalValues, $newValues);
+			}
 		} catch (\yii\db\exception $e) {
 			$this->clearCaches();
 			return ['error' => $e->getMessage()];
@@ -521,12 +538,14 @@ 
 	 */
 	public function editObjects($classType, array $uuids, array $changes)
 	{
-		if (count($uuids) == 0 || count($changes) == 0)
-			return false;
+		if (count($uuids) == 0 || count($changes) == 0) {
+					return false;
+		}
 		try {
 			$uuids = $this->checkUuidsAgainstClassType($uuids, $classType);
-			if (count($uuids) == 0)
-				return false;
+			if (count($uuids) == 0) {
+							return false;
+			}
 			$table = $this->getTableFromClassType($classType);
 			$memberStorage = $this->getMemberStorage($classType);
 			if (count($memberStorage) == 0) {
@@ -537,8 +556,9 @@ 
 			$class = null;
 			$this->findClass($classType, $class);
 			if ($class && $class->hasChangeLog()) {
-				foreach ($uuids as $uuid)
-					$beforeValues[$uuid] = array_intersect_key($this->getObject($uuid), $changes);
+				foreach ($uuids as $uuid) {
+									$beforeValues[$uuid] = array_intersect_key($this->getObject($uuid), $changes);
+				}
 				$newValues = $changes;
 			}
 
@@ -560,8 +580,9 @@ 
 					}
 				}
 			}
-			if (count($updates) == 0)
-				return false;
+			if (count($updates) == 0) {
+							return false;
+			}
 
 			// sort out the uuid clause for both the update on the table and in the object table
 			$uuidClause = [];
@@ -589,8 +610,9 @@ 
 
 			// and save entries into a change log
 			if ($class && $class->hasChangeLog()) {
-				foreach ($beforeValues as $uuid => $before)
-					$this->createChangeLogEntry($class, 'EDIT', $uuid, $before, $newValues);
+				foreach ($beforeValues as $uuid => $before) {
+									$this->createChangeLogEntry($class, 'EDIT', $uuid, $before, $newValues);
+				}
 			}
 		} catch (\yii\db\exception $e) {
 			$this->clearCaches();
@@ -610,8 +632,9 @@ 
 		if ($this->getObjectFromId($uuid, $object) && $object->_deleted == 0) {
 			$object->_deleted = 1;
 			$object->_updated = date('Y-m-d H:i:s');
-			if (!$object->save())
-				throw new \RuntimeException("Couldn't delete the object: " . print_r($object->errors, true));
+			if (!$object->save()) {
+							throw new \RuntimeException("Couldn't delete the object: " . print_r($object->errors, true));
+			}
 			// and update the class object deleted count
 			$class = null;
 			$this->findClass($object['_class_type'], $class);
@@ -619,8 +642,9 @@ 
 			$class->save();
 
 			// see if we need to store the change log
-			if ($class && $class->hasChangeLog())
-				$changeLogUuid = $this->createChangeLogEntry($class, 'DELETE', $uuid);
+			if ($class && $class->hasChangeLog()) {
+							$changeLogUuid = $this->createChangeLogEntry($class, 'DELETE', $uuid);
+			}
 
 			/** ---- KEEP ME ----
 			 * don't delete the links so any that haven't been deleted between
@@ -643,8 +667,9 @@ 
 	public function deleteObjects(array $uuids)
 	{
 		$objects = $this->getObjectRowsFromIds($uuids);
-		if (count($objects) == 0)
-			return;
+		if (count($objects) == 0) {
+					return;
+		}
 
 		$foundUuids = [];
 		$foundClasses = [];
@@ -661,8 +686,9 @@ 
 				$deletedUuids[$classType][$obj['_uuid']] = $obj['_uuid'];
 			}
 		}
-		if (count($foundUuids))
-			DdsObject::updateAll(['_deleted' => 1, '_updated' => date('Y-m-d H:i:s')], ['_uuid' => $foundUuids]);
+		if (count($foundUuids)) {
+					DdsObject::updateAll(['_deleted' => 1, '_updated' => date('Y-m-d H:i:s')], ['_uuid' => $foundUuids]);
+		}
 		foreach ($foundClasses as $type => $count) {
 			$class = null;
 			if ($this->findClass($type, $class)) {
@@ -670,8 +696,9 @@ 
 				$class->save();
 				// TODO 20200107 Make a bulk call for change log entries
 				if ($class->hasChangeLog()) {
-					foreach ($deletedUuids[$class->class_type] as $objUuid)
-						$this->createChangeLogEntry($class, 'DELETE', $objUuid);
+					foreach ($deletedUuids[$class->class_type] as $objUuid) {
+											$this->createChangeLogEntry($class, 'DELETE', $objUuid);
+					}
 				}
 			}
 		}
@@ -698,8 +725,9 @@ 
 		if ($this->getObjectFromId($uuid, $object) && $object->_deleted == 1) {
 			$object->_deleted = 0;
 			$object->_updated = date('Y-m-d H:i:s');
-			if (!$object->save())
-				throw new \RuntimeException("Couldn't undelete the object: " . print_r($object->errors, true));
+			if (!$object->save()) {
+							throw new \RuntimeException("Couldn't undelete the object: " . print_r($object->errors, true));
+			}
 
 			// and update the class object deleted count
 			$class = null;
@@ -708,8 +736,9 @@ 
 				$class->save();
 
 				// see if we need to store the change log
-				if ($class->hasChangeLog())
-					$changeLogUuid = $this->createChangeLogEntry($class, 'UNDELETE', $uuid);
+				if ($class->hasChangeLog()) {
+									$changeLogUuid = $this->createChangeLogEntry($class, 'UNDELETE', $uuid);
+				}
 			}
 			$this->clearCaches();
 		}
@@ -724,8 +753,9 @@ 
 		if ($this->getObjectFromId($uuid, $object)) {
 			$class = null;
 			$this->findClass($object['_class_type'], $class);
-			if ($class && $class->hasChangeLog())
-				$originalValues = $this->getObject($uuid);
+			if ($class && $class->hasChangeLog()) {
+							$originalValues = $this->getObject($uuid);
+			}
 
 			// delete any object table data
 			$table = $this->getTableFromClassType($object['_class_type']);
@@ -741,8 +771,9 @@ 
 			// then if ok, decrement the class's object counts
 			if ($class) {
 				$class->count_total = max(0, $class->count_total - 1);
-				if ($object['_deleted'] == 1)
-					$class->count_deleted = max(0, $class->count_deleted - 1);
+				if ($object['_deleted'] == 1) {
+									$class->count_deleted = max(0, $class->count_deleted - 1);
+				}
 				$class->save();
 			}
 
@@ -750,8 +781,9 @@ 
 			DdsLink::deleteAll(['or', ['from_id' => $uuid], ['to_id' => $uuid]]);
 
 			// and note this is in the change log
-			if ($class && $class->hasChangeLog())
-				$changeLogUuid = $this->createChangeLogEntry($class, 'DESTROY', $uuid, $originalValues);
+			if ($class && $class->hasChangeLog()) {
+							$changeLogUuid = $this->createChangeLogEntry($class, 'DESTROY', $uuid, $originalValues);
+			}
 
 			$this->clearCaches();
 		}
@@ -763,16 +795,18 @@ 
 	public function destroyObjects(array $uuids)
 	{
 		$objects = $this->getObjectRowsFromIds($uuids);
-		if (count($objects) == 0)
-			return;
+		if (count($objects) == 0) {
+					return;
+		}
 
 		$objectUuids = [];
 		$objectsDeleted = [];
 		$classUuids = [];
 		foreach ($objects as $obj) {
 			$objectUuids[$obj['_uuid']] = $obj['_uuid'];
-			if ($obj['_deleted'] == 1)
-				$objectsDeleted[$obj['_class_type']][$obj['_uuid']] = $obj['_uuid'];
+			if ($obj['_deleted'] == 1) {
+							$objectsDeleted[$obj['_class_type']][$obj['_uuid']] = $obj['_uuid'];
+			}
 			$classUuids[$obj['_class_type']][] = $obj['_uuid'];
 		}
 
@@ -799,8 +833,9 @@ 
 			// decrement the class's object counts
 			if ($class) {
 				$class->count_total = max(0, $class->count_total - count($uuids));
-				if (isset($objectsDeleted[$classType]))
-					$class->count_deleted = max(0, $class->count_deleted - count($objectsDeleted[$classType]));
+				if (isset($objectsDeleted[$classType])) {
+									$class->count_deleted = max(0, $class->count_deleted - count($objectsDeleted[$classType]));
+				}
 				$class->save();
 			}
 		}
@@ -852,8 +887,9 @@ 
 	 */
 	private function getObjectMapManager()
 	{
-		if (!$this->_objectMapManager)
-			$this->_objectMapManager = new DdsObjectMapManager;
+		if (!$this->_objectMapManager) {
+					$this->_objectMapManager = new DdsObjectMapManager;
+		}
 		return $this->_objectMapManager;
 	}
 
@@ -868,14 +904,16 @@ 
 		// then see if there are any multilinks to extract
 		if ($links === null) {
 			$objectIds = [];
-			foreach ($dataSet as $row)
-				$objectIds[] = $row['_uuid'];
+			foreach ($dataSet as $row) {
+							$objectIds[] = $row['_uuid'];
+			}
 			$links = $this->getMembersLinks($objectIds);
 		}
 		foreach ($dataSet as &$data) {
 			// convert full rows to database. Totals are left untouched
-			if (isset($data[$classTypeKey]))
-				$this->convertFromDBToPHP($data, $links[$data['_uuid']], $classTypeKey);
+			if (isset($data[$classTypeKey])) {
+							$this->convertFromDBToPHP($data, $links[$data['_uuid']], $classTypeKey);
+			}
 		}
 	}
 
@@ -942,8 +980,9 @@ 
 		// a randomly generated resultKey prevents db query chaching.
 		$resultKey = $resultKey ? $resultKey : $request['requestKey'] . '_' . $counter;
 		$request['resultKey'] = $resultKey;
-		if (!array_key_exists($request['requestKey'], self::$_requestResultsCache))
-			$this->convertRequestToSql($request);
+		if (!array_key_exists($request['requestKey'], self::$_requestResultsCache)) {
+					$this->convertRequestToSql($request);
+		}
 		// store the request
 		$this->objectRequests[$request['classType']][$resultKey] = $request;
 		return $resultKey;
@@ -1052,8 +1091,9 @@ 
 		$join = ['INNER JOIN dds_object `o` on `t`.`_uuid` = `o`.`_uuid`'];
 		// now see what else needs to be added
 		foreach ($links as $l) {
-			if (in_array($l, $filterKeys))
-				$join[] = "LEFT JOIN dds_link `link_$l` ON (`link_$l`.`from_id`=`o`.`_uuid` AND `link_$l`.`from_member`='$l')";
+			if (in_array($l, $filterKeys)) {
+							$join[] = "LEFT JOIN dds_link `link_$l` ON (`link_$l`.`from_id`=`o`.`_uuid` AND `link_$l`.`from_member`='$l')";
+			}
 		}
 		return implode(' ', $join);
 	}
@@ -1070,11 +1110,13 @@ 
 	{
 		$filterKeys = [];
 		$notDeleted = '`o`.`_deleted`=0';
-		if (count($filters) == 0)
-			return $includeDeleted ? '' : "WHERE $notDeleted";
+		if (count($filters) == 0) {
+					return $includeDeleted ? '' : "WHERE $notDeleted";
+		}
 		$this->createFilterClause($filters, $links, $filterKeys);
-		if (!$includeDeleted)
-			return "WHERE $notDeleted AND ($filters[sql])";
+		if (!$includeDeleted) {
+					return "WHERE $notDeleted AND ($filters[sql])";
+		}
 		return "WHERE $filters[sql]";
 	}
 
@@ -1092,8 +1134,9 @@ 
 		// filter down until we hit the actual filters
 		if (is_array($filters[0])) {
 			foreach ($filters as $k => &$filter) {
-				if ($k === 'logic')
-					continue;
+				if ($k === 'logic') {
+									continue;
+				}
 				$this->createFilterClause($filter, $links, $filterKeys);
 			}
 
@@ -1105,25 +1148,28 @@ 
 				// in which case we combine those with to get the filterSql
 				$filterSql = [];
 				foreach ($filters as $k => &$filter) {
-					if ($k === 'logic')
-						continue;
+					if ($k === 'logic') {
+											continue;
+					}
 					// use the logic keys if any for the items - these can be in position 3
 					// for most operators and position 2 for operators that don't take a key
 					if ($this->operatorTakesObject($filter[1])) {
-						if (isset($filter[3]))
-							$filterSql[$filter[3]] = $filter['itemSql'];
-						else
-							$filterSql[] = $filter['itemSql'];
+						if (isset($filter[3])) {
+													$filterSql[$filter[3]] = $filter['itemSql'];
+						} else {
+													$filterSql[] = $filter['itemSql'];
+						}
 					} else {
-						if (isset($filter[2]))
-							$filterSql[$filter[2]] = $filter['itemSql'];
-						else
-							$filterSql[] = $filter['itemSql'];
+						if (isset($filter[2])) {
+													$filterSql[$filter[2]] = $filter['itemSql'];
+						} else {
+													$filterSql[] = $filter['itemSql'];
+						}
 					}
 				}
-				if (empty($filters['logic']))
-					$filters['filterSql'] = $filters['sql'] = implode(' AND ', $filterSql);
-				else {
+				if (empty($filters['logic'])) {
+									$filters['filterSql'] = $filters['sql'] = implode(' AND ', $filterSql);
+				} else {
 					// make sure we test logic filters in order from longest key to shortest key
 					// otherwise we can end up with subkeys screwing things up
 					$orderedKeys = array_map('strlen', array_keys($filterSql));
@@ -1135,8 +1181,9 @@ 
 					// double conversion,
 					$keys = array_keys($filterSql);
 					$keys2hashed = array();
-					foreach ($keys as $k)
-						$keys2hashed[$k] = md5($k);
+					foreach ($keys as $k) {
+											$keys2hashed[$k] = md5($k);
+					}
 					$logicPass1 = str_replace($keys, $keys2hashed, $filters['logic']);
 					$filters['filterSql'] = $filters['sql'] = str_replace($keys2hashed, array_values($filterSql), $logicPass1);
 				}
@@ -1159,9 +1206,9 @@ 
 		//
 		$filter = (in_array($filters[0], $links) ? "`link_{$filters[0]}`.`to_id`" : $this->quoteField($filters[0]));
 		$filterKeys[$filters[0]] = $filters[0];
-		if ($this->operatorTakesObject($filters[1]) && isset($filters[2]))
-			$filters['itemSql'] = $filters['sql'] = "$filter " . $this->prepareForPDO($filters[1], $filters[2]);
-		else {
+		if ($this->operatorTakesObject($filters[1]) && isset($filters[2])) {
+					$filters['itemSql'] = $filters['sql'] = "$filter " . $this->prepareForPDO($filters[1], $filters[2]);
+		} else {
 			$filters['itemSql'] = $filters['sql'] = "$filter $filters[1]";
 		}
 	}
@@ -1170,21 +1217,25 @@ 
 	{
 		if (is_array($value)) {
 			$pdoValues = [];
-			if (!in_array($operator, ['=', '!=', 'IN', 'NOT IN']))
-				throw new \InvalidArgumentException("Daedalus: Cannot pass an *array* of values with an operator of $operator");
+			if (!in_array($operator, ['=', '!=', 'IN', 'NOT IN'])) {
+							throw new \InvalidArgumentException("Daedalus: Cannot pass an *array* of values with an operator of $operator");
+			}
 			foreach ($value as $v) {
-				if (is_array($v))
-					throw new \InvalidArgumentException("Daedalus: Cannot pass an *array* of *array* of values as filters (silly billy). You passed " . print_r($value, true));
+				if (is_array($v)) {
+									throw new \InvalidArgumentException("Daedalus: Cannot pass an *array* of *array* of values as filters (silly billy). You passed " . print_r($value, true));
+				}
 				$count = count($this->boundValues);
 				$variable = ":var{$count}end";
 				$this->boundValues[$variable] = $v;
 				$pdoValues[] = $variable;
 			}
 			// converts equals to ins and outs
-			if ($operator == '=')
-				$operator = 'IN';
-			if ($operator == '!=')
-				$operator = 'NOT IN';
+			if ($operator == '=') {
+							$operator = 'IN';
+			}
+			if ($operator == '!=') {
+							$operator = 'NOT IN';
+			}
 			return "$operator (" . implode(',', $pdoValues) . ')';
 		} else {
 			$count = count($this->boundValues);
@@ -1208,8 +1259,9 @@ 
 	 */
 	private function extractOrderClause($order)
 	{
-		if (count($order) == 0)
-			return '';
+		if (count($order) == 0) {
+					return '';
+		}
 		$clause = [];
 		foreach ($order as $o => $d) {
 			// if any of the order clauses are RAND then replace whole clause
@@ -1218,10 +1270,11 @@ 
 				break;
 			}
 			// allow negative ordering in mysql for nulls last
-			if (strpos($o, '-') === 0)
-				$clause[] = '-' . substr($o, 1) . " $d";
-			else
-				$clause[] = "$o $d";
+			if (strpos($o, '-') === 0) {
+							$clause[] = '-' . substr($o, 1) . " $d";
+			} else {
+							$clause[] = "$o $d";
+			}
 		}
 		return 'ORDER BY ' . implode(', ', $clause);
 	}
@@ -1233,8 +1286,9 @@ 
 	 */
 	private function extractLimitClause($limit)
 	{
-		if (count($limit) == 0)
-			return '';
+		if (count($limit) == 0) {
+					return '';
+		}
 		return "LIMIT $limit[start], $limit[length]";
 	}
 
@@ -1245,8 +1299,9 @@ 
 	 */
 	private function setMembersLinks($objectsLinks, $chunkSize)
 	{
-		if (count($objectsLinks) === 0)
-			return 0;
+		if (count($objectsLinks) === 0) {
+					return 0;
+		}
 		$totalCount = 0;
 		$db = neon()->db;
 		$ddsLink = DdsLink::tableName();
@@ -1256,15 +1311,18 @@ 
 			$deleteLinkClauses = [];
 			foreach ($chunkData as $fromUuid => $toMemberLinks) {
 				// check you've been supplied with rinky dinky data
-				if (!$this->isUUID($fromUuid))
-					throw new \InvalidArgumentException("The fromUuid should be a UUID64. You passed in $fromUuid");
+				if (!$this->isUUID($fromUuid)) {
+									throw new \InvalidArgumentException("The fromUuid should be a UUID64. You passed in $fromUuid");
+				}
 				foreach ($toMemberLinks as $member => $toLinks) {
-					if (!is_array($toLinks))
-						$toLinks = [$toLinks];
+					if (!is_array($toLinks)) {
+											$toLinks = [$toLinks];
+					}
 					// remove any empty toLinks that might have snuck through
 					$toLinks = array_filter($toLinks);
-					if (!$this->areUUIDs($toLinks))
-						throw new \InvalidArgumentException("The toMemberLinks should be UUID64s. You passed in " . print_r($toLinks, true));
+					if (!$this->areUUIDs($toLinks)) {
+											throw new \InvalidArgumentException("The toMemberLinks should be UUID64s. You passed in " . print_r($toLinks, true));
+					}
 					$toLinks = array_unique($toLinks);
 
 					//
@@ -1279,8 +1337,9 @@ 
 
 					// prepare the new links ready for bulk insert
 					$count += count($toLinks);
-					foreach ($toLinks as $toLink)
-						$batchInsert[] = ['from_id' => $fromUuid, 'from_member' => $member, 'to_id' => $toLink];
+					foreach ($toLinks as $toLink) {
+											$batchInsert[] = ['from_id' => $fromUuid, 'from_member' => $member, 'to_id' => $toLink];
+					}
 				}
 			}
 			if (count($deleteLinkClauses)) {
@@ -1289,10 +1348,12 @@ 
 			}
 
 			$insertedCount = 0;
-			if (count($batchInsert))
-				$insertedCount = $db->createCommand()->batchInsert($ddsLink, ['from_id', 'from_member', 'to_id'], $batchInsert)->execute();
-			if (YII_DEBUG && $insertedCount !== $count)
-				throw new \Exception("The link insertion failed - $count items should have been inserted vs $insertedCount actual");
+			if (count($batchInsert)) {
+							$insertedCount = $db->createCommand()->batchInsert($ddsLink, ['from_id', 'from_member', 'to_id'], $batchInsert)->execute();
+			}
+			if (YII_DEBUG && $insertedCount !== $count) {
+							throw new \Exception("The link insertion failed - $count items should have been inserted vs $insertedCount actual");
+			}
 			$totalCount += $count;
 		}
 		return $totalCount;
@@ -1333,8 +1394,9 @@ 
 	 */
 	protected function getMembersLinks(array $objectIds)
 	{
-		if (empty($objectIds))
-			return [];
+		if (empty($objectIds)) {
+					return [];
+		}
 		$objectIds = array_unique($objectIds);
 		sort($objectIds, SORT_STRING);
 		$cacheKey = md5(serialize($objectIds));
@@ -1346,8 +1408,9 @@ 
 				->asArray()
 				->all();
 			$multilinks = array_fill_keys($objectIds, []);
-			foreach ($rows as $row)
-				$multilinks[$row['from_id']][$row['from_member']][] = $row['to_id'];
+			foreach ($rows as $row) {
+							$multilinks[$row['from_id']][$row['from_member']][] = $row['to_id'];
+			}
 			self::$_linkResultsCache[$cacheKey] = $multilinks;
 		}
 		return self::$_linkResultsCache[$cacheKey];
@@ -1364,8 +1427,9 @@ 
 	private function createChangeLogEntry($class, $changeKey, $objectUuid, $originalValues = [], $newValues = [])
 	{
 		static $changeLog = null;
-		if (!$changeLog)
-			$changeLog = neon('dds')->IDdsChangeLogManagement;
+		if (!$changeLog) {
+					$changeLog = neon('dds')->IDdsChangeLogManagement;
+		}
 		return $changeLog->addLogEntry($objectUuid, $class->toArray(), $changeKey, $originalValues, $newValues);
 	}
 

neon/settings/forms/AppSettings.php

Indentation +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 			foreach ($appSettings as $key => $value) {
 				// if the fields value is valid then save it
 				if ($appForm->getField($key)->validate()) {
-				    $saved = $this->getSettingsManager()->set($appKey, $key, $value);
+					$saved = $this->getSettingsManager()->set($appKey, $key, $value);
 				}
 			}   
 		}

neon/cms/components/Editor.php

Braces +9 added lines, -3 removed lines

@@ -55,7 +55,9 @@ 
 	 */
 	public static function editor($params, $content, \Smarty_Internal_Template $template, &$repeat)
 	{
-		if ($repeat) return;
+		if ($repeat) {
+			return;
+		}
 		$id = Arr::get($params, 'id', neon()->cms->getPage()->getId());
 		$dbContent = neon()->getCms()->getICmsStaticData()->getStaticContent($id, null);
 		if (!$dbContent) {
@@ -73,7 +75,9 @@ 
 			$element->normalize();
 
 			$dom = simplexml_import_dom($x);
-			foreach($dom as $d) dp($d);
+			foreach($dom as $d) {
+				dp($d);
+			}
 			exit;
 
 
@@ -168,7 +172,9 @@ 
 		unset($props['children']);
 		$formatParams = '';
 		foreach($props as $key => $value) {
-			if (is_array($value)) $value = var_export($value, true);
+			if (is_array($value)) {
+				$value = var_export($value, true);
+			}
 			$formatParams .= $key.'="'. $value .'" ';
 		}
 		return $formatParams;