9 files changed, 252 insertions, 169 deletions
diff --git a/phpBB/adm/style/acp_ext_details.html b/phpBB/adm/style/acp_ext_details.html
index 776c44711c..5af9603a75 100644
--- a/phpBB/adm/style/acp_ext_details.html
+++ b/phpBB/adm/style/acp_ext_details.html
@@ -6,10 +6,12 @@
 
 	<fieldset>
 		<legend>{L_EXT_DETAILS}</legend>
+		<!-- IF MD_DISPLAY_NAME -->
 		<dl>
 			<dt><label for="md_display_name">{L_DISPLAY_NAME}:</label></dt>
 			<dd><strong id="md_display_name">{MD_DISPLAY_NAME}</strong></dd>
 		</dl>
+		<!-- ENDIF -->
 		<dl>
 			<dt><label for="md_name">{L_CLEAN_NAME}:</label></dt>
 			<dd><strong id="md_name">{MD_NAME}</strong></dd>
@@ -43,7 +45,7 @@
 			<dd><p id="md_license">{MD_LICENCE}</p></dd>
 		</dl>
 	</fieldset>
-	
+
 	<fieldset>
 		<legend>{L_REQUIREMENTS}</legend>
 		<!-- IF MD_REQUIRE_PHPBB -->
@@ -85,9 +87,9 @@
 			<dd><strong id="md_author_role">{md_authors.AUTHOR_ROLE}</strong></dd>
 		</dl>
 		<!-- ENDIF -->
-		
+
 		<br /><br />
 		<!-- END md_authors -->
 	</fieldset>
-	
+
 <!-- INCLUDE overall_footer.html -->
diff --git a/phpBB/common.php b/phpBB/common.php
index c7c5859c25..fc2b9ae21e 100644
--- a/phpBB/common.php
+++ b/phpBB/common.php
@@ -119,7 +119,7 @@ set_config(null, null, null, $config);
 set_config_count(null, null, null, $config);
 
 // load extensions
-$phpbb_extension_manager = new phpbb_extension_manager($db, EXT_TABLE, $phpbb_root_path, ".$phpEx", $cache->get_driver());
+$phpbb_extension_manager = new phpbb_extension_manager($db, $config, EXT_TABLE, $phpbb_root_path, ".$phpEx", $cache->get_driver());
 
 // Initialize style
 $phpbb_style_resource_locator = new phpbb_style_resource_locator();
diff --git a/phpBB/download/file.php b/phpBB/download/file.php
index b56d729a7b..706df6abab 100644
--- a/phpBB/download/file.php
+++ b/phpBB/download/file.php
@@ -77,7 +77,7 @@ if (isset($_GET['avatar']))
 	set_config_count(null, null, null, $config);
 
 	// load extensions
-	$phpbb_extension_manager = new phpbb_extension_manager($db, EXT_TABLE, $phpbb_root_path, ".$phpEx", $cache->get_driver());
+	$phpbb_extension_manager = new phpbb_extension_manager($db, $config, EXT_TABLE, $phpbb_root_path, ".$phpEx", $cache->get_driver());
 
 	$phpbb_subscriber_loader = new phpbb_event_extension_subscriber_loader($phpbb_dispatcher, $phpbb_extension_manager);
 	$phpbb_subscriber_loader->load();
diff --git a/phpBB/includes/acp/acp_extensions.php b/phpBB/includes/acp/acp_extensions.php
index c4d9497956..ce32640c33 100644
--- a/phpBB/includes/acp/acp_extensions.php
+++ b/phpBB/includes/acp/acp_extensions.php
@@ -25,7 +25,7 @@ class acp_extensions
 	function main()
 	{
 		// Start the page
-		global $user, $template, $request, $phpbb_extension_manager, $db, $phpbb_root_path, $phpEx;
+		global $config, $user, $template, $request, $phpbb_extension_manager, $db, $phpbb_root_path, $phpEx;
 
 		$user->add_lang(array('install', 'acp/extensions'));
 
@@ -34,6 +34,17 @@ class acp_extensions
 		$action = $request->variable('action', 'list');
 		$ext_name = $request->variable('ext_name', '');
 
+		// If they've specificed an extension, let's load the metadata manager and validate it.
+		if ($ext_name)
+		{
+			$md_manager = new phpbb_extension_metadata_manager($ext_name, $db, $phpbb_extension_manager, $phpbb_root_path, ".$phpEx", $template, $config);
+
+			if ($md_manager->get_metadata('all') === false)
+			{
+				trigger_error('EXTENSION_INVALID');
+			}
+		}
+
 		// What are we doing?
 		switch ($action)
 		{
@@ -47,6 +58,11 @@ class acp_extensions
 			break;
 
 			case 'enable_pre':
+				if (!$md_manager->validate_enable())
+				{
+					trigger_error('EXTENSION_NOT_AVAILABLE');
+				}
+
 				$this->tpl_name = 'acp_ext_enable';
 
 				$template->assign_vars(array(
@@ -56,10 +72,15 @@ class acp_extensions
 			break;
 
 			case 'enable':
+				if (!$md_manager->validate_enable())
+				{
+					trigger_error('EXTENSION_NOT_AVAILABLE');
+				}
+
 				if ($phpbb_extension_manager->enable_step($ext_name))
 				{
 					$template->assign_var('S_NEXT_STEP', true);
-					
+
 					meta_refresh(0, $this->u_action . '&amp;action=enable&amp;ext_name=' . $ext_name);
 				}
 
@@ -76,14 +97,14 @@ class acp_extensions
 				$template->assign_vars(array(
 					'PRE'		=> true,
 					'U_DISABLE'	=> $this->u_action . '&amp;action=disable&amp;ext_name=' . $ext_name,
-				));				
-			break;	
+				));
+			break;
 
 			case 'disable':
 				if ($phpbb_extension_manager->disable_step($ext_name))
 				{
 					$template->assign_var('S_NEXT_STEP', true);
-					
+
 					meta_refresh(0, $this->u_action . '&amp;action=disable&amp;ext_name=' . $ext_name);
 				}
 
@@ -101,13 +122,13 @@ class acp_extensions
 					'PRE'		=> true,
 					'U_PURGE'	=> $this->u_action . '&amp;action=purge&amp;ext_name=' . $ext_name,
 				));
-			break;		
+			break;
 
 			case 'purge':
 				if ($phpbb_extension_manager->purge_step($ext_name))
 				{
 					$template->assign_var('S_NEXT_STEP', true);
-					
+
 					meta_refresh(0, $this->u_action . '&amp;action=purge&amp;ext_name=' . $ext_name);
 				}
 
@@ -132,12 +153,8 @@ class acp_extensions
 			break;*/
 
 			case 'details':
-				$md_manager = new phpbb_extension_metadata_manager($ext_name, $db, $phpbb_extension_manager, $phpbb_root_path, ".$phpEx", $template);
-				
-				if ($md_manager->get_metadata('all', true) === false)
-				{
-					trigger_error('EXTENSION_INVALID');
-				}
+				// Output it to the template
+				$md_manager->output_template_data();
 
 				$this->tpl_name = 'acp_ext_details';
 			break;
@@ -146,7 +163,7 @@ class acp_extensions
 
 	/**
 	 * Lists all the enabled extensions and dumps to the template
-	 * 
+	 *
 	 * @param  $phpbb_extension_manager     An instance of the extension manager
 	 * @param  $template 					An instance of the template engine
 	 * @return null
@@ -156,7 +173,7 @@ class acp_extensions
 		foreach ($phpbb_extension_manager->all_enabled() as $name => $location)
 		{
 			$md_manager = $phpbb_extension_manager->get_extension_metadata($name, $template);
-			
+
 			$template->assign_block_vars('enabled', array(
 				'EXT_NAME'		=> $md_manager->get_metadata('display-name'),
 
@@ -169,7 +186,7 @@ class acp_extensions
 
 	/**
 	 * Lists all the disabled extensions and dumps to the template
-	 * 
+	 *
 	 * @param  $phpbb_extension_manager     An instance of the extension manager
 	 * @param  $template 					An instance of the template engine
 	 * @return null
@@ -179,7 +196,7 @@ class acp_extensions
 		foreach ($phpbb_extension_manager->all_disabled() as $name => $location)
 		{
 			$md_manager = $phpbb_extension_manager->get_extension_metadata($name, $template);
-			
+
 			$template->assign_block_vars('disabled', array(
 				'EXT_NAME'		=> $md_manager->get_metadata('display-name'),
 
@@ -193,7 +210,7 @@ class acp_extensions
 
 	/**
 	 * Lists all the available extensions and dumps to the template
-	 * 
+	 *
 	 * @param  $phpbb_extension_manager     An instance of the extension manager
 	 * @param  $template 					An instance of the template engine
 	 * @return null
diff --git a/phpBB/includes/extension/manager.php b/phpBB/includes/extension/manager.php
index e7ceef6ad5..9dfc0a067c 100644
--- a/phpBB/includes/extension/manager.php
+++ b/phpBB/includes/extension/manager.php
@@ -23,6 +23,7 @@ if (!defined('IN_PHPBB'))
 class phpbb_extension_manager
 {
 	protected $db;
+	protected $config;
 	protected $cache;
 	protected $php_ext;
 	protected $extensions;
@@ -34,16 +35,18 @@ class phpbb_extension_manager
 	* Creates a manager and loads information from database
 	*
 	* @param dbal $db A database connection
+	* @param phpbb_config $config phpbb_config
 	* @param string $extension_table The name of the table holding extensions
 	* @param string $phpbb_root_path Path to the phpbb includes directory.
 	* @param string $php_ext php file extension
 	* @param phpbb_cache_driver_interface $cache A cache instance or null
 	* @param string $cache_name The name of the cache variable, defaults to _ext
 	*/
-	public function __construct(dbal $db, $extension_table, $phpbb_root_path, $php_ext = '.php', phpbb_cache_driver_interface $cache = null, $cache_name = '_ext')
+	public function __construct(dbal $db, phpbb_config $config, $extension_table, $phpbb_root_path, $php_ext = '.php', phpbb_cache_driver_interface $cache = null, $cache_name = '_ext')
 	{
 		$this->phpbb_root_path = $phpbb_root_path;
 		$this->db = $db;
+		$this->config = $config;
 		$this->cache = $cache;
 		$this->php_ext = $php_ext;
 		$this->extension_table = $extension_table;
@@ -130,7 +133,7 @@ class phpbb_extension_manager
 	*/
 	public function get_extension_metadata($name, phpbb_template $template)
 	{
-		return new phpbb_extension_metadata_manager($name, $this->db, $this, $this->phpbb_root_path, $this->phpEx, $template);
+		return new phpbb_extension_metadata_manager($name, $this->db, $this, $this->phpbb_root_path, $this->php_ext, $template, $this->config);
 	}
 
 	/**
diff --git a/phpBB/includes/extension/metadata_manager.php b/phpBB/includes/extension/metadata_manager.php
index 6af02e47b7..0e0b609a68 100644
--- a/phpBB/includes/extension/metadata_manager.php
+++ b/phpBB/includes/extension/metadata_manager.php
@@ -32,17 +32,70 @@ class phpbb_extension_metadata_manager
 	protected $metadata_file;
 
 	/**
+	* Array of validation regular expressions, see __call()
+	*
+	* @var mixed
+	*/
+	protected $validation = array(
+		'name'					=> '#^[a-zA-Z0-9_\x7f-\xff]{2,}/[a-zA-Z0-9_\x7f-\xff]{2,}$#',
+		'type'					=> '#^phpbb3-extension$#',
+		'description'			=> '#.*#',
+		'version'				=> '#.+#',
+		'licence'				=> '#.+#',
+		'extra'					=> array(
+			'display-name'			=> '#.*#',
+		),
+	);
+
+	/**
+	* Magic method to catch validation calls
+	*
+	* @param string $name
+	* @param mixed $arguments
+	* @return int
+	*/
+	public function __call($name, $arguments)
+    {
+    	// Validation Magic methods
+        if (strpos($name, 'validate_') === 0)
+        {
+        	// Remove validate_
+        	$name = substr($name, 9);
+
+        	// Replace underscores with dashes (underscores are not used)
+        	$name = str_replace('_', '-', $name);
+
+        	if (strpos($name, 'extra-') === 0)
+        	{
+        		// Remove extra_
+        		$name = substr($name, 6);
+
+        		if (isset($this->validation['extra'][$name]))
+        		{
+        			// Extra means it's optional, so return true if it does not exist
+        			return (isset($this->metadata['extra'][$name])) ? preg_match($this->validation['extra'][$name], $this->metadata['extra'][$name]) : true;
+				}
+			}
+			else if (isset($this->validation[$name]))
+	        {
+        		return preg_match($this->validation[$name], $this->metadata[$name]);
+			}
+		}
+    }
+
+	/**
 	* Creates the metadata manager
-	* 
+	*
 	* @param dbal $db A database connection
 	* @param string $extension_manager An instance of the phpbb extension manager
 	* @param string $phpbb_root_path Path to the phpbb includes directory.
 	* @param string $phpEx php file extension
 	*/
-	public function __construct($ext_name, dbal $db, phpbb_extension_manager $extension_manager, $phpbb_root_path, $phpEx = '.php', phpbb_template $template)
+	public function __construct($ext_name, dbal $db, phpbb_extension_manager $extension_manager, $phpbb_root_path, $phpEx = '.php', phpbb_template $template, phpbb_config $config)
 	{
 		$this->phpbb_root_path = $phpbb_root_path;
 		$this->db = $db;
+		$this->config = $config;
 		$this->phpEx = $phpEx;
 		$this->template = $template;
 		$this->extension_manager = $extension_manager;
@@ -53,12 +106,11 @@ class phpbb_extension_metadata_manager
 
 	/**
 	 * Processes and gets the metadata requested
-	 * 
-	 * @param  string $element         All for all metadata that it has and is valid, otherwise specify which section you want by its shorthand term.
-	 * @param  boolean $template_output   True if you want the requested metadata assigned to template vars (only works on the 'all" case
-	 * @return array                   Contains all of the requested metadata
+	 *
+	 * @param  string $element			All for all metadata that it has and is valid, otherwise specify which section you want by its shorthand term.
+	 * @return bool|array				Contains all of the requested metadata or bool False if not valid
 	 */
-	public function get_metadata($element = 'all', $template_output = false)
+	public function get_metadata($element = 'all')
 	{
 		// TODO: Check ext_name exists and is an extension that exists
 		if (!$this->set_metadata_file())
@@ -66,34 +118,37 @@ class phpbb_extension_metadata_manager
 			return false;
 		}
 
+		// Fetch the metadata
 		if (!$this->fetch_metadata())
 		{
 			return false;
 		}
 
-		switch ($element) 
+		// Clean the metadata
+		if (!$this->clean_metadata_array())
+		{
+			return false;
+		}
+
+		switch ($element)
 		{
 			case 'all':
 			default:
-				if (!$this->clean_metadata_array())
+				// Validate the metadata
+				if (!$this->validate_metadata_array())
 				{
 					return false;
 				}
 
-				if ($template_output) 
-				{
-					$this->output_template_data();
-				}
-
 				return $this->metadata;
 			break;
-			
+
 			case 'name':
 				return ($this->validate_name()) ? $this->metadata['name'] : false;
 			break;
-			
+
 			case 'display-name':
-				if ($this->validate_extra_display_name())
+				if (isset($this->metadata['extra']['display-name']) && $this->validate_extra_display_name())
 				{
 					return $this->metadata['extra']['display-name'];
 				}
@@ -108,7 +163,7 @@ class phpbb_extension_metadata_manager
 
 	/**
 	 * Sets the filepath of the metadata file
-	 * 
+	 *
 	 * @return boolean  Set to true if it exists
 	 */
 	private function set_metadata_file()
@@ -129,122 +184,156 @@ class phpbb_extension_metadata_manager
 	}
 
 	/**
-	 * This array handles the validation and cleaning of the array
-	 * 
-	 * @return array Contains the cleaned and validated metadata array
+	 * Gets the contents of the composer.json file
+	 *
+	 * @return bool True of false (if loading succeeded or failed)
 	 */
-	private function clean_metadata_array()
-	{		
-		if (!$this->validate_name() || !$this->validate_type() || !$this->validate_licence() || !$this->validate_description() || !$this->validate_version() || !$this->validate_require_phpbb() || !$this->validate_extra_display_name())
+	private function fetch_metadata()
+	{
+		if (!file_exists($this->metadata_file))
 		{
 			return false;
 		}
-		
-		$this->check_for_optional(true);
+		else
+		{
+			if (!($file_contents = file_get_contents($this->metadata_file)))
+			{
+				return false;
+			}
 
-//		TODO: Remove all parts of the array we don't want or shouldn't be there due to nub mod authors
-//		$this->metadata = $metadata_finished;
+			if (($metadata = json_decode($file_contents, true)) === NULL)
+			{
+				return false;
+			}
 
-		return $this->metadata;
-	}
+			$this->metadata = $metadata;
 
-	/**
-	 * Validates the contents of the name field
-	 * 
-	 * @return boolean True when passes validation
-	 */
-	private function validate_name()
-	{
-		return preg_match('#^[a-zA-Z0-9_\x7f-\xff]{2,}/[a-zA-Z0-9_\x7f-\xff]{2,}$#', $this->metadata['name']);
+			return true;
+		}
 	}
 
 	/**
-	 * Validates the contents of the type field
-	 * 
-	 * @return boolean True when passes validation
+	 * This array handles the validation and cleaning of the array
+	 *
+	 * @return array Contains the cleaned and validated metadata array
 	 */
-	private function validate_type()
+	private function clean_metadata_array()
 	{
-		return $this->metadata['type'] == 'phpbb3-extension';
-	}
+//		TODO: Remove all parts of the array we don't want or shouldn't be there due to nub mod authors
+//		$this->metadata = $metadata_finished;
 
-	/**
-	 * Validates the contents of the description field
-	 * 
-	 * @return boolean True when passes validation
-	 */
-	private function validate_description()
-	{
-		return true;//preg_match('#^{10,}$#', $this->metadata['description']);
+		return $this->metadata;
 	}
 
 	/**
-	 * Validates the contents of the version field
-	 * 
-	 * @return boolean True when passes validation
+	 * This array handles the validation of strings
+	 *
+	 * @return bool True if validation succeeded, False if failed
 	 */
-	private function validate_version()
+	public function validate_metadata_array()
 	{
-		return preg_match('#^[0-9]{1,2}\.[0-9]{1,2}\.[0-9]{1,2}#', $this->metadata['version']);
+		$validate = array(
+			'name',
+			'type',
+			'licence',
+			'description',
+			'version',
+			'extra_display-name',
+		);
+
+		foreach ($validate as $type)
+		{
+			$type = 'validate_' . $type;
+
+			if (!$this->$type())
+			{
+				return false;
+			}
+		}
+
+		return true;
 	}
 
 	/**
-	 * Validates the contents of the license field
-	 * 
-	 * @return boolean True when passes validation
+	 * This array handles the verification that this extension can be enabled on this board
+	 *
+	 * @return bool True if validation succeeded, False if failed
 	 */
-	private function validate_licence()
+	public function validate_enable()
 	{
-		// Nothing to validate except existence
-		return isset($this->metadata['licence']);
+		$validate = array(
+			'require_phpbb',
+			'require_php',
+		);
+
+		foreach ($validate as $type)
+		{
+			$type = 'validate_' . $type;
+
+			if (!$this->$type())
+			{
+				return false;
+			}
+		}
+
+		return true;
 	}
 
+
 	/**
 	 * Validates the contents of the phpbb requirement field
-	 * 
+	 *
 	 * @return boolean True when passes validation
 	 */
 	private function validate_require_phpbb()
 	{
-		return (preg_match('#^[0-9]{1,2}\.[0-9]{1,2}\.[0-9]{1,2}$#', $this->metadata['require']['phpbb']) && version_compare($this->metadata['require']['phpbb'], '3.1.0', '>='));
+		if (!isset($this->metadata['require']['phpbb']))
+		{
+			return true;
+		}
+
+		return $this->_validate_version($this->metadata['require']['phpbb'], $this->config['version']);
 	}
 
 	/**
-	 * Validates the contents of the display name field
-	 * 
+	 * Validates the contents of the php requirement field
+	 *
 	 * @return boolean True when passes validation
 	 */
-	private function validate_extra_display_name()
+	private function validate_require_php()
 	{
-		return true;//preg_match('#^[a-zA-Z0-9_]{2,0}$#', $this->metadata['name']);
+		if (!isset($this->metadata['require']['php']))
+		{
+			return true;
+		}
+
+		return $this->_validate_version($this->metadata['require']['php'], phpversion());
 	}
 
 	/**
-	 * Checks which optional fields exist
-	 * 
-	 * @return boolean           False if any that exist fail validation, otherwise true.
-	 */
-	public function check_for_optional()
+	* Version validation helper
+	*
+	* @param string $string The string for comparing to a version
+	* @param string $current_version The version to compare to
+	* @return bool True/False if meets version requirements
+	*/
+	private function _validate_version($string, $current_version)
 	{
-		if ((isset($this->metadata['require']['php']) && !$this->validate_require_php()) || (isset($this->metadata['time']) && !$this->validate_time()) || (isset($this->metadata['validate_homepage']) && !$this->validate_homepage()))
+		// Allow them to specify their own comparison operator (ex: <3.1.2, >=3.1.0)
+		$comparison_matches = false;
+		preg_match('#[=<>]+#', $string, $comparison_matches);
+
+		if (!empty($comparison_matches))
 		{
-			return false;
+			return version_compare($current_version, str_replace(array($comparison_matches[0], ' '), '', $string), $comparison_matches[0]);
 		}
-	}
 
-	/**
-	 * Validates the contents of the php requirement field
-	 * 
-	 * @return boolean True when passes validation
-	 */
-	private function validate_require_php()
-	{
-		return (preg_match('#^[0-9]{1,2}\.[0-9]{1,2}\.[0-9]{1,2}$#', $this->metadata['require']['php']) && version_compare($this->metadata['require']['php'], phpversion(), '>='));
+		return version_compare($current_version, $string, '>=');
 	}
 
 	/**
 	 * Validates the contents of the time field
-	 * 
+	 *
 	 * @return boolean True when passes validation
 	 */
 	private function validate_time()
@@ -255,7 +344,7 @@ class phpbb_extension_metadata_manager
 
 	/**
 	 * Validates the contents of the homepage field
-	 * 
+	 *
 	 * @return boolean True when passes validation
 	 */
 	private function validate_homepage()
@@ -265,7 +354,7 @@ class phpbb_extension_metadata_manager
 
 	/**
 	 * Validates the contents of the authors field
-	 * 
+	 *
 	 * @return boolean True when passes validation
 	 */
 	private function validate_authors()
@@ -292,37 +381,8 @@ class phpbb_extension_metadata_manager
 	}
 
 	/**
-	 * Gets the contents of the composer.json file
-	 *
-	 * @return bool True of false (if loading succeeded or failed)
-	 */
-	private function fetch_metadata()
-	{
-		if (!file_exists($this->metadata_file))
-		{
-			return false;
-		}
-		else
-		{
-			if (!($file_contents = file_get_contents($this->metadata_file)))
-			{
-				return false;
-			}
-			
-			if (($metadata = json_decode($file_contents, true)) === NULL)
-			{
-				return false;
-			}
-			
-			$this->metadata = $metadata;
-			
-			return true;
-		}
-	}
-
-	/**
 	 * Outputs the metadata into the template
-	 * 
+	 *
 	 * @return null
 	 */
 	public function output_template_data()
@@ -350,7 +410,7 @@ class phpbb_extension_metadata_manager
 				'AUTHOR_ROLE'		=> (isset($author['role'])) ? htmlspecialchars($author['role']) : '',
 			));
 		}
-		
+
 		return;
 	}
 }
diff --git a/phpBB/install/database_update.php b/phpBB/install/database_update.php
index 502b3bb1a4..0b470ced26 100644
--- a/phpBB/install/database_update.php
+++ b/phpBB/install/database_update.php
@@ -683,12 +683,12 @@ function _write_result($no_updates, $errored, $error_ary)
 
 function _add_modules($modules_to_install)
 {
-	global $phpbb_root_path, $phpEx, $db, $phpbb_extension_manager;
+	global $phpbb_root_path, $phpEx, $db, $phpbb_extension_manager, $config;
 
 	// modules require an extension manager
 	if (empty($phpbb_extension_manager))
 	{
-		$phpbb_extension_manager = new phpbb_extension_manager($db, EXT_TABLE, $phpbb_root_path, ".$phpEx");
+		$phpbb_extension_manager = new phpbb_extension_manager($db, $config, EXT_TABLE, $phpbb_root_path, ".$phpEx");
 	}
 
 	include_once($phpbb_root_path . 'includes/acp/acp_modules.' . $phpEx);
diff --git a/phpBB/install/install_install.php b/phpBB/install/install_install.php
index 23593ee51f..9162d5ab60 100644
--- a/phpBB/install/install_install.php
+++ b/phpBB/install/install_install.php
@@ -250,7 +250,7 @@ class install_install extends module
 			'S_EXPLAIN'		=> true,
 			'S_LEGEND'		=> false,
 		));
-		
+
 		// Check for php json support
 		if (@extension_loaded('json'))
 		{
@@ -1481,12 +1481,12 @@ class install_install extends module
 	*/
 	function add_modules($mode, $sub)
 	{
-		global $db, $lang, $phpbb_root_path, $phpEx, $phpbb_extension_manager;
+		global $db, $lang, $phpbb_root_path, $phpEx, $phpbb_extension_manager, $config;
 
 		// modules require an extension manager
 		if (empty($phpbb_extension_manager))
 		{
-			$phpbb_extension_manager = new phpbb_extension_manager($db, EXT_TABLE, $phpbb_root_path, ".$phpEx");
+			$phpbb_extension_manager = new phpbb_extension_manager($db, $config, EXT_TABLE, $phpbb_root_path, ".$phpEx");
 		}
 
 		include_once($phpbb_root_path . 'includes/acp/acp_modules.' . $phpEx);
diff --git a/phpBB/language/en/acp/extensions.php b/phpBB/language/en/acp/extensions.php
index 66f3665757..547d14d2b9 100644
--- a/phpBB/language/en/acp/extensions.php
+++ b/phpBB/language/en/acp/extensions.php
@@ -1,11 +1,11 @@
 <?php
-/** 
+/**
 *
 * acp_extensions [English]
 *
 * @package language
-* @copyright (c) 2012 phpBB Group 
-* @license http://opensource.org/licenses/gpl-2.0.php GNU Public License 
+* @copyright (c) 2012 phpBB Group
+* @license http://opensource.org/licenses/gpl-2.0.php GNU Public License
 *
 */
 /**
@@ -23,7 +23,7 @@ if (empty($lang) || !is_array($lang))
 	$lang = array();
 }
 
-// DEVELOPERS PLEASE NOTE 
+// DEVELOPERS PLEASE NOTE
 //
 // Placeholders can now contain order information, e.g. instead of
 // 'Page %s of %s' you can (and should) write 'Page %1$s of %2$s', this allows
@@ -35,11 +35,12 @@ if (empty($lang) || !is_array($lang))
 
 
 $lang = array_merge($lang, array(
-	'EXTENSION'				=> 'Extension',
-	'EXTENSIONS'			=> 'Extensions',
-	'EXTENSIONS_ADMIN'		=> 'Extensions Manager',
-	'EXTENSIONS_EXPLAIN'	=> 'The Extensions Manager is a tool in your phpBB Board which allows you to manage all of your extensions statuses and view information about them.',
-	'EXTENSION_INVALID'		=> 'The selected extension is not valid.',
+	'EXTENSION'					=> 'Extension',
+	'EXTENSIONS'				=> 'Extensions',
+	'EXTENSIONS_ADMIN'			=> 'Extensions Manager',
+	'EXTENSIONS_EXPLAIN'		=> 'The Extensions Manager is a tool in your phpBB Board which allows you to manage all of your extensions statuses and view information about them.',
+	'EXTENSION_INVALID'			=> 'The selected extension is not valid.',
+	'EXTENSION_NOT_AVAILABLE'	=> 'The selected extension is not available for this board, please verify your phpBB and PHP versions are allowed (see the details page).',
 
 	'DETAILS'				=> 'Details',
 
@@ -63,27 +64,27 @@ $lang = array_merge($lang, array(
 	'DISABLE_IN_PROGRESS'	=> 'The extension is currently being disabled, please do not leave this page or refresh until it is completed.',
 	'ENABLE_IN_PROGRESS'	=> 'The extension is currently being installed, please do not leave this page or refresh until it is completed.',
 	'PURGE_IN_PROGRESS'		=> 'The extension is currently being purged, please do not leave this page or refresh until it is completed.',
-	'ENABLE_SUCCESS'			=> 'The extension was enabled successfully',
+	'ENABLE_SUCCESS'		=> 'The extension was enabled successfully',
 	'DISABLE_SUCCESS'		=> 'The extension was disabled successfully',
 	'PURGE_SUCCESS'			=> 'The extension was purged successfully',
-	'DELETE_SUCCESS'			=> 'The extension was deleted successfully',
+	'DELETE_SUCCESS'		=> 'The extension was deleted successfully',
 
 	'ENABLE_FAIL'			=> 'The extension could not be enabled',
 	'DISABLE_FAIL'			=> 'The extension could not be disabled',
 	'PURGE_FAIL'			=> 'The extension could not be purged',
 	'DELETE_FAIL'			=> 'The extension could not be deleted',
 
-	'EXTENSION_NAME'			=> 'Extension Name',
-	'EXTENSION_ACTIONS'			=> 'Actions',
-	'EXTENSION_OPTIONS'			=> 'Options',
+	'EXTENSION_NAME'		=> 'Extension Name',
+	'EXTENSION_ACTIONS'		=> 'Actions',
+	'EXTENSION_OPTIONS'		=> 'Options',
 
-	'ENABLE_CONFIRM'			=> 'Are you sure that you wish to enable this extension?',
-	'DISABLE_CONFIRM'			=> 'Are you sure that you wish to disable this extension?',
-	'PURGE_CONFIRM'				=> 'Are you sure that you wish to purge this extension&#39;s data? This cannot be undone.',
-	'DELETE_CONFIRM'			=> 'Are you sure that you wish to data this extension&#39;s files and clear its data? This cannot be undone.',
+	'ENABLE_CONFIRM'		=> 'Are you sure that you wish to enable this extension?',
+	'DISABLE_CONFIRM'		=> 'Are you sure that you wish to disable this extension?',
+	'PURGE_CONFIRM'			=> 'Are you sure that you wish to purge this extension&#39;s data? This cannot be undone.',
+	'DELETE_CONFIRM'		=> 'Are you sure that you wish to data this extension&#39;s files and clear its data? This cannot be undone.',
 
-	'WARNING'			=> 'Warning',
-	'RETURN'			=> 'Return',
+	'WARNING'				=> 'Warning',
+	'RETURN'				=> 'Return',
 
 	'EXT_DETAILS'			=> 'Extension Details',
 	'DISPLAY_NAME'			=> 'Display Name',