Studio-42 / elFinder

php/elFinderVolumeDropbox.class.php

Spacing +66 added lines, -67 removed lines

@@ -138,7 +138,7 @@ 
 		
 		if ($options['user'] === 'init') {
 
-			if (! $this->dropbox_phpFound || empty($options['consumerKey']) || empty($options['consumerSecret']) || !class_exists('PDO', false)) {
+			if (!$this->dropbox_phpFound || empty($options['consumerKey']) || empty($options['consumerSecret']) || !class_exists('PDO', false)) {
 				return array('exit' => true, 'body' => '{msg:errNetMountNoDriver}');
 			}
 			
@@ -148,7 +148,7 @@ 
 				if (class_exists('OAuth', false)) {
 					$this->oauth = new Dropbox_OAuth_PHP($options['consumerKey'], $options['consumerSecret']);
 				} else {
-					if (! class_exists('HTTP_OAuth_Consumer', false)) {
+					if (!class_exists('HTTP_OAuth_Consumer', false)) {
 						// We're going to try to load in manually
 						include 'HTTP/OAuth/Consumer.php';
 					}
@@ -158,7 +158,7 @@ 
 				}
 			}
 			
-			if (! $this->oauth) {
+			if (!$this->oauth) {
 				return array('exit' => true, 'body' => '{msg:errNetMountNoDriver}');
 			}
 
@@ -179,25 +179,25 @@ 
 						unset($_SESSION['elFinderDropboxTokens']);
 					}
 				}
-				if (! $html) {
+				if (!$html) {
 					// get customdata
 					$cdata = '';
 					$innerKeys = array('cmd', 'host', 'options', 'pass', 'protocol', 'user');
-					$this->ARGS = $_SERVER['REQUEST_METHOD'] === 'POST'? $_POST : $_GET;
-					foreach($this->ARGS as $k => $v) {
-						if (! in_array($k, $innerKeys)) {
-							$cdata .= '&' . $k . '=' . rawurlencode($v);
+					$this->ARGS = $_SERVER['REQUEST_METHOD'] === 'POST' ? $_POST : $_GET;
+					foreach ($this->ARGS as $k => $v) {
+						if (!in_array($k, $innerKeys)) {
+							$cdata .= '&'.$k.'='.rawurlencode($v);
 						}
 					}
-					if (strpos($options['url'], 'http') !== 0 ) {
+					if (strpos($options['url'], 'http') !== 0) {
 						$options['url'] = $this->getConnectorUrl();
 					}
-					$callback  = $options['url']
+					$callback = $options['url']
 					           . '?cmd=netmount&protocol=dropbox&host=dropbox.com&user=init&pass=return&node='.$options['id'].$cdata;
 					
 					try {
 						$tokens = $this->oauth->getRequestToken();
-						$url= $this->oauth->getAuthorizeUrl(rawurlencode($callback));
+						$url = $this->oauth->getAuthorizeUrl(rawurlencode($callback));
 					} catch (Dropbox_Exception $e) {
 						return array('exit' => true, 'body' => '{msg:errAccess}');
 					}
@@ -244,14 +244,14 @@ 
 		if (isset($netVolumes[$key])) {
 			$dropboxUid = $netVolumes[$key]['dropboxUid'];
 		}
-		foreach($netVolumes as $volume) {
+		foreach ($netVolumes as $volume) {
 			if (@$volume['host'] === 'dropbox' && @$volume['dropboxUid'] === $dropboxUid) {
 				$count++;
 			}
 		}
 		if ($count === 1) {
 			$this->DB->exec('drop table '.$this->DB_TableName);
-			foreach(glob(rtrim($this->options['tmbPath'], '\\/').DIRECTORY_SEPARATOR.$this->tmbPrefix.'*.png') as $tmb) {
+			foreach (glob(rtrim($this->options['tmbPath'], '\\/').DIRECTORY_SEPARATOR.$this->tmbPrefix.'*.png') as $tmb) {
 				unlink($tmb);
 			}
 		}
@@ -265,10 +265,10 @@ 
 	 * @author Naoki Sawada
 	 */
 	private function getConnectorUrl() {
-		$url  = ((isset($_SERVER['HTTPS']) && strtolower($_SERVER['HTTPS']) !== 'off')? 'https://' : 'http://')
+		$url = ((isset($_SERVER['HTTPS']) && strtolower($_SERVER['HTTPS']) !== 'off') ? 'https://' : 'http://')
 		       . $_SERVER['SERVER_NAME']                                              // host
-		      . ($_SERVER['SERVER_PORT'] == 80 ? '' : ':' . $_SERVER['SERVER_PORT'])  // port
-		       . $_SERVER['REQUEST_URI'];                                             // path & query
+		      . ($_SERVER['SERVER_PORT'] == 80 ? '' : ':'.$_SERVER['SERVER_PORT'])  // port
+		       . $_SERVER['REQUEST_URI']; // path & query
 		list($url) = explode('?', $url);
 		return $url;
 	}
@@ -304,14 +304,14 @@ 
 		// make net mount key
 		$this->netMountKey = md5(join('-', array('dropbox', $this->options['path'])));
 
-		if (! $this->oauth) {
+		if (!$this->oauth) {
 			if (defined('ELFINDER_DROPBOX_USE_CURL_PUT')) {
 				$this->oauth = new Dropbox_OAuth_Curl($this->options['consumerKey'], $this->options['consumerSecret']);
 			} else {
 				if (class_exists('OAuth', false)) {
 					$this->oauth = new Dropbox_OAuth_PHP($this->options['consumerKey'], $this->options['consumerSecret']);
 				} else {
-					if (! class_exists('HTTP_OAuth_Consumer', false)) {
+					if (!class_exists('HTTP_OAuth_Consumer', false)) {
 						// We're going to try to load in manually
 						include 'HTTP/OAuth/Consumer.php';
 					}
@@ -322,7 +322,7 @@ 
 			}
 		}
 		
-		if (! $this->oauth) {
+		if (!$this->oauth) {
 			return $this->setError('OAuth extension not loaded.');
 		}
 
@@ -330,7 +330,7 @@ 
 		$this->root = $this->options['path'] = $this->_normpath($this->options['path']);
 
 		if (empty($this->options['alias'])) {
-			$this->options['alias'] = ($this->options['path'] === '/')? 'Dropbox.com'  : 'Dropbox'.$this->options['path'];
+			$this->options['alias'] = ($this->options['path'] === '/') ? 'Dropbox.com' : 'Dropbox'.$this->options['path'];
 		}
 
 		$this->rootName = $this->options['alias'];
@@ -385,7 +385,7 @@ 
 		}
 		
 		// setup PDO
-		if (! $this->options['PDO_DSN']) {
+		if (!$this->options['PDO_DSN']) {
 			$this->options['PDO_DSN'] = 'sqlite:'.$this->metaCache.DIRECTORY_SEPARATOR.'.elFinder_dropbox_db_'.md5($this->dropboxUid.$this->options['consumerSecret']);
 		}
 		// DataBase table name
@@ -393,7 +393,7 @@ 
 		// DataBase check or make table
 		try {
 			$this->DB = new PDO($this->options['PDO_DSN'], $this->options['PDO_User'], $this->options['PDO_Pass'], $this->options['PDO_Options']);
-			if (! $this->checkDB()) {
+			if (!$this->checkDB()) {
 				return $this->setError('Can not make DB table');
 			}
 		} catch (PDOException $e) {
@@ -455,7 +455,7 @@ 
 				$res = false;
 			}
 		}
-		if (! $res) {
+		if (!$res) {
 			try {
 				$this->DB->exec('CREATE TABLE '.$this->DB_TableName.'(path text, fname text, dat blob, isdir integer);');
 				$this->DB->exec('CREATE UNIQUE INDEX nameidx ON '.$this->DB_TableName.'(path, fname)');
@@ -505,7 +505,7 @@ 
 	 */
 	private function updateDBdat($path, $dat) {
 		return $this->query('update '.$this->DB_TableName.' set dat='.$this->DB->quote(serialize($dat))
-				. ', isdir=' . ($dat['is_dir']? 1 : 0)
+				. ', isdir='.($dat['is_dir'] ? 1 : 0)
 				. ' where path='.$this->DB->quote(strtolower($this->_dirname($path))).' and fname='.$this->DB->quote(strtolower(basename($path))));
 	}
 	/*********************************************************************/
@@ -530,7 +530,7 @@ 
 	 */
 	protected function deltaCheck($refresh = true) {
 		$chk = false;
-		if (! $refresh && $chk = $this->query('select dat from '.$this->DB_TableName.' where path=\'\' and fname=\'\' limit 1')) {
+		if (!$refresh && $chk = $this->query('select dat from '.$this->DB_TableName.' where path=\'\' and fname=\'\' limit 1')) {
 			$chk = unserialize($chk[0]);
 		}
 		if ($chk && ($chk['mtime'] + $this->options['metaCacheTime']) > $_SERVER['REQUEST_TIME']) {
@@ -554,7 +554,7 @@ 
 			do {
 				@ ini_set('max_execution_time', 120);
 				$_info = $this->dropbox->delta($cursor);
-				if (! empty($_info['reset'])) {
+				if (!empty($_info['reset'])) {
 					$this->DB->exec('TRUNCATE table '.$this->DB_TableName);
 					$this->DB->exec('insert into '.$this->DB_TableName.' values(\'\', \'\', \''.serialize(array('cursor' => '', 'mtime' => 0)).'\', 0);');
 					$this->DB->exec('insert into '.$this->DB_TableName.' values(\'/\', \'\', \''.serialize(array(
@@ -567,7 +567,7 @@ 
 				}
 				$cursor = $_info['cursor'];
 				
-				foreach($_info['entries'] as $entry) {
+				foreach ($_info['entries'] as $entry) {
 					$key = strtolower($entry[0]);
 					$pkey = strtolower($this->_dirname($key));
 					
@@ -576,22 +576,22 @@ 
 					$where = 'where path='.$path.' and fname='.$fname;
 					
 					if (empty($entry[1])) {
-						$ptimes[$pkey] = isset($ptimes[$pkey])? max(array($now, $ptimes[$pkey])) : $now;
+						$ptimes[$pkey] = isset($ptimes[$pkey]) ? max(array($now, $ptimes[$pkey])) : $now;
 						$this->DB->exec('delete from '.$this->DB_TableName.' '.$where);
-						! $delete && $delete = true;
+						!$delete && $delete = true;
 						continue;
 					}
 
-					$_itemTime = strtotime(isset($entry[1]['client_mtime'])? $entry[1]['client_mtime'] : $entry[1]['modified']);
-					$ptimes[$pkey] = isset($ptimes[$pkey])? max(array($_itemTime, $ptimes[$pkey])) : $_itemTime;
+					$_itemTime = strtotime(isset($entry[1]['client_mtime']) ? $entry[1]['client_mtime'] : $entry[1]['modified']);
+					$ptimes[$pkey] = isset($ptimes[$pkey]) ? max(array($_itemTime, $ptimes[$pkey])) : $_itemTime;
 					$sql = 'select path from '.$this->DB_TableName.' '.$where.' limit 1';
-					if (! $reset && $this->query($sql)) {
-						$this->DB->exec('update '.$this->DB_TableName.' set dat='.$this->DB->quote(serialize($entry[1])).', isdir='.($entry[1]['is_dir']? 1 : 0).' ' .$where);
+					if (!$reset && $this->query($sql)) {
+						$this->DB->exec('update '.$this->DB_TableName.' set dat='.$this->DB->quote(serialize($entry[1])).', isdir='.($entry[1]['is_dir'] ? 1 : 0).' '.$where);
 					} else {
-						$this->DB->exec('insert into '.$this->DB_TableName.' values ('.$path.', '.$fname.', '.$this->DB->quote(serialize($entry[1])).', '.(int)$entry[1]['is_dir'].')');
+						$this->DB->exec('insert into '.$this->DB_TableName.' values ('.$path.', '.$fname.', '.$this->DB->quote(serialize($entry[1])).', '.(int) $entry[1]['is_dir'].')');
 					}
 				}
-			} while (! empty($_info['has_more']));
+			} while (!empty($_info['has_more']));
 			
 			// update time stamp of parent holder
 			foreach ($ptimes as $_p => $_t) {
@@ -613,14 +613,14 @@ 
 			}
 			
 			$this->DB->exec('update '.$this->DB_TableName.' set dat='.$this->DB->quote(serialize(array('cursor'=>$cursor, 'mtime'=>$_SERVER['REQUEST_TIME']))).' where path=\'\' and fname=\'\'');
-			if (! $this->DB->commit()) {
+			if (!$this->DB->commit()) {
 				$e = $this->DB->errorInfo();
 				return $e[2];
 			}
 			if ($delete) {
 				$this->DB->exec('vacuum');
 			}
-		} catch(Dropbox_Exception $e) {
+		} catch (Dropbox_Exception $e) {
 			return $e->getMessage();
 		}
 		return true;
@@ -636,15 +636,14 @@ 
 	protected function parseRaw($raw) {
 		$stat = array();
 
-		$stat['rev']   = isset($raw['rev'])? $raw['rev'] : 'root';
+		$stat['rev']   = isset($raw['rev']) ? $raw['rev'] : 'root';
 		$stat['name']  = basename($raw['path']);
-		$stat['mime']  = $raw['is_dir']? 'directory' : $raw['mime_type'];
+		$stat['mime']  = $raw['is_dir'] ? 'directory' : $raw['mime_type'];
 		$stat['size']  = $stat['mime'] == 'directory' ? 0 : $raw['bytes'];
-		$stat['ts']    = isset($raw['client_mtime'])? strtotime($raw['client_mtime']) :
-		                (isset($raw['modified'])? strtotime($raw['modified']) : $_SERVER['REQUEST_TIME']);
+		$stat['ts']    = isset($raw['client_mtime']) ? strtotime($raw['client_mtime']) : (isset($raw['modified']) ? strtotime($raw['modified']) : $_SERVER['REQUEST_TIME']);
 		$stat['dirs'] = 0;
 		if ($raw['is_dir']) {
-			$stat['dirs'] = (int)(bool)$this->query('select path from '.$this->DB_TableName.' where isdir=1 and path='.$this->DB->quote(strtolower($raw['path'])));
+			$stat['dirs'] = (int) (bool) $this->query('select path from '.$this->DB_TableName.' where isdir=1 and path='.$this->DB->quote(strtolower($raw['path'])));
 		}
 		
 		if (!empty($raw['url'])) {
@@ -670,7 +669,7 @@ 
 		$res = $this->query('select dat from '.$this->DB_TableName.' where path='.$this->DB->quote(strtolower($path)));
 		
 		if ($res) {
-			foreach($res as $raw) {
+			foreach ($res as $raw) {
 				$raw = unserialize($raw);
 				if ($stat = $this->parseRaw($raw)) {
 					$stat = $this->updateCache($raw['path'], $stat);
@@ -695,10 +694,10 @@ 
 	protected function doSearch($path, $q, $mimes) {
 		$result = array();
 		$sth = $this->DB->prepare('select dat from '.$this->DB_TableName.' WHERE path LIKE ? AND fname LIKE ?');
-		$sth->execute(array('%'.(($path === '/')? '' : strtolower($path)), '%'.strtolower($q).'%'));
+		$sth->execute(array('%'.(($path === '/') ? '' : strtolower($path)), '%'.strtolower($q).'%'));
 		$res = $sth->fetchAll(PDO::FETCH_COLUMN);
 		if ($res) {
-			foreach($res as $raw) {
+			foreach ($res as $raw) {
 				$raw = unserialize($raw);
 				if ($stat = $this->parseRaw($raw)) {
 					if (!isset($this->cache[$raw['path']])) {
@@ -789,10 +788,10 @@ 
 		$tmb  = $this->tmbPath.DIRECTORY_SEPARATOR.$name;
 	
 		// copy image into tmbPath so some drivers does not store files on local fs
-		if (! $data = $this->getThumbnail($path, $this->options['getTmbSize'])) {
+		if (!$data = $this->getThumbnail($path, $this->options['getTmbSize'])) {
 			return false;
 		}
-		if (! file_put_contents($tmb, $data)) {
+		if (!file_put_contents($tmb, $data)) {
 			return false;
 		}
 	
@@ -805,15 +804,15 @@ 
 		}
 	
 		/* If image smaller or equal thumbnail size - just fitting to thumbnail square */
-		if ($s[0] <= $tmbSize && $s[1]  <= $tmbSize) {
-			$result = $this->imgSquareFit($tmb, $tmbSize, $tmbSize, 'center', 'middle', $this->options['tmbBgColor'], 'png' );
+		if ($s[0] <= $tmbSize && $s[1] <= $tmbSize) {
+			$result = $this->imgSquareFit($tmb, $tmbSize, $tmbSize, 'center', 'middle', $this->options['tmbBgColor'], 'png');
 	
 		} else {
 	
 			if ($this->options['tmbCrop']) {
 	
 				/* Resize and crop if image bigger than thumbnail */
-				if (!(($s[0] > $tmbSize && $s[1] <= $tmbSize) || ($s[0] <= $tmbSize && $s[1] > $tmbSize) ) || ($s[0] > $tmbSize && $s[1] > $tmbSize)) {
+				if (!(($s[0] > $tmbSize && $s[1] <= $tmbSize) || ($s[0] <= $tmbSize && $s[1] > $tmbSize)) || ($s[0] > $tmbSize && $s[1] > $tmbSize)) {
 					$result = $this->imgResize($tmb, $tmbSize, $tmbSize, true, false, 'png');
 				}
 	
@@ -827,7 +826,7 @@ 
 				$result = $this->imgResize($tmb, $tmbSize, $tmbSize, true, true, 'png');
 			}
 		
-			$result = $this->imgSquareFit($tmb, $tmbSize, $tmbSize, 'center', 'middle', $this->options['tmbBgColor'], 'png' );
+			$result = $this->imgSquareFit($tmb, $tmbSize, $tmbSize, 'center', 'middle', $this->options['tmbBgColor'], 'png');
 		}
 		
 		if (!$result) {
@@ -884,7 +883,7 @@ 
 					}
 				}
 			}
-			if (! $url) {
+			if (!$url) {
 				try {
 					$res = $this->dropbox->share($path, null, false);
 					$url = $res['url'];
@@ -896,7 +895,7 @@ 
 					}
 					list($url) = explode('?', $url);
 					$url = str_replace('www.dropbox.com', $this->dropbox_dlhost, $url);
-					if (! isset($cache['share']) || $cache['share'] !== $url) {
+					if (!isset($cache['share']) || $cache['share'] !== $url) {
 						$cache['share'] = $url;
 						$this->updateDBdat($path, $cache);
 					}
@@ -920,12 +919,12 @@ 
 		if (function_exists('curl_exec')) {
 
 			$c = curl_init();
-			curl_setopt( $c, CURLOPT_RETURNTRANSFER, true );
-			curl_setopt( $c, CURLOPT_CUSTOMREQUEST, 'HEAD' );
-			curl_setopt( $c, CURLOPT_HEADER, 1 );
-			curl_setopt( $c, CURLOPT_NOBODY, true );
-			curl_setopt( $c, CURLOPT_URL, $url );
-			$res = curl_exec( $c );
+			curl_setopt($c, CURLOPT_RETURNTRANSFER, true);
+			curl_setopt($c, CURLOPT_CUSTOMREQUEST, 'HEAD');
+			curl_setopt($c, CURLOPT_HEADER, 1);
+			curl_setopt($c, CURLOPT_NOBODY, true);
+			curl_setopt($c, CURLOPT_URL, $url);
+			$res = curl_exec($c);
 			
 		} else {
 			
@@ -941,13 +940,13 @@ 
 				$result = $request2->send();
 				$res = array();
 				$res[] = 'HTTP/'.$result->getVersion().' '.$result->getStatus().' '.$result->getReasonPhrase();
-				foreach($result->getHeader() as $key => $val) {
-					$res[] = $key . ': ' . $val;
+				foreach ($result->getHeader() as $key => $val) {
+					$res[] = $key.': '.$val;
 				}
 				$res = join("\r\n", $res);
-			} catch( HTTP_Request2_Exception $e ){
+			} catch (HTTP_Request2_Exception $e) {
 				$res = '';
-			} catch (Exception $e){
+			} catch (Exception $e) {
 				$res = '';
 			}
 		
@@ -1002,7 +1001,7 @@ 
 		if (DIRECTORY_SEPARATOR !== '/') {
 			$path = str_replace(DIRECTORY_SEPARATOR, '/', $path);
 		}
-		$path = '/' . ltrim($path, '/');
+		$path = '/'.ltrim($path, '/');
 		return $path;
 	}
 
@@ -1145,7 +1144,7 @@ 
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fopen($path, $mode='rb') {
+	protected function _fopen($path, $mode = 'rb') {
 
 		if (($mode == 'rb' || $mode == 'r')) {
 			try {
@@ -1155,7 +1154,7 @@ 
  				fputs($fp, "GET {$url['path']} HTTP/1.0\r\n");
  				fputs($fp, "Host: {$url['host']}\r\n");
  				fputs($fp, "\r\n");
- 				while(trim(fgets($fp)) !== ''){};
+ 				while (trim(fgets($fp)) !== '') {};
  				return $fp;
 			} catch (Dropbox_Exception $e) {
 				return false;
@@ -1186,7 +1185,7 @@ 
 	 * @return bool
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fclose($fp, $path='') {
+	protected function _fclose($fp, $path = '') {
 		@fclose($fp);
 		if ($path) {
 			@unlink($this->getTempFile($path));

php/elFinderVolumeFTP.class.php

Spacing +53 added lines, -53 removed lines

@@ -103,7 +103,7 @@ 
 		);
 		$this->options = array_merge($this->options, $opts); 
 		$this->options['mimeDetect'] = 'internal';
-		$this->options['maxArcFilesSize'] = 0;     // max allowed archive files size (0 - no limit)
+		$this->options['maxArcFilesSize'] = 0; // max allowed archive files size (0 - no limit)
 	}
 	
 	/**
@@ -163,7 +163,7 @@ 
 		$scheme = parse_url($this->options['host'], PHP_URL_SCHEME);
 
 		if ($scheme) {
-			$this->options['host'] = substr($this->options['host'], strlen($scheme)+3);
+			$this->options['host'] = substr($this->options['host'], strlen($scheme) + 3);
 		}
 
 		// normalize root path
@@ -239,11 +239,11 @@ 
 		if ($this->encoding) {
 			@ftp_exec($this->connect, 'OPTS UTF8 OFF');
 		} else {
-			@ftp_exec($this->connect, 'OPTS UTF8 ON' );
+			@ftp_exec($this->connect, 'OPTS UTF8 ON');
 		}
 		
 		// switch off extended passive mode - may be usefull for some servers
-		@ftp_exec($this->connect, 'epsv4 off' );
+		@ftp_exec($this->connect, 'epsv4 off');
 		// enter passive mode if required
 		ftp_pasv($this->connect, $this->options['mode'] == 'passive');
 
@@ -311,7 +311,7 @@ 
 		$name = $info[8];
 		
 		if (preg_match('|(.+)\-\>(.+)|', $name, $m)) {
-			$name   = trim($m[1]);
+			$name = trim($m[1]);
 			// check recursive processing
 			if ($this->cacheDirTarget && $this->_joinPath($base, $name) !== $this->cacheDirTarget) {
 				return array();
@@ -347,7 +347,7 @@ 
 			$stat['owner'] = $info[2];
 			$stat['group'] = $info[3];
 			$stat['perm']  = substr($info[0], 1);
-			$stat['isowner'] = $stat['owner']? ($stat['owner'] == $this->options['user']) : $this->options['owner'];
+			$stat['isowner'] = $stat['owner'] ? ($stat['owner'] == $this->options['user']) : $this->options['owner'];
 		}
 		
 		$perm = $this->parsePermissions($info[0], $stat['owner']);
@@ -379,7 +379,7 @@ 
 			$info[4] = 0;
 			$info[0] = 'drwxr-xr-x';
 		} else {
-			$info[4] = (int)$size;
+			$info[4] = (int) $size;
 			$info[0] = '-rw-r--r--';
 		}
 		return $info;
@@ -395,7 +395,7 @@ 
 	protected function parsePermissions($perm, $user = '') {
 		$res   = array();
 		$parts = array();
-		$owner = $user? ($user == $this->options['user']) : $this->options['owner'];
+		$owner = $user ? ($user == $this->options['user']) : $this->options['owner'];
 		for ($i = 0, $l = strlen($perm); $i < $l; $i++) {
 			$parts[] = substr($perm, $i, 1);
 		}
@@ -426,10 +426,10 @@ 
 			}
 		}
 		$list = $this->convEncOut($list);
-		$prefix = ($path === $this->separator)? $this->separator : $path . $this->separator;
+		$prefix = ($path === $this->separator) ? $this->separator : $path.$this->separator;
 		$targets = array();
-		foreach($list as $stat) {
-			$p = $prefix . $stat['name'];
+		foreach ($list as $stat) {
+			$p = $prefix.$stat['name'];
 			if (isset($stat['target'])) {
 				// stat later
 				$targets[$stat['name']] = $stat['target'];
@@ -441,10 +441,10 @@ 
 			}
 		}
 		// stat link targets
-		foreach($targets as $name => $target) {
+		foreach ($targets as $name => $target) {
 			$stat = array();
 			$stat['name'] = $name;
-			$p = $prefix . $name;
+			$p = $prefix.$name;
 			$cacheDirTarget = $this->cacheDirTarget;
 			$this->cacheDirTarget = $this->convEncIn($target, true);
 			if ($tstat = $this->stat($target)) {
@@ -453,12 +453,12 @@ 
 				$stat['thash'] = $tstat['hash'];
 				$stat['mime']  = $tstat['mime'];
 				$stat['read']  = $tstat['read'];
-				$stat['write']  = $tstat['write'];
+				$stat['write'] = $tstat['write'];
 				
-				if (isset($tstat['ts']))      { $stat['ts']      = $tstat['ts']; }
-				if (isset($tstat['owner']))   { $stat['owner']   = $tstat['owner']; }
-				if (isset($tstat['group']))   { $stat['group']   = $tstat['group']; }
- 				if (isset($tstat['perm']))    { $stat['perm']    = $tstat['perm']; }
+				if (isset($tstat['ts'])) { $stat['ts']      = $tstat['ts']; }
+				if (isset($tstat['owner'])) { $stat['owner']   = $tstat['owner']; }
+				if (isset($tstat['group'])) { $stat['group']   = $tstat['group']; }
+ 				if (isset($tstat['perm'])) { $stat['perm']    = $tstat['perm']; }
  				if (isset($tstat['isowner'])) { $stat['isowner'] = $tstat['isowner']; }
 			} else {
 				
@@ -571,7 +571,7 @@ 
 		$comps = $new_comps;
 		$path = implode($this->separator, $comps);
 		if ($initial_slashes) {
-			$path = str_repeat($this->separator, $initial_slashes) . $path;
+			$path = str_repeat($this->separator, $initial_slashes).$path;
 		}
 		
 		return $path ? $path : '.';
@@ -637,7 +637,7 @@ 
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _inpath($path, $parent) {
-		return $path == $parent || strpos($path, $parent. $this->separator) === 0;
+		return $path == $parent || strpos($path, $parent.$this->separator) === 0;
 	}
 	
 	/***************** file stat ********************/
@@ -669,7 +669,7 @@ 
 		}
 		if (!$this->MLSTsupprt) {
 			//if ($path == $this->root && (empty($this->ARGS['reload']) || !isset($this->ARGS['target']) || strpos($this->ARGS['target'], $this->id) !== 0)) {
-			if ($path == $this->root && ! $this->isMyReload()) {
+			if ($path == $this->root && !$this->isMyReload()) {
 				return array(
 					'name' => $this->root,
 					'mime' => 'directory',
@@ -677,9 +677,9 @@ 
 				);
 			}
 			$this->cacheDir($this->convEncOut($this->_dirname($path)));
-			return $this->convEncIn(isset($this->cache[$outPath])? $this->cache[$outPath] : array());
+			return $this->convEncIn(isset($this->cache[$outPath]) ? $this->cache[$outPath] : array());
 		}
-		$raw = ftp_raw($this->connect, 'MLST ' . $path);
+		$raw = ftp_raw($this->connect, 'MLST '.$path);
 		if (is_array($raw) && count($raw) > 1 && substr(trim($raw[0]), 0, 1) == 2) {
 			$parts = explode(';', trim($raw[1]));
 			array_pop($parts);
@@ -709,8 +709,8 @@ 
 
 					case 'perm':
 						$val = strtolower($val);
-						$stat['read']  = (int)preg_match('/e|l|r/', $val);
-						$stat['write'] = (int)preg_match('/w|m|c/', $val);
+						$stat['read']  = (int) preg_match('/e|l|r/', $val);
+						$stat['write'] = (int) preg_match('/w|m|c/', $val);
 						if (!preg_match('/f|d/', $val)) {
 							$stat['locked'] = 1;
 						}
@@ -795,7 +795,7 @@ 
 			if (!$this->ftpOsUnix) {
 				$info = $this->normalizeRawWindows($str);
 			}
-			$name = isset($info[8])? trim($info[8]) : '';
+			$name = isset($info[8]) ? trim($info[8]) : '';
 			if ($name && $name !== '.' && $name !== '..' && substr(strtolower($info[0]), 0, 1) === 'd') {
 				return true;
 			}
@@ -850,7 +850,7 @@ 
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fopen($path, $mode='rb') {
+	protected function _fopen($path, $mode = 'rb') {
 		// try ftp stream wrapper
 		if (ini_get('allow_url_fopen')) {
 			$url = 'ftp://'.$this->options['user'].':'.$this->options['pass'].'@'.$this->options['host'].':'.$this->options['port'].$path;
@@ -887,7 +887,7 @@ 
 	 * @return bool
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fclose($fp, $path='') {
+	protected function _fclose($fp, $path = '') {
 		@fclose($fp);
 		if ($path) {
 			@unlink($this->getTempFile($path));
@@ -1062,7 +1062,7 @@ 
 			if (@file_put_contents($local, $content, LOCK_EX) !== false
 			&& ($fp = @fopen($local, 'rb'))) {
 				clearstatcache();
-				$res  = ftp_fput($this->connect, $path, $fp, $this->ftpMode($path));
+				$res = ftp_fput($this->connect, $path, $fp, $this->ftpMode($path));
 				@fclose($fp);
 			}
 			file_exists($local) && @unlink($local);
@@ -1087,7 +1087,7 @@ 
 	 * @return bool
 	 **/
 	protected function _chmod($path, $mode) {
-		$modeOct = is_string($mode) ? octdec($mode) : octdec(sprintf("%04o",$mode));
+		$modeOct = is_string($mode) ? octdec($mode) : octdec(sprintf("%04o", $mode));
 		return @ftp_chmod($this->connect, $modeOct, $path);
 	}
 
@@ -1140,7 +1140,7 @@ 
 		}
 
 		$basename = $this->_basename($path);
-		$localPath = $dir . DIRECTORY_SEPARATOR . $basename;
+		$localPath = $dir.DIRECTORY_SEPARATOR.$basename;
 
 		if (!ftp_get($this->connect, $localPath, $path, FTP_BINARY)) {
 			//cleanup
@@ -1177,7 +1177,7 @@ 
 		
 		// archive contains one item - extract in archive dir
 		$name = '';
-		$src = $dir . DIRECTORY_SEPARATOR . $filesToProcess[0];
+		$src = $dir.DIRECTORY_SEPARATOR.$filesToProcess[0];
 		if (($extractTo === 'auto' || !$extractTo) && count($filesToProcess) === 1 && is_file($src)) {
 			$name = $filesToProcess[0];
 		} else if ($extractTo === 'auto' || $extractTo) {
@@ -1186,7 +1186,7 @@ 
 			$src = $dir;
 			$name = basename($path);
 			if (preg_match('/\.((tar\.(gz|bz|bz2|z|lzo))|cpio\.gz|ps\.gz|xcf\.(gz|bz2)|[a-z0-9]{1,4})$/i', $name, $m)) {
-				$name = substr($name, 0,  strlen($name)-strlen($m[0]));
+				$name = substr($name, 0, strlen($name) - strlen($m[0]));
 			}
 			$test = $this->_joinPath(dirname($path), $name);
 			if ($this->stat($test)) {
@@ -1197,7 +1197,7 @@ 
 		if ($name !== '' && is_file($src)) {
 			$result = $this->_joinPath(dirname($path), $name);
 
-			if (! ftp_put($this->connect, $result, $src, FTP_BINARY)) {
+			if (!ftp_put($this->connect, $result, $src, FTP_BINARY)) {
 				$this->rmdirRecursive($dir);
 				return false;
 			}
@@ -1211,19 +1211,19 @@ 
 				}
 				$result[] = $dstDir;
 			}
-			foreach($filesToProcess as $name) {
+			foreach ($filesToProcess as $name) {
 				$name = rtrim($name, DIRECTORY_SEPARATOR);
-				$src = $dir . DIRECTORY_SEPARATOR . $name;
+				$src = $dir.DIRECTORY_SEPARATOR.$name;
 				if (is_dir($src)) {
 					$p = dirname($name);
 					$name = basename($name);
-					if (! $target = $this->_mkdir($this->_joinPath($dstDir, $p), $name)) {
+					if (!$target = $this->_mkdir($this->_joinPath($dstDir, $p), $name)) {
 						$this->rmdirRecursive($dir);
 						return false;
 					}
 				} else {
 					$target = $this->_joinPath($dstDir, $name);
-					if (! ftp_put($this->connect, $target, $src, FTP_BINARY)) {
+					if (!ftp_put($this->connect, $target, $src, FTP_BINARY)) {
 						$this->rmdirRecursive($dir);
 						return false;
 					}
@@ -1239,7 +1239,7 @@ 
 		is_dir($dir) && $this->rmdirRecursive($dir);
 		
 		$this->clearcache();
-		return $result? $result : false;
+		return $result ? $result : false;
 	}
 
 	/**
@@ -1279,7 +1279,7 @@ 
 		}
 
 		//cleanup
-		if(!$this->rmdirRecursive($tmpDir)) {
+		if (!$this->rmdirRecursive($tmpDir)) {
 			return false;
 		}
 
@@ -1383,7 +1383,7 @@ 
 		$remoteDirLen = strlen($remote_directory);
 		foreach ($contents as $item) {
 			$relative_path = substr($item['path'], $remoteDirLen);
-			$local_path = $dest_local_directory . DIRECTORY_SEPARATOR . $relative_path;
+			$local_path = $dest_local_directory.DIRECTORY_SEPARATOR.$relative_path;
 			switch ($item['type']) {
 				case 'd':
 					$success = mkdir($local_path);
@@ -1414,8 +1414,8 @@ 
 		} else {
 			$success = true;
 			foreach (array_reverse(elFinderVolumeFTP::listFilesInDirectory($dirPath, false)) as $path) {
-				$path = $dirPath . DIRECTORY_SEPARATOR . $path;
-				if(is_link($path)) {
+				$path = $dirPath.DIRECTORY_SEPARATOR.$path;
+				if (is_link($path)) {
 					unlink($path);
 				} else if (is_dir($path)) {
 					$success = rmdir($path);
@@ -1426,11 +1426,11 @@ 
 					break;
 				}
 			}
-			if($success) {
+			if ($success) {
 				$success = rmdir($dirPath);
 			}
 		}
-		if(!$success) {
+		if (!$success) {
 			$this->setError(elFinder::ERROR_RM, $dirPath);
 			return false;
 		}
@@ -1452,25 +1452,25 @@ 
 		if (!is_dir($dir)) {
 			return false;
 		}
-		$excludes = array(".","..");
+		$excludes = array(".", "..");
 		$result = array();
 		$files = scandir($dir);
-		if(!$files) {
+		if (!$files) {
 			return array();
 		}
-		foreach($files as $file) {
-			if(!in_array($file, $excludes)) {
+		foreach ($files as $file) {
+			if (!in_array($file, $excludes)) {
 				$path = $dir.DIRECTORY_SEPARATOR.$file;
-				if(is_link($path)) {
-					if($omitSymlinks) {
+				if (is_link($path)) {
+					if ($omitSymlinks) {
 						continue;
 					} else {
 						$result[] = $prefix.$file;
 					}
-				} else if(is_dir($path)) {
+				} else if (is_dir($path)) {
 					$result[] = $prefix.$file.DIRECTORY_SEPARATOR;
 					$subs = elFinderVolumeFTP::listFilesInDirectory($path, $omitSymlinks, $prefix.$file.DIRECTORY_SEPARATOR);
-					if($subs) {
+					if ($subs) {
 						$result = array_merge($result, $subs);
 					}
 					

php/elFinderVolumeLocalFileSystem.class.php

Spacing +36 added lines, -36 removed lines

@@ -32,11 +32,11 @@ 
 	 * @author Dmitry (dio) Levashov
 	 **/
 	public function __construct() {
-		$this->options['alias']    = '';              // alias to replace root dir name
-		$this->options['dirMode']  = 0755;            // new dirs mode
-		$this->options['fileMode'] = 0644;            // new files mode
-		$this->options['quarantine'] = '.quarantine';  // quarantine folder name - required to check archive (must be hidden)
-		$this->options['maxArcFilesSize'] = 0;        // max allowed archive files size (0 - no limit)
+		$this->options['alias']    = ''; // alias to replace root dir name
+		$this->options['dirMode']  = 0755; // new dirs mode
+		$this->options['fileMode'] = 0644; // new files mode
+		$this->options['quarantine'] = '.quarantine'; // quarantine folder name - required to check archive (must be hidden)
+		$this->options['maxArcFilesSize'] = 0; // max allowed archive files size (0 - no limit)
 		$this->options['rootCssClass'] = 'elfinder-navbar-root-local';
 	}
 	
@@ -53,7 +53,7 @@ 
 	protected function init() {
 		// Normalize directory separator for windows
 		if (DIRECTORY_SEPARATOR !== '/') {
-			foreach(array('path', 'tmbPath', 'tmpPath', 'quarantine') as $key) {
+			foreach (array('path', 'tmbPath', 'tmpPath', 'quarantine') as $key) {
 				if (!empty($this->options[$key])) {
 					$this->options[$key] = str_replace('/', DIRECTORY_SEPARATOR, $this->options[$key]);
 				}
@@ -117,7 +117,7 @@ 
 		// if no thumbnails url - try detect it
 		if ($root['read'] && !$this->tmbURL && $this->URL) {
 			if (strpos($this->tmbPath, $this->root) === 0) {
-				$this->tmbURL = $this->URL.str_replace(DIRECTORY_SEPARATOR, '/', substr($this->tmbPath, strlen($this->root)+1));
+				$this->tmbURL = $this->URL.str_replace(DIRECTORY_SEPARATOR, '/', substr($this->tmbPath, strlen($this->root) + 1));
 				if (preg_match("|[^/?&=]$|", $this->tmbURL)) {
 					$this->tmbURL .= '/';
 				}
@@ -175,12 +175,12 @@ 
 		if ($mtime != $compare) {
 			return $mtime;
 		}
-		$inotifywait = defined('ELFINER_INOTIFYWAIT_PATH')? ELFINER_INOTIFYWAIT_PATH : 'inotifywait';
+		$inotifywait = defined('ELFINER_INOTIFYWAIT_PATH') ? ELFINER_INOTIFYWAIT_PATH : 'inotifywait';
 		$path = escapeshellarg($path);
 		$standby = max(1, intval($standby));
 		$cmd = $inotifywait.' '.$path.' -t '.$standby.' -e moved_to,moved_from,move,create,delete,delete_self';
 		$o = $r = '';
-		$this->procExec($cmd , $o, $r);
+		$this->procExec($cmd, $o, $r);
 		if ($r === 0) {
 			// changed
 			clearstatcache();
@@ -191,7 +191,7 @@ 
 		}
 		// error
 		// cache to $_SESSION
-		$sessionClose = true;;
+		$sessionClose = true; ;
 		try {
 			$sessionStart = session_start();
 		} catch (Exception $e) {
@@ -243,7 +243,7 @@ 
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _joinPath($dir, $name) {
-		return rtrim($dir, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . $name;
+		return rtrim($dir, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$name;
 	}
 	
 	/**
@@ -295,7 +295,7 @@ 
 		$comps = $new_comps;
 		$path = implode('/', $comps);
 		if ($initial_slashes) {
-			$path = str_repeat('/', $initial_slashes) . $path;
+			$path = str_repeat('/', $initial_slashes).$path;
 		}
 		
 		if ($changeSep) {
@@ -366,7 +366,7 @@ 
 	 **/
 	protected function _inpath($path, $parent) {
 		$cwd = getcwd();
-		$real_path   = $this->getFullPath($path,   $cwd);
+		$real_path   = $this->getFullPath($path, $cwd);
 		$real_parent = $this->getFullPath($parent, $cwd);
 		if ($real_path && $real_parent) {
 			return $real_path === $real_parent || strpos($real_path, rtrim($real_parent, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR) === 0;
@@ -425,7 +425,7 @@ 
 					$stat = array();
 					return $stat;
 				} else {
-					$stat['mime']  = 'symlink-broken';
+					$stat['mime'] = 'symlink-broken';
 					$target = readlink($path);
 					$lstat = lstat($path);
 					$ostat = $this->getOwnerStat($lstat['uid'], $lstat['gid']);
@@ -441,7 +441,7 @@ 
 			$fstat = stat($path);
 			$uid = $fstat['uid'];
 			$gid = $fstat['gid'];
-			$stat['perm'] = substr((string)decoct($fstat['mode']), -4);
+			$stat['perm'] = substr((string) decoct($fstat['mode']), -4);
 			$stat = array_merge($stat, $this->getOwnerStat($uid, $gid));
 		}
 		
@@ -451,8 +451,8 @@ 
 			$stat['mime'] = $dir ? 'directory' : $this->mimetype($path);
 		}
 		//logical rights first
-		$stat['read'] = ($linkreadable || is_readable($path))? null : false;
-		$stat['write'] = is_writable($path)? null : false;
+		$stat['read'] = ($linkreadable || is_readable($path)) ? null : false;
+		$stat['write'] = is_writable($path) ? null : false;
 
 		if (is_null($stat['read'])) {
 			$stat['size'] = $dir ? 0 : $size;
@@ -522,7 +522,7 @@ 
 
 		if (is_dir($path)) {
 			$path = strtr($path, array('['  => '\\[', ']'  => '\\]', '*'  => '\\*', '?'  => '\\?'));
-			return (bool)glob(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . '*', GLOB_ONLYDIR);
+			return (bool) glob(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.'*', GLOB_ONLYDIR);
 		}
 		return false;
 	}
@@ -603,7 +603,7 @@ 
 							$br = true;
 						} else {
 							$_path = $fpath;
-							$stat['mime']  = 'symlink-broken';
+							$stat['mime'] = 'symlink-broken';
 							$target = readlink($_path);
 							$lstat = lstat($_path);
 							$ostat = $this->getOwnerStat($lstat['uid'], $lstat['gid']);
@@ -628,13 +628,13 @@ 
 					if ($statOwner && !$linkreadable) {
 						$uid = $file->getOwner();
 						$gid = $file->getGroup();
-						$stat['perm'] = substr((string)decoct($file->getPerms()), -4);
+						$stat['perm'] = substr((string) decoct($file->getPerms()), -4);
 						$stat = array_merge($stat, $this->getOwnerStat($uid, $gid));
 					}
 					
 					//logical rights first
-					$stat['read'] = ($linkreadable || $file->isReadable())? null : false;
-					$stat['write'] = $file->isWritable()? null : false;
+					$stat['read'] = ($linkreadable || $file->isReadable()) ? null : false;
+					$stat['write'] = $file->isWritable() ? null : false;
 					
 					if (is_null($stat['read'])) {
 						$stat['size'] = $dir ? 0 : $size;
@@ -650,7 +650,7 @@ 
 		
 		if ($cache) {
 			$cache = $this->convEncOut($cache, false);
-			foreach($cache as $d) {
+			foreach ($cache as $d) {
 				$this->updateCache($d[0], $d[1]);
 			}
 		}
@@ -666,7 +666,7 @@ 
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fopen($path, $mode='rb') {
+	protected function _fopen($path, $mode = 'rb') {
 		return @fopen($path, $mode);
 	}
 	
@@ -677,7 +677,7 @@ 
 	 * @return bool
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fclose($fp, $path='') {
+	protected function _fclose($fp, $path = '') {
 		return @fclose($fp);
 	}
 	
@@ -809,7 +809,7 @@ 
 		$path = $this->_joinPath($dir, $name);
 
 		$meta = stream_get_meta_data($fp);
-		$uri = isset($meta['uri'])? $meta['uri'] : '';
+		$uri = isset($meta['uri']) ? $meta['uri'] : '';
 		if ($uri && @is_file($uri)) {
 			fclose($fp);
 			$isCmdPaste = ($this->ARGS['cmd'] === 'paste');
@@ -873,7 +873,7 @@ 
 	 * @return bool
 	 **/
 	protected function _chmod($path, $mode) {
-		$modeOct = is_string($mode) ? octdec($mode) : octdec(sprintf("%04o",$mode));
+		$modeOct = is_string($mode) ? octdec($mode) : octdec(sprintf("%04o", $mode));
 		$ret = @chmod($path, $modeOct);
 		$ret && clearstatcache();
 		return  $ret;
@@ -991,7 +991,7 @@ 
 				$src = $dir;
 				$name = basename($path);
 				if (preg_match('/\.((tar\.(gz|bz|bz2|z|lzo))|cpio\.gz|ps\.gz|xcf\.(gz|bz2)|[a-z0-9]{1,4})$/i', $name, $m)) {
-					$name = substr($name, 0,  strlen($name)-strlen($m[0]));
+					$name = substr($name, 0, strlen($name) - strlen($m[0]));
 				}
 				$test = dirname($path).DIRECTORY_SEPARATOR.$name;
 				if (file_exists($test) || is_link($test)) {
@@ -1000,9 +1000,9 @@ 
 			}
 			
 			if ($name !== '') {
-				$result  = dirname($path).DIRECTORY_SEPARATOR.$name;
+				$result = dirname($path).DIRECTORY_SEPARATOR.$name;
 
-				if (! @rename($src, $result)) {
+				if (!@rename($src, $result)) {
 					$this->delTree($dir);
 					return false;
 				}
@@ -1010,7 +1010,7 @@ 
 				$dstDir = dirname($path);
 				$res = false;
 				$result = array();
-				foreach($ls as $name) {
+				foreach ($ls as $name) {
 					$target = $dstDir.DIRECTORY_SEPARATOR.$name;
 					if (is_dir($target)) {
 						$this->delTree($target);
@@ -1102,10 +1102,10 @@ 
 		
 		$path = strtr($path, $escaper);
 		$_q = strtr($q, $escaper);
-		$dirs = glob(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . '*', GLOB_ONLYDIR);
-		$match = glob(rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . '*'.$_q.'*', GLOB_NOSORT);
+		$dirs = glob(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.'*', GLOB_ONLYDIR);
+		$match = glob(rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.'*'.$_q.'*', GLOB_NOSORT);
 		if ($match) {
-			foreach($match as $p) {
+			foreach ($match as $p) {
 				$stat = $this->stat($p);
 		
 				if (!$stat) { // invalid links
@@ -1122,7 +1122,7 @@ 
 					$stat['path'] = $this->path($stat['hash']);
 					if ($this->URL && !isset($stat['url'])) {
 						$path = str_replace(DIRECTORY_SEPARATOR, '/', substr($p, strlen($this->root) + 1));
-						$stat['url'] = $this->URL . $path;
+						$stat['url'] = $this->URL.$path;
 					}
 		
 					$result[] = $stat;
@@ -1130,7 +1130,7 @@ 
 			}
 		}
 		if ($dirs) {
-			foreach($dirs as $dir) {
+			foreach ($dirs as $dir) {
 				$stat = $this->stat($dir);
 				if ($stat['read'] && !isset($stat['alias'])) {
 					@set_time_limit(30);