From 1f595ef4016bc4a1512b752aa23be72f44e49c87 Mon Sep 17 00:00:00 2001
From: Matthew Mongeau <halogenandtoast@gmail.com>
Date: Thu, 19 Jul 2012 10:26:39 -0400
Subject: [PATCH] Order courses according to univesity on courses page.

---
 lms/djangoapps/courseware/views.py | 10 ++++++++--
 lms/templates/courses.html         | 18 +++++++++++++++---
 2 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py
index 7b05cb0e9b..521e434f6a 100644
--- a/lms/djangoapps/courseware/views.py
+++ b/lms/djangoapps/courseware/views.py
@@ -1,5 +1,6 @@
 import logging
 import urllib
+import itertools
 
 from django.conf import settings
 from django.core.context_processors import csrf
@@ -54,8 +55,13 @@ def format_url_params(params):
 @cache_if_anonymous
 def courses(request):
     # TODO: Clean up how 'error' is done.
-    context = {'courses': modulestore().get_courses()}
-    return render_to_response("courses.html", context)
+    courses = modulestore().get_courses()
+    universities = dict()
+    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 })
 
 @cache_control(no_cache=True, no_store=True, must_revalidate=True)
 def gradebook(request, course_id):
diff --git a/lms/templates/courses.html b/lms/templates/courses.html
index 3179f67753..d7302565d9 100644
--- a/lms/templates/courses.html
+++ b/lms/templates/courses.html
@@ -18,9 +18,21 @@
     ## I'm removing this for now since we aren't using it for the fall.
     ## <%include file="course_filter.html" />
     <section class="courses">
-      %for course in courses:
-        <%include file="course.html" args="course=course" />
-      %endfor
+      <section class='university-column'>
+        %for course in universities['MITx']:
+          <%include file="course.html" args="course=course" />
+        %endfor
+      </section>
+      <section class='university-column'>
+        %for course in universities['HarvardX']:
+          <%include file="course.html" args="course=course" />
+        %endfor
+      </section>
+      <section class='university-column'>
+        %for course in universities['BerkeleyX']:
+          <%include file="course.html" args="course=course" />
+        %endfor
+      </section>
     </section>
   </section>
 </section>