newicon / neon

neon/cms/components/Theme.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 	 * @param array &$read - the set of read components to prevent infinite looping
 	 * @return array  the set of components
 	 */
-	private static function &getComponents($path, &$components=[], &$read=[])
+	private static function &getComponents($path, &$components = [], &$read = [])
 	{
 		$root = self::getRoot();
 		$configFile = $root.$path.'/_config.php';
@@ -91,7 +91,7 @@ 
 					}
 				}
 			} catch (\Exception $e) {
-				throw new \InvalidArgumentException("Something went wrong when trying to parse the config file $configFile. The error was ".$e->getMessage(). ". The file contained <pre>".print_r($config,true)."</pre>.");
+				throw new \InvalidArgumentException("Something went wrong when trying to parse the config file $configFile. The error was ".$e->getMessage().". The file contained <pre>".print_r($config, true)."</pre>.");
 			}
 		}
 		return $components;

Braces +6 added lines, -4 removed lines

@@ -29,8 +29,9 @@ 
 			if (is_string($root) && count($themes)) {
 				foreach ($themes as $theme) {
 					$themePath = $root.$theme;
-					if (!isset($hierarchy[$theme]))
-						$hierarchy[$theme] = [];
+					if (!isset($hierarchy[$theme])) {
+											$hierarchy[$theme] = [];
+					}
 
 					// add the default template and plugin directories
 					$hierarchy[$theme]['templates'][] = $themePath;
@@ -104,8 +105,9 @@ 
 	private static function getRoot()
 	{
 		static $root = null;
-		if ($root === null)
-			$root = Neon::getAlias('@root/themes/');
+		if ($root === null) {
+					$root = Neon::getAlias('@root/themes/');
+		}
 		return $root;
 	}
 }

neon/cms/components/Widget.php

Spacing +4 added lines, -4 removed lines

@@ -116,7 +116,7 @@ 
 		if (isset($params['type'])) {
 			$defaults['type'] = $params['type'];
 			// path to the file is determined by the theme hierarchy
-			$defaults['file'] = $params['type'] . '.tpl';
+			$defaults['file'] = $params['type'].'.tpl';
 			if (isset($params['key']))
 				self::buildDataFromCmsKeyAttributes($params);
  		}
@@ -231,7 +231,7 @@ 
 			return $this->createStaticEditable($content);
 		} else {
 			$file = $this->params['file'];
-			$this->params['hasData'] = count($this->data)>0;
+			$this->params['hasData'] = count($this->data) > 0;
 			$this->params['hasResponse'] = true;
 			$this->params['data'] = $this->data;
 			$this->params['edit'] = $this->createDynamicEditable();
@@ -297,7 +297,7 @@ 
 	public function getId()
 	{
 		if ($this->_id === null) {
-			$this->_id = $this->getPage()->getId() . '_' . static::$autoIdPrefix . static::$counter++;
+			$this->_id = $this->getPage()->getId().'_'.static::$autoIdPrefix.static::$counter++;
 		}
 		return $this->_id;
 	}
@@ -339,7 +339,7 @@ 
 			$dataKey = $this->params['data-key'];
 			$dataPageId = $this->params['data-page-id'];
 			if ((isset($this->params['wys']) || isset($this->params['wysiwyg'])) || (isset($this->params['editor']) && strpos('wys', $this->params['editor'] === 0)))
-				return '<div id="'.$this->getId().'" data-id="'.$this->getId().'" data-key="'.$dataKey.'" data-page-id="'.$dataPageId.'" class="cobe-editable cobe-static-block" contenteditable="true">' . $value . '</div>';
+				return '<div id="'.$this->getId().'" data-id="'.$this->getId().'" data-key="'.$dataKey.'" data-page-id="'.$dataPageId.'" class="cobe-editable cobe-static-block" contenteditable="true">'.$value.'</div>';
 			else
 				return "<div style='display:inline-block;' class='cobe-editable cobe-static'  data-key='$dataKey' data-page-id='$dataPageId' contenteditable='true' />$value</div>";
 		} else {

Braces +29 added lines, -20 removed lines

@@ -117,11 +117,13 @@ 
 			$defaults['type'] = $params['type'];
 			// path to the file is determined by the theme hierarchy
 			$defaults['file'] = $params['type'] . '.tpl';
-			if (isset($params['key']))
-				self::buildDataFromCmsKeyAttributes($params);
+			if (isset($params['key'])) {
+							self::buildDataFromCmsKeyAttributes($params);
+			}
  		}
-		if (isset($params['dds']) || isset($params['dds_page']) || isset($params['dds_class']))
-			Daedalus::buildDataFromDdsAttributes($params);
+		if (isset($params['dds']) || isset($params['dds_page']) || isset($params['dds_class'])) {
+					Daedalus::buildDataFromDdsAttributes($params);
+		}
 		$resolvedParams = ArrayHelper::merge($defaults, $params);
 		$out = $widget->render($resolvedParams, $template);
 		return $out;
@@ -170,8 +172,9 @@ 
 		}
 		// scb = static content block
 		$params['type'] = '_scb';
-		if (isset($params['key']))
-			self::buildDataFromCmsKeyAttributes($params);
+		if (isset($params['key'])) {
+					self::buildDataFromCmsKeyAttributes($params);
+		}
 		$params['content'] = $content;
 		$params['wys'] = (isset($params['wys']) ? $params['wys'] : true);
 		return $widget->render($params, $template);
@@ -207,8 +210,9 @@ 
 		$data = [];
 		if (($dataRequest = $this->getDataRequest()) !== null) {
 			$id = $meta = null;
-			if (!$this->getPage()->requestWidgetData($dataRequest, $data, $id, $this->isStatic, $meta))
-				return $id;
+			if (!$this->getPage()->requestWidgetData($dataRequest, $data, $id, $this->isStatic, $meta)) {
+							return $id;
+			}
 			$this->saveData($data);
 			$this->saveDataMetaInformation($meta);
 		}
@@ -238,8 +242,9 @@ 
 			$this->params['meta'] = $this->meta;
 
 			// createTemplate is an expensive call so only call it once per file and reset all variables on the stored template.
-			if (!isset($templates[$file]))
-				$templates[$file] = $this->renderer->createTemplate($file);
+			if (!isset($templates[$file])) {
+							$templates[$file] = $this->renderer->createTemplate($file);
+			}
 			$tpl = $templates[$file];
 			$tpl->tpl_vars = $tpl->config_vars = [];
 
@@ -257,8 +262,9 @@ 
 	 */
 	protected function getDataRequest()
 	{
-		if (isset($this->params['data']))
-			return Daedalus::convertDataRequestToJson($this->params['data']);
+		if (isset($this->params['data'])) {
+					return Daedalus::convertDataRequestToJson($this->params['data']);
+		}
 		return null;
 	}
 
@@ -314,15 +320,17 @@ 
 	 */
 	protected static function buildDataFromCmsKeyAttributes(&$params)
 	{
-		if (!isset($params['type'], $params['key']))
-			return;
+		if (!isset($params['type'], $params['key'])) {
+					return;
+		}
 		$key = "$params[type]_$params[key]";
 		$pageId = isset($params['page']) ? self::$page->getId() : null;
 		$params['data-key'] = $key;
 		$params['data-page-id'] = $pageId;
 		$params['data'] = "'key':'$key','pageId':'$pageId'";
-		if (!isset($params['content']))
-			$params['content'] = "Add Content for $params[key] here";
+		if (!isset($params['content'])) {
+					$params['content'] = "Add Content for $params[key] here";
+		}
 		$params['is_static'] = true;
 	}
 
@@ -338,10 +346,11 @@ 
 		if ($this->getPage()->isInEditMode()) {
 			$dataKey = $this->params['data-key'];
 			$dataPageId = $this->params['data-page-id'];
-			if ((isset($this->params['wys']) || isset($this->params['wysiwyg'])) || (isset($this->params['editor']) && strpos('wys', $this->params['editor'] === 0)))
-				return '<div id="'.$this->getId().'" data-id="'.$this->getId().'" data-key="'.$dataKey.'" data-page-id="'.$dataPageId.'" class="cobe-editable cobe-static-block" contenteditable="true">' . $value . '</div>';
-			else
-				return "<div style='display:inline-block;' class='cobe-editable cobe-static'  data-key='$dataKey' data-page-id='$dataPageId' contenteditable='true' />$value</div>";
+			if ((isset($this->params['wys']) || isset($this->params['wysiwyg'])) || (isset($this->params['editor']) && strpos('wys', $this->params['editor'] === 0))) {
+							return '<div id="'.$this->getId().'" data-id="'.$this->getId().'" data-key="'.$dataKey.'" data-page-id="'.$dataPageId.'" class="cobe-editable cobe-static-block" contenteditable="true">' . $value . '</div>';
+			} else {
+							return "<div style='display:inline-block;' class='cobe-editable cobe-static'  data-key='$dataKey' data-page-id='$dataPageId' contenteditable='true' />$value</div>";
+			}
 		} else {
 			return $value;
 		}

neon/cms/components/Daedalus.php

Spacing +13 added lines, -13 removed lines

@@ -173,7 +173,7 @@ 
 	 * @param array $params  any paramters from @see dds and dds_choice
 	 * @param object $renderer  the render object
 	 */
-	public function __construct($params, $renderer=null)
+	public function __construct($params, $renderer = null)
 	{
 		$this->params = $params;
 		$this->renderer = $renderer;
@@ -198,7 +198,7 @@ 
 			$request = self::convertDataRequestToJson($this->params['data']);
 			$responded = self::$page->requestWidgetData($request, $data, $id, false, $meta);
 			if ($responded) {
-				$result['hasData'] = count($data)>0;
+				$result['hasData'] = count($data) > 0;
 				$result['hasResponse'] = true;
 				$result['data'] = $data;
 				$result['meta'] = $meta;
@@ -223,21 +223,21 @@ 
 	 * @return null|errors|data  returns errors if not valid, null if called
 	 *   from within a template and the data if from within a plugin
 	 */
-	protected function queryChoice($template=true)
+	protected function queryChoice($template = true)
 	{
 		// check usage
 		if (!isset($this->params['member']))
-			return "Usage: you must provide a 'member' parameter. You provided ".print_r($this->params,true);
+			return "Usage: you must provide a 'member' parameter. You provided ".print_r($this->params, true);
 		if ($this->renderer && !isset($this->params['assign']))
-			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
-		if (strpos($this->params['member'],':') === false && !isset($this->params['class']))
+			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params, true);
+		if (strpos($this->params['member'], ':') === false && !isset($this->params['class']))
 			return "Usage: if you do not scope the member as class::member then you must provide the class parameter";
 
 		// extract out class and member
 		$class = null;
 		$member = null;
-		if (strpos($this->params['member'],':') !== false)
-			list ($class, $member) = explode(':', str_replace('::',':',$this->params['member']));
+		if (strpos($this->params['member'], ':') !== false)
+			list ($class, $member) = explode(':', str_replace('::', ':', $this->params['member']));
 		else {
 			$class = $this->params['class'];
 			$member = $this->params['member'];
@@ -285,10 +285,10 @@ 
 	{
 		// check usage
 		if (!isset($this->params['class']))
-			return "Usage: you must provide a 'class' parameter. You provided ".print_r($this->params,true);
+			return "Usage: you must provide a 'class' parameter. You provided ".print_r($this->params, true);
 
 		if ($this->renderer && !isset($this->params['assign']))
-			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
+			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params, true);
 
 		$fields = isset($this->params['fields']) ? $this->params['fields'] : [];
 
@@ -385,7 +385,7 @@ 
 				if (is_array($dps['dds_filter']))
 					$filters[] = json_encode($dps['dds_filter']);
 				// else deprecated json approach
-				else if (strpos(trim($dps['dds_filter']), '[')!==0)
+				else if (strpos(trim($dps['dds_filter']), '[') !== 0)
 					$filters[] = "[$dps[dds_filter]]";
 				else
 					$filters[] = $dps['dds_filter'];
@@ -393,7 +393,7 @@ 
 
 			// create the filters clause
 			if (count($filters))
-				$data[] = "'filters':".implode(',',$filters);
+				$data[] = "'filters':".implode(',', $filters);
 
 			// sort out the order
 			if (isset($dps['dds_order'])) {
@@ -458,7 +458,7 @@ 
 				$classType = $d['_class_type'];
 				foreach ($d as $field=>$value) {
 					// ignore all bookkeeping fields
-					if (strpos($field, '_')===0)
+					if (strpos($field, '_') === 0)
 						continue;
 					$edit[$i][$field] = self::createFieldEditable($classType, $field, $d['_uuid'], $value);
 				}

Braces +94 added lines, -68 removed lines

@@ -205,10 +205,11 @@ 
 				$result['edit'] = $this->createEditable($data);
 			}
 		}
-		if ($this->renderer)
-			$this->renderer->assign($this->params['assign'], $result);
-		else
-			return $result;
+		if ($this->renderer) {
+					$this->renderer->assign($this->params['assign'], $result);
+		} else {
+					return $result;
+		}
 	}
 
 	/**
@@ -226,19 +227,22 @@ 
 	protected function queryChoice($template=true)
 	{
 		// check usage
-		if (!isset($this->params['member']))
-			return "Usage: you must provide a 'member' parameter. You provided ".print_r($this->params,true);
-		if ($this->renderer && !isset($this->params['assign']))
-			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
-		if (strpos($this->params['member'],':') === false && !isset($this->params['class']))
-			return "Usage: if you do not scope the member as class::member then you must provide the class parameter";
+		if (!isset($this->params['member'])) {
+					return "Usage: you must provide a 'member' parameter. You provided ".print_r($this->params,true);
+		}
+		if ($this->renderer && !isset($this->params['assign'])) {
+					return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
+		}
+		if (strpos($this->params['member'],':') === false && !isset($this->params['class'])) {
+					return "Usage: if you do not scope the member as class::member then you must provide the class parameter";
+		}
 
 		// extract out class and member
 		$class = null;
 		$member = null;
-		if (strpos($this->params['member'],':') !== false)
-			list ($class, $member) = explode(':', str_replace('::',':',$this->params['member']));
-		else {
+		if (strpos($this->params['member'],':') !== false) {
+					list ($class, $member) = explode(':', str_replace('::',':',$this->params['member']));
+		} else {
 			$class = $this->params['class'];
 			$member = $this->params['member'];
 		}
@@ -250,10 +254,11 @@ 
 		if (!isset(self::$_choices[$key])) {
 			$dds = neon('dds')->iDdsClassManagement;
 			$result = $dds->getMember($class, $member, ['choices']);
-			if (!empty($result) && isset($result['choices']))
-				self::$_choices[$key] = $result['choices'];
-			else
-				self::$_choices[$key] = [];
+			if (!empty($result) && isset($result['choices'])) {
+							self::$_choices[$key] = $result['choices'];
+			} else {
+							self::$_choices[$key] = [];
+			}
 		}
 		$choices = self::$_choices[$key];
 		if (isset($this->params['order'])) {
@@ -263,10 +268,11 @@ 
 				asort($choices);
 			}
 		}
-		if ($this->renderer)
-			$this->renderer->assign($this->params['assign'], $choices);
-		else
-			return $choices;
+		if ($this->renderer) {
+					$this->renderer->assign($this->params['assign'], $choices);
+		} else {
+					return $choices;
+		}
 	}
 
 	/**
@@ -284,11 +290,13 @@ 
 	protected function queryMap()
 	{
 		// check usage
-		if (!isset($this->params['class']))
-			return "Usage: you must provide a 'class' parameter. You provided ".print_r($this->params,true);
+		if (!isset($this->params['class'])) {
+					return "Usage: you must provide a 'class' parameter. You provided ".print_r($this->params,true);
+		}
 
-		if ($this->renderer && !isset($this->params['assign']))
-			return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
+		if ($this->renderer && !isset($this->params['assign'])) {
+					return "Usage: you must provide an 'assign' parameter. You provided ".print_r($this->params,true);
+		}
 
 		$fields = isset($this->params['fields']) ? $this->params['fields'] : [];
 
@@ -307,11 +315,13 @@ 
 			self::$_maps[$class] = $result;
 		}
 		$map = self::$_maps[$class];
-		if (isset($this->params['flip']))
-			$map = array_flip($map);
+		if (isset($this->params['flip'])) {
+					$map = array_flip($map);
+		}
 
-		if ($this->renderer)
-			$this->renderer->assign($this->params['assign'], $map);
+		if ($this->renderer) {
+					$this->renderer->assign($this->params['assign'], $map);
+		}
 
 		return $map;
 	}
@@ -328,11 +338,13 @@ 
 		$json = trim(str_replace("'", '"', $data));
 		if (strlen($json) > 0) {
 			// for convenience the beginning and ending brackets are optional so add if not supplied
-			if (strpos($json, '{') !== 0)
-				$json = '{'.$json.'}';
+			if (strpos($json, '{') !== 0) {
+							$json = '{'.$json.'}';
+			}
 			$request = json_decode($json, true);
-			if ($request === null)
-				throw new \Exception("Failing to json decode the data request: $json converted from $data. ");
+			if ($request === null) {
+							throw new \Exception("Failing to json decode the data request: $json converted from $data. ");
+			}
 		}
 		return $request;
 	}
@@ -357,10 +369,12 @@ 
 		$dps = array_intersect_key($params, array_flip($dpa));
 		if (count($dps)) {
 			$data = [];
-			if ((isset($dps['dds']) || isset($dps['dds_page'])) && empty($dps['dds_class']))
-				$dps['dds_class'] = $params['type'];
-			if (isset($dps['dds_class']))
-				$data[] = "'class':'$dps[dds_class]'";
+			if ((isset($dps['dds']) || isset($dps['dds_page'])) && empty($dps['dds_class'])) {
+							$dps['dds_class'] = $params['type'];
+			}
+			if (isset($dps['dds_class'])) {
+							$data[] = "'class':'$dps[dds_class]'";
+			}
 
 			// create the filters
 			$filters = [];
@@ -368,51 +382,60 @@ 
 			// create the page specific filter
 			if (!empty($dps['dds_page'])) {
 				$pageId = null;
-				if ($dps['dds_page'] === true || $dps['dds_page'] === 'true')
-					$pageId = self::$page->getId();
-				else {
+				if ($dps['dds_page'] === true || $dps['dds_page'] === 'true') {
+									$pageId = self::$page->getId();
+				} else {
 					// either a nice id or a page id
 					$pageId = self::$page->niceId2PageId($dps['dds_page']);
-					if (!$pageId)
-						$pageId = $dps['dds_page'];
+					if (!$pageId) {
+											$pageId = $dps['dds_page'];
+					}
+				}
+				if ($pageId) {
+									$filters[] = "['page_id','=','$pageId']";
 				}
-				if ($pageId)
-					$filters[] = "['page_id','=','$pageId']";
 			}
 
 			// add other filters
 			if (isset($dps['dds_filter']) && !empty($dps['dds_filter'])) {
-				if (is_array($dps['dds_filter']))
-					$filters[] = json_encode($dps['dds_filter']);
+				if (is_array($dps['dds_filter'])) {
+									$filters[] = json_encode($dps['dds_filter']);
+				}
 				// else deprecated json approach
-				else if (strpos(trim($dps['dds_filter']), '[')!==0)
-					$filters[] = "[$dps[dds_filter]]";
-				else
-					$filters[] = $dps['dds_filter'];
+				else if (strpos(trim($dps['dds_filter']), '[')!==0) {
+									$filters[] = "[$dps[dds_filter]]";
+				} else {
+									$filters[] = $dps['dds_filter'];
+				}
 			}
 
 			// create the filters clause
-			if (count($filters))
-				$data[] = "'filters':".implode(',',$filters);
+			if (count($filters)) {
+							$data[] = "'filters':".implode(',',$filters);
+			}
 
 			// sort out the order
 			if (isset($dps['dds_order'])) {
-				if (is_array($dps['dds_order']))
-					$data[] = "'order':".json_encode($dps['dds_order']);
+				if (is_array($dps['dds_order'])) {
+									$data[] = "'order':".json_encode($dps['dds_order']);
+				}
 				// else deprecated json approach
-				else
-					$data[] = "'order':{".$dps['dds_order']."}";
+				else {
+									$data[] = "'order':{".$dps['dds_order']."}";
+				}
 			}
 
 			// sort out the limit
 			if (isset($dps['dds_limit'])) {
-				if (is_numeric($dps['dds_limit']))
-					$data[] = "'limit':{'length':$dps[dds_limit]}";
-				else if (is_array($dps['dds_limit']))
-					$data[] = "'limit':".json_encode($dps['dds_limit']);
+				if (is_numeric($dps['dds_limit'])) {
+									$data[] = "'limit':{'length':$dps[dds_limit]}";
+				} else if (is_array($dps['dds_limit'])) {
+									$data[] = "'limit':".json_encode($dps['dds_limit']);
+				}
 				// else deprecated json approach
-				else
-					$data[] = "'limit':{".$dps['dds_limit']."}";
+				else {
+									$data[] = "'limit':{".$dps['dds_limit']."}";
+				}
 			}
 			$params['data'] = '{'.implode(', ', $data).'}';
 		}
@@ -429,8 +452,9 @@ 
 	protected function canonicalise($params)
 	{
 		$canon = [];
-		if (empty($params['assign']) && $this->renderer !== null)
-			throw new \InvalidArgumentException("You must set an 'assign' parameter when using the 'dds' tag");
+		if (empty($params['assign']) && $this->renderer !== null) {
+					throw new \InvalidArgumentException("You must set an 'assign' parameter when using the 'dds' tag");
+		}
 		foreach ($params as $k => $v) {
 			if ($k == 'assign') {
 				$canon[$k] = $v;
@@ -458,8 +482,9 @@ 
 				$classType = $d['_class_type'];
 				foreach ($d as $field=>$value) {
 					// ignore all bookkeeping fields
-					if (strpos($field, '_')===0)
-						continue;
+					if (strpos($field, '_')===0) {
+											continue;
+					}
 					$edit[$i][$field] = self::createFieldEditable($classType, $field, $d['_uuid'], $value);
 				}
 			}
@@ -477,8 +502,9 @@ 
 	 */
 	public static function createFieldEditable($classType, $field, $key, $value)
 	{
-		if (is_array($value))
-			return $value;
+		if (is_array($value)) {
+					return $value;
+		}
 
 		/**
 		 * This needs to be changed to generate an appropriate form to edit the

neon/cms/grid/PageGrid.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 
 
 		$this->addButtonColumn('edit')
-			->addButton('edit', ['/cms/editor/index' , 'id' => '{{nice_id}}'], '', '', '', ['data-toggle' => 'tooltip', 'title'=>'Edit']);
+			->addButton('edit', ['/cms/editor/index', 'id' => '{{nice_id}}'], '', '', '', ['data-toggle' => 'tooltip', 'title'=>'Edit']);
 
 		$this->addScope('published', 'Published');
 		$this->addScope('draft', 'Draft');

neon/cms/services/cmsManager/CmsManager.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	public function bulkStaticContentRequest($requests)
 	{
 		$requestedData = [];
-		if (count($requests)>0) {
+		if (count($requests) > 0) {
 			$query = CmsStaticContent::find()
 				->select("`key`, `page_id` as `pageId`, `content`");
 			foreach ($requests as $id => $request) {

neon/cms/form/Page.php

Spacing +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
 	{
 		$theme = neon()->getAlias(neon()->cms->getThemeAlias());
 		$templates = [];
-		foreach(File::findFiles("$theme", ['only' => ['*.tpl', '*.twig']]) as $file) {
+		foreach (File::findFiles("$theme", ['only' => ['*.tpl', '*.twig']]) as $file) {
 			$path = str_replace("$theme/", '', $file);
 			if (Str::startsWith($path, 'pages/') || Str::startsWith($path, 'partials/')) {
 				$templates[$path] = $path;
@@ -177,7 +177,7 @@ 
 		// create a migration file
 		$templateFile = __DIR__.'/templates/ddsMigration.php';
 		$input = file_get_contents($templateFile);
-		$output = str_replace('__MIGRATION__',$this->_classname,$input);
+		$output = str_replace('__MIGRATION__', $this->_classname, $input);
 		file_put_contents($this->_file, $output, LOCK_EX);
 		return $this->_filename;
 

neon/cms/controllers/DataController.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 	 * Update a static content widget
 	 * @return void
 	 */
-	public function actionUpdateContentWidget($content=null)
+	public function actionUpdateContentWidget($content = null)
 	{
 		$key = neon()->request->post('key');
 		$content = $content ? $content : neon()->request->postUnsafe('content');

Braces +13 added lines, -10 removed lines

@@ -39,17 +39,19 @@ 
 	{
 		// see if we have a key and if not we can do nothing
 		$key = neon()->request->post('key');
-		if (empty($key))
-			return;
+		if (empty($key)) {
+					return;
+		}
 		$content = neon()->request->postUnsafe('content');
 		$content = Html::sanitise($content, $this->inlineAllowedTags);
 		$pageId = neon()->request->post('pageId', null);
 		try {
 			$result = neon('cms')->getICmsStaticData()->editStaticContent($key, $pageId, $content);
-			if ($result === true)
-				return ['success'=>true, 'saved'=>$content];
-			else
-				return neon()->debug ? ['success'=>false, 'errors'=>$result] : ['success'=>false];
+			if ($result === true) {
+							return ['success'=>true, 'saved'=>$content];
+			} else {
+							return neon()->debug ? ['success'=>false, 'errors'=>$result] : ['success'=>false];
+			}
 		} catch (\Exception $e) {
 			return neon()->debug ? ['success'=>false, 'errors'=>$e->getMessage()] : ['success'=>false];
 		}
@@ -67,10 +69,11 @@ 
 		$pageId = neon()->request->post('pageId', null);
 		try {
 			$result = neon()->cms->getICmsStaticData()->editStaticContent($key, $pageId, $content);
-			if ($result === true)
-				return ['success'=>true, 'saved'=>$content];
-			else if (neon()->debug)
-				return neon()->debug ? ['success'=>false, 'errors'=>$result] : ['success'=>false];
+			if ($result === true) {
+							return ['success'=>true, 'saved'=>$content];
+			} else if (neon()->debug) {
+							return neon()->debug ? ['success'=>false, 'errors'=>$result] : ['success'=>false];
+			}
 		} catch (\Exception $e) {
 			return neon()->debug ? ['success'=>false, 'errors'=>$e->getMessage()] : ['success'=>false];
 		}

neon/cms/controllers/EditorController.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 {
 	public $layout = '/admin-workbench';
 
-	public function actionIndex($id=null)
+	public function actionIndex($id = null)
 	{
 		$grid = new \neon\cms\grid\PageGrid();
 		$pages = $grid->getRows();
@@ -40,7 +40,7 @@ 
 		if ($pageForm->processRequest()) {
 			if (!$pageForm->save($errors)) {
 				// something went wrong
-				throw new \Exception( 'Something went wrong ' . print_r($errors, true));
+				throw new \Exception('Something went wrong '.print_r($errors, true));
 			}
 			$data = $pageForm->getData();
 			return $this->redirect(['/cms/editor/index', 'id' => $data['nice_id']]);

neon/cms/controllers/RenderController.php

Braces +6 added lines, -4 removed lines

@@ -97,8 +97,9 @@ 
 		}
 
 		$render = $page->render();
-		if ($render === false)
-			throw new \yii\web\HttpException(404, 'No template found');
+		if ($render === false) {
+					throw new \yii\web\HttpException(404, 'No template found');
+		}
 
 		\Neon::endProfile('COBE::RENDER_ACTION', 'cobe');
 		return $render;
@@ -112,8 +113,9 @@ 
 	{
 		$request = neon()->request;
 		$url = $request->get('page');
-		if (!$url)
-			$this->pageNotFound();
+		if (!$url) {
+					$this->pageNotFound();
+		}
 		\Neon::app('cms')->page->setByUrl($url);
 		$pageData = neon()->getCms()->getPage()->getPageData();
 		return $this->actionPage($pageData['nice_id']);