Index: branches/5.2.x/units/link_validation/link_validation_eh.php
===================================================================
diff -u -N -r14254 -r14626
--- branches/5.2.x/units/link_validation/link_validation_eh.php	(.../link_validation_eh.php)	(revision 14254)
+++ branches/5.2.x/units/link_validation/link_validation_eh.php	(.../link_validation_eh.php)	(revision 14626)
@@ -1,6 +1,6 @@
 <?php
 /**
-* @version	$Id: link_validation_eh.php 14254 2011-03-16 21:30:58Z alex $
+* @version	$Id: link_validation_eh.php 14626 2011-10-04 09:34:41Z alex $
 * @package	In-Link
 * @copyright	Copyright (C) 1997 - 2009 Intechnic. All rights reserved.
 * @license      GNU/GPL
@@ -50,11 +50,13 @@
 		}
 
 		/**
-		 * Checks permissions of user
+		 * Checks user permission to execute given $event
 		 *
 		 * @param kEvent $event
+		 * @return bool
+		 * @access public
 		 */
-		function CheckPermission(&$event)
+		public function CheckPermission(&$event)
 		{
 			$check_events = Array ('OnApproveLinks', 'OnDeclineLinks', 'OnDeleteLinks');
 			if (in_array($event->Name, $check_events)) {
@@ -89,10 +91,12 @@
 		/**
 		 * Adds calculates fields for category name
 		 *
-		 * @param kDBItem $object
+		 * @param kDBItem|kDBList $object
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function prepareObject(&$object, &$event)
+		protected function prepareObject(&$object, &$event)
 		{
 			parent::prepareObject($object, $event);
 
@@ -103,8 +107,11 @@
 		 * Allows to show only invalid links
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
+		 * @see kDBEventHandler::OnListBuild()
 		 */
-		function SetCustomQuery(&$event)
+		protected function SetCustomQuery(&$event)
 		{
 			$object =& $event->getObject();
 			/* @var $object kDBList */
@@ -400,26 +407,29 @@
 		 * Approves/declines selected links
 		 *
 		 * @param kEvent $event
+		 * @return void
+		 * @access protected
 		 */
-		function iterateItems(&$event)
+		protected function iterateItems(&$event)
 		{
-			if ($this->Application->CheckPermission('SYSTEM_ACCESS.READONLY', 1)) {
+			if ( $this->Application->CheckPermission('SYSTEM_ACCESS.READONLY', 1) ) {
 				$event->status = kEvent::erFAIL;
 				return;
 			}
 
 			$ids = $this->_getSelectedIds($event);
-			if (!$ids) {
-				return ;
+			if ( !$ids ) {
+				return;
 			}
 
 			$object =& $this->Application->recallObject('l.-item', null, Array ('skip_autoload' => true));
 			/* @var $object kCatDBItem */
 
 			foreach ($ids as $id) {
+				$ret = true;
 				$object->Load($id);
 
-				switch ($event->Name) {
+				switch ( $event->Name ) {
 					case 'OnApproveLinks':
 						$ret = $object->ApproveChanges();
 						break;
@@ -429,7 +439,7 @@
 						break;
 				}
 
-				if (!$ret) {
+				if ( !$ret ) {
 					$event->status = kEvent::erFAIL;
 					$event->redirect = false;
 					break;