Skip to content
Extraits de code Groupes Projets
Valider 7c2fba6d rédigé par Thomas Steur's avatar Thomas Steur Validation de mattab
Parcourir les fichiers

fix exit page was not correct when last action was download or outlink

(cherry picked from commit 092ccb6)
parent 27c45269
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
...@@ -248,12 +248,12 @@ abstract class Action ...@@ -248,12 +248,12 @@ abstract class Action
public function getIdActionUrlForEntryAndExitIds() public function getIdActionUrlForEntryAndExitIds()
{ {
return $this->getIdActionUrl(); return false;
} }
public function getIdActionNameForEntryAndExitIds() public function getIdActionNameForEntryAndExitIds()
{ {
return $this->getIdActionName(); return false;
} }
public function getIdActionName() public function getIdActionName()
......
...@@ -51,6 +51,16 @@ class ActionPageview extends Action ...@@ -51,6 +51,16 @@ class ActionPageview extends Action
return true; return true;
} }
public function getIdActionUrlForEntryAndExitIds()
{
return $this->getIdActionUrl();
}
public function getIdActionNameForEntryAndExitIds()
{
return $this->getIdActionName();
}
private function cleanupActionName($actionName) private function cleanupActionName($actionName)
{ {
// get the delimiter, by default '/'; BC, we read the old action_category_delimiter first (see #1067) // get the delimiter, by default '/'; BC, we read the old action_category_delimiter first (see #1067)
......
...@@ -79,8 +79,9 @@ class VisitTotalActions extends VisitDimension ...@@ -79,8 +79,9 @@ class VisitTotalActions extends VisitDimension
} }
$actionType = $action->getActionType(); $actionType = $action->getActionType();
$types = array(Action::TYPE_SITE_SEARCH, Action::TYPE_EVENT, Action::TYPE_OUTLINK, Action::TYPE_DOWNLOAD);
if (in_array($actionType, array(Action::TYPE_SITE_SEARCH, Action::TYPE_EVENT))) { if (in_array($actionType, $types)) {
return $increment; return $increment;
} }
......
...@@ -40,15 +40,4 @@ class ActionContent extends Action ...@@ -40,15 +40,4 @@ class ActionContent extends Action
); );
} }
// Do not track this Event URL as Entry/Exit Page URL (leave the existing entry/exit)
public function getIdActionUrlForEntryAndExitIds()
{
return false;
}
// Do not track this Event Name as Entry/Exit Page Title (leave the existing entry/exit)
public function getIdActionNameForEntryAndExitIds()
{
return false;
}
} }
...@@ -73,18 +73,6 @@ class ActionEvent extends Action ...@@ -73,18 +73,6 @@ class ActionEvent extends Action
return array('idaction_url' => $actionUrl); return array('idaction_url' => $actionUrl);
} }
// Do not track this Event URL as Entry/Exit Page URL (leave the existing entry/exit)
public function getIdActionUrlForEntryAndExitIds()
{
return false;
}
// Do not track this Event Name as Entry/Exit Page Title (leave the existing entry/exit)
public function getIdActionNameForEntryAndExitIds()
{
return false;
}
public function writeDebugInfo() public function writeDebugInfo()
{ {
$write = parent::writeDebugInfo(); $write = parent::writeDebugInfo();
......
0% Chargement en cours ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter