Studio-42 / elFinder

php/elFinder.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -1344,7 +1344,7 @@
 	 * 
 	 * @param  string $str
 	 * @param  array  $extTable
-	 * @return array
+	 * @return string[]
 	 * @author Naoki Sawada
 	 */
 	protected function parse_data_scheme( $str, $extTable ) {

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);
+			$sleep = max(1, (int) $volume->getOption('lsPlSleep'));
+			$limit = max(1, (int) $volume->getOption('plStandby')/$sleep);
 			$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++;
 			}

php/elFinderVolumeDropbox.class.php

Doc Comments +5 added lines, -12 removed lines

@@ -234,7 +234,6 @@ 
 	 * process of on netunmount
 	 * Drop table `dropbox` & rm thumbs
 	 * 
-	 * @param array $options
 	 * @return boolean
 	 */
 	public function netunmount($netVolumes, $key) {
@@ -431,7 +430,7 @@ 
 	/**
 	 * Check DB for delta cache
 	 * 
-	 * @return void
+	 * @return boolean
 	 */
 	private function checkDB() {
 		$res = $this->query('SELECT * FROM sqlite_master WHERE type=\'table\' AND name=\''.$this->DB_TableName.'\'');
@@ -685,7 +684,7 @@ 
 	* @param  string  $src   source path
 	* @param  string  $dst   destination dir path
 	* @param  string  $name  new file name (optionaly)
-	* @return string|false
+	* @return string|boolean
 	* @author Dmitry (dio) Levashov
 	* @author Naoki Sawada
 	**/
@@ -739,10 +738,6 @@ 
 	* Create thumnbnail and return it's URL on success
 	*
 	* @param  string  $path  file path
-	* @param  string  $mime  file mime type
-	* @return string|false
-	* @author Dmitry (dio) Levashov
-	* @author Naoki Sawada
 	**/
 	protected function createTmb($path, $stat) {
 		if (!$stat || !$this->canCreateTmb($path, $stat)) {
@@ -1099,7 +1094,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -1141,7 +1135,7 @@ 
 	 * Close opened file
 	 *
 	 * @param  resource  $fp  file pointer
-	 * @return bool
+	 * @return boolean|null
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _fclose($fp, $path='') {
@@ -1225,7 +1219,7 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
 	 * @return string|bool
 	 * @author Dmitry (dio) Levashov
@@ -1274,7 +1268,6 @@ 
 	 * Return new file path or false on error.
 	 *
 	 * @param  resource  $fp   file pointer
-	 * @param  string    $dir  target dir path
 	 * @param  string    $name file name
 	 * @param  array     $stat file stat (required by some virtual fs)
 	 * @return bool|string
@@ -1320,7 +1313,7 @@ 
 	 *
 	 * @param  string  $path     file path
 	 * @param  string  $content  new file content
-	 * @return bool
+	 * @return boolean|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _filePutContents($path, $content) {

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

Doc Comments +12 added lines, -12 removed lines

@@ -477,7 +477,7 @@ 
 	 * Return ftp transfer mode for file
 	 *
 	 * @param  string  $path  file path
-	 * @return string
+	 * @return integer
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function ftpMode($path) {
@@ -841,7 +841,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -866,7 +865,7 @@ 
 	 * Close opened file
 	 *
 	 * @param  resource  $fp  file pointer
-	 * @return bool
+	 * @return boolean|null
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _fclose($fp, $path='') {
@@ -883,7 +882,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -901,7 +900,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -933,7 +932,7 @@ 
 	 * @param  string  $source     source file path
 	 * @param  string  $targetDir  target directory path
 	 * @param  string  $name       new file name
-	 * @return bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _copy($source, $targetDir, $name) {
@@ -958,9 +957,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -998,7 +997,7 @@ 
 	 * @param  string    $dir  target dir path
 	 * @param  string    $name file name
 	 * @param  array     $stat file stat (required by some virtual fs)
-	 * @return bool|string
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _save($fp, $dir, $name, $stat) {
@@ -1231,7 +1230,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov,
 	 * @author Alexey Sukhotin
 	 **/
@@ -1350,9 +1349,9 @@ 
 	 * Downloads specified files from remote directory
 	 * if there is a directory among files it is downloaded recursively (omitting symbolic links).
 	 * 
-	 * @param $remote_directory string remote FTP path to a source directory to download from.
+	 * @param string $remote_directory string remote FTP path to a source directory to download from.
 	 * @param array $files list of files to download from remote directory.
-	 * @param $dest_local_directory string destination folder to store downloaded files.
+	 * @param string $dest_local_directory string destination folder to store downloaded files.
 	 * @return bool true on success and false on failure.
 	 */
 	private function ftp_download_files($remote_directory, array $files, $dest_local_directory)
@@ -1423,6 +1422,7 @@ 
 	 * Returns array of strings containing all files and folders in the specified local directory.
 	 * @param $dir
 	 * @param string $prefix
+	 * @param boolean $omitSymlinks
 	 * @internal param string $path path to directory to scan.
 	 * @return array array of files and folders names relative to the $path
 	 * or an empty array if the directory $path is empty,

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

Doc Comments +7 added lines, -8 removed lines

@@ -593,7 +593,6 @@ 
 	 * Open file and return file pointer
 	 *
 	 * @param  string  $path  file path
-	 * @param  bool    $write open file for writing
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
@@ -619,7 +618,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -639,7 +638,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -687,9 +686,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -733,7 +732,7 @@ 
 	 * @param  string    $dir  target dir path
 	 * @param  string    $name file name
 	 * @param  array     $stat file stat (required by some virtual fs)
-	 * @return bool|string
+	 * @return false|string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _save($fp, $dir, $name, $stat) {
@@ -761,7 +760,7 @@ 
 	 * Get file contents
 	 *
 	 * @param  string  $path  file path
-	 * @return string|false
+	 * @return string
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _getContents($path) {
@@ -965,7 +964,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/

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/elFinderVolumeMySQL.class.php

Doc Comments +7 added lines, -7 removed lines

@@ -611,7 +611,7 @@ 
 	 * Close opened file
 	 *
 	 * @param  resource  $fp  file pointer
-	 * @return bool
+	 * @return boolean|null
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _fclose($fp, $path='') {
@@ -628,7 +628,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new directory name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkdir($path, $name) {
@@ -640,7 +640,7 @@ 
 	 *
 	 * @param  string  $path  parent dir path
 	 * @param string  $name  new file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _mkfile($path, $name) {
@@ -684,9 +684,9 @@ 
 	 * Return new file path or false.
 	 *
 	 * @param  string  $source  source file path
-	 * @param  string  $target  target dir path
+	 * @param  string  $targetDir  target dir path
 	 * @param  string  $name    file name
-	 * @return string|bool
+	 * @return string|false
 	 * @author Dmitry (dio) Levashov
 	 **/
 	protected function _move($source, $targetDir, $name) {
@@ -869,7 +869,7 @@ 
 	 *
 	 * @param  string  $path  archive path
 	 * @param  array   $arc   archiver command and arguments (same as in $this->archivers)
-	 * @return true
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/
@@ -884,7 +884,7 @@ 
 	 * @param  array   $files  files names list
 	 * @param  string  $name   archive name
 	 * @param  array   $arc    archiver options
-	 * @return string|bool
+	 * @return boolean
 	 * @author Dmitry (dio) Levashov, 
 	 * @author Alexey Sukhotin
 	 **/

Spacing +6 added lines, -6 removed lines

@@ -311,7 +311,7 @@ 
 		$dirs = array();
 		if ($path != $this->root) {
 			$inpath = array(intval($path));
-			while($inpath) {
+			while ($inpath) {
 				$in = '('.join(',', $inpath).')';
 				$inpath = array();
 				$sql = 'SELECT f.id FROM %s AS f WHERE f.parent_id IN '.$in.' AND `mime` = \'directory\'';
@@ -330,7 +330,7 @@ 
 		
 		if ($mimes) {
 			$whrs = array();
-			foreach($mimes as $mime) {
+			foreach ($mimes as $mime) {
 				if (strpos($mime, '/') === false) {
 					$whrs[] = sprintf('f.mime LIKE "%s/%%"', $this->db->real_escape_string($mime));
 				} else {
@@ -342,7 +342,7 @@ 
 			$whr = sprintf('f.name RLIKE "%s"', $this->db->real_escape_string($q));
 		}
 		if ($dirs) {
-			$whr = '(' . $whr . ') AND (`parent_id` IN (' . join(',', $dirs) . '))';
+			$whr = '('.$whr.') AND (`parent_id` IN ('.join(',', $dirs).'))';
 		}
 		
 		$sql = 'SELECT f.id, f.parent_id, f.name, f.size, f.mtime AS ts, f.mime, f.read, f.write, f.locked, f.hidden, f.width, f.height, 0 AS dirs 
@@ -587,7 +587,7 @@ 
 	 * @return resource|false
 	 * @author Dmitry (dio) Levashov
 	 **/
-	protected function _fopen($path, $mode='rb') {
+	protected function _fopen($path, $mode = 'rb') {
 		$fp = $this->tmbPath
 			? @fopen($this->getTempFile($path), 'w+')
 			: @tmpfile();
@@ -614,7 +614,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));
@@ -746,7 +746,7 @@ 
 		$this->clearcache();
 		
 		$mime = $stat['mime'];
-		$w = !empty($stat['width'])  ? $stat['width']  : 0;
+		$w = !empty($stat['width']) ? $stat['width'] : 0;
 		$h = !empty($stat['height']) ? $stat['height'] : 0;
 		
 		$id = $this->_joinPath($dir, $name);

php/plugins/AutoResize/plugin.php

Doc Comments +8 added lines

@@ -107,6 +107,10 @@ 
 		}
 	}
 	
+	/**
+	 * @param double $width
+	 * @param double $height
+	 */
 	private function resize_gd($src, $width, $height, $quality, $srcImgInfo) {
 		switch ($srcImgInfo['mime']) {
 			case 'image/gif':
@@ -177,6 +181,10 @@ 
 		return false;
 	}
 	
+	/**
+	 * @param double $width
+	 * @param double $height
+	 */
 	private function resize_imagick($src, $width, $height, $quality) {
 		try {
 			$img = new imagick($src);

Spacing +14 added lines, -14 removed lines

@@ -52,12 +52,12 @@ 
 
 	public function __construct($opts) {
 		$defaults = array(
-			'enable'         => true,       // For control by volume driver
-			'maxWidth'       => 1024,       // Path to Water mark image
-			'maxHeight'      => 1024,       // Margin right pixel
-			'quality'        => 95,         // JPEG image save quality
-			'preserveExif'   => false,      // Preserve EXIF data (Imagick only)
-			'targetType'     => IMG_GIF|IMG_JPG|IMG_PNG|IMG_WBMP // Target image formats ( bit-field )
+			'enable'         => true, // For control by volume driver
+			'maxWidth'       => 1024, // Path to Water mark image
+			'maxHeight'      => 1024, // Margin right pixel
+			'quality'        => 95, // JPEG image save quality
+			'preserveExif'   => false, // Preserve EXIF data (Imagick only)
+			'targetType'     => IMG_GIF | IMG_JPG | IMG_PNG | IMG_WBMP // Target image formats ( bit-field )
 		);
 
 		$this->opts = array_merge($defaults, $opts);
@@ -71,7 +71,7 @@ 
 			$opts = array_merge($this->opts, $volOpts);
 		}
 		
-		if (! $opts['enable']) {
+		if (!$opts['enable']) {
 			return false;
 		}
 		
@@ -87,7 +87,7 @@ 
 				IMAGETYPE_PNG => IMG_PNG,
 				IMAGETYPE_WBMP => IMG_WBMP,
 		);
-		if (! ($opts['targetType'] & $imgTypes[$srcImgInfo[2]])) {
+		if (!($opts['targetType'] & $imgTypes[$srcImgInfo[2]])) {
 			return false;
 		}
 		
@@ -99,9 +99,9 @@ 
 	}
 	
 	private function resize($src, $srcImgInfo, $maxWidth, $maxHeight, $quality, $preserveExif) {
-		$zoom = min(($maxWidth/$srcImgInfo[0]),($maxHeight/$srcImgInfo[1]));
-		$width = round($srcImgInfo[0] * $zoom);
-		$height = round($srcImgInfo[1] * $zoom);
+		$zoom = min(($maxWidth/$srcImgInfo[0]), ($maxHeight/$srcImgInfo[1]));
+		$width = round($srcImgInfo[0]*$zoom);
+		$height = round($srcImgInfo[1]*$zoom);
 		
 		if (class_exists('Imagick', false)) {
 			return $this->resize_imagick($src, $width, $height, $quality, $preserveExif);
@@ -121,14 +121,14 @@ 
 				break;
 			case 'image/jpeg':
 				if (@imagetypes() & IMG_JPG) {
-					$oSrcImg = @imagecreatefromjpeg($src) ;
+					$oSrcImg = @imagecreatefromjpeg($src);
 				} else {
 					$ermsg = 'JPEG images are not supported';
 				}
 				break;
 			case 'image/png':
 				if (@imagetypes() & IMG_PNG) {
-					$oSrcImg = @imagecreatefrompng($src) ;
+					$oSrcImg = @imagecreatefrompng($src);
 				} else {
 					$ermsg = 'PNG images are not supported';
 				}
@@ -146,7 +146,7 @@ 
 				break;
 		}
 		
-		if ($oSrcImg &&  false != ($tmp = imagecreatetruecolor($width, $height))) {
+		if ($oSrcImg && false != ($tmp = imagecreatetruecolor($width, $height))) {
 			
 			if (!imagecopyresampled($tmp, $oSrcImg, 0, 0, 0, 0, $width, $height, $srcImgInfo[0], $srcImgInfo[1])) {
 				return false;

php/chars-test.php

Spacing +2 added lines, -2 removed lines

@@ -24,10 +24,10 @@
 
 //                     й                 ё              Й               Ё              Ø         Å
 $patterns = array("\u0438\u0306", "\u0435\u0308", "\u0418\u0306", "\u0415\u0308", "\u00d8A", "\u030a");
-$replace  = array("\u0439",        "\u0451",       "\u0419",       "\u0401",       "\u00d8", "\u00c5");
+$replace  = array("\u0439", "\u0451", "\u0419", "\u0401", "\u00d8", "\u00c5");
 
 
-foreach(scandir($path) as $f) {
+foreach (scandir($path) as $f) {
 	if ($f != '.' && $f != '..' && substr($f, 0, 1) != '.') {
 		
 		// echo mb_detect_encoding($f);

php/elFinderConnector.class.php

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,7 @@ 
 	 * @return void
 	 * @author Dmitry (dio) Levashov
 	 **/
-	public function __construct($elFinder, $debug=false) {
+	public function __construct($elFinder, $debug = false) {
 		
 		$this->elFinder = $elFinder;
 		if ($debug) {
@@ -54,7 +54,7 @@ 
 		if ($isPost && !$src && $rawPostData = @file_get_contents('php://input')) {
 			// for support IE XDomainRequest()
 			$parts = explode('&', $rawPostData);
-			foreach($parts as $part) {
+			foreach ($parts as $part) {
 				list($key, $value) = array_pad(explode('=', $part), 2, '');
 				$key = rawurldecode($key);
 				if (substr($key, -2) === '[]') {
@@ -121,7 +121,7 @@ 
 	 **/
 	protected function output(array $data) {
 		// clear output buffer
-		while(@ob_get_level()){ @ob_end_clean(); }
+		while (@ob_get_level()) { @ob_end_clean(); }
 		
 		$header = isset($data['header']) ? $data['header'] : $this->header;
 		unset($data['header']);
@@ -162,14 +162,14 @@ 
 							$psize = $end - $start + 1;
 							
 							header('HTTP/1.1 206 Partial Content');
-							header('Content-Length: ' . $psize);
-							header('Content-Range: bytes ' . $start . '-' . $end . '/' . $size);
+							header('Content-Length: '.$psize);
+							header('Content-Range: bytes '.$start.'-'.$end.'/'.$size);
 							
 							fseek($fp, $start);
 						}
 					}
 				}
-				if (is_null($psize)){
+				if (is_null($psize)) {
 					rewind($fp);
 				}
 			} else {

php/elFinderVolumeS3.class.php

Spacing +20 added lines, -20 removed lines

@@ -61,8 +61,8 @@ 
 	 **/

 	protected function _dirname($path) {

 	

-		$newpath =  preg_replace("/\/$/", "", $path);

-		$dn = substr($path, 0, strrpos($newpath, '/')) ;

+		$newpath = preg_replace("/\/$/", "", $path);

+		$dn = substr($path, 0, strrpos($newpath, '/'));

 		

 		if (substr($dn, 0, 1) != '/') {

 		 $dn = "/$dn";

@@ -105,9 +105,9 @@ 
 	 * @author Troex Nevelin

 	 **/

 	protected function _normpath($path) {

-		$tmp =  preg_replace("/^\//", "", $path);

-		$tmp =  preg_replace("/\/\//", "/", $tmp);

-		$tmp =  preg_replace("/\/$/", "", $tmp);

+		$tmp = preg_replace("/^\//", "", $path);

+		$tmp = preg_replace("/\/\//", "/", $tmp);

+		$tmp = preg_replace("/\/$/", "", $tmp);

 		return $tmp;

 	}

 

@@ -128,9 +128,9 @@ 
 			$newpath = "/$newpath";

 		}

 		

-		$newpath =  preg_replace("/\/$/", "", $newpath);

+		$newpath = preg_replace("/\/$/", "", $newpath);

 	

-		$ret = ($newpath == $this->root) ? '' : substr($newpath, strlen($this->root)+1);

+		$ret = ($newpath == $this->root) ? '' : substr($newpath, strlen($this->root) + 1);

 		

 		return $ret;

 	}

@@ -231,7 +231,7 @@ 
 		$np = $this->_normpath($path);

 		

 		try {

-			$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $np , 'GetMetadata' => true, 'InlineData' => false, 'GetData' => false));

+			$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $np, 'GetMetadata' => true, 'InlineData' => false, 'GetData' => false));

 		} catch (Exception $e) {

 		

 		}

@@ -239,7 +239,7 @@ 
 		if (!isset($obj) || ($obj->GetObjectResponse->Status->Code != 200)) {

 			$np .= '/';

 			try {

-				$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $np , 'GetMetadata' => true, 'InlineData' => false, 'GetData' => false));

+				$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $np, 'GetMetadata' => true, 'InlineData' => false, 'GetData' => false));

 			} catch (Exception $e) {

 		

 			}

@@ -341,14 +341,14 @@ 
 	 **/

 	protected function _scandir($path) {

 		

-		$s3path = preg_replace("/^\//", "", $path) . '/';

+		$s3path = preg_replace("/^\//", "", $path).'/';

 		

 		$files = $this->s3->ListBucket(array('Bucket' => $this->options['bucket'], 'delimiter' => '/', 'Prefix' => $s3path))->ListBucketResponse->Contents;

 	

 		$finalfiles = array();

 		

 		foreach ($files as $file) {

-			if (preg_match("|^" . $s3path . "[^/]*/?$|", $file->Key)) {

+			if (preg_match("|^".$s3path."[^/]*/?$|", $file->Key)) {

 				$fname = preg_replace("/\/$/", "", $file->Key);

 				$fname = $file->Key;

 				

@@ -373,7 +373,7 @@ 
 	 * @author Dmitry (dio) Levashov,

 	 * @author Alexey Sukhotin

 	 **/

-	protected function _fopen($path, $mode="rb") {

+	protected function _fopen($path, $mode = "rb") {

 	

 		$tn = $this->getTempFile($path);

 	

@@ -385,8 +385,8 @@ 
 		if ($fp) {

 

 			try {

-				$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $this->_normpath($path) , 'GetMetadata' => true, 'InlineData' => true, 'GetData' => true));

-			}	catch (Exception $e) {

+				$obj = $this->s3->GetObject(array('Bucket' => $this->options['bucket'], 'Key' => $this->_normpath($path), 'GetMetadata' => true, 'InlineData' => true, 'GetData' => true));

+			} catch (Exception $e) {

 		

 			}

 				

@@ -410,7 +410,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));

@@ -435,7 +435,7 @@ 
 		$newkey = "$newkey/$name/";

 

 		try {

-			$obj = $this->s3->PutObjectInline(array('Bucket' => $this->options['bucket'], 'Key' => $newkey , 'ContentLength' => 0, 'Data' => ''));

+			$obj = $this->s3->PutObjectInline(array('Bucket' => $this->options['bucket'], 'Key' => $newkey, 'ContentLength' => 0, 'Data' => ''));

 		} catch (Exception $e) {

 		

 		}

@@ -462,7 +462,7 @@ 
 		$newkey = "$newkey/$name";

 

 		try {

-			$obj = $this->s3->PutObjectInline(array('Bucket' => $this->options['bucket'], 'Key' => $newkey , 'ContentLength' => 0, 'Data' => '', 'Metadata' => array(array('Name' => 'Content-Type', 'Value' => 'text/plain'))));

+			$obj = $this->s3->PutObjectInline(array('Bucket' => $this->options['bucket'], 'Key' => $newkey, 'ContentLength' => 0, 'Data' => '', 'Metadata' => array(array('Name' => 'Content-Type', 'Value' => 'text/plain'))));

 		} catch (Exception $e) {

 		

 		}

@@ -563,7 +563,7 @@ 
 	 * @author Dmitry (dio) Levashov

 	 **/

 	protected function _rmdir($path) {

-		return $this->_unlink($path . '/');

+		return $this->_unlink($path.'/');

 	}

 

 	/**

@@ -682,7 +682,7 @@ 
 		/* Getting list of S3 web service functions which requires signing */

 		$funcs = $this->__getFunctions();

 		

-		$funcnames  = array();

+		$funcnames = array();

 		

 		foreach ($funcs as $func) {

 			preg_match("/\S+\s+([^\)]+)\(/", $func, $m);

@@ -723,7 +723,7 @@ 
 			'Timestamp' => gmdate('Y-m-d\TH:i:s.000\Z'),

 		);

 

-		$sign_str = 'AmazonS3' . $operation . $params['Timestamp'];

+		$sign_str = 'AmazonS3'.$operation.$params['Timestamp'];

 		

 		$params['Signature'] = base64_encode(hash_hmac('sha1', $sign_str, $this->secretkey, TRUE));

 		

Doc Comments +5 added lines, -5 removed lines

@@ -407,7 +407,7 @@ 
 	 * 

 	 * @param  resource  $fp    file pointer

 	 * @param  string    $path  file path

-	 * @return bool

+	 * @return boolean|null

 	 * @author Dmitry (dio) Levashov

 	 **/

 	protected function _fclose($fp, $path='') {

@@ -508,7 +508,7 @@ 
 	 * @param  string  $source  source file path

 	 * @param  string  $targetDir  target dir path

 	 * @param  string  $name    file name

-	 * @return string|bool

+	 * @return boolean

 	 * @author Dmitry (dio) Levashov

 	 **/

 	protected function _move($source, $targetDir, $name) {

@@ -574,7 +574,7 @@ 
 	 * @param  string $dir target dir path

 	 * @param  string $name file name

 	 * @param  array $stat

-	 * @return bool|string

+	 * @return boolean

 	 * @author Dmitry (dio) Levashov

 	 */

 	protected function _save($fp, $dir, $name, $stat) {

@@ -585,7 +585,7 @@ 
 	 * Get file contents

 	 *

 	 * @param  string  $path  file path

-	 * @return string|false

+	 * @return boolean

 	 * @author Dmitry (dio) Levashov

 	 **/

 	protected function _getContents($path) {

@@ -624,7 +624,7 @@ 
 	 * @param  array   $files  files names list

 	 * @param  string  $name   archive name

 	 * @param  array   $arc    archiver options

-	 * @return string|bool

+	 * @return boolean

 	 * @author Dmitry (dio) Levashov, 

 	 * @author Alexey Sukhotin

 	 **/