From 14b59d6434c75a1debbac63b6f1e580ad6d93821 Mon Sep 17 00:00:00 2001 From: Michael Roytman Date: Fri, 13 Jul 2018 10:24:47 -0400 Subject: [PATCH] add condition for existence of course end date to graded_only option --- .../api/views/course_validation.py | 24 ++++++++++++------- 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/cms/djangoapps/contentstore/api/views/course_validation.py b/cms/djangoapps/contentstore/api/views/course_validation.py index b0b7ee3532..ef4bcc718f 100644 --- a/cms/djangoapps/contentstore/api/views/course_validation.py +++ b/cms/djangoapps/contentstore/api/views/course_validation.py @@ -139,15 +139,23 @@ class CourseValidationView(DeveloperErrorViewMixin, GenericAPIView): if a.due and a.graded ] - assignments_with_dates_before_start = [ - {'id': unicode(a.location), 'display_name': a.display_name} - for a in assignments_with_dates if a.due < course.start - ] + assignments_with_dates_before_start = ( + [ + {'id': unicode(a.location), 'display_name': a.display_name} + for a in assignments_with_dates if a.due < course.start + ] + if self._has_start_date(course) + else [] + ) - assignments_with_dates_after_end = [ - {'id': unicode(a.location), 'display_name': a.display_name} - for a in assignments_with_dates if a.due > course.end - ] + assignments_with_dates_after_end = ( + [ + {'id': unicode(a.location), 'display_name': a.display_name} + for a in assignments_with_dates if a.due > course.end + ] + if course.end + else [] + ) return dict( total_number=len(assignments),