1 files changed, 94 insertions, 71 deletions
diff --git a/phpBB/includes/acp/acp_captcha.php b/phpBB/includes/acp/acp_captcha.php
index e61b8c2170..57de03c4c9 100644
--- a/phpBB/includes/acp/acp_captcha.php
+++ b/phpBB/includes/acp/acp_captcha.php
@@ -10,6 +10,8 @@
 /**
 * @ignore
 */
+
+
 if (!defined('IN_PHPBB'))
 {
 	exit;
@@ -28,94 +30,115 @@ class acp_captcha
 
 		$user->add_lang('acp/board');
 
+		include(PHPBB_ROOT_PATH . 'includes/captcha/captcha_factory.' . PHP_EXT);
 
-		$captcha_vars = array(
-			'captcha_gd_x_grid'				=> 'CAPTCHA_GD_X_GRID',
-			'captcha_gd_y_grid'				=> 'CAPTCHA_GD_Y_GRID',
-			'captcha_gd_foreground_noise'	=> 'CAPTCHA_GD_FOREGROUND_NOISE',
-			'captcha_gd'					=> 'CAPTCHA_GD_PREVIEWED'
-		);
-
-		if (isset($_GET['demo']))
+		$selected = request_var('select_captcha', $config['captcha_plugin']);
+		$configure = request_var('configure', false);
+		
+		// Oh, they are just here for the view
+		if (isset($_GET['captcha_demo']))
 		{
-			$captcha_vars = array_keys($captcha_vars);
-			foreach ($captcha_vars as $captcha_var)
-			{
-				$config[$captcha_var] = (isset($_REQUEST[$captcha_var])) ? request_var($captcha_var, 0) : $config[$captcha_var];
-			}
-
-			if ($config['captcha_gd'])
-			{
-				include(PHPBB_ROOT_PATH . 'includes/captcha/captcha_gd.' . PHP_EXT);
-			}
-			else
-			{
-				include(PHPBB_ROOT_PATH . 'includes/captcha/captcha_non_gd.' . PHP_EXT);
-			}
-			captcha::execute(gen_rand_string(mt_rand(5, 8)), time());
-			exit;
+			$this->deliver_demo($selected);
+		}
+		
+		// Delegate
+		if ($configure)
+		{
+			$config_captcha = phpbb_captcha_factory::get_instance($selected);
+			$config_captcha->acp_page($id, $this);
 		}
+		else
+		{
+			$captchas = phpbb_captcha_factory::get_captcha_types();
 
-		$config_vars = array(
-			'enable_confirm'		=> 'REG_ENABLE',
-			'enable_post_confirm'	=> 'POST_ENABLE',
-			'captcha_gd'			=> 'CAPTCHA_GD',
-		);
+			$config_vars = array(
+				'enable_confirm'		=> 'REG_ENABLE',
+				'enable_post_confirm'	=> 'POST_ENABLE',
+			);
 
-		$this->tpl_name = 'acp_captcha';
-		$this->page_title = 'ACP_VC_SETTINGS';
-		$form_key = 'acp_captcha';
-		add_form_key($form_key);
+			$this->tpl_name = 'acp_captcha';
+			$this->page_title = 'ACP_VC_SETTINGS';
+			$form_key = 'acp_captcha';
+			add_form_key($form_key);
 
-		$submit = request_var('submit', '');
+			$submit = request_var('main_submit', false);
 
-		if ($submit && check_form_key($form_key))
-		{
-			$config_vars = array_keys($config_vars);
-			foreach ($config_vars as $config_var)
-			{
-				set_config($config_var, request_var($config_var, ''));
-			}
-			$captcha_vars = array_keys($captcha_vars);
-			foreach ($captcha_vars as $captcha_var)
+			if ($submit && check_form_key($form_key))
 			{
-				$value = request_var($captcha_var, 0);
-				if ($value >= 0)
+				$config_vars = array_keys($config_vars);
+				foreach ($config_vars as $config_var)
 				{
-					set_config($captcha_var, $value);
+					set_config($config_var, request_var($config_var, false));
 				}
+				if ($selected !== $config['captcha_plugin'])
+				{
+					// sanity check
+					if (isset($captchas['available'][$selected]))
+					{
+						$old_captcha = phpbb_captcha_factory::get_instance($config['captcha_plugin']);
+						$old_captcha->uninstall();
+						set_config('captcha_plugin', $selected);
+						$new_captcha = phpbb_captcha_factory::get_instance($config['captcha_plugin']);
+						$old_captcha->install();
+					}
+					else
+					{
+						trigger_error($user->lang['CAPTCHA_UNAVAILABLE'] . adm_back_link($this->u_action));
+					}
+				}
+				trigger_error($user->lang['CONFIG_UPDATED'] . adm_back_link($this->u_action));
 			}
-			trigger_error($user->lang['CONFIG_UPDATED'] . adm_back_link($this->u_action));
-		}
-		else if ($submit)
-		{
-				trigger_error($user->lang['FORM_INVALID'] . adm_back_link($this->u_action));
-		}
-		else
-		{
-
-			$preview_image_src = append_sid(append_sid(PHPBB_ADMIN_PATH . 'index.' . PHP_EXT, "i=$id&amp;demo=demo"));
-			if (@extension_loaded('gd'))
-			{
-				$template->assign_var('GD', true);
-			}
-			foreach ($config_vars as $config_var => $template_var)
+			else if ($submit)
 			{
-				$template->assign_var($template_var, (isset($_REQUEST[$config_var])) ? request_var($config_var, '') : $config[$config_var]) ;
+				trigger_error($user->lang['FORM_INVALID'] . adm_back_link());
 			}
-			foreach ($captcha_vars as $captcha_var => $template_var)
+			else
 			{
-				$var = (isset($_REQUEST[$captcha_var])) ? request_var($captcha_var, 0) : $config[$captcha_var];
-				$template->assign_var($template_var, $var);
-				$preview_image_src .= "&amp;$captcha_var=" . $var;
-			}
-			$template->assign_vars(array(
-				'CAPTCHA_PREVIEW'	=> $preview_image_src,
-				'PREVIEW'			=> isset($_POST['preview']),
-			));
+				$captcha_select = '';
+				foreach ($captchas['available'] as $value => $title)
+				{
+					$current = ($selected !== false && $value == $selected) ? ' selected="selected"' : '';
+					$captcha_select .= '<option value="' . $value . '"' . $current . '>' . $user->lang[$title] . '</option>';
+				}
+				foreach ($captchas['unavailable'] as $value => $title)
+				{
+					$captcha_select .= '<option value="' . $value . '"' . $current . ' class="disabled-option" >' . $user->lang[$title] . '</option>';
+				}
+
+				$demo_captcha = phpbb_captcha_factory::get_instance($selected);
+				
+				foreach ($config_vars as $config_var => $template_var)
+				{
+					$template->assign_var($template_var, (isset($_REQUEST[$config_var])) ? request_var($config_var, '') : $config[$config_var]) ;
+				}
 
+				$template->assign_vars(array(
+					'CAPTCHA_PREVIEW'	=> $demo_captcha->get_demo_template($id),
+					'CAPTCHA_SELECT'	=> $captcha_select,
+				));
+			}
+			
 		}
 	}
+	
+	
+	/**
+	* Entry point for delivering image CAPTCHAs in the ACP.
+	*/
+	function deliver_demo($selected)
+	{
+		global $db, $user, $config;
+		
+		$captcha = phpbb_captcha_factory::get_instance($selected);
+		$captcha->init(CONFIRM_REG);
+		$captcha->execute_demo();
+		garbage_collection();
+		exit_handler();
+	}
+	
+	
+	
+	
 }
 
 ?>
 \ No newline at end of file