add card owner

src/modules/payments/dto/create-payment-method.dto.ts

@@ -1,4 +1,4 @@
-import { IsString, IsOptional, IsBoolean, IsNumber, IsEnum } from 'class-validator';
+import { IsString, IsOptional, IsBoolean, IsNumber, IsEnum, ValidateIf, IsNotEmpty } from 'class-validator';
 import { ApiProperty } from '@nestjs/swagger';
 import { PaymentMethodEnum, PaymentGatewayEnum } from '../interface/payment';
 
@@ -23,6 +23,12 @@ export class CreatePaymentMethodDto {
   @IsString()
   cardNumber?: string;
 
+  @ApiProperty({ description: 'Card owner name for card-to-card payments', required: false })
+  @ValidateIf((dto: CreatePaymentMethodDto) => dto.method === PaymentMethodEnum.CreditCard)
+  @IsNotEmpty()
+  @IsString()
+  cardOwner?: string;
+
   @ApiProperty({ description: 'Is payment method enabled', required: false, default: true })
   @IsOptional()
   @IsBoolean()

src/modules/payments/entities/payment-method.entity.ts

@@ -23,6 +23,9 @@ export class PaymentMethod extends BaseEntity {
   @Property({ nullable: true })
   cardNumber?: string;
 
+  @Property({ nullable: true })
+  cardOwner?: string;
+
   @Property({ default: true })
   enabled: boolean = true;
 

src/modules/payments/services/payment-method.service.ts

@@ -1,4 +1,4 @@
-import { Injectable, NotFoundException, ConflictException } from '@nestjs/common';
+import { BadRequestException, Injectable, NotFoundException } from '@nestjs/common';
 import { EntityManager } from '@mikro-orm/postgresql';
 import { PaymentMethod } from '../entities/payment-method.entity';
 import { PaymentMethodRepository } from '../repositories/payment-method.repository';
@@ -7,6 +7,7 @@ import { UpdatePaymentMethodDto } from '../dto/update-payment-method.dto';
 import { RequiredEntityData } from '@mikro-orm/core';
 import { PaymentMessage } from 'src/common/enums/message.enum';
 import { RestaurantsService } from 'src/modules/restaurants/providers/restaurants.service';
+import { PaymentMethodEnum } from '../interface/payment';
 
 @Injectable()
 export class PaymentMethodService {
@@ -18,7 +19,8 @@ export class PaymentMethodService {
 
   async create(restId: string, createPaymentMethodDto: CreatePaymentMethodDto): Promise<PaymentMethod> {
     // Check if restaurant exists
-    const restaurant = await this.restaurantsService.findOneOrFail(restId)
+    const restaurant = await this.restaurantsService.findOneOrFail(restId);
+    this.assertCreditCardOwner(createPaymentMethodDto.method, createPaymentMethodDto.cardOwner);
 
     const paymentMethod = this.paymentMethodRepository.create({
       restaurant,
@@ -43,10 +45,17 @@ export class PaymentMethodService {
   async update(id: string, updatePaymentMethodDto: UpdatePaymentMethodDto): Promise<PaymentMethod> {
     const paymentMethod = await this.findOneOrFail(id);
     this.em.assign(paymentMethod, updatePaymentMethodDto);
+    this.assertCreditCardOwner(paymentMethod.method, paymentMethod.cardOwner);
     await this.em.flush();
     return paymentMethod;
   }
 
+  private assertCreditCardOwner(method: PaymentMethodEnum, cardOwner?: string): void {
+    if (method === PaymentMethodEnum.CreditCard && !cardOwner?.trim()) {
+      throw new BadRequestException(PaymentMessage.CARD_OWNER_REQUIRED);
+    }
+  }
+
   async remove(id: string): Promise<PaymentMethod> {
     const paymentMethod = await this.findOneOrFail(id);
     await this.em.removeAndFlush(paymentMethod);

src/modules/payments/services/payments.service.ts

@@ -83,6 +83,10 @@ export class PaymentsService {
       throw new BadRequestException(PaymentMessage.CARD_NUMBER_REQUIRED);
     }
 
+    if (pm.method === PaymentMethodEnum.CreditCard && !pm.cardOwner?.trim()) {
+      throw new BadRequestException(PaymentMessage.CARD_OWNER_REQUIRED);
+    }
+
     return {
       order,
       amount: order.total,