Skip to content

Conversation

@billynoah
Copy link
Contributor

Rewrite logic for to get mimeDetect for clarity and to resolve a small bug.

This line causes a bug where it becomes impossible to set mimeDetect to 'mime_content_type':

$type = 'internal';

The logic here first makes an array of methods available in the current environment. Then we check to see if the type set is among them. If not, set to auto. If set to auto, shift the first element from available types.

Rewrite logic for to get mimeDetect for clarity and to resolve a small bug.

This line causes a bug where it becomes impossible to set mimeDetect to 'mime_content_type': https://github.com/Studio-42/elFinder/blob/33eea861cb07196edbde5d9188f65a6ce81be21f/php/elFinderVolumeDriver.class.php#L1290

The logic here first makes an array of methods available in the current environment.  Then we check to see if the type set is among them.  If not, set to auto.  If set to auto, shift the first element from available types.
@nao-pon nao-pon merged commit 398d29c into Studio-42:master Jun 8, 2021
@nao-pon
Copy link
Member

nao-pon commented Jun 8, 2021

@billynoah Thank you for fixing a bug. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants