total-questions added to the attributes

[Manit1]

Fixes #13

[Manit1]

@abhishek97
I used this as the reference

[abhishek97]

This would not work if i don't include all questions along with quiz. Ideally this should at controller level

[Manit1]

@abhishek97 sure !
Doing this for /:id is pretty straightforward but when querying all the quizzes should again iterate through them to include the questions count. Something like

const {rows, count} = this.findAll(...arguments)
rows = rows.map( _ => _.get({plain: true}))
rows.map(async row => {
        const res = await QuizQuestions.find({
          attributes: [Sequelize.fn('count', Sequelize.col('id')), 'questions'],
          where: {
            quizId: row.id
          }
        })
        row['total-questions'] = res.questions
  })

[abhishek97]

Can be done for /quizzes by group by

DB.quizzes.findAll({
  includeIgnoreAttributes: false,
  attributes: [ 'id' ,[Sequelize.fn('count', Sequelize.col('questions->quizQuestions.id')), 'total']],
  include: {
    model: DB.questions,
  },
 where: {
    $in: quizIds
  },
  group: ['quizzes.id']
})

and then map total to the quizzes object

[Manit1]

@abhishek97 Thanks and done! 😄 🚀

[abhishek97]

This isn't mean to be like this

[abhishek97]

these those map calls can be combined into one.