[ISSUE] When maxTargets = 1. Elfinder can't call loadThumbnails if images are more than one.

[ethaizone]

Reproduce step.

1. Set option maxTargets be 1.
2. Upload multiple images by drag and drop.
3. Image uploaded success but problem is incoming. (LOL)

After that ELFinder try to refresh UI and call loadThumbnails

elFinder/js/ui/cwd.js

Line 1254 in 632bc97

loadThumbnails = function(reloads) {

then request will call

elFinder/js/ui/cwd.js

Lines 1285 to 1288 in 632bc97

	fm.request({
	data : {cmd : 'tmb', targets : tmbs},
	preventFail : true
	})

but it will stop because these lines.

elFinder/js/elFinder.js

Lines 2272 to 2275 in 632bc97

	if (self.maxTargets && data.targets && data.targets.length > self.maxTargets) {
	syncOnFail = false;
	return dfrd.reject(['errMaxTargets', self.maxTargets]);
	}

after that UI have problems.

• Can't show thumbnail.
• Can't delete file or directory.
• Can't create new directory.

[ethaizone]

Right now I make stupid patch for use right now by add cmd != "tmb" inside if like this.

if (cmd != "tmb" && self.maxTargets && data.targets && data.targets.length > self.maxTargets) {
	syncOnFail = false;
	return dfrd.reject(['errMaxTargets', self.maxTargets]);
}

I don't think this is good solution so I create this issue.

[nao-pon]

@ethaizone On the handling of maxTargets option some problems on the client side were found and I fixed it. Thanks! 👍

[ethaizone]

@ethaizone On the handling of maxTargets option some problems on the client side were found and I fixed it. Thanks! 👍

Good to hear. Thx. :)