[Feature #164489789] pagination support for article ratings

config/config.js

@@ -8,20 +8,23 @@ module.exports = {
     password: process.env.DBPASSWORD,
     database: process.env.DBNAME,
     host: process.env.DBHOST,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    logging: false
   },
   test: {
     username: process.env.DBUSERNAME,
     password: process.env.DBPASSWORD,
     database: process.env.TESTDBNAME,
     host: process.env.DBHOST,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    logging: false
   },
   production: {
     username: process.env.DBUSERNAME,
     password: process.env.DBPASSWORD,
     database: process.env.DBNAME,
     host: process.env.DBHOST,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    logging: false
   }
 };

controllers/article.js

@@ -697,7 +697,6 @@ class Article {
         error: 'Article can not be found.'
       });
     }
-
     const allArticles = await ratingModel.allRatings(UserModel, ArticleModel, slug);
     const { rows } = allArticles;
     if (rows.length === 0) {
@@ -720,5 +719,37 @@ class Article {
       averageRating: objKey(Math.ceil(AvgRate))
     });
   }
+
+  /**
+  * @author Innocent Nkunzi
+  * @param {*} req
+  * @param {*} res
+  * @returns {object} it returns an object of articles
+  */
+  static async articleRatingPagination(req, res) {
+    const pageNumber = parseInt(req.query.page, 10);
+    const limitRatings = parseInt(req.query.limit, 10);
+
+    if (pageNumber <= 0) {
+      return res.status(403).json({
+        error: 'Invalid page number'
+      });
+    }
+    if (limitRatings <= 0) {
+      return res.status(403).json({
+        error: 'Invalid limit'
+      });
+    }
+    const offset = limitRatings * (pageNumber - 1);
+    const listOfRatings = await ratingModel.paginateArticleRatings(limitRatings, offset);
+    if (listOfRatings.length) {
+      res.status(200).json({
+        articles: listOfRatings,
+        ratingCounts: listOfRatings.length
+      });
+    } else {
+      res.status(404).json({ error: 'No article found' });
+    }
+  }
 }
 export default Article;

migrations/20190410060956-create-resetpassword.js

@@ -10,18 +10,6 @@ module.exports = {
     token: {
       type: Sequelize.STRING
     },
-    userId: {
-      type: Sequelize.INTEGER,
-      allowNull: false
-    },
-    articleSlug: {
-      type: Sequelize.STRING,
-      allowNull: false
-    },
-    rating: {
-      type: Sequelize.INTEGER,
-      allowNull: false
-    },
     createdAt: {
       allowNull: false,
       type: Sequelize.DATE

models/article.js

@@ -37,7 +37,6 @@ const ArticleModel = (sequelize, DataTypes) => {
   Article.createArticle = article => Article.create(article);
   Article.getAll = () => Article.findAll();
   Article.getOneArticle = slug => Article.findOne({ where: { slug } });
-<<<<<<< HEAD
   Article.findArticleSlug = (authorid, slug) => Article.findOne({ where: { authorid, slug } });
   Article.deleteArticle = slug => Article.destroy({ where: { id: slug } });
   Article.getAllPages = (limit, offset) => Article.findAll({ limit, offset });
@@ -55,9 +54,6 @@ const ArticleModel = (sequelize, DataTypes) => {
     return data;
   };
   Article.addViewer = id => Article.increment('views', { by: 1, where: { id } });
-=======
-  Article.verifyArticle = slug => Article.findOne({ where: { slug } });
->>>>>>> get user from token
 
   Article.associate = (models) => {
     Article.belongsTo(models.user, {

models/rating.js

@@ -54,11 +54,7 @@ module.exports = (sequelize, DataTypes) => {
     }
     ],
   });
-
-  rating.rateCheck = (rate, article, id) => rating.findOrCreate({ where: { userId: id, articleSlug: article }, defaults: { rating: rate } });
-  rating.addRate = (rate, article, id) => rating.findOrCreate({ where: { userId: id, articleSlug: article }, defaults: { rating: rate } });
-  rating.rateUpdate = (rateId, rate) => rating.update({ rating: rate }, { returning: true, where: { id: rateId } });
-  rating.avgFind = id => rating.findAll({ where: { articleId: id }, attributes: ['articleId', [sequelize.fn('AVG', sequelize.col('rating')), 'avgRating']], group: 'rating.articleId' });
+  rating.paginateArticleRatings = (limit, offset) => rating.findAll({ limit, offset });
   rating.associate = (models) => {
     rating.belongsTo(models.user, { foreignKey: 'userId', onDelete: 'CASCADE' });
     rating.belongsTo(models.article, { foreignKey: 'articleSlug', onDelete: 'CASCADE' });

routes/api/articles.js

@@ -19,6 +19,7 @@ router.post('/:slug/rate/:rate', AuthToken, articleController.rateArticle);
 router.get('/:slug/rates', AuthToken, articleController.fetchArticleRating);
 router.delete('/:slug', AuthToken, errorHandler(articleController.deleteArticle));
 router.put('/:slug', AuthToken, errorHandler(articleController.updateArticle));
+router.get('/rating/articles', errorHandler(articleController.articleRatingPagination));
 router.patch(
   '/:slug/:likeState',
   Strategy.verifyToken,