Actualizar estado de pagos al cancelar viajes

- Añade función update_payments_on_booking_cancel en payments/public.py para actualizar pagos a REFUNDED
- Soluciona el problema donde los pagos quedaban en estado activo después de cancelar reservas

Author: alejandroGM0

README.md

@@ -46,16 +46,15 @@ El siguiente diagrama muestran la planificación del proyecto:
 ## Instalación
 
 ### Requisitos previos
-- Python 3.8+
+- Python 3.13.3 (en caso de otra versión los paquetes de requirements.txt podrían usar versiones diferentes).
 - pip
 - Virtualenv (recomendado)
 
 ### Pasos de instalación
 
 1. Clonar el repositorio:
    ```
-   git clone https://github.com/tu-usuario/charlacar.git
-   cd charlacar
+   git clone https://github.com/alejandroGM0proyecto_software.git
    ```
 
 2. Crear y activar entorno virtual:

codigo/payments/public.py

@@ -8,7 +8,7 @@
 from datetime import timedelta
 
 from .models import Payment
-from .constants import PAYMENT_STATUS_COMPLETED
+from .constants import PAYMENT_STATUS_COMPLETED, PAYMENT_STATUS_REFUNDED
 
 # Clave pública de Stripe para usar en el frontend
 STRIPE_PUBLIC_KEY = getattr(settings, 'STRIPE_PUBLIC_KEY', '')
@@ -83,7 +83,26 @@ def get_recent_payments(days=7, limit=5):
     """
     Obtiene los pagos más recientes realizados en los últimos días especificados.
     """
-    recent_date = timezone.now() - timedelta(days=days)
+    recent_date = timezone.now() - timedelta(days=7)
     return Payment.objects.filter(
         created_at__gte=recent_date
     ).order_by('-created_at')[:limit]
+    
+def update_payments_on_booking_cancel(user, ride):
+    """
+    Actualiza el estado de los pagos asociados a un usuario y un viaje cuando
+    se cancela una reserva de viaje.
+
+    """
+    payments = Payment.objects.filter(
+        payer=user,
+        ride=ride,
+    )
+    
+    count = 0
+    for payment in payments:
+        payment.status = PAYMENT_STATUS_REFUNDED
+        payment.save()
+        count += 1
+        
+    return count

codigo/payments/views.py

@@ -125,11 +125,14 @@ def create_payment(request, ride_id):
         )
         return redirect("rides:ride_detail", ride_id=ride_id)
 
+    # Modificar esta consulta para buscar correctamente solo pagos activos o completados
     existing_payment = Payment.objects.filter(
         payer=request.user,
         ride=ride,
-        status__in=[PAYMENT_STATUS_PENDING, PAYMENT_STATUS_COMPLETED],
+    ).exclude(
+        status__in=[PAYMENT_STATUS_CANCELLED, PAYMENT_STATUS_FAILED, PAYMENT_STATUS_REFUNDED]
     ).first()
+    
 
     if existing_payment:
         if existing_payment.status == PAYMENT_STATUS_COMPLETED:
@@ -148,6 +151,7 @@ def create_payment(request, ride_id):
                 amount=ride.price,
                 ride=ride,
                 status=PAYMENT_STATUS_PENDING,
+                payment_method=PAYMENT_METHOD_STRIPE,
             )
             payment.save()
 

codigo/rides/views.py

@@ -6,6 +6,7 @@
 from django.contrib.auth.decorators import login_required
 from django.shortcuts import get_object_or_404, redirect, render
 from django.utils import timezone
+from payments.public import update_payments_on_booking_cancel
 
 from ._utils import filter_rides_complex, get_ride_context, paginate_rides, process_search_params
 from .constants import (AVAILABLE_SEATS_KEY, DATE_KEY, DESTINATION_KEY,
@@ -152,7 +153,6 @@ def edit_ride(request, ride_id):
     return render(request, RIDE_FORM_TEMPLATE, {FORM_KEY: form, EDIT_MODE_KEY: True})
 
 
-#DEBERIA DE HABER UN ENDPOINT PARA PAYMENTS
 @login_required
 def cancel_booking(request, ride_id):
     """Cancela la reserva de un viaje para el usuario actual."""
@@ -170,5 +170,8 @@ def cancel_booking(request, ride_id):
     ride.passengers.remove(request.user)
     ride.save()
 
+    update_payments_on_booking_cancel(request.user, ride)
+
+    
     messages.success(request, RIDE_BOOKING_CANCELLED_SUCCESS)
     return redirect(RIDE_DETAIL_URL, ride_id=ride.id)