src/command/configuration/newsletterConfig/newsletterConfigCommand.ts

@@ -1,8 +1,8 @@
-import { NewsletterConfigEnum } from "../../../enums/newsletterConfigEnum";
+import { NewsletterConfigType } from "../../../enums/newsletterConfigType";
 
 export interface SetNewsletterConfigCommand {
   comTypeId: number;
-  config: NewsletterConfigEnum;
+  config: NewsletterConfigType;
 }
 
 export interface DeleteNewsletterConfigCommand {

src/entity/configuration/newsletterConfig.ts

@@ -1,5 +1,5 @@
 import { Column, Entity, ManyToOne, PrimaryColumn } from "typeorm";
-import { NewsletterConfigEnum } from "../../enums/newsletterConfigEnum";
+import { NewsletterConfigType } from "../../enums/newsletterConfigType";
 import { communicationType } from "./communicationType";
 
 @Entity()
@@ -11,15 +11,15 @@ export class newsletterConfig {
     type: "varchar",
     length: "255",
     transformer: {
-      to(value: NewsletterConfigEnum) {
+      to(value: NewsletterConfigType) {
         return value.toString();
       },
       from(value: string) {
-        return NewsletterConfigEnum[value as keyof typeof NewsletterConfigEnum];
+        return NewsletterConfigType[value as keyof typeof NewsletterConfigType];
       },
     },
   })
-  config: NewsletterConfigEnum;
+  config: NewsletterConfigType;
 
   @ManyToOne(() => communicationType, {
     nullable: false,

src/enums/newsletterConfigEnum.ts

@@ -1,5 +0,0 @@
-export enum NewsletterConfigEnum {
-  pdf = "pdf",
-  mail = "mail",
-  none = "none",
-}

src/enums/newsletterConfigType.ts

@@ -0,0 +1,4 @@
+export enum NewsletterConfigType {
+  pdf = "pdf",
+  mail = "mail",
+}

src/helpers/newsletterHelper.ts

@@ -14,7 +14,7 @@ import { CLUB_NAME } from "../env.defaults";
 import { TemplateHelper } from "./templateHelper";
 import { PdfExport } from "./pdfExport";
 import NewsletterConfigService from "../service/configuration/newsletterConfigService";
-import { NewsletterConfigEnum } from "../enums/newsletterConfigEnum";
+import { NewsletterConfigType } from "../enums/newsletterConfigType";
 import InternalException from "../exceptions/internalException";
 import EventEmitter from "events";
 
@@ -145,7 +145,7 @@ export abstract class NewsletterHelper {
       return [];
     } else {
       let members = await MemberService.getAll({ noLimit: true, ids: queryMemberIds });
-      return members[0].filter((m) => m.sendNewsletter != null);
+      return members[0];
     }
   }
 
@@ -154,11 +154,14 @@ export abstract class NewsletterHelper {
     let recipients = await NewsletterRecipientsService.getAll(newsletterId);
     let config = await NewsletterConfigService.getAll();
 
-    let allowedForMail = config.filter((c) => c.config == NewsletterConfigEnum.mail).map((c) => c.comTypeId);
+    let allowedForMail = config.filter((c) => c.config == NewsletterConfigType.mail).map((c) => c.comTypeId);
 
     const members = await this.transformRecipientsToMembers(newsletter, recipients);
     const mailRecipients = members.filter(
-      (m) => m.sendNewsletter?.email != "" && allowedForMail.includes(m.sendNewsletter?.type?.id)
+      (m) =>
+        m.sendNewsletter != null &&
+        m.sendNewsletter?.email != null &&
+        allowedForMail.includes(m.sendNewsletter?.type?.id)
     );
 
     return mailRecipients;
@@ -169,12 +172,12 @@ export abstract class NewsletterHelper {
     let recipients = await NewsletterRecipientsService.getAll(newsletterId);
     let config = await NewsletterConfigService.getAll();
 
-    let notAllowedForPdf = config
-      .filter((c) => c.config == NewsletterConfigEnum.none || c.config == NewsletterConfigEnum.mail)
-      .map((c) => c.comTypeId);
+    let notAllowedForPdf = config.filter((c) => c.config == NewsletterConfigType.mail).map((c) => c.comTypeId);
 
     const members = await this.transformRecipientsToMembers(newsletter, recipients);
-    const pdfRecipients = members.filter((m) => !notAllowedForPdf.includes(m.sendNewsletter?.type?.id));
+    const pdfRecipients = members.filter(
+      (m) => !notAllowedForPdf.includes(m.sendNewsletter?.type?.id) || m.sendNewsletter == null
+    );
 
     pdfRecipients.unshift({
       id: "0",
@@ -275,7 +278,7 @@ export abstract class NewsletterHelper {
     if (error) throw new InternalException("Failed Building ICS form Pdf", error);
     this.saveIcsToFile(newsletter, value);
 
-    let printWithAdress = config.filter((c) => c.config == NewsletterConfigEnum.pdf).map((c) => c.comTypeId);
+    let printWithAdress = config.filter((c) => c.config == NewsletterConfigType.pdf).map((c) => c.comTypeId);
 
     const pdfRecipients = await this.getPrintRecipients(newsletterId);
 

src/viewmodel/admin/configuration/newsletterConfig.models.ts

@@ -1,8 +1,8 @@
-import { NewsletterConfigEnum } from "../../../enums/newsletterConfigEnum";
+import { NewsletterConfigType } from "../../../enums/newsletterConfigType";
 import { CommunicationTypeViewModel } from "./communicationType.models";
 
 export interface NewsletterConfigViewModel {
   comTypeId: number;
-  config: NewsletterConfigEnum;
+  config: NewsletterConfigType;
   comType: CommunicationTypeViewModel;
 }