diff --git a/packages/component-helper-service/src/services/Collection.js b/packages/component-helper-service/src/services/Collection.js index 5b05427bc524f8766deb2ddf60b88be7d522d463..f39e84990cf553b1585dc421138ff9dae86852e9 100644 --- a/packages/component-helper-service/src/services/Collection.js +++ b/packages/component-helper-service/src/services/Collection.js @@ -1,3 +1,5 @@ +const last = require('lodash/last') + class Collection { constructor({ collection = {} }) { this.collection = collection @@ -73,8 +75,8 @@ class Collection { await this.updateStatus({ newStatus: status }) } - async updateStatusByNumberOfReviewers() { - const reviewerInvitations = this.collection.invitations.filter( + async updateStatusByNumberOfReviewers({ invitations }) { + const reviewerInvitations = invitations.filter( inv => inv.role === 'reviewer', ) if (reviewerInvitations.length === 0) diff --git a/packages/component-invite/src/routes/fragmentsInvitations/patch.js b/packages/component-invite/src/routes/fragmentsInvitations/patch.js index 7ecb11d6b286fd76b69d892e5222c71d424296c3..6cb2ca49d9cd4aa8dcbac67b254963ad2edb3ee9 100644 --- a/packages/component-invite/src/routes/fragmentsInvitations/patch.js +++ b/packages/component-invite/src/routes/fragmentsInvitations/patch.js @@ -78,7 +78,10 @@ module.exports = models => async (req, res) => { if (reason) invitation.reason = reason await fragment.save() - collectionHelper.updateStatusByNumberOfReviewers() + collectionHelper.updateStatusByNumberOfReviewers({ + invitations: fragment.invitations, + }) + emailHelper.setupReviewerDecisionEmail({ agree: false, user,