elFinderVolumeDriver

Complexity

Total Complexity

1369

Size/Duplication

Total Lines	6810
Duplicated Lines	4.86 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

182 Methods

Rating	Name	Duplication	Size	Complexity
A	driverId()	0	4	1
A	id()	0	4	1
A	setSession()	0	4	1
A	saveSessionCache()	0	4	1
A	debug()	0	9	1
C	chmod()	8	43	12
A	fstat()	0	6	1
A	clearstatcache()	0	5	1
A	clearcaches()	0	9	2
F	mount()	57	368	150
A	umount()	0	3	1
A	clearSessionCache()	0	4	1
A	error()	0	4	1
A	isUploadableByName()	0	6	3
A	getMimeTable()	0	7	2
B	getExtentionByMime()	0	20	8
A	setMimesFilter()	0	6	2
A	root()	0	4	1
A	getHash()	0	8	2
A	getPath()	0	4	1
A	defaultPath()	0	4	2
F	options()	0	55	17
A	getOption()	0	4	2
A	getOptionsPlugin()	0	8	3
A	commandDisabled()	0	4	1
B	mimeAccepted()	0	13	7
A	isReadable()	0	6	1
A	copyFromAllowed()	0	4	1
A	path()	0	4	1
A	realpath()	0	6	2
C	removed()	0	23	8
A	added()	0	4	1
A	resetRemoved()	0	4	1
A	resetResultStat()	0	5	1
A	closest()	0	4	2
A	file()	0	6	2
B	dir()	0	14	7
C	scandir()	3	29	12
C	ls()	0	22	9
B	tree()	3	13	6
C	parents()	0	40	14
B	tmb()	0	23	6
A	size()	0	4	1
A	open()	4	9	3
A	close()	0	4	1
D	mkdir()	0	35	9
D	mkfile()	3	33	10
C	rename()	0	52	12
B	duplicate()	0	22	5
D	upload()	18	86	23
F	paste()	10	125	40
F	zipdl()	0	69	20
B	getContents()	0	22	6
C	putContents()	3	46	13
C	extract()	0	41	11
C	archive()	3	50	14
F	resize()	6	89	27
A	rm()	0	6	2
F	search()	5	53	18
A	dimensions()	0	8	2
A	subdirs()	0	4	1
D	getContentUrl()	0	39	9
B	getTempPath()	0	18	11
B	getUploadTaget()	0	29	5
A	getUploadMaxSize()	0	4	1
A	setUploadOverwrite()	0	4	1
D	imageUtil()	4	46	15
A	ffmpegToImg()	0	22	3
C	uniqueName()	0	31	7
A	convEncIn()	0	4	2
A	convEncOut()	0	4	2
B	getImageSize()	0	14	6
A	rmdirRecursive()	0	4	1
A	init()	0	4	1
F	configure()	20	105	40
A	sessionRestart()	0	6	1
A	setError()	0	7	1
A	addError()	0	12	3
A	dirnameCE()	0	6	3
A	basenameCE()	0	4	2
A	joinPathCE()	0	4	2
A	normpathCE()	0	4	2
A	relpathCE()	0	4	2
A	abspathCE()	0	4	2
A	inpathCE()	0	4	2
A	fopenCE()	0	4	2
A	fcloseCE()	0	4	2
A	saveCE()	0	4	2
B	subdirsCE()	5	12	5
A	scandirCE()	0	4	2
A	symlinkCE()	0	4	2
A	encode()	0	23	3
A	decode()	0	14	2
A	crypt()	0	4	1
A	uncrypt()	0	4	1
B	nameAccepted()	0	19	6
A	getRootstatCachekey()	0	4	2
D	convEnc()	0	31	9
B	getTempFile()	0	28	6
B	getWorkFile()	0	18	5
A	delTree()	0	11	3
F	getItemsInHand()	0	74	25
C	attr()	0	31	11
A	allowCreate()	0	4	1
B	allowPutMime()	0	20	6
D	stat()	8	45	18
A	getRootStatExtra()	0	11	2
A	isNameExists()	0	4	1
F	updateCache()	0	149	58
B	cacheDir()	0	16	6
A	clearcache()	0	4	1
F	mimetype()	0	62	21
C	mimetypeInternalDetect()	13	31	13
C	countSize()	9	45	16
A	isSameType()	0	4	4
B	closestByAttr()	0	18	5
A	childsByAttr()	0	10	3
C	isMyReload()	0	25	12
B	updateSubdirsCache()	0	16	6
B	getScandir()	0	14	5
B	gettree()	0	19	9
C	doSearch()	16	62	26
C	copy()	11	65	18
A	move()	0	18	4
C	copyFrom()	6	68	21
C	remove()	5	32	8
A	tmbname()	0	4	1
B	gettmb()	0	16	5
C	canCreateTmb()	0	23	11
A	canResize()	0	4	1
F	createTmb()	6	135	43
F	imgResize()	25	147	40
D	imgCrop()	9	103	25
D	imgSquareFit()	0	106	23
F	imgRotate()	10	137	30
C	procExec()	0	51	7
B	rmTmb()	0	14	8
C	gdImageCreate()	0	28	8
D	gdImage()	0	31	10
D	imagickImage()	0	36	10
A	gdImageBackground()	5	11	2
C	imageMagickConvertPrepare()	0	47	15
A	stripos()	0	10	4
F	getArchivers()	39	116	30
C	getFullPath()	0	45	12
B	makeArchive()	3	25	6
D	unpackArchive()	3	31	9
D	checkExtractItems()	11	85	28
B	localScandir()	0	18	5
C	localRmdirRecursive()	0	30	11
C	localMoveRecursive()	0	25	8
D	zipArchiveZip()	0	40	13
A	zipArchiveUnzip()	0	14	3
B	localFindSymlinks()	0	20	7
	_dirname()	0	1	?
	_basename()	0	1	?
	_joinPath()	0	1	?
	_normpath()	0	1	?
	_relpath()	0	1	?
	_abspath()	0	1	?
	_path()	0	1	?
	_inpath()	0	1	?
	_stat()	0	1	?
	_subdirs()	0	1	?
	_dimensions()	0	1	?
	_scandir()	0	1	?
	_fopen()	0	1	?
	_fclose()	0	1	?
	_mkdir()	0	1	?
	_mkfile()	0	1	?
	_symlink()	0	1	?
	_copy()	0	1	?
	_move()	0	1	?
	_unlink()	0	1	?
	_rmdir()	0	1	?
	_save()	0	1	?
	_getContents()	0	1	?
	_filePutContents()	0	1	?
	_extract()	0	1	?
	_archive()	0	1	?
	_checkArchivers()	0	1	?
	_chmod()	0	1	?

<?php
/**
 * Base class for elFinder volume.
 * Provide 2 layers:
 *  1. Public API (commands)
 *  2. abstract fs API.
 *
 * All abstract methods begin with "_"
 *
 * @author Dmitry (dio) Levashov
 * @author Troex Nevelin
 * @author Alexey Sukhotin
 **/
abstract class elFinderVolumeDriver

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
	 * Net mount key.
	 *
	 * @var string
	 **/
	public $netMountKey = '';

	/**
	 * Request args
	 * $_POST or $_GET values.
	 *
	 * @var array
	 */
	protected $ARGS = [];

	/**
	 * Driver id
	 * Must be started from letter and contains [a-z0-9]
	 * Used as part of volume id.
	 *
	 * @var string
	 **/
	protected $driverId = 'a';

	/**
	 * Volume id - used as prefix for files hashes.
	 *
	 * @var string
	 **/
	protected $id = '';

	/**
	 * Flag - volume "mounted" and available.
	 *
	 * @var bool
	 **/
	protected $mounted = false;

	/**
	 * Root directory path.
	 *
	 * @var string
	 **/
	protected $root = '';

	/**
	 * Root basename | alias.
	 *
	 * @var string
	 **/
	protected $rootName = '';

	/**
	 * Default directory to open.
	 *
	 * @var string
	 **/
	protected $startPath = '';

	/**
	 * Base URL.
	 *
	 * @var string
	 **/
	protected $URL = '';

	/**
	 * A file save destination path when a temporary content URL is required
	 * on a network volume or the like
	 * If not specified, it tries to use "Connector Path/../files/.tmb".
	 *
	 * @var string
	 */
	protected $tmpLinkPath = '';

	/**
	 * A file save destination URL when a temporary content URL is required
	 * on a network volume or the like
	 * If not specified, it tries to use "Connector URL/../files/.tmb".
	 *
	 * @var string
	 */
	protected $tmpLinkUrl = '';

	/**
	 * Thumbnails dir path.
	 *
	 * @var string
	 **/
	protected $tmbPath = '';

	/**
	 * Is thumbnails dir writable.
	 *
	 * @var bool
	 **/
	protected $tmbPathWritable = false;

	/**
	 * Thumbnails base URL.
	 *
	 * @var string
	 **/
	protected $tmbURL = '';

	/**
	 * Thumbnails size in px.
	 *
	 * @var int
	 **/
	protected $tmbSize = 48;

	/**
	 * Image manipulation lib name
	 * auto|imagick|gd|convert.
	 *
	 * @var string
	 **/
	protected $imgLib = 'auto';

	/**
	 * Video to Image converter.
	 *
	 * @var array
	 */
	protected $imgConverter = [];

	/**
	 * Library to crypt files name.
	 *
	 * @var string
	 **/
	protected $cryptLib = '';

	/**
	 * Archivers config.
	 *
	 * @var array
	 **/
	protected $archivers = [
		'create' => [],
		'extract' => [],
	];

	/**
	 * Server character encoding.
	 *
	 * @var string or null
	 **/
	protected $encoding = null;

	/**
	 * How many subdirs levels return for tree.
	 *
	 * @var int
	 **/
	protected $treeDeep = 1;

	/**
	 * Errors from last failed action.
	 *
	 * @var array
	 **/
	protected $error = [];

	/**
	 * Today 24:00 timestamp.
	 *
	 * @var int
	 **/
	protected $today = 0;

	/**
	 * Yesterday 24:00 timestamp.
	 *
	 * @var int
	 **/
	protected $yesterday = 0;

	/**
	 * Force make dirctory on extract.
	 *
	 * @var int
	 **/
	protected $extractToNewdir = 'auto';

	/**
	 * Object configuration.
	 *
	 * @var array
	 **/
	protected $options = [
		// Driver ID (Prefix of volume ID), Normally, the value specified for each volume driver is used.
		'driverId' => '',
		// Id (Suffix of volume ID), Normally, the number incremented according to the specified number of volumes is used.
		'id' => '',
		// revision id of root directory that uses for caching control of root stat
		'rootRev' => '',
		// driver type it uses volume root's CSS class name. e.g. 'group' -> Adds 'elfinder-group' to CSS class name.
		'type' => '',
		// root directory path
		'path' => '',
		// Folder hash value on elFinder to be the parent of this volume
		'phash' => '',
		// Folder hash value on elFinder to trash bin of this volume, it require 'copyJoin' to true
		'trashHash' => '',
		// open this path on initial request instead of root path
		'startPath' => '',
		// how many subdirs levels return per request
		'treeDeep' => 1,
		// root url, not set to disable sending URL to client (replacement for old "fileURL" option)
		'URL' => '',
		// directory link url to own manager url with folder hash (`true`, `false` or default `'auto'`: URL is empty then `true` else `false`)
		'dirUrlOwn' => 'auto',
		// directory separator. required by client to show paths correctly
		'separator' => DIRECTORY_SEPARATOR,
		// Server character encoding (default is '': UTF-8)
		'encoding' => '',
		// for convert character encoding (default is '': Not change locale)
		'locale' => '',
		// URL of volume icon (16x16 pixel image file)
		'icon' => '',
		// CSS Class of volume root in tree
		'rootCssClass' => '',
		// Items to disable session caching
		'noSessionCache' => [],
		// enable i18n folder name that convert name to elFinderInstance.messages['folder_'+name]
		'i18nFolderName' => false,
		// Search timeout (sec)
		'searchTimeout' => 30,
		// Search exclusion directory regex pattern (require demiliter e.g. '#/path/to/exclude_directory#i')
		'searchExDirReg' => '',
		// library to crypt/uncrypt files names (not implemented)
		'cryptLib' => '',
		// how to detect files mimetypes. (auto/internal/finfo/mime_content_type)
		'mimeDetect' => 'auto',
		// mime.types file path (for mimeDetect==internal)
		'mimefile' => '',
		// Static extension/MIME of general server side scripts to security issues
		'staticMineMap' => [
			'php:*' => 'text/x-php',
			'php3:*' => 'text/x-php',
			'php4:*' => 'text/x-php',
			'php5:*' => 'text/x-php',
			'phtml:*' => 'text/x-php',
			'cgi:*' => 'text/x-httpd-cgi',
			'pl:*' => 'text/x-perl',
			'asp:*' => 'text/x-asap',
			'aspx:*' => 'text/x-asap',
			'py:*' => 'text/x-python',
			'rb:*' => 'text/x-ruby',
			'jsp:*' => 'text/x-jsp',
		],
		// mime type normalize map : Array '[ext]:[detected mime type]' => '[normalized mime]'
		'mimeMap' => [
			'md:application/x-genesis-rom' => 'text/x-markdown',
			'md:text/plain' => 'text/x-markdown',
			'markdown:text/plain' => 'text/x-markdown',
			'css:text/x-asm' => 'text/css',
			'ico:image/vnd.microsoft.icon' => 'image/x-icon',
			'csv:text/plain' => 'text/csv',
			'm4a:video/mp4' => 'audio/mp4',
			'oga:application/ogg' => 'audio/ogg',
			'ogv:application/ogg' => 'video/ogg',
			'zip:application/x-zip' => 'application/zip',
		],
		// An option to add MimeMap to the `mimeMap` option
		// Array '[ext]:[detected mime type]' => '[normalized mime]'

50% of this comment could be valid code. Did you maybe forget this after debugging?

		'additionalMimeMap' => [],
		// MIME regex of send HTTP header "Content-Disposition: inline" or allow preview in quicklook
		// '.' is allow inline of all of MIME types
		// '$^' is not allow inline of all of MIME types
		'dispInlineRegex' => '^(?:(?:image|video|audio)|(?:text/plain|application/pdf)$)',
		// temporary content URL's base path
		'tmpLinkPath' => '',
		// temporary content URL's base URL
		'tmpLinkUrl' => '',
		// directory for thumbnails
		'tmbPath' => '.tmb',
		// mode to create thumbnails dir
		'tmbPathMode' => 0777,
		// thumbnails dir URL. Set it if store thumbnails outside root directory
		'tmbURL' => '',
		// thumbnails size (px)
		'tmbSize' => 48,
		// thumbnails crop (true - crop, false - scale image to fit thumbnail size)
		'tmbCrop' => true,
		// thumbnails background color (hex #rrggbb or 'transparent')
		'tmbBgColor' => 'transparent',
		// image rotate fallback background color (hex #rrggbb)
		'bgColorFb' => '#ffffff',
		// image manipulations library
		'imgLib' => 'auto',
		// Fallback self image to thumbnail (nothing imgLib)
		'tmbFbSelf' => true,
		// Video to Image converters ['TYPE or MIME' => ['func' => function($file){ /* Converts $file to Image */ return true; }, 'maxlen' => (int)TransferLength]]

39% of this comment could be valid code. Did you maybe forget this after debugging?

		'imgConverter' => [],
		// Max length of transfer to image converter
		'tmbVideoConvLen' => 10000000,
		// Captre point seccond
		'tmbVideoConvSec' => 6,
		// Resource path of fallback icon images defailt: php/resouces
		'resourcePath' => '',
		// Jpeg image saveing quality
		'jpgQuality' => 100,
		// on paste file -  if true - old file will be replaced with new one, if false new file get name - original_name-number.ext
		'copyOverwrite' => true,
		// if true - join new and old directories content on paste
		'copyJoin' => true,
		// on upload -  if true - old file will be replaced with new one, if false new file get name - original_name-number.ext
		'uploadOverwrite' => true,
		// mimetypes allowed to upload
		'uploadAllow' => [],
		// mimetypes not allowed to upload
		'uploadDeny' => [],
		// order to proccess uploadAllow and uploadDeny options
		'uploadOrder' => ['deny', 'allow'],
		// maximum upload file size. NOTE - this is size for every uploaded files
		'uploadMaxSize' => 0,
		// maximum number of chunked upload connection. `-1` to disable chunked upload
		'uploadMaxConn' => 3,
		// maximum get file size. NOTE - Maximum value is 50% of PHP memory_limit
		'getMaxSize' => 0,
		// files dates format
		'dateFormat' => 'j M Y H:i',
		// files time format
		'timeFormat' => 'H:i',
		// if true - every folder will be check for children folders, -1 - every folder will be check asynchronously, false -  all folders will be marked as having subfolders
		'checkSubfolders' => true, // true, false or -1

50% of this comment could be valid code. Did you maybe forget this after debugging?

		// allow to copy from this volume to other ones?
		'copyFrom' => true,
		// allow to copy from other volumes to this one?
		'copyTo' => true,
		// cmd duplicate suffix format e.g. '_%s_' to without spaces
		'duplicateSuffix' => ' %s ',
		// unique name numbar format e.g. '(%d)' to (1), (2)...
		'uniqueNumFormat' => '%d',
		// list of commands disabled on this root
		'disabled' => [],
		// enable file owner, group & mode info, `false` to inactivate "chmod" command.
		'statOwner' => false,
		// allow exec chmod of read-only files
		'allowChmodReadOnly' => false,
		// regexp or function name to validate new file name
		'acceptedName' => '/^[^\.].*/', // Notice: overwritten it in some volume drivers contractor
		// regexp or function name to validate new directory name
		'acceptedDirname' => '', // used `acceptedName` if empty value
			// function/class method to control files permissions
		'accessControl' => null,
		// some data required by access control
		'accessControlData' => null,
		// default permissions.
		'defaults' => [
			'read' => true,
			'write' => true,
			'locked' => false,
			'hidden' => false,
		],
		// files attributes
		'attributes' => [],
		// max allowed archive files size (0 - no limit)
		'maxArcFilesSize' => 0,
		// Allowed archive's mimetypes to create. Leave empty for all available types.
		'archiveMimes' => [],
		// Manual config for archivers. See example below. Leave empty for auto detect
		'archivers' => [],
		// plugin settings
		'plugin' => [],
		// Is support parent directory time stamp update on add|remove|rename item
		// Default `null` is auto detection that is LocalFileSystem, FTP or Dropbox are `true`
		'syncChkAsTs' => null,
		// Long pooling sync checker function for syncChkAsTs is true
		// Calls with args (TARGET DIRCTORY PATH, STAND-BY(sec), OLD TIMESTAMP, VOLUME DRIVER INSTANCE, ELFINDER INSTANCE)
		// This function must return the following values. Changed: New Timestamp or Same: Old Timestamp or Error: false
		// Default `null` is try use elFinderVolumeLocalFileSystem::localFileSystemInotify() on LocalFileSystem driver
		// another driver use elFinder stat() checker
		'syncCheckFunc' => null,
		// Long polling sync stand-by time (sec)
		'plStandby' => 30,
		// Sleep time (sec) for elFinder stat() checker (syncChkAsTs is true)
		'tsPlSleep' => 10,
		// Sleep time (sec) for elFinder ls() checker (syncChkAsTs is false)
		'lsPlSleep' => 30,
		// Client side sync interval minimum (ms)
		// Default `null` is auto set to ('tsPlSleep' or 'lsPlSleep') * 1000
		// `0` to disable auto sync
		'syncMinMs' => null,
		// required to fix bug on macos
		// However, we recommend to use the Normalizer plugin instead this option
		'utf8fix' => false,
		 //                           й                 ё              Й               Ё              Ø         Å
		'utf8patterns' => ["\u0438\u0306", "\u0435\u0308", "\u0418\u0306", "\u0415\u0308", "\u00d8A", "\u030a"],
		'utf8replace' => ["\u0439",        "\u0451",       "\u0419",       "\u0401",       "\u00d8", "\u00c5"],
	];

	/**
	 * Defaults permissions.
	 *
	 * @var array
	 **/
	protected $defaults = [
		'read' => true,
		'write' => true,
		'locked' => false,
		'hidden' => false,
	];

	/**
	 * Access control function/class.
	 *
	 * @var mixed
	 **/
	protected $attributes = [];

	/**
	 * Access control function/class.
	 *
	 * @var mixed
	 **/
	protected $access = null;

	/**
	 * Mime types allowed to upload.
	 *
	 * @var array
	 **/
	protected $uploadAllow = [];

	/**
	 * Mime types denied to upload.
	 *
	 * @var array
	 **/
	protected $uploadDeny = [];

	/**
	 * Order to validate uploadAllow and uploadDeny.
	 *
	 * @var array
	 **/
	protected $uploadOrder = [];

	/**
	 * Maximum allowed upload file size.
	 * Set as number or string with unit - "10M", "500K", "1G".
	 *
	 * @var int|string
	 **/
	protected $uploadMaxSize = 0;

	/**
	 * Run time setting of overwrite items on upload.
	 *
	 * @var string
	 */
	protected $uploadOverwrite = true;

	/**
	 * Maximum allowed get file size.
	 * Set as number or string with unit - "10M", "500K", "1G".
	 *
	 * @var int|string
	 **/
	protected $getMaxSize = -1;

	/**
	 * Mimetype detect method.
	 *
	 * @var string
	 **/
	protected $mimeDetect = 'auto';

	/**
	 * Finfo object for mimeDetect == 'finfo'.
	 *
	 * @var object
	 **/
	protected $finfo = null;

	/**
	 * List of disabled client's commands.
	 *
	 * @var array
	 **/
	protected $disabled = [];

	/**
	 * default extensions/mimetypes for mimeDetect == 'internal'.
	 *
	 * @var array
	 **/
	protected static $mimetypes = [
		// applications
		'ai' => 'application/postscript',
		'eps' => 'application/postscript',
		'exe' => 'application/x-executable',
		'doc' => 'application/msword',
		'dot' => 'application/msword',
		'xls' => 'application/vnd.ms-excel',
		'xlt' => 'application/vnd.ms-excel',
		'xla' => 'application/vnd.ms-excel',
		'ppt' => 'application/vnd.ms-powerpoint',
		'pps' => 'application/vnd.ms-powerpoint',
		'pdf' => 'application/pdf',
		'xml' => 'application/xml',
		'swf' => 'application/x-shockwave-flash',
		'torrent' => 'application/x-bittorrent',
		'jar' => 'application/x-jar',
		// open office (finfo detect as application/zip)
		'odt' => 'application/vnd.oasis.opendocument.text',
		'ott' => 'application/vnd.oasis.opendocument.text-template',
		'oth' => 'application/vnd.oasis.opendocument.text-web',
		'odm' => 'application/vnd.oasis.opendocument.text-master',
		'odg' => 'application/vnd.oasis.opendocument.graphics',
		'otg' => 'application/vnd.oasis.opendocument.graphics-template',
		'odp' => 'application/vnd.oasis.opendocument.presentation',
		'otp' => 'application/vnd.oasis.opendocument.presentation-template',
		'ods' => 'application/vnd.oasis.opendocument.spreadsheet',
		'ots' => 'application/vnd.oasis.opendocument.spreadsheet-template',
		'odc' => 'application/vnd.oasis.opendocument.chart',
		'odf' => 'application/vnd.oasis.opendocument.formula',
		'odb' => 'application/vnd.oasis.opendocument.database',
		'odi' => 'application/vnd.oasis.opendocument.image',
		'oxt' => 'application/vnd.openofficeorg.extension',
		// MS office 2007 (finfo detect as application/zip)
		'docx' => 'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
		'docm' => 'application/vnd.ms-word.document.macroEnabled.12',
		'dotx' => 'application/vnd.openxmlformats-officedocument.wordprocessingml.template',
		'dotm' => 'application/vnd.ms-word.template.macroEnabled.12',
		'xlsx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
		'xlsm' => 'application/vnd.ms-excel.sheet.macroEnabled.12',
		'xltx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.template',
		'xltm' => 'application/vnd.ms-excel.template.macroEnabled.12',
		'xlsb' => 'application/vnd.ms-excel.sheet.binary.macroEnabled.12',
		'xlam' => 'application/vnd.ms-excel.addin.macroEnabled.12',
		'pptx' => 'application/vnd.openxmlformats-officedocument.presentationml.presentation',
		'pptm' => 'application/vnd.ms-powerpoint.presentation.macroEnabled.12',
		'ppsx' => 'application/vnd.openxmlformats-officedocument.presentationml.slideshow',
		'ppsm' => 'application/vnd.ms-powerpoint.slideshow.macroEnabled.12',
		'potx' => 'application/vnd.openxmlformats-officedocument.presentationml.template',
		'potm' => 'application/vnd.ms-powerpoint.template.macroEnabled.12',
		'ppam' => 'application/vnd.ms-powerpoint.addin.macroEnabled.12',
		'sldx' => 'application/vnd.openxmlformats-officedocument.presentationml.slide',
		'sldm' => 'application/vnd.ms-powerpoint.slide.macroEnabled.12',
		// archives
		'gz' => 'application/x-gzip',
		'tgz' => 'application/x-gzip',
		'bz' => 'application/x-bzip2',
		'bz2' => 'application/x-bzip2',
		'tbz' => 'application/x-bzip2',
		'xz' => 'application/x-xz',
		'zip' => 'application/zip',
		'rar' => 'application/x-rar',
		'tar' => 'application/x-tar',
		'7z' => 'application/x-7z-compressed',
		// texts
		'txt' => 'text/plain',
		'php' => 'text/x-php',
		'html' => 'text/html',
		'htm' => 'text/html',
		'js' => 'text/javascript',
		'css' => 'text/css',
		'rtf' => 'text/rtf',
		'rtfd' => 'text/rtfd',
		'py' => 'text/x-python',
		'java' => 'text/x-java-source',
		'rb' => 'text/x-ruby',
		'sh' => 'text/x-shellscript',
		'pl' => 'text/x-perl',
		//'xml'   => 'text/xml',

67% of this comment could be valid code. Did you maybe forget this after debugging?

		'sql' => 'text/x-sql',
		'c' => 'text/x-csrc',
		'h' => 'text/x-chdr',
		'cpp' => 'text/x-c++src',
		'hh' => 'text/x-c++hdr',
		'log' => 'text/plain',
		'csv' => 'text/csv',
		'md' => 'text/x-markdown',
		'markdown' => 'text/x-markdown',
		// images
		'bmp' => 'image/x-ms-bmp',
		'jpg' => 'image/jpeg',
		'jpeg' => 'image/jpeg',
		'gif' => 'image/gif',
		'png' => 'image/png',
		'tif' => 'image/tiff',
		'tiff' => 'image/tiff',
		'tga' => 'image/x-targa',
		'psd' => 'image/vnd.adobe.photoshop',
		//'ai'    => 'image/vnd.adobe.photoshop',

67% of this comment could be valid code. Did you maybe forget this after debugging?

		'xbm' => 'image/xbm',
		'pxm' => 'image/pxm',
		//audio
		'mp3' => 'audio/mpeg',
		'mid' => 'audio/midi',
		'ogg' => 'audio/ogg',
		'oga' => 'audio/ogg',
		'm4a' => 'audio/mp4',
		'wav' => 'audio/wav',
		'wma' => 'audio/x-ms-wma',
		// video
		'avi' => 'video/x-msvideo',
		'dv' => 'video/x-dv',
		'mp4' => 'video/mp4',
		'mpeg' => 'video/mpeg',
		'mpg' => 'video/mpeg',
		'mov' => 'video/quicktime',
		'wm' => 'video/x-ms-wmv',
		'flv' => 'video/x-flv',
		'mkv' => 'video/x-matroska',
		'webm' => 'video/webm',
		'ogv' => 'video/ogg',
		'ogm' => 'video/ogg',
		];

	/**
	 * Directory separator - required by client.
	 *
	 * @var string
	 **/
	protected $separator = DIRECTORY_SEPARATOR;

	/**
	 * System Root path (Unix like: '/', Windows: '\', 'C:\' or 'D:\'...).
	 *
	 * @var string
	 **/
	protected $systemRoot = DIRECTORY_SEPARATOR;

	/**
	 * Mimetypes allowed to display.
	 *
	 * @var array
	 **/
	protected $onlyMimes = [];

	/**
	 * Store files moved or overwrited files info.
	 *
	 * @var array
	 **/
	protected $removed = [];

	/**
	 * Store files added files info.
	 *
	 * @var array
	 **/
	protected $added = [];

	/**
	 * Cache storage.
	 *
	 * @var array
	 **/
	protected $cache = [];

	/**
	 * Cache by folders.
	 *
	 * @var array
	 **/
	protected $dirsCache = [];

	/**
	 * You should use `$this->sessionCache['subdirs']` instead.
	 *
	 * @var array
	 * @deprecated
	 */
	protected $subdirsCache = [];

	/**
	 * This volume session cache.
	 *
	 * @var array
	 */
	protected $sessionCache;

	/**
	 * Session caching item list.
	 *
	 * @var array
	 */
	protected $sessionCaching = ['rootstat' => true, 'subdirs' => true];

	/**
	 * elFinder session wrapper object.
	 *
	 * @var elFinderSessionInterface
	 */
	protected $session;

	/**
	 * Search start time.
	 *
	 * @var int
	 */
	protected $searchStart;

	/**
	 * Current query word on doSearch.
	 *
	 * @var string
	 **/
	protected $doSearchCurrentQuery = [];

	/**
	 * Is root modified (for clear root stat cache).
	 *
	 * @var bool
	 */
	protected $rootModified = false;

	/**
	 * Is disable of command `url`.
	 *
	 * @var string
	 */
	protected $disabledGetUrl = false;

	/**
	 * Accepted filename validator.
	 *
	 * @var string | callable
	 */
	protected $nameValidator;

	/**
	 * Accepted dirname validator.
	 *
	 * @var string | callable
	 */
	protected $dirnameValidator;

	/**
	 * Flag - mimetypes from externail file was loaded.
	 *
	 * @var bool
	 **/
	private static $mimetypesLoaded = false;

	/*********************************************************************/
	/*                              PUBLIC API                           */
	/*********************************************************************/

	/**
	 * Return driver id. Used as a part of volume id.
	 *
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	public function driverId()
	{
	    return $this->driverId;
	}

	/**
	 * Return volume id.
	 *
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	public function id()
	{
	    return $this->id;
	}

	/**
	 * Assign elFinder session wrapper object.
	 *
	 * @param  $session  elFinderSessionInterface
	 */
	public function setSession($session)
	{
	    $this->session = $session;
	}

	/**
	 * Save session cache data
	 * Calls this function before umount this volume on elFinder::exec().
	 *
	 * @return void
	 */
	public function saveSessionCache()
	{
	    $this->session->set($this->id, $this->sessionCache);
	}

	/**
	 * Return debug info for client.
	 *
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function debug()
	{
	    return [
			'id' => $this->id(),
			'name' => strtolower(substr(get_class($this), strlen('elfinderdriver'))),
			'mimeDetect' => $this->mimeDetect,
			'imgLib' => $this->imgLib,
		];
	}

	/**
	 * chmod a file or folder.
	 *
	 * @param  string   $hash    file or folder hash to chmod
	 * @param  string   $mode    octal string representing new permissions
	 * @return array|false
	 * @author David Bartle
	 **/
	public function chmod($hash, $mode)
	{
	    if ($this->commandDisabled('chmod')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (! ($file = $this->file($hash))) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    if (! $this->options['allowChmodReadOnly']) {

This code seems to be duplicated across your project.

	        if (! $this->attr($this->decode($hash), 'write', null, ($file['mime'] === 'directory'))) {
	            return $this->setError(elFinder::ERROR_PERM_DENIED, $file['name']);
	        }
	    }

	    $path = $this->decode($hash);
	    $write = $file['write'];

	    if ($this->convEncOut(! $this->_chmod($this->convEncIn($path), $mode))) {

This code seems to be duplicated across your project.

	        return $this->setError(elFinder::ERROR_PERM_DENIED, $file['name']);
	    }

	    $this->clearcache();
	    if ($path == $this->root) {
	        $this->rootModified = true;
	    }

	    if ($file = $this->stat($path)) {
	        $files = [$file];
	        if ($file['mime'] === 'directory' && $write !== $file['write']) {
	            foreach ($this->getScandir($path) as $stat) {
	                if ($this->mimeAccepted($stat['mime'])) {
	                    $files[] = $stat;
	                }
	            }
	        }

	        return $files;
	    } else {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }
	}

	/**
	 * stat a file or folder for elFinder cmd exec.
	 *
	 * @param  string   $hash    file or folder hash to chmod
	 * @return array
	 * @author Naoki Sawada
	 **/
	public function fstat($hash)
	{
	    $path = $this->decode($hash);

	    return $this->stat($path);
	}

	/**
	 * Clear PHP stat cache & all of inner stat caches.
	 */
	public function clearstatcache()
	{
	    clearstatcache();
	    $this->clearcache();
	}

	/**
	 * Clear inner stat caches for target hash.
	 *
	 * @param string $hash
	 */
	public function clearcaches($hash = null)
	{
	    if ($hash === null) {
	        $this->clearcache();
	    } else {
	        $path = $this->decode($hash);
	        unset($this->cache[$path], $this->dirsCache[$path]);
	    }
	}

	/**
	 * "Mount" volume.
	 * Return true if volume available for read or write,
	 * false - otherwise.
	 *
	 * @param array $opts
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 * @author Alexey Sukhotin
	 */
	public function mount(array $opts)

mount uses the super-global variable $_SERVER which is generally not recommended.

mount uses the super-global variable $_POST which is generally not recommended.

mount uses the super-global variable $_GET which is generally not recommended.

	{
	    $this->options = array_merge($this->options, $opts);

	    if (! isset($this->options['path']) || $this->options['path'] === '') {

This code seems to be duplicated across your project.

	        return $this->setError('Path undefined.');
	    }

	    if (! $this->session) {
	        return $this->setError('Session wrapper dose not set. Need to `$volume->setSession(elFinderSessionInterface);` before mount.');
	    }
	    if (! ($this->session instanceof elFinderSessionInterface)) {
	        return $this->setError('Session wrapper instance must be "elFinderSessionInterface".');
	    }

		// set driverId
		if (! empty($this->options['driverId'])) {
		    $this->driverId = $this->options['driverId'];
		}

	    $this->id = $this->driverId.(! empty($this->options['id']) ? $this->options['id'] : elFinder::$volumesCnt++).'_';
	    $this->root = $this->normpathCE($this->options['path']);
	    $this->separator = isset($this->options['separator']) ? $this->options['separator'] : DIRECTORY_SEPARATOR;
	    $this->systemRoot = isset($this->options['systemRoot']) ? $this->options['systemRoot'] : $this->separator;

		// set server encoding
		if (! empty($this->options['encoding']) && strtoupper($this->options['encoding']) !== 'UTF-8') {
		    $this->encoding = $this->options['encoding'];
		} else {
		    $this->encoding = null;
		}

		// set ARGS
		$this->ARGS = $_SERVER['REQUEST_METHOD'] === 'POST' ? $_POST : $_GET;

	    $argInit = ! empty($this->ARGS['init']);

		// session cache
		if ($argInit) {
		    $this->session->set($this->id, []);
		}
	    $this->sessionCache = $this->session->get($this->id, []);

It seems like $this->session->get($this->id, array()) of type * is incompatible with the declared type array of property $sessionCache.

		// default file attribute
		$this->defaults = [
			'read' => isset($this->options['defaults']['read']) ? (bool) $this->options['defaults']['read'] : true,
			'write' => isset($this->options['defaults']['write']) ? (bool) $this->options['defaults']['write'] : true,
			'locked' => isset($this->options['defaults']['locked']) ? (bool) $this->options['defaults']['locked'] : false,
			'hidden' => isset($this->options['defaults']['hidden']) ? (bool) $this->options['defaults']['hidden'] : false,
		];

		// root attributes
		$this->attributes[] = [
			'pattern' => '~^'.preg_quote($this->separator).'$~',
			'locked' => true,
			'hidden' => false,
		];
		// set files attributes
		if (! empty($this->options['attributes']) && is_array($this->options['attributes'])) {

This code seems to be duplicated across your project.

		    foreach ($this->options['attributes'] as $a) {
		        // attributes must contain pattern and at least one rule
				if (! empty($a['pattern']) || count($a) > 1) {
				    $this->attributes[] = $a;
				}
		    }
		}

	    if (! empty($this->options['accessControl']) && is_callable($this->options['accessControl'])) {
	        $this->access = $this->options['accessControl'];
	    }

	    $this->today = mktime(0, 0, 0, date('m'), date('d'), date('Y'));
	    $this->yesterday = $this->today - 86400;

	    if (! $this->init()) {
	        return false;
	    }

		// check some options is arrays
		$this->uploadAllow = isset($this->options['uploadAllow']) && is_array($this->options['uploadAllow'])
			? $this->options['uploadAllow']
			: [];

	    $this->uploadDeny = isset($this->options['uploadDeny']) && is_array($this->options['uploadDeny'])
			? $this->options['uploadDeny']
			: [];

	    $this->options['uiCmdMap'] = (isset($this->options['uiCmdMap']) && is_array($this->options['uiCmdMap']))
			? $this->options['uiCmdMap']
			: [];

	    if (is_string($this->options['uploadOrder'])) { // telephat_mode on, compatibility with 1.x
			$parts = explode(',', isset($this->options['uploadOrder']) ? $this->options['uploadOrder'] : 'deny,allow');
	        $this->uploadOrder = [trim($parts[0]), trim($parts[1])];
	    } else { // telephat_mode off
			$this->uploadOrder = ! empty($this->options['uploadOrder']) ? $this->options['uploadOrder'] : ['deny', 'allow'];
	    }

	    if (! empty($this->options['uploadMaxSize'])) {
	        $this->uploadMaxSize = elFinder::getIniBytes('', $this->options['uploadMaxSize']);

It seems like \elFinder::getIniBytes('', $this->options['uploadMaxSize']) can also be of type double. However, the property $uploadMaxSize is declared as type integer|string. Maybe add an additional type check?

	    }
		// Set maximum to PHP_INT_MAX
		if (! defined('PHP_INT_MAX')) {
		    define('PHP_INT_MAX', 2147483647);
		}
	    if ($this->uploadMaxSize < 1 || $this->uploadMaxSize > PHP_INT_MAX) {
	        $this->uploadMaxSize = PHP_INT_MAX;
	    }

		// Set to get maximum size to 50% of memory_limit
		$memLimit = elFinder::getIniBytes('memory_limit') / 2;
	    if ($memLimit > 0) {
	        $this->getMaxSize = empty($this->options['getMaxSize']) ? $memLimit : min($memLimit, elFinder::getIniBytes('', $this->options['getMaxSize']));

It seems like empty($this->options['getMaxSize']) ? $memLimit : min($memLimit, \elFinder::getIniBytes('', $this->options['getMaxSize'])) can also be of type double. However, the property $getMaxSize is declared as type integer|string. Maybe add an additional type check?

	    } else {
	        $this->getMaxSize = -1;
	    }

	    $this->disabled = isset($this->options['disabled']) && is_array($this->options['disabled'])
			? array_values(array_diff($this->options['disabled'], ['open'])) // 'open' is required
			: [];

	    $this->cryptLib = $this->options['cryptLib'];
	    $this->mimeDetect = $this->options['mimeDetect'];

		// find available mimetype detect method
		$type = strtolower($this->options['mimeDetect']);
	    $type = preg_match('/^(finfo|mime_content_type|internal|auto)$/i', $type) ? $type : 'auto';
	    $regexp = '/text\/x\-(php|c\+\+)/';

	    if (($type == 'finfo' || $type == 'auto')
		&& class_exists('finfo', false)) {
	        $tmpFileInfo = explode(';', finfo_file(finfo_open(FILEINFO_MIME), __FILE__));
	    } else {
	        $tmpFileInfo = false;
	    }

	    $type = 'internal';
	    if ($tmpFileInfo && preg_match($regexp, array_shift($tmpFileInfo))) {
	        $type = 'finfo';
	        $this->finfo = finfo_open(FILEINFO_MIME);
	    } elseif (($type == 'mime_content_type' || $type == 'auto') && function_exists('mime_content_type')) {
	        $_mimetypes = explode(';', mime_content_type(__FILE__));
	        if (preg_match($regexp, array_shift($_mimetypes))) {
	            $type = 'mime_content_type';
	        }
	    }
	    $this->mimeDetect = $type;

		// load mimes from external file for mimeDetect == 'internal'
		// based on Alexey Sukhotin idea and patch: http://elrte.org/redmine/issues/163
		// file must be in file directory or in parent one
		if ($this->mimeDetect == 'internal' && ! self::$mimetypesLoaded) {
		    self::$mimetypesLoaded = true;
		    $this->mimeDetect = 'internal';
		    $file = false;
		    if (! empty($this->options['mimefile']) && file_exists($this->options['mimefile'])) {
		        $file = $this->options['mimefile'];
		    } elseif (elFinder::$defaultMimefile && file_exists(elFinder::$defaultMimefile)) {
		        $file = elFinder::$defaultMimefile;
		    } elseif (file_exists(dirname(__FILE__).DIRECTORY_SEPARATOR.'mime.types')) {
		        $file = dirname(__FILE__).DIRECTORY_SEPARATOR.'mime.types';
		    } elseif (file_exists(dirname(dirname(__FILE__)).DIRECTORY_SEPARATOR.'mime.types')) {
		        $file = dirname(dirname(__FILE__)).DIRECTORY_SEPARATOR.'mime.types';
		    }

		    if ($file && file_exists($file)) {

This code seems to be duplicated across your project.

		        $mimecf = file($file);

		        foreach ($mimecf as $line_num => $line) {
		            if (! preg_match('/^\s*#/', $line)) {
		                $mime = preg_split('/\s+/', $line, -1, PREG_SPLIT_NO_EMPTY);
		                for ($i = 1, $size = count($mime); $i < $size; $i++) {
		                    if (! isset(self::$mimetypes[$mime[$i]])) {
		                        self::$mimetypes[$mime[$i]] = $mime[0];
		                    }
		                }
		            }
		        }
		    }
		}

	    $this->rootName = empty($this->options['alias']) ? $this->basenameCE($this->root) : $this->options['alias'];

		// This get's triggered if $this->root == '/' and alias is empty.
		// Maybe modify _basename instead?
		if ($this->rootName === '') {
		    $this->rootName = $this->separator;
		}

	    $root = $this->stat($this->root);

	    if (! $root) {

The expression $root of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	        return $this->setError('Root folder does not exist.');
	    }
	    if (! $root['read'] && ! $root['write']) {
	        return $this->setError('Root folder has not read and write permissions.');
	    }

	    if ($root['read']) {
	        if ($argInit) {
	            // check startPath - path to open by default instead of root
				$startPath = $this->options['startPath'] ? $this->normpathCE($this->options['startPath']) : '';
	            if ($startPath) {
	                $start = $this->stat($startPath);
	                if (! empty($start)
					&& $start['mime'] == 'directory'
					&& $start['read']
					&& empty($start['hidden'])
					&& $this->inpathCE($startPath, $this->root)) {
	                    $this->startPath = $startPath;
	                    if (substr($this->startPath, -1, 1) == $this->options['separator']) {
	                        $this->startPath = substr($this->startPath, 0, -1);
	                    }
	                }
	            }
	        }
	    } else {
	        $this->options['URL'] = '';
	        $this->options['tmbURL'] = '';
	        $this->options['tmbPath'] = '';
			// read only volume
			array_unshift($this->attributes, [
				'pattern' => '/.*/',
				'read' => false,
			]);
	    }
	    $this->treeDeep = $this->options['treeDeep'] > 0 ? (int) $this->options['treeDeep'] : 1;
	    $this->tmbSize = $this->options['tmbSize'] > 0 ? (int) $this->options['tmbSize'] : 48;
	    $this->URL = $this->options['URL'];
	    if ($this->URL && preg_match('|[^/?&=]$|', $this->URL)) {
	        $this->URL .= '/';
	    }
	    if (strtolower($this->options['dirUrlOwn']) === 'auto') {
	        $this->options['dirUrlOwn'] = $this->URL ? false : true;
	    } else {
	        $this->options['dirUrlOwn'] = (bool) $this->options['dirUrlOwn'];
	    }

	    $this->tmbURL = ! empty($this->options['tmbURL']) ? $this->options['tmbURL'] : '';
	    if ($this->tmbURL && $this->tmbURL !== 'self' && preg_match('|[^/?&=]$|', $this->tmbURL)) {
	        $this->tmbURL .= '/';
	    }

	    $this->nameValidator = ! empty($this->options['acceptedName']) && (is_string($this->options['acceptedName']) || is_callable($this->options['acceptedName']))

It seems like !empty($this->options['acceptedName']) && (is_string($this->options['acceptedName']) || is_callable($this->options['acceptedName'])) ? $this->options['acceptedName'] : '' of type callable is incompatible with the declared type string of property $nameValidator.

			? $this->options['acceptedName']
			: '';

	    $this->dirnameValidator = ! empty($this->options['acceptedDirname']) && (is_callable($this->options['acceptedDirname']) || (is_string($this->options['acceptedDirname']) && preg_match($this->options['acceptedDirname'], '') !== false))

It seems like !empty($this->options['acceptedDirname']) && (is_callable($this->options['acceptedDirname']) || is_string($this->options['acceptedDirname']) && preg_match($this->options['acceptedDirname'], '') !== false) ? $this->options['acceptedDirname'] : $this->nameValidator of type callable is incompatible with the declared type string of property $dirnameValidator.

			? $this->options['acceptedDirname']
			: $this->nameValidator;

	    $this->_checkArchivers();
		// manual control archive types to create
		if (! empty($this->options['archiveMimes']) && is_array($this->options['archiveMimes'])) {

This code seems to be duplicated across your project.

		    foreach ($this->archivers['create'] as $mime => $v) {
		        if (! in_array($mime, $this->options['archiveMimes'])) {
		            unset($this->archivers['create'][$mime]);
		        }
		    }
		}

		// manualy add archivers
		if (! empty($this->options['archivers']['create']) && is_array($this->options['archivers']['create'])) {

This code seems to be duplicated across your project.

		    foreach ($this->options['archivers']['create'] as $mime => $conf) {
		        if (strpos($mime, 'application/') === 0
				&& ! empty($conf['cmd'])
				&& isset($conf['argc'])
				&& ! empty($conf['ext'])
				&& ! isset($this->archivers['create'][$mime])) {
		            $this->archivers['create'][$mime] = $conf;
		        }
		    }
		}

	    if (! empty($this->options['archivers']['extract']) && is_array($this->options['archivers']['extract'])) {

This code seems to be duplicated across your project.

	        foreach ($this->options['archivers']['extract'] as $mime => $conf) {
	            if (strpos($mime, 'application/') === 0
				&& ! empty($conf['cmd'])
				&& isset($conf['argc'])
				&& ! empty($conf['ext'])
				&& ! isset($this->archivers['extract'][$mime])) {
	                $this->archivers['extract'][$mime] = $conf;
	            }
	        }
	    }

	    if (! empty($this->options['noSessionCache']) && is_array($this->options['noSessionCache'])) {
	        foreach ($this->options['noSessionCache'] as $_key) {
	            $this->sessionCaching[$_key] = false;
	            unset($this->sessionCache[$_key]);
	        }
	    }
	    if ($this->sessionCaching['subdirs']) {
	        if (! isset($this->sessionCache['subdirs'])) {
	            $this->sessionCache['subdirs'] = [];
	        }
	    }

	    $this->configure();

		// Normarize disabled (array_merge`for type array of JSON)
		$this->disabled = array_values(array_unique($this->disabled));

		// fix sync interval
		if ($this->options['syncMinMs'] !== 0) {

This code seems to be duplicated across your project.

		    $this->options['syncMinMs'] = max($this->options[$this->options['syncChkAsTs'] ? 'tsPlSleep' : 'lsPlSleep'] * 1000, intval($this->options['syncMinMs']));
		}

		// ` copyJoin` is required for the trash function
		if ($this->options['trashHash'] && empty($this->options['copyJoin'])) {
		    $this->options['trashHash'] = '';
		}

		// set tmpLinkPath
		if (elFinder::$tmpLinkPath && ! $this->options['tmpLinkPath']) {
		    $this->options['tmpLinkPath'] = elFinder::$tmpLinkPath;
		}
	    if ($this->options['tmpLinkPath'] && is_writable($this->options['tmpLinkPath'])) {
	        $this->tmpLinkPath = realpath($this->options['tmpLinkPath']);
	    } elseif (! $this->options['URL'] && is_writable('../files/.tmb')) {
	        $this->tmpLinkPath = realpath('../files/.tmb');
	        $this->options['tmpLinkUrl'] = '';
	        if (! elFinder::$tmpLinkPath) {
	            elFinder::$tmpLinkPath = $this->tmpLinkPath;
	            elFinder::$tmpLinkUrl = '';
	        }
	    }

		// set tmpLinkUrl
		if (elFinder::$tmpLinkUrl && ! $this->options['tmpLinkUrl']) {
		    $this->options['tmpLinkUrl'] = elFinder::$tmpLinkUrl;
		}
	    if ($this->options['tmpLinkUrl']) {
	        $this->tmpLinkUrl = $this->options['tmpLinkUrl'];
	    }
	    if ($this->tmpLinkPath && ! $this->tmpLinkUrl) {
	        $cur = realpath('./');
	        $i = 0;
	        while ($cur !== $this->systemRoot && strpos($this->tmpLinkPath, $cur) !== 0) {
	            $i++;
	            $cur = dirname($cur);
	        }
	        list($req) = explode('?', $_SERVER['REQUEST_URI']);
	        $reqs = explode('/', dirname($req));
	        $uri = implode('/', array_slice($reqs, 0, count($reqs) - 1)).substr($this->tmpLinkPath, strlen($cur));
	        $https = (isset($_SERVER['HTTPS']) && strtolower($_SERVER['HTTPS']) !== 'off');
	        $this->tmpLinkUrl = ($https ? 'https://' : 'http://')
				.$_SERVER['SERVER_NAME'] // host
				.(((! $https && $_SERVER['SERVER_PORT'] == 80) || ($https && $_SERVER['SERVER_PORT'] == 443)) ? '' : (':'.$_SERVER['SERVER_PORT']))  // port
				.$uri;
	        if (! elFinder::$tmpLinkUrl) {
	            elFinder::$tmpLinkUrl = $this->tmpLinkUrl;
	        }
	    }

		// remove last '/'
		if ($this->tmpLinkPath) {
		    $this->tmpLinkPath = rtrim($this->tmpLinkPath, '/');
		}
	    if ($this->tmpLinkUrl) {
	        $this->tmpLinkUrl = rtrim($this->tmpLinkUrl, '/');
	    }

		// to update options cache
		$this->updateCache($this->root, $root);

	    return $this->mounted = true;
	}

	/**
	 * Some "unmount" stuffs - may be required by virtual fs.
	 *
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	public function umount()
	{
	}

	/**
	 * Remove session cache of this volume.
	 */
	public function clearSessionCache()
	{
	    $this->sessionCache = [];
	}

	/**
	 * Return error message from last failed action.
	 *
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function error()
	{
	    return $this->error;
	}

	/**
	 * Return is uploadable that given file name.
	 *
	 * @param  string  $name  file name
	 * @param  bool    $allowUnknown
	 * @return bool
	 * @author Naoki Sawada
	 **/
	public function isUploadableByName($name, $allowUnknown = false)
	{
	    $mimeByName = $this->mimetype($name, true);

	    return ($allowUnknown && $mimeByName === 'unknown') || $this->allowPutMime($mimeByName);
	}

	/**
	 * Return Extention/MIME Table (elFinderVolumeDriver::$mimetypes).
	 *
	 * @return array
	 * @author Naoki Sawada
	 */
	public function getMimeTable()
	{
	    // load mime.types
		! self::$mimetypesLoaded && self::mimetypeInternalDetect();

	    return self::$mimetypes;
	}

	/**
	 * Return file extention detected by MIME type.
	 *
	 * @param  string  $mime    MIME type
	 * @param  string  $suffix  Additional suffix
	 * @return string
	 * @author Naoki Sawada
	 */
	public function getExtentionByMime($mime, $suffix = '')
	{
	    static $extTable = null;

	    if (is_null($extTable)) {
	        $extTable = array_flip(array_unique($this->getMimeTable()));
	        foreach (array_keys($this->options['mimeMap']) as $pair) {
	            list($ext, $_mime) = explode(':', $pair);
	            if ($_mime !== '*' && ! isset($extTable[$_mime])) {
	                $extTable[$_mime] = $ext;
	            }
	        }
	    }

	    if ($mime && isset($extTable[$mime])) {
	        return $suffix ? ($extTable[$mime].$suffix) : $extTable[$mime];
	    }

	    return '';
	}

	/**
	 * Set mimetypes allowed to display to client.
	 *
	 * @param  array  $mimes
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	public function setMimesFilter($mimes)
	{
	    if (is_array($mimes)) {
	        $this->onlyMimes = $mimes;
	    }
	}

	/**
	 * Return root folder hash.
	 *
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	public function root()
	{
	    return $this->encode($this->root);
	}

	/**
	 * Return target path hash.
	 *
	 * @param  string $path
	 * @param  string $name
	 * @author Naoki Sawada
	 * @return string
	 */
	public function getHash($path, $name = '')
	{
	    if ($name !== '') {
	        $path = $this->joinPathCE($path, $name);
	    }

	    return $this->encode($path);
	}

	/**
	 * Return decoded path of target hash
	 * This method do not check the stat of target
	 * Use method `realpath()` to do check of the stat of target.
	 *
	 * @param  string $hash
	 * @author Naoki Sawada
	 * @return string
	 */
	public function getPath($hash)
	{
	    return $this->decode($hash);
	}

	/**
	 * Return root or startPath hash.
	 *
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	public function defaultPath()
	{
	    return $this->encode($this->startPath ? $this->startPath : $this->root);
	}

	/**
	 * Return volume options required by client:.
	 *
	 * @param $hash
	 * @return array
	 * @author Dmitry (dio) Levashov
	 */
	public function options($hash)
	{
	    $create = $createext = [];
	    if (isset($this->archivers['create']) && is_array($this->archivers['create'])) {
	        foreach ($this->archivers['create'] as $m => $v) {
	            $create[] = $m;
	            $createext[$m] = $v['ext'];
	        }
	    }
	    $opts = [
			'path' => $hash ? $this->path($hash) : '',
			'url' => $this->URL,
			'tmbUrl' => (! $this->imgLib && $this->options['tmbFbSelf']) ? 'self' : $this->tmbURL,
			'disabled' => $this->disabled,
			'separator' => $this->separator,
			'copyOverwrite' => intval($this->options['copyOverwrite']),
			'uploadOverwrite' => intval($this->options['uploadOverwrite']),
			'uploadMaxSize' => intval($this->uploadMaxSize),
			'uploadMaxConn' => intval($this->options['uploadMaxConn']),
			'uploadMime' => [
				'firstOrder' => isset($this->uploadOrder[0]) ? $this->uploadOrder[0] : 'deny',
				'allow' => $this->uploadAllow,
				'deny' => $this->uploadDeny,
			],
			'dispInlineRegex' => $this->options['dispInlineRegex'],
			'jpgQuality' => intval($this->options['jpgQuality']),
			'archivers' => [
				'create' => $create,
				'extract' => isset($this->archivers['extract']) && is_array($this->archivers['extract']) ? array_keys($this->archivers['extract']) : [],
				'createext' => $createext,
			],
			'uiCmdMap' => (isset($this->options['uiCmdMap']) && is_array($this->options['uiCmdMap'])) ? $this->options['uiCmdMap'] : [],
			'syncChkAsTs' => intval($this->options['syncChkAsTs']),
			'syncMinMs' => intval($this->options['syncMinMs']),
			'i18nFolderName' => intval($this->options['i18nFolderName']),
			'tmbCrop' => intval($this->options['tmbCrop']),
		];
	    if (! empty($this->options['trashHash'])) {
	        $opts['trashHash'] = $this->options['trashHash'];
	    }
	    if ($hash === null) {
	        // call from getRootStatExtra()
			if (! empty($this->options['icon'])) {
			    $opts['icon'] = $this->options['icon'];
			}
	        if (! empty($this->options['rootCssClass'])) {
	            $opts['csscls'] = $this->options['rootCssClass'];
	        }
	        if (isset($this->options['netkey'])) {
	            $opts['netkey'] = $this->options['netkey'];
	        }
	    }

	    return $opts;
	}

	/**
	 * Get option value of this volume.
	 *
	 * @param string $name  target option name
	 * @return null|mixed   target option value
	 * @author Naoki Sawada
	 */
	public function getOption($name)
	{
	    return isset($this->options[$name]) ? $this->options[$name] : null;
	}

	/**
	 * Get plugin values of this options.
	 *
	 * @param string $name  Plugin name
	 * @return null|array   Plugin values
	 * @author Naoki Sawada
	 */
	public function getOptionsPlugin($name = '')
	{
	    if ($name) {
	        return isset($this->options['plugin'][$name]) ? $this->options['plugin'][$name] : [];
	    } else {
	        return $this->options['plugin'];
	    }
	}

	/**
	 * Return true if command disabled in options.
	 *
	 * @param  string  $cmd  command name
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	public function commandDisabled($cmd)
	{
	    return in_array($cmd, $this->disabled);
	}

	/**
	 * Return true if mime is required mimes list.
	 *
	 * @param  string     $mime   mime type to check
	 * @param  array      $mimes  allowed mime types list or not set to use client mimes list
	 * @param  bool|null  $empty  what to return on empty list
	 * @return bool|null
	 * @author Dmitry (dio) Levashov
	 * @author Troex Nevelin
	 **/
	public function mimeAccepted($mime, $mimes = null, $empty = true)
	{
	    $mimes = is_array($mimes) ? $mimes : $this->onlyMimes;
	    if (empty($mimes)) {
	        return $empty;
	    }

	    return $mime == 'directory'
			|| in_array('all', $mimes)
			|| in_array('All', $mimes)
			|| in_array($mime, $mimes)
			|| in_array(substr($mime, 0, strpos($mime, '/')), $mimes);
	}

	/**
	 * Return true if voume is readable.
	 *
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	public function isReadable()
	{
	    $stat = $this->stat($this->root);

	    return $stat['read'];
	}

	/**
	 * Return true if copy from this volume allowed.
	 *
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	public function copyFromAllowed()
	{
	    return (bool) $this->options['copyFrom'];
	}

	/**
	 * Return file path related to root with convert encoging.
	 *
	 * @param  string   $hash  file hash
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	public function path($hash)
	{
	    return $this->convEncOut($this->_path($this->convEncIn($this->decode($hash))));
	}

	/**
	 * Return file real path if file exists.
	 *
	 * @param  string  $hash  file hash
	 * @return string | false
	 * @author Dmitry (dio) Levashov
	 **/
	public function realpath($hash)
	{
	    $path = $this->decode($hash);

	    return $this->stat($path) ? $path : false;
	}

	/**
	 * Return list of moved/overwrited files.
	 *
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function removed()
	{
	    if ($this->removed) {

The expression $this->removed of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	        $unsetSubdir = isset($this->sessionCache['subdirs']) ? true : false;
	        foreach ($this->removed as $item) {
	            if ($item['mime'] === 'directory') {
	                $path = $this->decode($item['hash']);
	                if ($unsetSubdir) {
	                    unset($this->sessionCache['subdirs'][$path]);
	                }
	                if ($item['phash'] !== '') {
	                    $parent = $this->decode($item['phash']);
	                    unset($this->cache[$parent]);
	                    if ($unsetSubdir) {
	                        unset($this->sessionCache['subdirs'][$parent]);
	                    }
	                }
	            }
	        }
	    }

	    return $this->removed;
	}

	/**
	 * Return list of added files.
	 *
	 * @deprecated
	 * @return array
	 * @author Naoki Sawada
	 **/
	public function added()
	{
	    return $this->added;
	}

	/**
	 * Clean removed files list.
	 *
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	public function resetRemoved()
	{
	    $this->resetResultStat();
	}

	/**
	 * Clean added/removed files list.
	 *
	 * @return void
	 **/
	public function resetResultStat()
	{
	    $this->removed = [];
	    $this->added = [];
	}

	/**
	 * Return file/dir hash or first founded child hash with required attr == $val.
	 *
	 * @param  string   $hash  file hash
	 * @param  string   $attr  attribute name
	 * @param  bool     $val   attribute value
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function closest($hash, $attr, $val)
	{
	    return ($path = $this->closestByAttr($this->decode($hash), $attr, $val)) ? $this->encode($path) : false;
	}

	/**
	 * Return file info or false on error.
	 *
	 * @param  string $hash file hash
	 * @return array|false
	 * @internal param bool $realpath add realpath field to file info
	 * @author Dmitry (dio) Levashov
	 */
	public function file($hash)
	{
	    $file = $this->stat($this->decode($hash));

	    return ($file) ? $file : $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	}

	/**
	 * Return folder info.
	 *
	 * @param  string $hash folder hash
	 * @param bool $resolveLink
	 * @return array|false
	 * @internal param bool $hidden return hidden file info
	 * @author Dmitry (dio) Levashov
	 */
	public function dir($hash, $resolveLink = false)
	{
	    if (($dir = $this->file($hash)) == false) {
	        return $this->setError(elFinder::ERROR_DIR_NOT_FOUND);
	    }

	    if ($resolveLink && ! empty($dir['thash'])) {
	        $dir = $this->file($dir['thash']);
	    }

	    return $dir && $dir['mime'] == 'directory' && empty($dir['hidden'])
			? $dir
			: $this->setError(elFinder::ERROR_NOT_DIR);
	}

	/**
	 * Return directory content or false on error.
	 *
	 * @param  string   $hash   file hash
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function scandir($hash)
	{
	    if (($dir = $this->dir($hash)) == false) {
	        return false;
	    }

	    $path = $this->decode($hash);
	    if ($res = $dir['read']
			? $this->getScandir($path)
			: $this->setError(elFinder::ERROR_PERM_DENIED)) {
	        $dirs = null;
	        if ($this->sessionCaching['subdirs'] && isset($this->sessionCache['subdirs'][$path])) {

This code seems to be duplicated across your project.

	            $dirs = $this->sessionCache['subdirs'][$path];
	        }
	        if ($dirs !== null || (isset($dir['dirs']) && $dir['dirs'] != 1)) {
	            $_dir = $dir;
	            if ($dirs || $this->subdirs($hash)) {
	                $dir['dirs'] = 1;
	            } else {
	                unset($dir['dirs']);
	            }
	            if ($dir !== $_dir) {
	                $this->updateCache($path, $dir);

It seems like $dir defined by $this->dir($hash) on line 1747 can also be of type boolean; however, elFinderVolumeDriver::updateCache() does only seem to accept array, maybe add an additional type check?

	            }
	        }
	    }

	    return $res;
	}

	/**
	 * Return dir files names list.
	 *
	 * @param  string $hash file hash
	 * @param null $intersect
	 * @return array
	 * @author Dmitry (dio) Levashov
	 */
	public function ls($hash, $intersect = null)
	{
	    if (($dir = $this->dir($hash)) == false || ! $dir['read']) {
	        return false;
	    }

	    $list = [];
	    $path = $this->decode($hash);

	    $check = [];
	    if ($intersect) {
	        $check = array_flip($intersect);
	    }

	    foreach ($this->getScandir($path) as $stat) {
	        if (empty($stat['hidden']) && (! $check || isset($check[$stat['name']])) && $this->mimeAccepted($stat['mime'])) {

The expression $check of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	            $list[$stat['hash']] = $stat['name'];
	        }
	    }

	    return $list;
	}

	/**
	 * Return subfolders for required folder or false on error.
	 *
	 * @param  string   $hash  folder hash or empty string to get tree from root folder
	 * @param  int      $deep  subdir deep
	 * @param  string   $exclude  dir hash which subfolders must be exluded from result, required to not get stat twice on cwd subfolders
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function tree($hash = '', $deep = 0, $exclude = '')
	{
	    $path = $hash ? $this->decode($hash) : $this->root;

	    if (($dir = $this->stat($path)) == false || $dir['mime'] != 'directory') {

This code seems to be duplicated across your project.

	        return false;
	    }

	    $dirs = $this->gettree($path, $deep > 0 ? $deep - 1 : $this->treeDeep - 1, $exclude ? $this->decode($exclude) : null);
	    array_unshift($dirs, $dir);

	    return $dirs;
	}

	/**
	 * Return part of dirs tree from required dir up to root dir.
	 *
	 * @param  string    $hash   directory hash
	 * @param  bool|null $lineal only lineal parents
	 * @param  string    $until  hash that is enough to that extent >= 2.1.24

There is no parameter named $until. Was it maybe removed?

	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function parents($hash, $lineal = false)
	{
	    if (($current = $this->dir($hash)) == false) {
	        return false;
	    }

	    $args = func_get_args();
		// checks 3rd param `$until` (elFinder >= 2.1.24)

39% of this comment could be valid code. Did you maybe forget this after debugging?

		$until = '';
	    if (isset($args[2])) {
	        $until = $args[2];
	    }

	    $path = $this->decode($hash);
	    $tree = [];

	    while ($path && $path != $this->root) {

The expression $path of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

	        elFinder::extendTimeLimit();
	        $path = $this->dirnameCE($path);
	        if (! ($stat = $this->stat($path)) || ! empty($stat['hidden']) || ! $stat['read']) {
	            return false;
	        }

	        array_unshift($tree, $stat);
	        if (! $lineal) {

The expression $lineal of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

	            foreach ($this->gettree($path, 0) as $dir) {
	                elFinder::extendTimeLimit();
	                if (! isset($tree[$dir['hash']])) {
	                    $tree[$dir['hash']] = $dir;
	                }
	            }
	        }

	        if ($until && $until === $this->encode($path)) {
	            break;
	        }
	    }

	    return $tree ? array_values($tree) : [$current];
	}

	/**
	 * Create thumbnail for required file and return its name of false on failed.
	 *
	 * @param $hash
	 * @return false|string
	 * @author Dmitry (dio) Levashov
	 */
	public function tmb($hash)
	{
	    $path = $this->decode($hash);
	    $stat = $this->stat($path);

	    if (isset($stat['tmb'])) {
	        $res = $stat['tmb'] == '1' ? $this->createTmb($path, $stat) : $stat['tmb'];
	        if (! $res) {
	            list($type) = explode('/', $stat['mime']);
	            $fallback = $this->options['resourcePath'].DIRECTORY_SEPARATOR.strtolower($type).'.png';
	            if (is_file($fallback)) {
	                $res = $this->tmbname($stat);
	                if (! copy($fallback, $this->tmbPath.DIRECTORY_SEPARATOR.$res)) {
	                    $res = false;
	                }
	            }
	        }

	        return $res;
	    }

	    return false;
	}

	/**
	 * Return file size / total directory size.
	 *
	 * @param  string   file hash
	 * @return int
	 * @author Dmitry (dio) Levashov
	 **/
	public function size($hash)
	{
	    return $this->countSize($this->decode($hash));
	}

	/**
	 * Open file for reading and return file pointer.
	 *
	 * @param  string   file hash
	 * @return resource
	 * @author Dmitry (dio) Levashov
	 **/
	public function open($hash)
	{
	    if (($file = $this->file($hash)) == false

This code seems to be duplicated across your project.

		|| $file['mime'] == 'directory') {
	        return false;
	    }

	    return $this->fopenCE($this->decode($hash), 'rb');
	}

	/**
	 * Close file pointer.
	 *
	 * @param  resource  $fp   file pointer
	 * @param  string    $hash file hash
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	public function close($fp, $hash)
	{
	    $this->fcloseCE($fp, $this->decode($hash));
	}

	/**
	 * Create directory and return dir info.
	 *
	 * @param  string   $dsthash  destination directory hash
	 * @param  string   $name directory name
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function mkdir($dsthash, $name)
	{
	    if ($this->commandDisabled('mkdir')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (! $this->nameAccepted($name, true)) {
	        return $this->setError(elFinder::ERROR_INVALID_DIRNAME);
	    }

	    if (($dir = $this->dir($dsthash)) == false) {
	        return $this->setError(elFinder::ERROR_TRGDIR_NOT_FOUND, '#'.$dsthash);
	    }

	    $path = $this->decode($dsthash);

	    if (! $dir['write'] || ! $this->allowCreate($path, $name, true)) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    $dst = $this->joinPathCE($path, $name);
	    $stat = $this->isNameExists($dst);
	    if (! empty($stat)) {
	        return $this->setError(elFinder::ERROR_EXISTS, $name);
	    }
	    $this->clearcache();

	    $mkpath = $this->convEncOut($this->_mkdir($this->convEncIn($path), $this->convEncIn($name)));
	    if ($mkpath) {
	        $this->updateSubdirsCache($path, true);
	        $this->updateSubdirsCache($mkpath, false);
	    }

	    return $mkpath ? $this->stat($mkpath) : false;
	}

	/**
	 * Create empty file and return its info.
	 *
	 * @param  string   $dst  destination directory
	 * @param  string   $name file name
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function mkfile($dst, $name)
	{
	    if ($this->commandDisabled('mkfile')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (! $this->nameAccepted($name, false)) {
	        return $this->setError(elFinder::ERROR_INVALID_NAME);
	    }

	    $mimeByName = $this->mimetype($name, true);
	    if ($mimeByName && ! $this->allowPutMime($mimeByName)) {
	        return $this->setError(elFinder::ERROR_UPLOAD_FILE_MIME, $name);
	    }

	    if (($dir = $this->dir($dst)) == false) {

This code seems to be duplicated across your project.

	        return $this->setError(elFinder::ERROR_TRGDIR_NOT_FOUND, '#'.$dst);
	    }

	    $path = $this->decode($dst);

	    if (! $dir['write'] || ! $this->allowCreate($path, $name, false)) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if ($this->isNameExists($this->joinPathCE($path, $name))) {
	        return $this->setError(elFinder::ERROR_EXISTS, $name);
	    }

	    $this->clearcache();

	    return ($path = $this->convEncOut($this->_mkfile($this->convEncIn($path), $this->convEncIn($name)))) ? $this->stat($path) : false;
	}

	/**
	 * Rename file and return file info.
	 *
	 * @param  string  $hash  file hash
	 * @param  string  $name  new file name
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function rename($hash, $name)
	{
	    if ($this->commandDisabled('rename')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (! ($file = $this->file($hash))) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    if ($name === $file['name']) {
	        return $file;
	    }

	    if (! empty($file['locked'])) {
	        return $this->setError(elFinder::ERROR_LOCKED, $file['name']);
	    }

	    $isDir = ($file['mime'] === 'directory');

	    if (! $this->nameAccepted($name, $isDir)) {
	        return $this->setError(elFinder::ERROR_INVALID_DIRNAME);
	    }

	    if (! $isDir) {
	        $mimeByName = $this->mimetype($name, true);
	        if ($mimeByName && ! $this->allowPutMime($mimeByName)) {
	            return $this->setError(elFinder::ERROR_UPLOAD_FILE_MIME, $name);
	        }
	    }

	    $path = $this->decode($hash);
	    $dir = $this->dirnameCE($path);
	    $stat = $this->isNameExists($this->joinPathCE($dir, $name));
	    if ($stat) {

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	        return $this->setError(elFinder::ERROR_EXISTS, $name);
	    }

	    if (! $this->allowCreate($dir, $name, ($file['mime'] === 'directory'))) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    $this->rmTmb($file); // remove old name tmbs, we cannot do this after dir move

$file is of type array|boolean, but the function expects a string.

		if ($path = $this->convEncOut($this->_move($this->convEncIn($path), $this->convEncIn($dir), $this->convEncIn($name)))) {
		    $this->clearcache();

		    return $this->stat($path);
		}

	    return false;
	}

	/**
	 * Create file copy with suffix "copy number" and return its info.
	 *
	 * @param  string   $hash    file hash
	 * @param  string   $suffix  suffix to add to file name
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function duplicate($hash, $suffix = 'copy')
	{
	    if ($this->commandDisabled('duplicate')) {
	        return $this->setError(elFinder::ERROR_COPY, '#'.$hash, elFinder::ERROR_PERM_DENIED);
	    }

	    if (($file = $this->file($hash)) == false) {
	        return $this->setError(elFinder::ERROR_COPY, elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    $path = $this->decode($hash);
	    $dir = $this->dirnameCE($path);
	    $name = $this->uniqueName($dir, $file['name'], sprintf($this->options['duplicateSuffix'], $suffix));

	    if (! $this->allowCreate($dir, $name, ($file['mime'] === 'directory'))) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    return ($path = $this->copy($path, $dir, $name)) == false
			? false
			: $this->stat($path);

It seems like $path defined by $this->copy($path, $dir, $name) on line 2125 can also be of type boolean or null; however, elFinderVolumeDriver::stat() does only seem to accept string, maybe add an additional type check?

	}

	/**
	 * Save uploaded file.
	 * On success return array with new file stat and with removed file hash (if existed file was replaced).
	 *
	 * @param  resource $fp file pointer
	 * @param  string $dst destination folder hash
	 * @param $name
	 * @param  string $tmpname file tmp name - required to detect mime type
	 * @param  array $hashes exists files hash array with filename as key
	 * @return array|false
	 * @internal param string $src file name
	 * @author Dmitry (dio) Levashov
	 */
	public function upload($fp, $dst, $name, $tmpname, $hashes = [])
	{
	    if ($this->commandDisabled('upload')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (($dir = $this->dir($dst)) == false) {

This code seems to be duplicated across your project.

	        return $this->setError(elFinder::ERROR_TRGDIR_NOT_FOUND, '#'.$dst);
	    }

	    if (empty($dir['write'])) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (! $this->nameAccepted($name, false)) {
	        return $this->setError(elFinder::ERROR_INVALID_NAME);
	    }

	    $mimeByName = '';
	    if ($this->mimeDetect === 'internal') {
	        $mime = $this->mimetype($name, true);
	    } else {
	        $mime = $this->mimetype($tmpname, $name);
	        $mimeByName = $this->mimetype($name, true);
	        if ($mime === 'unknown') {
	            $mime = $mimeByName;
	        }
	    }

	    if (! $this->allowPutMime($mime) || ($mimeByName && ! $this->allowPutMime($mimeByName))) {

This code seems to be duplicated across your project.

	        return $this->setError(elFinder::ERROR_UPLOAD_FILE_MIME);
	    }

	    $tmpsize = sprintf('%u', filesize($tmpname));
	    if ($this->uploadMaxSize > 0 && $tmpsize > $this->uploadMaxSize) {
	        return $this->setError(elFinder::ERROR_UPLOAD_FILE_SIZE);
	    }

	    $dstpath = $this->decode($dst);
	    if (isset($hashes[$name])) {

This code seems to be duplicated across your project.

	        $test = $this->decode($hashes[$name]);
	        $file = $this->stat($test);
	    } else {
	        $test = $this->joinPathCE($dstpath, $name);
	        $file = $this->isNameExists($test);
	    }

	    $this->clearcache();

	    if ($file && $file['name'] === $name) { // file exists and check filename for item ID based filesystem

The expression $file of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			if ($this->uploadOverwrite) {
			    if (! $file['write']) {

This code seems to be duplicated across your project.

			        return $this->setError(elFinder::ERROR_PERM_DENIED);
			    } elseif ($file['mime'] == 'directory') {
			        return $this->setError(elFinder::ERROR_NOT_REPLACE, $name);
			    }
			    $this->remove($test);
			} else {
			    $name = $this->uniqueName($dstpath, $name, '-', false);
			}
	    }

	    $stat = [
			'mime' => $mime,
			'width' => 0,
			'height' => 0,
			'size' => $tmpsize, ];

		// $w = $h = 0;

37% of this comment could be valid code. Did you maybe forget this after debugging?

		if (strpos($mime, 'image') === 0 && ($s = getimagesize($tmpname))) {
		    $stat['width'] = $s[0];
		    $stat['height'] = $s[1];
		}
		// $this->clearcache();

72% of this comment could be valid code. Did you maybe forget this after debugging?

		if (($path = $this->saveCE($fp, $dstpath, $name, $stat)) == false) {
		    return false;
		}

	    $stat = $this->stat($path);

It seems like $path defined by $this->saveCE($fp, $dstpath, $name, $stat) on line 2217 can also be of type boolean; however, elFinderVolumeDriver::stat() does only seem to accept string, maybe add an additional type check?

		// Try get URL
		if (empty($stat['url']) && ($url = $this->getContentUrl($stat['hash']))) {
		    $stat['url'] = $url;
		}

	    return $stat;
	}

	/**
	 * Paste files.
	 *
	 * @param  object $volume source volume
	 * @param $src
	 * @param  string $dst destination dir hash
	 * @param  bool $rmSrc remove source after copy?
	 * @param array $hashes
	 * @return array|false
	 * @internal param string $source file hash
	 * @author Dmitry (dio) Levashov
	 */
	public function paste($volume, $src, $dst, $rmSrc = false, $hashes = [])
	{
	    $err = $rmSrc ? elFinder::ERROR_MOVE : elFinder::ERROR_COPY;

	    if ($this->commandDisabled('paste')) {
	        return $this->setError($err, '#'.$src, elFinder::ERROR_PERM_DENIED);
	    }

	    if (($file = $volume->file($src, $rmSrc)) == false) {
	        return $this->setError($err, '#'.$src, elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    $name = $file['name'];
	    $errpath = $volume->path($file['hash']);

	    if (($dir = $this->dir($dst)) == false) {

This code seems to be duplicated across your project.

	        return $this->setError($err, $errpath, elFinder::ERROR_TRGDIR_NOT_FOUND, '#'.$dst);
	    }

	    if (! $dir['write'] || ! $file['read']) {
	        return $this->setError($err, $errpath, elFinder::ERROR_PERM_DENIED);
	    }

	    $destination = $this->decode($dst);

	    if (($test = $volume->closest($src, $rmSrc ? 'locked' : 'read', $rmSrc))) {
	        return $rmSrc
				? $this->setError($err, $errpath, elFinder::ERROR_LOCKED, $volume->path($test))
				: $this->setError($err, $errpath, empty($file['thash']) ? elFinder::ERROR_PERM_DENIED : elFinder::ERROR_MKOUTLINK);
	    }

	    if (isset($hashes[$name])) {

This code seems to be duplicated across your project.

	        $test = $this->decode($hashes[$name]);
	        $stat = $this->stat($test);
	    } else {
	        $test = $this->joinPathCE($destination, $name);
	        $stat = $this->isNameExists($test);
	    }
	    $this->clearcache();
	    $dstDirExists = false;
	    if ($stat && $stat['name'] === $name) { // file exists and check filename for item ID based filesystem

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			if ($this->options['copyOverwrite']) {
			    // do not replace file with dir or dir with file
				if (! $this->isSameType($file['mime'], $stat['mime'])) {
				    return $this->setError(elFinder::ERROR_NOT_REPLACE, $this->path($stat['hash']));
				}
				// existed file is not writable
				if (! $stat['write']) {
				    return $this->setError($err, $errpath, elFinder::ERROR_PERM_DENIED);
				}
			    if ($this->options['copyJoin']) {
			        if ($stat['locked']) {
			            return $this->setError(elFinder::ERROR_LOCKED, $this->path($stat['hash']));
			        }
			    } else {
			        // existed file locked or has locked child
					if (($locked = $this->closestByAttr($test, 'locked', true))) {
					    $stat = $this->stat($locked);

					    return $this->setError(elFinder::ERROR_LOCKED, $this->path($stat['hash']));
					}
			    }
				// target is entity file of alias
				if ($volume === $this && ((isset($file['target']) && $test == $file['target']) || $test == $this->decode($src))) {
				    return $this->setError(elFinder::ERROR_REPLACE, $errpath);
				}
				// remove existed file
				if (! $this->options['copyJoin'] || $stat['mime'] !== 'directory') {
				    if (! $this->remove($test)) {
				        return $this->setError(elFinder::ERROR_REPLACE, $this->path($stat['hash']));
				    }
				} elseif ($stat['mime'] === 'directory') {
				    $dstDirExists = true;
				}
			} else {
			    $name = $this->uniqueName($destination, $name, ' ', false);
			}
	    }

		// copy/move inside current volume
		if ($volume === $this) { //  changing == operand to === fixes issue #1285 - Paul Canning 24/03/2016
			$source = $this->decode($src);
			// do not copy into itself
			if ($this->inpathCE($destination, $source)) {
			    return $this->setError(elFinder::ERROR_COPY_INTO_ITSELF, $errpath);
			}
		    $rmDir = false;
		    if ($rmSrc) {
		        if ($dstDirExists) {
		            $rmDir = true;
		            $method = 'copy';
		        } else {
		            $method = 'move';
		        }
		    } else {
		        $method = 'copy';
		    }
		    $this->clearcache();
		    if ($res = ($path = $this->$method($source, $destination, $name)) ? $this->stat($path) : false) {
		        if ($rmDir) {
		            $this->remove($source);
		        }
		    } else {
		        return false;
		    }
		} else {
		    // copy/move from another volume
			if (! $this->options['copyTo'] || ! $volume->copyFromAllowed()) {
			    return $this->setError(elFinder::ERROR_COPY, $errpath, elFinder::ERROR_PERM_DENIED);
			}

		    if (($path = $this->copyFrom($volume, $src, $destination, $name)) == false) {
		        return false;
		    }

		    if ($rmSrc) {
		        if (! $volume->rm($src)) {
		            return $this->setError(elFinder::ERROR_MOVE, $errpath, elFinder::ERROR_RM_SRC);
		        }
		    }
		    $res = $this->stat($path);

It seems like $path defined by $this->copyFrom($volume, $src, $destination, $name) on line 2353 can also be of type boolean or null; however, elFinderVolumeDriver::stat() does only seem to accept string, maybe add an additional type check?

		}

	    return $res;
	}

	/**
	 * Return path to archive of target items.
	 *
	 * @param  array  $hashes
	 * @return string archive path
	 * @author Naoki Sawada
	 */
	public function zipdl($hashes)
	{
	    if ($this->commandDisabled('zipdl')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    $archivers = $this->getArchivers();
	    $cmd = null;
	    if (! $archivers || empty($archivers['create'])) {

The expression $archivers of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	        return false;
	    }
	    $archivers = $archivers['create'];
	    foreach (['zip', 'tgz'] as $ext) {
	        $mime = self::$mimetypes[$ext];
	        if (isset($archivers[$mime])) {
	            $cmd = $archivers[$mime];
	            break;
	        }
	    }
	    if (! $cmd) {
	        $cmd = $archivers[0];
	        $ext = $cmd['ext'];
	        $mime = $this->mimetype('file.'.$ext, true);
	    }
	    $res = false;
	    $mixed = false;
	    $hashes = array_values($hashes);
	    $dirname = dirname(str_replace($this->separator, DIRECTORY_SEPARATOR, $this->path($hashes[0])));
	    $cnt = count($hashes);
	    if ($cnt > 1) {
	        for ($i = 1; $i < $cnt; $i++) {
	            if ($dirname !== dirname(str_replace($this->separator, DIRECTORY_SEPARATOR, $this->path($hashes[$i])))) {
	                $mixed = true;
	                break;
	            }
	        }
	    }
	    if ($mixed || $this->root == $this->dirnameCE($this->decode($hashes[0]))) {
	        $prefix = $this->rootName;
	    } else {
	        $prefix = basename($dirname);
	    }
	    if ($dir = $this->getItemsInHand($hashes)) {
	        $tmppre = (substr(PHP_OS, 0, 3) === 'WIN') ? 'zdl' : 'elfzdl';
	        $pdir = dirname($dir);
			// garbage collection
			$ttl = 7200; // expire 2h
			$time = time();
	        foreach (glob($pdir.DIRECTORY_SEPARATOR.$tmppre.'*') as $_file) {
	            if (filemtime($_file) + $ttl < $time) {
	                unlink($_file);
	            }
	        }
	        $files = self::localScandir($dir);
	        if ($files && ($arc = tempnam($dir, $tmppre))) {

The expression $files of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	            unlink($arc);
	            $arc = $arc.'.'.$ext;

The variable $ext does not seem to be defined for all execution paths leading up to this point.

	            $name = basename($arc);
	            if ($arc = $this->makeArchive($dir, $files, $name, $cmd)) {
	                $file = tempnam($pdir, $tmppre);
	                unlink($file);
	                $res = rename($arc, $file);
	                $this->rmdirRecursive($dir);
	            }
	        }
	    }

	    return $res ? ['path' => $file, 'ext' => $ext, 'mime' => $mime, 'prefix' => $prefix] : false;

The variable $file does not seem to be defined for all execution paths leading up to this point.

The variable $mime does not seem to be defined for all execution paths leading up to this point.

	}

	/**
	 * Return file contents.
	 *
	 * @param  string  $hash  file hash
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	public function getContents($hash)
	{
	    $file = $this->file($hash);

	    if (! $file) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    if ($file['mime'] == 'directory') {
	        return $this->setError(elFinder::ERROR_NOT_FILE);
	    }

	    if (! $file['read']) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if ($this->getMaxSize > 0 && $file['size'] > $this->getMaxSize) {
	        return $this->setError(elFinder::ERROR_UPLOAD_FILE_SIZE);
	    }

	    return $this->_getContents($this->convEncIn($this->decode($hash), true));
	}

	/**
	 * Put content in text file and return file info.
	 *
	 * @param  string  $hash     file hash
	 * @param  string  $content  new file content
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function putContents($hash, $content)
	{
	    if ($this->commandDisabled('edit')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    $path = $this->decode($hash);

	    if (! ($file = $this->file($hash))) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    if (! $file['write']) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

		// check data cheme
		if (preg_match('~^data:(.+?/.+?);base64,~', $content, $m)) {
		    $dMime = $m[1];
		    if ($file['size'] > 0 && $dMime !== $file['mime']) {
		        return $this->setError(elFinder::ERROR_PERM_DENIED);
		    }
		    $content = base64_decode(substr($content, strlen($m[0])));
		}

		// check MIME
		$name = $this->basenameCE($path);
	    $mime = '';
	    $mimeByName = $this->mimetype($name, true);
	    if ($this->mimeDetect !== 'internal') {
	        if ($tp = tmpfile()) {
	            fwrite($tp, $content);
	            $info = stream_get_meta_data($tp);
	            $filepath = $info['uri'];
	            $mime = $this->mimetype($filepath, $name);
	            fclose($tp);
	        }
	    }
	    if (! $this->allowPutMime($mimeByName) || ($mime && ! $this->allowPutMime($mime))) {

This code seems to be duplicated across your project.

	        return $this->setError(elFinder::ERROR_UPLOAD_FILE_MIME);
	    }

	    $this->clearcache();

	    return $this->convEncOut($this->_filePutContents($this->convEncIn($path), $content)) ? $this->stat($path) : false;
	}

	/**
	 * Extract files from archive.
	 *
	 * @param  string $hash archive hash
	 * @param null $makedir
	 * @return array|bool
	 * @author Dmitry (dio) Levashov,
	 * @author Alexey Sukhotin
	 */
	public function extract($hash, $makedir = null)
	{
	    if ($this->commandDisabled('extract')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (($file = $this->file($hash)) == false) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    $archiver = isset($this->archivers['extract'][$file['mime']])
			? $this->archivers['extract'][$file['mime']]
			: false;

	    if (! $archiver) {
	        return $this->setError(elFinder::ERROR_NOT_ARCHIVE);
	    }

	    $path = $this->decode($hash);
	    $parent = $this->stat($this->dirnameCE($path));

	    if (! $file['read'] || ! $parent['write']) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }
	    $this->clearcache();
	    $this->extractToNewdir = is_null($makedir) ? 'auto' : (bool) $makedir;

It seems like is_null($makedir) ? 'auto' : (bool) $makedir of type string or boolean is incompatible with the declared type integer of property $extractToNewdir.

	    if ($path = $this->convEncOut($this->_extract($this->convEncIn($path), $archiver))) {
	        if (is_array($path)) {
	            foreach ($path as $_k => $_p) {
	                $path[$_k] = $this->stat($_p);
	            }
	        } else {
	            $path = $this->stat($path);
	        }

	        return $path;
	    } else {
	        return false;
	    }
	}

	/**
	 * Add files to archive.
	 *
	 * @param $hashes
	 * @param $mime
	 * @param string $name
	 * @return array|bool
	 */
	public function archive($hashes, $mime, $name = '')
	{
	    if ($this->commandDisabled('archive')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if ($name !== '' && ! $this->nameAccepted($name, false)) {
	        return $this->setError(elFinder::ERROR_INVALID_NAME);
	    }

	    $archiver = isset($this->archivers['create'][$mime])
			? $this->archivers['create'][$mime]
			: false;

	    if (! $archiver) {
	        return $this->setError(elFinder::ERROR_ARCHIVE_TYPE);
	    }

	    $files = [];

	    foreach ($hashes as $hash) {
	        if (($file = $this->file($hash)) == false) {

This code seems to be duplicated across your project.

	            return $this->setError(elFinder::ERROR_FILE_NOT_FOUND, '#' + $hash);
	        }
	        if (! $file['read']) {
	            return $this->setError(elFinder::ERROR_PERM_DENIED);
	        }
	        $path = $this->decode($hash);
	        if (! isset($dir)) {
	            $dir = $this->dirnameCE($path);
	            $stat = $this->stat($dir);
	            if (! $stat['write']) {
	                return $this->setError(elFinder::ERROR_PERM_DENIED);
	            }
	        }

	        $files[] = $this->basenameCE($path);
	    }

	    if ($name === '') {
	        $name = count($files) == 1 ? $files[0] : 'Archive';
	    } else {
	        $name = str_replace(['/', '\\'], '_', preg_replace('/\.'.preg_quote($archiver['ext'], '/').'$/i', '', $name));
	    }
	    $name .= '.'.$archiver['ext'];
	    $name = $this->uniqueName($dir, $name, '');

The variable $dir does not seem to be defined for all execution paths leading up to this point.

	    $this->clearcache();

	    return ($path = $this->convEncOut($this->_archive($this->convEncIn($dir), $this->convEncIn($files), $this->convEncIn($name), $archiver))) ? $this->stat($path) : false;
	}

	/**
	 * Resize image.
	 *
	 * @param  string   $hash    image file
	 * @param  int      $width   new width
	 * @param  int      $height  new height
	 * @param  int      $x       X start poistion for crop
	 * @param  int      $y       Y start poistion for crop
	 * @param  string   $mode    action how to mainpulate image
	 * @param  string   $bg      background color
	 * @param  int      $degree  rotete degree
	 * @param  int      $jpgQuality  JEPG quality (1-100)
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 * @author Alexey Sukhotin
	 * @author nao-pon
	 * @author Troex Nevelin
	 **/
	public function resize($hash, $width, $height, $x, $y, $mode = 'resize', $bg = '', $degree = 0, $jpgQuality = null)
	{
	    if ($this->commandDisabled('resize')) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if (($file = $this->file($hash)) == false) {
	        return $this->setError(elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    if (! $file['write'] || ! $file['read']) {
	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    $path = $this->decode($hash);

	    $work_path = $this->getWorkFile($this->encoding ? $this->convEncIn($path, true) : $path);

	    if (! $work_path || ! is_writable($work_path)) {

The expression $work_path of type string|false is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === false instead.

	        if ($work_path && $path !== $work_path && is_file($work_path)) {

The expression $work_path of type string|false is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

	            unlink($work_path);
	        }

	        return $this->setError(elFinder::ERROR_PERM_DENIED);
	    }

	    if ($this->imgLib !== 'imagick' && $this->imgLib !== 'convert') {
	        if (elFinder::isAnimationGif($work_path)) {
	            return $this->setError(elFinder::ERROR_UNSUPPORT_TYPE);
	        }
	    }

	    switch ($mode) {

			case 'propresize':

This code seems to be duplicated across your project.

				$result = $this->imgResize($work_path, $width, $height, true, true, null, $jpgQuality);
				break;

			case 'crop':
				$result = $this->imgCrop($work_path, $width, $height, $x, $y, null, $jpgQuality);

$x is of type integer, but the function expects a boolean.

$y is of type integer, but the function expects a boolean.

				break;

			case 'fitsquare':
				$result = $this->imgSquareFit($work_path, $width, $height, 'center', 'middle', ($bg ? $bg : $this->options['tmbBgColor']), null, $jpgQuality);
				break;

			case 'rotate':
				$result = $this->imgRotate($work_path, $degree, ($bg ? $bg : $this->options['bgColorFb']), null, $jpgQuality);
				break;

			default:

This code seems to be duplicated across your project.

				$result = $this->imgResize($work_path, $width, $height, false, true, null, $jpgQuality);
				break;
		}

	    $ret = false;
	    if ($result) {

The expression $result of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

	        $this->rmTmb($file);

$file is of type array|boolean, but the function expects a string.

	        $this->clearstatcache();
	        $stat = $this->stat($path);
	        $fstat = stat($work_path);
	        $stat['size'] = $fstat['size'];
	        $stat['ts'] = $fstat['mtime'];
	        if ($imgsize = getimagesize($work_path)) {
	            $stat['width'] = $imgsize[0];
	            $stat['height'] = $imgsize[1];
	            $stat['mime'] = $imgsize['mime'];
	        }
	        if ($path !== $work_path) {
	            if ($fp = fopen($work_path, 'rb')) {
	                $ret = $this->saveCE($fp, $this->dirnameCE($path), $this->basenameCE($path), $stat);
	                fclose($fp);
	            }
	        } else {
	            $ret = true;
	        }
	        if ($ret) {
	            $this->clearcache();
	            $ret = $this->stat($path);
	            $ret['width'] = $stat['width'];
	            $ret['height'] = $stat['height'];
	        }
	    }
	    if ($path !== $work_path) {
	        is_file($work_path) && unlink($work_path);
	    }

	    return $ret;
	}

	/**
	 * Remove file/dir.
	 *
	 * @param  string  $hash  file hash
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	public function rm($hash)
	{
	    return $this->commandDisabled('rm')
			? $this->setError(elFinder::ERROR_PERM_DENIED)
			: $this->remove($this->decode($hash));
	}

	/**
	 * Search files.
	 *
	 * @param  string $q search string
	 * @param  array $mimes
	 * @param null $hash
	 * @return array
	 * @author Dmitry (dio) Levashov
	 */
	public function search($q, $mimes, $hash = null)
	{
	    $dir = null;
	    if ($hash) {
	        $dir = $this->decode($hash);
	        $stat = $this->stat($dir);
	        if (! $stat || $stat['mime'] !== 'directory' || ! $stat['read']) {

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	            $q = '';
	        }
	    }
	    if ($mimes && $this->onlyMimes) {

The expression $mimes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

The expression $this->onlyMimes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	        $mimes = array_intersect($mimes, $this->onlyMimes);
	        if (! $mimes) {

The expression $mimes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	            $q = '';
	        }
	    }
	    $this->searchStart = time();

	    $qs = preg_split('/"([^"]+)"| +/', $q, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);
	    $query = $excludes = [];
	    foreach ($qs as $_q) {
	        $_q = trim($_q);
	        if ($_q !== '') {
	            if ($_q[0] === '-') {
	                if (isset($_q[1])) {
	                    $excludes[] = substr($_q, 1);
	                }
	            } else {
	                $query[] = $_q;
	            }
	        }
	    }
	    if (! $query) {

The expression $query of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	        $q = '';
	    } else {
	        $q = implode(' ', $query);
	        $this->doSearchCurrentQuery = [

It seems like array('q' => $q, 'excludes' => $excludes) of type array<string,string|array,{"q":"string","excludes":"array"}> is incompatible with the declared type string of property $doSearchCurrentQuery.

				'q' => $q,
				'excludes' => $excludes,
			];
	    }

		// valided regex $this->options['searchExDirReg']

46% of this comment could be valid code. Did you maybe forget this after debugging?

		if ($this->options['searchExDirReg']) {

This code seems to be duplicated across your project.

		    if (false === preg_match($this->options['searchExDirReg'], '')) {
		        $this->options['searchExDirReg'] = '';
		    }
		}

	    return ($q === '' || $this->commandDisabled('search'))
			? []
			: $this->doSearch(is_null($dir) ? $this->root : $dir, $q, $mimes);
	}

	/**
	 * Return image dimensions.
	 *
	 * @param  string  $hash  file hash
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	public function dimensions($hash)
	{
	    if (($file = $this->file($hash)) == false) {
	        return false;
	    }

	    return $this->convEncOut($this->_dimensions($this->convEncIn($this->decode($hash)), $file['mime']));
	}

	/**
	 * Return has subdirs.
	 *
	 * @param  string  $hash  file hash
	 * @return bool
	 * @author Naoki Sawada
	 **/
	public function subdirs($hash)
	{
	    return (bool) $this->subdirsCE($this->decode($hash));
	}

	/**
	 * Return content URL (for netmout volume driver)
	 * If file.url == 1 requests from JavaScript client with XHR.
	 *
	 * @param string $hash  file hash
	 * @param array $options  options array
	 * @return bool|string
	 * @author Naoki Sawada
	 */
	public function getContentUrl($hash, $options = [])
	{
	    if (($file = $this->file($hash)) === false) {
	        return false;
	    }
	    if (empty($file['url'])) {
	        if ($this->URL) {
	            $path = str_replace($this->separator, '/', substr($this->decode($hash), strlen($this->root) + 1));
	            if ($this->encoding) {
	                $path = $this->convEncIn($path, true);
	            }
	            $path = str_replace('%2F', '/', rawurlencode($path));

	            return $this->URL.$path;
	        }

	        return false;
	    } else {
	        if ($file['url'] != 1) {
	            return $file['url'];
	        } elseif (! empty($options['temporary']) && $this->tmpLinkPath) {
	            $name = 'temp_'.md5($hash);
	            $path = $this->tmpLinkPath.DIRECTORY_SEPARATOR.$name;
	            $contents = $this->getContents($hash);

$contents is not used, you could remove the assignment.

	            $gc = create_function('$p,$t', 'foreach(glob($p) as $f) { (filemtime($f) < (time() - $t)) && unlink($f); }');

The use of create_function is highly discouraged, better use a closure.

				/*$gc = function($p,$t) {

61% of this comment could be valid code. Did you maybe forget this after debugging?

					foreach(glob($p) as $f) {
						(filemtime($f) < (time() - $t)) && unlink($f);
					}
				};*/
				register_shutdown_function($gc, $this->tmpLinkPath.DIRECTORY_SEPARATOR.'temp_*', elFinder::$tmpLinkLifeTime);
	            if (file_put_contents($path, $this->getContents($hash))) {
	                return $this->tmpLinkUrl.'/'.$name;
	            }
	        }

	        return false;
	    }
	}

	/**
	 * Return temp path.
	 *
	 * @return string
	 * @author Naoki Sawada
	 */
	public function getTempPath()
	{
	    $tempPath = null;
	    if (isset($this->tmpPath) && $this->tmpPath && is_writable($this->tmpPath)) {
	        $tempPath = $this->tmpPath;

The property tmpPath does not exist. Did you maybe forget to declare it?

	    } elseif (isset($this->tmp) && $this->tmp && is_writable($this->tmp)) {

The property tmp does not exist. Did you maybe forget to declare it?

	        $tempPath = $this->tmp;
	    } elseif (function_exists('sys_get_temp_dir')) {
	        $tempPath = sys_get_temp_dir();
	    } elseif ($this->tmbPathWritable) {
	        $tempPath = $this->tmbPath;
	    }
	    if ($tempPath && DIRECTORY_SEPARATOR !== '/') {
	        $tempPath = str_replace('/', DIRECTORY_SEPARATOR, $tempPath);
	    }

	    return $tempPath;
	}

	/**
	 * (Make &) Get upload taget dirctory hash.
	 *
	 * @param string $baseTargetHash
	 * @param string $path
	 * @param array  $result
	 * @return bool|string
	 * @author Naoki Sawada
	 */
	public function getUploadTaget($baseTargetHash, $path, &$result)
	{
	    $base = $this->decode($baseTargetHash);
	    $targetHash = $baseTargetHash;
	    $path = ltrim($path, $this->separator);
	    $dirs = explode($this->separator, $path);
	    array_pop($dirs);
	    foreach ($dirs as $dir) {
	        $targetPath = $this->joinPathCE($base, $dir);
	        if (! $_realpath = $this->realpath($this->encode($targetPath))) {
	            if ($stat = $this->mkdir($targetHash, $dir)) {
	                $result['added'][] = $stat;
	                $targetHash = $stat['hash'];
	                $base = $this->decode($targetHash);
	            } else {
	                return false;
	            }
	        } else {
	            $targetHash = $this->encode($_realpath);
	            if ($this->dir($targetHash)) {
	                $base = $this->decode($targetHash);
	            } else {
	                return false;
	            }
	        }
	    }

	    return $targetHash;
	}

	/**
	 * Return this uploadMaxSize value.
	 *
	 * @return int
	 * @author Naoki Sawada
	 */
	public function getUploadMaxSize()
	{
	    return $this->uploadMaxSize;
	}

    public function setUploadOverwrite($var)
    {
        $this->uploadOverwrite = (bool) $var;

The property $uploadOverwrite was declared of type string, but (bool) $var is of type boolean. Maybe add a type cast?

    }

	/**
	 * Image file utility.
	 *
	 * @param string $mode     'resize', 'rotate', 'propresize', 'crop', 'fitsquare'
	 * @param string $src      Image file local path
	 * @param array  $options  excute options
	 * @return bool
	 * @author Naoki Sawada
	 */
	public function imageUtil($mode, $src, $options = [])
	{
	    if (! isset($options['jpgQuality'])) {
	        $options['jpgQuality'] = intval($this->options['jpgQuality']);
	    }
	    if (! isset($options['bgcolor'])) {
	        $options['bgcolor'] = '#ffffff';
	    }
	    if (! isset($options['bgColorFb'])) {
	        $options['bgColorFb'] = $this->options['bgColorFb'];
	    }

		// check 'width' ,'height'

43% of this comment could be valid code. Did you maybe forget this after debugging?

		if (in_array($mode, ['resize', 'propresize', 'crop', 'fitsquare'])) {
		    if (empty($options['width']) || empty($options['height'])) {
		        return false;
		    }
		}

	    switch ($mode) {
			case 'rotate':
				if (empty($options['degree'])) {
				    return true;
				}

				return (bool) $this->imgRotate($src, $options['degree'], $options['bgColorFb'], null, $options['jpgQuality']);

			case 'resize':

This code seems to be duplicated across your project.

				return (bool) $this->imgResize($src, $options['width'], $options['height'], false, true, null, $options['jpgQuality'], $options);

			case 'propresize':

This code seems to be duplicated across your project.

				return (bool) $this->imgResize($src, $options['width'], $options['height'], true, true, null, $options['jpgQuality'], $options);

			case 'crop':
				if (isset($options['x']) && isset($options['y'])) {
				    return (bool) $this->imgCrop($src, $options['width'], $options['height'], $options['x'], $options['y'], null, $options['jpgQuality']);
				}
				break;

			case 'fitsquare':
				return (bool) $this->imgSquareFit($src, $options['width'], $options['height'], 'center', 'middle', $options['bgcolor'], null, $options['jpgQuality']);

		}

	    return false;
	}

	/**
	 * Convert Video To Image by ffmpeg.
	 *
	 * @param  $file video source file path
	 * @param  $stat file stat array
	 * @return bool
	 * @author Naoki Sawada
	 */
	public function ffmpegToImg($file, $stat)

ffmpegToImg uses the super-global variable $GLOBALS which is generally not recommended.

	{
	    $name = basename($file);
	    $path = dirname($file);
	    $tmp = $path.DIRECTORY_SEPARATOR.md5($name);
	    $GLOBALS['elFinderTempFiles'][] = $tmp; // regist to remove at the end
		if (rename($file, $tmp)) {
		    // specific start time by file name (xxx^[sec].[extention] - video^3.mp4)
			if (preg_match('/\^(\d+(?:\.\d+)?)\.[^.]+$/', $stat['name'], $_m)) {
			    $ss = $_m[1];
			} else {
			    $ss = $this->options['tmbVideoConvSec'];
			}
		    $cmd = sprintf('ffmpeg -i %s -ss 00:00:%.3f -vframes 1 -f image2 %s', escapeshellarg($tmp), $ss, escapeshellarg($file));
		    $r = $this->procExec($cmd);
		    unlink($tmp);

		    return $r === 0;
		}

	    return false;
	}

	/**
	 * Return new unique name based on file name and suffix.
	 *
	 * @param $dir
	 * @param $name
	 * @param  string $suffix suffix append to name
	 * @param bool $checkNum
	 * @param int $start
	 * @return string
	 * @internal param string $path file path
	 * @author Dmitry (dio) Levashov
	 */
	public function uniqueName($dir, $name, $suffix = ' copy', $checkNum = true, $start = 1)
	{
	    $ext = '';

	    if (preg_match('/\.((tar\.(gz|bz|bz2|z|lzo))|cpio\.gz|ps\.gz|xcf\.(gz|bz2)|[a-z0-9]{1,4})$/i', $name, $m)) {
	        $ext = '.'.$m[1];
	        $name = substr($name, 0, strlen($name) - strlen($m[0]));
	    }

	    if ($checkNum && preg_match('/('.preg_quote($suffix, '/').')(\d*)$/i', $name, $m)) {
	        $i = (int) $m[2];
	        $name = substr($name, 0, strlen($name) - strlen($m[2]));
	    } else {
	        $i = $start;
	        $name .= $suffix;
	    }
	    $max = $i + 100000;

	    while ($i <= $max) {
	        $n = $name.($i > 0 ? sprintf($this->options['uniqueNumFormat'], $i) : '').$ext;

	        if (! $this->isNameExists($this->joinPathCE($dir, $n))) {
	            $this->clearcache();

	            return $n;
	        }
	        $i++;
	    }

	    return $name.md5($dir).$ext;
	}

	/**
	 * Converts character encoding from UTF-8 to server's one.
	 *
	 * @param  mixed  $var           target string or array var
	 * @param  bool   $restoreLocale do retore global locale, default is false
	 * @param  string $unknown       replaces character for unknown
	 * @return mixed
	 * @author Naoki Sawada
	 */
	public function convEncIn($var = null, $restoreLocale = false, $unknown = '_')
	{
	    return (! $this->encoding) ? $var : $this->convEnc($var, 'UTF-8', $this->encoding, $this->options['locale'], $restoreLocale, $unknown);
	}

	/**
	 * Converts character encoding from server's one to UTF-8.
	 *
	 * @param  mixed  $var           target string or array var
	 * @param  bool   $restoreLocale do retore global locale, default is true
	 * @param  string $unknown       replaces character for unknown
	 * @return mixed
	 * @author Naoki Sawada
	 */
	public function convEncOut($var = null, $restoreLocale = true, $unknown = '_')
	{
	    return (! $this->encoding) ? $var : $this->convEnc($var, $this->encoding, 'UTF-8', $this->options['locale'], $restoreLocale, $unknown);
	}

	/**
	 * Get image size array with `dimensions`.
	 *
	 * @param string $path path need convert encoding to server encoding
	 * @param string $mime file mime type
	 * @return array|false
	 */
	public function getImageSize($path, $mime = '')
	{
	    $size = false;
	    if ($mime === '' || strtolower(substr($mime, 0, 5)) === 'image') {
	        if ($work = $this->getWorkFile($path)) {
	            if ($size = getimagesize($work)) {
	                $size['dimensions'] = $size[0].'x'.$size[1];
	            }
	        }
	        is_file($work) && unlink($work);
	    }

	    return $size;
	}

	/**
	 * Remove directory recursive on local file system.
	 *
	 * @param string $dir Target dirctory path
	 * @return bool
	 * @author Naoki Sawada
	 */
	public function rmdirRecursive($dir)
	{
	    return self::localRmdirRecursive($dir);
	}

	/*********************************************************************/
	/*                            INITIALIZATION                         */
	/*********************************************************************/

	/**
	 * Prepare driver before mount volume.
	 * Return true if volume is ready.
	 *
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function init()
	{
	    return true;
	}

	/**
	 * Configure after successfull mount.
	 * By default set thumbnails path and image manipulation library.
	 *
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	protected function configure()
	{
	    // set thumbnails path
		$path = $this->options['tmbPath'];
	    if ($path) {
	        if (! file_exists($path)) {

This code seems to be duplicated across your project.

	            if (mkdir($path)) {
	                chmod($path, $this->options['tmbPathMode']);
	            } else {
	                $path = '';
	            }
	        }

	        if (is_dir($path) && is_readable($path)) {
	            $this->tmbPath = $path;
	            $this->tmbPathWritable = is_writable($path);
	        }
	    }
		// set resouce path
		if (! is_dir($this->options['resourcePath'])) {
		    $this->options['resourcePath'] = dirname(__FILE__).DIRECTORY_SEPARATOR.'resources';
		}

		// set image manipulation library
		$type = preg_match('/^(imagick|gd|convert|auto)$/i', $this->options['imgLib'])
			? strtolower($this->options['imgLib'])
			: 'auto';

	    $imgLibFallback = extension_loaded('imagick') ? 'imagick' : (function_exists('gd_info') ? 'gd' : '');

$imgLibFallback is not used, you could remove the assignment.

	    if (($type === 'imagick' || $type === 'auto') && extension_loaded('imagick')) {
	        $this->imgLib = 'imagick';
	    } elseif (($type === 'gd' || $type === 'auto') && function_exists('gd_info')) {
	        $this->imgLib = 'gd';
	    } else {
	        $convertCache = 'imgLibConvert';
	        if (($convertCmd = $this->session->get($convertCache, false)) !== false) {
	            $this->imgLib = $convertCmd;
	        } else {
	            $this->imgLib = ($this->procExec('convert -version') === 0) ? 'convert' : '';
	            $this->session->set($convertCache, $this->imgLib);
	        }
	    }
	    if ($type !== 'auto' && $this->imgLib === '') {
	        // fallback
			$this->imgLib = extension_loaded('imagick') ? 'imagick' : (function_exists('gd_info') ? 'gd' : '');
	    }

		// check video to img converter
		if (! empty($this->options['imgConverter']) && is_array($this->options['imgConverter'])) {

This code seems to be duplicated across your project.

		    foreach ($this->options['imgConverter'] as $_type => $_converter) {
		        if (isset($_converter['func'])) {
		            $this->imgConverter[strtolower($_type)] = $_converter;
		        }
		    }
		}
	    if (! isset($this->imgConverter['video'])) {
	        $videoLibCache = 'videoLib';
	        if (($videoLibCmd = $this->session->get($videoLibCache, false)) === false) {
	            $videoLibCmd = ($this->procExec('ffmpeg -version') === 0) ? 'ffmpeg' : '';
	            $this->session->set($videoLibCache, $videoLibCmd);
	        }
	        if ($videoLibCmd) {
	            $this->imgConverter['video'] = [
					'func' => [$this, $videoLibCmd.'ToImg'],
					'maxlen' => $this->options['tmbVideoConvLen'],
				];
	        }
	    }

		// check archivers
		if (empty($this->archivers['create'])) {
		    $this->disabled[] = 'archive';
		}
	    if (empty($this->archivers['extract'])) {
	        $this->disabled[] = 'extract';
	    }
	    $_arc = $this->getArchivers();
	    if (empty($_arc['create'])) {
	        $this->disabled[] = 'zipdl';
	    }

		// check 'statOwner' for command `chmod`
		if (empty($this->options['statOwner'])) {
		    $this->disabled[] = 'chmod';
		}

		// check 'mimeMap'
		if (! is_array($this->options['mimeMap'])) {
		    $this->options['mimeMap'] = [];
		}
	    if (is_array($this->options['staticMineMap']) && $this->options['staticMineMap']) {

This code seems to be duplicated across your project.

	        $this->options['mimeMap'] = array_merge($this->options['mimeMap'], $this->options['staticMineMap']);
	    }
	    if (is_array($this->options['additionalMimeMap']) && $this->options['additionalMimeMap']) {

This code seems to be duplicated across your project.

	        $this->options['mimeMap'] = array_merge($this->options['mimeMap'], $this->options['additionalMimeMap']);
	    }

		// check 'url' in disabled commands
		if (in_array('url', $this->disabled)) {
		    $this->disabledGetUrl = true;

The property $disabledGetUrl was declared of type string, but true is of type boolean. Maybe add a type cast?

		}

		// set run time setting uploadOverwrite
		$this->uploadOverwrite = $this->options['uploadOverwrite'];
	}

	/**
	 * @deprecated
	 */
	protected function sessionRestart()
	{
	    $this->sessionCache = $this->session->start()->get($this->id, []);

It seems like $this->session->start()->get($this->id, array()) of type * is incompatible with the declared type array of property $sessionCache.

	    return true;
	}

	/**
	 * Save error message.
	 *
	 * @param  array  error
	 * @return false
	 * @author Naoki Sawada
	 **/
	protected function setError()
	{
	    $this->error = [];
	    $this->addError(func_get_args());

	    return false;
	}

	/**
	 * Add error message.
	 *
	 * @param  array  error
	 * @return false
	 * @author Dmitry(dio) Levashov
	 **/
	protected function addError()
	{
	    foreach (func_get_args() as $err) {
	        if (is_array($err)) {
	            $this->error = array_merge($this->error, $err);
	        } else {
	            $this->error[] = $err;
	        }
	    }

	    return false;
	}

	/*********************************************************************/
	/*                               FS API                              */
	/*********************************************************************/

	/***************** server encoding support *******************/

	/**
	 * Return parent directory path (with convert encoding).
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function dirnameCE($path)
	{
	    $dirname = (! $this->encoding) ? $this->_dirname($path) : $this->convEncOut($this->_dirname($this->convEncIn($path)));
		// check to infinite loop prevention
		return ($dirname != $path) ? $dirname : '';
	}

	/**
	 * Return file name (with convert encoding).
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function basenameCE($path)
	{
	    return (! $this->encoding) ? $this->_basename($path) : $this->convEncOut($this->_basename($this->convEncIn($path)));
	}

	/**
	 * Join dir name and file name and return full path. (with convert encoding)
	 * Some drivers (db) use int as path - so we give to concat path to driver itself.
	 *
	 * @param  string  $dir   dir path
	 * @param  string  $name  file name
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function joinPathCE($dir, $name)
	{
	    return (! $this->encoding) ? $this->_joinPath($dir, $name) : $this->convEncOut($this->_joinPath($this->convEncIn($dir), $this->convEncIn($name)));
	}

	/**
	 * Return normalized path (with convert encoding).
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function normpathCE($path)
	{
	    return (! $this->encoding) ? $this->_normpath($path) : $this->convEncOut($this->_normpath($this->convEncIn($path)));
	}

	/**
	 * Return file path related to root dir (with convert encoding).
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function relpathCE($path)
	{
	    return (! $this->encoding) ? $this->_relpath($path) : $this->convEncOut($this->_relpath($this->convEncIn($path)));
	}

	/**
	 * Convert path related to root dir into real path (with convert encoding).
	 *
	 * @param  string  $path  rel file path
	 * @return string
	 * @author Naoki Sawada
	 **/
	protected function abspathCE($path)
	{
	    return (! $this->encoding) ? $this->_abspath($path) : $this->convEncOut($this->_abspath($this->convEncIn($path)));
	}

	/**
	 * Return true if $path is children of $parent (with convert encoding).
	 *
	 * @param  string  $path    path to check
	 * @param  string  $parent  parent path
	 * @return bool
	 * @author Naoki Sawada
	 **/
	protected function inpathCE($path, $parent)
	{
	    return (! $this->encoding) ? $this->_inpath($path, $parent) : $this->convEncOut($this->_inpath($this->convEncIn($path), $this->convEncIn($parent)));
	}

	/**
	 * Open file and return file pointer (with convert encoding).
	 *
	 * @param  string $path file path
	 * @param string $mode
	 * @return false|resource
	 * @internal param bool $write open file for writing
	 * @author Naoki Sawada
	 */
	protected function fopenCE($path, $mode = 'rb')
	{
	    return (! $this->encoding) ? $this->_fopen($path, $mode) : $this->convEncOut($this->_fopen($this->convEncIn($path), $mode));
	}

	/**
	 * Close opened file (with convert encoding).
	 *
	 * @param  resource  $fp    file pointer
	 * @param  string    $path  file path
	 * @return bool
	 * @author Naoki Sawada
	 **/
	protected function fcloseCE($fp, $path = '')
	{
	    return (! $this->encoding) ? $this->_fclose($fp, $path) : $this->convEncOut($this->_fclose($fp, $this->convEncIn($path)));
	}

	/**
	 * Create new file and write into it from file pointer. (with convert encoding)
	 * 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
	 * @author Naoki Sawada
	 **/
	protected function saveCE($fp, $dir, $name, $stat)
	{
	    return (! $this->encoding) ? $this->_save($fp, $dir, $name, $stat) : $this->convEncOut($this->_save($fp, $this->convEncIn($dir), $this->convEncIn($name), $this->convEncIn($stat)));
	}

	/**
	 * Return true if path is dir and has at least one childs directory (with convert encoding).
	 *
	 * @param  string  $path  dir path
	 * @return bool
	 * @author Naoki Sawada
	 **/
	protected function subdirsCE($path)
	{
	    if ($this->sessionCaching['subdirs']) {

This code seems to be duplicated across your project.

	        if (isset($this->sessionCache['subdirs'][$path]) && ! $this->isMyReload()) {
	            return $this->sessionCache['subdirs'][$path];
	        }
	    }
	    $hasdir = (bool) ((! $this->encoding) ? $this->_subdirs($path) : $this->convEncOut($this->_subdirs($this->convEncIn($path))));
	    $this->updateSubdirsCache($path, $hasdir);

	    return $hasdir;
	}

	/**
	 * Return files list in directory (with convert encoding).
	 *
	 * @param  string  $path  dir path
	 * @return array
	 * @author Naoki Sawada
	 **/
	protected function scandirCE($path)
	{
	    return (! $this->encoding) ? $this->_scandir($path) : $this->convEncOut($this->_scandir($this->convEncIn($path)));
	}

	/**
	 * Create symlink (with convert encoding).
	 *
	 * @param  string  $source     file to link to
	 * @param  string  $targetDir  folder to create link in
	 * @param  string  $name       symlink name
	 * @return bool
	 * @author Naoki Sawada
	 **/
	protected function symlinkCE($source, $targetDir, $name)
	{
	    return (! $this->encoding) ? $this->_symlink($source, $targetDir, $name) : $this->convEncOut($this->_symlink($this->convEncIn($source), $this->convEncIn($targetDir), $this->convEncIn($name)));
	}

	/***************** paths *******************/

	/**
	 * Encode path into hash.
	 *
	 * @param  string  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 * @author Troex Nevelin
	 **/
	protected function encode($path)
	{
	    if ($path !== '') {

			// cut ROOT from $path for security reason, even if hacker decodes the path he will not know the root
			$p = $this->relpathCE($path);
			// if reqesting root dir $path will be empty, then assign '/' as we cannot leave it blank for crypt
			if ($p === '') {
			    $p = $this->separator;
			}

			// TODO crypt path and return hash
			$hash = $this->crypt($p);
			// hash is used as id in HTML that means it must contain vaild chars
			// make base64 html safe and append prefix in begining
			$hash = strtr(base64_encode($hash), '+/=', '-_.');
			// remove dots '.' at the end, before it was '=' in base64
			$hash = rtrim($hash, '.');
			// append volume id to make hash unique
			return $this->id.$hash;
	    }
	    //TODO: Add return statement here
	}

	/**
	 * Decode path from hash.
	 *
	 * @param  string  file hash
	 * @return string
	 * @author Dmitry (dio) Levashov
	 * @author Troex Nevelin
	 **/
	protected function decode($hash)
	{
	    if (strpos($hash, $this->id) === 0) {
	        // cut volume id after it was prepended in encode
			$h = substr($hash, strlen($this->id));
			// replace HTML safe base64 to normal
			$h = base64_decode(strtr($h, '-_.', '+/='));
			// TODO uncrypt hash and return path
			$path = $this->uncrypt($h);
			// append ROOT to path after it was cut in encode
			return $this->abspathCE($path); //$this->root.($path === $this->separator ? '' : $this->separator.$path);

54% of this comment could be valid code. Did you maybe forget this after debugging?

	    }
	    //TODO: Add return statement here
	}

	/**
	 * Return crypted path
	 * Not implemented.
	 *
	 * @param  string  path
	 * @return mixed
	 * @author Dmitry (dio) Levashov
	 **/
	protected function crypt($path)
	{
	    return $path;
	}

	/**
	 * Return uncrypted path
	 * Not implemented.
	 *
	 * @param  mixed  hash
	 * @return mixed
	 * @author Dmitry (dio) Levashov
	 **/
	protected function uncrypt($hash)
	{
	    return $hash;
	}

	/**
	 * Validate file name based on $this->options['acceptedName'] regexp or function.
	 *
	 * @param  string  $name  file name
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function nameAccepted($name, $isDir = false)
	{
	    if (json_encode($name) === false) {
	        return false;
	    }
	    $nameValidator = $isDir ? $this->dirnameValidator : $this->nameValidator;
	    if ($nameValidator) {
	        if (is_callable($nameValidator)) {
	            $res = call_user_func($nameValidator, $name);

	            return $res;
	        }
	        if (preg_match($nameValidator, '') !== false) {
	            return preg_match($nameValidator, $name);
	        }
	    }

	    return true;
	}

	/**
	 * Return session rootstat cache key.
	 *
	 * @return string
	 */
	protected function getRootstatCachekey()
	{
	    return md5($this->root.(isset($this->options['alias']) ? $this->options['alias'] : ''));
	}

	/**
	 * Converts character encoding (base function).
	 *
	 * @param  mixed $var target string or array var
	 * @param  string $from from character encoding
	 * @param  string $to to character encoding
	 * @param  string $locale local locale
	 * @param $restoreLocale
	 * @param  string $unknown replaces character for unknown
	 * @return mixed
	 */
	protected function convEnc($var, $from, $to, $locale, $restoreLocale, $unknown = '_')
	{
	    if (strtoupper($from) !== strtoupper($to)) {
	        if ($locale) {
	            setlocale(LC_ALL, $locale);
	        }
	        if (is_array($var)) {
	            $_ret = [];
	            foreach ($var as $_k => $_v) {
	                $_ret[$_k] = $this->convEnc($_v, $from, $to, '', false, $unknown = '_');
	            }
	            $var = $_ret;
	        } else {
	            $_var = false;
	            if (is_string($var)) {
	                $_var = $var;
	                if (false !== ($_var = iconv($from, $to.'//TRANSLIT', $_var))) {
	                    $_var = str_replace('?', $unknown, $_var);
	                }
	            }
	            if ($_var !== false) {
	                $var = $_var;
	            }
	        }
	        if ($restoreLocale) {
	            setlocale(LC_ALL, elFinder::$locale);
	        }
	    }

	    return $var;
	}

	/*********************** util mainly for inheritance class *********************/

	/**
	 * Get temporary filename. Tempfile will be removed when after script execution finishes or exit() is called.
	 * When needing the unique file to a path, give $path to parameter.
	 *
	 * @param  string       $path for get unique file to a path
	 * @return string|false
	 * @author Naoki Sawada
	 */
	protected function getTempFile($path = '')
	{
	    static $cache = [];
	    static $rmfunc;

	    $key = '';
	    if ($path !== '') {
	        $key = $this->id.'#'.$path;
	        if (isset($cache[$key])) {
	            return $cache[$key];
	        }
	    }

	    if ($tmpdir = $this->getTempPath()) {
	        if (! $rmfunc) {
	            $rmfunc = create_function('$f', 'is_file($f) && unlink($f);');

The use of create_function is highly discouraged, better use a closure.

	        }
	        $name = tempnam($tmpdir, 'ELF');
	        if ($key) {
	            $cache[$key] = $name;
	        }
	        register_shutdown_function($rmfunc, $name);

	        return $name;
	    }

	    return false;
	}

	/**
	 * File path of local server side work file path.
	 *
	 * @param  string $path path need convert encoding to server encoding
	 * @return string
	 * @author Naoki Sawada
	 */
	protected function getWorkFile($path)
	{
	    if ($work = $this->getTempFile()) {
	        if ($wfp = fopen($work, 'wb')) {
	            if ($fp = $this->_fopen($path)) {
	                while (! feof($fp)) {
	                    fwrite($wfp, fread($fp, 8192));
	                }
	                $this->_fclose($fp, $path);
	                fclose($wfp);

	                return $work;
	            }
	        }
	    }

	    return false;
	}

	/**
	 * Delete dirctory trees.
	 *
	 * @param string $localpath path need convert encoding to server encoding
	 * @return bool
	 * @author Naoki Sawada
	 */
	protected function delTree($localpath)
	{
	    foreach ($this->_scandir($localpath) as $p) {
	        elFinder::extendTimeLimit();
	        $stat = $this->stat($this->convEncOut($p));
	        $this->convEncIn();
	        ($stat['mime'] === 'directory') ? $this->delTree($p) : $this->_unlink($p);
	    }

	    return $this->_rmdir($localpath);
	}

	/**
	 * Copy items to a new temporary directory on the local server.
	 *
	 * @param  array  $hashes  target hashes
	 * @param  string $dir     destination directory (for recurcive)
	 * @param  string $canLink it can use link() (for recurcive)
	 * @return string|false    saved path name
	 * @author Naoki Sawada
	 */
	protected function getItemsInHand($hashes, $dir = null, $canLink = null)
	{
	    static $totalSize = 0;
	    if (is_null($dir)) {
	        $totalSize = 0;
	        if (! $tmpDir = $this->getTempPath()) {
	            return false;
	        }
	        $dir = tempnam($tmpDir, 'elf');
	        if (! unlink($dir) || ! mkdir($dir, 0700, true)) {
	            return false;
	        }
	        register_shutdown_function([$this, 'rmdirRecursive'], $dir);
	    }
	    if (is_null($canLink)) {
	        $canLink = ($this instanceof elFinderVolumeLocalFileSystem);
	    }
	    elFinder::extendTimeLimit();
	    $res = true;
	    $files = [];
	    foreach ($hashes as $hash) {
	        if (($file = $this->file($hash)) == false) {
	            continue;
	        }
	        if (! $file['read']) {
	            continue;
	        }

	        $name = $file['name'];
			// for call from search results
			if (isset($files[$name])) {
			    $name = preg_replace('/^(.*?)(\..*)?$/', '$1_'.$files[$name]++.'$2', $name);
			} else {
			    $files[$name] = 1;
			}
	        $target = $dir.DIRECTORY_SEPARATOR.$name;

	        if ($file['mime'] === 'directory') {
	            $chashes = [];
	            $_files = $this->scandir($hash);
	            foreach ($_files as $_file) {

The expression $_files of type array|boolean is not guaranteed to be traversable. How about adding an additional type check?

	                if ($file['read']) {
	                    $chashes[] = $_file['hash'];
	                }
	            }
	            if (($res = mkdir($target, 0700, true)) && $chashes) {

The expression $chashes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	                $res = $this->getItemsInHand($chashes, $target, $canLink);

It seems like $canLink can also be of type boolean; however, elFinderVolumeDriver::getItemsInHand() does only seem to accept string|null, maybe add an additional type check?

	            }
	            if (! $res) {
	                break;
	            }
	            ! empty($file['ts']) && touch($target, $file['ts']);
	        } else {
	            $path = $this->decode($hash);
	            if (! $canLink || ! ($canLink = link($path, $target))) {
	                if ($fp = $this->fopenCE($path)) {
	                    if ($tfp = fopen($target, 'wb')) {
	                        $totalSize += stream_copy_to_stream($fp, $tfp);
	                        fclose($tfp);
	                    }
	                    ! empty($file['ts']) && touch($target, $file['ts']);
	                    $this->fcloseCE($fp, $path);
	                }
	            } else {
	                $totalSize += filesize($path);
	            }
	            if ($this->options['maxArcFilesSize'] > 0 && $this->options['maxArcFilesSize'] < $totalSize) {
	                $res = $this->setError(elFinder::ERROR_ARC_MAXSIZE);
	            }
	        }
	    }

	    return $res ? $dir : false;
	}

	/*********************** file stat *********************/

	/**
	 * Check file attribute.
	 *
	 * @param  string  $path  file path
	 * @param  string  $name  attribute name (read|write|locked|hidden)
	 * @param  bool    $val   attribute value returned by file system
	 * @param  bool    $isDir path is directory (true: directory, false: file)
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function attr($path, $name, $val = null, $isDir = null)
	{
	    if (! isset($this->defaults[$name])) {
	        return false;
	    }

	    $relpath = $this->relpathCE($path);
	    if ($this->separator !== '/') {
	        $relpath = str_replace($this->separator, '/', $relpath);
	    }
	    $relpath = '/'.$relpath;

	    $perm = null;

	    if ($this->access) {
	        $perm = call_user_func($this->access, $name, $path, $this->options['accessControlData'], $this, $isDir, $relpath);
	        if ($perm !== null) {
	            return (bool) $perm;
	        }
	    }

	    for ($i = 0, $c = count($this->attributes); $i < $c; $i++) {
	        $attrs = $this->attributes[$i];

	        if (isset($attrs[$name]) && isset($attrs['pattern']) && preg_match($attrs['pattern'], $relpath)) {
	            $perm = $attrs[$name];
	        }
	    }

	    return $perm === null ? (is_null($val) ? $this->defaults[$name] : $val) : (bool) $perm;
	}

	/**
	 * Return true if file with given name can be created in given folder.
	 *
	 * @param string $dir parent dir path
	 * @param string $name new file name
	 * @param null $isDir
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 */
	protected function allowCreate($dir, $name, $isDir = null)
	{
	    return $this->attr($this->joinPathCE($dir, $name), 'write', true, $isDir);
	}

	/**
	 * Return true if file MIME type can save with check uploadOrder config.
	 *
	 * @param string $mime
	 * @return bool
	 */
	protected function allowPutMime($mime)
	{
	    // logic based on http://httpd.apache.org/docs/2.2/mod/mod_authz_host.html#order
		$allow = $this->mimeAccepted($mime, $this->uploadAllow, null);
	    $deny = $this->mimeAccepted($mime, $this->uploadDeny, null);
	    $res = true; // default to allow

$res is not used, you could remove the assignment.

		if (strtolower($this->uploadOrder[0]) == 'allow') { // array('allow', 'deny'), default is to 'deny'

53% of this comment could be valid code. Did you maybe forget this after debugging?

			$res = false; // default is deny
			if (! $deny && ($allow === true)) { // match only allow

The expression $deny of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

				$res = true;
			}// else (both match | no match | match only deny) { deny }
		} else { // array('deny', 'allow'), default is to 'allow' - this is the default rule
			$res = true; // default is allow
			if (($deny === true) && ! $allow) { // match only deny

The expression $allow of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

				$res = false;
			} // else (both match | no match | match only allow) { allow }
		}

	    return $res;
	}

	/**
	 * Return fileinfo.
	 *
	 * @param  string  $path  file cache
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	protected function stat($path)
	{
	    if ($path === false || is_null($path)) {
	        return false;
	    }
	    $is_root = ($path == $this->root);
	    if ($is_root) {
	        $rootKey = $this->getRootstatCachekey();
	        if ($this->sessionCaching['rootstat'] && ! isset($this->sessionCache['rootstat'])) {
	            $this->sessionCache['rootstat'] = [];
	        }
	        if (! isset($this->cache[$path]) && ! $this->isMyReload()) {
	            // need $path as key for netmount/netunmount
				if ($this->sessionCaching['rootstat'] && isset($this->sessionCache['rootstat'][$rootKey])) {
				    if ($ret = $this->sessionCache['rootstat'][$rootKey]) {
				        if ($this->options['rootRev'] === $ret['rootRev']) {
				            if (isset($this->options['phash'])) {

This code seems to be duplicated across your project.

				                $ret['isroot'] = 1;
				                $ret['phash'] = $this->options['phash'];
				            }

				            return $ret;
				        }
				    }
				}
	        }
	    }
	    $ret = isset($this->cache[$path])
			? $this->cache[$path]
			: $this->updateCache($path, $this->convEncOut($this->_stat($this->convEncIn($path))));
	    if ($is_root && $this->sessionCaching['rootstat']) {
	        if ($ret) {
	            $this->rootModified = false;
	            $this->sessionCache['rootstat'][$rootKey] = $ret;

The variable $rootKey does not seem to be defined for all execution paths leading up to this point.

	            if (isset($this->options['phash'])) {

This code seems to be duplicated across your project.

	                $ret['isroot'] = 1;
	                $ret['phash'] = $this->options['phash'];
	            }
	        } else {
	            unset($this->sessionCache['rootstat'][$rootKey]);
	        }
	    }

	    return $ret;
	}

	/**
	 * Get root stat extra key values.
	 *
	 * @return array stat extras
	 * @author Naoki Sawada
	 */
	protected function getRootStatExtra()
	{
	    $stat = [];
	    if ($this->rootName) {
	        $stat['name'] = $this->rootName;
	    }
	    $stat['rootRev'] = $this->options['rootRev'];
	    $stat['options'] = $this->options(null);

	    return $stat;
	}

	/**
	 * Return fileinfo based on filename
	 * For item ID based path file system
	 * Please override if needed on each drivers.
	 *
	 * @param  string  $path  file cache
	 * @return array
	 */
	protected function isNameExists($path)
	{
	    return $this->stat($path);
	}

	/**
	 * Put file stat in cache and return it.
	 *
	 * @param  string  $path   file path
	 * @param  array   $stat   file stat
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	protected function updateCache($path, $stat)
	{
	    if (empty($stat) || ! is_array($stat)) {
	        return $this->cache[$path] = [];
	    }

	    $stat['hash'] = $this->encode($path);

	    $root = $path == $this->root;
	    $parent = '';

	    if ($root) {
	        $stat = array_merge($stat, $this->getRootStatExtra());
	    } else {
	        if (! isset($stat['name']) || $stat['name'] === '') {
	            $stat['name'] = $this->basenameCE($path);
	        }
	        if (empty($stat['phash'])) {
	            $parent = $this->dirnameCE($path);
	            $stat['phash'] = $this->encode($parent);
	        } else {
	            $parent = $this->decode($stat['phash']);
	        }
	    }

		// name check
		if (! $jeName = json_encode($stat['name'])) {
		    return $this->cache[$path] = [];
		}
		// fix name if required
		if ($this->options['utf8fix'] && $this->options['utf8patterns'] && $this->options['utf8replace']) {
		    $stat['name'] = json_decode(str_replace($this->options['utf8patterns'], $this->options['utf8replace'], $jeName));
		}

	    if (empty($stat['mime'])) {
	        $stat['mime'] = $this->mimetype($stat['name'], true);
	    }

		// @todo move dateformat to client
		// $stat['date'] = isset($stat['ts'])

74% of this comment could be valid code. Did you maybe forget this after debugging?

		// 	? $this->formatDate($stat['ts'])

75% of this comment could be valid code. Did you maybe forget this after debugging?

		// 	: 'unknown';

		if (! isset($stat['size'])) {
		    $stat['size'] = 'unknown';
		}

	    if ($isDir = ($stat['mime'] === 'directory')) {
	        $stat['volumeid'] = $this->id;
	    }

	    $stat['read'] = intval($this->attr($path, 'read', isset($stat['read']) ? (bool) $stat['read'] : null, $isDir));
	    $stat['write'] = intval($this->attr($path, 'write', isset($stat['write']) ? (bool) $stat['write'] : null, $isDir));
	    if ($root) {
	        $stat['locked'] = 1;
	        if ($this->options['type'] !== '') {
	            $stat['type'] = $this->options['type'];
	        }
	    } else {
	        // lock when parent directory is not writable
			if (! isset($stat['locked'])) {
			    $pstat = $this->stat($parent);
			    if (isset($pstat['write']) && ! $pstat['write']) {
			        $stat['locked'] = true;
			    }
			}
	        if ($this->attr($path, 'locked', isset($stat['locked']) ? (bool) $stat['locked'] : null, $isDir)) {
	            $stat['locked'] = 1;
	        } else {
	            unset($stat['locked']);
	        }
	    }

	    if ($root) {
	        unset($stat['hidden']);
	    } elseif ($this->attr($path, 'hidden', isset($stat['hidden']) ? (bool) $stat['hidden'] : null, $isDir)
		|| ! $this->mimeAccepted($stat['mime'])) {

The expression $this->mimeAccepted($stat['mime']) of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

	        $stat['hidden'] = 1;
	    } else {
	        unset($stat['hidden']);
	    }

	    if ($stat['read'] && empty($stat['hidden'])) {
	        if ($isDir) {
	            // caching parent's subdirs
				if ($parent) {

The expression $parent of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

				    $this->updateSubdirsCache($parent, true);
				}
				// for dir - check for subdirs
				if ($this->options['checkSubfolders']) {
				    if (! isset($stat['dirs']) && intval($this->options['checkSubfolders']) === -1) {
				        $stat['dirs'] = -1;
				    }
				    if (isset($stat['dirs'])) {
				        if ($stat['dirs']) {
				            if ($stat['dirs'] == -1) {
				                $stat['dirs'] = ($this->sessionCaching['subdirs'] && isset($this->sessionCache['subdirs'][$path])) ? (int) $this->sessionCache['subdirs'][$path] : -1;
				            } else {
				                $stat['dirs'] = 1;
				            }
				        } else {
				            unset($stat['dirs']);
				        }
				    } elseif (! empty($stat['alias']) && ! empty($stat['target'])) {
				        $stat['dirs'] = isset($this->cache[$stat['target']])
							? intval(isset($this->cache[$stat['target']]['dirs']))
							: $this->subdirsCE($stat['target']);
				    } elseif ($this->subdirsCE($path)) {
				        $stat['dirs'] = 1;
				    }
				} else {
				    $stat['dirs'] = 1;
				}
	            if ($this->options['dirUrlOwn'] === true) {
	                $stat['url'] = '#elf_'.$stat['hash'];
	            }
	        } else {
	            // for files - check for thumbnails
				$p = isset($stat['target']) ? $stat['target'] : $path;
	            if ($this->tmbURL && ! isset($stat['tmb']) && $this->canCreateTmb($p, $stat)) {
	                $tmb = $this->gettmb($p, $stat);
	                $stat['tmb'] = $tmb ? $tmb : 1;
	            }
	        }
	        if (! isset($stat['url']) && $this->URL && $this->encoding) {
	            $_path = str_replace($this->separator, '/', substr($path, strlen($this->root) + 1));
	            $stat['url'] = rtrim($this->URL, '/').'/'.str_replace('%2F', '/', rawurlencode((substr(PHP_OS, 0, 3) === 'WIN') ? $_path : $this->convEncIn($_path, true)));
	        }
	    } else {
	        if ($isDir) {
	            unset($stat['dirs']);
	        }
	    }

	    if (! empty($stat['alias']) && ! empty($stat['target'])) {
	        $stat['thash'] = $this->encode($stat['target']);
			//$this->cache[$stat['target']] = $stat;

72% of this comment could be valid code. Did you maybe forget this after debugging?

			unset($stat['target']);
	    }

	    $this->cache[$path] = $stat;

	    if ($root && $this->sessionCaching['rootstat']) {
	        // to update session cache
			$this->stat($path);
	    }

	    return $stat;
	}

	/**
	 * Get stat for folder content and put in cache.
	 *
	 * @param  string  $path
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	protected function cacheDir($path)
	{
	    $this->dirsCache[$path] = [];
	    $hasDir = false;

	    foreach ($this->scandirCE($path) as $p) {
	        if (($stat = $this->stat($p)) && empty($stat['hidden'])) {
	            if (! $hasDir && $stat['mime'] === 'directory') {
	                $hasDir = true;
	            }
	            $this->dirsCache[$path][] = $p;
	        }
	    }

	    $this->updateSubdirsCache($path, $hasDir);
	}

	/**
	 * Clean cache.
	 *
	 * @return void
	 * @author Dmitry (dio) Levashov
	 **/
	protected function clearcache()
	{
	    $this->cache = $this->dirsCache = [];
	}

	/**
	 * Return file mimetype.
	 *
	 * @param  string      $path file path
	 * @param  string|bool $name
	 * @return string
	 * @author Dmitry (dio) Levashov
	 */
	protected function mimetype($path, $name = '')
	{
	    $type = '';
	    $nameCheck = false;

	    if ($name === '') {
	        $name = $path;
	    } elseif ($name === true) {
	        $name = $path;
	        $nameCheck = true;
	    }
	    $ext = (false === $pos = strrpos($name, '.')) ? '' : substr($name, $pos + 1);
	    if (! $nameCheck && is_readable($path)) {
	        if (filesize($path) > 0) {
	            if ($this->mimeDetect == 'finfo') {
	                if ($type = finfo_file($this->finfo, $path)) {
	                    if ($ext && preg_match('~^application/(?:octet-stream|(?:x-)?zip)~', $type)) {
	                        if (isset(self::$mimetypes[$ext])) {
	                            $type = self::$mimetypes[$ext];
	                        }
	                    } elseif ($ext === 'js' && preg_match('~^text/~', $type)) {
	                        $type = 'text/javascript';
	                    }
	                } else {
	                    $type = 'unknown';
	                }
	            } elseif ($this->mimeDetect == 'mime_content_type') {
	                $type = mime_content_type($path);
	            }
	        } else {
	            $type = 'text/plain';
	        }
	    }
	    if (! $type) {
	        $type = self::mimetypeInternalDetect($path);
	    }

	    if ($type === 'unknown' && $this->mimeDetect != 'internal') {
	        $type = self::mimetypeInternalDetect($path);
	    }

	    $type = explode(';', $type);
	    $type = trim($type[0]);

	    if (in_array($type, ['application/x-empty', 'inode/x-empty'])) {
	        // finfo return this mime for empty files
			$type = 'text/plain';
	    }

		// mime type normalization
		$_checkKey = strtolower($ext.':'.$type);
	    if (isset($this->options['mimeMap'][$_checkKey])) {
	        $type = $this->options['mimeMap'][$_checkKey];
	    } else {
	        $_checkKey = strtolower($ext.':*');
	        if (isset($this->options['mimeMap'][$_checkKey])) {
	            $type = $this->options['mimeMap'][$_checkKey];
	        }
	    }

	    return $type;
	}

	/**
	 * Detect file mimetype using "internal" method or Loading mime.types with $path = ''.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	protected static function mimetypeInternalDetect($path = '')
	{
	    // load default MIME table file "mime.types"
		if (! self::$mimetypesLoaded) {
		    self::$mimetypesLoaded = true;
		    $file = elFinder::$defaultMimefile;
		    if ($file === '' || ! is_readable($file)) {
		        $file = dirname(__FILE__).DIRECTORY_SEPARATOR.'mime.types';
		    }
		    if (is_readable($file)) {

This code seems to be duplicated across your project.

		        $mimecf = file($file);
		        foreach ($mimecf as $line_num => $line) {
		            if (! preg_match('/^\s*#/', $line)) {
		                $mime = preg_split('/\s+/', $line, -1, PREG_SPLIT_NO_EMPTY);
		                for ($i = 1, $size = count($mime); $i < $size; $i++) {
		                    if (! isset(self::$mimetypes[$mime[$i]])) {
		                        self::$mimetypes[$mime[$i]] = $mime[0];
		                    }
		                }
		            }
		        }
		    }
		}
	    $ext = '';
	    if ($path) {
	        $pinfo = pathinfo($path);
	        $ext = isset($pinfo['extension']) ? strtolower($pinfo['extension']) : '';
	    }

	    return ($ext && isset(self::$mimetypes[$ext])) ? self::$mimetypes[$ext] : 'unknown';
	}

	/**
	 * Return file/total directory size.
	 *
	 * @param  string  $path  file path
	 * @return int
	 * @author Dmitry (dio) Levashov
	 **/
	protected function countSize($path)
	{
	    elFinder::extendTimeLimit();

	    $result = ['size' => 0, 'files' => 0, 'dirs' => 0];
	    $stat = $this->stat($path);

	    if (empty($stat) || ! $stat['read'] || ! empty($stat['hidden'])) {
	        $result['size'] = 'unknown';

	        return $result;
	    }

	    if ($stat['mime'] != 'directory') {
	        $result['size'] = intval($stat['size']);
	        $result['files'] = 1;

	        return $result;
	    }

	    $subdirs = $this->options['checkSubfolders'];
	    $this->options['checkSubfolders'] = true;
	    foreach ($this->getScandir($path) as $stat) {
	        if ($isDir = ($stat['mime'] === 'directory' && $stat['read'])) {
	            ++$result['dirs'];
	        } else {
	            ++$result['files'];
	        }
	        $res = $isDir
				? $this->countSize($this->decode($stat['hash']))
				: (isset($stat['size']) ? ['size' => intval($stat['size'])] : []);
	        if (! empty($res['size']) && is_numeric($res['size'])) {

This code seems to be duplicated across your project.

	            $result['size'] += $res['size'];
	        }
	        if (! empty($res['files']) && is_numeric($res['files'])) {

This code seems to be duplicated across your project.

	            $result['files'] += $res['files'];
	        }
	        if (! empty($res['dirs']) && is_numeric($res['dirs'])) {

This code seems to be duplicated across your project.

	            $result['dirs'] += $res['dirs'];
	        }
	    }
	    $this->options['checkSubfolders'] = $subdirs;

	    return $result;
	}

	/**
	 * Return true if all mimes is directory or files.
	 *
	 * @param  string  $mime1  mimetype
	 * @param  string  $mime2  mimetype
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function isSameType($mime1, $mime2)
	{
	    return ($mime1 == 'directory' && $mime1 == $mime2) || ($mime1 != 'directory' && $mime2 != 'directory');
	}

	/**
	 * If file has required attr == $val - return file path,
	 * If dir has child with has required attr == $val - return child path.
	 *
	 * @param  string   $path  file path
	 * @param  string   $attr  attribute name
	 * @param  bool     $val   attribute value
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function closestByAttr($path, $attr, $val)
	{
	    $stat = $this->stat($path);

	    if (empty($stat)) {
	        return false;
	    }

	    $v = isset($stat[$attr]) ? $stat[$attr] : false;

	    if ($v == $val) {
	        return $path;
	    }

	    return $stat['mime'] == 'directory'
			? $this->childsByAttr($path, $attr, $val)
			: false;
	}

	/**
	 * Return first found children with required attr == $val.
	 *
	 * @param  string   $path  file path
	 * @param  string   $attr  attribute name
	 * @param  bool     $val   attribute value
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function childsByAttr($path, $attr, $val)
	{
	    foreach ($this->scandirCE($path) as $p) {
	        if (($_p = $this->closestByAttr($p, $attr, $val)) != false) {

It seems like you are loosely comparing $_p = $this->closestByAttr($p, $attr, $val) of type string|false against false; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

	            return $_p;
	        }
	    }

	    return false;
	}

    protected function isMyReload($target = '', $ARGtarget = '')
    {
        if ($this->rootModified || (! empty($this->ARGS['cmd']) && $this->ARGS['cmd'] === 'parents')) {
            return true;
        }
        if (! empty($this->ARGS['reload'])) {
            if ($ARGtarget === '') {
                $ARGtarget = isset($this->ARGS['target']) ? $this->ARGS['target']
					: ((isset($this->ARGS['targets']) && is_array($this->ARGS['targets']) && count($this->ARGS['targets']) === 1) ?
						$this->ARGS['targets'][0] : '');
            }
            if ($ARGtarget !== '') {
                $ARGtarget = strval($ARGtarget);
                if ($target === '') {
                    return strpos($ARGtarget, $this->id) === 0;
                } else {
                    $target = strval($target);

                    return $target === $ARGtarget;
                }
            }
        }

        return false;
    }

	/**
	 * Update subdirs cache data.
	 *
	 * @param string $path
	 * @param bool   $subdirs
	 *
	 * @returnv void
	 */
	protected function updateSubdirsCache($path, $subdirs)
	{
	    if (isset($this->cache[$path])) {
	        if ($subdirs) {
	            $this->cache[$path]['dirs'] = 1;
	        } else {
	            unset($this->cache[$path]['dirs']);
	        }
	    }
	    if ($this->sessionCaching['subdirs']) {
	        $this->sessionCache['subdirs'][$path] = $subdirs;
	    }
	    if ($this->sessionCaching['rootstat'] && $path == $this->root) {
	        unset($this->sessionCache['rootstat'][$this->getRootstatCachekey()]);
	    }
	}

	/*****************  get content *******************/

	/**
	 * Return required dir's files info.
	 * If onlyMimes is set - return only dirs and files of required mimes.
	 *
	 * @param  string  $path  dir path
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	protected function getScandir($path)
	{
	    $files = [];

	    ! isset($this->dirsCache[$path]) && $this->cacheDir($path);

	    foreach ($this->dirsCache[$path] as $p) {
	        if (($stat = $this->stat($p)) && empty($stat['hidden'])) {
	            $files[] = $stat;
	        }
	    }

	    return $files;
	}

	/**
	 * Return subdirs tree.
	 *
	 * @param  string $path parent dir path
	 * @param  int $deep tree deep
	 * @param string $exclude
	 * @return array
	 * @author Dmitry (dio) Levashov
	 */
	protected function gettree($path, $deep, $exclude = '')
	{
	    $dirs = [];

	    ! isset($this->dirsCache[$path]) && $this->cacheDir($path);

	    foreach ($this->dirsCache[$path] as $p) {
	        $stat = $this->stat($p);

	        if ($stat && empty($stat['hidden']) && $p != $exclude && $stat['mime'] == 'directory') {

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	            $dirs[] = $stat;
	            if ($deep > 0 && ! empty($stat['dirs'])) {
	                $dirs = array_merge($dirs, $this->gettree($p, $deep - 1));
	            }
	        }
	    }

	    return $dirs;
	}

	/**
	 * Recursive files search.
	 *
	 * @param  string  $path   dir path
	 * @param  string  $q      search string
	 * @param  array   $mimes
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	protected function doSearch($path, $q, $mimes)
	{
	    $result = [];

	    $timeout = $this->options['searchTimeout'] ? $this->searchStart + $this->options['searchTimeout'] : 0;
	    if ($timeout && $timeout < time()) {

This code seems to be duplicated across your project.

	        $this->setError(elFinder::ERROR_SEARCH_TIMEOUT, $this->path($this->encode($path)));

	        return $result;
	    }

	    foreach ($this->scandirCE($path) as $p) {
	        elFinder::extendTimeLimit($this->options['searchTimeout'] + 30);

	        if ($timeout && ($this->error || $timeout < time())) {

The expression $this->error of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This code seems to be duplicated across your project.

	            ! $this->error && $this->setError(elFinder::ERROR_SEARCH_TIMEOUT, $this->path($this->encode($path)));

The expression $this->error of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	            break;
	        }

	        $stat = $this->stat($p);

	        if (! $stat) { // invalid links

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

				continue;
	        }

	        if (! empty($stat['hidden']) || ! $this->mimeAccepted($stat['mime'], $mimes)) {
	            continue;
	        }

	        $name = $stat['name'];

	        if ($this->doSearchCurrentQuery['excludes']) {

This code seems to be duplicated across your project.

	            foreach ($this->doSearchCurrentQuery['excludes'] as $exclude) {

The expression $this->doSearchCurrentQuery['excludes'] of type string is not traversable.

	                if ($this->stripos($name, $exclude) !== false) {
	                    continue 2;
	                }
	            }
	        }

	        if ((! $mimes || $stat['mime'] !== 'directory') && $this->stripos($name, $q) !== false) {

The expression $mimes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	            $stat['path'] = $this->path($stat['hash']);
	            if ($this->URL && ! isset($stat['url'])) {
	                $path = str_replace($this->separator, '/', substr($p, strlen($this->root) + 1));
	                if ($this->encoding) {
	                    $path = str_replace('%2F', '/', rawurlencode($this->convEncIn($path, true)));
	                } else {
	                    $path = str_replace('%2F', '/', rawurlencode($path));
	                }
	                $stat['url'] = $this->URL.$path;
	            }

	            $result[] = $stat;
	        }
	        if ($stat['mime'] == 'directory' && $stat['read'] && ! isset($stat['alias'])) {
	            if (! $this->options['searchExDirReg'] || ! preg_match($this->options['searchExDirReg'], $p)) {
	                $result = array_merge($result, $this->doSearch($p, $q, $mimes));
	            }
	        }
	    }

	    return $result;
	}

	/**********************  manuipulations  ******************/

	/**
	 * Copy file/recursive copy dir only in current volume.
	 * Return new file path or false.
	 *
	 * @param  string  $src   source path
	 * @param  string  $dst   destination dir path
	 * @param  string  $name  new file name (optionaly)
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function copy($src, $dst, $name)
	{
	    elFinder::extendTimeLimit();

	    $srcStat = $this->stat($src);

	    if (! empty($srcStat['thash'])) {
	        $target = $this->decode($srcStat['thash']);
	        if (! $this->inpathCE($target, $this->root)) {
	            return $this->setError(elFinder::ERROR_COPY, $this->path($srcStat['hash']), elFinder::ERROR_MKOUTLINK);
	        }
	        $stat = $this->stat($target);
	        $this->clearcache();

	        return $stat && $this->symlinkCE($target, $dst, $name)

The expression $stat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

				? $this->joinPathCE($dst, $name)
				: $this->setError(elFinder::ERROR_COPY, $this->path($srcStat['hash']));
	    }

	    if ($srcStat['mime'] === 'directory') {
	        $testStat = $this->isNameExists($this->joinPathCE($dst, $name));
	        $this->clearcache();

	        if (($testStat && $testStat['mime'] !== 'directory') || (! $testStat && ! $testStat = $this->mkdir($this->encode($dst), $name))) {

The expression $testStat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

The expression $testStat of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This code seems to be duplicated across your project.

	            return $this->setError(elFinder::ERROR_COPY, $this->path($srcStat['hash']));
	        }

	        $dst = $this->decode($testStat['hash']);

	        foreach ($this->getScandir($src) as $stat) {
	            if (empty($stat['hidden'])) {
	                $name = $stat['name'];
	                $_src = $this->decode($stat['hash']);
	                if (! $this->copy($_src, $dst, $name)) {
	                    $this->remove($dst, true); // fall back
						return $this->setError($this->error, elFinder::ERROR_COPY, $this->_path($src));
	                }
	            }
	        }

	        $this->added[] = $testStat;

	        return $dst;
	    }

	    if ($this->options['copyJoin']) {

This code seems to be duplicated across your project.

	        $test = $this->joinPathCE($dst, $name);
	        if ($testStat = $this->isNameExists($test)) {

$testStat is not used, you could remove the assignment.

	            $this->remove($test);
	        }
	    } else {
	        $testStat = false;

$testStat is not used, you could remove the assignment.

	    }
	    if ($res = $this->convEncOut($this->_copy($this->convEncIn($src), $this->convEncIn($dst), $this->convEncIn($name)))) {
	        $path = is_string($res) ? $res : $this->joinPathCE($dst, $name);
	        $this->clearcache();
	        if ($this->ARGS['cmd'] !== 'duplicate') {
	            $this->added[] = $this->stat($path);
	        }

	        return $path;
	    }

	    return $this->setError(elFinder::ERROR_COPY, $this->path($srcStat['hash']));
	}

	/**
	 * Move file
	 * Return new file path or false.
	 *
	 * @param  string  $src   source path
	 * @param  string  $dst   destination dir path
	 * @param  string  $name  new file name
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function move($src, $dst, $name)
	{
	    $stat = $this->stat($src);
	    $stat['realpath'] = $src;
	    $this->rmTmb($stat); // can not do rmTmb() after _move()

$stat is of type array<string,string,{"realpath":"string"}>, but the function expects a string.

		$this->clearcache();

	    if ($res = $this->convEncOut($this->_move($this->convEncIn($src), $this->convEncIn($dst), $this->convEncIn($name)))) {
	        $this->removed[] = $stat;
	        if ($stat['mime'] === 'directory') {
	            $this->updateSubdirsCache($dst, true);
	        }

	        return is_string($res) ? $res : $this->joinPathCE($dst, $name);
	    }

	    return $this->setError(elFinder::ERROR_MOVE, $this->path($stat['hash']));
	}

	/**
	 * Copy file from another volume.
	 * Return new file path or false.
	 *
	 * @param  object  $volume       source volume
	 * @param  string  $src          source file hash
	 * @param  string  $destination  destination dir path
	 * @param  string  $name         file name
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function copyFrom($volume, $src, $destination, $name)
	{
	    elFinder::extendTimeLimit();

	    if (($source = $volume->file($src)) == false) {
	        return $this->setError(elFinder::ERROR_COPY, '#'.$src, $volume->error());
	    }

	    $srcIsDir = ($source['mime'] === 'directory');

	    $errpath = $volume->path($source['hash']);

	    if (! $this->nameAccepted($source['name'], $srcIsDir)) {
	        return $this->setError(elFinder::ERROR_COPY, $errpath, $srcIsDir ? elFinder::ERROR_INVALID_DIRNAME : elFinder::ERROR_INVALID_NAME);
	    }

	    if (! $source['read']) {
	        return $this->setError(elFinder::ERROR_COPY, $errpath, elFinder::ERROR_PERM_DENIED);
	    }

	    if ($srcIsDir) {
	        $test = $this->isNameExists($this->joinPathCE($destination, $name));
	        $this->clearcache();

	        if (($test && $test['mime'] != 'directory') || (! $test && ! $test = $this->mkdir($this->encode($destination), $name))) {

The expression $test of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

The expression $test of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This code seems to be duplicated across your project.

	            return $this->setError(elFinder::ERROR_COPY, $errpath);
	        }

			//$path = $this->joinPathCE($destination, $name);

65% of this comment could be valid code. Did you maybe forget this after debugging?

			$path = $this->decode($test['hash']);

	        foreach ($volume->scandir($src) as $entr) {
	            if (! $this->copyFrom($volume, $entr['hash'], $path, $entr['name'])) {
	                $this->remove($path, true); // fall back
					return $this->setError($this->error, elFinder::ERROR_COPY, $errpath);
	            }
	        }

	        $this->added[] = $test;
	    } else {
	        // MIME check
			$mimeByName = $this->mimetype($source['name'], true);
	        if ($source['mime'] === $mimeByName) {
	            $mimeByName = '';
	        }
	        if (! $this->allowPutMime($source['mime']) || ($mimeByName && ! $this->allowPutMime($mimeByName))) {

This code seems to be duplicated across your project.

	            return $this->setError(elFinder::ERROR_UPLOAD_FILE_MIME, $errpath);
	        }

	        if (strpos($source['mime'], 'image') === 0 && ($dim = $volume->dimensions($src))) {
	            $s = explode('x', $dim);
	            $source['width'] = $s[0];
	            $source['height'] = $s[1];
	        }

	        if (($fp = $volume->open($src)) == false
			|| ($path = $this->saveCE($fp, $destination, $name, $source)) == false) {
	            $fp && $volume->close($fp, $src);

	            return $this->setError(elFinder::ERROR_COPY, $errpath);
	        }
	        $volume->close($fp, $src);

	        $this->added[] = $this->stat($path);

It seems like $path defined by $this->saveCE($fp, $destination, $name, $source) on line 4802 can also be of type boolean; however, elFinderVolumeDriver::stat() does only seem to accept string, maybe add an additional type check?

	    }

	    return $path;
	}

	/**
	 * Remove file/ recursive remove dir.
	 *
	 * @param  string  $path   file path
	 * @param  bool    $force  try to remove even if file locked
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function remove($path, $force = false)
	{
	    $stat = $this->stat($path);

	    if (empty($stat)) {
	        return $this->setError(elFinder::ERROR_RM, $path, elFinder::ERROR_FILE_NOT_FOUND);
	    }

	    $stat['realpath'] = $path;
	    $this->rmTmb($stat);

$stat is of type array<string,string,{"realpath":"string"}>, but the function expects a string.

	    $this->clearcache();

	    if (! $force && ! empty($stat['locked'])) {
	        return $this->setError(elFinder::ERROR_LOCKED, $this->path($stat['hash']));
	    }

	    if ($stat['mime'] == 'directory' && empty($stat['thash'])) {
	        $ret = $this->delTree($this->convEncIn($path));
	        $this->convEncOut();
	        if (! $ret) {
	            return $this->setError(elFinder::ERROR_RM, $this->path($stat['hash']));
	        }
	    } else {

This code seems to be duplicated across your project.

	        if ($this->convEncOut(! $this->_unlink($this->convEncIn($path)))) {
	            return $this->setError(elFinder::ERROR_RM, $this->path($stat['hash']));
	        }
	    }

	    $this->removed[] = $stat;

	    return true;
	}

	/************************* thumbnails **************************/

	/**
	 * Return thumbnail file name for required file.
	 *
	 * @param  array  $stat  file stat
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	protected function tmbname($stat)
	{
	    return $stat['hash'].$stat['ts'].'.png';
	}

	/**
	 * Return thumnbnail name if exists.
	 *
	 * @param  string  $path file path
	 * @param  array   $stat file stat
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	protected function gettmb($path, $stat)
	{
	    if ($this->tmbURL && $this->tmbPath) {
	        // file itself thumnbnail
			if (strpos($path, $this->tmbPath) === 0) {
			    return basename($path);
			}

	        $name = $this->tmbname($stat);
	        if (file_exists($this->tmbPath.DIRECTORY_SEPARATOR.$name)) {
	            return $name;
	        }
	    }

	    return false;
	}

	/**
	 * Return true if thumnbnail for required file can be created.
	 *
	 * @param  string  $path  thumnbnail path
	 * @param  array   $stat  file stat
	 * @param  bool    $checkTmbPath
	 * @return string|bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function canCreateTmb($path, $stat, $checkTmbPath = true)
	{
	    if ((! $checkTmbPath || $this->tmbPathWritable)
			&& (! $this->tmbPath || strpos($path, $this->tmbPath) === false) // do not create thumnbnail for thumnbnail
		) {
	        $mime = strtolower($stat['mime']);
	        list($type) = explode('/', $mime);
	        if (! empty($this->imgConverter)) {
	            if (isset($this->imgConverter[$mime])) {
	                return true;
	            }
	            if (isset($this->imgConverter[$type])) {
	                return true;
	            }
	        }

	        return $this->imgLib
				&& ($type === 'image')
				&& ($this->imgLib == 'gd' ? in_array($stat['mime'], ['image/jpeg', 'image/png', 'image/gif', 'image/x-ms-bmp']) : true);
	    }

	    return false;
	}

	/**
	 * Return true if required file can be resized.
	 * By default - the same as canCreateTmb.
	 *
	 * @param  string  $path  thumnbnail path
	 * @param  array   $stat  file stat
	 * @return string|bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected function canResize($path, $stat)
	{
	    return $this->canCreateTmb($path, $stat, false);
	}

	/**
	 * Create thumnbnail and return it's URL on success.
	 *
	 * @param  string $path file path
	 * @param $stat
	 * @return false|string
	 * @internal param string $mime file mime type
	 * @author Dmitry (dio) Levashov
	 */
	protected function createTmb($path, $stat)
	{
	    if (! $stat || ! $this->canCreateTmb($path, $stat)) {
	        return false;
	    }

	    $name = $this->tmbname($stat);
	    $tmb = $this->tmbPath.DIRECTORY_SEPARATOR.$name;

	    $maxlength = -1;
	    $imgConverter = null;

		// check imgConverter
		$mime = strtolower($stat['mime']);
	    list($type) = explode('/', $mime);
	    if (isset($this->imgConverter[$mime])) {
	        $imgConverter = $this->imgConverter[$mime]['func'];
	        if (! empty($this->imgConverter[$mime]['maxlen'])) {

This code seems to be duplicated across your project.

	            $maxlength = intval($this->imgConverter[$mime]['maxlen']);
	        }
	    } elseif (isset($this->imgConverter[$type])) {
	        $imgConverter = $this->imgConverter[$type]['func'];
	        if (! empty($this->imgConverter[$type]['maxlen'])) {

This code seems to be duplicated across your project.

	            $maxlength = intval($this->imgConverter[$type]['maxlen']);
	        }
	    }
	    if ($imgConverter && ! is_callable($imgConverter)) {
	        return false;
	    }

		// copy image into tmbPath so some drivers does not store files on local fs
		if (($src = $this->fopenCE($path, 'rb')) == false) {
		    return false;
		}

	    if (($trg = fopen($tmb, 'wb')) == false) {
	        $this->fcloseCE($src, $path);

	        return false;
	    }

	    stream_copy_to_stream($src, $trg, $maxlength);

	    $this->fcloseCE($src, $path);
	    fclose($trg);

		// call imgConverter
		if ($imgConverter) {
		    if (! call_user_func_array($imgConverter, [$tmb, $stat, $this])) {
		        file_exists($tmb) && unlink($tmb);

		        return false;
		    }
		}

	    $result = false;

	    $tmbSize = $this->tmbSize;

	    if ($this->imgLib === 'imagick') {
	        try {
	            $imagickTest = new imagick($tmb);
	            $imagickTest->clear();
	            $imagickTest = true;
	        } catch (Exception $e) {
	            $imagickTest = false;
	        }
	    }

	    if (($this->imgLib === 'imagick' && ! $imagickTest) || ($s = getimagesize($tmb)) === false) {

The variable $imagickTest does not seem to be defined for all execution paths leading up to this point.

	        if ($this->imgLib === 'imagick') {
	            $bgcolor = $this->options['tmbBgColor'];
	            if ($bgcolor === 'transparent') {
	                $bgcolor = 'rgba(255, 255, 255, 0.0)';
	            }
	            try {
	                $imagick = new imagick();
	                $imagick->setBackgroundColor(new ImagickPixel($bgcolor));
	                $imagick->readImage($this->getExtentionByMime($stat['mime'], ':').$tmb);
	                $imagick->setImageFormat('png');
	                $imagick->writeImage($tmb);
	                $imagick->clear();
	                if (($s = getimagesize($tmb)) !== false) {
	                    $result = true;
	                }
	            } catch (Exception $e) {

Consider adding a comment why this CATCH block is empty.

	            }
	        }
	        if (! $result) {
	            file_exists($tmb) && unlink($tmb);

	            return false;
	        }
	        $result = false;

$result is not used, you could remove the assignment.

	    }

		/* 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');
		} else {
		    if ($this->options['tmbCrop']) {
		        $result = $tmb;
				/* 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)) {

The variable $s does not seem to be defined for all execution paths leading up to this point.

				    $result = $this->imgResize($tmb, $tmbSize, $tmbSize, true, false, 'png');
				}

		        if ($result && ($s = getimagesize($tmb)) != false) {

The expression $result of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

		            $x = $s[0] > $tmbSize ? intval(($s[0] - $tmbSize) / 2) : 0;
		            $y = $s[1] > $tmbSize ? intval(($s[1] - $tmbSize) / 2) : 0;
		            $result = $this->imgCrop($result, $tmbSize, $tmbSize, $x, $y, 'png');

$x is of type integer, but the function expects a boolean.

$y is of type integer, but the function expects a boolean.

		        } else {
		            $result = false;
		        }
		    } else {
		        $result = $this->imgResize($tmb, $tmbSize, $tmbSize, true, true, 'png');
		    }

		    if ($result) {

The expression $result of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

		        if ($s = getimagesize($tmb)) {
		            if ($s[0] !== $tmbSize || $s[1] !== $tmbSize) {
		                $result = $this->imgSquareFit($result, $tmbSize, $tmbSize, 'center', 'middle', $this->options['tmbBgColor'], 'png');
		            }
		        }
		    }
		}

	    if (! $result) {

The expression $result of type false|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === false instead.

	        unlink($tmb);

	        return false;
	    }

	    return $name;
	}

	/**
	 * Resize image.
	 *
	 * @param  string   $path               image file
	 * @param  int      $width              new width
	 * @param  int      $height             new height
	 * @param  bool	    $keepProportions    crop image
	 * @param  bool	    $resizeByBiggerSide resize image based on bigger side if true
	 * @param  string   $destformat         image destination format
	 * @param  int      $jpgQuality         JEPG quality (1-100)
	 * @param  array    $options            Other extra options
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 * @author Alexey Sukhotin
	 **/
	protected function imgResize($path, $width, $height, $keepProportions = false, $resizeByBiggerSide = true, $destformat = null, $jpgQuality = null, $options = [])
	{
	    if (($s = getimagesize($path)) == false) {
	        return false;
	    }

	    $result = false;

$result is not used, you could remove the assignment.

	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

	        $jpgQuality = $this->options['jpgQuality'];
	    }

	    list($orig_w, $orig_h) = [$s[0], $s[1]];
	    list($size_w, $size_h) = [$width, $height];

	    if (empty($options['unenlarge']) || $orig_w > $size_w || $orig_h > $size_h) {
	        if ($keepProportions == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

	            /* Resizing by biggest side */
				if ($resizeByBiggerSide) {
				    if ($orig_w > $orig_h) {

This code seems to be duplicated across your project.

				        $size_h = round($orig_h * $width / $orig_w);
				        $size_w = $width;
				    } else {
				        $size_w = round($orig_w * $height / $orig_h);
				        $size_h = $height;
				    }
				} else {

This code seems to be duplicated across your project.

				    if ($orig_w > $orig_h) {
				        $size_w = round($orig_w * $height / $orig_h);
				        $size_h = $height;
				    } else {
				        $size_h = round($orig_h * $width / $orig_w);
				        $size_w = $width;
				    }
				}
	        }
	    } else {
	        $size_w = $orig_w;
	        $size_h = $orig_h;
	    }

	    elFinder::extendTimeLimit(300);
	    switch ($this->imgLib) {
			case 'imagick':

The case body in a switch statement must start on the line following the statement.

				try {
				    $img = new imagick($path);
				} catch (Exception $e) {
				    return false;
				}

				// Imagick::FILTER_BOX faster than FILTER_LANCZOS so use for createTmb
				// resize bench: http://app-mgng.rhcloud.com/9
				// resize sample: http://www.dylanbeattie.net/magick/filters/result.html
				$filter = ($destformat === 'png' /* createTmb */) ? Imagick::FILTER_BOX : Imagick::FILTER_LANCZOS;

				$ani = ($img->getNumberImages() > 1);
				if ($ani && is_null($destformat)) {
				    $img = $img->coalesceImages();
				    do {
				        $img->resizeImage($size_w, $size_h, $filter, 1);
				    } while ($img->nextImage());
				    $img = $img->optimizeImageLayers();
				    $result = $img->writeImages($path, true);

The method writeImages cannot be called on $img (of type boolean).

				} else {
				    if ($ani) {
				        $img->setFirstIterator();
				    }
				    if (strtoupper($img->getImageFormat()) === 'JPEG') {
				        $img->setImageCompression(imagick::COMPRESSION_JPEG);
				        $img->setImageCompressionQuality($jpgQuality);
				        if (isset($options['preserveExif']) && ! $options['preserveExif']) {
				            try {
				                $orientation = $img->getImageOrientation();
				            } catch (ImagickException $e) {
				                $orientation = 0;
				            }
				            $img->stripImage();
				            if ($orientation) {
				                $img->setImageOrientation($orientation);
				            }
				        }
				    }
				    $img->resizeImage($size_w, $size_h, $filter, true);
				    if ($destformat) {

The expression $destformat of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

				        $result = $this->imagickImage($img, $path, $destformat, $jpgQuality);

$img is of type object<Imagick>, but the function expects a resource.

				    } else {
				        $result = $img->writeImage($path);
				    }
				}

				$img->clear();

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'convert':
				extract($this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s));

$this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s) cannot be passed to extract() as the parameter $var_array expects a reference.

				$filter = ($destformat === 'png' /* createTmb */) ? '-filter Box' : '-filter Lanczos';
				$strip = (isset($options['preserveExif']) && ! $options['preserveExif']) ? ' -strip' : '';

$strip is not used, you could remove the assignment.

				//$cmd = sprintf('convert%s%s +repage %s -resize %dx%d +repage%s %s %s', $coalesce, $jpgQuality, $filter, $size_w, $size_h, $deconstruct, $quotedPath, $quotedDstPath);

64% of this comment could be valid code. Did you maybe forget this after debugging?

				$cmd = sprintf('convert %s%s%s %s -geometry %dx%d! %s %s', $quotedPath, $coalesce, $jpgQuality, $filter, $size_w, $size_h, $deconstruct, $quotedDstPath);

				$result = false;
				if ($this->procExec($cmd) === 0) {
				    $result = true;
				}

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'gd':
				$img = $this->gdImageCreate($path, $s['mime']);

				if ($img && false != ($tmp = imagecreatetruecolor($size_w, $size_h))) {
				    $bgNum = false;
				    if ($s[2] === IMAGETYPE_GIF && (! $destformat || $destformat === 'gif')) {

The expression $destformat of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

This code seems to be duplicated across your project.

				        $bgIdx = imagecolortransparent($img);
				        if ($bgIdx !== -1) {
				            $c = imagecolorsforindex($img, $bgIdx);
				            $bgNum = imagecolorallocate($tmp, $c['red'], $c['green'], $c['blue']);
				            imagefill($tmp, 0, 0, $bgNum);
				            imagecolortransparent($tmp, $bgNum);
				        }
				    }
				    if ($bgNum === false) {
				        $this->gdImageBackground($tmp, 'transparent');
				    }

				    if (! imagecopyresampled($tmp, $img, 0, 0, 0, 0, $size_w, $size_h, $s[0], $s[1])) {
				        return false;
				    }

				    $result = $this->gdImage($tmp, $path, $destformat, $s['mime'], $jpgQuality);

				    imagedestroy($img);
				    imagedestroy($tmp);

				    return $result ? $path : false;
				}
				break;
		}

	    return false;
	}

  	/**
  	 * Crop image.
  	 *
  	 * @param  string   $path               image file
  	 * @param  int      $width              crop width
  	 * @param  int      $height             crop height
  	 * @param  bool	    $x                  crop left offset
  	 * @param  bool	    $y                  crop top offset
  	 * @param  string   $destformat         image destination format
  	 * @param  int      $jpgQuality         JEPG quality (1-100)
  	 * @return string|false
  	 * @author Dmitry (dio) Levashov
  	 * @author Alexey Sukhotin
  	 **/
  	protected function imgCrop($path, $width, $height, $x, $y, $destformat = null, $jpgQuality = null)
  	{
  	    if (($s = getimagesize($path)) == false) {
  	        return false;
  	    }

  	    $result = false;

$result is not used, you could remove the assignment.

  	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

  	        $jpgQuality = $this->options['jpgQuality'];
  	    }

  	    elFinder::extendTimeLimit(300);
  	    switch ($this->imgLib) {
			case 'imagick':

The case body in a switch statement must start on the line following the statement.

				try {
				    $img = new imagick($path);
				} catch (Exception $e) {
				    return false;
				}

				$ani = ($img->getNumberImages() > 1);
				if ($ani && is_null($destformat)) {
				    $img = $img->coalesceImages();
				    do {
				        $img->setImagePage($s[0], $s[1], 0, 0);
				        $img->cropImage($width, $height, $x, $y);
				        $img->setImagePage($width, $height, 0, 0);
				    } while ($img->nextImage());
				    $img = $img->optimizeImageLayers();
				    $result = $img->writeImages($path, true);

The method writeImages cannot be called on $img (of type boolean).

				} else {
				    if ($ani) {
				        $img->setFirstIterator();
				    }
				    $img->setImagePage($s[0], $s[1], 0, 0);
				    $img->cropImage($width, $height, $x, $y);
				    $img->setImagePage($width, $height, 0, 0);
				    $result = $this->imagickImage($img, $path, $destformat, $jpgQuality);

$img is of type object<Imagick>, but the function expects a resource.

				}

				$img->clear();

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'convert':
				extract($this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s));

$this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s) cannot be passed to extract() as the parameter $var_array expects a reference.

				$cmd = sprintf('convert %s%s%s -crop %dx%d+%d+%d%s %s', $quotedPath, $coalesce, $jpgQuality, $width, $height, $x, $y, $deconstruct, $quotedDstPath);

				$result = false;
				if ($this->procExec($cmd) === 0) {
				    $result = true;
				}

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'gd':
				$img = $this->gdImageCreate($path, $s['mime']);

				if ($img && false != ($tmp = imagecreatetruecolor($width, $height))) {
				    $bgNum = false;
				    if ($s[2] === IMAGETYPE_GIF && (! $destformat || $destformat === 'gif')) {

The expression $destformat of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

This code seems to be duplicated across your project.

				        $bgIdx = imagecolortransparent($img);
				        if ($bgIdx !== -1) {
				            $c = imagecolorsforindex($img, $bgIdx);
				            $bgNum = imagecolorallocate($tmp, $c['red'], $c['green'], $c['blue']);
				            imagefill($tmp, 0, 0, $bgNum);
				            imagecolortransparent($tmp, $bgNum);
				        }
				    }
				    if ($bgNum === false) {
				        $this->gdImageBackground($tmp, 'transparent');
				    }

				    $size_w = $width;
				    $size_h = $height;

				    if ($s[0] < $width || $s[1] < $height) {
				        $size_w = $s[0];
				        $size_h = $s[1];
				    }

				    if (! imagecopy($tmp, $img, 0, 0, $x, $y, $size_w, $size_h)) {
				        return false;
				    }

				    $result = $this->gdImage($tmp, $path, $destformat, $s['mime'], $jpgQuality);

				    imagedestroy($img);
				    imagedestroy($tmp);

				    return $result ? $path : false;
				}
				break;
		}

  	    return false;
  	}

	/**
	 * Put image to square.
	 *
	 * @param  string $path image file
	 * @param  int $width square width
	 * @param  int $height square height
	 * @param int|string $align reserved
	 * @param int|string $valign reserved
	 * @param  string $bgcolor square background color in #rrggbb format
	 * @param  string $destformat image destination format
	 * @param  int $jpgQuality JEPG quality (1-100)
	 * @return false|string
	 * @author Dmitry (dio) Levashov
	 * @author Alexey Sukhotin
	 */
	protected function imgSquareFit($path, $width, $height, $align = 'center', $valign = 'middle', $bgcolor = '#0000ff', $destformat = null, $jpgQuality = null)

The parameter $align is not used and could be removed.

The parameter $valign is not used and could be removed.

	{
	    if (($s = getimagesize($path)) == false) {
	        return false;
	    }

	    $result = false;

$result is not used, you could remove the assignment.

		/* Coordinates for image over square aligning */
		$y = ceil(abs($height - $s[1]) / 2);
	    $x = ceil(abs($width - $s[0]) / 2);

	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

	        $jpgQuality = $this->options['jpgQuality'];
	    }

	    elFinder::extendTimeLimit(300);
	    switch ($this->imgLib) {
			case 'imagick':
				try {
				    $img = new imagick($path);
				} catch (Exception $e) {
				    return false;
				}

				if ($bgcolor === 'transparent') {
				    $bgcolor = 'rgba(255, 255, 255, 0.0)';
				}
				$ani = ($img->getNumberImages() > 1);
				if ($ani && is_null($destformat)) {
				    $img1 = new Imagick();
				    $img1->setFormat('gif');
				    $img = $img->coalesceImages();
				    do {
				        $gif = new Imagick();
				        $gif->newImage($width, $height, new ImagickPixel($bgcolor));
				        $gif->setImageColorspace($img->getImageColorspace());
				        $gif->setImageFormat('gif');
				        $gif->compositeImage($img, imagick::COMPOSITE_OVER, $x, $y);
				        $gif->setImageDelay($img->getImageDelay());
				        $gif->setImageIterations($img->getImageIterations());
				        $img1->addImage($gif);
				        $gif->clear();
				    } while ($img->nextImage());
				    $img1 = $img1->optimizeImageLayers();
				    $result = $img1->writeImages($path, true);

The method writeImages cannot be called on $img1 (of type boolean).

				} else {
				    if ($ani) {
				        $img->setFirstIterator();
				    }
				    $img1 = new Imagick();
				    $img1->newImage($width, $height, new ImagickPixel($bgcolor));
				    $img1->setImageColorspace($img->getImageColorspace());
				    $img1->compositeImage($img, imagick::COMPOSITE_OVER, $x, $y);
				    $result = $this->imagickImage($img1, $path, $destformat, $jpgQuality);

$img1 is of type object<Imagick>, but the function expects a resource.

				}

				$img1->clear();
				$img->clear();

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'convert':
				extract($this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s));

$this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s) cannot be passed to extract() as the parameter $var_array expects a reference.

				if ($bgcolor === 'transparent') {
				    $bgcolor = 'rgba(255, 255, 255, 0.0)';
				}
				$cmd = sprintf('convert -size %dx%d "xc:%s" png:- | convert%s%s png:-  %s -geometry +%d+%d -compose over -composite%s %s', $width, $height, $bgcolor, $coalesce, $jpgQuality, $quotedPath, $x, $y, $deconstruct, $quotedDstPath);

				$result = false;
				if ($this->procExec($cmd) === 0) {
				    $result = true;
				}

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'gd':
				$img = $this->gdImageCreate($path, $s['mime']);

				if ($img && false != ($tmp = imagecreatetruecolor($width, $height))) {
				    $this->gdImageBackground($tmp, $bgcolor);
				    if ($bgcolor === 'transparent' && ($destformat === 'png' || $s[2] === IMAGETYPE_PNG)) {
				        $bgNum = imagecolorallocatealpha($tmp, 255, 255, 255, 127);
				        imagefill($tmp, 0, 0, $bgNum);
				    }

				    if (! imagecopy($tmp, $img, $x, $y, 0, 0, $s[0], $s[1])) {
				        return false;
				    }

				    $result = $this->gdImage($tmp, $path, $destformat, $s['mime'], $jpgQuality);

				    imagedestroy($img);
				    imagedestroy($tmp);

				    return $result ? $path : false;
				}
				break;
		}

	    return false;
	}

	/**
	 * Rotate image.
	 *
	 * @param  string   $path               image file
	 * @param  int      $degree             rotete degrees
	 * @param  string   $bgcolor            square background color in #rrggbb format
	 * @param  string   $destformat         image destination format
	 * @param  int      $jpgQuality         JEPG quality (1-100)
	 * @return string|false
	 * @author nao-pon
	 * @author Troex Nevelin
	 **/
	protected function imgRotate($path, $degree, $bgcolor = '#ffffff', $destformat = null, $jpgQuality = null)
	{
	    if (($s = getimagesize($path)) == false || $degree % 360 === 0) {
	        return false;
	    }

	    $result = false;

		// try lossless rotate
		if ($degree % 90 === 0 && in_array($s[2], [IMAGETYPE_JPEG, IMAGETYPE_JPEG2000])) {
		    $count = ($degree / 90) % 4;
		    $exiftran = [
				1 => '-9',
				2 => '-1',
				3 => '-2',
			];
		    $jpegtran = [
				1 => '90',
				2 => '180',
				3 => '270',
			];
		    $quotedPath = escapeshellarg($path);
		    $cmds = [];
		    if ($this->procExec('exiftran -h') === 0) {
		        $cmds[] = 'exiftran -i '.$exiftran[$count].' '.$path;
		    }
		    if ($this->procExec('jpegtran -version') === 0) {
		        $cmds[] = 'jpegtran -rotate '.$jpegtran[$count].' -copy all -outfile '.$quotedPath.' '.$quotedPath;
		    }
		    foreach ($cmds as $cmd) {
		        if ($this->procExec($cmd) === 0) {
		            $result = true;
		            break;
		        }
		    }
		    if ($result) {
		        return $path;
		    }
		}

	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

	        $jpgQuality = $this->options['jpgQuality'];
	    }

	    elFinder::extendTimeLimit(300);
	    switch ($this->imgLib) {
			case 'imagick':
				try {
				    $img = new imagick($path);
				} catch (Exception $e) {
				    return false;
				}

				if ($s[2] === IMAGETYPE_GIF || $s[2] === IMAGETYPE_PNG) {

This code seems to be duplicated across your project.

				    $bgcolor = 'rgba(255, 255, 255, 0.0)';
				}
				if ($img->getNumberImages() > 1) {
				    $img = $img->coalesceImages();
				    do {
				        $img->rotateImage(new ImagickPixel($bgcolor), $degree);
				    } while ($img->nextImage());
				    $img = $img->optimizeImageLayers();
				    $result = $img->writeImages($path, true);

The method writeImages cannot be called on $img (of type boolean).

				} else {
				    $img->rotateImage(new ImagickPixel($bgcolor), $degree);
				    $result = $this->imagickImage($img, $path, $destformat, $jpgQuality);

$img is of type object<Imagick>, but the function expects a resource.

				}
				$img->clear();

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'convert':
				extract($this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s));

$this->imageMagickConvertPrepare($path, $destformat, $jpgQuality, $s) cannot be passed to extract() as the parameter $var_array expects a reference.

				if ($s[2] === IMAGETYPE_GIF || $s[2] === IMAGETYPE_PNG) {

This code seems to be duplicated across your project.

				    $bgcolor = 'rgba(255, 255, 255, 0.0)';
				}
				$cmd = sprintf('convert %s%s%s -background "%s" -rotate %d%s %s', $quotedPath, $coalesce, $jpgQuality, $bgcolor, $degree, $deconstruct, $quotedDstPath);

				$result = false;
				if ($this->procExec($cmd) === 0) {
				    $result = true;
				}

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

			case 'gd':
				$img = $this->gdImageCreate($path, $s['mime']);

				$degree = 360 - $degree;

				$bgNum = -1;
				$bgIdx = false;
				if ($s[2] === IMAGETYPE_GIF) {
				    $bgIdx = imagecolortransparent($img);
				    if ($bgIdx !== -1) {
				        $c = imagecolorsforindex($img, $bgIdx);
				        $w = imagesx($img);
				        $h = imagesy($img);
				        $newImg = imagecreatetruecolor($w, $h);
				        imagepalettecopy($newImg, $img);
				        $bgNum = imagecolorallocate($newImg, $c['red'], $c['green'], $c['blue']);
				        imagefill($newImg, 0, 0, $bgNum);
				        imagecolortransparent($newImg, $bgNum);
				        imagecopy($newImg, $img, 0, 0, 0, 0, $w, $h);
				        imagedestroy($img);
				        $img = $newImg;
				        $newImg = null;

$newImg is not used, you could remove the assignment.

				    }
				} elseif ($s[2] === IMAGETYPE_PNG) {
				    $bgNum = imagecolorallocatealpha($img, 255, 255, 255, 127);
				}
				if ($bgNum === -1) {

This code seems to be duplicated across your project.

				    list($r, $g, $b) = sscanf($bgcolor, '#%02x%02x%02x');
				    $bgNum = imagecolorallocate($img, $r, $g, $b);
				}

				$tmp = imagerotate($img, $degree, $bgNum);
				if ($bgIdx !== -1) {
				    imagecolortransparent($tmp, $bgNum);
				}

				$result = $this->gdImage($tmp, $path, $destformat, $s['mime'], $jpgQuality);

				imagedestroy($img);
				imagedestroy($tmp);

				return $result ? $path : false;

				break;

break is not strictly necessary here and could be removed.

		}

	    return false;
	}

	/**
	 * Execute shell command.
	 *
	 * @param  string $command command line
	 * @param  array $output stdout strings
	 * @param array|int $return_var process exit code
	 * @param  array $error_output stderr strings
	 * @return int exit code
	 * @author Alexey Sukhotin
	 */
	protected function procExec($command, array &$output = null, &$return_var = -1, array &$error_output = null)
	{
	    static $allowed = null;

	    if ($allowed === null) {
	        if ($allowed = function_exists('proc_open')) {
	            if ($disabled = ini_get('disable_functions')) {
	                $funcs = array_map('trim', explode(',', $disabled));
	                $allowed = ! in_array('proc_open', $funcs);
	            }
	        }
	    }

	    if (! $allowed) {

The expression $allowed of type boolean|null is loosely compared to false; this is ambiguous if the boolean can be false. You might want to explicitly use !== null instead.

	        $return_var = -1;

	        return $return_var;
	    }

	    if (! $command) {
	        $return_var = 0;

	        return $return_var;
	    }

	    $descriptorspec = [
			0 => ['pipe', 'r'],  // stdin
			1 => ['pipe', 'w'],  // stdout
			2 => ['pipe', 'w'],   // stderr
		];

	    $process = proc_open($command, $descriptorspec, $pipes, null, null);

	    if (is_resource($process)) {
	        fclose($pipes[0]);

	        $tmpout = '';

$tmpout is not used, you could remove the assignment.

	        $tmperr = '';

$tmperr is not used, you could remove the assignment.

	        $output = stream_get_contents($pipes[1]);
	        $error_output = stream_get_contents($pipes[2]);

	        fclose($pipes[1]);
	        fclose($pipes[2]);
	        $return_var = proc_close($process);
	    } else {
	        $return_var = -1;
	    }

	    return $return_var;
	}

	/**
	 * Remove thumbnail, also remove recursively if stat is directory.
	 *
	 * @param  string  $stat  file stat
	 * @return void
	 * @author Dmitry (dio) Levashov
	 * @author Naoki Sawada
	 * @author Troex Nevelin
	 **/
	protected function rmTmb($stat)
	{
	    if ($stat['mime'] === 'directory') {
	        foreach ($this->scandirCE($this->decode($stat['hash'])) as $p) {
	            elFinder::extendTimeLimit(30);
	            $name = $this->basenameCE($p);
	            $name != '.' && $name != '..' && $this->rmTmb($this->stat($p));

$this->stat($p) is of type array, but the function expects a string.

	        }
	    } elseif (! empty($stat['tmb']) && $stat['tmb'] != '1') {
	        $tmb = $this->tmbPath.DIRECTORY_SEPARATOR.$stat['tmb'];
	        file_exists($tmb) && unlink($tmb);
	        clearstatcache();
	    }
	}

	/**
	 * Create an gd image according to the specified mime type.
	 *
	 * @param string $path image file
	 * @param string $mime
	 * @return gd image resource identifier
	 */
	protected function gdImageCreate($path, $mime)
	{
	    switch ($mime) {
			case 'image/jpeg':
			return imagecreatefromjpeg($path);

			case 'image/png':
			return imagecreatefrompng($path);

			case 'image/gif':
			return imagecreatefromgif($path);

			case 'image/x-ms-bmp':
			if (! function_exists('imagecreatefrombmp')) {
			    include_once dirname(__FILE__).'/libs/GdBmp.php';
			}

			return imagecreatefrombmp($path);

			case 'image/xbm':
			return imagecreatefromxbm($path);

			case 'image/xpm':
			return imagecreatefromxpm($path);
		}

	    return false;
	}

	/**
	 * Output gd image to file.
	 *
	 * @param resource $image gd image resource
	 * @param string $filename The path to save the file to.
	 * @param string $destformat The Image type to use for $filename
	 * @param string $mime The original image mime type
	 * @param int $jpgQuality JEPG quality (1-100)
	 * @return bool
	 */
	protected function gdImage($image, $filename, $destformat, $mime, $jpgQuality = null)
	{
	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

	        $jpgQuality = $this->options['jpgQuality'];
	    }
	    if ($destformat) {
	        switch ($destformat) {
				case 'jpg':
					$mime = 'image/jpeg';
					break;
				case 'gif':
					$mime = 'image/gif';
					break;
				case 'png':
				default:
					$mime = 'image/png';
					break;
			}
	    }
	    switch ($mime) {
			case 'image/gif':
				return imagegif($image, $filename);
			case 'image/jpeg':
				return imagejpeg($image, $filename, $jpgQuality);
			case 'image/wbmp':
				return imagewbmp($image, $filename);
			case 'image/png':
			default:
				return imagepng($image, $filename);
		}
	}

	/**
	 * Output imagick image to file.
	 *
	 * @param resource $img imagick image resource
	 * @param string $filename The path to save the file to.
	 * @param string $destformat The Image type to use for $filename
	 * @param int $jpgQuality JEPG quality (1-100)
	 * @return bool
	 */
	protected function imagickImage($img, $filename, $destformat, $jpgQuality = null)
	{
	    if (! $jpgQuality) {

The expression $jpgQuality of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

	        $jpgQuality = $this->options['jpgQuality'];
	    }

	    try {
	        if ($destformat) {
	            if ($destformat === 'gif') {
	                $img->setImageFormat('gif');

The method setImageFormat cannot be called on $img (of type resource).

	            } elseif ($destformat === 'png') {
	                $img->setImageFormat('png');

The method setImageFormat cannot be called on $img (of type resource).

	            } elseif ($destformat === 'jpg') {
	                $img->setImageFormat('jpeg');

The method setImageFormat cannot be called on $img (of type resource).

	            }
	        }
	        if (strtoupper($img->getImageFormat()) === 'JPEG') {

The method getImageFormat cannot be called on $img (of type resource).

	            $img->setImageCompression(imagick::COMPRESSION_JPEG);

The method setImageCompression cannot be called on $img (of type resource).

	            $img->setImageCompressionQuality($jpgQuality);

The method setImageCompressionQuality cannot be called on $img (of type resource).

	            try {
	                $orientation = $img->getImageOrientation();

The method getImageOrientation cannot be called on $img (of type resource).

	            } catch (ImagickException $e) {
	                $orientation = 0;
	            }
	            $img->stripImage();

The method stripImage cannot be called on $img (of type resource).

	            if ($orientation) {
	                $img->setImageOrientation($orientation);

The method setImageOrientation cannot be called on $img (of type resource).

	            }
	        }
	        $result = $img->writeImage($filename);

The method writeImage cannot be called on $img (of type resource).

	    } catch (Exception $e) {
	        $result = false;
	    }

	    return $result;
	}

	/**
	 * Assign the proper background to a gd image.
	 *
	 * @param resource $image gd image resource
	 * @param string $bgcolor background color in #rrggbb format
	 */
	protected function gdImageBackground($image, $bgcolor)
	{
	    if ($bgcolor === 'transparent') {
	        imagealphablending($image, false);
	        imagesavealpha($image, true);
	    } else {

This code seems to be duplicated across your project.

	        list($r, $g, $b) = sscanf($bgcolor, '#%02x%02x%02x');
	        $bgcolor1 = imagecolorallocate($image, $r, $g, $b);
	        imagefill($image, 0, 0, $bgcolor1);
	    }
	}

	/**
	 * Prepare variables for exec convert of ImageMagick.
	 *
	 * @param  string  $path
	 * @param  string  $destformat
	 * @param  int     $jpgQuality
	 * @param  array   $imageSize
	 * @return array
	 */
	protected function imageMagickConvertPrepare($path, $destformat, $jpgQuality, $imageSize = null)
	{
	    if (is_null($imageSize)) {
	        $imageSize = getimagesize($path);
	    }
	    if (! $imageSize) {

The expression $imageSize of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

	        return [];
	    }
	    $srcType = $this->getExtentionByMime($imageSize['mime'], ':');
	    $ani = false;
	    $cmd = 'identify '.escapeshellarg($srcType.$path);
	    if ($this->procExec($cmd, $o) === 0) {
	        $ani = preg_split('/(?:\r\n|\n|\r)/', trim($o));
	        if (count($ani) < 2) {
	            $ani = false;
	        }
	    }
	    $coalesce = $index = '';
	    $deconstruct = ' +repage';
	    if ($ani) {
	        if (is_null($destformat)) {
	            $coalesce = ' -coalesce -repage 0x0';
	            $deconstruct = ' +repage -deconstruct -layers optimize';
	        } else {
	            $index = '[0]';
	            if ($srcType === 'ico:') {
	                foreach ($ani as $_i => $_info) {
	                    if (preg_match('/ (\d+)x(\d+) /', $_info, $m)) {
	                        if ($m[1] == $imageSize[0] && $m[2] == $imageSize[1]) {
	                            $index = '['.$_i.']';
	                            break;
	                        }
	                    }
	                }
	            }
	        }
	    }
	    if ($imageSize[2] === IMAGETYPE_JPEG || $imageSize[2] === IMAGETYPE_JPEG2000) {
	        $jpgQuality = ' -quality '.$jpgQuality;
	    } else {
	        $jpgQuality = '';
	    }
	    $quotedPath = escapeshellarg($srcType.$path.$index);
	    $quotedDstPath = escapeshellarg(($destformat ? ($destformat.':') : $srcType).$path);

	    return compact('ani', 'index', 'coalesce', 'deconstruct', 'jpgQuality', 'quotedPath', 'quotedDstPath');
	}

	/*********************** misc *************************/

	/**
	 * Return smart formatted date.
	 *
	 * @param  int     $ts  file timestamp
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	// protected function formatDate($ts) {

55% of this comment could be valid code. Did you maybe forget this after debugging?

	// 	if ($ts > $this->today) {

50% of this comment could be valid code. Did you maybe forget this after debugging?

	// 		return 'Today '.date($this->options['timeFormat'], $ts);

67% of this comment could be valid code. Did you maybe forget this after debugging?

	// 	}
	//
	// 	if ($ts > $this->yesterday) {

50% of this comment could be valid code. Did you maybe forget this after debugging?

	// 		return 'Yesterday '.date($this->options['timeFormat'], $ts);

67% of this comment could be valid code. Did you maybe forget this after debugging?

	// 	}
	//
	// 	return date($this->options['dateFormat'], $ts);

69% of this comment could be valid code. Did you maybe forget this after debugging?

	// }

	/**
	 * Find position of first occurrence of string in a string with multibyte support.
	 *
	 * @param  string  $haystack  The string being checked.
	 * @param  string  $needle    The string to find in haystack.
	 * @param  int     $offset    The search offset. If it is not specified, 0 is used.
	 * @return int|bool
	 * @author Alexey Sukhotin
	 **/
	protected function stripos($haystack, $needle, $offset = 0)
	{
	    if (function_exists('mb_stripos')) {
	        return mb_stripos($haystack, $needle, $offset, 'UTF-8');
	    } elseif (function_exists('mb_strtolower') && function_exists('mb_strpos')) {
	        return mb_strpos(mb_strtolower($haystack, 'UTF-8'), mb_strtolower($needle, 'UTF-8'), $offset);
	    }

	    return stripos($haystack, $needle, $offset);
	}

	/**
	 * Get server side available archivers.
	 *
	 * @param bool $use_cache
	 * @return array
	 */
	protected function getArchivers($use_cache = true)
	{
	    $sessionKey = 'ARCHIVERS_CACHE';
	    if ($use_cache && isset($this->sessionCache[$sessionKey]) && is_array($this->sessionCache[$sessionKey])) {
	        return $this->sessionCache[$sessionKey];
	    }

	    $arcs = [
			'create' => [],
			'extract' => [],
		];

	    if ($this->procExec('') === 0) {
	        $this->procExec('tar --version', $o, $ctar);

	        if ($ctar == 0) {
	            $arcs['create']['application/x-tar'] = ['cmd' => 'tar', 'argc' => '-cf', 'ext' => 'tar'];
	            $arcs['extract']['application/x-tar'] = ['cmd' => 'tar', 'argc' => '-xf', 'ext' => 'tar', 'toSpec' => '-C '];
	            unset($o);
	            $this->procExec('gzip --version', $o, $c);
	            if ($c == 0) {

This code seems to be duplicated across your project.

	                $arcs['create']['application/x-gzip'] = ['cmd' => 'tar', 'argc' => '-czf', 'ext' => 'tgz'];
	                $arcs['extract']['application/x-gzip'] = ['cmd' => 'tar', 'argc' => '-xzf', 'ext' => 'tgz', 'toSpec' => '-C '];
	            }
	            unset($o);
	            $this->procExec('bzip2 --version', $o, $c);
	            if ($c == 0) {

This code seems to be duplicated across your project.

	                $arcs['create']['application/x-bzip2'] = ['cmd' => 'tar', 'argc' => '-cjf', 'ext' => 'tbz'];
	                $arcs['extract']['application/x-bzip2'] = ['cmd' => 'tar', 'argc' => '-xjf', 'ext' => 'tbz', 'toSpec' => '-C '];
	            }
	            unset($o);
	            $this->procExec('xz --version', $o, $c);
	            if ($c == 0) {

This code seems to be duplicated across your project.

	                $arcs['create']['application/x-xz'] = ['cmd' => 'tar', 'argc' => '-cJf', 'ext' => 'xz'];
	                $arcs['extract']['application/x-xz'] = ['cmd' => 'tar', 'argc' => '-xJf', 'ext' => 'xz', 'toSpec' => '-C '];
	            }
	        }
	        unset($o);
	        $this->procExec('zip -v', $o, $c);
	        if ($c == 0) {

This code seems to be duplicated across your project.

	            $arcs['create']['application/zip'] = ['cmd' => 'zip', 'argc' => '-r9', 'ext' => 'zip'];
	        }
	        unset($o);
	        $this->procExec('unzip --help', $o, $c);
	        if ($c == 0) {

This code seems to be duplicated across your project.

	            $arcs['extract']['application/zip'] = ['cmd' => 'unzip', 'argc' => '',  'ext' => 'zip', 'toSpec' => '-d '];
	        }
	        unset($o);
	        $this->procExec('rar --version', $o, $c);
	        if ($c == 0 || $c == 7) {

This code seems to be duplicated across your project.

	            $arcs['create']['application/x-rar'] = ['cmd' => 'rar', 'argc' => 'a -inul', 'ext' => 'rar'];
	        }
	        unset($o);
	        $this->procExec('unrar', $o, $c);
	        if ($c == 0 || $c == 7) {

This code seems to be duplicated across your project.

	            $arcs['extract']['application/x-rar'] = ['cmd' => 'unrar', 'argc' => 'x -y', 'ext' => 'rar', 'toSpec' => ''];
	        }
	        unset($o);
	        $this->procExec('7za --help', $o, $c);
	        if ($c == 0) {
	            $arcs['create']['application/x-7z-compressed'] = ['cmd' => '7za', 'argc' => 'a', 'ext' => '7z'];
	            $arcs['extract']['application/x-7z-compressed'] = ['cmd' => '7za', 'argc' => 'x -y', 'ext' => '7z', 'toSpec' => '-o'];

	            if (empty($arcs['create']['application/zip'])) {
	                $arcs['create']['application/zip'] = ['cmd' => '7za', 'argc' => 'a -tzip', 'ext' => 'zip'];
	            }
	            if (empty($arcs['extract']['application/zip'])) {

This code seems to be duplicated across your project.

	                $arcs['extract']['application/zip'] = ['cmd' => '7za', 'argc' => 'x -tzip -y', 'ext' => 'zip', 'toSpec' => '-o'];
	            }
	            if (empty($arcs['create']['application/x-tar'])) {
	                $arcs['create']['application/x-tar'] = ['cmd' => '7za', 'argc' => 'a -ttar', 'ext' => 'tar'];
	            }
	            if (empty($arcs['extract']['application/x-tar'])) {

This code seems to be duplicated across your project.

	                $arcs['extract']['application/x-tar'] = ['cmd' => '7za', 'argc' => 'x -ttar -y', 'ext' => 'tar', 'toSpec' => '-o'];
	            }
	        } elseif (substr(PHP_OS, 0, 3) === 'WIN') {
	            // check `7z` for Windows server.
				unset($o);
	            $this->procExec('7z', $o, $c);
	            if ($c == 0) {
	                $arcs['create']['application/x-7z-compressed'] = ['cmd' => '7z', 'argc' => 'a', 'ext' => '7z'];
	                $arcs['extract']['application/x-7z-compressed'] = ['cmd' => '7z', 'argc' => 'x -y', 'ext' => '7z', 'toSpec' => '-o'];

	                if (empty($arcs['create']['application/zip'])) {
	                    $arcs['create']['application/zip'] = ['cmd' => '7z', 'argc' => 'a -tzip', 'ext' => 'zip'];
	                }
	                if (empty($arcs['extract']['application/zip'])) {

This code seems to be duplicated across your project.

	                    $arcs['extract']['application/zip'] = ['cmd' => '7z', 'argc' => 'x -tzip -y', 'ext' => 'zip', 'toSpec' => '-o'];
	                }
	                if (empty($arcs['create']['application/x-tar'])) {
	                    $arcs['create']['application/x-tar'] = ['cmd' => '7z', 'argc' => 'a -ttar', 'ext' => 'tar'];
	                }
	                if (empty($arcs['extract']['application/x-tar'])) {

This code seems to be duplicated across your project.

	                    $arcs['extract']['application/x-tar'] = ['cmd' => '7z', 'argc' => 'x -ttar -y', 'ext' => 'tar', 'toSpec' => '-o'];
	                }
	                if (empty($arcs['extract']['application/x-rar'])) {

This code seems to be duplicated across your project.

	                    $arcs['extract']['application/x-rar'] = ['cmd' => '7z', 'argc' => 'x -trar -y', 'ext' => 'rar', 'toSpec' => '-o'];
	                }
	            }
	        }
	    }

		// Use PHP ZipArchive Class
		if (class_exists('ZipArchive', false)) {
		    if (empty($arcs['create']['application/zip'])) {
		        $arcs['create']['application/zip'] = ['cmd' => 'phpfunction', 'argc' => ['self', 'zipArchiveZip'], 'ext' => 'zip'];
		    }
		    if (empty($arcs['extract']['application/zip'])) {
		        $arcs['extract']['application/zip'] = ['cmd' => 'phpfunction', 'argc' => ['self', 'zipArchiveUnzip'], 'ext' => 'zip'];
		    }
		}

	    $this->sessionCache[$sessionKey] = $arcs;

	    return $arcs;
	}

	/**
	 * Resolve relative / (Unix-like)absolute path.
	 *
	 * @param string $path  target path
	 * @param string $base  base path
	 * @return string
	 */
	protected function getFullPath($path, $base)
	{
	    $separator = $this->separator;
	    $systemroot = $this->systemRoot;

	    if ($base[0] === $separator && strpos($base, 0, strlen($systemroot)) !== $systemroot) {

The strict comparison !== seems to always evaluate to true as the types of strpos($base, 0, strlen($systemroot)) (integer) and $systemroot (string) can never be identical. Maybe you want to use a loose comparison != instead?

	        $base = $systemroot.substr($base, 1);
	    }

		// 'Here'
		if ($path === '' || $path === '.'.$separator) {
		    return $base;
		}

	    $sepquoted = preg_quote($separator, '#');

	    if (substr($path, 0, 3) === '..'.$separator) {
	        $path = $base.$separator.$path;
	    }
		// normalize `/../`
		$normreg = '#('.$sepquoted.')[^'.$sepquoted.']+'.$sepquoted.'\.\.'.$sepquoted.'#'; // '#(/)[^\/]+/\.\./#'
		while (preg_match($normreg, $path)) {
		    $path = preg_replace($normreg, '$1', $path, 1);
		}

		// Absolute path
		if ($path[0] === $separator || strpos($path, $systemroot) === 0) {
		    return $path;
		}

	    $preg_separator = '#'.$sepquoted.'#';

		// Relative path from 'Here'
		if (substr($path, 0, 2) === '.'.$separator || $path[0] !== '.') {
		    $arrn = preg_split($preg_separator, $path, -1, PREG_SPLIT_NO_EMPTY);
		    if ($arrn[0] !== '.') {
		        array_unshift($arrn, '.');
		    }
		    $arrn[0] = $base;

		    return implode($separator, $arrn);
		}

	    return $path;
	}

	/**
	 * Create archive and return its path.
	 *
	 * @param  string  $dir    target dir
	 * @param  array   $files  files names list
	 * @param  string  $name   archive name
	 * @param  array   $arc    archiver options
	 * @return string|bool
	 * @author Dmitry (dio) Levashov,
	 * @author Alexey Sukhotin
	 * @author Naoki Sawada
	 **/
	protected function makeArchive($dir, $files, $name, $arc)
	{
	    if ($arc['cmd'] === 'phpfunction') {
	        if (is_callable($arc['argc'])) {

This code seems to be duplicated across your project.

	            call_user_func_array($arc['argc'], [$dir, $files, $name]);
	        }
	    } else {
	        $cwd = getcwd();
	        if (chdir($dir)) {
	            foreach ($files as $i => $file) {
	                $files[$i] = '.'.DIRECTORY_SEPARATOR.$file;
	            }
	            $files = array_map('escapeshellarg', $files);

	            $cmd = $arc['cmd'].' '.$arc['argc'].' '.escapeshellarg($name).' '.implode(' ', $files);
	            $this->procExec($cmd, $o, $c);
	            chdir($cwd);
	        } else {
	            return false;
	        }
	    }
	    $path = $dir.DIRECTORY_SEPARATOR.$name;

	    return file_exists($path) ? $path : false;
	}

	/**
	 * Unpack archive.
	 *
	 * @param  string      $path  archive path
	 * @param  array       $arc   archiver command and arguments (same as in $this->archivers)
	 * @param  bool|string $mode  bool: remove archive ( unlink($path) ) | string: extract to directory
	 * @return void
	 * @author Dmitry (dio) Levashov
	 * @author Alexey Sukhotin
	 * @author Naoki Sawada
	 **/
	protected function unpackArchive($path, $arc, $mode = true)
	{
	    if (is_string($mode)) {
	        $dir = $mode;
	        $chdir = null;
	        $remove = false;
	    } else {
	        $dir = dirname($path);
	        $chdir = $dir;
	        $remove = $mode;
	    }
	    $dir = realpath($dir);
	    $path = realpath($path);
	    if ($arc['cmd'] === 'phpfunction') {
	        if (is_callable($arc['argc'])) {

This code seems to be duplicated across your project.

	            call_user_func_array($arc['argc'], [$path, $dir]);
	        }
	    } else {
	        $cwd = getcwd();
	        if (! $chdir || chdir($dir)) {

The expression $chdir of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

	            if ($chdir) {

The expression $chdir of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

	                $cmd = $arc['cmd'].' '.$arc['argc'].' '.escapeshellarg(basename($path));
	            } else {
	                $cmd = $arc['cmd'].' '.$arc['argc'].' '.escapeshellarg($path).' '.$arc['toSpec'].escapeshellarg($dir);
	            }
	            $this->procExec($cmd, $o, $c);
	            $chdir && chdir($cwd);

The expression $chdir of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

	        }
	    }
	    $remove && unlink($path);
	}

	/**
	 * Check and filter the extracted items.
	 *
	 * @param  string $path    target local path
	 * @param  array  $checks  types to check default: ['symlink', 'name', 'writable', 'mime']
	 * @return array  ['symlinks' => [], 'names' => [], 'writables' => [], 'mimes' => [], 'rmNames' => [], 'totalSize' => 0]
	 * @author Naoki Sawada
	 */
	protected function checkExtractItems($path, $checks = null)
	{
	    if (is_null($checks) || ! is_array($checks)) {
	        $checks = ['symlink', 'name', 'writable', 'mime'];
	    }
	    $chkSymlink = in_array('symlink', $checks);
	    $chkName = in_array('name', $checks);
	    $chkWritable = in_array('writable', $checks);
	    $chkMime = in_array('mime', $checks);

	    $res = [
			'symlinks' => [],
			'names' => [],
			'writables' => [],
			'mimes' => [],
			'rmNames' => [],
			'totalSize' => 0,
		];

	    if (is_dir($path)) {
	        foreach (self::localScandir($path) as $name) {
	            $p = $path.DIRECTORY_SEPARATOR.$name;
	            if ($chkSymlink && is_link($p)) {
	                self::localRmdirRecursive($p);
	                $res['symlinks'][] = $p;
	                $res['rmNames'][] = $name;
	                continue;
	            }
	            $isDir = is_dir($p);
	            if ($chkName && ! $this->nameAccepted($name, $isDir)) {
	                self::localRmdirRecursive($p);
	                $res['names'][] = $p;
	                $res['rmNames'][] = $name;
	                continue;
	            }
	            if ($chkWritable && ! $this->attr($p, 'write', null, $isDir)) {
	                self::localRmdirRecursive($p);
	                $res['writables'][] = $p;
	                $res['rmNames'][] = $name;
	                continue;
	            }
	            if ($chkMime && ($mimeByName = self::mimetypeInternalDetect($name)) && $mimeByName !== 'unknown' && ! $this->allowPutMime($mimeByName)) {

This code seems to be duplicated across your project.

	                self::localRmdirRecursive($p);
	                $res['mimes'][] = $p;
	                $res['rmNames'][] = $name;
	                continue;
	            }
	            if ($isDir) {
	                $cRes = $this->checkExtractItems($p, $checks);
	                foreach ($cRes as $k => $v) {
	                    if (is_array($v)) {
	                        $res[$k] = array_merge($res[$k], $cRes[$k]);
	                    } else {
	                        $res[$k] += $cRes[$k];
	                    }
	                }
	            } else {
	                $res['totalSize'] += sprintf('%u', filesize($p));
	            }
	        }
	        $res['rmNames'] = array_unique($res['rmNames']);
	    } else {
	        if ($chkSymlink && is_link($path)) {
	            unlink($path);
	            $res['symlinks'][] = $path;
	            $res['rmNames'][] = basename($path);
	        } elseif ($chkName && ! $this->nameAccepted($name, false)) {

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

	            unlink($path);
	            $res['names'][] = $path;
	            $res['rmNames'][] = $name;

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

	        } elseif ($chkWritable && ! $this->attr($path, 'write', null, $isDir)) {

The variable $isDir seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

	            unlink($path);
	            $res['writables'][] = $path;
	            $res['rmNames'][] = $name;

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

	        } elseif ($chkMime && ($mimeByName = self::mimetypeInternalDetect($name)) && $mimeByName !== 'unknown' && ! $this->allowPutMime($mimeByName)) {

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

This code seems to be duplicated across your project.

	            unlink($path);
	            $res['mimes'][] = $path;
	            $res['rmNames'][] = $name;

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

	        } else {
	            $res['totalSize'] += sprintf('%u', filesize($path));
	        }
	    }

	    return $res;
	}

	/**
	 * Return files of target directory that is dotfiles excludes.
	 *
	 * @param  string $dir target directory path
	 * @return array
	 * @throws Exception
	 * @author Naoki Sawada
	 */
	protected static function localScandir($dir)
	{
	    // PHP function scandir() is not work well in specific environment. I dont know why.
		// ref. https://github.com/Studio-42/elFinder/issues/1248
		$files = [];
	    if ($dh = opendir($dir)) {
	        while (false !== ($file = readdir($dh))) {
	            if ($file !== '.' && $file !== '..') {
	                $files[] = $file;
	            }
	        }
	        closedir($dh);
	    } else {
	        throw new Exception('Can not open local directory.');
	    }

	    return $files;
	}

	/**
	 * Remove directory recursive on local file system.
	 *
	 * @param string $dir Target dirctory path
	 * @return bool
	 * @author Naoki Sawada
	 */
	protected static function localRmdirRecursive($dir)
	{
	    if (! is_link($dir) && is_dir($dir)) {
	        chmod($dir, 0777);
	        if ($handle = opendir($dir)) {
	            while (false !== ($file = readdir($handle))) {
	                if ($file === '.' || $file === '..') {
	                    continue;
	                }
	                elFinder::extendTimeLimit(30);
	                $path = $dir.DIRECTORY_SEPARATOR.$file;
	                if (! is_link($dir) && is_dir($path)) {
	                    self::localRmdirRecursive($path);
	                } else {
	                    chmod($path, 0666);
	                    unlink($path);
	                }
	            }
	            closedir($handle);
	        }

	        return rmdir($dir);
	    } elseif (is_file($dir) || is_link($dir)) {
	        chmod($dir, 0666);

	        return unlink($dir);
	    }

	    return false;
	}

	/**
	 * Move item recursive on local file system.
	 *
	 * @param string $src
	 * @param string $target
	 * @param string $overWrite
	 * @param string $copyJoin
	 * @return bool
	 * @author Naoki Sawada
	 */
	protected static function localMoveRecursive($src, $target, $overWrite = true, $copyJoin = true)
	{
	    $res = false;
	    if (! file_exists($target)) {
	        return rename($src, $target);
	    }
	    if (! $copyJoin || ! is_dir($target)) {
	        if ($overWrite) {
	            if (is_dir($target)) {
	                $del = self::localRmdirRecursive($target);
	            } else {
	                $del = unlink($target);
	            }
	            if ($del) {
	                return rename($src, $target);
	            }
	        }
	    } else {
	        foreach (self::localScandir($src) as $item) {
	            $res |= self::localMoveRecursive($src.DIRECTORY_SEPARATOR.$item, $target.DIRECTORY_SEPARATOR.$item, $overWrite, $copyJoin);
	        }
	    }

	    return (bool) $res;
	}

	/**
	 * Create Zip archive using PHP class ZipArchive.
	 *
	 * @param  string        $dir      target dir
	 * @param  array         $files    files names list
	 * @param  string|object $zipPath  Zip archive name
	 * @return bool
	 * @author Naoki Sawada
	 */
	protected static function zipArchiveZip($dir, $files, $zipPath)
	{
	    try {
	        if ($start = is_string($zipPath)) {
	            $zip = new ZipArchive();
	            if ($zip->open($dir.DIRECTORY_SEPARATOR.$zipPath, ZipArchive::CREATE) !== true) {
	                $zip = false;
	            }
	        } else {
	            $zip = $zipPath;
	        }
	        if ($zip) {
	            foreach ($files as $file) {
	                $path = $dir.DIRECTORY_SEPARATOR.$file;
	                if (is_dir($path)) {
	                    $zip->addEmptyDir($file);
	                    $_files = [];
	                    if ($handle = opendir($path)) {
	                        while (false !== ($entry = readdir($handle))) {
	                            if ($entry !== '.' && $entry !== '..') {
	                                $_files[] = $file.DIRECTORY_SEPARATOR.$entry;
	                            }
	                        }
	                        closedir($handle);
	                    }
	                    if ($_files) {

The expression $_files of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	                        self::zipArchiveZip($dir, $_files, $zip);
	                    }
	                } else {
	                    $zip->addFile($path, $file);
	                }
	            }
	            $start && $zip->close();
	        }
	    } catch (Exception $e) {
	        return false;
	    }

	    return true;
	}

	/**
	 * Unpack Zip archive using PHP class ZipArchive.
	 *
	 * @param  string $zipPath  Zip archive name
	 * @param  string $toDir    Extract to path
	 * @return bool
	 * @author Naoki Sawada
	 */
	protected static function zipArchiveUnzip($zipPath, $toDir)
	{
	    try {
	        $zip = new ZipArchive();
	        if ($zip->open($zipPath) === true) {
	            $zip->extractTo($toDir);
	            $zip->close();
	        }
	    } catch (Exception $e) {
	        return false;
	    }

	    return true;
	}

	/**
	 * Recursive symlinks search.
	 *
	 * @param  string  $path  file/dir path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	protected static function localFindSymlinks($path)
	{
	    if (is_link($path)) {
	        return true;
	    }

	    if (is_dir($path)) {
	        foreach (self::localScandir($path) as $name) {
	            $p = $path.DIRECTORY_SEPARATOR.$name;
	            if (is_link($p)) {
	                return true;
	            }
	            if (is_dir($p) && $this->_findSymlinks($p)) {

The variable $this does not exist. Did you forget to declare it?

	                return true;
	            }
	        }
	    }

	    return false;
	}

	/**==================================* abstract methods *====================================**/

	/*********************** paths/urls *************************/

	/**
	 * Return parent directory path.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _dirname($path);

	/**
	 * Return file name.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _basename($path);

	/**
	 * Join dir name and file name and return full path.
	 * Some drivers (db) use int as path - so we give to concat path to driver itself.
	 *
	 * @param  string  $dir   dir path
	 * @param  string  $name  file name
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _joinPath($dir, $name);

	/**
	 * Return normalized path.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _normpath($path);

	/**
	 * Return file path related to root dir.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _relpath($path);

	/**
	 * Convert path related to root dir into real path.
	 *
	 * @param  string  $path  rel file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _abspath($path);

	/**
	 * Return fake path started from root dir.
	 * Required to show path on client side.
	 *
	 * @param  string  $path  file path
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _path($path);

	/**
	 * Return true if $path is children of $parent.
	 *
	 * @param  string  $path    path to check
	 * @param  string  $parent  parent path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _inpath($path, $parent);

	/**
	 * Return stat for given path.
	 * Stat contains following fields:
	 * - (int)    size    file size in b. required
	 * - (int)    ts      file modification time in unix time. required
	 * - (string) mime    mimetype. required for folders, others - optionally
	 * - (bool)   read    read permissions. required
	 * - (bool)   write   write permissions. required
	 * - (bool)   locked  is object locked. optionally
	 * - (bool)   hidden  is object hidden. optionally
	 * - (string) alias   for symlinks - link target path relative to root path. optionally
	 * - (string) target  for symlinks - link target path. optionally.
	 *
	 * If file does not exists - returns empty array or false.
	 *
	 * @param  string  $path    file path
	 * @return array|false
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _stat($path);

	/***************** file stat ********************/

	/**
	 * Return true if path is dir and has at least one childs directory.
	 *
	 * @param  string  $path  dir path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _subdirs($path);

	/**
	 * Return object width and height
	 * Ususaly used for images, but can be realize for video etc...
	 *
	 * @param  string  $path  file path
	 * @param  string  $mime  file mime type
	 * @return string
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _dimensions($path, $mime);

	/******************** file/dir content *********************/

	/**
	 * Return files list in directory.
	 *
	 * @param  string  $path  dir path
	 * @return array
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _scandir($path);

	/**
	 * Open file and return file pointer.
	 *
	 * @param  string $path file path
	 * @param  string $mode open mode
	 * @return resource|false
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _fopen($path, $mode = 'rb');

	/**
	 * Close opened file.
	 *
	 * @param  resource  $fp    file pointer
	 * @param  string    $path  file path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _fclose($fp, $path = '');

	/********************  file/dir manipulations *************************/

	/**
	 * Create dir and return created dir path or false on failed.
	 *
	 * @param  string  $path  parent dir path
	 * @param string  $name  new directory name
	 * @return string|bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _mkdir($path, $name);

	/**
	 * Create file and return it's path or false on failed.
	 *
	 * @param  string  $path  parent dir path
	 * @param string  $name  new file name
	 * @return string|bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _mkfile($path, $name);

	/**
	 * Create symlink.
	 *
	 * @param  string  $source     file to link to
	 * @param  string  $targetDir  folder to create link in
	 * @param  string  $name       symlink name
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _symlink($source, $targetDir, $name);

	/**
	 * Copy file into another file (only inside one volume).
	 *
	 * @param  string $source source file path
	 * @param $targetDir
	 * @param  string $name file name
	 * @return bool|string
	 * @internal param string $target target dir path
	 * @author Dmitry (dio) Levashov
	 */
	abstract protected function _copy($source, $targetDir, $name);

	/**
	 * Move file into another parent dir.
	 * Return new file path or false.
	 *
	 * @param  string $source source file path
	 * @param $targetDir
	 * @param  string $name file name
	 * @return bool|string
	 * @internal param string $target target dir path
	 * @author Dmitry (dio) Levashov
	 */
	abstract protected function _move($source, $targetDir, $name);

	/**
	 * Remove file.
	 *
	 * @param  string  $path  file path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _unlink($path);

	/**
	 * Remove dir.
	 *
	 * @param  string  $path  dir path
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _rmdir($path);

	/**
	 * Create new file and write into it from file pointer.
	 * 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
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _save($fp, $dir, $name, $stat);

	/**
	 * Get file contents.
	 *
	 * @param  string  $path  file path
	 * @return string|false
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _getContents($path);

	/**
	 * Write a string to a file.
	 *
	 * @param  string  $path     file path
	 * @param  string  $content  new file content
	 * @return bool
	 * @author Dmitry (dio) Levashov
	 **/
	abstract protected function _filePutContents($path, $content);

	/**
	 * Extract files from archive.
	 *
	 * @param  string  $path file path
	 * @param  array   $arc  archiver options
	 * @return bool
	 * @author Dmitry (dio) Levashov,
	 * @author Alexey Sukhotin
	 **/
	abstract protected function _extract($path, $arc);

	/**
	 * Create archive and return its path.
	 *
	 * @param  string  $dir    target dir
	 * @param  array   $files  files names list
	 * @param  string  $name   archive name
	 * @param  array   $arc    archiver options
	 * @return string|bool
	 * @author Dmitry (dio) Levashov,
	 * @author Alexey Sukhotin
	 **/
	abstract protected function _archive($dir, $files, $name, $arc);

	/**
	 * Detect available archivers.
	 *
	 * @return void
	 * @author Dmitry (dio) Levashov,
	 * @author Alexey Sukhotin
	 **/
	abstract protected function _checkArchivers();

	/**
	 * Change file mode (chmod).
	 *
	 * @param  string  $path  file path
	 * @param  string  $mode  octal string such as '0755'
	 * @return bool
	 * @author David Bartle,
	 **/
	abstract protected function _chmod($path, $mode);
} // END class