From 5ab2533b1e168500b20cebe351f6df9843e19870 Mon Sep 17 00:00:00 2001
From: jaebradley <jae.b.bradley@gmail.com>
Date: Tue, 23 Jan 2018 10:28:01 -0500
Subject: [PATCH] add try/catch for analytics api calls

add comment

update jsx comment

simplify try block and update comment

fix comment
---
 .../LearnerAnalyticsDashboard.jsx               |  1 +
 openedx/features/learner_analytics/views.py     | 17 +++++++++++++----
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/lms/static/js/learner_analytics_dashboard/LearnerAnalyticsDashboard.jsx b/lms/static/js/learner_analytics_dashboard/LearnerAnalyticsDashboard.jsx
index baea0e0cf1..08f5632cc1 100644
--- a/lms/static/js/learner_analytics_dashboard/LearnerAnalyticsDashboard.jsx
+++ b/lms/static/js/learner_analytics_dashboard/LearnerAnalyticsDashboard.jsx
@@ -94,6 +94,7 @@ export function LearnerAnalyticsDashboard(props) {
           }
 
           <h3 className="section-heading">Graded Assignments</h3>
+          {/* TODO: LEARNER-3854: If implementing Learner Analytics, rename to graded-assignments-wrapper. */}
           <div className="graded-assessments-wrapper">
             <GradeTable assignmentTypes={assignmentTypes}
                         grades={grades}
diff --git a/openedx/features/learner_analytics/views.py b/openedx/features/learner_analytics/views.py
index 2d87c1bb22..49dca9e64c 100644
--- a/openedx/features/learner_analytics/views.py
+++ b/openedx/features/learner_analytics/views.py
@@ -96,6 +96,17 @@ class LearnerAnalyticsView(View):
             schedule_data = self.get_assignments_with_due_date(request, course_key)
             (grade_data, schedule_data) = self.sort_grade_and_schedule_data(grade_data, schedule_data)
 
+            # TODO: LEARNER-3854: Fix hacked defaults with real error handling if implementing Learner Analytics.
+            try:
+                weekly_active_users = self.get_weekly_course_activity_count(course_key)
+                week_streak = self.consecutive_weeks_of_course_activity_for_user(
+                    request.user.username, course_key
+                )
+            except Exception as e:
+                logging.exception(e)
+                weekly_active_users = 134
+                week_streak = 1
+
             context.update({
                 'grading_policy': grading_policy,
                 'assignment_grades': grade_data,
@@ -103,12 +114,10 @@ class LearnerAnalyticsView(View):
                 'assignment_schedule': schedule_data,
                 'profile_image_urls': get_profile_image_urls_for_user(request.user, request),
                 'discussion_info': self.get_discussion_data(request, course_key),
-                'weekly_active_users': self.get_weekly_course_activity_count(course_key),
-                'week_streak': self.consecutive_weeks_of_course_activity_for_user(
-                    request.user.username, course_key
-                ),
                 'passing_grade': math.ceil(100 * course.lowest_passing_grade),
                 'percent_grade': math.ceil(100 * percent_grade),
+                'weekly_active_users': weekly_active_users,
+                'week_streak': week_streak,
             })
 
         return render_to_response('learner_analytics/dashboard.html', context)