Index: branches/5.2.x/core/units/reviews/reviews_config.php
===================================================================
diff -u -N -r14726 -r14769
--- branches/5.2.x/core/units/reviews/reviews_config.php	(.../reviews_config.php)	(revision 14726)
+++ branches/5.2.x/core/units/reviews/reviews_config.php	(.../reviews_config.php)	(revision 14769)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: reviews_config.php 14726 2011-11-03 08:43:30Z alex $
+* @version	$Id: reviews_config.php 14769 2011-11-21 16:53:53Z alex $
 * @package	In-Portal
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -31,6 +31,8 @@
 			5 => 'mode',
 		),
 
+		'RewriteListener' => 'CategoryItemRewrite:ReviewRewriteListener',
+
 		'ParentPrefix' 		=> 'p', // replace all usage of rev to "p-rev" and then remove this param from here and Prefix too
 		'ConfigMapping' => Array (
 			'PerPage'				=>	'Comm_Perpage_Reviews',
@@ -58,17 +60,17 @@
 
 		'CalculatedFields' => Array (
 			'' => Array (
-					'ReviewedBy' => 'IF( ISNULL(pu.Username), IF (%1$s.CreatedById = ' . USER_ROOT . ', \'root\', IF (%1$s.CreatedById = ' . USER_GUEST . ', \'Guest\', \'n/a\')), pu.Username )',
+				'ReviewedBy' => 'CASE %1$s.CreatedById WHEN ' . USER_ROOT . ' THEN "root" WHEN ' . USER_GUEST . ' THEN "Guest" ELSE IF(CONCAT(pu.FirstName, pu.LastName) <> "", CONCAT(pu.FirstName, " ", pu.LastName), pu.Username) END',
 			),
 
 			'products' => Array (
-				'ReviewedBy'	=>	'IF( ISNULL(pu.Username), IF (%1$s.CreatedById = ' . USER_ROOT . ', \'root\', IF (%1$s.CreatedById = ' . USER_GUEST . ', \'Guest\', \'n/a\')), pu.Username )',
+				'ReviewedBy'	=>	'CASE %1$s.CreatedById WHEN ' . USER_ROOT . ' THEN "root" WHEN ' . USER_GUEST . ' THEN "Guest" ELSE IF(CONCAT(pu.FirstName, pu.LastName) <> "", CONCAT(pu.FirstName, " ", pu.LastName), pu.Username) END',
 				'ItemName'		=>	'pr.l1_Name',
 				'ProductId'		=>	'pr.ProductId',
 			),
 
 			'product' => Array (
-				'ReviewedBy'	=>	'IF( ISNULL(pu.Username), IF (%1$s.CreatedById = ' . USER_ROOT . ', \'root\', IF (%1$s.CreatedById = ' . USER_GUEST . ', \'Guest\', \'n/a\')), pu.Username )',
+				'ReviewedBy'	=>	'CASE %1$s.CreatedById WHEN ' . USER_ROOT . ' THEN "root" WHEN ' . USER_GUEST . ' THEN "Guest" ELSE IF(CONCAT(pu.FirstName, pu.LastName) <> "", CONCAT(pu.FirstName, " ", pu.LastName), pu.Username) END',
 				'ItemName'		=>	'pr.l1_Name',
 				'ProductId'		=>	'pr.ProductId',
 			),
@@ -96,7 +98,7 @@
 		'ListSortings' => Array (
 			'' => Array (
 				'ForcedSorting' => Array ('Priority' => 'desc'),
-				'Sorting' => Array ('CreatedOn' => 'desc'),
+				'Sorting' => Array ('ReviewId' => 'desc'),
 			)
 		),