Merge pull request #219 from MITx/mm-course-page-ordering

Order courses according to univesity on courses page.

lms/djangoapps/courseware/views.py

 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):

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:
+      <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>