src/command/communicationCommand.ts

@@ -1,6 +1,5 @@
 export interface CreateCommunicationCommand {
   preferred: boolean;
-  isSMSAlarming: boolean;
   mobile: string;
   email: string;
   city: string;
@@ -14,7 +13,6 @@ export interface CreateCommunicationCommand {
 export interface UpdateCommunicationCommand {
   id: number;
   preferred: boolean;
-  isSMSAlarming: boolean;
   mobile: string;
   email: string;
   city: string;

src/command/communicationCommandHandler.ts

@@ -22,7 +22,6 @@ export default abstract class CommunicationCommandHandler {
       .into(communication)
       .values({
         preferred: createCommunication.preferred,
-        isSMSAlarming: createCommunication.isSMSAlarming,
         mobile: createCommunication.mobile,
         email: createCommunication.email,
         city: createCommunication.city,
@@ -60,7 +59,6 @@ export default abstract class CommunicationCommandHandler {
       .update(communication)
       .set({
         preferred: updateCommunication.preferred,
-        isSMSAlarming: updateCommunication.isSMSAlarming,
         mobile: updateCommunication.mobile,
         email: updateCommunication.email,
         city: updateCommunication.city,

src/command/memberCommandHandler.ts

@@ -68,6 +68,7 @@ export default abstract class MemberCommandHandler {
    * @returns {Promise<void>}
    */
   static async updateNewsletter(updateMember: UpdateMemberNewsletterCommand): Promise<void> {
+    console.log(updateMember);
     return await dataSource
       .createQueryBuilder()
       .update(member)
@@ -87,26 +88,6 @@ export default abstract class MemberCommandHandler {
       });
   }
 
-  /**
-   * @description update member newsletter to unset
-   * @param memberId string
-   * @returns {Promise<void>}
-   */
-  static async unsetNewsletter(memberId: number): Promise<void> {
-    return await dataSource
-      .createQueryBuilder()
-      .update(member)
-      .set({
-        sendNewsletter: null,
-      })
-      .where("id = :id", { id: memberId })
-      .execute()
-      .then(() => {})
-      .catch((err) => {
-        throw new InternalException("Failed updating member", err);
-      });
-  }
-
   /**
    * @description delete member
    * @param DeleteMemberCommand

src/controller/admin/memberController.ts

@@ -346,7 +346,6 @@ export async function addExecutivePositionToMember(req: Request, res: Response):
 export async function addCommunicationToMember(req: Request, res: Response): Promise<any> {
   const memberId = parseInt(req.params.memberId);
   const preferred = req.body.preferred;
-  const isSMSAlarming = req.body.isSMSAlarming;
   const mobile = req.body.mobile;
   const email = req.body.email;
   const city = req.body.city;
@@ -358,7 +357,6 @@ export async function addCommunicationToMember(req: Request, res: Response): Pro
 
   let createCommunication: CreateCommunicationCommand = {
     preferred,
-    isSMSAlarming,
     mobile,
     email,
     city,
@@ -530,7 +528,6 @@ export async function updateCommunicationOfMember(req: Request, res: Response):
   const memberId = parseInt(req.params.memberId);
   const recordId = parseInt(req.params.recordId);
   const preferred = req.body.preferred;
-  const isSMSAlarming = req.body.isSMSAlarming;
   const mobile = req.body.mobile;
   const email = req.body.email;
   const city = req.body.city;
@@ -543,7 +540,6 @@ export async function updateCommunicationOfMember(req: Request, res: Response):
   let updateCommunication: UpdateCommunicationCommand = {
     id: recordId,
     preferred,
-    isSMSAlarming,
     mobile,
     email,
     city,
@@ -554,16 +550,12 @@ export async function updateCommunicationOfMember(req: Request, res: Response):
   };
   await CommunicationCommandHandler.update(updateCommunication);
 
-  let currentUserNewsletterMain = await MemberService.getNewsletterById(memberId);
-
   if (isNewsletterMain) {
     let updateNewsletter: UpdateMemberNewsletterCommand = {
       id: memberId,
       communicationId: recordId,
     };
     await MemberCommandHandler.updateNewsletter(updateNewsletter);
-  } else if (currentUserNewsletterMain.sendNewsletter.id == recordId) {
-    await MemberCommandHandler.unsetNewsletter(memberId);
   }
 
   res.sendStatus(204);

src/data-source.ts

@@ -42,7 +42,6 @@ import { calendarType } from "./entity/calendarType";
 import { Calendar1729947763295 } from "./migrations/1729947763295-calendar";
 import { reset } from "./entity/reset";
 import { ResetToken1732358596823 } from "./migrations/1732358596823-resetToken";
-import { SMSAlarming1732696919191 } from "./migrations/1732696919191-SMSAlarming";
 
 const dataSource = new DataSource({
   type: DB_TYPE as any,
@@ -95,7 +94,6 @@ const dataSource = new DataSource({
     Protocol1729347911107,
     Calendar1729947763295,
     ResetToken1732358596823,
-    SMSAlarming1732696919191,
   ],
   migrationsRun: true,
   migrationsTransactionMode: "each",

src/entity/communication.ts

@@ -10,9 +10,6 @@ export class communication {
   @Column({ type: "boolean", default: false })
   preferred: boolean;
 
-  @Column({ type: "boolean", default: false })
-  isSMSAlarming: boolean;
-
   @Column({ type: "varchar", length: 255, nullable: true })
   mobile: string;
 

src/entity/member.ts

@@ -65,5 +65,4 @@ export class member {
   firstMembershipEntry?: membership;
   lastMembershipEntry?: membership;
   preferredCommunication?: Array<communication>;
-  smsAlarming?: Array<communication>;
 }

src/factory/admin/communication.ts

@@ -20,7 +20,6 @@ export default abstract class CommunicationFactory {
       streetNumberAddition: record.streetNumberAddition,
       type: CommunicationTypeFactory.mapToSingle(record.type),
       isNewsletterMain: isMain ? isMain : record?.member?.sendNewsletter?.id == record.id,
-      isSMSAlarming: record.isSMSAlarming,
     };
   }
 

src/factory/admin/member.ts

@@ -27,7 +27,6 @@ export default abstract class MemberFactory {
       preferredCommunication: record?.preferredCommunication
         ? CommunicationFactory.mapToBase(record.preferredCommunication)
         : null,
-      smsAlarming: record?.smsAlarming ? CommunicationFactory.mapToBase(record.smsAlarming) : null,
     };
   }
 

src/migrations/1732696919191-SMSAlarming.ts

@@ -1,21 +0,0 @@
-import { MigrationInterface, QueryRunner, TableColumn } from "typeorm";
-
-export class SMSAlarming1732696919191 implements MigrationInterface {
-  name = "SMSAlarming1732696919191";
-
-  public async up(queryRunner: QueryRunner): Promise<void> {
-    await queryRunner.addColumn(
-      "communication",
-      new TableColumn({
-        name: "isSMSAlarming",
-        type: "tinyint",
-        default: 0,
-        isNullable: false,
-      })
-    );
-  }
-
-  public async down(queryRunner: QueryRunner): Promise<void> {
-    await queryRunner.dropColumn("communication", "isSMSAlarming");
-  }
-}

src/service/communicationService.ts

@@ -56,6 +56,6 @@ export default abstract class CommunicationService {
   static getAvailableColumnsForCommunication(): Array<string> {
     let metadata = dataSource.getMetadata(communication);
     let columns = metadata.columns.map((c) => c.propertyName);
-    return columns.filter((c) => !["id", "preferred", "isSMSAlarming", "type", "member"].includes(c));
+    return columns.filter((c) => !["id", "preferred", "type", "member"].includes(c));
   }
 }

src/service/memberService.ts

@@ -35,8 +35,6 @@ export default abstract class MemberService {
         "preferredCommunication.preferred = 1"
       )
       .leftJoinAndSelect("preferredCommunication.type", "communicationtype_preferred")
-      .leftJoinAndMapMany("member.smsAlarming", "member.communications", "smsAlarming", "smsAlarming.isSMSAlarming = 1")
-      .leftJoinAndSelect("smsAlarming.type", "communicationtype_smsAlarming")
       .offset(offset)
       .limit(count)
       .orderBy("member.lastname")
@@ -54,7 +52,7 @@ export default abstract class MemberService {
   /**
    * @description get member by id
    * @param {number} id
-   * @returns {Promise<member>}
+   * @returns {Promise<Array<member>>}
    */
   static async getById(id: number): Promise<member> {
     return await dataSource
@@ -82,9 +80,6 @@ export default abstract class MemberService {
         "preferredCommunication",
         "preferredCommunication.preferred = 1"
       )
-
-      .leftJoinAndMapMany("member.smsAlarming", "member.communications", "smsAlarming", "smsAlarming.isSMSAlarming = 1")
-      .leftJoinAndSelect("smsAlarming.type", "communicationtype_smsAlarming")
       .leftJoinAndSelect("preferredCommunication.type", "communicationtype_preferred")
       .where("member.id = :id", { id: id })
       .getOneOrFail()
@@ -95,24 +90,4 @@ export default abstract class MemberService {
         throw new InternalException("member not found by id", err);
       });
   }
-
-  /**
-   * @description get newsletter by member by id
-   * @param {number} id
-   * @returns {Promise<member>}
-   */
-  static async getNewsletterById(id: number): Promise<member> {
-    return await dataSource
-      .getRepository(member)
-      .createQueryBuilder("member")
-      .leftJoinAndSelect("member.sendNewsletter", "sendNewsletter")
-      .where("member.id = :id", { id: id })
-      .getOneOrFail()
-      .then((res) => {
-        return res;
-      })
-      .catch((err) => {
-        throw new InternalException("member not found by id", err);
-      });
-  }
 }

src/viewmodel/admin/communication.models.ts

@@ -11,5 +11,4 @@ export interface CommunicationViewModel {
   streetNumberAddition: string;
   type: CommunicationTypeViewModel;
   isNewsletterMain: boolean;
-  isSMSAlarming: boolean;
 }

src/viewmodel/admin/member.models.ts

@@ -12,6 +12,5 @@ export interface MemberViewModel {
   firstMembershipEntry?: MembershipViewModel;
   lastMembershipEntry?: MembershipViewModel;
   sendNewsletter?: CommunicationViewModel;
-  smsAlarming?: Array<CommunicationViewModel>;
   preferredCommunication?: Array<CommunicationViewModel>;
 }