Merge pull request #1061 from MITx/feature/jarv/cert-status-fix

Missing course_id filter for certificate summary
2012-11-19 12:44:25 -08:00
parent a9e2b77d6b 9c9dd05666
commit 35528d9c22
1 changed files with 3 additions and 2 deletions
--- a/lms/djangoapps/certificates/management/commands/gen_cert_report.py
+++ b/lms/djangoapps/certificates/management/commands/gen_cert_report.py
@@ -70,8 +70,9 @@ class Command(BaseCommand):
            cert_data[course_id] = {'enrolled': enrolled_students.count()}
            cert_data[course_id].update({'unavailable': unavailable_count})

-            tallies = GeneratedCertificate.objects.values(
-                    'status').annotate(dcount=Count('status'))
+            tallies = GeneratedCertificate.objects.filter(
+                        course_id__exact=course_id).values('status').annotate(
+                            dcount=Count('status'))
            cert_data[course_id].update(
                    {status['status']: status['dcount']
                                        for status in tallies})