Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

use .get when checking combined notifications to handle missing keys

Browse Source
This commit is contained in:
Jason Bau
2013-09-18 23:47:31 -07:00
parent 0e2d833a2e
commit aa0801907d
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lms/djangoapps/open_ended_grading/open_ended_notifications.py
View File

@@ -152,8 +152,9 @@ def combined_notifications(course, user):
controller_response = controller_qs.check_combined_notifications(course.id, student_id, user_is_staff,
last_time_viewed)
notifications = json.loads(controller_response)
if notifications['success']:
if notifications['staff_needs_to_grade'] or notifications['student_needs_to_peer_grade']:
if notifications.get('success'):
if (notifications.get('staff_needs_to_grade') or
notifications.get('student_needs_to_peer_grade')):
pending_grading = True
except:
#Non catastrophic error, so no real action