From 7428b2fe9411dc0ff263652e6a6e1dd3254d3945 Mon Sep 17 00:00:00 2001 From: Ahsan Ulhaq Date: Mon, 26 Mar 2018 17:54:16 +0500 Subject: [PATCH 1/2] Discovery for pdf cert generation LEARNER-4520 --- lms/djangoapps/certificates/api.py | 4 ++++ lms/djangoapps/certificates/queue.py | 8 ++++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/certificates/api.py b/lms/djangoapps/certificates/api.py index da0dea0123..3da013851a 100644 --- a/lms/djangoapps/certificates/api.py +++ b/lms/djangoapps/certificates/api.py @@ -207,6 +207,10 @@ def regenerate_user_certificates(student, course_key, course=None, course = modulestore().get_course(course_key, depth=0) generate_pdf = not has_html_certificates_enabled(course) + log.info( + "Started regenerating certificates for user %s in course %s with generate_pdf status: %s", + student.username, unicode(course_key), generate_pdf + ) return xqueue.regen_cert( student, diff --git a/lms/djangoapps/certificates/queue.py b/lms/djangoapps/certificates/queue.py index 56702c7a6f..c6febba19c 100644 --- a/lms/djangoapps/certificates/queue.py +++ b/lms/djangoapps/certificates/queue.py @@ -298,14 +298,16 @@ class XQueueCertInterface(object): u"Certificate generated for student %s in the course: %s with template: %s. " u"given template: %s, " u"user is verified: %s, " - u"mode is verified: %s" + u"mode is verified: %s," + u"generate_pdf is: %s" ), student.username, unicode(course_id), template_pdf, template_file, user_is_verified, - mode_is_verified + mode_is_verified, + generate_pdf ) cert, created = GeneratedCertificate.objects.get_or_create(user=student, course_id=course_id) @@ -442,6 +444,8 @@ class XQueueCertInterface(object): cert.verify_uuid = uuid4().hex cert.save() + logging.info(u'certificate generated for user: %s with generate_pdf status: %s', + student.id, generate_pdf) if generate_pdf: try: From ea9a679c27253a4d3be09f833aec902560e5301f Mon Sep 17 00:00:00 2001 From: Ahsan Ulhaq Date: Tue, 27 Mar 2018 16:59:07 +0500 Subject: [PATCH 2/2] small nit --- lms/djangoapps/certificates/queue.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lms/djangoapps/certificates/queue.py b/lms/djangoapps/certificates/queue.py index c6febba19c..66b8e35c87 100644 --- a/lms/djangoapps/certificates/queue.py +++ b/lms/djangoapps/certificates/queue.py @@ -445,7 +445,7 @@ class XQueueCertInterface(object): cert.save() logging.info(u'certificate generated for user: %s with generate_pdf status: %s', - student.id, generate_pdf) + student.username, generate_pdf) if generate_pdf: try: