added instrumentation to count: user logins, registrations, correct answers, wrong answers, enrollment
This commit is contained in:
@@ -39,6 +39,8 @@ from collections import namedtuple
|
||||
from courseware.courses import get_courses_by_university
|
||||
from courseware.access import has_access
|
||||
|
||||
from statsd import statsd
|
||||
|
||||
log = logging.getLogger("mitx.student")
|
||||
Article = namedtuple('Article', 'title url author image deck publication publish_date')
|
||||
|
||||
@@ -204,7 +206,7 @@ def change_enrollment(request):
|
||||
return {'success': False,
|
||||
'error': 'enrollment in {} not allowed at this time'
|
||||
.format(course.display_name)}
|
||||
|
||||
statsd.increment("user.enrollment_in_" + str(course_id))
|
||||
enrollment, created = CourseEnrollment.objects.get_or_create(user=user, course_id=course.id)
|
||||
return {'success': True}
|
||||
|
||||
@@ -212,6 +214,7 @@ def change_enrollment(request):
|
||||
try:
|
||||
enrollment = CourseEnrollment.objects.get(user=user, course_id=course_id)
|
||||
enrollment.delete()
|
||||
statsd.increment("user.unenrollment_in_" + str(course_id))
|
||||
return {'success': True}
|
||||
except CourseEnrollment.DoesNotExist:
|
||||
return {'success': False, 'error': 'You are not enrolled for this course.'}
|
||||
@@ -260,7 +263,7 @@ def login_user(request, error=""):
|
||||
log.info("Login success - {0} ({1})".format(username, email))
|
||||
|
||||
try_change_enrollment(request)
|
||||
|
||||
statsd.increment("user.successful_login")
|
||||
return HttpResponse(json.dumps({'success': True}))
|
||||
|
||||
log.warning("Login failed - Account not active for user {0}, resending activation".format(username))
|
||||
@@ -466,7 +469,7 @@ def create_account(request, post_override=None):
|
||||
log.debug('bypassing activation email')
|
||||
login_user.is_active = True
|
||||
login_user.save()
|
||||
|
||||
statsd.increment("user.account_created")
|
||||
js = {'success': True}
|
||||
return HttpResponse(json.dumps(js), mimetype="application/json")
|
||||
|
||||
|
||||
@@ -2,7 +2,7 @@
|
||||
# class used to store graded responses to CAPA questions
|
||||
#
|
||||
# Used by responsetypes and capa_problem
|
||||
|
||||
from statsd import statsd
|
||||
|
||||
class CorrectMap(object):
|
||||
"""
|
||||
@@ -53,6 +53,11 @@ class CorrectMap(object):
|
||||
'hintmode': hintmode,
|
||||
'queuestate': queuestate,
|
||||
}
|
||||
|
||||
if correctness=="correct":
|
||||
statsd.increment("user.correct_answer")
|
||||
else:
|
||||
statsd.increment("user.incorrect_answer)
|
||||
|
||||
def __repr__(self):
|
||||
return repr(self.cmap)
|
||||
|
||||
Reference in New Issue
Block a user