7 files changed, 205 insertions, 188 deletions
diff --git a/phpBB/includes/captcha/captcha_factory.php b/phpBB/includes/captcha/captcha_factory.php
index fbe615a043..b5fa69990d 100644
--- a/phpBB/includes/captcha/captcha_factory.php
+++ b/phpBB/includes/captcha/captcha_factory.php
@@ -16,8 +16,11 @@ if (!defined('IN_PHPBB'))
 	exit;
 }
 
-
-/** A small class until we get the autoloader done */
+/**
+* A small class for 3.0.x (no autoloader in 3.0.x)
+*
+* @package VC
+*/
 class phpbb_captcha_factory
 {
 	/**
@@ -26,7 +29,7 @@ class phpbb_captcha_factory
 	function get_instance($name)
 	{
 		global $phpbb_root_path, $phpEx;
-		
+
 		$name = basename($name);
 		if (!class_exists($name))
 		{
@@ -34,7 +37,7 @@ class phpbb_captcha_factory
 		}
 		return call_user_func(array($name, 'get_instance'));
 	}
-	
+
 	/**
 	* Call the garbage collector
 	*/
@@ -49,18 +52,19 @@ class phpbb_captcha_factory
 		}
 		call_user_func(array($name, 'garbage_collect'), 0);
 	}
-	
+
 	/**
 	* return a list of all discovered CAPTCHA plugins
 	*/
 	function get_captcha_types()
 	{
 		global $phpbb_root_path, $phpEx;
-	
-		$captchas = array();
-		$captchas['available'] = array();
-		$captchas['unavailable'] = array();
-	
+
+		$captchas = array(
+			'available'		=> array(),
+			'unavailable'	=> array(),
+		);
+
 		$dp = @opendir($phpbb_root_path . 'includes/captcha/plugins');
 
 		if ($dp)
@@ -74,6 +78,7 @@ class phpbb_captcha_factory
 					{
 						include($phpbb_root_path . "includes/captcha/plugins/$file");
 					}
+
 					if (call_user_func(array($name, 'is_available')))
 					{
 						$captchas['available'][$name] = call_user_func(array($name, 'get_name'));
diff --git a/phpBB/includes/captcha/captcha_gd_wave.php b/phpBB/includes/captcha/captcha_gd_wave.php
index d911d574f7..f706c98d43 100644
--- a/phpBB/includes/captcha/captcha_gd_wave.php
+++ b/phpBB/includes/captcha/captcha_gd_wave.php
@@ -1,16 +1,18 @@
 <?php
-/** 
+/**
 *
 * @package VC
 * @version $Id$
-* @copyright (c) 2006 phpBB Group 
-* @license http://opensource.org/licenses/gpl-license.php GNU Public License 
+* @copyright (c) 2006 phpBB Group
+* @license http://opensource.org/licenses/gpl-license.php GNU Public License
 *
 */
 
-
-/** 
-* Wave3D CAPTCHA by Robert Hetzler
+/**
+* Wave3D CAPTCHA
+*
+* @author Robert Hetzler
+* @package VC
 */
 class captcha
 {
@@ -20,10 +22,10 @@ class captcha
 	function execute($code, $seed)
 	{
 		global $starttime;
-		
+
 		// seed the random generator
 		mt_srand($seed);
-	
+
 		// set height and width
 		$img_x = $this->width;
 		$img_y = $this->height;
@@ -32,7 +34,7 @@ class captcha
 		$img	= imagecreatetruecolor($img_x, $img_y);
 		$x_grid = mt_rand(6, 10);
 		$y_grid = mt_rand(6, 10);
- 
+
 		// Ok, so lets cut to the chase. We could accurately represent this in 3d and
 		// do all the appropriate linear transforms. my questions is... why bother?
 		// The computational overhead is unnecessary when you consider the simple fact:
@@ -47,28 +49,29 @@ class captcha
 		$plane_x	= 100;
 		$plane_y	= 30;
 
-		$subdivision_factor	= 3;
+		$subdivision_factor = 3;
+
 		// $box is the 4 points in img_space that correspond to the corners of the plane in 3-space
 		$box = array(
 			'upper_left'	=> array(
 				'x' => mt_rand(5, 15),
 				'y' => mt_rand(10, 15)
-			), 
+			),
 			'upper_right'	=> array(
 				'x' => mt_rand($img_x - 35, $img_x - 19),
 				'y' => mt_rand(10, 17)
-			), 
+			),
 			'lower_left'	=> array(
 				'x' => mt_rand($img_x - 5, $img_x - 45),
 				'y' => mt_rand($img_y - 0, $img_y - 15)
-			), 
+			),
 		);
+
 		$box['lower_right'] = array(
-				'x' => $box['lower_left']['x'] + $box['upper_left']['x'] - $box['upper_right']['x'],
-				'y' => $box['lower_left']['y'] + $box['upper_left']['y'] - $box['upper_right']['y'],
+			'x' => $box['lower_left']['x'] + $box['upper_left']['x'] - $box['upper_right']['x'],
+			'y' => $box['lower_left']['y'] + $box['upper_left']['y'] - $box['upper_right']['y'],
 		);
 
-
 		// TODO
 		$background = imagecolorallocate($img, mt_rand(155, 255), mt_rand(155, 255), mt_rand(155, 255));
 		imagefill($img, 0, 0, $background);
@@ -83,7 +86,7 @@ class captcha
 		}
 
 		$fontcolors[0] = imagecolorallocate($img, mt_rand(0, 120), mt_rand(0, 120), mt_rand(0, 120));
-		
+
  		$colors = array();
 
 		$minr = mt_rand(20, 30);
@@ -159,7 +162,7 @@ class captcha
 			$cur_height		= $this->wave_height($x, 0, $subdivision_factor);
 			$offset			= $cur_height - $prev_height;
 			$img_pos_cur	= array($img_pos_prev[0] + $dxx, $img_pos_prev[1] + $dxy + $offset);
-									
+
 			$img_buffer[0][$x]	= $img_pos_cur;
 			$img_pos_prev		= $img_pos_cur;
 			$prev_height		= $cur_height;
@@ -170,7 +173,7 @@ class captcha
 			// swap buffers
 			$buffer_cur		= $y & 1;
 			$buffer_prev	= 1 - $buffer_cur;
-			
+
 			$prev_height	= $this->wave_height(0, $y, $subdivision_factor);
 			$offset			= $prev_height - $this->wave_height(0, $y - 1, $subdivision_factor);
 			$img_pos_cur	= array($img_buffer[$buffer_prev][0][0] + $dyx, min($img_buffer[$buffer_prev][0][1] + $dyy + $offset, $img_y - 1));
@@ -179,7 +182,7 @@ class captcha
 			$img_pos_prev	= $img_pos_cur;
 
 			$img_buffer[$buffer_cur][0]	= $img_pos_cur;
-			
+
 			for ($x = 1; $x <= $full_x; ++$x)
 			{
 				$cur_height		= $this->wave_height($x, $y, $subdivision_factor) + $this->grid_height($x, $y, 1, $x_grid, $y_grid);
@@ -496,7 +499,7 @@ class captcha
 					array(0,0,0,0,0,0,0,0,0),
 					array(0,0,0,0,0,0,0,0,0),
 					array(0,0,0,0,0,0,0,0,0),
-				),		
+				),
 				'O' => array(
 					array(0,0,0,1,1,1,0,0,0),
 					array(0,0,1,0,0,0,1,0,0),
diff --git a/phpBB/includes/captcha/plugins/captcha_abstract.php b/phpBB/includes/captcha/plugins/captcha_abstract.php
index f88d82b2a0..0666a3ca67 100644
--- a/phpBB/includes/captcha/plugins/captcha_abstract.php
+++ b/phpBB/includes/captcha/plugins/captcha_abstract.php
@@ -18,7 +18,9 @@ if (!defined('IN_PHPBB'))
 
 
 /**
-* This class holds the code shared by the two default 3.0 CAPTCHAs.
+* This class holds the code shared by the two default 3.0.x CAPTCHAs.
+*
+* @package VC
 */
 class phpbb_default_captcha
 {
@@ -29,18 +31,17 @@ class phpbb_default_captcha
 	var $type;
 	var $solved = false;
 
-
 	function init($type)
 	{
 		global $config, $db, $user;
-		
+
 		// read input
 		$this->confirm_id = request_var('confirm_id', '');
 		$this->confirm_code = request_var('confirm_code', '');
 		$refresh = request_var('refresh_vc', false) && $config['confirm_refresh'];
-		
+
 		$this->type = (int) $type;
-		
+
 		if (!strlen($this->confirm_id))
 		{
 			// we have no confirm ID, better get ready to display something
@@ -50,24 +51,22 @@ class phpbb_default_captcha
 		{
 			$this->regenerate_code();
 		}
-
 	}
-	
+
 	function execute_demo()
 	{
 		global $user;
-	
-		$this->code = gen_rand_string(mt_rand(5, 8));
+
+		$this->code = gen_rand_string(mt_rand(CAPTCHA_MIN_CHARS, CAPTCHA_MAX_CHARS));
 		$this->seed = hexdec(substr(unique_id(), 4, 10));
-				
+
 		// compute $seed % 0x7fffffff
 		$this->seed -= 0x7fffffff * floor($this->seed / 0x7fffffff);
-		
+
 		$captcha = new captcha();
 		$captcha->execute($this->code, $this->seed);
 	}
-	
-	
+
 	function execute()
 	{
 		if (empty($this->code))
@@ -81,46 +80,46 @@ class phpbb_default_captcha
 		$captcha = new captcha();
 		$captcha->execute($this->code, $this->seed);
 	}
-	
-	
+
 	function get_template()
 	{
 		global $config, $user, $template, $phpEx, $phpbb_root_path;
-		
+
 		$template->set_filenames(array(
 			'captcha' => 'captcha_default.html')
 		);
-		
+
 		$template->assign_vars(array(
 			'CONFIRM_IMAGE'				=> append_sid($phpbb_root_path . 'ucp.' . $phpEx . '?mode=confirm&amp;confirm_id=' . $this->confirm_id . '&amp;type=' . $this->type),
-			'CONFIRM_ID'				=> $this->confirm_id,			
+			'CONFIRM_ID'				=> $this->confirm_id,
 			'S_REFRESH'					=> (bool) $config['confirm_refresh'],
 
 		));
-	
+
 		return $template->assign_display('captcha');
 	}
-	
+
 	function get_demo_template($id)
 	{
 		global $config, $user, $template, $phpbb_admin_path, $phpEx;
-		
+
 		$template->set_filenames(array(
 			'captcha_demo' => 'captcha_default_acp_demo.html')
 		);
+
 		// acp_captcha has a delivery function; let's use it
 		$template->assign_vars(array(
 			'CONFIRM_IMAGE'		=> append_sid($phpbb_admin_path . 'index.' . $phpEx . '?captcha_demo=1&amp;mode=visual&amp;i=' . $id . '&amp;select_captcha=' . $this->get_class_name()),
 			'CONFIRM_ID'		=> $this->confirm_id,
 		));
-		
+
 		return $template->assign_display('captcha_demo');
 	}
-		
+
 	function get_hidden_fields()
 	{
 		$hidden_fields = array();
-		
+
 		// this is required for postig.php - otherwise we would forget about the captcha being already solved
 		if ($this->solved)
 		{
@@ -129,16 +128,16 @@ class phpbb_default_captcha
 		$hidden_fields['confirm_id'] = $this->confirm_id;
 		return $hidden_fields;
 	}
-	
+
 	function garbage_collect($type)
 	{
 		global $db, $config;
 
 		$sql = 'SELECT DISTINCT c.session_id
-				FROM ' . CONFIRM_TABLE . ' c
-				LEFT JOIN ' . SESSIONS_TABLE . ' s ON (c.session_id = s.session_id)
-				WHERE s.session_id IS NULL' .
-					((empty($type)) ? '' : ' AND c.confirm_type = ' . (int) $type);
+			FROM ' . CONFIRM_TABLE . ' c
+			LEFT JOIN ' . SESSIONS_TABLE . ' s ON (c.session_id = s.session_id)
+			WHERE s.session_id IS NULL' .
+				((empty($type)) ? '' : ' AND c.confirm_type = ' . (int) $type);
 		$result = $db->sql_query($sql);
 
 		if ($row = $db->sql_fetchrow($result))
@@ -159,21 +158,21 @@ class phpbb_default_captcha
 		}
 		$db->sql_freeresult($result);
 	}
-	
+
 	function uninstall()
 	{
-		self::garbage_collect(0);
+		$this->garbage_collect(0);
 	}
-	
+
 	function install()
 	{
 		return;
 	}
-	
+
 	function validate()
 	{
 		global $config, $db, $user;
-		
+
 		$this->confirm_code = request_var('confirm_code', '');
 		if (!$this->confirm_id)
 		{
@@ -191,10 +190,10 @@ class phpbb_default_captcha
 				$error = $user->lang['CONFIRM_CODE_WRONG'];
 			}
 		}
-	
+
 		if (strlen($error))
 		{
-			// okay, inorect answer. Let's ask a new question
+			// okay, incorrect answer. Let's ask a new question.
 			$this->generate_code();
 			return $error;
 		}
@@ -203,16 +202,15 @@ class phpbb_default_captcha
 			return false;
 		}
 	}
-	
-	
+
 	/**
 	* The old way to generate code, suitable for GD and non-GD. Resets the internal state.
 	*/
 	function generate_code()
 	{
 		global $db, $user;
-		
-		$this->code = gen_rand_string(mt_rand(5, 8));
+
+		$this->code = gen_rand_string(mt_rand(CAPTCHA_MIN_CHARS, CAPTCHA_MAX_CHARS));
 		$this->confirm_id = md5(unique_id($user->ip));
 		$this->seed = hexdec(substr(unique_id(), 4, 10));
 		$this->solved = false;
@@ -228,19 +226,20 @@ class phpbb_default_captcha
 		);
 		$db->sql_query($sql);
 	}
-	
+
 	/**
 	* New Question, if desired.
 	*/
 	function regenerate_code()
 	{
 		global $db, $user;
-		
-		$this->code = gen_rand_string(mt_rand(5, 8));
+
+		$this->code = gen_rand_string(mt_rand(CAPTCHA_MIN_CHARS, CAPTCHA_MAX_CHARS));
 		$this->seed = hexdec(substr(unique_id(), 4, 10));
 		$this->solved = false;
 		// compute $seed % 0x7fffffff
 		$this->seed -= 0x7fffffff * floor($this->seed / 0x7fffffff);
+
 		$sql = 'UPDATE ' . CONFIRM_TABLE . ' SET ' . $db->sql_build_array('UPDATE', array(
 				'code'			=> (string) $this->code,
 				'seed'			=> (int) $this->seed)) . '
@@ -249,35 +248,37 @@ class phpbb_default_captcha
 				session_id = \'' . $db->sql_escape($user->session_id) . '\'';
 		$db->sql_query($sql);
 	}
-	
+
 	/**
-	* Look up everything we need for painting&checking. 
+	* Look up everything we need for painting&checking.
 	*/
 	function load_code()
 	{
 		global $db, $user;
+
 		$sql = 'SELECT code, seed
-				FROM ' . CONFIRM_TABLE . "
-				WHERE confirm_id = '" . $db->sql_escape($this->confirm_id) . "'
-				AND session_id = '" . $db->sql_escape($user->session_id) . "'
-					AND confirm_type = " . $this->type;
+			FROM ' . CONFIRM_TABLE . "
+			WHERE confirm_id = '" . $db->sql_escape($this->confirm_id) . "'
+			AND session_id = '" . $db->sql_escape($user->session_id) . "'
+				AND confirm_type = " . $this->type;
 		$result = $db->sql_query($sql);
 		$row = $db->sql_fetchrow($result);
 		$db->sql_freeresult($result);
+
 		if ($row)
 		{
 			$this->code = $row['code'];
 			$this->seed = $row['seed'];
 			return true;
 		}
+
 		return false;
-		
 	}
-	
+
 	function check_code()
 	{
 		global $db;
-		
+
 		if (empty($this->code))
 		{
 			if (!$this->load_code())
@@ -287,47 +288,45 @@ class phpbb_default_captcha
 		}
 		return (strcasecmp($this->code, $this->confirm_code) === 0);
 	}
-	
+
 	function delete_code()
 	{
 		global $db, $user;
-		
+
 		$sql = 'DELETE FROM ' . CONFIRM_TABLE . "
-				WHERE confirm_id = '" . $db->sql_escape($confirm_id) . "'
-					AND session_id = '" . $db->sql_escape($user->session_id) . "'
-					AND confirm_type = " . $this->type;
+			WHERE confirm_id = '" . $db->sql_escape($confirm_id) . "'
+				AND session_id = '" . $db->sql_escape($user->session_id) . "'
+				AND confirm_type = " . $this->type;
 		$db->sql_query($sql);
 	}
-	
+
 	function get_attempt_count()
 	{
 		global $db, $user;
-		
+
 		$sql = 'SELECT COUNT(session_id) AS attempts
-				FROM ' . CONFIRM_TABLE . "
-				WHERE session_id = '" . $db->sql_escape($user->session_id) . "'
-					AND confirm_type = " . $this->type;
+			FROM ' . CONFIRM_TABLE . "
+			WHERE session_id = '" . $db->sql_escape($user->session_id) . "'
+				AND confirm_type = " . $this->type;
 		$result = $db->sql_query($sql);
 		$attempts = (int) $db->sql_fetchfield('attempts');
 		$db->sql_freeresult($result);
-		
+
 		return $attempts;
 	}
-	
-	
+
 	function reset()
 	{
 		global $db, $user;
-		
+
 		$sql = 'DELETE FROM ' . CONFIRM_TABLE . "
-					WHERE session_id = '" . $db->sql_escape($user->session_id) . "'
-						AND confirm_type = " . (int) $this->type;
+			WHERE session_id = '" . $db->sql_escape($user->session_id) . "'
+				AND confirm_type = " . (int) $this->type;
 		$db->sql_query($sql);
-		
+
 		// we leave the class usable by generating a new question
 		$this->generate_code();
 	}
-	
 }
 
 ?>
 \ No newline at end of file
diff --git a/phpBB/includes/captcha/plugins/phpbb_captcha_gd_plugin.php b/phpBB/includes/captcha/plugins/phpbb_captcha_gd_plugin.php
index 5b3c09f32d..ac78b3d1c4 100644
--- a/phpBB/includes/captcha/plugins/phpbb_captcha_gd_plugin.php
+++ b/phpBB/includes/captcha/plugins/phpbb_captcha_gd_plugin.php
@@ -16,7 +16,7 @@ if (!defined('IN_PHPBB'))
 	exit;
 }
 
-/** 
+/**
 * Placeholder for autoload
 */
 if (!class_exists('phpbb_default_captcha'))
@@ -24,6 +24,9 @@ if (!class_exists('phpbb_default_captcha'))
 	include($phpbb_root_path . 'includes/captcha/plugins/captcha_abstract.' . $phpEx);
 }
 
+/**
+* @package VC
+*/
 class phpbb_captcha_gd extends phpbb_default_captcha
 {
 	function phpbb_captcha_gd()
@@ -35,7 +38,7 @@ class phpbb_captcha_gd extends phpbb_default_captcha
 			include($phpbb_root_path . 'includes/captcha/captcha_gd.' . $phpEx);
 		}
 	}
-	
+
 	function get_instance()
 	{
 		return new phpbb_captcha_gd();
@@ -45,17 +48,17 @@ class phpbb_captcha_gd extends phpbb_default_captcha
 	{
 		return (@extension_loaded('gd') || can_load_dll('gd'));
 	}
-	
+
 	function get_name()
 	{
 		return 'CAPTCHA_GD';
 	}
-	
+
 	function get_class_name()
 	{
 		return 'phpbb_captcha_gd';
 	}
-		
+
 	function acp_page($id, &$module)
 	{
 		global $db, $user, $auth, $template;
@@ -80,7 +83,6 @@ class phpbb_captcha_gd extends phpbb_default_captcha
 			'captcha_gd'			=> 'CAPTCHA_GD',
 		);
 
-
 		$module->tpl_name = 'captcha_gd_acp';
 		$module->page_title = 'ACP_VC_SETTINGS';
 		$form_key = 'acp_captcha';
@@ -112,11 +114,11 @@ class phpbb_captcha_gd extends phpbb_default_captcha
 				$var = (isset($_REQUEST[$captcha_var])) ? request_var($captcha_var, 0) : $config[$captcha_var];
 				$template->assign_var($template_var, $var);
 			}
+
 			$template->assign_vars(array(
 				'CAPTCHA_PREVIEW'	=> $this->get_demo_template($id),
 				'CAPTCHA_NAME'		=> $this->get_class_name(),
 			));
-
 		}
 	}
 }
diff --git a/phpBB/includes/captcha/plugins/phpbb_captcha_gd_wave_plugin.php b/phpBB/includes/captcha/plugins/phpbb_captcha_gd_wave_plugin.php
index 38e5aabedd..36cbaf7d79 100644
--- a/phpBB/includes/captcha/plugins/phpbb_captcha_gd_wave_plugin.php
+++ b/phpBB/includes/captcha/plugins/phpbb_captcha_gd_wave_plugin.php
@@ -24,6 +24,9 @@ if (!class_exists('captcha_abstract'))
 	include_once($phpbb_root_path . 'includes/captcha/plugins/captcha_abstract.' . $phpEx);
 }
 
+/**
+* @package VC
+*/
 class phpbb_captcha_gd_wave extends phpbb_default_captcha
 {
 
@@ -60,7 +63,7 @@ class phpbb_captcha_gd_wave extends phpbb_default_captcha
 	function acp_page($id, &$module)
 	{
 		global $config, $db, $template, $user;
-		
+
 		trigger_error($user->lang['CAPTCHA_NO_OPTIONS'] . adm_back_link($module->u_action));
 	}
 }
diff --git a/phpBB/includes/captcha/plugins/phpbb_captcha_nogd_plugin.php b/phpBB/includes/captcha/plugins/phpbb_captcha_nogd_plugin.php
index 8df11bfe8a..517b55f09e 100644
--- a/phpBB/includes/captcha/plugins/phpbb_captcha_nogd_plugin.php
+++ b/phpBB/includes/captcha/plugins/phpbb_captcha_nogd_plugin.php
@@ -16,7 +16,7 @@ if (!defined('IN_PHPBB'))
 	exit;
 }
 
-/** 
+/**
 * Placeholder for autoload
 */
 if (!class_exists('phpbb_default_captcha'))
@@ -24,6 +24,9 @@ if (!class_exists('phpbb_default_captcha'))
 	include_once($phpbb_root_path . 'includes/captcha/plugins/captcha_abstract.' . $phpEx);
 }
 
+/**
+* @package VC
+*/
 class phpbb_captcha_nogd extends phpbb_default_captcha
 {
 
@@ -36,7 +39,7 @@ class phpbb_captcha_nogd extends phpbb_default_captcha
 			include_once($phpbb_root_path . 'includes/captcha/captcha_non_gd.' . $phpEx);
 		}
 	}
-	
+
 	function get_instance()
 	{
 		return new phpbb_captcha_nogd();
@@ -46,25 +49,23 @@ class phpbb_captcha_nogd extends phpbb_default_captcha
 	{
 		return true;
 	}
-	
+
 	function get_name()
 	{
-		global $user;
-		
-		return 'CAPTCHA_NO_GD'; 
+		return 'CAPTCHA_NO_GD';
 	}
-	
+
 	function get_class_name()
 	{
 		return 'phpbb_captcha_nogd';
 	}
-	
-	
+
 	function acp_page($id, &$module)
 	{
 		global $user;
-		
+
 		trigger_error($user->lang['CAPTCHA_NO_OPTIONS'] . adm_back_link($module->u_action));
 	}
 }
 
+?>
+\ No newline at end of file
diff --git a/phpBB/includes/captcha/plugins/phpbb_recaptcha_plugin.php b/phpBB/includes/captcha/plugins/phpbb_recaptcha_plugin.php
index 42ca25ae54..a96f5ef9c6 100644
--- a/phpBB/includes/captcha/plugins/phpbb_recaptcha_plugin.php
+++ b/phpBB/includes/captcha/plugins/phpbb_recaptcha_plugin.php
@@ -22,6 +22,9 @@ if (!class_exists('phpbb_default_captcha'))
 	include_once($phpbb_root_path . 'includes/captcha/plugins/captcha_abstract.' . $phpEx);
 }
 
+/**
+* @package VC
+*/
 class phpbb_recaptcha extends phpbb_default_captcha
 {
 	var $recaptcha_server = 'http://api.recaptcha.net';
@@ -29,18 +32,16 @@ class phpbb_recaptcha extends phpbb_default_captcha
 	var $challenge;
 	var $response;
 
-	
 	function init($type)
 	{
 		global $config, $db, $user;
-		
+
 		$user->add_lang('recaptcha');
 		parent::init($type);
 		$this->challenge = request_var('recaptcha_challenge_field', '');
 		$this->response = request_var('recaptcha_response_field', '');
 	}
-	
-	
+
 	function get_instance()
 	{
 		return new phpbb_recaptcha();
@@ -48,25 +49,25 @@ class phpbb_recaptcha extends phpbb_default_captcha
 
 	function is_available()
 	{
-		global $config, $user; 
+		global $config, $user;
 		$user->add_lang('recaptcha');
 		return (isset($config['recaptcha_pubkey']) && !empty($config['recaptcha_pubkey']));
 	}
-	
+
 	function get_name()
 	{
 		return 'CAPTCHA_RECAPTCHA';
 	}
-	
+
 	function get_class_name()
 	{
 		return 'phpbb_recaptcha';
 	}
-		
+
 	function acp_page($id, &$module)
 	{
 		global $config, $db, $template, $user;
-		
+
 		$captcha_vars = array(
 			'recaptcha_pubkey'				=> 'RECAPTCHA_PUBKEY',
 			'recaptcha_privkey'				=> 'RECAPTCHA_PRIVKEY',
@@ -103,6 +104,7 @@ class phpbb_recaptcha extends phpbb_default_captcha
 				$var = (isset($_REQUEST[$captcha_var])) ? request_var($captcha_var, '') : ((isset($config[$captcha_var])) ? $config[$captcha_var] : '');
 				$template->assign_var($template_var, $var);
 			}
+
 			$template->assign_vars(array(
 				'CAPTCHA_PREVIEW'	=> $this->get_demo_template($id),
 				'CAPTCHA_NAME'		=> $this->get_class_name(),
@@ -110,47 +112,44 @@ class phpbb_recaptcha extends phpbb_default_captcha
 
 		}
 	}
-	
-	
+
 	// not needed
 	function execute_demo()
 	{
 	}
-	
-	
+
 	// not needed
 	function execute()
 	{
 	}
-	
-	
+
 	function get_template()
 	{
 		global $config, $user, $template;
-		
+
 		$template->set_filenames(array(
 			'captcha' => 'captcha_recaptcha.html')
 		);
-		
+
 		$template->assign_vars(array(
 			'RECAPTCHA_SERVER'			=> $this->recaptcha_server,
 			'RECAPTCHA_PUBKEY'			=> isset($config['recaptcha_pubkey']) ? $config['recaptcha_pubkey'] : '',
 			'RECAPTCHA_ERRORGET'		=> '',
 			'S_RECAPTCHA_AVAILABLE'		=> $this->is_available(),
 		));
-	
+
 		return $template->assign_display('captcha');
 	}
-	
+
 	function get_demo_template($id)
 	{
 		return $this->get_template();
 	}
-		
+
 	function get_hidden_fields()
 	{
 		$hidden_fields = array();
-		
+
 		// this is required for postig.php - otherwise we would forget about the captcha being already solved
 		if ($this->solved)
 		{
@@ -159,17 +158,17 @@ class phpbb_recaptcha extends phpbb_default_captcha
 		$hidden_fields['confirm_id'] = $this->confirm_id;
 		return $hidden_fields;
 	}
-		
+
 	function uninstall()
 	{
-		self::garbage_collect(0);
+		$this->garbage_collect(0);
 	}
-	
+
 	function install()
 	{
 		return;
 	}
-	
+
 	function validate()
 	{
 		if (!parent::validate())
@@ -181,7 +180,6 @@ class phpbb_recaptcha extends phpbb_default_captcha
 			return $this->recaptcha_check_answer();
 		}
 	}
-	
 
 // Code from here on is based on recaptchalib.php
 /*
@@ -218,14 +216,14 @@ class phpbb_recaptcha extends phpbb_default_captcha
  */
 
 	/**
-	 * Submits an HTTP POST to a reCAPTCHA server
-	 * @param string $host
-	 * @param string $path
-	 * @param array $data
-	 * @param int port
-	 * @return array response
-	 */
-	function _recaptcha_http_post($host, $path, $data, $port = 80) 
+	* Submits an HTTP POST to a reCAPTCHA server
+	* @param string $host
+	* @param string $path
+	* @param array $data
+	* @param int port
+	* @return array response
+	*/
+	function _recaptcha_http_post($host, $path, $data, $port = 80)
 	{
 		$req = $this->_recaptcha_qsencode ($data);
 
@@ -238,52 +236,56 @@ class phpbb_recaptcha extends phpbb_default_captcha
 		$http_request .= $req;
 
 		$response = '';
-		if( false == ( $fs = @fsockopen($host, $port, $errno, $errstr, 10) ) ) {
-				die ('Could not open socket');
+		if (false == ($fs = @fsockopen($host, $port, $errno, $errstr, 10)))
+		{
+			trigger_error('Could not open socket', E_USER_ERROR);
 		}
 
 		fwrite($fs, $http_request);
 
-		while ( !feof($fs) )
-				$response .= fgets($fs, 1160); // One TCP-IP packet
+		while (!feof($fs))
+		{
+			// One TCP-IP packet
+			$response .= fgets($fs, 1160);
+		}
 		fclose($fs);
 		$response = explode("\r\n\r\n", $response, 2);
 
 		return $response;
 	}
 
-
 	/**
-	  * Calls an HTTP POST function to verify if the user's guess was correct
-	  * @param array $extra_params an array of extra variables to post to the server
-	  * @return ReCaptchaResponse
-	  */
-	function recaptcha_check_answer ($extra_params = array())
+	* Calls an HTTP POST function to verify if the user's guess was correct
+	* @param array $extra_params an array of extra variables to post to the server
+	* @return ReCaptchaResponse
+	*/
+	function recaptcha_check_answer($extra_params = array())
 	{
 		global $config, $user;
+
 		//discard spam submissions
-		if ($this->challenge == null || strlen($this->challenge) == 0 || $this->response == null || strlen($this->response) == 0) 
+		if ($this->challenge == null || strlen($this->challenge) == 0 || $this->response == null || strlen($this->response) == 0)
 		{
-				return $user->lang['RECAPTCHA_INCORRECT'];
+			return $user->lang['RECAPTCHA_INCORRECT'];
 		}
 
-		$response = $this->_recaptcha_http_post ($this->recaptcha_verify_server, "/verify",
-										  array (
-												 'privatekey'	=> $config['recaptcha_privkey'],
-												 'remoteip'		=> $user->ip,
-												 'challenge'	=> $this->challenge,
-												 'response'		=> $this->response
-												 ) + $extra_params
-										  );
-
-		$answers = explode ("\n", $response[1]);
-		
-		if (trim ($answers[0]) === 'true') 
+		$response = $this->_recaptcha_http_post($this->recaptcha_verify_server, '/verify',
+			array(
+				'privatekey'	=> $config['recaptcha_privkey'],
+				'remoteip'		=> $user->ip,
+				'challenge'		=> $this->challenge,
+				'response'		=> $this->response
+			) + $extra_params
+		);
+
+		$answers = explode("\n", $response[1]);
+
+		if (trim($answers[0]) === 'true')
 		{
 			$this->solved = true;
 			return false;
 		}
-		else 
+		else
 		{
 			if ($answers[1] === 'incorrect-captcha-sol')
 			{
@@ -291,22 +293,24 @@ class phpbb_recaptcha extends phpbb_default_captcha
 			}
 		}
 	}
-	
-		/**
-	 * Encodes the given data into a query string format
-	 * @param $data - array of string elements to be encoded
-	 * @return string - encoded request
-	 */
-	function _recaptcha_qsencode ($data)
+
+	/**
+	* Encodes the given data into a query string format
+	* @param $data - array of string elements to be encoded
+	* @return string - encoded request
+	*/
+	function _recaptcha_qsencode($data)
 	{
 		$req = '';
-		foreach ( $data as $key => $value )
+		foreach ($data as $key => $value)
 		{
-			$req .= $key . '=' . urlencode( stripslashes($value) ) . '&';
+			$req .= $key . '=' . urlencode(stripslashes($value)) . '&';
 		}
+
 		// Cut the last '&'
-		$req=substr($req,0,strlen($req)-1);
+		$req = substr($req, 0, strlen($req) - 1);
 		return $req;
 	}
 }
 
+?>
+\ No newline at end of file