newicon / neon

neon/core/form/traits/BuilderTrait.php

Spacing +23 added lines, -23 removed lines

@@ -84,7 +84,7 @@ 
 	 * @param boolean $overwrite (optional) - whether to overwrite an existing field of the same name defaults to false
 	 * @return fields\Field|Form|FormRepeater
 	 */
-	abstract public function add($field, $defaults=[], $overwrite=false);
+	abstract public function add($field, $defaults = [], $overwrite = false);
 
 	/**
 	 * Adds a sub form box to the form
@@ -93,7 +93,7 @@ 
 	 * @param array $options
 	 * @return IField|Form
 	 */
-	public function addSubForm($name, $options=[])
+	public function addSubForm($name, $options = [])
 	{
 		if (is_object($name)) {
 			return $this->add($name);
@@ -109,7 +109,7 @@ 
 	 * @param array $options
 	 * @return IField|fields\Text
 	 */
-	public function addFieldText($name, $options=[])
+	public function addFieldText($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'text']);
 	}
@@ -121,7 +121,7 @@ 
 	 * @param array $options
 	 * @return IField|fields\Password
 	 */
-	public function addFieldPassword($name, $options=[])
+	public function addFieldPassword($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'password']);
 	}
@@ -132,7 +132,7 @@ 
 	 * @param string $name
 	 * @return IField|fields\Text
 	 */
-	public function addFieldFile($name, $options=[])
+	public function addFieldFile($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'file', 'name' => $name]);
 	}
@@ -145,7 +145,7 @@ 
 	 * @param string $name
 	 * @return IField|fields\Text
 	 */
-	public function addFieldImage($name, $options=[])
+	public function addFieldImage($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'image', 'name' => $name]);
 	}
@@ -156,7 +156,7 @@ 
 	 * @param string $name
 	 * @return IField|fields\Email
 	 */
-	public function addFieldEmail($name, $options=[])
+	public function addFieldEmail($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'email', 'name' => $name]);
 	}
@@ -168,7 +168,7 @@ 
 	 * @param array $options
 	 * @return IField|fields\Submit
 	 */
-	public function addFieldSubmit($name, $options=[])
+	public function addFieldSubmit($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'submit', 'name' => $name]);
 	}
@@ -180,7 +180,7 @@ 
 	 * @param array $options
 	 * @return IField|fields\Text
 	 */
-	public function addFieldHidden($name, $options=[])
+	public function addFieldHidden($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'hidden', 'name' => $name]);
 	}
@@ -192,7 +192,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Checkbox
 	 */
-	public function addFieldCheckbox($name, $options=[])
+	public function addFieldCheckbox($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'checkbox', 'name' => $name]);
 	}
@@ -204,7 +204,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Checkbox
 	 */
-	public function addFieldCheckList($name, $options=[])
+	public function addFieldCheckList($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'checklist', 'name' => $name]);
 	}
@@ -216,7 +216,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Date
 	 */
-	public function addFieldDate($name, $options=[])
+	public function addFieldDate($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'date']);
 	}
@@ -228,7 +228,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\DateTime
 	 */
-	public function addFieldDateTime($name, $options=[])
+	public function addFieldDateTime($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'datetime']);
 	}
@@ -240,7 +240,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Integer
 	 */
-	public function addFieldInteger($name, $options=[])
+	public function addFieldInteger($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'integer']);
 	}
@@ -252,7 +252,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Select
 	 */
-	public function addFieldSelect($name, $options=[])
+	public function addFieldSelect($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'select', 'name' => $name]);
 	}
@@ -264,7 +264,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Section
 	 */
-	public function addFieldSection($name, $options=[])
+	public function addFieldSection($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'section', 'name' => $name]);
 	}
@@ -276,7 +276,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\SwitchButton
 	 */
-	public function addFieldSwitch($name, $options=[])
+	public function addFieldSwitch($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'switch', 'name' => $name]);
 	}
@@ -288,7 +288,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Textarea
 	 */
-	public function addFieldTextArea($name, $options=[])
+	public function addFieldTextArea($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'textarea', 'name' => $name]);
 	}
@@ -300,7 +300,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Time
 	 */
-	public function addFieldTime($name, $options=[])
+	public function addFieldTime($name, $options = [])
 	{
 		return $this->add($options, ['name' => $name, 'class' => 'time']);
 	}
@@ -312,7 +312,7 @@ 
 	 * @param array  $options
 	 * @return IField|fields\Wysiwyg
 	 */
-	public function addFieldWysiwyg($name, $options=[])
+	public function addFieldWysiwyg($name, $options = [])
 	{
 		return $this->add($options, ['class' => 'wysiwyg', 'name' => $name]);
 	}
@@ -350,14 +350,14 @@ 
 	 * @throws UnknownFieldClass if the class is not found
 	 * @return IField
 	 */
-	public function makeField($field, $defaults=[])
+	public function makeField($field, $defaults = [])
 	{
 		if (is_array($field))
 			$field = $this->createFieldObjectFromConfig($field, $defaults);
 
 		if (!($field instanceof IField || $field instanceof Form)) {
 			// add this form as the parent form reference
-			throw new \InvalidArgumentException('The $field parameter was not a valid \neon\core\form\fields\Field or \neon\core\form\Form object "' . print_r($field, true) . '" given');
+			throw new \InvalidArgumentException('The $field parameter was not a valid \neon\core\form\fields\Field or \neon\core\form\Form object "'.print_r($field, true).'" given');
 		}
 		return $field;
 	}
@@ -381,7 +381,7 @@ 
 	 * @throws UnknownFieldClass
 	 * @return IField
 	 */
-	public function createFieldObjectFromConfig($config, $defaults=[])
+	public function createFieldObjectFromConfig($config, $defaults = [])
 	{
 		$config = Arr::merge($defaults, $config);
 		// allows class config property to be specified by an alias "switch"

Braces +6 added lines, -4 removed lines

@@ -352,8 +352,9 @@ 
 	 */
 	public function makeField($field, $defaults=[])
 	{
-		if (is_array($field))
-			$field = $this->createFieldObjectFromConfig($field, $defaults);
+		if (is_array($field)) {
+					$field = $this->createFieldObjectFromConfig($field, $defaults);
+		}
 
 		if (!($field instanceof IField || $field instanceof Form)) {
 			// add this form as the parent form reference
@@ -422,8 +423,9 @@ 
 	 */
 	public function getFieldClassByAlias($alias)
 	{
-		if (!$this->fieldAliasExists($alias))
-			throw new \InvalidArgumentException("There is no alias with the key '$alias' in the Forms fieldAliases array");
+		if (!$this->fieldAliasExists($alias)) {
+					throw new \InvalidArgumentException("There is no alias with the key '$alias' in the Forms fieldAliases array");
+		}
 		return $this->fieldAliases[strtolower($alias)];
 	}
 

neon/core/form/Deprecated.php

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
 	 * @return \neon\core\form\Form
 	 * @deprecated should be using phoebe this should be replaced with using the loadFromDefinition via phoebe
 	 */
-	public static function ddsLoadFrom($form, $classType, $includeDeleted=false)
+	public static function ddsLoadFrom($form, $classType, $includeDeleted = false)
 	{
 		\Neon::warning('neon\core\form\Form::ddsLoadFrom is a deprecated function - phoebe is responsible for loading form definitions');
 		$class = neon()->getDds()->getIDdsClassManagement()->getClass($classType);
@@ -58,11 +58,11 @@ 
 		$formDefinition = Arr::merge($formDefinition, $immutable);
 		// remove the class option
 		Arr::remove($formDefinition, 'class');
-		foreach($formDefinition as $key => $value) {
+		foreach ($formDefinition as $key => $value) {
 			$form->$key = $value;
 		}
 		// Add fields to the form from the individual members
-		if (! empty($members) ) {
+		if (!empty($members)) {
 			foreach ($members as $member) {
 				$form->ddsAddField($member);
 			}
@@ -78,7 +78,7 @@ 
 	 * @return void
 	 * @deprecated
 	 */
-	public static function ddsSaveDefinition($form, $classType=null, $ddsMembers=null)
+	public static function ddsSaveDefinition($form, $classType = null, $ddsMembers = null)
 	{
 		\Neon::warning('neon\core\form\Form::ddsSaveDefinition is a deprecated function - phoebe is now responsible for saving form definitions');
 		// \neon\phoebe\builders\database\builder\Dds::saveDefinition($form);
@@ -97,7 +97,7 @@ 
 			]);
 		}
 		$members = $ddsClassManager->listMembers($classType, true);
-		foreach($form->getFields() as $key => $field) {
+		foreach ($form->getFields() as $key => $field) {
 			// set the order value to be the same as how we received them
 			$field->ddsSaveDefinition($classType, $members, $key);
 		}
@@ -143,7 +143,7 @@ 
 		]);
 		// the definition might supply more information
 		$widgetDefinition = [];
-		if (!empty($member['definition'])){
+		if (!empty($member['definition'])) {
 			// if the definition is already array then assume it is decoded - if it is not then assume it is a json string
 			$widgetDefinition = is_array($member['definition'])
 				? $member['definition']
@@ -151,7 +151,7 @@ 
 		}
 		// remove the value from the widget definition as this should be populated by the actual data
 		Arr::remove($widgetDefinition, 'value');
-		$config = Arr::merge($defaults, $widgetDefinition , $immutable);
+		$config = Arr::merge($defaults, $widgetDefinition, $immutable);
 
 		$form->add($config);
 	}
@@ -166,7 +166,7 @@ 
 	 * @param array|null $ddsMembers if not specified it will look up the members of the classType
 	 * @deprecated
 	 */
-	public static function ddsSaveDefinitionField($field, $classType=null, $ddsMembers=null)
+	public static function ddsSaveDefinitionField($field, $classType = null, $ddsMembers = null)
 	{
 		\Neon::warning('neon\core\form\field::ddsSaveDefinition is a deprecated function - you should be using phoebe');
 		$iDdsClassManagement = neon()->getDds()->getIDdsClassManagement();

Braces +3 added lines, -2 removed lines

@@ -180,8 +180,9 @@
 		}
 
 		// ignore if this is a deleted field
-		if ($field->deleted == 1)
-			return;
+		if ($field->deleted == 1) {
+					return;
+		}
 
 		// check if member exists
 		$memberRef = $field->getDdsName();

neon/core/form/interfaces/IValidate.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 	 * @param bool $required
 	 * @return $this method can be chained
 	 */
-	public function setRequired($required=true);
+	public function setRequired($required = true);
 
 	/**
 	 * Add a validator to a form field
@@ -52,7 +52,7 @@ 
 	 *
 	 * @return $this Is a chainable function
 	 */
-	public function addValidator($validator, $options=[]);
+	public function addValidator($validator, $options = []);
 
 	/**
 	 * @param array $validators

neon/core/form/assets/CkeditorAsset.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
 	public function init()
 	{
-		$this->sourcePath = __DIR__ . '/vendor/ckeditor';
+		$this->sourcePath = __DIR__.'/vendor/ckeditor';
 		parent::init();
 	}
 }

neon/core/form/FormRepeater.php

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
 		if (!is_array($value))
 			throw new \InvalidArgumentException('The value should be an array');
 
-		foreach($value as $position => $values) {
+		foreach ($value as $position => $values) {
 			if (!$this->hasField($position)) {
 				$this->addInstance($position);
 			}
@@ -120,7 +120,7 @@ 
 	{
 		$subFormCount = count(parent::getSubForms());
 		if ($subFormCount < $this->count) {
-			for ($i = $subFormCount; $i <= $this->count-1; $i++) {
+			for ($i = $subFormCount; $i <= $this->count - 1; $i++) {
 				$this->addInstance($i);
 			}
 		}
@@ -131,7 +131,7 @@ 
 	 * The fields in the template must register their scripts
 	 * @param null $view
 	 */
-	public function registerScripts($view=null, $mount=true)
+	public function registerScripts($view = null, $mount = true)
 	{
 		$view = ($view === null) ? $this->getView() : $view;
 		foreach ($this->getTemplate()->getFields() as $field)
@@ -209,11 +209,11 @@ 
 	 * Get fields - also add additional instances as appropriate
 	 * @inheritdoc
 	 */
-	public function getFields($names=null)
+	public function getFields($names = null)
 	{
 		$subFormCount = count(parent::getSubForms());
 		if ($subFormCount < $this->count) {
-			for ($i = $subFormCount; $i <= $this->count-1; $i++) {
+			for ($i = $subFormCount; $i <= $this->count - 1; $i++) {
 				$this->addInstance(Hash::uuid64());
 			}
 		}

Braces +15 added lines, -10 removed lines

@@ -60,8 +60,9 @@ 
 	public function loadFromDb($data)
 	{
 		foreach ($data as $name => $value) {
-			if (!$this->hasField($name))
-				$this->addInstance($name);
+			if (!$this->hasField($name)) {
+							$this->addInstance($name);
+			}
 			$this->getField($name)->setValueFromDb($value);
 		}
 	}
@@ -102,8 +103,9 @@ 
 	{
 		// throw an exception - should probably cope with this scenario but set
 		// a validation to fail rather than throwing toys out of the pram
-		if (!is_array($value))
-			throw new \InvalidArgumentException('The value should be an array');
+		if (!is_array($value)) {
+					throw new \InvalidArgumentException('The value should be an array');
+		}
 
 		foreach($value as $position => $values) {
 			if (!$this->hasField($position)) {
@@ -134,8 +136,9 @@ 
 	public function registerScripts($view=null, $mount=true)
 	{
 		$view = ($view === null) ? $this->getView() : $view;
-		foreach ($this->getTemplate()->getFields() as $field)
-			$field->registerScripts($view, $mount);
+		foreach ($this->getTemplate()->getFields() as $field) {
+					$field->registerScripts($view, $mount);
+		}
 	}
 
 	/**
@@ -240,16 +243,18 @@ 
 	{
 		// the root path bit can be the the name or the id.
 		$bits = explode('.', $path);
-		if (!($bits[0] === $this->name))
-			throw new \Exception("The first part of the path should be the current repeater name. Repeater form name=$this->name, id=$this->id, path=$path.");
+		if (!($bits[0] === $this->name)) {
+					throw new \Exception("The first part of the path should be the current repeater name. Repeater form name=$this->name, id=$this->id, path=$path.");
+		}
 
 		// if the repeater has an existing instance - then lets return that
 		if ($this->hasField($bits[1])) {
 			$field = $this->getField($bits[1]);
 			// fields within repeaters should always be forms - so this should never happen
 			// but just to ensure we trap the error - lets throw an exception
-			if (!$field->isForm())
-				throw new \InvalidArgumentException('FormRepeaters should only contain form instances.');
+			if (!$field->isForm()) {
+							throw new \InvalidArgumentException('FormRepeaters should only contain form instances.');
+			}
 			array_splice($bits, 0, 1);
 			return $field->getFieldByPath(implode('.', $bits));
 		}

neon/core/view/MarkdownNeon.php

Spacing +5 added lines, -5 removed lines

@@ -72,7 +72,7 @@ 
 		$block = ['neonPlugin'];
 
 		$block['plugin']  = $this->_getPluginNameFromLines($lines, $current);
-		$block['attributes']  = $this->_getPluginAttributesFromLines($lines, $current);
+		$block['attributes'] = $this->_getPluginAttributesFromLines($lines, $current);
 		$block['content'] = $this->_getPluginContentFromLines($lines, $current, $endLine);
 
 		return [$block, $endLine];
@@ -81,11 +81,11 @@ 
 	protected function renderNeonPlugin($block)
 	{
 		$plugin = $block['plugin'];
-		$content = implode("\n",$block['content']);
+		$content = implode("\n", $block['content']);
 		$attributes = $block['attributes'];
 		if (class_exists($plugin))
 			return $plugin::widget(array_merge($attributes, ['content' => $content]));
-		return '<div class="alert alert-danger">unknown plugin: &quot;' .  $plugin . '&quot;</div>';
+		return '<div class="alert alert-danger">unknown plugin: &quot;'.$plugin.'&quot;</div>';
 	}
 
 	protected function _getPluginAttributesFromLines($lines, $current)
@@ -105,7 +105,7 @@ 
 		}
 		// remove tag and open delimiter from first and last lines
 		$plugin = $this->_getPluginNameFromLines($lines, $current);
-		$lastLine = \count($content)-1;
+		$lastLine = \count($content) - 1;
 		$content[0] = Str::replaceFirst($openDelimiter, '', $content[0]);
 		$content[0] = Str::replaceFirst($plugin, '', $content[0]);
 		$content[$lastLine] = Str::replaceLast($closeDelimiter, '', $content[$lastLine]);
@@ -114,7 +114,7 @@ 
 		// xml part
 		$xml = simplexml_load_string("<attr $attrs></attr>");
 		$attributes = [];
-		foreach($xml[0]->attributes() as $a => $b) {
+		foreach ($xml[0]->attributes() as $a => $b) {
 			$attributes[$a] = (string) $b;
 		}
 		return $attributes;

Braces +6 added lines, -4 removed lines

@@ -83,8 +83,9 @@ 
 		$plugin = $block['plugin'];
 		$content = implode("\n",$block['content']);
 		$attributes = $block['attributes'];
-		if (class_exists($plugin))
-			return $plugin::widget(array_merge($attributes, ['content' => $content]));
+		if (class_exists($plugin)) {
+					return $plugin::widget(array_merge($attributes, ['content' => $content]));
+		}
 		return '<div class="alert alert-danger">unknown plugin: &quot;' .  $plugin . '&quot;</div>';
 	}
 
@@ -151,8 +152,9 @@ 
 		for ($i = $current + 1, $count = count($lines); $i < $count; $i++) {
 			$line = $lines[$i];
 			// stop consuming lines if we find the closing tag
-			if (strpos($line, $closingTag) !== false)
-				break;
+			if (strpos($line, $closingTag) !== false) {
+							break;
+			}
 			$content[] = $line;
 		}
 		return $content;

neon/core/view/SmartyExtension.php

Spacing +4 added lines, -4 removed lines

@@ -72,7 +72,7 @@ 
 			trigger_error("path: missing 'route' parameter");
 		}
 
-		array_unshift($params, $params['route']) ;
+		array_unshift($params, $params['route']);
 		unset($params['route']);
 
 		return Url::to($params);
@@ -119,7 +119,7 @@ 
 		if ($type === 'block') {
 			// Register widget tag during compile time
 			$this->viewRenderer->widgets['blocks'][$alias] = $class;
-			$this->smarty->registerPlugin('block', $alias, [$this->viewRenderer, '_widget_block__' . $alias]);
+			$this->smarty->registerPlugin('block', $alias, [$this->viewRenderer, '_widget_block__'.$alias]);
 
 			// Inject code to re-register widget tag during run-time
 			return <<<PHP
@@ -137,7 +137,7 @@ 
 		} elseif ($type === 'function') {
 			// Register widget tag during compile time
 			$this->viewRenderer->widgets['functions'][$alias] = $class;
-			$this->smarty->registerPlugin('function', $alias, [$this->viewRenderer, '_widget_function__' . $alias]);
+			$this->smarty->registerPlugin('function', $alias, [$this->viewRenderer, '_widget_function__'.$alias]);
 
 			// Inject code to re-register widget tag during run-time
 			return <<<PHP
@@ -283,7 +283,7 @@ 
 	 */
 	protected function getViewConstVal($string, $default)
 	{
-		$val = @constant('yii\web\View::' . $string);
+		$val = @constant('yii\web\View::'.$string);
 		return isset($val) ? $val : $default;
 	}
 }

neon/core/assets/JQueryUiAsset.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
  */
 class JQueryUiAsset extends AssetBundle
 {
-	public $sourcePath = __DIR__ . '/publish/vendor/jqueryui';
+	public $sourcePath = __DIR__.'/publish/vendor/jqueryui';
 
 	public $js = [
 		'jquery-ui.min.js'

neon/core/assets/CoreAsset.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
  */
 class CoreAsset extends AssetBundle
 {
-	public $sourcePath = __DIR__ . '/publish';
+	public $sourcePath = __DIR__.'/publish';
 
 	public $js = [
 		(YII_DEBUG ? 'vendor/vue.js' : 'vendor/vue.min.js'),