From e9a2fc66d635a976962471b95e6252f4361dcecf Mon Sep 17 00:00:00 2001
From: Hugh Bragg <hughbragg@dodo.com.au>
Date: Fri, 5 Apr 2013 12:18:21 +1000
Subject: [PATCH] Graceful degradation for mime type determination

---
 controller/site/thumbs.process.php      |  6 +++---
 controller/site/userPicture.process.php |  5 ++---
 lib/getMimeType.php                     | 24 ++++++++++++++++++++++++
 model/classes/upload.class.php          |  6 +++---
 4 files changed, 32 insertions(+), 9 deletions(-)
 create mode 100644 lib/getMimeType.php

diff --git a/controller/site/thumbs.process.php b/controller/site/thumbs.process.php
index 5891bef..bcabdcd 100644
--- a/controller/site/thumbs.process.php
+++ b/controller/site/thumbs.process.php
@@ -1,12 +1,12 @@
 <?php
 
+require_once SYSTEM_PATH."/lib/getMimeType.php";
+
 // temp variable for uploaded file (and path)
 $uploadedFile = $targetDir . DIRECTORY_SEPARATOR . $fileName;
 
 // get MIME type
-$finfo = finfo_open(FILEINFO_MIME_TYPE);
-$mime = finfo_file($finfo, $uploadedFile);
-finfo_close($finfo);
+$mime = getMimeType($uploadedFile);
 
 // get extension
 $ext = pathinfo($uploadedFile, PATHINFO_EXTENSION);
diff --git a/controller/site/userPicture.process.php b/controller/site/userPicture.process.php
index a5e2dca..1d2ff0b 100644
--- a/controller/site/userPicture.process.php
+++ b/controller/site/userPicture.process.php
@@ -1,5 +1,6 @@
 <?php
 require_once("../../global.php");
+require_once SYSTEM_PATH."/lib/getMimeType.php";
 
 if(!Session::isLoggedIn()) {
 	$json = array('error' => 'You must be logged in.');
@@ -168,9 +169,7 @@
 	$uploadedFile = $targetDir . DIRECTORY_SEPARATOR . $fileName;
 
 	// get MIME type
-	$finfo = finfo_open(FILEINFO_MIME_TYPE);
-	$mime = finfo_file($finfo, $uploadedFile);
-	finfo_close($finfo);
+	$mime = getMimeType($uploadedFile);
 
 	// get extension
 	$ext = pathinfo($uploadedFile, PATHINFO_EXTENSION);
diff --git a/lib/getMimeType.php b/lib/getMimeType.php
new file mode 100644
index 0000000..aa7f30e
--- /dev/null
+++ b/lib/getMimeType.php
@@ -0,0 +1,24 @@
+<?php
+// Hugh Bragg
+function getMimeType($filename)
+{
+	$mimetype = false;
+	if(function_exists('finfo_open')) {
+		// open with FileInfo
+		$finfo = finfo_open(FILEINFO_MIME_TYPE);
+		$mimetype = finfo_file($finfo, $filename);
+		finfo_close($finfo);
+	} elseif(function_exists('exif_imagetype')) {
+		// open with EXIF
+		$extifType = exif_imagetype($filename);
+		$mimetype =  image_type_to_mime_type($extifType);
+	} elseif(function_exists('getimagesize')) {
+		// open with GD
+		$info = getimagesize($filename);
+		$mimetype = $info['mime'];
+	} elseif(function_exists('mime_content_type')) {
+		// PHP < 5.3
+		$mimetype = mime_content_type($filename);
+	}
+	return $mimetype;
+}
\ No newline at end of file
diff --git a/model/classes/upload.class.php b/model/classes/upload.class.php
index 1d13a3f..6f5562e 100644
--- a/model/classes/upload.class.php
+++ b/model/classes/upload.class.php
@@ -1,5 +1,7 @@
 <?php
 
+require_once SYSTEM_PATH."/lib/getMimeType.php";
+
 class Upload extends DbObject
 {
 	protected $id;
@@ -190,9 +192,7 @@ public static function saveToDatabase($originalName=null, $storedName=null, $ite
 		// get file size
 		$size = filesize($absPath);
 		// get mime type
-		$finfo = finfo_open(FILEINFO_MIME_TYPE);
-		$mime = finfo_file($finfo, $absPath);
-		finfo_close($finfo);	
+		$mime = getMimeType($absPath);
 		// get height and width (if image)
 		$imgSize = getimagesize($absPath);
 		if($imgSize) {