Commit 44141b6a by John Jarvis

pep8 cleanup

parent 491027a8
...@@ -23,26 +23,24 @@ class Command(BaseCommand): ...@@ -23,26 +23,24 @@ class Command(BaseCommand):
option_list = BaseCommand.option_list + ( option_list = BaseCommand.option_list + (
make_option('-n', '--noop', make_option('-n', '--noop',
action='store_true', action='store_true',
dest='noop', dest='noop',
default=False, default=False,
help="Don't add certificate requests to the queue"), help="Don't add certificate requests to the queue"),
make_option('-c', '--course', make_option('-c', '--course',
metavar='COURSE_ID', metavar='COURSE_ID',
dest='course', dest='course',
default=False, default=False,
help='Grade and generate certificates for a specific course'), help='Grade and generate certificates '
'for a specific course'),
) )
def handle(self, *args, **options): def handle(self, *args, **options):
# Will only generate a certificate if the current # Will only generate a certificate if the current
# status is in this state # status is in this state
VALID_STATUSES = [ VALID_STATUSES = [CertificateStatuses.unavailable]
CertificateStatuses.unavailable
]
# Print update after this many students # Print update after this many students
...@@ -54,8 +52,8 @@ class Command(BaseCommand): ...@@ -54,8 +52,8 @@ class Command(BaseCommand):
# Find all courses that have ended # Find all courses that have ended
ended_courses = [] ended_courses = []
for course_id in [course # all courses in COURSE_LISTINGS for course_id in [course # all courses in COURSE_LISTINGS
for sub in settings.COURSE_LISTINGS for sub in settings.COURSE_LISTINGS
for course in settings.COURSE_LISTINGS[sub]]: for course in settings.COURSE_LISTINGS[sub]]:
course_loc = CourseDescriptor.id_to_location(course_id) course_loc = CourseDescriptor.id_to_location(course_id)
course = modulestore().get_instance(course_id, course_loc) course = modulestore().get_instance(course_id, course_loc)
if course.has_ended(): if course.has_ended():
...@@ -64,8 +62,8 @@ class Command(BaseCommand): ...@@ -64,8 +62,8 @@ class Command(BaseCommand):
for course_id in ended_courses: for course_id in ended_courses:
print "Fetching enrolled students for {0}".format(course_id) print "Fetching enrolled students for {0}".format(course_id)
enrolled_students = User.objects.filter( enrolled_students = User.objects.filter(
courseenrollment__course_id=course_id).prefetch_related( courseenrollment__course_id=course_id).prefetch_related(
"groups").order_by('username') "groups").order_by('username')
xq = XQueueCertInterface() xq = XQueueCertInterface()
total = enrolled_students.count() total = enrolled_students.count()
count = 0 count = 0
...@@ -81,11 +79,11 @@ class Command(BaseCommand): ...@@ -81,11 +79,11 @@ class Command(BaseCommand):
hours, remainder = divmod(timeleft.seconds, 3600) hours, remainder = divmod(timeleft.seconds, 3600)
minutes, seconds = divmod(remainder, 60) minutes, seconds = divmod(remainder, 60)
print "{0}/{1} completed ~{2:02}:{3:02}m remaining".format( print "{0}/{1} completed ~{2:02}:{3:02}m remaining".format(
count, total, hours, minutes) count, total, hours, minutes)
start = datetime.datetime.now() start = datetime.datetime.now()
if certificate_status_for_student( if certificate_status_for_student(
student, course_id)['status'] in VALID_STATUSES: student, course_id)['status'] in VALID_STATUSES:
if not options['noop']: if not options['noop']:
# Add the certificate request to the queue # Add the certificate request to the queue
ret = xq.add_cert(student, course_id) ret = xq.add_cert(student, course_id)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment