Index: branches/5.1.x/core/units/users/users_event_handler.php
===================================================================
diff -u -N -r14433 -r14445
--- branches/5.1.x/core/units/users/users_event_handler.php	(.../users_event_handler.php)	(revision 14433)
+++ branches/5.1.x/core/units/users/users_event_handler.php	(.../users_event_handler.php)	(revision 14445)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: users_event_handler.php 14433 2011-07-07 15:02:49Z alex $
+* @version	$Id: users_event_handler.php 14445 2011-07-08 12:56:13Z alex $
 * @package	In-Portal
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -519,10 +519,9 @@
 			if (!$this->isSubscriberOnly($event, true)) {
 				$cs_helper->CheckStateField($event, 'State', 'Country');
 			}
-			else {
-				// no need to make password required when guest user tries to subscribe to mailing
-				$this->_makePasswordRequired($event);
-			}
+
+			$this->_makePasswordRequired($event);
+
 			$cs_helper->PopulateStates($event, 'State', 'Country');
 
 			$object =& $event->getObject();
@@ -722,14 +721,13 @@
 					}
 				}
 				else {
-					$object->SetField('Email', $user_email);
-					$object->SetField('Login', $user_email);
-					$object->SetDBField('dob', 1);
-					$object->SetDBField('dob_date', 1);
-					$object->SetDBField('dob_time', 1);
+					$password = makepassword4();
+			      	$object->SetField('Password', $password);
+			      	$object->SetField('VerifyPassword', $password);
+					$object->SetDBField('Email', $user_email);
+					$object->SetDBField('Login', $user_email);
 					$object->SetDBField('Status', STATUS_ACTIVE); // make user subscriber Active by default
-					$ip = getenv('HTTP_X_FORWARDED_FOR')?getenv('HTTP_X_FORWARDED_FOR'):getenv('REMOTE_ADDR');
-					$object->SetDBField('ip', $ip);
+					$object->SetDBField('ip', $_SERVER['REMOTE_ADDR']);
 
 					$this->Application->SetVar('IsSubscriber', 1);