Feedback from code review.

This commit is contained in:
cahrens
2013-04-02 14:10:56 -04:00
parent ceababbf6d
commit 42fbc16006
3 changed files with 3 additions and 3 deletions

View File

@@ -544,7 +544,7 @@ class CourseDescriptor(CourseFields, SequenceDescriptor):
except (ValueError, AttributeError):
start = time_to_datetime(self.start)
now = time_to_datetime(time.gmtime())
now = datetime.utcnow()
return announcement, start, now

View File

@@ -11,7 +11,7 @@
<li class="${'active' if 'active' in section and section['active'] else ''} ${'graded' if 'graded' in section and section['graded'] else ''}">
<a href="${reverse('courseware_section', args=[course_id, chapter['url_name'], section['url_name']])}">
<p>${section['display_name']}</p>
<p class="subtitle">${section['format']} ${"due " + get_default_time_display(section['due']) if 'due' in section and section['due'] else ''}</p>
<p class="subtitle">${section['format']} ${"due " + get_default_time_display(section['due']) if section.get('due') is not None else ''}</p>
</a>
</li>
% endfor

View File

@@ -62,7 +62,7 @@ ${progress_graph.body(grade_summary, course.grade_cutoffs, "grade-detail-graph",
<p>
${section['format']}
%if 'due' in section and section['due']:
%if section.get('due') is not None:
<em>
due ${get_default_time_display(section['due'])}
</em>