link to author of article in article show page

controllers/articles.js

@@ -1,7 +1,7 @@
 var express = require('express');
 var router = express.Router();
 var Article = require('../models/articles.js');
-var Author = require('../models/authors.js');
+var Author = require('../models/authors.js')
 
 router.get('/', function(req, res){
 	Article.find({}, function(err, foundArticles){
@@ -12,55 +12,71 @@ router.get('/', function(req, res){
 });
 
 router.get('/new', function(req, res){
-	res.render('articles/new.ejs');
+	Author.find({}, function(err, allAuthors){
+		res.render('articles/new.ejs', {
+			authors:allAuthors
+		});
+	});
 });
 
 router.post('/', function(req, res){
+	Author.findById(req.body.authorId, function(err, foundAuthor){
 		Article.create(req.body, function(err, createdArticle){
+			foundAuthor.articles.push(createdArticle);
+			foundAuthor.save(function(){
 				res.redirect('/articles');
 			})
 		});
+	});
+});
 
 router.get('/:id', function(req,res){
 	Article.findById(req.params.id, function(err, foundArticle){
+		Author.findOne({'articles._id':req.params.id}, function(err, foundAuthor){
 			res.render('articles/show.ejs', {
+				author: foundAuthor,
 				article: foundArticle
 			});
 		});
 	});
+});
 
 router.delete('/:id', function(req, res){
-	Author.find({'articles._id':req.params.id}, function(err, foundAuthors){
-		for(var i = 0; i < foundAuthors.length; i++){
-			foundAuthors[i].articles.id(req.params.id).remove();
-			foundAuthors[i].save();
-		}
+	Author.findOne({'articles._id':req.params.id}, function(err, author){
+		author.articles.id(req.params.id).remove();
+		author.save();
+	});
 	Article.findByIdAndRemove(req.params.id, function(){
 		res.redirect('/articles');
-		})
 	});
 });
 
 router.get('/:id/edit', function(req, res){
+	Author.find({}, function(err, allAuthors){
 		Article.findById(req.params.id, function(err, foundArticle){
 			res.render('articles/edit.ejs', {
-			article: foundArticle
+				article: foundArticle,
+				authors: allAuthors
+			});
 		});
 	});
 });
 
 router.put('/:id', function(req, res){
+	Author.findOne({'articles._id':req.params.id}, function(err, previousAuthor){
+		previousAuthor.articles.id(req.params.id).remove();
+		previousAuthor.save(function(){
+			Author.findById(req.body.authorId, function(err, newAuthor){
 				Article.findByIdAndUpdate(req.params.id, req.body, {new:true}, function(err, updatedArticle){
-		Author.find({'articles._id':req.params.id}, function(err, foundAuthors){
-			for(var i = 0; i < foundAuthors.length; i++){
-				foundAuthors[i].articles.id(req.params.id).remove();
-				foundAuthors[i].articles.push(updatedArticle);
-				foundAuthors[i].save();
-			}
+					newAuthor.articles.push(updatedArticle);
+					newAuthor.save(function(){
 						res.redirect('/articles');
 					});
 				});
 			});
+		});
+	});
+});
 
 
 module.exports = router;

controllers/authors.js

@@ -1,6 +1,5 @@
 var express = require('express');
 var Author = require('../models/authors.js');
-var Article = require('../models/articles.js');
 var router = express.Router();
 
 router.get('/', function(req, res){
@@ -42,33 +41,10 @@ router.put('/:id', function(req, res){
 });
 
 router.get('/:id/edit', function(req, res){
-	Article.find({}, function(err, foundArticles){
 	Author.findById(req.params.id, function(err, foundAuthor){
 		res.render('authors/edit.ejs', {
-				author: foundAuthor,
-				articles: foundArticles
-			});
-		});
-	});
-});
-
-router.post('/:id/articles', function(req, res){
-	Article.findById(req.body.articleId, function(err, foundArticle){
-		Author.findById(req.params.id, function(err, foundAuthor){
-			foundAuthor.articles.push(foundArticle);
-			foundAuthor.save(function(){
-				res.redirect('/authors/' + req.params.id + '/edit');
-			})
-		});
-	});
+			author: foundAuthor
 		});
-
-router.delete('/:authorId/articles/:articleId', function(req, res){
-	Author.findById(req.params.authorId, function(err, foundAuthor){
-		foundAuthor.articles.id(req.params.articleId).remove();
-		foundAuthor.save(function(){
-			res.redirect('/authors/' + req.params.authorId + '/edit');
-		})
 	});
 });
 

views/articles/edit.ejs

@@ -22,9 +22,21 @@
 			<h2>Article Attributes:</h2>
 			<form action="/articles/<%=article._id%>?_method=PUT" method="post">
 				<input type="text" name="title" value="<%=article.title%>"/><br/>
-				<textarea name="body">
-					<%=article.body%>
-				</textarea><br/>
+				<textarea name="body"><%=article.body%></textarea><br/>
+				Choose and author:
+				<select name="authorId">
+					<% for(var currentAuthorIndex = 0; currentAuthorIndex < authors.length; currentAuthorIndex++) { %>
+						<%
+							var isAuthor = false;
+							for(var currentArticleIndex = 0; currentArticleIndex < authors[currentAuthorIndex].articles.length; currentArticleIndex++){
+								if(authors[currentAuthorIndex].articles[currentArticleIndex]._id.toString() == article._id.toString()){
+									isAuthor = true;
+								}
+							}
+						%>
+						<option value="<%=authors[currentAuthorIndex]._id %>" <%=(isAuthor)?'selected':''%>><%=authors[currentAuthorIndex].name %></option>
+					<% } %>
+				</select><br/>
 				<input type="submit" value="Update Article"/>
 			</form>
 		</main>

views/articles/new.ejs

@@ -22,6 +22,12 @@
 			<form action="/articles" method="post">
 				<input type="text" name="title" /><br/>
 				<textarea name="body"></textarea><br/>
+				Choose and author:
+				<select name="authorId">
+					<% for(var i = 0; i < authors.length; i++) { %>
+						<option value="<%=authors[i]._id %>"><%=authors[i].name %></option>
+					<% } %>
+				</select><br/>
 				<input type="submit" value="Publish Article"/>
 			</form>
 		</main>

views/articles/show.ejs

@@ -6,7 +6,7 @@
 	</head>
 	<body>
 		<header>
-			<h1><%=article.title%></h1>
+			<h1><%=article.title%> by <a href="/authors/<%=author._id%>"><%=author.name%></a></h1>
 			<nav>
 				<ul>
 					<li>

views/authors/edit.ejs

@@ -13,46 +13,17 @@
 						<a href="/">Home</a>
 					</li>
 					<li>
-						<a href="/authors/<%=author._id%>"><%=author.name%>'s Show Page</a>
+						<a href="/authors">Authors Index</a>
 					</li>
 				</ul>
 			</nav>
 		</header>
 		<main>
-			<section>
 			<h2>Author Attributes:</h2>
 			<form action="/authors/<%=author._id%>?_method=PUT" method="post">
 				<input type="text" name="name" value="<%=author.name%>"/><br/>
 				<input type="submit" value="Update Author"/>
 			</form>
-			</section>
-			<section>
-				<h2>Articles Written By This Author</h2>
-				<ul>
-					<% for(var i = 0; i < author.articles.length; i++) { %>
-						<li>
-							<%=author.articles[i].title%>
-							<form action="/authors/<%=author._id%>/articles/<%=author.articles[i]._id%>?_method=DELETE" method="post">
-								<input type="submit" value="Remove This Author's Attribution from This Article" />
-							</form>
-						</li>
-					<% } %>
-				</ul>
-			</section>
-			<section>
-				<h2>Articles Not Written By This Author</h2>
-				<ul>
-					<% for(var i = 0; i < articles.length; i++) { %>
-						<li>
-							<%=articles[i].title%>
-							<form action="/authors/<%=author._id%>/articles" method="post">
-								<input type="hidden" name="articleId" value="<%=articles[i]._id%>">
-								<input type="submit" value="Attribute Article To This Author" />
-							</form>
-						</li>
-					<% } %>
-				</ul>
-			</section>
 		</main>
 	</body>
 </html>

views/authors/show.ejs

@@ -29,7 +29,7 @@
 				<h2>Articles Written By This Author</h2>
 				<ul>
 					<% for(var i = 0; i < author.articles.length; i++){ %>
-						<li><%=author.articles[i].title%></li>
+						<li><a href="/articles/<%=author.articles[i]._id%>"><%=author.articles[i].title%></a></li>
 					<% } %>
 				</ul>
 			</section>