From 17263dcd1b9ac7f9644ea505bb65221c2569392e Mon Sep 17 00:00:00 2001 From: David Ormsbee Date: Thu, 19 Jul 2012 16:19:16 -0400 Subject: [PATCH] Keep courses and front page from erroring when we're missing an org in the data. --- common/djangoapps/student/views.py | 4 ++-- lms/djangoapps/courseware/views.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/common/djangoapps/student/views.py b/common/djangoapps/student/views.py index 61dac7b905..7d521931a2 100644 --- a/common/djangoapps/student/views.py +++ b/common/djangoapps/student/views.py @@ -8,6 +8,7 @@ import uuid import feedparser import urllib import itertools +from collections import defaultdict from django.conf import settings from django.contrib.auth import logout, authenticate, login @@ -72,9 +73,8 @@ def index(request): entry.image = soup.img['src'] if soup.img else None courses = modulestore().get_courses() - universities = dict() + universities = defaultdict(list) for university, group in itertools.groupby(courses, lambda course: course.org): - universities.setdefault(university, []) [universities[university].append(course) for course in group] return render_to_response('index.html', {'universities': universities, 'entries': entries}) diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py index 521e434f6a..c03fcec94f 100644 --- a/lms/djangoapps/courseware/views.py +++ b/lms/djangoapps/courseware/views.py @@ -1,3 +1,4 @@ +from collections import defaultdict import logging import urllib import itertools @@ -56,9 +57,8 @@ def format_url_params(params): def courses(request): # TODO: Clean up how 'error' is done. courses = modulestore().get_courses() - universities = dict() + universities = defaultdict(list) for university, group in itertools.groupby(courses, lambda course: course.org): - universities.setdefault(university, []) [universities[university].append(course) for course in group] return render_to_response("courses.html", { 'universities': universities })