From 4263cd5e24f56d1a0154dc3ff17a28def7c9a7af Mon Sep 17 00:00:00 2001
From: saeedbashir <sbashir@edx.org>
Date: Fri, 14 Jul 2023 15:30:53 +0500
Subject: [PATCH] chore: update find courses clicked analytics

---
 Source/CoursesContainerViewController.swift | 1 +
 Source/EnrolledTabBarViewController.swift   | 3 ---
 Source/OEXAnalytics.h                       | 2 +-
 Source/OEXAnalytics.m                       | 4 ++--
 Source/OEXRouter+Swift.swift                | 1 -
 5 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/Source/CoursesContainerViewController.swift b/Source/CoursesContainerViewController.swift
index 8ae216a04f..8636da4c63 100644
--- a/Source/CoursesContainerViewController.swift
+++ b/Source/CoursesContainerViewController.swift
@@ -201,6 +201,7 @@ class CoursesContainerViewController: UICollectionViewController {
             let footerView = collectionView.dequeueReusableSupplementaryView(ofKind: kind, withReuseIdentifier: EnrolledCoursesFooterView.identifier, for: indexPath) as! EnrolledCoursesFooterView
             footerView.findCoursesAction = {[weak self] in
                 self?.environment.router?.showCourseCatalog(fromController: self, bottomBar: nil)
+                self?.environment.analytics.trackUserFindsCourses(self?.courses.count ?? 0)
             }
             return footerView
         }
diff --git a/Source/EnrolledTabBarViewController.swift b/Source/EnrolledTabBarViewController.swift
index fc5098f148..9a0fea82e7 100644
--- a/Source/EnrolledTabBarViewController.swift
+++ b/Source/EnrolledTabBarViewController.swift
@@ -169,8 +169,5 @@ class EnrolledTabBarViewController: UITabBarController, UITabBarControllerDelega
 extension EnrolledTabBarViewController {
     func tabBarController(_ tabBarController: UITabBarController, didSelect viewController: UIViewController){
         navigationItem.title = viewController.navigationItem.title
-        if TabBarOptions.options[tabBarController.selectedIndex] == .CourseCatalog {
-            environment.analytics.trackUserFindsCourses()
-        }
     }
 }
diff --git a/Source/OEXAnalytics.h b/Source/OEXAnalytics.h
index d203d30fb3..913b77f1bd 100644
--- a/Source/OEXAnalytics.h
+++ b/Source/OEXAnalytics.h
@@ -132,7 +132,7 @@ NS_ASSUME_NONNULL_BEGIN
 
 - (void)trackUserDoesNotHaveAccount;
 
-- (void)trackUserFindsCourses;
+- (void)trackUserFindsCourses:(NSInteger) enrolledCounses;
 
 //Discussion search screen event
 - (void) trackDiscussionSearchScreenWithName:(NSString *) screenName courseId:(NSString *) courseID value:(nullable NSString *) value searchQuery:(NSString *) query;
diff --git a/Source/OEXAnalytics.m b/Source/OEXAnalytics.m
index dee3d380af..fdaeed96bf 100644
--- a/Source/OEXAnalytics.m
+++ b/Source/OEXAnalytics.m
@@ -482,13 +482,13 @@ - (void)trackUserDoesNotHaveAccount {
     [self trackEvent:event forComponent:nil withInfo:@{}];
 }
 
-- (void)trackUserFindsCourses {
+- (void)trackUserFindsCourses:(NSInteger) enrolledCounses {
     OEXAnalyticsEvent* event = [[OEXAnalyticsEvent alloc] init];
     event.name = value_find_courses;
     event.displayName = @"Find Courses Clicked";
     event.category = OEXAnalyticsCategoryUserEngagement;
     event.label = @"course-discovery";
-    [self trackEvent:event forComponent:nil withInfo:@{}];
+    [self trackEvent:event forComponent:nil withInfo:@{@"enrolled_courses_count": [NSNumber numberWithInteger:enrolledCounses]}];
 }
 
 #pragma mark- Discussion
diff --git a/Source/OEXRouter+Swift.swift b/Source/OEXRouter+Swift.swift
index d14e2c964d..5a0b43fc3c 100644
--- a/Source/OEXRouter+Swift.swift
+++ b/Source/OEXRouter+Swift.swift
@@ -492,7 +492,6 @@ extension OEXRouter {
         } else {
             showControllerFromStartupScreen(controller: controller)
         }
-        self.environment.analytics.trackUserFindsCourses()
     }
     
     func discoveryViewController(bottomBar: UIView? = nil, searchQuery: String? = nil) -> UIViewController? {