newicon / neon

neon/core/test/WebBrowserDriver.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
 		parent::__construct($config, $options);
 		// Set default path for ChromeDriver
 		if (!isset($this->config['path'])) {
-			$this->config['path'] = __DIR__ . '/bin/'.$this->chromeVersion.'/chromedriver-' . $this->getDriverSuffix();
+			$this->config['path'] = __DIR__.'/bin/'.$this->chromeVersion.'/chromedriver-'.$this->getDriverSuffix();
 		}
 		// Set default WebDriver port
 		if (!isset($this->config['port'])) {
@@ -81,8 +81,8 @@ 
 		}
 		$descriptorSpec = array(
 			array('pipe', 'r'),
-			array('file', $this->getLogDir() . 'ChromeDriver.output.txt', 'w'),
-			array('file', $this->getLogDir() . 'ChromeDriver.errors.txt', 'a')
+			array('file', $this->getLogDir().'ChromeDriver.output.txt', 'w'),
+			array('file', $this->getLogDir().'ChromeDriver.errors.txt', 'a')
 		);
 		$this->resource = proc_open($command, $descriptorSpec, $this->pipes, null, null, array('bypass_shell' => true));
 		if (!is_resource($this->resource) || !proc_get_status($this->resource)['running']) {
@@ -170,7 +170,7 @@ 
 					// Make sure the value is true/false and not 1/0
 					$configValue = ($configValue) ? 'true' : 'false';
 				}
-				$params[] = $mapping[$configKey] . '=' . $configValue;
+				$params[] = $mapping[$configKey].'='.$configValue;
 			}
 		}
 		return implode(' ', $params);
@@ -183,7 +183,7 @@ 
 		// Prefix command with exec on non Windows systems to ensure that we receive the correct pid.
 		// See http://php.net/manual/en/function.proc-get-status.php#93382
 		$commandPrefix = $this->isWindows() ? '' : 'exec ';
-		return $commandPrefix . escapeshellarg(realpath($this->config['path'])) . ' ' . $this->getCommandParameters() . ' --url-base=/wd/hub';
+		return $commandPrefix.escapeshellarg(realpath($this->config['path'])).' '.$this->getCommandParameters().' --url-base=/wd/hub';
 	}
 	/**
 	 * Checks if the current machine is Windows.

neon/core/db/Connection.php

Spacing +3 added lines, -3 removed lines

@@ -86,7 +86,7 @@ 
 	{
 		$currentVersion = neon()->getVersion();
 		$dbName = env('DB_NAME');
-		$filename = gmdate('ymd_His') . '_' . $dbName . '_' . $currentVersion . '.sql';
+		$filename = gmdate('ymd_His').'_'.$dbName.'_'.$currentVersion.'.sql';
 		return $this->getBackupDirectory()."/$filename";
 	}
 
@@ -109,7 +109,7 @@ 
 			'--dump-date',
 			'--no-autocommit',
 			'--routines',
-			'--default-character-set=' . $this->charset,
+			'--default-character-set='.$this->charset,
 			'--set-charset',
 			'--triggers',
 			'--host='.env('DB_HOST'),
@@ -135,7 +135,7 @@ 
 	 */
 	public function restore($path)
 	{
-		$process = new Process('mysql ' . env('DB_NAME') . ' < "' . $path . '"');
+		$process = new Process('mysql '.env('DB_NAME').' < "'.$path.'"');
 		$result = $process->run();
 		if (!$process->isSuccessful()) {
 			throw new ProcessFailedException($process);

neon/core/db/TreeRecord.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 	 * @param int $depth
 	 * @return int
 	 */
-	public function rebuildTree($parent, $left, $depth=0)
+	public function rebuildTree($parent, $left, $depth = 0)
 	{
 		// the right value of this node is the left value + 1
 		$right = $left + 1;
@@ -117,7 +117,7 @@ 
 	 * @throws \Exception|\Throwable
 	 * @return TreeRecord the child added
 	 */
-	public function addChild(TreeRecord $node, callable $callable=null) : TreeRecord
+	public function addChild(TreeRecord $node, callable $callable = null) : TreeRecord
 	{
 		$result = static::addAsLastChildOf($this, $node);
 		if ($callable) $callable($node);

Braces +18 added lines, -11 removed lines

@@ -60,8 +60,9 @@ 
 	 */
 	public function makeRoot()
 	{
-		if (static::find()->root()->exists())
-			throw new \Exception('You can only create one root node');
+		if (static::find()->root()->exists()) {
+					throw new \Exception('You can only create one root node');
+		}
 		$this->tree_left = 1;
 		$this->tree_right = 2;
 		$this->tree_depth = 0;
@@ -120,7 +121,9 @@ 
 	public function addChild(TreeRecord $node, callable $callable=null) : TreeRecord
 	{
 		$result = static::addAsLastChildOf($this, $node);
-		if ($callable) $callable($node);
+		if ($callable) {
+			$callable($node);
+		}
 		return $node;
 	}
 
@@ -155,8 +158,9 @@ 
 	 */
 	public function moveAsFirstChildOf(TreeRecord $parent)
 	{
-		if ($parent === $this || $this->isAncestorOf($parent))
-			throw new \Excepton("Cannot move node as first child of itself or into a descendant");
+		if ($parent === $this || $this->isAncestorOf($parent)) {
+					throw new \Excepton("Cannot move node as first child of itself or into a descendant");
+		}
 		$oldDepth = $this->tree_depth;
 		$this->tree_left = $parent->tree_depth + 1;
 		$this->tree_parent = $parent->getPrimaryKey();
@@ -230,12 +234,15 @@ 
 	public static function validateAdd(TreeRecord $parent, TreeRecord $node)
 	{
 		// cannot insert as child of itself
-		if ($parent === $node)
-			throw new \Exception("Cannot insert node as last child of itself");
-		if ($parent->getIsNewRecord())
-			throw new \Exception('The parent node must exist.');
-		if ($node->tree_depth === 0 && $parent->isRoot())
-			throw new \Exception('Can not put a root node inside another root node');
+		if ($parent === $node) {
+					throw new \Exception("Cannot insert node as last child of itself");
+		}
+		if ($parent->getIsNewRecord()) {
+					throw new \Exception('The parent node must exist.');
+		}
+		if ($node->tree_depth === 0 && $parent->isRoot()) {
+					throw new \Exception('Can not put a root node inside another root node');
+		}
 	}
 
 	/**

neon/core/db/TreeQuery.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
 			$columns = ['tree_root' => SORT_ASC] + $columns;
 		}
 
-		$this->andWhere(['tree_right' => new Expression($db->quoteColumnName('tree_left') . ' + 1')])
+		$this->andWhere(['tree_right' => new Expression($db->quoteColumnName('tree_left').' + 1')])
 			->addOrderBy($columns);
 
 		return $this;
@@ -69,7 +69,7 @@ 
 	 * @param null $parent
 	 * @param null $root
 	 */
-	public function getProperties(&$left=null, &$right=null, &$parent=null, &$root=null)
+	public function getProperties(&$left = null, &$right = null, &$parent = null, &$root = null)
 	{
 		$model = new $this->modelClass();
 		$model->getPropertes($left, $right, $parent, $root);

neon/core/console/CacheController.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 		$r = new Renderer();
 		$r->smarty->clearCompiledTemplate();
 		$r->smarty->clearAllCache();
-		foreach(neon()->view->renderers as $ext => $config) {
+		foreach (neon()->view->renderers as $ext => $config) {
 			$renderer = neon()->view->getRendererByExtension($ext);
 			if ($renderer instanceof ICanFlushTemplates) {
 				$renderer->flushTemplates();

neon/daedalus/migrations/m170121_171048_dds_dds_changes_for_choices.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 //		$connection->createCommand($sql)->execute();
 
 		// 1. add the choices datatype
-$sql =<<<EOQ
+$sql = <<<EOQ
 SET foreign_key_checks = 0;
 REPLACE INTO `dds_data_type` (`data_type_ref`, `label`, `description`, `definition`, `storage_ref`, `deleted`)
 VALUES
@@ -46,7 +46,7 @@ 
 	{
 		$connection = neon()->db;
 
-$sql =<<<EOQ
+$sql = <<<EOQ
 SET foreign_key_checks = 0;
 DELETE FROM {{%dds_data_type}} WHERE `data_type_ref`='choice' LIMIT 1;
 SET foreign_key_checks = 1;

neon/daedalus/migrations/m160913_121403__neon_dds_changes.php

Indentation +7 added lines, -7 removed lines

@@ -4,9 +4,9 @@ 
 
 class m160913_121403__neon_dds_changes extends Migration
 {
-    // Use safeUp/safeDown to run migration code within a transaction
-    public function safeUp()
-    {
+	// Use safeUp/safeDown to run migration code within a transaction
+	public function safeUp()
+	{
 		// add a series of default types to the datatypes
 		$sql =<<<EOQ
 INSERT INTO `dds_data_type` (`data_type_ref`, `label`, `description`, `definition`, `storage_ref`, `deleted`)
@@ -40,10 +40,10 @@ 
 		$command = $connection->createCommand($sql);
 		$command->execute();
 
-    }
+	}
 
-    public function safeDown()
-    {
+	public function safeDown()
+	{
 		// delete the additional rows in the datatypes
 		$sql =<<<EOQ
 DELETE FROM `dds_data_type` WHERE `data_type_ref` IN
@@ -55,5 +55,5 @@ 
 		$command = $connection->createCommand($sql);
 		$command->execute();
 
-    }
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
     public function safeUp()
     {
 		// add a series of default types to the datatypes
-		$sql =<<<EOQ
+		$sql = <<<EOQ
 INSERT INTO `dds_data_type` (`data_type_ref`, `label`, `description`, `definition`, `storage_ref`, `deleted`)
 VALUES
 ('binary','Normal Binary','For normal binaries such as images up to 16MB in size','','binary',0),
@@ -45,7 +45,7 @@ 
     public function safeDown()
     {
 		// delete the additional rows in the datatypes
-		$sql =<<<EOQ
+		$sql = <<<EOQ
 DELETE FROM `dds_data_type` WHERE `data_type_ref` IN
 ('binary', 'binarylong', 'binaryshort', 'date', 'datetime', 'float', 'floatdouble',
 'integer', 'integerlong', 'integershort', 'integertiny', 'text', 'textlong', 'textshort',

neon/daedalus/migrations/m170904_201142_dds_multichoice_creation.php

Indentation +10 added lines, -10 removed lines

@@ -4,26 +4,26 @@
 
 class m170904_201142_dds_multichoice_creation extends Migration
 {
-    public function safeUp()
-    {
-	    $sql =<<<EOQ
+	public function safeUp()
+	{
+		$sql =<<<EOQ
 SET foreign_key_checks = 0;
 REPLACE INTO `dds_data_type` (`data_type_ref`, `label`, `description`, `definition`, `storage_ref`, `deleted`)
 VALUES
 ('choice_multiple','Multiple Choice','For storing multiple choice keys','{"size":10000}','text',0);
 SET foreign_key_checks = 1;
 EOQ;
-	    $this->execute($sql);
+		$this->execute($sql);
 
-    }
+	}
 
-    public function safeDown()
-    {
-	    $sql =<<<EOQ
+	public function safeDown()
+	{
+		$sql =<<<EOQ
 SET foreign_key_checks=0;
 DELETE FROM `dds_data_type` WHERE `data_type_ref`='choice_multiple';
 SET foreign_key_checks=1;
 EOQ;
-	    $this->execute($sql);
-    }
+		$this->execute($sql);
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 {
     public function safeUp()
     {
-	    $sql =<<<EOQ
+	    $sql = <<<EOQ
 SET foreign_key_checks = 0;
 REPLACE INTO `dds_data_type` (`data_type_ref`, `label`, `description`, `definition`, `storage_ref`, `deleted`)
 VALUES
@@ -19,7 +19,7 @@ 
 
     public function safeDown()
     {
-	    $sql =<<<EOQ
+	    $sql = <<<EOQ
 SET foreign_key_checks=0;
 DELETE FROM `dds_data_type` WHERE `data_type_ref`='choice_multiple';
 SET foreign_key_checks=1;

neon/daedalus/migrations/m161111_161448__dds_removal_of_ids.php

Indentation +6 added lines, -6 removed lines

@@ -4,8 +4,8 @@ 
 
 class m161111_161448__dds_removal_of_ids extends Migration
 {
-    public function safeUp()
-    {
+	public function safeUp()
+	{
 		// delete ids
 		$this->dropColumn('dds_class', 'id');
 		$this->dropColumn('dds_data_type', 'id');
@@ -14,10 +14,10 @@ 
 
 		// add a uuid to object tables
 		$this->execute('ALTER TABLE `dds_object` ADD `_uuid` CHAR(36) NOT NULL FIRST, ADD PRIMARY KEY (`_uuid`)');
-    }
+	}
 
-    public function safeDown()
-    {
+	public function safeDown()
+	{
 		// remove the uuid
 		$this->dropColumn('dds_object', '_uuid');
 		// add ids back again
@@ -25,5 +25,5 @@ 
 		$this->execute('ALTER TABLE `dds_data_type` ADD `id` INT UNSIGNED NOT NULL AUTO_INCREMENT FIRST, ADD PRIMARY KEY (`id`)');
 		$this->execute('ALTER TABLE `dds_member` ADD `id` INT UNSIGNED NOT NULL AUTO_INCREMENT FIRST, ADD PRIMARY KEY (`id`)');
 		$this->execute('ALTER TABLE `dds_object` ADD `_id` INT UNSIGNED NOT NULL AUTO_INCREMENT FIRST, ADD PRIMARY KEY (`_id`)');
-    }
+	}
 }