Index: branches/5.2.x/units/private_messages/private_message_eh.php
===================================================================
diff -u -N -r14848 -r14985
--- branches/5.2.x/units/private_messages/private_message_eh.php	(.../private_message_eh.php)	(revision 14848)
+++ branches/5.2.x/units/private_messages/private_message_eh.php	(.../private_message_eh.php)	(revision 14985)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: private_message_eh.php 14848 2011-12-08 12:28:28Z alex $
+* @version	$Id: private_message_eh.php 14985 2012-01-04 16:04:11Z alex $
 * @package	In-Bulletin
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -16,13 +16,21 @@
 
 	class PrivateMessageEventHandler extends kDBEventHandler {
 
-		function mapPermissions()
+		/**
+		 * Allows to override standard permission mapping
+		 *
+		 * @return void
+		 * @access protected
+		 * @see kEventHandler::$permMapping
+		 */
+		protected function mapPermissions()
 		{
 			parent::mapPermissions();
-			$permissions = Array(
-				'OnItemBuild' => Array('self' => true),
-				'OnCreate' => Array('self' => true),
-				'OnDelete' => Array('self' => true),
+
+			$permissions = Array (
+				'OnItemBuild' => Array ('self' => true),
+				'OnCreate' => Array ('self' => true),
+				'OnDelete' => Array ('self' => true),
 			);
 
 			$this->permMapping = array_merge($this->permMapping, $permissions);
@@ -36,10 +44,13 @@
 		 * @access protected
 		 * @see kDBEventHandler::OnListBuild()
 		 */
-		protected function SetCustomQuery(&$event)
+		protected function SetCustomQuery(kEvent &$event)
 		{
+			parent::SetCustomQuery($event);
+
 			$folder_id = $this->Application->GetVar('folder_id');
-			if ($folder_id === false) {
+
+			if ( $folder_id === false ) {
 				$folder_id = PM_FOLDER_INBOX;
 				$this->Application->SetVar('folder_id', $folder_id);
 			}
@@ -48,13 +59,15 @@
 			/* @var $object kDBList */
 
 			$user_id = $this->Application->RecallVar('user_id');
-			if ($folder_id == PM_FOLDER_INBOX) {
-				$object->addFilter('owner_filter', '%1$s.ToId = '.$user_id);
+
+			if ( $folder_id == PM_FOLDER_INBOX ) {
+				$object->addFilter('owner_filter', '%1$s.ToId = ' . $user_id);
 			}
 			else {
-				$object->addFilter('owner_filter', '%1$s.FromId = '.$user_id);
+				$object->addFilter('owner_filter', '%1$s.FromId = ' . $user_id);
 			}
-			$object->addFilter('folder_filter', '%1$s.FolderId = '.$folder_id);
+
+			$object->addFilter('folder_filter', '%1$s.FolderId = ' . $folder_id);
 		}
 
 		/**
@@ -224,11 +237,14 @@
 		 *
 		 * @param kEvent $event
 		 * @return bool
+		 * @access protected
 		 */
-		function checkItemStatus(&$event)
+		protected function checkItemStatus(kEvent &$event)
 		{
 			$object =& $event->getObject();
-			if (!$object->isLoaded()) {
+			/* @var $object kDBItem */
+
+			if ( !$object->isLoaded() ) {
 				return true;
 			}