Index: branches/5.2.x/units/topics/topics_event_handler.php
===================================================================
diff -u -N -r14252 -r14624
--- branches/5.2.x/units/topics/topics_event_handler.php	(.../topics_event_handler.php)	(revision 14252)
+++ branches/5.2.x/units/topics/topics_event_handler.php	(.../topics_event_handler.php)	(revision 14624)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: topics_event_handler.php 14252 2011-03-16 21:26:29Z alex $
+* @version	$Id: topics_event_handler.php 14624 2011-10-04 09:33:21Z alex $
 * @package	In-Bulletin
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -16,13 +16,14 @@
 
 	class TopicsEventHandler extends kCatDBEventHandler {
 
-
 		/**
 		 * Checks topic lock permission
 		 *
 		 * @param kEvent $event
+		 * @return bool
+		 * @access public
 		 */
-		function CheckPermission(&$event)
+		public function CheckPermission(&$event)
 		{
 			if ($event->Name == 'OnTopicLockToggle') {
 				$object =& $event->getObject();
@@ -64,8 +65,10 @@
 		 * Cache topic owner
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function OnBeforeItemUpdate(&$event)
+		protected function OnBeforeItemUpdate(&$event)
 		{
 			parent::OnBeforeItemUpdate($event);
 
@@ -76,8 +79,10 @@
 		 * Cache topic owner
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function OnBeforeItemCreate(&$event)
+		protected function OnBeforeItemCreate(&$event)
 		{
 			parent::OnBeforeItemCreate($event);
 
@@ -86,34 +91,36 @@
 			$object =& $event->getObject();
 			/* @var $object kCatDBItem */
 
-			if (!$object->GetDBField('TodayDate')) {
+			if ( !$object->GetDBField('TodayDate') ) {
 				$object->SetDBField('TodayDate', adodb_date('Y-m-d'));
 			}
 
 			$post_helper =& $this->Application->recallObject('PostHelper');
 			/* @var $post_helper PostHelper */
 
-			$object->SetDBField('TopicText', $post_helper->CensorText( $object->GetDBField('TopicText') ));
+			$object->SetDBField('TopicText', $post_helper->CensorText($object->GetDBField('TopicText')));
 		}
 
 		/**
 		 * Creates 1st post when topic is created
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function OnAfterItemCreate(&$event)
+		protected function OnAfterItemCreate(&$event)
 		{
 			parent::OnAfterItemCreate($event);
 
-			if ($event->Special == '-item') {
+			if ( $event->Special == '-item' ) {
 				// don't create first post when cloning
 				return ;
 			}
 
 			$object =& $event->getObject();
 			/* @var $object kDBItem */
 
-			$post =& $this->Application->recallObject($event->Prefix.'-post', null, Array ('skip_autoload' => true));
+			$post =& $this->Application->recallObject($event->Prefix . '-post', null, Array ('skip_autoload' => true));
 			/* @var $post kDBItem */
 
 			$post->SetDBField('Pending', $object->GetDBField('Status') == STATUS_ACTIVE ? 0 : 1);
@@ -157,18 +164,20 @@
 		}
 
 		/**
-		 * Makes first post body field non-requered when topic has posts already
+		 * Makes first post body field non-required when topic has posts already
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function OnAfterItemLoad(&$event)
+		protected function OnAfterItemLoad(&$event)
 		{
 			parent::OnAfterItemLoad($event);
 
 			$object =& $event->getObject();
 			/* @var $object kCatDBItem */
 
-			if ($object->GetDBField('Posts') > 0 || !$this->Application->isAdminUser) {
+			if ( $object->GetDBField('Posts') > 0 || !$this->Application->isAdminUser ) {
 				$object->setRequired('PostingText', false);
 			}
 		}