From c82c9395a4e9d0ab883dd7830804db3402c4fe82 Mon Sep 17 00:00:00 2001
From: ivanm <ivanm@verdnatura.es>
Date: Thu, 19 Sep 2024 12:49:45 +0200
Subject: [PATCH] feat: refs #7343 Requested changes

---
 .../back/methods/route/driverRouteEmail.js    | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/modules/route/back/methods/route/driverRouteEmail.js b/modules/route/back/methods/route/driverRouteEmail.js
index 4675289893..62147db87d 100644
--- a/modules/route/back/methods/route/driverRouteEmail.js
+++ b/modules/route/back/methods/route/driverRouteEmail.js
@@ -39,8 +39,8 @@ module.exports = Self => {
         const {reportMail} = agencyMode();
         let user;
         let account;
-        let userEmail = null;
-        let reportEmails = reportMail ? reportMail.split(',').map(email => email.trim()) : [];
+        let userEmail;
+        ctx.args.recipients = reportMail ? reportMail.split(',').map(email => email.trim()) : [];
 
         if (workerFk) {
             user = await models.VnUser.findById(workerFk, {
@@ -50,22 +50,17 @@ module.exports = Self => {
             account = await models.Account.findById(workerFk);
         }
 
-        if (user?.active && account) {
+        if (user?.active && account)
             userEmail = user.emailUser().email;
-        }
 
-        let recipients = reportEmails;
-        if (userEmail) {
-            recipients.push(userEmail);
-        }
+        if (userEmail)
+            ctx.args.recipients.push(userEmail);
 
-        recipients = [...new Set(recipients)];
+        ctx.args.recipients = [...new Set(ctx.args.recipients)];
 
-        if (recipients.length === 0) {
+        if (!ctx.args.recipients.length)
             throw new UserError('An email is necessary');
-        }
 
-        ctx.args.recipients = recipients;
         return Self.sendTemplate(ctx, 'driver-route');
     };
 };