From aedb02543cdaf1bc63b1f114ece6d7f2cc43e2c3 Mon Sep 17 00:00:00 2001 From: Victor Shnayder Date: Fri, 9 Nov 2012 14:00:05 -0500 Subject: [PATCH] always return grade as part of cert status --- lms/djangoapps/certificates/models.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/lms/djangoapps/certificates/models.py b/lms/djangoapps/certificates/models.py index 1434510920..b1db8ef288 100644 --- a/lms/djangoapps/certificates/models.py +++ b/lms/djangoapps/certificates/models.py @@ -93,14 +93,12 @@ def certificate_status_for_student(student, course_id): try: generated_certificate = GeneratedCertificate.objects.get( user=student, course_id=course_id) + d = {'status': generated_certificate.status, + 'grade' : generated_certificate.grade,} if generated_certificate.status == CertificateStatuses.downloadable: - return { - 'status': CertificateStatuses.downloadable, - 'download_url': generated_certificate.download_url, - 'grade': generated_certificate.grade, - } - else: - return {'status': generated_certificate.status} + d['download_url'] = generated_certificate.download_url + + return d except GeneratedCertificate.DoesNotExist: pass return {'status': CertificateStatuses.unavailable}