newicon / neon

neon/core/helpers.php

Spacing +37 added lines, -37 removed lines

@@ -16,7 +16,7 @@ 
 use \neon\core\Env;
 use \Carbon\Carbon;
 
-if (! function_exists('dp')) {
+if (!function_exists('dp')) {
 	/**
 	 * Debug function. Prints all passed in arguments to the screen.
 	 * arguments containing objects or arrays are output using print_r
@@ -38,7 +38,7 @@ 
 	}
 }
 
-if (! function_exists('backtrace_line_by_function')) {
+if (!function_exists('backtrace_line_by_function')) {
 	/**
 	 * Tries to get the line and file that called the specified functions
 	 * - will return the earliest match in execution order that it found.
@@ -50,9 +50,9 @@ 
 	{
 		$bt = debug_backtrace();
 		$lineInfo = '';
-		foreach($bt as $trace) {
+		foreach ($bt as $trace) {
 			if (isset($trace['function']) && $trace['function'] === $functionToShow) {
-				$lineInfo = '==> [' . (isset($trace['file']) ? $trace['file'] : ' unknown file '). ':' . (isset($trace['line']) ? $trace['line'] : ' unknown line ' ) . ']';
+				$lineInfo = '==> ['.(isset($trace['file']) ? $trace['file'] : ' unknown file ').':'.(isset($trace['line']) ? $trace['line'] : ' unknown line ').']';
 				break;
 			}
 		}
@@ -60,7 +60,7 @@ 
 	}
 }
 
-if (! function_exists('dp_string')) {
+if (!function_exists('dp_string')) {
 	/**
 	 * This generates a nice debug string and returns it. It will only
 	 * print to the screen if you are in debug mode. Otherwise it prints
@@ -92,7 +92,7 @@ 
 	}
 }
 
-if (! function_exists('dd')) {
+if (!function_exists('dd')) {
 	/**
 	 * This is the same as the dp (debug print) function but exits after
 	 * **Note:** If not in debug will print to log error and not exit
@@ -112,7 +112,7 @@ 
 	}
 }
 
-if (! function_exists('debug_message')) {
+if (!function_exists('debug_message')) {
 	/**
 	 * This is the same as the dp_string (debug print) function however it returns the string
 	 * The main difference is that it will only output a message if neon is in debug mode
@@ -131,7 +131,7 @@ 
 }
 
 
-if (! function_exists('neon')) {
+if (!function_exists('neon')) {
 	/**
 	 * Shortcut method to ```\Neon::$app```
 	 * @param string $name
@@ -143,7 +143,7 @@ 
 	}
 }
 
-if (! function_exists('setting')) {
+if (!function_exists('setting')) {
 	/**
 	 * @param string $app the app responsible for this setting
 	 * @param string $name the setting name
@@ -152,13 +152,13 @@ 
 	 * the setting has been set in the current session
 	 * @return string
 	 */
-	function setting($app, $name, $default=null, $refresh=false)
+	function setting($app, $name, $default = null, $refresh = false)
 	{
 		return neon()->getSettingsManager()->get($app, $name, $default, $refresh);
 	}
 }
 
-if (! function_exists('set_setting')) {
+if (!function_exists('set_setting')) {
 	/**
 	 * @param string $app the app responsible for this setting
 	 * @param string $name the setting name
@@ -171,7 +171,7 @@ 
 	}
 }
 
-if (! function_exists('url')) {
+if (!function_exists('url')) {
 	/**
 	 * Alias of \neon\core\helpers\Url::to
 	 * @see \neon\core\helpers\Url::to()
@@ -191,7 +191,7 @@ 
 	}
 }
 
-if (! function_exists('url_base')) {
+if (!function_exists('url_base')) {
 	/**
 	 * Returns the base URL of the current request.
 	 * @param boolean|string $scheme the URI scheme to use in the returned base URL:
@@ -207,7 +207,7 @@ 
 	}
 }
 
-if (! function_exists('to_bytes')) {
+if (!function_exists('to_bytes')) {
 	/**
 	 * Returns the number of bytes as integer from a strings like `128K` | `128M` | `2G` | `1T` | `1P`
 	 *
@@ -219,7 +219,7 @@ 
 	}
 }
 
-if (! function_exists('format_bytes')) {
+if (!function_exists('format_bytes')) {
 	/**
 	 * Output a formatted size like '128M' from bytes.
 	 *
@@ -232,7 +232,7 @@ 
 	}
 }
 
-if (! function_exists('truthy')) {
+if (!function_exists('truthy')) {
 	/**
 	 * Php if statement handles truthy and falsey string conversions by default e.g. evaluating a '0' string to false
 	 * however a string of 'false' is evaluated as true. This function resolves that difference. This should be used
@@ -248,7 +248,7 @@ 
 	}
 }
 
-if (! function_exists('env')) {
+if (!function_exists('env')) {
 	/**
 	 * Gets the value of an environment variable.
 	 * Interprets 'true', 'false', 'empty' and 'null' strings appropriately
@@ -283,7 +283,7 @@ 
 	}
 }
 
-if (! function_exists('neon_secret')) {
+if (!function_exists('neon_secret')) {
 	/**
 	 * Gets the value of NEON_SECRET creating it if missing
 	 * @return string
@@ -293,7 +293,7 @@ 
 	}
 }
 
-if (! function_exists('neon_encryption_key')) {
+if (!function_exists('neon_encryption_key')) {
 	/**
 	 * Gets the value of NEON_SECRET creating it if missing
 	 * @return string
@@ -303,7 +303,7 @@ 
 	}
 }
 
-if (! function_exists('value')) {
+if (!function_exists('value')) {
 	/**
 	 * Return the value of the given value.
 	 * If the $value is a Closure, it will first be executed then its result returned
@@ -317,7 +317,7 @@ 
 	}
 }
 
-if (! function_exists('collect')) {
+if (!function_exists('collect')) {
 	/**
 	 * Create a collection from the given value.
 	 *
@@ -330,7 +330,7 @@ 
 	}
 }
 
-if (! function_exists('now')) {
+if (!function_exists('now')) {
 	/**
 	 * Create a new Carbon instance for the current time.
 	 *
@@ -343,27 +343,27 @@ 
 	}
 }
 
-if (! function_exists('profile_begin')) {
+if (!function_exists('profile_begin')) {
 	/**
 	 * @param string $name
 	 */
-	function profile_begin($name, $category='neon')
+	function profile_begin($name, $category = 'neon')
 	{
 		\Neon::beginProfile($name, $category);
 	}
 }
 
-if (! function_exists('profile_end')) {
+if (!function_exists('profile_end')) {
 	/**
 	 * @param string $name
 	 */
-	function profile_end($name=null, $category='neon')
+	function profile_end($name = null, $category = 'neon')
 	{
 		\Neon::endProfile($name, $category);
 	}
 }
 
-if (! function_exists('encrypt')) {
+if (!function_exists('encrypt')) {
 	/**
 	 * @param string $data
 	 * @return string - encrypted data
@@ -374,7 +374,7 @@ 
 	}
 }
 
-if (! function_exists('decrypt')) {
+if (!function_exists('decrypt')) {
 	/**
 	 * @param string $data
 	 * @return string - encrypted data
@@ -385,7 +385,7 @@ 
 	}
 }
 
-if (! function_exists('get_alias')) {
+if (!function_exists('get_alias')) {
 	/**
 	 * @param string $path
 	 * @return string - resolved path
@@ -396,7 +396,7 @@ 
 	}
 }
 
-if (! function_exists('on_url')) {
+if (!function_exists('on_url')) {
 	/**
 	 * This function is useful for checking if links should be active,
 	 *
@@ -417,7 +417,7 @@ 
 	}
 }
 
-if (! function_exists('_t')) {
+if (!function_exists('_t')) {
 	/**
 	 * Translates a message to the target language
 	 * This function is a wrapper to the \Neon::t method
@@ -437,12 +437,12 @@ 
 	 * @param string $category  if provided will save the text to that category. Defaults to 'neon'
 	 * @return string the translated message.
 	 */
-	function _t($text, $params=[], $category='neon')
+	function _t($text, $params = [], $category = 'neon')
 	{
 		return \Neon::t($category, $text, $params);
 	}
 }
-if (! function_exists('faker')) {
+if (!function_exists('faker')) {
 	/**
 	 * Generate a faker object
 	 * @return \Faker\Generator;
@@ -455,7 +455,7 @@ 
 		return $faker;
 	}
 }
-if (! function_exists('user_is')) {
+if (!function_exists('user_is')) {
 	/**
 	 * @return boolean whether the user is/has a particular role
 	 */
@@ -465,7 +465,7 @@ 
 	}
 }
 
-if (! function_exists('user_can')) {
+if (!function_exists('user_can')) {
 	/**
 	 * Whether the user can perform a permission
 	 * @param $permission
@@ -477,7 +477,7 @@ 
 	}
 }
 
-if (! function_exists('is_route')) {
+if (!function_exists('is_route')) {
 	/**
 	 * Test whether the current application is on a particular route
 	 * This is url rewrite safe! However you must reference the module/controller/action and not the url directly
@@ -486,7 +486,7 @@ 
 	 * @param string $pattern
 	 * @param array $params
 	 */
-	function is_route($pattern, $params=[])
+	function is_route($pattern, $params = [])
 	{
 		return neon()->request->isRoute($pattern, $params);
 	}

Braces +17 added lines, -12 removed lines

@@ -31,10 +31,11 @@ 
 	{
 		echo dp_string(...$dump);
 		$backtrace = backtrace_line_by_function('dp');
-		if (neon()->debug)
-			echo $backtrace;
-		else
-			\Neon::error($backtrace);
+		if (neon()->debug) {
+					echo $backtrace;
+		} else {
+					\Neon::error($backtrace);
+		}
 	}
 }
 
@@ -85,8 +86,9 @@ 
 			$str .= "<pre>$debug</pre>";
 		}
 
-		if (neon()->debug)
-			return $str;
+		if (neon()->debug) {
+					return $str;
+		}
 		\Neon::error($str);
 		return '';
 	}
@@ -124,8 +126,9 @@ 
 	function debug_message(...$dump)
 	{
 		// security feature to ensure debug is not displayed unless in debug mode
-		if (!neon()->debug)
-			return '';
+		if (!neon()->debug) {
+					return '';
+		}
 		return dp_string(...$dump);
 	}
 }
@@ -242,8 +245,9 @@ 
 	 */
 	function truthy($value)
 	{
-		if ($value && strtolower($value) !== "false")
-			return true;
+		if ($value && strtolower($value) !== "false") {
+					return true;
+		}
 		return false;
 	}
 }
@@ -450,8 +454,9 @@ 
 	function faker()
 	{
 		static $faker = null;
-		if ($faker === null)
-			$faker = \Faker\Factory::create();
+		if ($faker === null) {
+					$faker = \Faker\Factory::create();
+		}
 		return $faker;
 	}
 }

neon/core/test/WebBrowserDriver.php

Spacing +5 added lines, -5 removed lines

@@ -32,7 +32,7 @@ 
 		parent::__construct($config, $options);
 		// Set default path for ChromeDriver
 		if (!isset($this->config['path'])) {
-			$this->config['path'] = __DIR__ . '/bin/72/chromedriver-' . $this->getDriverSuffix();
+			$this->config['path'] = __DIR__.'/bin/72/chromedriver-'.$this->getDriverSuffix();
 		}
 		// Set default WebDriver port
 		if (!isset($this->config['port'])) {
@@ -78,8 +78,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']) {
@@ -167,7 +167,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);
@@ -180,7 +180,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/test/Unit.php

Spacing +5 added lines, -5 removed lines

@@ -33,13 +33,13 @@ 
 	 * @param  integer  $size  size of the string required
 	 * @return  string
 	 */
-	protected static function randomString($size=20)
+	protected static function randomString($size = 20)
 	{
 		$source = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
 		$sourceLen = strlen($source);
 		$string = '';
-		for ($i=0; $i<$size; $i++)
-			$string .= substr($source, rand(0, $sourceLen-1), 1);
+		for ($i = 0; $i < $size; $i++)
+			$string .= substr($source, rand(0, $sourceLen - 1), 1);
 		return $string;
 	}
 
@@ -84,9 +84,9 @@ 
 	 * @param int $time  the time if not now
 	 * @return datetime in form yyyy-mm-dd hh-mm-ss
 	 */
-	protected function datetime($time=null, $format='Y-m-d H:i:s')
+	protected function datetime($time = null, $format = 'Y-m-d H:i:s')
 	{
-		return date($format, ($time===null ? time() : $time));
+		return date($format, ($time === null ? time() : $time));
 	}
 
 	/**

Braces +3 added lines, -2 removed lines

@@ -38,8 +38,9 @@
 		$source = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
 		$sourceLen = strlen($source);
 		$string = '';
-		for ($i=0; $i<$size; $i++)
-			$string .= substr($source, rand(0, $sourceLen-1), 1);
+		for ($i=0; $i<$size; $i++) {
+					$string .= substr($source, rand(0, $sourceLen-1), 1);
+		}
 		return $string;
 	}
 

neon/core/test/helpers.php

Spacing +15 added lines, -15 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('neon_test_database_create')) {
+if (!function_exists('neon_test_database_create')) {
 	/**
 	 * Creates the test database and user
 	 * Note this does not run the neon installation and create the
@@ -9,7 +9,7 @@ 
 	 */
 	function neon_test_database_create()
 	{
-		$installDb  = env('INSTALL_TEST_DB', true);
+		$installDb = env('INSTALL_TEST_DB', true);
 		if ($installDb) {
 			$db = env('DB_NAME');
 			$host = env('DB_HOST', 'localhost');
@@ -50,7 +50,7 @@ 
 	}
 }
 
-if (! function_exists('neon_test_db_cmd')) {
+if (!function_exists('neon_test_db_cmd')) {
 	function neon_test_db_cmd($cmd)
 	{
 		// A security risk here?  Mind you if you can run PHP your system is compromised anyway.
@@ -62,11 +62,11 @@ 
 		# use the following line to see any warnings and errors
 		#exec($c);
 		# or use this one to suppress them (inc the one about the password)
-		exec($c . " 2>/dev/null");
+		exec($c." 2>/dev/null");
 	}
 }
 
-if (! function_exists('neon_test_server_restart')) {
+if (!function_exists('neon_test_server_restart')) {
 	/**
 	 * Restart a test server
 	 * @param $name
@@ -74,7 +74,7 @@ 
 	 * @param int $port
 	 * @throws Exception
 	 */
-	function neon_test_server_restart($name, $docRoot=DIR_TEST.'/_root/public', $port=6969)
+	function neon_test_server_restart($name, $docRoot = DIR_TEST.'/_root/public', $port = 6969)
 	{
 		echo "\nRestarting test server...\n";
 		neon_test_server_stop($name);
@@ -82,7 +82,7 @@ 
 	}
 }
 
-if (! function_exists('neon_test_server_start')) {
+if (!function_exists('neon_test_server_start')) {
 	/**
 	 * Spin up the test server to run the acceptance tests against
 	 * @param string $name
@@ -91,7 +91,7 @@ 
 	 * @return \Symfony\Component\Process\Process
 	 * @throws Exception
 	 */
-	function neon_test_server_start($name, $docRoot=DIR_TEST.'/_root/public', $port=6969, $silent=false)
+	function neon_test_server_start($name, $docRoot = DIR_TEST.'/_root/public', $port = 6969, $silent = false)
 	{
 		if (preg_match("/[^a-z0-9_]/", $name) !== 0) {
 			throw new \Exception('The name must contain only [a-z0-9_] characters');
@@ -101,7 +101,7 @@ 
 		 * Tests are run against this server
 		 */
 		$address = "localhost:$port";
-		$command = PHP_BINARY . " -S {$address} -t {$docRoot}";
+		$command = PHP_BINARY." -S {$address} -t {$docRoot}";
 		if (!$silent) {
 			echo "\n|======================================================";
 			echo "\n| Creating Test Server";
@@ -118,19 +118,19 @@ 
 		}
 		$process = \neon\core\test\ProcessForker::run($name, $command);
 		try {
-			retry(10, function () use ($address) {
+			retry(10, function() use ($address) {
 				$client = new GuzzleHttp\Client();
 				$res = $client->request('GET', $address);
-				echo ' - Server running with code: ' . $res->getStatusCode() . "\n";
+				echo ' - Server running with code: '.$res->getStatusCode()."\n";
 			}, 1000);
-		} catch(GuzzleHttp\Exception\ConnectException $e) {
+		} catch (GuzzleHttp\Exception\ConnectException $e) {
 			throw new \Exception('Unable to connect to the local test server');
 		}
 		return $process;
 	}
 }
 
-if (! function_exists('retry')) {
+if (!function_exists('retry')) {
 	/**
 	 * Retry an operation a given number of times.
 	 *
@@ -148,7 +148,7 @@ 
 		try {
 			return $callback();
 		} catch (\Exception $e) {
-			if (! $times) {
+			if (!$times) {
 				throw $e;
 			}
 			$times--;
@@ -160,7 +160,7 @@ 
 	}
 }
 
-if (! function_exists('neon_test_server_stop')) {
+if (!function_exists('neon_test_server_stop')) {
 	/**
 	 * Spin up the test server to run the acceptance tests against
 	 * @param string $name

neon/core/Yii.php

Spacing +1 added lines, -1 removed lines

@@ -24,5 +24,5 @@
 
 spl_autoload_register(['Yii', 'autoload'], true, true);
 // Ability to replace any yii core class with an alternative
-Yii::$classMap = require YII2_PATH . '/classes.php';
+Yii::$classMap = require YII2_PATH.'/classes.php';
 Yii::$container = new yii\di\Container();
\ No newline at end of file

neon/core/BaseApp.php

Spacing +9 added lines, -9 removed lines

@@ -65,7 +65,7 @@ 
 			$class = \get_class($this);
 			if (($pos = strrpos($class, '\\')) !== false) {
 				$controllers = neon()->isConsoleApp() ? 'console' : 'controllers';
-				$this->controllerNamespace = substr($class, 0, $pos) . '\\' . $controllers;
+				$this->controllerNamespace = substr($class, 0, $pos).'\\'.$controllers;
 			}
 		}
 		parent::init();
@@ -145,7 +145,7 @@ 
 	 * @param boolean $refresh - default false
 	 * @return type
 	 */
-	public function getSetting($name, $default=null, $refresh=false)
+	public function getSetting($name, $default = null, $refresh = false)
 	{
 		$settingManager = neon()->settingsManager;
 		return $settingManager->get($this->id, $name, $default, $refresh);
@@ -185,7 +185,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function uninstall(){}
+	public function uninstall() {}
 
 	/**
 	 * Get the alias key to this module (App)
@@ -195,7 +195,7 @@ 
 	 */
 	public function getClassAlias()
 	{
-		return '@' . rtrim(str_replace(['\\', 'App'], ['/', ''], get_called_class()), "/");
+		return '@'.rtrim(str_replace(['\\', 'App'], ['/', ''], get_called_class()), "/");
 	}
 
 	/**
@@ -232,7 +232,7 @@ 
 	 */
 	public function getAssetsPath()
 	{
-		return $this->getPath() . '/assets';
+		return $this->getPath().'/assets';
 	}
 
 	/**
@@ -240,7 +240,7 @@ 
 	 */
 	public function getDocsPath()
 	{
-		return $this->getPath() . '/docs';
+		return $this->getPath().'/docs';
 	}
 
 	/**
@@ -250,7 +250,7 @@ 
 	 * @see $this->getAssetsPath()
 	 * @return string
 	 */
-	public function publishAssets(&$url='', &$path='')
+	public function publishAssets(&$url = '', &$path = '')
 	{
 		$assets = neon()->assetManager->publish($this->getAssetsPath());
 		list($path, $url) = $assets;
@@ -266,7 +266,7 @@ 
 	public function getDataMapProviders()
 	{
 		$availableMaps = [];
-		foreach(neon()->getAllApps() as $name => $config) {
+		foreach (neon()->getAllApps() as $name => $config) {
 			$app = neon($name);
 			if ($app instanceof IDataMapProvider) {
 				$maps = $app->getDataMapTypes();
@@ -292,7 +292,7 @@ 
 		if ($provider === null) {
 			throw new \Exception("The map provider '$name' specified can not be found.");
 		}
-		if (! $provider instanceof IDataMapProvider) {
+		if (!$provider instanceof IDataMapProvider) {
 			throw new \Exception("The Provider '$name' must implement the 'neon\\core\\interfaces\\IDataMapProvider' interface.");
 		}
 		return $provider;

Braces +3 added lines, -2 removed lines

@@ -106,8 +106,9 @@
 	public function getMenu()
 	{
 		// if the menu has been overridden
-		if ($this->_menu !== null)
-			return $this->_menu;
+		if ($this->_menu !== null) {
+					return $this->_menu;
+		}
 		return $this->getDefaultMenu();
 	}
 

neon/firefly/services/mediaManager/interfaces/IMediaManager.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 	 *        ],
 	 *     ]
 	 */
-	public function addFile($id, $path='/');
+	public function addFile($id, $path = '/');
 
 	/**
 	 * Get the information about the root node.

neon/firefly/services/mediaManager/models/Media.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
 	 * @param boolean $withDeleted
 	 * @return Media|null|\yii\db\ActiveRecord
 	 */
-	public static function findDirectoryByPath($path, $withDeleted=false)
+	public static function findDirectoryByPath($path, $withDeleted = false)
 	{
 		$path = static::getPath($path);
 		return static::query($withDeleted)
@@ -84,7 +84,7 @@ 
 	 * @param string $parentPath - the parent path used to find the parent node
 	 * @return Media|null|\yii\db\ActiveRecord
 	 */
-	public static function findParentByPath($path, &$parentPath='')
+	public static function findParentByPath($path, &$parentPath = '')
 	{
 		$info = pathinfo($path);
 		$parentPath = $info['dirname'];
@@ -109,7 +109,7 @@ 
 		$node = new static(['path' => $path, 'type' => 'dir', 'parent_id' => $parent->id]);
 		$node->save();
 		if ($node->hasErrors()) {
-			throw new \Exception('Model validation errors: ' . print_r($node->getErrors(), true));
+			throw new \Exception('Model validation errors: '.print_r($node->getErrors(), true));
 		}
 		return $node->toArray();
 	}
@@ -226,7 +226,7 @@ 
 		// update directories - and their child paths
 		if ($this->isDir()) {
 			$originalPath = $this->path;
-			$newPath = $intoNode->path . $this->path;
+			$newPath = $intoNode->path.$this->path;
 			$this->path = $newPath;
 			$success = $this->save();
 			if ($success) {
@@ -250,7 +250,7 @@ 
 	public static function replacePaths($findPath, $replacePath, $wherePath)
 	{
 		$tableName = static::tableName();
-		$sql = 'UPDATE ' . $tableName . ' SET path = REPLACE(path, :findPath, :replacePath) WHERE path LIKE :wherePath';
+		$sql = 'UPDATE '.$tableName.' SET path = REPLACE(path, :findPath, :replacePath) WHERE path LIKE :wherePath';
 		return Media::getDb()->createCommand($sql)
 			->bindValues([
 				':findPath' => $findPath,
@@ -265,7 +265,7 @@ 
 	 *
 	 * @return \neon\core\db\ActiveQuery
 	 */
-	public function getChildrenQuery($withDeleted=false)
+	public function getChildrenQuery($withDeleted = false)
 	{
 		return static::query($withDeleted)->alias('media')
 			->where(['parent_id' => $this->id]);

Braces +12 added lines, -8 removed lines

@@ -104,8 +104,9 @@ 
 	public static function createDirectory($path)
 	{
 		$parent = static::findParentByPath($path, $parentPath);
-		if ($parent === null)
-			throw new \Exception("Directory does not exist with path '$parentPath'");
+		if ($parent === null) {
+					throw new \Exception("Directory does not exist with path '$parentPath'");
+		}
 		$node = new static(['path' => $path, 'type' => 'dir', 'parent_id' => $parent->id]);
 		$node->save();
 		if ($node->hasErrors()) {
@@ -155,11 +156,13 @@ 
 	{
 		$path = $this->$attribute;
 		// must be absolute and start with /
-		if ($path[0] !== '/')
-			$this->addError('path', "Must be absolute path and start with a '/' provided: '$path'");
+		if ($path[0] !== '/') {
+					$this->addError('path', "Must be absolute path and start with a '/' provided: '$path'");
+		}
 		// prevent folders with no names being created
-		if (strpos($path, '//') !== false)
-			$this->addError('path', "Path can not contain a double forward slash");
+		if (strpos($path, '//') !== false) {
+					$this->addError('path', "Path can not contain a double forward slash");
+		}
 	}
 
 	/**
@@ -303,8 +306,9 @@ 
 	 */
 	public static function getPath($path)
 	{
-		if ($path === '/' && neon()->firefly->mediaManager->getRootPath() !== '/')
-			$path = neon()->firefly->mediaManager->getRootPath();
+		if ($path === '/' && neon()->firefly->mediaManager->getRootPath() !== '/') {
+					$path = neon()->firefly->mediaManager->getRootPath();
+		}
 		return $path;
 	}
 }
\ No newline at end of file

neon/firefly/services/driveManager/interfaces/IFileSystem.php

Spacing +2 added lines, -2 removed lines

@@ -165,7 +165,7 @@ 
 	 * @throws \InvalidArgumentException if the $contents is an object and it does not have a getRealPath function
 	 * @return bool  True on success, false on failure.
 	 */
-	public function put($path, $contents, $visibility=null);
+	public function put($path, $contents, $visibility = null);
 
 	/**
 	 * Get meta data for this file
@@ -235,7 +235,7 @@ 
 	 * @param  string  $algorithm  - Any algorithm supported by hash()
 	 * @return bool|string  return false on failure the hash output string on success
 	 */
-	public function hash($path, $algorithm='md5');
+	public function hash($path, $algorithm = 'md5');
 
 	/**
 	 * Get the underlying Flysystem driver