PR: https://github.com/mantisbt/mantisbt/pull/1263

@vboctor isn't this fixed in version 2.9.0 0023639 ?
https://github.com/mantisbt/mantisbt/commit/5f6292fbabea4de238cb10cb28e3449f71fa8679

@atrol You partially addressed finfo, but I did more. I think there is also more cleanup to be done here. For example, utility.php has finfo_get_if_available, where finfo is always available, I added methods to get mime type in file_api that we probably should use instead of using finfo_get_if_available, etc.

I think there is also more cleanup to be done here.

Right

To reduce redundant code, wouldn't it be better to give function finfo_get_if_available a better name (remove _if_available) and use it in your new functions file_get_mime_type and file_get_mime_type_for_content (bad name btw)?

Will also add comments for this in PR.