From bc2acaea51e857bd048d9745474462afd3dda1d3 Mon Sep 17 00:00:00 2001 From: Albert Santoni Date: Tue, 24 Mar 2015 11:18:03 -0400 Subject: [PATCH] Don't fire conversion tracking for the sourcefabric_admin user --- airtime_mvc/application/common/GoogleAnalytics.php | 4 +++- airtime_mvc/application/models/User.php | 9 +++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/airtime_mvc/application/common/GoogleAnalytics.php b/airtime_mvc/application/common/GoogleAnalytics.php index 44d951cc7..00a95a51e 100644 --- a/airtime_mvc/application/common/GoogleAnalytics.php +++ b/airtime_mvc/application/common/GoogleAnalytics.php @@ -77,7 +77,9 @@ class Application_Common_GoogleAnalytics $oldPlan = Application_Model_Preference::GetOldPlanLevel(); - if ($user->isSuperAdmin() && $request->getControllerKey() !== "thank-you") + if ($user->isSuperAdmin() && + !$user->isSourcefabricAdmin() && + $request->getControllerKey() !== "thank-you") { //Only tracking trial->paid conversions for now. if ($oldPlan == "trial") diff --git a/airtime_mvc/application/models/User.php b/airtime_mvc/application/models/User.php index 2001a97f8..ac428fefc 100644 --- a/airtime_mvc/application/models/User.php +++ b/airtime_mvc/application/models/User.php @@ -69,6 +69,15 @@ class Application_Model_User return $result; } + public function isSourcefabricAdmin() + { + $username = $this->getLogin(); + if ($username == "sourcefabric_admin") { + return true; + } + return false; + } + // TODO : refactor code to only accept arrays for isUserType and // simplify code even further public function isUserType($type)