Command can be disabled on mutualized servers

[jerrywham]

No description provided.

[lGuillaume124]

Hi,

Thank you for your contribution ! We were not expecting so much success and we need to organise the repository and its branches before accept your contribution.

[MightyCreak]

I'm not sure about the function name is_shell_exec_available which just seems to check if shell_exec is available, but it also run the given command. This is pretty misleading.

It might be better to have a ShellExec component which would have an exec method that will first verify (in the constructor?) if shell_exec availability before executing the given command.

[MightyCreak]

About the exif_read_data commit (fd0ded2).

First, it mixes CheckCmd changes and the actual bug. The CheckCmd changes should be in your first commit (f488700).

I don't think this bugfix should be in this PR, but should have it's own PR. It would be cleaner.

Finally, the exif_read_data documentation doesn't mention handling gif images, but you added it in the extension array.

[jerrywham]

@MightyCreak : I agree with all what you said. Sorry for these mistakes.

[MightyCreak]

No worries ;)

I'm just helping the Sonerezh devs, hoping they'll find time to merge all these interesting PR.