From e56abaca4fddf3f000e1dca183ee1ffaefb85662 Mon Sep 17 00:00:00 2001
From: Will Rossiter <will.rossiter@gmail.com>
Date: Sat, 11 May 2013 18:31:02 +1200
Subject: [PATCH] Suggest users install curl, tidy on their machines.

---
 dev/install/install.php5 | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/dev/install/install.php5 b/dev/install/install.php5
index 7e02105da..23565b44b 100644
--- a/dev/install/install.php5
+++ b/dev/install/install.php5
@@ -459,6 +459,10 @@ class InstallRequirements {
 
 		$this->suggestClass('finfo', array('PHP Configuration', 'fileinfo support', 'fileinfo should be enabled in PHP. SilverStripe uses it for MIME type detection of files. SilverStripe will still operate, but email attachments and sending files to browser (e.g. export data to CSV) may not work correctly without finfo.'));
 
+		$this->suggestFunction('curl_init', array('PHP Configuration', 'curl support', 'curl should be enabled in PHP. SilverStripe uses it for consuming web services via the RestfulService class and many modules rely on it.'));
+
+		$this->suggestClass('tidy', array('PHP Configuration', 'tidy support', 'Tidy provides a library of code to clean up your html. SilverStripe will operate fine without tidy but HTMLCleaner will not be effective.'));
+
 		$this->suggestPHPSetting('asp_tags', array(false,0,''), array('PHP Configuration', 'asp_tags option', 'This should be turned off as it can cause issues with SilverStripe'));
 		$this->suggestPHPSetting('magic_quotes_gpc', array(false,0,''), array('PHP Configuration', 'magic_quotes_gpc option', 'This should be turned off, as it can cause issues with cookies. More specifically, unserializing data stored in cookies.'));
 		$this->suggestPHPSetting('display_errors', array(false,0,''), array('PHP Configuration', 'display_errors option', 'Unless you\'re in a development environment, this should be turned off, as it can expose sensitive data to website users.'));
@@ -487,6 +491,14 @@ class InstallRequirements {
 		}
 	}
 
+	function suggestFunction($class, $testDetails) {
+		$this->testing($testDetails);
+
+		if(!function_exists($class)) {
+			$this->warning($testDetails);
+		}
+	}
+
 	function requireDateTimezone($testDetails) {
 		$this->testing($testDetails);