From d39f7ac2fb0346ff0268d5ff07905f361d615934 Mon Sep 17 00:00:00 2001
From: chapel <jacob.chapel@gmail.com>
Date: Sat, 12 Nov 2011 16:17:19 -0800
Subject: [PATCH] Add course list for school and sanitize course

---
 app.js    | 67 +++++++++++++++++++++++++++----------------------------
 models.js | 14 ++++++++++++
 2 files changed, 47 insertions(+), 34 deletions(-)

diff --git a/app.js b/app.js
index 601dda1..34b4c75 100644
--- a/app.js
+++ b/app.js
@@ -493,40 +493,9 @@ app.get( '/schools', checkAjax, loadUser, function( req, res ) {
     if( schools ) {
       // If schools are found, loop through them gathering any courses that are
       // associated with them and then render the page with that information.
-      async.forEach(
-        schools,
-        function( school, callback ) {
-          // Check if user is authorized with each school
-          school.authorize( user, function( authorized ) {
-            // This is used to display interface elements for those users
-            // that are are allowed to see them, for instance a 'New Course' button.
-            var sanitizedSchool = school.sanitized;
-            sanitizedSchool.authorized = authorized;
-            // Find all courses for school by it's id and sort by name
-            Course.find( { 'school' : school._id } ).sort( 'name', '1' ).run( function( err, courses ) {
-              // If any courses are found, set them to the appropriate school, otherwise
-              // leave empty.
-              if( courses.length > 0 ) {
-                sanitizedSchool.courses = courses.filter(function(course) {
-                  if (!course.deleted) return course;
-                });
-              } else {
-                sanitizedSchool.courses = [];
-              }
-              schoolList.push(sanitizedSchool);
-              // This tells async (the module) that each iteration of forEach is
-              // done and will continue to call the rest until they have all been
-              // completed, at which time the last function below will be called.
-              callback();
-            });
-          });
-        },
-        // After all schools and courses have been found, render them
-        function( err ) {
-          //res.render( 'schools', { 'schools' : schools } );
-          res.json({ 'schools' : schoolList });
-        }
-      );
+      res.json({ 'schools' : schools.map(function(school) {
+        return school.sanitized;
+      })})
     } else {
       // If no schools have been found, display none
       //res.render( 'schools', { 'schools' : [] } );
@@ -535,6 +504,36 @@ app.get( '/schools', checkAjax, loadUser, function( req, res ) {
   });
 });
 
+app.get( '/school/:id', checkAjax, loadUser, loadSchool, function( req, res ) {
+  var school = req.school;
+  var user = req.user;
+
+  school.authorize( user, function( authorized ) {
+    // This is used to display interface elements for those users
+    // that are are allowed to see th)m, for instance a 'New Course' button.
+    var sanitizedSchool = school.sanitized;
+    sanitizedSchool.authorized = authorized;
+    // Find all courses for school by it's id and sort by name
+    Course.find( { 'school' : school._id } ).sort( 'name', '1' ).run( function( err, courses ) {
+      // If any courses are found, set them to the appropriate school, otherwise
+      // leave empty.
+      if( courses.length > 0 ) {
+        sanitizedSchool.courses = courses.filter(function(course) {
+          if (!course.deleted) return course;
+        }).map(function(course) {
+          return course.sanitized;
+        });
+      } else {
+        sanitizedSchool.courses = [];
+      }
+      // This tells async (the module) that each iteration of forEach is
+      // done and will continue to call the rest until they have all been
+      // completed, at which time the last function below will be called.
+      res.json({ 'school': sanitizedSchool })
+    });
+  });
+});
+
 // New course page
 // Displays form to create new course
 // Private, requires user to be authorized
diff --git a/models.js b/models.js
index 87d5c72..f9bea51 100644
--- a/models.js
+++ b/models.js
@@ -134,6 +134,7 @@ var SchoolSchema = new Schema( {
 
 SchoolSchema.virtual( 'sanitized' ).get(function() {
   var school = {
+    _id: this._id,
     name: this.name,
     description: this.description,
     url: this.url
@@ -169,6 +170,19 @@ var CourseSchema = new Schema( {
 	users				: Array
 });
 
+CourseSchema.virtual( 'sanitized' ).get(function() {
+  var course = {
+    _id: this._id,
+    name: this.name,
+    number: this.number,
+    description: this.description,
+    subject: this.subject,
+    department: this.department
+  }
+
+  return course;
+});
+
 CourseSchema.virtual( 'displayName' )
 	.get( function() {
 		if( this.number ) {
-- 
2.25.1