From edd45dce824fe0cac07a3274f18de91f029f330d Mon Sep 17 00:00:00 2001
From: Andrei Cioromila <andrei.cioromila@thinslices.com>
Date: Fri, 19 Oct 2018 16:46:19 +0300
Subject: [PATCH] refactor(notifications): extract author notification when eic
 makes a decision

---
 .../notifications/helpers.js                  |  4 +-
 .../notifications/notification.js             | 99 +++++++++++++++++--
 .../routes/fragmentsRecommendations/post.js   |  6 +-
 3 files changed, 95 insertions(+), 14 deletions(-)

diff --git a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/helpers.js b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/helpers.js
index 3405bc686..87301e860 100644
--- a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/helpers.js
+++ b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/helpers.js
@@ -107,14 +107,14 @@ module.exports = {
     comments,
     emailType,
     titleText,
-    reviewerLastName,
+    targetUserName,
   }) => {
     const { html, text } = email.getNotificationBody({
       emailBodyProps: getEmailCopy({
         emailType,
         titleText,
         comments,
-        targetUserName: reviewerLastName,
+        targetUserName,
       }),
     })
     email.sendEmail({ html, text })
diff --git a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/notification.js b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/notification.js
index 1260ae7dc..aa684d544 100644
--- a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/notification.js
+++ b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/notifications/notification.js
@@ -1,5 +1,5 @@
 const config = require('config')
-const { get } = require('lodash')
+const { get, isEmpty } = require('lodash')
 const Email = require('@pubsweet/component-email-templating')
 
 const unsubscribeSlug = config.get('unsubscribe.url')
@@ -66,7 +66,7 @@ class Notification {
     helpers.sendHandlingEditorEmail({
       email,
       titleText,
-      reviewerLastName,
+      targetUserName: reviewerLastName,
       emailType: 'he-review-submitted',
     })
   }
@@ -104,13 +104,13 @@ class Notification {
     helpers.sendHandlingEditorEmail({
       email,
       titleText,
-      reviewerLastName,
+      targetUserName: eicName,
       emailType: 'he-review-submitted',
     })
   }
 
-  notifyEAWhenEiCMakesFinalDecision() {
-    const { eicName, titleText } = this.props
+  async notifyEAWhenEiCMakesFinalDecision() {
+    const { eicName, titleText } = await this.getNotificationProperties()
     const subjectBaseText = `${this.collection.customId}: Manuscript`
 
     const email = new Email({
@@ -135,8 +135,8 @@ class Notification {
     email.sendEmail({ html, text })
   }
 
-  notifyEAWhenEiCRequestsEQAApproval() {
-    const { eicName } = this.props
+  async notifyEAWhenEiCRequestsEQAApproval() {
+    const { eicName } = await this.getNotificationProperties()
     const subjectBaseText = `${this.collection.customId}: Manuscript`
 
     const email = new Email({
@@ -171,7 +171,75 @@ class Notification {
     email.sendEmail({ html, text })
   }
 
-  notifyAuthorsWhenEiCMakesDecision() {}
+  async notifyAuthorsWhenEiCMakesDecision() {
+    const {
+      eicName,
+      titleText,
+      activeAuthors,
+      recommendation,
+      parsedFragment,
+    } = await this.getNotificationProperties()
+
+    activeAuthors.forEach(async author => {
+      const subjectOpts = {
+        publish: `${this.collection.customId}: Manuscript accepted`,
+        reject: `${this.collection.customId}: Manuscript rejected`,
+      }
+      const subject = subjectOpts[recommendation]
+
+      if (isEmpty(subject)) {
+        throw new Error(`Undefined recommendation: ${recommendation}`)
+      }
+
+      const email = new Email({
+        type: 'user',
+        toUser: {
+          email: author.email,
+          name: author.lastName,
+        },
+        fromEmail: `${eicName} <${staffEmail}>`,
+        content: {
+          signatureName: eicName,
+          ctaText: 'MANUSCRIPT DETAILS',
+          subject,
+          unsubscribeLink: services.createUrl(this.baseUrl, unsubscribeSlug, {
+            id: author.id,
+            token: author.accessTokens.unsubscribe,
+          }),
+          ctaLink: services.createUrl(
+            this.baseUrl,
+            `/projects/${this.collection.id}/versions/${
+              this.fragment.id
+            }/details`,
+          ),
+          signatureJournal: journalName,
+        },
+      })
+
+      const hasPeerReview = !isEmpty(this.collection.handlingEditor)
+      const emailType = helpers.getEmailTypeByRecommendationForAuthors({
+        recommendation,
+        hasPeerReview,
+      })
+      let comments
+      if (hasPeerReview) {
+        comments = helpers.getHEComments({
+          heRecommendation: parsedFragment.heRecommendation,
+        })
+      } else {
+        comments = this.newRecommendation.comments[0].content
+      }
+
+      const { html, text } = email.getNotificationBody({
+        emailBodyProps: getEmailCopy({
+          titleText,
+          emailType,
+          comments,
+        }),
+      })
+      email.sendEmail({ html, text })
+    })
+  }
 
   notifyReviewersWhenEiCMakesDecision() {}
 
@@ -186,7 +254,10 @@ class Notification {
     const parsedFragment = await fragmentHelper.getFragmentData({
       handlingEditor: this.collection.handlingEditor,
     })
-    const { submittingAuthor } = await fragmentHelper.getAuthorData({
+    const {
+      submittingAuthor,
+      activeAuthors,
+    } = await fragmentHelper.getAuthorData({
       UserModel: this.UserModel,
     })
 
@@ -199,7 +270,15 @@ class Notification {
 
     const { recommendation, recommendationType } = this.newRecommendation
 
-    return { recommendation, recommendationType, eicName, titleText }
+    return {
+      recommendation,
+      recommendationType,
+      eicName,
+      titleText,
+      submittingAuthor,
+      activeAuthors,
+      parsedFragment,
+    }
   }
 }
 
diff --git a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/post.js b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/post.js
index f9df5c8b9..31a2723fc 100644
--- a/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/post.js
+++ b/packages/component-manuscript-manager/src/routes/fragmentsRecommendations/post.js
@@ -156,13 +156,15 @@ module.exports = models => async (req, res) => {
       if (
         recommendation === 'publish' &&
         collection.technicalChecks.token &&
-        ['accepted', 'inQA'].includes(this.collection.status)
+        ['accepted', 'inQA'].includes(collection.status)
       ) {
-        if (this.collection.status === 'inQA') {
+        if (collection.status === 'inQA') {
           notification.notifyEAWhenEiCRequestsEQAApproval()
         } else {
           notification.notifyEAWhenEiCMakesFinalDecision()
         }
+
+        notification.notifyAuthorsWhenEiCMakesDecision()
       }
     } else {
       // notifications.sendNotificationsWhenHEMakesRecommendation({
-- 
GitLab