Handle duplicate repayment attempts
This commit is contained in:
@@ -57,7 +57,7 @@ class RepaymentService(BaseService):
|
|||||||
return ResponseHelper.error(result_description="Failed to save repayment details.")
|
return ResponseHelper.error(result_description="Failed to save repayment details.")
|
||||||
|
|
||||||
#Update Loan status
|
#Update Loan status
|
||||||
Loan.update_status(loan_id = loan_id, status = LoanStatus.START_REPAY) # repay started bu user
|
Loan.update_status(loan_id = loan_id, status = LoanStatus.START_REPAY) # repay started by user
|
||||||
transaction = RepaymentService.log_transaction(validated_data = validated_data)
|
transaction = RepaymentService.log_transaction(validated_data = validated_data)
|
||||||
|
|
||||||
if not transaction:
|
if not transaction:
|
||||||
|
|||||||
+17
-11
@@ -4,6 +4,8 @@ from app.models import account
|
|||||||
from sqlalchemy.exc import IntegrityError
|
from sqlalchemy.exc import IntegrityError
|
||||||
from sqlalchemy import and_, or_, not_
|
from sqlalchemy import and_, or_, not_
|
||||||
from sqlalchemy.sql import func
|
from sqlalchemy.sql import func
|
||||||
|
from app.api.enums import TransactionType
|
||||||
|
|
||||||
|
|
||||||
class Transaction(db.Model):
|
class Transaction(db.Model):
|
||||||
__tablename__ = 'transactions'
|
__tablename__ = 'transactions'
|
||||||
@@ -20,7 +22,7 @@ class Transaction(db.Model):
|
|||||||
phone_number = db.Column(db.String(50), nullable=True)
|
phone_number = db.Column(db.String(50), nullable=True)
|
||||||
created_at = db.Column(db.DateTime(timezone=True), server_default=func.now())
|
created_at = db.Column(db.DateTime(timezone=True), server_default=func.now())
|
||||||
updated_at = db.Column(db.DateTime(timezone=True), server_default=func.now(), onupdate=func.now())
|
updated_at = db.Column(db.DateTime(timezone=True), server_default=func.now(), onupdate=func.now())
|
||||||
|
|
||||||
def __repr__(self):
|
def __repr__(self):
|
||||||
return f'<Transaction {self.id}>'
|
return f'<Transaction {self.id}>'
|
||||||
|
|
||||||
@@ -30,17 +32,21 @@ class Transaction(db.Model):
|
|||||||
# if cls.query.filter_by(transaction_id=transaction_id).first():
|
# if cls.query.filter_by(transaction_id=transaction_id).first():
|
||||||
# raise ValueError("Duplicate Transaction")
|
# raise ValueError("Duplicate Transaction")
|
||||||
|
|
||||||
if cls.query.filter( and_( cls.transaction_id ==transaction_id, cls.type==type) ).first():
|
if cls.query.filter(and_(cls.transaction_id == transaction_id, cls.type == type)).first():
|
||||||
raise ValueError("Duplicate Transaction")
|
if type == TransactionType.REPAYMENT:
|
||||||
|
logger.info('Repayment transaction already exists :::: But we like to continue.')
|
||||||
|
now = datetime.now()
|
||||||
|
type = TransactionType.REPAYMENT + '.'+ now.strftime("%Y%m%d%H%M%S")
|
||||||
|
logger.info('Modify Type :::: {0}'.format(type))
|
||||||
|
else:
|
||||||
|
raise ValueError("Duplicate Transaction")
|
||||||
|
|
||||||
transaction = cls(
|
transaction = cls(
|
||||||
transaction_id = transaction_id,
|
transaction_id=transaction_id,
|
||||||
customer_id = customer_id,
|
customer_id=customer_id,
|
||||||
account_id = account_id,
|
account_id=account_id,
|
||||||
type = type,
|
type=type,
|
||||||
channel = channel,
|
channel=channel,
|
||||||
created_at=datetime.now(timezone.utc),
|
created_at=datetime.now(timezone.utc),
|
||||||
updated_at=datetime.now(timezone.utc)
|
updated_at=datetime.now(timezone.utc)
|
||||||
)
|
)
|
||||||
@@ -54,4 +60,4 @@ class Transaction(db.Model):
|
|||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def get_transaction_by_id(cls, transaction_id):
|
def get_transaction_by_id(cls, transaction_id):
|
||||||
return cls.query.get(transaction_id)
|
return cls.query.get(transaction_id)
|
||||||
|
|||||||
Reference in New Issue
Block a user