Index: branches/5.2.x/units/product_option_combinations/product_option_formatters.php
===================================================================
diff -u -N -r14258 -r14625
--- branches/5.2.x/units/product_option_combinations/product_option_formatters.php	(.../product_option_formatters.php)	(revision 14258)
+++ branches/5.2.x/units/product_option_combinations/product_option_formatters.php	(.../product_option_formatters.php)	(revision 14625)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: product_option_formatters.php 14258 2011-03-16 21:43:52Z alex $
+* @version	$Id: product_option_formatters.php 14625 2011-10-04 09:34:12Z alex $
 * @package	In-Commerce
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license	Commercial License
@@ -15,6 +15,15 @@
 
 	class kCombinationFormatter extends kFormatter {
 
+		/**
+		 * Formats value of a given field
+		 *
+		 * @param string $value
+		 * @param string $field_name
+		 * @param kDBItem|kDBList $object
+		 * @param string $format
+		 * @return string
+		 */
 		function Format($value, $field_name, $object, $format=null)
 		{
 			$o = '';
@@ -45,18 +54,18 @@
 		 * @param string $field_name
 		 * @param kDBItem $object
 		 * @return mixed
+		 * @access public
 		 */
-		function Parse($value, $field_name, &$object)
+		public function Parse($value, $field_name, &$object)
 		{
-			if (!is_array($value)) {
+			if ( !is_array($value) ) {
 				$object->SetError($field_name, 'required');
 				return '';
 			}
 			else {
 				if ( $object->isRequired($field_name) ) {
-					foreach ($value as $key=>$val)
-					{
-						if ($val == '') {
+					foreach ($value as $key => $val) {
+						if ( $val == '' ) {
 							$object->SetError($field_name, 'required');
 						}
 					}
@@ -69,11 +78,11 @@
 	class kCombPriceFormatter extends kFormatter {
 
 		/**
-		 * Enter description here...
+		 * Formats value of a given field
 		 *
-		 * @param mixed $value
+		 * @param string $value
 		 * @param string $field_name
-		 * @param kDBItem $object
+		 * @param kDBItem|kDBList $object
 		 * @param string $format
 		 * @return string
 		 */