update an article updates articles in author’s array

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,70 +12,54 @@ router.get('/', function(req, res){
 });
 
 router.get('/new', function(req, res){
-	Author.find({}, function(err, allAuthors){
+	res.render('articles/new.ejs');
-		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.findOne({'articles._id':req.params.id}, function(err, author){
+	Author.find({'articles._id':req.params.id}, function(err, foundAuthors){
-		author.articles.id(req.params.id).remove();
+		for(var i = 0; i < foundAuthors.length; i++){
-		author.save();
+			foundAuthors[i].articles.id(req.params.id).remove();
-	});
+			foundAuthors[i].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){
-					newAuthor.articles.push(updatedArticle);
+		Author.find({'articles._id':req.params.id}, function(err, foundAuthors){
-					newAuthor.save(function(){
+			for(var i = 0; i < foundAuthors.length; i++){
+				foundAuthors[i].articles.id(req.params.id).remove();
+				foundAuthors[i].articles.push(updatedArticle);
+				foundAuthors[i].save();
+			}
 			res.redirect('/articles');
 		});
 	});
-			});
-		});
-	});
 });
 
 

controllers/authors.js

@@ -1,5 +1,6 @@
 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){
@@ -41,10 +42,33 @@ 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
+				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');
+			})
+		});
 	});
+});
+
+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,21 +22,9 @@
 			<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">
-				Choose and author:
+					<%=article.body%>
-				<select name="authorId">
+				</textarea><br/>
-					<% 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,12 +22,6 @@
 			<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%> by <a href="/authors/<%=author._id%>"><%=author.name%></a></h1>
+			<h1><%=article.title%></h1>
 			<nav>
 				<ul>
 					<li>

views/authors/edit.ejs

@@ -13,17 +13,46 @@
 						<a href="/">Home</a>
 					</li>
 					<li>
-						<a href="/authors">Authors Index</a>
+						<a href="/authors/<%=author._id%>"><%=author.name%>'s Show Page</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

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