Studio-42 / elFinder

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);

php/elFinder.class.php

Spacing +162 added lines, -162 removed lines

@@ -203,14 +203,14 @@ 
 	const ERROR_INV_PARAMS        = 'errCmdParams';
 	const ERROR_OPEN              = 'errOpen';
 	const ERROR_DIR_NOT_FOUND     = 'errFolderNotFound';
-	const ERROR_FILE_NOT_FOUND    = 'errFileNotFound';     // 'File not found.'
+	const ERROR_FILE_NOT_FOUND    = 'errFileNotFound'; // 'File not found.'
 	const ERROR_TRGDIR_NOT_FOUND  = 'errTrgFolderNotFound'; // 'Target folder "$1" not found.'
 	const ERROR_NOT_DIR           = 'errNotFolder';
 	const ERROR_NOT_FILE          = 'errNotFile';
 	const ERROR_PERM_DENIED       = 'errPerm';
-	const ERROR_LOCKED            = 'errLocked';        // '"$1" is locked and can not be renamed, moved or removed.'
-	const ERROR_EXISTS            = 'errExists';        // 'File named "$1" already exists.'
-	const ERROR_INVALID_NAME      = 'errInvName';       // 'Invalid file name.'
+	const ERROR_LOCKED            = 'errLocked'; // '"$1" is locked and can not be renamed, moved or removed.'
+	const ERROR_EXISTS            = 'errExists'; // 'File named "$1" already exists.'
+	const ERROR_INVALID_NAME      = 'errInvName'; // 'Invalid file name.'
 	const ERROR_MKDIR             = 'errMkdir';
 	const ERROR_MKFILE            = 'errMkfile';
 	const ERROR_RENAME            = 'errRename';
@@ -219,20 +219,20 @@ 
 	const ERROR_COPY_FROM         = 'errCopyFrom';
 	const ERROR_COPY_TO           = 'errCopyTo';
 	const ERROR_COPY_ITSELF       = 'errCopyInItself';
-	const ERROR_REPLACE           = 'errReplace';          // 'Unable to replace "$1".'
-	const ERROR_RM                = 'errRm';               // 'Unable to remove "$1".'
-	const ERROR_RM_SRC            = 'errRmSrc';            // 'Unable remove source file(s)'
-	const ERROR_MKOUTLINK         = 'errMkOutLink';        // 'Unable to create a link to outside the volume root.'
-	const ERROR_UPLOAD            = 'errUpload';           // 'Upload error.'
-	const ERROR_UPLOAD_FILE       = 'errUploadFile';       // 'Unable to upload "$1".'
-	const ERROR_UPLOAD_NO_FILES   = 'errUploadNoFiles';    // 'No files found for upload.'
-	const ERROR_UPLOAD_TOTAL_SIZE = 'errUploadTotalSize';  // 'Data exceeds the maximum allowed size.'
-	const ERROR_UPLOAD_FILE_SIZE  = 'errUploadFileSize';   // 'File exceeds maximum allowed size.'
-	const ERROR_UPLOAD_FILE_MIME  = 'errUploadMime';       // 'File type not allowed.'
-	const ERROR_UPLOAD_TRANSFER   = 'errUploadTransfer';   // '"$1" transfer error.'
-	const ERROR_UPLOAD_TEMP       = 'errUploadTemp';       // 'Unable to make temporary file for upload.'
+	const ERROR_REPLACE           = 'errReplace'; // 'Unable to replace "$1".'
+	const ERROR_RM                = 'errRm'; // 'Unable to remove "$1".'
+	const ERROR_RM_SRC            = 'errRmSrc'; // 'Unable remove source file(s)'
+	const ERROR_MKOUTLINK         = 'errMkOutLink'; // 'Unable to create a link to outside the volume root.'
+	const ERROR_UPLOAD            = 'errUpload'; // 'Upload error.'
+	const ERROR_UPLOAD_FILE       = 'errUploadFile'; // 'Unable to upload "$1".'
+	const ERROR_UPLOAD_NO_FILES   = 'errUploadNoFiles'; // 'No files found for upload.'
+	const ERROR_UPLOAD_TOTAL_SIZE = 'errUploadTotalSize'; // 'Data exceeds the maximum allowed size.'
+	const ERROR_UPLOAD_FILE_SIZE  = 'errUploadFileSize'; // 'File exceeds maximum allowed size.'
+	const ERROR_UPLOAD_FILE_MIME  = 'errUploadMime'; // 'File type not allowed.'
+	const ERROR_UPLOAD_TRANSFER   = 'errUploadTransfer'; // '"$1" transfer error.'
+	const ERROR_UPLOAD_TEMP       = 'errUploadTemp'; // 'Unable to make temporary file for upload.'
 	// const ERROR_ACCESS_DENIED     = 'errAccess';
-	const ERROR_NOT_REPLACE       = 'errNotReplace';       // Object "$1" already exists at this location and can not be replaced with object of another type.
+	const ERROR_NOT_REPLACE       = 'errNotReplace'; // Object "$1" already exists at this location and can not be replaced with object of another type.
 	const ERROR_SAVE              = 'errSave';
 	const ERROR_EXTRACT           = 'errExtract';
 	const ERROR_ARCHIVE           = 'errArchive';
@@ -247,14 +247,14 @@ 
 	const ERROR_NETMOUNT          = 'errNetMount';
 	const ERROR_NETUNMOUNT        = 'errNetUnMount';
 	const ERROR_NETMOUNT_NO_DRIVER = 'errNetMountNoDriver';
-	const ERROR_NETMOUNT_FAILED       = 'errNetMountFailed';
+	const ERROR_NETMOUNT_FAILED = 'errNetMountFailed';
 
-	const ERROR_SESSION_EXPIRES 	= 'errSessionExpires';
+	const ERROR_SESSION_EXPIRES = 'errSessionExpires';
 
 	const ERROR_CREATING_TEMP_DIR 	= 'errCreatingTempDir';
 	const ERROR_FTP_DOWNLOAD_FILE 	= 'errFtpDownloadFile';
-	const ERROR_FTP_UPLOAD_FILE 	= 'errFtpUploadFile';
-	const ERROR_FTP_MKDIR 		= 'errFtpMkdir';
+	const ERROR_FTP_UPLOAD_FILE = 'errFtpUploadFile';
+	const ERROR_FTP_MKDIR = 'errFtpMkdir';
 	const ERROR_ARCHIVE_EXEC 	= 'errArchiveExec';
 	const ERROR_EXTRACT_EXEC 	= 'errExtractExec';
 
@@ -281,17 +281,17 @@ 
 		
 		$this->time  = $this->utime();
 		$this->debug = (isset($opts['debug']) && $opts['debug'] ? true : false);
-		$this->sessionCloseEarlier = isset($opts['sessionCloseEarlier'])? (bool)$opts['sessionCloseEarlier'] : true;
+		$this->sessionCloseEarlier = isset($opts['sessionCloseEarlier']) ? (bool) $opts['sessionCloseEarlier'] : true;
 		$this->sessionUseCmds = array_flip($sessionUseCmds);
 		$this->timeout = (isset($opts['timeout']) ? $opts['timeout'] : 0);
 		$this->uploadTempPath = (isset($opts['uploadTempPath']) ? $opts['uploadTempPath'] : '');
-		$this->netVolumesSessionKey = !empty($opts['netVolumesSessionKey'])? $opts['netVolumesSessionKey'] : 'elFinderNetVolumes';
+		$this->netVolumesSessionKey = !empty($opts['netVolumesSessionKey']) ? $opts['netVolumesSessionKey'] : 'elFinderNetVolumes';
 		$this->callbackWindowURL = (isset($opts['callbackWindowURL']) ? $opts['callbackWindowURL'] : '');
 		self::$sessionCacheKey = !empty($opts['sessionCacheKey']) ? $opts['sessionCacheKey'] : 'elFinderCaches';
 		
 		// check session cache
 		$_optsMD5 = md5(json_encode($opts['roots']));
-		if (! isset($_SESSION[self::$sessionCacheKey]) || $_SESSION[self::$sessionCacheKey]['_optsMD5'] !== $_optsMD5) {
+		if (!isset($_SESSION[self::$sessionCacheKey]) || $_SESSION[self::$sessionCacheKey]['_optsMD5'] !== $_optsMD5) {
 			$_SESSION[self::$sessionCacheKey] = array(
 				'_optsMD5' => $_optsMD5
 			);
@@ -310,20 +310,20 @@ 
 			$_reqCmd = isset($_req['cmd']) ? $_req['cmd'] : '';
 			foreach ($opts['bind'] as $cmd => $handlers) {
 				$doRegist = (strpos($cmd, '*') !== false);
-				if (! $doRegist) {
+				if (!$doRegist) {
 					$_getcmd = create_function('$cmd', 'list($ret) = explode(\'.\', $cmd);return trim($ret);');
 					$doRegist = ($_reqCmd && in_array($_reqCmd, array_map($_getcmd, explode(' ', $cmd))));
 				}
 				if ($doRegist) {
-					if (! is_array($handlers) || is_object($handlers[0])) {
+					if (!is_array($handlers) || is_object($handlers[0])) {
 						$handlers = array($handlers);
 					}
-					foreach($handlers as $handler) {
+					foreach ($handlers as $handler) {
 						if ($handler) {
 							if (is_string($handler) && strpos($handler, '.')) {
 								list($_domain, $_name, $_method) = array_pad(explode('.', $handler), 3, '');
 								if (strcasecmp($_domain, 'plugin') === 0) {
-									if ($plugin = $this->getPluginInstance($_name, isset($opts['plugin'][$_name])? $opts['plugin'][$_name] : array())
+									if ($plugin = $this->getPluginInstance($_name, isset($opts['plugin'][$_name]) ? $opts['plugin'][$_name] : array())
 											and method_exists($plugin, $_method)) {
 										$this->bind($cmd, array($plugin, $_method));
 									}
@@ -410,13 +410,13 @@ 
 	public function bind($cmd, $handler) {
 		$allCmds = array_keys($this->commands);
 		$cmds = array();
-		foreach(explode(' ', $cmd) as $_cmd) {
+		foreach (explode(' ', $cmd) as $_cmd) {
 			if ($_cmd !== '') {
 				if ($all = strpos($_cmd, '*') !== false) {
 					list(, $sub) = array_pad(explode('.', $_cmd), 2, '');
 					if ($sub) {
 						$sub = str_replace('\'', '\\\'', $sub);
-						$addSub = create_function('$cmd', 'return $cmd . \'.\' . trim(\'' . $sub . '\');');
+						$addSub = create_function('$cmd', 'return $cmd . \'.\' . trim(\''.$sub.'\');');
 						$cmds = array_merge($cmds, array_map($addSub, $allCmds));
 					} else {
 						$cmds = array_merge($cmds, $allCmds);
@@ -496,16 +496,16 @@ 
 
 	private function session_expires() {
 		
-		if (!isset($_SESSION[self::$sessionCacheKey . ':LAST_ACTIVITY'])) {
-			$_SESSION[self::$sessionCacheKey . ':LAST_ACTIVITY'] = time();
+		if (!isset($_SESSION[self::$sessionCacheKey.':LAST_ACTIVITY'])) {
+			$_SESSION[self::$sessionCacheKey.':LAST_ACTIVITY'] = time();
 			return false;
 		}
 
-		if ( ($this->timeout > 0) && (time() - $_SESSION[self::$sessionCacheKey . ':LAST_ACTIVITY'] > $this->timeout) ) {
+		if (($this->timeout > 0) && (time() - $_SESSION[self::$sessionCacheKey.':LAST_ACTIVITY'] > $this->timeout)) {
 			return true;
 		}
 
-		$_SESSION[self::$sessionCacheKey . ':LAST_ACTIVITY'] = time();
+		$_SESSION[self::$sessionCacheKey.':LAST_ACTIVITY'] = time();
 		return false;	
 	}
 	
@@ -538,9 +538,9 @@ 
 		}
 
 		// call pre handlers for this command
-		$args['sessionCloseEarlier'] = isset($this->sessionUseCmds[$cmd])? false : $this->sessionCloseEarlier;
+		$args['sessionCloseEarlier'] = isset($this->sessionUseCmds[$cmd]) ? false : $this->sessionCloseEarlier;
 		if (!empty($this->listeners[$cmd.'.pre'])) {
-			$volume = isset($args['target'])? $this->volume($args['target']) : false;
+			$volume = isset($args['target']) ? $this->volume($args['target']) : false;
 			foreach ($this->listeners[$cmd.'.pre'] as $handler) {
 				call_user_func_array($handler, array($cmd, &$args, $this, $volume));
 			}
@@ -550,7 +550,7 @@ 
 		$this->sessionCloseEarlier && $args['sessionCloseEarlier'] && session_id() && session_write_close();
 		unset($this->sessionCloseEarlier);
 		
-		if (substr(PHP_OS,0,3) === 'WIN') {
+		if (substr(PHP_OS, 0, 3) === 'WIN') {
 			// set time out
 			if (($_max_execution_time = ini_get('max_execution_time')) && $_max_execution_time < 300) {
 				@set_time_limit(300);
@@ -569,7 +569,7 @@ 
 		// call handlers for this command
 		if (!empty($this->listeners[$cmd])) {
 			foreach ($this->listeners[$cmd] as $handler) {
-				if (call_user_func_array($handler,array($cmd,&$result,$args,$this))) {
+				if (call_user_func_array($handler, array($cmd, &$result, $args, $this))) {
 					// handler return true to force sync client after command completed
 					$result['sync'] = true;
 				}
@@ -628,7 +628,7 @@ 
 	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
-	public function realpath($hash)	{
+	public function realpath($hash) {
 		if (($volume = $this->volume($hash)) == false) {
 			return false;
 		}
@@ -684,11 +684,11 @@ 
 	 */
 	protected function getPluginInstance($name, $opts = array()) {
 		$key = strtolower($name);
-		if (! isset($this->plugins[$key])) {
-			$p_file = dirname(__FILE__) . DIRECTORY_SEPARATOR . 'plugins' . DIRECTORY_SEPARATOR . $name . DIRECTORY_SEPARATOR . 'plugin.php';
+		if (!isset($this->plugins[$key])) {
+			$p_file = dirname(__FILE__).DIRECTORY_SEPARATOR.'plugins'.DIRECTORY_SEPARATOR.$name.DIRECTORY_SEPARATOR.'plugin.php';
 			if (is_file($p_file)) {
 				require_once $p_file;
-				$class = 'elFinderPlugin' . $name;
+				$class = 'elFinderPlugin'.$name;
 				$this->plugins[$key] = new $class($opts);
 			} else {
 				$this->plugins[$key] = false;
@@ -780,8 +780,8 @@ 
 		
 		$netVolumes = $this->getNetVolumes();
 		if ($volume->mount($options)) {
-			if (! $key = @ $volume->netMountKey) {
-				$key = md5($protocol . '-' . join('-', $options));
+			if (!$key = @ $volume->netMountKey) {
+				$key = md5($protocol.'-'.join('-', $options));
 			}
 			$options['driver'] = $driver;
 			$options['netkey'] = $key;
@@ -850,14 +850,14 @@ 
 		}
 		// long polling mode
 		if ($args['compare']) {
-			$sleep = max(1, (int)$volume->getOption('lsPlSleep'));
-			$limit = max(1, (int)$volume->getOption('plStandby') / $sleep) + 1;
+			$sleep = max(1, (int) $volume->getOption('lsPlSleep'));
+			$limit = max(1, (int) $volume->getOption('plStandby')/$sleep) + 1;
 			$timelimit = ini_get('max_execution_time');
 			$compare = $args['compare'];
 			do {
 				$timelimit && @ set_time_limit($timelimit + $sleep);
 				$_mtime = 0;
-				foreach($ls as $_f) {
+				foreach ($ls as $_f) {
 					$_mtime = max($_mtime, $_f['ts']);
 				}
 				$compare = strval(count($ls)).':'.strval($_mtime);
@@ -871,7 +871,7 @@ 
 						break;
 					}
 				}
-			} while($limit);
+			} while ($limit);
 			if ($ls === false) {
 				return array('error' => $this->error(self::ERROR_OPEN, $cwd['name'], $volume->error()));
 			}
@@ -1014,24 +1014,24 @@ 
 		}
 
 		// allow change MIME type by 'file.pre' callback functions
-		$mime = isset($args['mime'])? $args['mime'] : $file['mime'];
+		$mime = isset($args['mime']) ? $args['mime'] : $file['mime'];
 		if ($download) {
 			$disp = 'attachment';
 		} else {
 			$dispInlineRegex = $volume->getOption('dispInlineRegex');
 			$inlineRegex = false;
 			if ($dispInlineRegex) {
-				$inlineRegex = '#' . str_replace('#', '\\#', $dispInlineRegex) . '#';
+				$inlineRegex = '#'.str_replace('#', '\\#', $dispInlineRegex).'#';
 				try {
 					preg_match($inlineRegex, '');
-				} catch(Exception $e) {
+				} catch (Exception $e) {
 					$inlineRegex = false;
 				}
 			}
 			if (!$inlineRegex) {
 				$inlineRegex = '#^(?:(?:image|text)|application/x-shockwave-flash$)#';
 			}
-			$disp  = preg_match($inlineRegex, $mime)? 'inline' : 'attachment';
+			$disp = preg_match($inlineRegex, $mime) ? 'inline' : 'attachment';
 		}
 		
 		$filenameEncoded = rawurlencode($file['name']);
@@ -1140,7 +1140,7 @@ 
 		$name   = $args['name'];
 		
 		if (($volume = $this->volume($target)) == false
-		||  ($rm  = $volume->file($target)) == false) {
+		||  ($rm = $volume->file($target)) == false) {
 			return array('error' => $this->error(self::ERROR_RENAME, '#'.$target, self::ERROR_FILE_NOT_FOUND));
 		}
 		$rm['realpath'] = $volume->realpath($target);
@@ -1218,9 +1218,9 @@ 
 	* @retval false  error
 	* @author Naoki Sawada
 	**/
-	protected function get_remote_contents( &$url, $timeout = 30, $redirect_max = 5, $ua = 'Mozilla/5.0', $fp = null ) {
-		$method = (function_exists('curl_exec') && !ini_get('safe_mode'))? 'curl_get_contents' : 'fsock_get_contents'; 
-		return $this->$method( $url, $timeout, $redirect_max, $ua, $fp );
+	protected function get_remote_contents(&$url, $timeout = 30, $redirect_max = 5, $ua = 'Mozilla/5.0', $fp = null) {
+		$method = (function_exists('curl_exec') && !ini_get('safe_mode')) ? 'curl_get_contents' : 'fsock_get_contents'; 
+		return $this->$method($url, $timeout, $redirect_max, $ua, $fp);
 	}
 	
 	/**
@@ -1236,26 +1236,26 @@ 
 	 * @retval false  error
 	 * @author Naoki Sawada
 	 **/
-	 protected function curl_get_contents( &$url, $timeout, $redirect_max, $ua, $outfp ){
+	 protected function curl_get_contents(&$url, $timeout, $redirect_max, $ua, $outfp) {
 		$ch = curl_init();
-		curl_setopt( $ch, CURLOPT_URL, $url );
-		curl_setopt( $ch, CURLOPT_HEADER, false );
+		curl_setopt($ch, CURLOPT_URL, $url);
+		curl_setopt($ch, CURLOPT_HEADER, false);
 		if ($outfp) {
-			curl_setopt( $ch, CURLOPT_FILE, $outfp );
+			curl_setopt($ch, CURLOPT_FILE, $outfp);
 		} else {
-			curl_setopt( $ch, CURLOPT_RETURNTRANSFER, true );
-			curl_setopt( $ch, CURLOPT_BINARYTRANSFER, true );
-		}
-		curl_setopt( $ch, CURLOPT_LOW_SPEED_LIMIT, 1 );
-		curl_setopt( $ch, CURLOPT_LOW_SPEED_TIME, $timeout );
-		curl_setopt( $ch, CURLOPT_SSL_VERIFYPEER, false );
-		curl_setopt( $ch, CURLOPT_FOLLOWLOCATION, 1);
-		curl_setopt( $ch, CURLOPT_MAXREDIRS, $redirect_max);
-		curl_setopt( $ch, CURLOPT_USERAGENT, $ua);
-		$result = curl_exec( $ch );
+			curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
+			curl_setopt($ch, CURLOPT_BINARYTRANSFER, true);
+		}
+		curl_setopt($ch, CURLOPT_LOW_SPEED_LIMIT, 1);
+		curl_setopt($ch, CURLOPT_LOW_SPEED_TIME, $timeout);
+		curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
+		curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1);
+		curl_setopt($ch, CURLOPT_MAXREDIRS, $redirect_max);
+		curl_setopt($ch, CURLOPT_USERAGENT, $ua);
+		$result = curl_exec($ch);
 		$url = curl_getinfo($ch, CURLINFO_EFFECTIVE_URL);
-		curl_close( $ch );
-		return $outfp? $outfp : $result;
+		curl_close($ch);
+		return $outfp ? $outfp : $result;
 	}
 	
 	/**
@@ -1271,7 +1271,7 @@ 
 	 * @retval false  error
 	 * @author Naoki Sawada
 	 */
-	protected function fsock_get_contents( &$url, $timeout, $redirect_max, $ua, $outfp ) {
+	protected function fsock_get_contents(&$url, $timeout, $redirect_max, $ua, $outfp) {
 
 		$connect_timeout = 3;
 		$connect_try = 3;
@@ -1282,7 +1282,7 @@ 
 		$headers = '';
 		
 		$arr = parse_url($url);
-		if (!$arr){
+		if (!$arr) {
 			// Bad request
 			return false;
 		}
@@ -1290,7 +1290,7 @@ 
 		// query
 		$arr['query'] = isset($arr['query']) ? '?'.$arr['query'] : '';
 		// port
-		$arr['port'] = isset($arr['port']) ? $arr['port'] : (!empty($arr['https'])? 443 : 80);
+		$arr['port'] = isset($arr['port']) ? $arr['port'] : (!empty($arr['https']) ? 443 : 80);
 		
 		$url_base = $arr['scheme'].'://'.$arr['host'].':'.$arr['port'];
 		$url_path = isset($arr['path']) ? $arr['path'] : '/';
@@ -1299,21 +1299,21 @@ 
 		$query = $method.' '.$uri." HTTP/1.0\r\n";
 		$query .= "Host: ".$arr['host']."\r\n";
 		if (!empty($ua)) $query .= "User-Agent: ".$ua."\r\n";
-		if (!is_null($getSize)) $query .= 'Range: bytes=0-' . ($getSize - 1) . "\r\n";
+		if (!is_null($getSize)) $query .= 'Range: bytes=0-'.($getSize - 1)."\r\n";
 		
 		$query .= $headers;
 
 		$query .= "\r\n";
 
 		$fp = $connect_try_count = 0;
-		while( !$fp && $connect_try_count < $connect_try ) {
+		while (!$fp && $connect_try_count < $connect_try) {
 	
 			$errno = 0;
 			$errstr = "";
 			$fp = @ fsockopen(
 			$arr['https'].$arr['host'],
 			$arr['port'],
-			$errno,$errstr,$connect_timeout);
+			$errno, $errstr, $connect_timeout);
 			if ($fp) break;
 			$connect_try_count++;
 			if (connection_aborted()) {
@@ -1338,13 +1338,13 @@ 
 		
 		$_response = '';
 		$header = '';
-		while($_response !== "\r\n"){
+		while ($_response !== "\r\n") {
 			$_response = fgets($fp, $readsize);
 			$header .= $_response;
 		};
 		
-		$rccd = array_pad(explode(' ',$header,3), 3, ''); // array('HTTP/1.1','200','OK\r\n...')
-		$rc = (int)$rccd[1];
+		$rccd = array_pad(explode(' ', $header, 3), 3, ''); // array('HTTP/1.1','200','OK\r\n...')
+		$rc = (int) $rccd[1];
 		
 		// Redirect
 		switch ($rc) {
@@ -1353,19 +1353,19 @@ 
 			case 302: // Moved Temporarily
 			case 301: // Moved Permanently
 				$matches = array();
-				if (preg_match('/^Location: (.+?)(#.+)?$/im',$header,$matches) && --$redirect_max > 0) {
+				if (preg_match('/^Location: (.+?)(#.+)?$/im', $header, $matches) && --$redirect_max > 0) {
 					$url = trim($matches[1]);
-					$hash = isset($matches[2])? trim($matches[2]) : '';
-					if (!preg_match('/^https?:\//',$url)) { // no scheme
+					$hash = isset($matches[2]) ? trim($matches[2]) : '';
+					if (!preg_match('/^https?:\//', $url)) { // no scheme
 						if ($url{0} != '/') { // Relative path
 							// to Absolute path
-							$url = substr($url_path,0,strrpos($url_path,'/')).'/'.$url;
+							$url = substr($url_path, 0, strrpos($url_path, '/')).'/'.$url;
 						}
 						// add sheme,host
 						$url = $url_base.$url;
 					}
 					fclose($fp);
-					return $this->fsock_get_contents( $url, $timeout, $redirect_max, $ua, $outfp );
+					return $this->fsock_get_contents($url, $timeout, $redirect_max, $ua, $outfp);
 				}
 		}
 		
@@ -1374,7 +1374,7 @@ 
 			$outfp = fopen('php://temp', 'rwb');
 			$body = true;
 		}
-		while(fwrite($outfp, fread($fp, $readsize))) {
+		while (fwrite($outfp, fread($fp, $readsize))) {
 			if ($timeout) {
 				$_status = socket_get_status($fp);
 				if ($_status['timed_out']) {
@@ -1393,7 +1393,7 @@ 
 		
 		fclose($fp);
 		
-		return $outfp? $outfp : $body; // Data
+		return $outfp ? $outfp : $body; // Data
 	}
 	
 	/**
@@ -1404,13 +1404,13 @@ 
 	 * @return array
 	 * @author Naoki Sawada
 	 */
-	protected function parse_data_scheme( $str, $extTable ) {
+	protected function parse_data_scheme($str, $extTable) {
 		$data = $name = '';
 		if ($fp = fopen('data://'.substr($str, 5), 'rb')) {
 			if ($data = stream_get_contents($fp)) {
 				$meta = stream_get_meta_data($fp);
-				$ext = isset($extTable[$meta['mediatype']])? '.' . $extTable[$meta['mediatype']] : '';
-				$name = substr(md5($data), 0, 8) . $ext;
+				$ext = isset($extTable[$meta['mediatype']]) ? '.'.$extTable[$meta['mediatype']] : '';
+				$name = substr(md5($data), 0, 8).$ext;
 			}
 			fclose($fp);
 		}
@@ -1474,7 +1474,7 @@ 
 			}
 		}
 		
-		return ($mime && isset($extTable[$mime]))? ('.' . $extTable[$mime]) : '';
+		return ($mime && isset($extTable[$mime])) ? ('.'.$extTable[$mime]) : '';
 	}
 	
 	/**
@@ -1496,12 +1496,12 @@ 
 			$testDirs[] = sys_get_temp_dir();
 		}
 		$tempDir = '';
-		foreach($testDirs as $testDir) {
+		foreach ($testDirs as $testDir) {
 			if (!$testDir || !is_dir($testDir)) continue;
 			if (is_writable($testDir)) {
 				$tempDir = $testDir;
 				$gc = time() - 3600;
-				foreach(glob($tempDir . DIRECTORY_SEPARATOR .'ELF*') as $cf) {
+				foreach (glob($tempDir.DIRECTORY_SEPARATOR.'ELF*') as $cf) {
 					if (filemtime($cf) < $gc) {
 						@unlink($cf);
 					}
@@ -1521,7 +1521,7 @@ 
 	 **/
 	protected function chmod($args) {
 		$targets = $args['targets'];
-		$mode    = intval((string)$args['mode'], 8);
+		$mode    = intval((string) $args['mode'], 8);
 
 		if (!is_array($targets)) {
 			$targets = array($targets);
@@ -1536,10 +1536,10 @@ 
 
 		$files = array();
 		$errors = array();
-		foreach($targets as $target) {
+		foreach ($targets as $target) {
 			$file = $volume->chmod($target, $mode);
 			if ($file) {
-				$files = array_merge($files, is_array($file)? $file : array($file));
+				$files = array_merge($files, is_array($file) ? $file : array($file));
 			} else {
 				$errors = array_merge($errors, $volume->error());
 			}
@@ -1570,26 +1570,26 @@ 
 	private function checkChunkedFile($tmpname, $chunk, $cid, $tempDir, $volume = null) {
 		if (preg_match('/^(.+)(\.\d+_(\d+))\.part$/s', $chunk, $m)) {
 			$fname = $m[1];
-			$encname = md5($cid . '_' . $fname);
-			$base = $tempDir . DIRECTORY_SEPARATOR . 'ELF' . $encname;
+			$encname = md5($cid.'_'.$fname);
+			$base = $tempDir.DIRECTORY_SEPARATOR.'ELF'.$encname;
 			$clast = intval($m[3]);
 			if (is_null($tmpname)) {
 				ignore_user_abort(true);
 				sleep(10); // wait 10 sec
 				// chunked file upload fail
-				foreach(glob($base . '*') as $cf) {
+				foreach (glob($base.'*') as $cf) {
 					@unlink($cf);
 				}
 				ignore_user_abort(false);
 				return;
 			}
 			
-			$range = isset($_POST['range'])? trim($_POST['range']) : '';
+			$range = isset($_POST['range']) ? trim($_POST['range']) : '';
 			if ($range && preg_match('/^(\d+),(\d+),(\d+)$/', $range, $ranges)) {
 				$start = $ranges[1];
 				$len   = $ranges[2];
 				$size  = $ranges[3];
-				$tmp = $base . '.part';
+				$tmp = $base.'.part';
 				$csize = filesize($tmpname);
 				
 				$tmpExists = is_file($tmp);
@@ -1615,7 +1615,7 @@ 
 				} else {
 					// wait until makeing temp file (for anothor session)
 					$cnt = 1200; // Time limit 120 sec
-					while(!is_file($base) && --$cnt) {
+					while (!is_file($base) && --$cnt) {
 						usleep(100000); // wait 100ms
 					}
 					if (!$cnt) {
@@ -1650,13 +1650,13 @@ 
 				}
 			} else {
 				// old way
-				$part = $base . $m[2];
+				$part = $base.$m[2];
 				if (move_uploaded_file($tmpname, $part)) {
 					@chmod($part, 0600);
-					if ($clast < count(glob($base . '*'))) {
+					if ($clast < count(glob($base.'*'))) {
 						$parts = array();
 						for ($i = 0; $i <= $clast; $i++) {
-							$name = $base . '.' . $i . '_' . $clast;
+							$name = $base.'.'.$i.'_'.$clast;
 							if (is_readable($name)) {
 								$parts[] = $name;
 							} else {
@@ -1669,7 +1669,7 @@ 
 								touch($base);
 								if ($resfile = tempnam($tempDir, 'ELF')) {
 									$target = fopen($resfile, 'wb');
-									foreach($parts as $f) {
+									foreach ($parts as $f) {
 										$fp = fopen($f, 'rb');
 										while (!feof($fp)) {
 											fwrite($target, fread($fp, 8192));
@@ -1705,15 +1705,15 @@ 
 		$files  = isset($args['FILES']['upload']) && is_array($args['FILES']['upload']) ? $args['FILES']['upload'] : array();
 		$header = empty($args['html']) ? array() : array('header' => 'Content-Type: text/html; charset=utf-8');
 		$result = array_merge(array('added' => array()), $header);
-		$paths  = $args['upload_path']? $args['upload_path'] : array();
-		$chunk  = $args['chunk']? $args['chunk'] : '';
-		$cid    = $args['cid']? (int)$args['cid'] : '';
+		$paths  = $args['upload_path'] ? $args['upload_path'] : array();
+		$chunk  = $args['chunk'] ? $args['chunk'] : '';
+		$cid    = $args['cid'] ? (int) $args['cid'] : '';
 		
-		$renames= array();
+		$renames = array();
 		$suffix = '~';
 		if ($args['renames'] && is_array($args['renames'])) {
 			$renames = array_flip($args['renames']);
-			if (is_string($args['suffix']) && ! preg_match($ngReg, $args['suffix'])) {
+			if (is_string($args['suffix']) && !preg_match($ngReg, $args['suffix'])) {
 				$suffix = $args['suffix'];
 			}
 		}
@@ -1746,7 +1746,7 @@ 
 			if (!$args['upload'] && $args['name'] && is_array($args['name'])) {
 				$error = '';
 				$result['name'] = array();
-				foreach($args['name'] as $_i => $_name) {
+				foreach ($args['name'] as $_i => $_name) {
 					if (!$volume->isUploadableByName($_name)) {
 						$error = $this->error(self::ERROR_UPLOAD_FILE, $_name, self::ERROR_UPLOAD_FILE_MIME);
 						break;
@@ -1767,7 +1767,7 @@ 
 			}
 			if (isset($args['upload']) && is_array($args['upload']) && ($tempDir = $this->getTempDir($volume->getTempPath()))) {
 				$names = array();
-				foreach($args['upload'] as $i => $url) {
+				foreach ($args['upload'] as $i => $url) {
 					// check chunked file upload commit
 					if ($args['chunk']) {
 						if ($url === 'chunkfail' && $args['mimes'] === 'chunkfail') {
@@ -1777,7 +1777,7 @@ 
 							}
 							return $result;
 						} else {
-							$tmpfname = $tempDir . '/' . $args['chunk'];
+							$tmpfname = $tempDir.'/'.$args['chunk'];
 							$files['tmp_name'][$i] = $tmpfname;
 							$files['name'][$i] = $url;
 							$files['error'][$i] = 0;
@@ -1786,7 +1786,7 @@ 
 						}
 					}
 					
-					$tmpfname = $tempDir . DIRECTORY_SEPARATOR . 'ELF_FATCH_' . md5($url.microtime(true));
+					$tmpfname = $tempDir.DIRECTORY_SEPARATOR.'ELF_FATCH_'.md5($url.microtime(true));
 					
 					$_name = '';
 					// check is data:
@@ -1822,8 +1822,8 @@ 
 								list($_a, $_b) = array_pad(explode('.', $_name, 2), 2, '');
 								if ($_b === '') {
 									if ($_ext) {
-										rename($tmpfname, $tmpfname . $_ext);
-										$tmpfname = $tmpfname . $_ext;
+										rename($tmpfname, $tmpfname.$_ext);
+										$tmpfname = $tmpfname.$_ext;
 									}
 									$_b = $this->detectFileExtension($tmpfname);
 									$_name = $_a.$_b;
@@ -1858,7 +1858,7 @@ 
 			}
 			
 			$tmpname = $files['tmp_name'][$i];
-			$path = ($paths && !empty($paths[$i]))? $paths[$i] : '';
+			$path = ($paths && !empty($paths[$i])) ? $paths[$i] : '';
 			if ($name === 'blob') {
 				if ($chunk) {
 					if ($tempDir = $this->getTempDir($volume->getTempPath())) {
@@ -1868,7 +1868,7 @@ 
 								preg_match('/^(.+)(\.\d+_(\d+))\.part$/s', $chunk, $m);
 								$result['error'] = $this->error(self::ERROR_UPLOAD_FILE, $m[1], $tmpname);
 								$result['_chunkfailure'] = true;
-								$this->uploadDebug = 'Upload error: ' . $tmpname;
+								$this->uploadDebug = 'Upload error: '.$tmpname;
 							} else if ($name) {
 								$result['_chunkmerged'] = basename($tmpname);
 								$result['_name'] = $name;
@@ -1882,14 +1882,14 @@ 
 				} else {
 					// for form clipboard with Google Chrome
 					$type = $files['type'][$i];
-					$ext = isset($extTable[$type])? '.' . $extTable[$type] : '';
-					$name = substr(md5(basename($tmpname)), 0, 8) . $ext;
+					$ext = isset($extTable[$type]) ? '.'.$extTable[$type] : '';
+					$name = substr(md5(basename($tmpname)), 0, 8).$ext;
 				}
 			}
 			
 			// do hook function 'upload.presave'
-			if (! empty($this->listeners['upload.presave'])) {
-				foreach($this->listeners['upload.presave'] as $handler) {
+			if (!empty($this->listeners['upload.presave'])) {
+				foreach ($this->listeners['upload.presave'] as $handler) {
 					call_user_func_array($handler, array(&$path, &$name, $tmpname, $this, $volume));
 				}
 			}
@@ -1897,7 +1897,7 @@ 
 			if (($fp = fopen($tmpname, 'rb')) == false) {
 				$result['warning'] = $this->error(self::ERROR_UPLOAD_FILE, $name, self::ERROR_UPLOAD_TRANSFER);
 				$this->uploadDebug = 'Upload error: unable open tmp file';
-				if (! is_uploaded_file($tmpname)) {
+				if (!is_uploaded_file($tmpname)) {
 					if (@ unlink($tmpname)) unset($GLOBALS['elFinderTempFiles'][$tmpfname]);
 					continue;
 				}
@@ -1919,18 +1919,18 @@ 
 					}
 				}
 			}
-			if (! $_target || ($file = $volume->upload($fp, $_target, $name, $tmpname)) === false) {
+			if (!$_target || ($file = $volume->upload($fp, $_target, $name, $tmpname)) === false) {
 				$result['warning'] = $this->error(self::ERROR_UPLOAD_FILE, $name, $volume->error());
 				fclose($fp);
-				if (! is_uploaded_file($tmpname)) {
-					if (@ unlink($tmpname)) unset($GLOBALS['elFinderTempFiles'][$tmpname]);;
+				if (!is_uploaded_file($tmpname)) {
+					if (@ unlink($tmpname)) unset($GLOBALS['elFinderTempFiles'][$tmpname]); ;
 					continue;
 				}
 				break;
 			}
 			
 			is_resource($fp) && fclose($fp);
-			if (! is_uploaded_file($tmpname)){
+			if (!is_uploaded_file($tmpname)) {
 				clearstatcache();
 				if (!is_file($tmpname) || @ unlink($tmpname)) {
 					unset($GLOBALS['elFinderTempFiles'][$tmpname]);
@@ -1942,7 +1942,7 @@ 
 			}
 		}
 		if ($GLOBALS['elFinderTempFiles']) {
-			foreach(array_keys($GLOBALS['elFinderTempFiles']) as $_temp) {
+			foreach (array_keys($GLOBALS['elFinderTempFiles']) as $_temp) {
 				@ unlink($_temp);
 			}
 		}
@@ -1979,7 +1979,7 @@ 
 		$suffix = '~';
 		if (!empty($args['renames'])) {
 			$renames = array_flip($args['renames']);
-			if (is_string($args['suffix']) && ! preg_match('/[\/\\?*:|"<>]/', $args['suffix'])) {
+			if (is_string($args['suffix']) && !preg_match('/[\/\\?*:|"<>]/', $args['suffix'])) {
 				$suffix = $args['suffix'];
 			}
 		}
@@ -2037,9 +2037,9 @@ 
 		}
 		
 		if ($args['conv'] && function_exists('mb_detect_encoding') && function_exists('mb_convert_encoding')) {
-			$mime = isset($file['mime'])? $file['mime'] : '';
+			$mime = isset($file['mime']) ? $file['mime'] : '';
 			if ($mime && strtolower(substr($mime, 0, 4)) === 'text') {
-				if ($enc = mb_detect_encoding ( $content , mb_detect_order(), true)) {
+				if ($enc = mb_detect_encoding($content, mb_detect_order(), true)) {
 					if (strtolower($enc) !== 'utf-8') {
 						$content = mb_convert_encoding($content, 'UTF-8', $enc);
 					}
@@ -2051,7 +2051,7 @@ 
 
 		if ($json === false || strlen($json) < strlen($content)) {
 			if ($args['conv']) {
-				return array('error' => $this->error(self::ERROR_CONV_UTF8,self::ERROR_NOT_UTF8_CONTENT, $volume->path($target)));
+				return array('error' => $this->error(self::ERROR_CONV_UTF8, self::ERROR_NOT_UTF8_CONTENT, $volume->path($target)));
 			} else {
 				return array('doconv' => true);
 			}
@@ -2093,7 +2093,7 @@ 
 		$target = $args['target'];
 		$mimes  = !empty($args['mimes']) && is_array($args['mimes']) ? $args['mimes'] : array();
 		$error  = array(self::ERROR_EXTRACT, '#'.$target);
-		$makedir = isset($args['makedir'])? (bool)$args['makedir'] : null;
+		$makedir = isset($args['makedir']) ? (bool) $args['makedir'] : null;
 
 		if (($volume = $this->volume($target)) == false
 		|| ($file = $volume->file($target)) == false) {
@@ -2101,7 +2101,7 @@ 
 		}  
 
 		return ($file = $volume->extract($target, $makedir))
-			? array('added' => isset($file['read'])? array($file) : $file)
+			? array('added' => isset($file['read']) ? array($file) : $file)
 			: array('error' => $this->error(self::ERROR_EXTRACT, $volume->path($target), $volume->error()));
 	}
 	
@@ -2116,7 +2116,7 @@ 
 	protected function archive($args) {
 		$type    = $args['type'];
 		$targets = isset($args['targets']) && is_array($args['targets']) ? $args['targets'] : array();
-		$name    = isset($args['name'])? $args['name'] : '';
+		$name    = isset($args['name']) ? $args['name'] : '';
 	
 		if (($volume = $this->volume($targets[0])) == false) {
 			return $this->error(self::ERROR_ARCHIVE, self::ERROR_TRGDIR_NOT_FOUND);
@@ -2137,7 +2137,7 @@ 
 	protected function search($args) {
 		$q      = trim($args['q']);
 		$mimes  = !empty($args['mimes']) && is_array($args['mimes']) ? $args['mimes'] : array();
-		$target = !empty($args['target'])? $args['target'] : null;
+		$target = !empty($args['target']) ? $args['target'] : null;
 		$result = array();
 
 		if (!is_null($target)) {
@@ -2168,7 +2168,7 @@ 
 			$compare = intval($args['compare']);
 			$hash = $args['targets'][0];
 			if ($volume = $this->volume($hash)) {
-				$standby = (int)$volume->getOption('plStandby');
+				$standby = (int) $volume->getOption('plStandby');
 				$_compare = false;
 				if (($syncCheckFunc = $volume->getOption('syncCheckFunc')) && is_callable($syncCheckFunc)) {
 					$_compare = call_user_func_array($syncCheckFunc, array($volume->realpath($hash), $standby, $compare, $volume, $this));
@@ -2176,8 +2176,8 @@ 
 				if ($_compare !== false) {
 					$compare = $_compare;
 				} else {
-					$sleep = max(1, (int)$volume->getOption('tsPlSleep'));
-					$limit = max(1, $standby / $sleep) + 1;
+					$sleep = max(1, (int) $volume->getOption('tsPlSleep'));
+					$limit = max(1, $standby/$sleep) + 1;
 					$timelimit = ini_get('max_execution_time');
 					do {
 						$timelimit && @ set_time_limit($timelimit + $sleep);
@@ -2194,7 +2194,7 @@ 
 						if (--$limit) {
 							sleep($sleep);
 						}
-					} while($limit);
+					} while ($limit);
 				}
 			}
 		} else {
@@ -2242,12 +2242,12 @@ 
 		$target = $args['target'];
 		$width  = $args['width'];
 		$height = $args['height'];
-		$x      = (int)$args['x'];
-		$y      = (int)$args['y'];
+		$x      = (int) $args['x'];
+		$y      = (int) $args['y'];
 		$mode   = $args['mode'];
 		$bg     = null;
-		$degree = (int)$args['degree'];
-		$quality= (int)$args['quality'];
+		$degree = (int) $args['degree'];
+		$quality = (int) $args['quality'];
 		
 		if (($volume = $this->volume($target)) == false
 		|| ($file = $volume->file($target)) == false) {
@@ -2268,7 +2268,7 @@ 
 	**/
 	protected function url($args) {
 		$target = $args['target'];
-		$options = isset($args['options'])? $args['options'] : array();
+		$options = isset($args['options']) ? $args['options'] : array();
 		if (($volume = $this->volume($target)) != false) {
 			$url = $volume->getContentUrl($target, $options);
 			return $url ? array('url' => $url) : array();
@@ -2285,18 +2285,18 @@ 
 	 */
 	protected function callback($args) {
 		$checkReg = '/[^a-zA-Z0-9;._-]/';
-		$node = (isset($args['node']) && !preg_match($checkReg, $args['node']))? $args['node'] : '';
-		$json = (isset($args['json']) && @json_decode($args['json']))? $args['json'] : '{}';
-		$bind  = (isset($args['bind']) && !preg_match($checkReg, $args['bind']))? $args['bind'] : '';
+		$node = (isset($args['node']) && !preg_match($checkReg, $args['node'])) ? $args['node'] : '';
+		$json = (isset($args['json']) && @json_decode($args['json'])) ? $args['json'] : '{}';
+		$bind = (isset($args['bind']) && !preg_match($checkReg, $args['bind'])) ? $args['bind'] : '';
 		$done = (!empty($args['done']));
 		
-		while( ob_get_level() ) {
-			if (! ob_end_clean()) {
+		while (ob_get_level()) {
+			if (!ob_end_clean()) {
 				break;
 			}
 		}
 		
-		if ($done || ! $this->callbackWindowURL) {
+		if ($done || !$this->callbackWindowURL) {
 			$script = '';
 			if ($node) {
 				$script .= '
@@ -2338,13 +2338,13 @@ 
 			
 		} else {
 			$url = $this->callbackWindowURL;
-			$url .= ((strpos($url, '?') === false)? '?' : '&')
-				 . '&node=' . rawurlencode($node)
-				 . (($json !== '{}')? ('&json=' . rawurlencode($json)) : '')
-				 . ($bind? ('&bind=' .  rawurlencode($bind)) : '')
+			$url .= ((strpos($url, '?') === false) ? '?' : '&')
+				 . '&node='.rawurlencode($node)
+				 . (($json !== '{}') ? ('&json='.rawurlencode($json)) : '')
+				 . ($bind ? ('&bind='.rawurlencode($bind)) : '')
 				 . '&done=1';
 			
-			header('Location: ' . $url);
+			header('Location: '.$url);
 			
 		}
 		exit();
@@ -2414,7 +2414,7 @@ 
 	
 	protected function utime() {
 		$time = explode(" ", microtime());
-		return (double)$time[1] + (double)$time[0];
+		return (double) $time[1] + (double) $time[0];
 	}
 	
 	
@@ -2440,7 +2440,7 @@ 
 		$imgcnt = 0;
 		$fp = fopen($path, 'rb');
 		@fread($fp, 4);
-		$c = @fread($fp,1);
+		$c = @fread($fp, 1);
 		if (ord($c) != 0x39) {  // GIF89a
 			return false;
 		}
@@ -2448,13 +2448,13 @@ 
 		while (!feof($fp)) {
 			do {
 				$c = fread($fp, 1);
-			} while(ord($c) != 0x21 && !feof($fp));
+			} while (ord($c) != 0x21 && !feof($fp));
 	
 			if (feof($fp)) {
 				break;
 			}
 	
-			$c2 = fread($fp,2);
+			$c2 = fread($fp, 2);
 			if (bin2hex($c2) == "f904") {
 				$imgcnt++;
 			}