Index: branches/5.2.x/core/units/helpers/mailing_list_helper.php
===================================================================
diff -u -N -r14861 -r14870
--- branches/5.2.x/core/units/helpers/mailing_list_helper.php	(.../mailing_list_helper.php)	(revision 14861)
+++ branches/5.2.x/core/units/helpers/mailing_list_helper.php	(.../mailing_list_helper.php)	(revision 14870)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: mailing_list_helper.php 14861 2011-12-13 09:39:34Z alex $
+* @version	$Id: mailing_list_helper.php 14870 2011-12-16 11:54:01Z alex $
 * @package	In-Portal
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -111,38 +111,50 @@
 		 */
 		function generateRecipients($id, $fields_hash)
 		{
-			$recipients = explode(';', $fields_hash['To']);
+			// for each group convert ids to names
+			$recipient_emails = Array ();
+			$recipients_grouped = $this->groupRecipientsByType(explode(';', $fields_hash['To']));
 
-			// 1. group recipients by types
+			foreach ($recipients_grouped as $recipient_type => $group_recipients) {
+				$recipient_emails = array_merge($recipient_emails, $this->_getRecipientEmails($recipient_type, $group_recipients));
+			}
+
+			$recipient_emails = array_unique($recipient_emails);
+
+			return Array (
+				'ToParsed' => serialize($recipient_emails),
+				'EmailsTotal' => count($recipient_emails),
+			);
+		}
+
+		/**
+		 * Groups recipients by type
+		 *
+		 * @param Array $recipients
+		 * @return Array
+		 * @access public
+		 */
+		public function groupRecipientsByType($recipients)
+		{
 			$recipients_grouped = Array ();
+
 			foreach ($recipients as $recipient) {
-				if (strpos($recipient, '_') !== false) {
+				if ( strpos($recipient, '_') !== false ) {
 					list ($recipient_type, $recipient_id) = explode('_', $recipient);
 				}
 				else {
 					$recipient_type = 'direct';
 					$recipient_id = $recipient;
 				}
 
-				if (!array_key_exists($recipient_type, $recipients_grouped)) {
+				if ( !array_key_exists($recipient_type, $recipients_grouped) ) {
 					$recipients_grouped[$recipient_type] = Array ();
 				}
 
 				$recipients_grouped[$recipient_type][] = $recipient_id;
 			}
 
-			// for each group convert ids to names
-			$recpient_emails = Array ();
-			foreach ($recipients_grouped as $recipient_type => $group_recipients) {
-				$recpient_emails = array_merge($recpient_emails, $this->_getRecipientEmails($recipient_type, $group_recipients));
-			}
-
-			$recpient_emails = array_unique($recpient_emails);
-
-			return Array (
-				'ToParsed' => serialize($recpient_emails),
-				'EmailsTotal' => count($recpient_emails),
-			);
+			return $recipients_grouped;
 		}
 
 		function _getRecipientEmails($recipient_type, $recipient_ids)