Update provide_loan.py

app/api/services/eligibility_check.py

@@ -51,13 +51,6 @@ class EligibilityCheckService(BaseService):
                 
                 db.session.flush()
 
-                # Determine if there is any loan of 3MPC active
-                current_loan  =  EligibilityCheckService.get_current_active_loans_by_account_id(account_id = account_id)
-                if current_loan:
-                    logger.info(f"Account {current_loan.account_id} has active loan {current_loan}")
-                    if current_loan.product_id =='3MPC':
-                        return ResponseHelper.error(result_description="Max loan count for 3MPC reached")
-
                 # Determine Loan count
                 is_eligible = EligibilityCheckService.check_loan_limits(customer_id)
 
@@ -174,12 +167,7 @@ class EligibilityCheckService(BaseService):
             logger.error(f"An error occurred: {str(e)}", exc_info=True)
             db.session.rollback()
             return ResponseHelper.internal_server_error()
-
-    @staticmethod
-    def get_current_active_loans_by_account_id(account_id):
-        current_loan  = Loan.get_current_active_loans_by_account_id(account_id)
-        return current_loan
-
+        
 
     @staticmethod
     def check_loan_limits(customer_id):

app/api/services/provide_loan.py

@@ -18,7 +18,6 @@ from app.api.integrations import EventServiceIntegration
 from app.models import LoanRepaymentSchedule 
 from app.api.services.offer_analysis import OfferAnalysis
 from app.api.helpers.response_helper import ResponseHelper
-from datetime import datetime
 
 class ProvideLoanService(BaseService):
     TRANSACTION_TYPE = TransactionType.PROVIDE_LOAN 
@@ -114,12 +113,9 @@ class ProvideLoanService(BaseService):
                     management = charges["management"]
                     insurance = charges["insurance"]
                     vat = charges["vat"]
-
-                    # Generate Loan Reference
-                    loan_ref = f"SIM{datetime.now().strftime('%Y%m%d%H%M%S%f')}"
                     
-                    # padded_id = str(transaction_id).zfill(12)
-                    # loan_ref = f"{padded_id}{channel}{offer.product_id}"
+                    padded_id = str(transaction_id).zfill(12)
+                    loan_ref = f"{padded_id}{channel}{offer.product_id}"
 
 
                     # Save the loan details

app/api/services/repayment.py

@@ -57,7 +57,7 @@ class RepaymentService(BaseService):
                         return ResponseHelper.error(result_description="Failed to save repayment details.")
 
                     #Update Loan status
-                    Loan.update_status(loan_id = loan_id, status = LoanStatus.START_REPAY) # repay started by user
+                    Loan.update_status(loan_id = loan_id, status = LoanStatus.START_REPAY) # repay started bu user
                     transaction = RepaymentService.log_transaction(validated_data = validated_data)
 
                     if not transaction:

app/models/loan.py

@@ -1,6 +1,5 @@
 from datetime import datetime, timezone, timedelta
 from itertools import product
-from app.api.enums.loan_status import LoanStatus
 from app.extensions import db
 from app.models.customer import Customer
 from app.models.account import Account
@@ -193,32 +192,13 @@ class Loan(db.Model):
         Get all active loans with the same original_transaction ID.
         """
 
-        active_loans = cls.query.filter(
-        cls.original_transaction == original_transaction_id,
-        or_(
-            cls.status == LoanStatus.ACTIVE.value,
-            cls.status == LoanStatus.START_REPAY.value,
-            cls.status == LoanStatus.ACTIVE_PARTIAL.value,
-        )
+        active_loans = cls.query.filter_by(
+        original_transaction=original_transaction_id,
+        # status='active'
         ).all()
 
         return active_loans
 
-    @classmethod
-    def get_current_active_loans_by_account_id(cls, account_id):
-        """
-        Get the first active loan based on the accountID.
-        """
-        first_active_loan = cls.query.filter(
-        cls.account_id == account_id,
-        or_(
-            cls.status == LoanStatus.ACTIVE.value,
-            cls.status == LoanStatus.START_REPAY.value,
-            cls.status == LoanStatus.ACTIVE_PARTIAL.value,
-        )
-        ).order_by(cls.id.desc()).first()
-
-        return first_active_loan
 
     @classmethod
     def update_status(cls, loan_id, status):

app/models/repayment.py

@@ -36,8 +36,9 @@ class Repayment(db.Model):
     def create_repayment(cls, customer_id, loan, transaction_id):
         
         # Check that the loan is active
-        if loan.status not in [LoanStatus.ACTIVE, LoanStatus.START_REPAY, LoanStatus.ACTIVE_PARTIAL]:
+        if loan.status not in [LoanStatus.ACTIVE, LoanStatus.START_REPAY]:
             raise ValueError(f"Repayment cannot be processed. Loan status: ({loan.status})")
+        
 
         repayment = cls(
             customer_id=customer_id,

app/models/transaction.py

@@ -4,12 +4,6 @@ from app.models import account
 from sqlalchemy.exc import IntegrityError
 from sqlalchemy import and_, or_, not_
 from sqlalchemy.sql import func
-from app.api.enums import TransactionType
-
-import logging
-logger = logging.getLogger(__name__)
-
-
 
 class Transaction(db.Model):
     __tablename__ = 'transactions'
@@ -26,7 +20,7 @@ class Transaction(db.Model):
     phone_number = db.Column(db.String(50), nullable=True)
     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())
-
+    
     def __repr__(self):
         return f'<Transaction {self.id}>'
 
@@ -36,21 +30,17 @@ class Transaction(db.Model):
         # if cls.query.filter_by(transaction_id=transaction_id).first():
         #     raise ValueError("Duplicate Transaction")
 
-        if cls.query.filter(and_(cls.transaction_id == transaction_id, cls.type == type)).first():
-            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")
+        if cls.query.filter( and_( cls.transaction_id ==transaction_id, cls.type==type) ).first():
+            raise ValueError("Duplicate Transaction")
+
+
 
         transaction = cls(
-            transaction_id=transaction_id,
-            customer_id=customer_id,
-            account_id=account_id,
-            type=type,
-            channel=channel,
+            transaction_id = transaction_id,
+            customer_id = customer_id,
+            account_id = account_id,
+            type = type,
+            channel = channel,
             created_at=datetime.now(timezone.utc),
             updated_at=datetime.now(timezone.utc)
         )
@@ -64,4 +54,4 @@ class Transaction(db.Model):
 
     @classmethod
     def get_transaction_by_id(cls, transaction_id):
-        return cls.query.get(transaction_id)
+        return cls.query.get(transaction_id)