Index: branches/5.3.x/core/units/reviews/reviews_event_handler.php =================================================================== diff -u -N -r15938 -r16519 --- branches/5.3.x/core/units/reviews/reviews_event_handler.php (.../reviews_event_handler.php) (revision 15938) +++ branches/5.3.x/core/units/reviews/reviews_event_handler.php (.../reviews_event_handler.php) (revision 16519) @@ -1,6 +1,6 @@ Name == 'OnAddReview' || $event->Name == 'OnCreate' ) { + /** @var kPermissionsHelper $perm_helper */ $perm_helper = $this->Application->recallObject('PermissionsHelper'); - /* @var $perm_helper kPermissionsHelper */ $parent_prefix = $event->getUnitConfig()->getParentPrefix(); + + /** @var kCatDBItem $main_object */ $main_object = $this->Application->recallObject($parent_prefix); - /* @var $main_object kCatDBItem */ $perm_name = $this->getPermPrefix($event).'.REVIEW'; $res = $this->Application->CheckPermission($perm_name, 0, $main_object->GetDBField('CategoryId')) || @@ -71,8 +72,8 @@ if ( in_array($event->Name, $check_events) ) { // check for PRODUCT.VIEW permission + /** @var kPermissionsHelper $perm_helper */ $perm_helper = $this->Application->recallObject('PermissionsHelper'); - /* @var $perm_helper kPermissionsHelper */ $perm_prefix = $this->getPermPrefix($event); @@ -133,8 +134,8 @@ return false; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ // 1. get main item resource id (use object, because of temp tables in admin) $sql = 'SELECT ItemId @@ -182,8 +183,8 @@ { parent::SetCustomQuery($event); + /** @var kDBList $object */ $object = $event->getObject(); - /* @var $object kDBList */ if ( !$this->Application->isAdminUser ) { $object->addFilter('active', '%1$s.Status = ' . STATUS_ACTIVE); @@ -198,8 +199,8 @@ $object->clearFilters(); $parent_info = $object->getLinkedInfo(); + /** @var kDBItem $parent */ $parent = $this->Application->recallObject($parent_info['ParentPrefix']); - /* @var $parent kDBItem */ $object->addFilter('item_reviews', '%1$s.ItemId = ' . $parent->GetDBField('ResourceId')); break; @@ -247,8 +248,9 @@ { $parent_prefix = $event->getUnitConfig()->getParentPrefix(); + + /** @var kCatDBItem $main_object */ $main_object = $this->Application->recallObject($parent_prefix); - /* @var $main_object kCatDBItem */ $ret = STATUS_DISABLED; $perm_name = $this->getPermPrefix($event).'.REVIEW'; @@ -273,8 +275,8 @@ { parent::OnBeforeItemCreate($event); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $parent_info = $object->getLinkedInfo(); $item_type = $this->Application->getUnitConfig($parent_info['ParentPrefix'])->getItemType(); @@ -288,8 +290,8 @@ return ; } + /** @var SpamHelper $spam_helper */ $spam_helper = $this->Application->recallObject('SpamHelper'); - /* @var $spam_helper SpamHelper */ $spam_helper->InitHelper($parent_info['ParentId'], 'Review', 0); @@ -322,8 +324,8 @@ { parent::OnBeforeItemUpdate($event); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $rating = $object->GetDBField('Rating'); @@ -346,11 +348,11 @@ $this->updateSubitemCounters($event); if ( !$this->Application->isAdminUser ) { + /** @var SpamHelper $spam_helper */ $spam_helper = $this->Application->recallObject('SpamHelper'); - /* @var $spam_helper SpamHelper */ + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $parent_info = $object->getLinkedInfo(); @@ -385,8 +387,8 @@ $this->updateSubitemCounters($event); + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ if ( $this->Application->isAdminUser && !$object->IsTempTable() ) { // send email on review status change from reviews grid in admin @@ -410,14 +412,14 @@ */ function _loadMainObject($event) { + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $config = $event->getUnitConfig(); $parent_prefix = $config->getParentPrefix(); + /** @var kDBItem $main_object */ $main_object = $this->Application->recallObject($parent_prefix, null, Array ('skip_autoload' => true)); - /* @var $main_object kDBItem */ $main_object->Load($object->GetDBField($config->getForeignKey()), $config->getParentTableKey()); } @@ -434,8 +436,8 @@ return; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ $parent_prefix = $event->getUnitConfig()->getParentPrefix(); $parent_table = $this->Application->getUnitConfig($parent_prefix)->getTableName(); @@ -542,12 +544,12 @@ return; } + /** @var kDBItem $object */ $object = $event->getObject(); - /* @var $object kDBItem */ if ( $this->Application->GetVar('ajax') == 'yes' ) { + /** @var AjaxFormHelper $ajax_form_helper */ $ajax_form_helper = $this->Application->recallObject('AjaxFormHelper'); - /* @var $ajax_form_helper AjaxFormHelper */ $params = Array ('status' => 'OK');