refactor RecipientsNotFoundError with recipients as array, #2975

src/api/common/error/RecipientsNotFoundError.js

@@ -3,8 +3,12 @@
 import {TutanotaError} from "./TutanotaError"
 
 export class RecipientsNotFoundError extends TutanotaError {
-	constructor(m: string) {
-		super("RecipientsNotFoundError", m)
+
+	+recipients: Array<string>
+
+	constructor(recipients: Array<string>) {
+		super("RecipientsNotFoundError", recipients.join("\n"))
+		this.recipients = recipients
 	}
 
 }

src/api/main/WorkerClient.js

@@ -611,7 +611,7 @@ export class WorkerClient implements EntityRestInterface {
 		return this._leaderStatus.leaderStatus
 	}
 
-	createTemplateGroup(name: string):Promise<Id> {
+	createTemplateGroup(name: string): Promise<Id> {
 		return this._postRequest(new Request('createTemplateGroup', arguments))
 	}
 }

src/api/worker/facades/MailFacade.js

@@ -391,7 +391,7 @@ export class MailFacade {
 	}
 
 	_addRecipientKeyData(bucketKey: Aes128Key, service: SendDraftData, recipientInfos: RecipientInfo[], senderMailGroupId: Id): Promise<void> {
-		let notFoundRecipients = []
+		let notFoundRecipients: Array<string> = []
 		return Promise.all(recipientInfos.map(recipientInfo => {
 				if (recipientInfo.mailAddress === "system@tutanota.de" || !recipientInfo) {
 					notFoundRecipients.push(recipientInfo.mailAddress)
@@ -438,7 +438,7 @@ export class MailFacade {
 				}
 			})
 		).then(() => {
-			if (notFoundRecipients.length > 0) throw new RecipientsNotFoundError(notFoundRecipients.join("\n"))
+			if (notFoundRecipients.length > 0) throw new RecipientsNotFoundError(notFoundRecipients)
 		})
 	}
 

src/api/worker/facades/ShareFacade.js

@@ -64,7 +64,7 @@ export class ShareFacade {
 						}
 					})
 				}).then(() => {
-					if (notFoundRecipients.length > 0) throw new RecipientsNotFoundError(notFoundRecipients.join("\n"))
+					if (notFoundRecipients.length > 0) throw new RecipientsNotFoundError(notFoundRecipients)
 					return serviceRequest(TutanotaService.GroupInvitationService, HttpMethod.POST, invitationData, GroupInvitationPostReturnTypeRef)
 				})
 			})

src/calendar/CalendarUpdateDistributor.js

@@ -76,7 +76,7 @@ export class CalendarMailDistributor implements CalendarUpdateDistributor {
 		}).catch(RecipientsNotFoundError, e => {
 			// we want to delete the event even if the recipient is not an existing tutanota address
 			// and just exclude them from sending out updates but leave the event untouched for other recipients
-			const invalidRecipients = e.message.split("\n")
+			const invalidRecipients = e.recipients
 			let hasRemovedRecipient = false
 			invalidRecipients.forEach(invalidRecipient => {
 				const recipientInfo = sendMailModel.bccRecipients().find(r => r.mailAddress === invalidRecipient)

src/mail/editor/SendMailModel.js

@@ -757,7 +757,7 @@ export class SendMailModel {
 			.catch(RecipientNotResolvedError, () => {throw new UserError("tooManyAttempts_msg")})
 			.catch(RecipientsNotFoundError, (e) => {
 				if (mailMethod === MailMethod.ICAL_CANCEL) {
-					//in case of calendar event cancellation we willremove invalid recipients and then delete the event without sending updates
+					//in case of calendar event cancellation we will remove invalid recipients and then delete the event without sending updates
 					throw e
 				} else {
 					let invalidRecipients = e.message