Article Cleanup

Author: amoshaviv

app/controllers/articles.js

@@ -88,7 +88,7 @@ exports.list = function(req, res) {
  * Article middleware
  */
 exports.articleByID = function(req, res, next, id) {
-	Article.load(id, function(err, article) {
+	Article.findById(id).populate('user', 'displayName').exec(function(err, article) {
 		if (err) return next(err);
 		if (!article) return next(new Error('Failed to load article ' + id));
 		req.article = article;

app/models/article.js

@@ -17,7 +17,8 @@ var ArticleSchema = new Schema({
 	title: {
 		type: String,
 		default: '',
-		trim: true
+		trim: true,
+		required: 'Title cannot be blank'
 	},
 	content: {
 		type: String,
@@ -30,22 +31,4 @@ var ArticleSchema = new Schema({
 	}
 });
 
-/**
- * Validations
- */
-ArticleSchema.path('title').validate(function(title) {
-	return title.length;
-}, 'Title cannot be blank');
-
-/**
- * Statics
- */
-ArticleSchema.statics = {
-	load: function(id, cb) {
-		this.findOne({
-			_id: id
-		}).populate('user', 'displayName').exec(cb);
-	}
-};
-
 mongoose.model('Article', ArticleSchema);