Merge pull request #71 from andela/bg-fix-update-article-image-bug

fix missing images ptoperty bug in article update
andela · Nov 7, 2019 · e219388 · e219388
2 parents 1cb2e38 + 4dfa2a7
commit e219388
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/src/controllers/articles.controller.js b/src/controllers/articles.controller.js
@@ -128,7 +128,9 @@ class Articles {
             lastname,
             image
           };
-          const rating = await RateService.getArticleRatingStatistic(article.slug);
+          const rating = await RateService.getArticleRatingStatistic(
+            article.slug
+          );
           let claps = await likeService.getAllAClaps(article.slug);
           claps = Object.values(claps)[0];
           const readTime = Helper.calculateReadTime(article.body);
@@ -333,7 +335,10 @@ class Articles {
       return util.send(res);
     }
     if (req.auth.id !== findArticle.authorId) {
-      util.setError(403, 'Sorry you can not DELETE an article that does not belong to you.');
+      util.setError(
+        403,
+        'Sorry you can not DELETE an article that does not belong to you.'
+      );
       return util.send(res);
     }
     await db.destroy({
@@ -361,15 +366,20 @@ class Articles {
       return util.send(res);
     }
     if (req.auth.id !== findArticle.authorId) {
-      util.setError(401, 'Sorry you can not UPDATE an article that does not belong to you.');
+      util.setError(
+        401,
+        'Sorry you can not UPDATE an article that does not belong to you.'
+      );
       return util.send(res);
     }
-    const { title, body, description } = req.body;
+    const {
+      title, body, description, images
+    } = req.body;
     const updatedArticle = await articleService.updateArticle(req.params.slug, {
-      slug: `${slug(title)}-${uniqid()}`,
       title,
       body,
       description,
+      images,
       taglist: req.body.taglist.split(' ')
     });
     util.setSuccess(200, 'Article Updated successfully!', updatedArticle);

diff --git a/src/routes/api/article/article.routes.js b/src/routes/api/article/article.routes.js
@@ -50,7 +50,7 @@ router.delete(
   [auth, confirmEmailAuth],
   articleController.deleteArticle
 );
-router.patch(
+router.put(
   '/:slug',
   [auth, confirmEmailAuth],
   imageUpload.array('images', 10),