Index: branches/5.2.x/core/units/visits/visits_event_handler.php =================================================================== diff -u -N -r14675 -r14989 --- branches/5.2.x/core/units/visits/visits_event_handler.php (.../visits_event_handler.php) (revision 14675) +++ branches/5.2.x/core/units/visits/visits_event_handler.php (.../visits_event_handler.php) (revision 14989) @@ -1,6 +1,6 @@ Array('self' => true), + + $permissions = Array ( + 'OnItemBuild' => Array ('self' => true), ); $this->permMapping = array_merge($this->permMapping, $permissions); @@ -67,8 +71,10 @@ * @access protected * @see kDBEventHandler::OnListBuild() */ - protected function SetCustomQuery(&$event) + protected function SetCustomQuery(kEvent &$event) { + parent::SetCustomQuery($event); + $object =& $event->getObject(); /* @var $object kDBList */ @@ -101,20 +107,19 @@ * @return void * @access protected */ - protected function prepareObject(&$object, &$event) + protected function prepareObject(&$object, kEvent &$event) { $types = $event->getEventParam('types'); - if(method_exists($object, 'AddGroupByField')) - { - if( ($types == 'myvisitors' || !$types) && $object->Special == 'incommerce') - { + + if ( method_exists($object, 'AddGroupByField') ) { + if ( (!$types || $types == 'myvisitors') && $object->Special == 'incommerce' ) { $object->addCalculatedField('OrderTotalAmountSum', 'SUM(IF(ord.Status = 4, ord.SubTotal+ord.ShippingCost+ord.VAT, 0))'); $object->addCalculatedField('OrderAffiliateCommissionSum', 'SUM( IF(ord.Status = 4,ord.AffiliateCommission,0))'); $object->addCalculatedField('OrderCountByVisit', 'SUM( IF(ord.Status = 4, 1, 0) )'); } - if (!$types){ - $object->AddGroupByField('%1$s.VisitId'); + if ( !$types ) { + $object->AddGroupByField('%1$s.VisitId'); } } }