Merge pull request #3261 from edx/jarv/limit-to-active-users

Only display active users for the certificate report

lms/djangoapps/certificates/management/commands/gen_cert_report.py

@@ -61,8 +61,8 @@ class Command(BaseCommand):
             # find students who are enrolled
             print "Looking up certificate states for {0}".format(course_id)
             enrolled_students = User.objects.filter(
-                    courseenrollment__course_id=course_id).prefetch_related(
-                            "groups").order_by('username')
+                    courseenrollment__course_id=course_id,
+                    courseenrollment__is_active=True)
             cert_data[course_id] = {'enrolled': enrolled_students.count()}
 
             tallies = GeneratedCertificate.objects.filter(

lms/djangoapps/certificates/management/commands/ungenerated_certs.py

@@ -84,8 +84,7 @@ class Command(BaseCommand):
 
             print "Fetching enrolled students for {0}".format(course_id)
             enrolled_students = User.objects.filter(
-                courseenrollment__course_id=course_id).prefetch_related(
-                    "groups").order_by('username')
+                courseenrollment__course_id=course_id)
 
             xq = XQueueCertInterface()
             if options['insecure']: