فهرست منبع

Arreglo editar movimiento y quito cosas de repostajes y lyrics de settings

Celestino Rey 1 سال پیش
والد
کامیت
8f487db32f
2فایلهای تغییر یافته به همراه44 افزوده شده و 14 حذف شده
  1. 44 4
      src/finanzas/views.py
  2. 0 10
      src/misfinanzas/settings.py

+ 44 - 4
src/finanzas/views.py

@@ -150,6 +150,9 @@ def editar_movimiento(request, movimiento_id):
     saldo_original_destino = movimiento.cuenta_destino
     monto_original = movimiento.monto
 
+    logger.warning(f"editar_movimiento: monto_original: {monto_original}")
+    logger.warning(f"editar_movimiento: saldo_original_origen: {saldo_original_origen.saldo}")
+    logger.warning(f"editar_movimiento: saldo_original_destino: {saldo_original_destino.saldo}")
     # logger.warning(f"editar_movimiento: fecha: {movimiento.fecha}")
 
     if request.method == 'POST':
@@ -163,18 +166,55 @@ def editar_movimiento(request, movimiento_id):
             movimiento_nuevo = form.save(commit=False)
 
             # Revertir efecto anterior
+            # Cuenta origen
+
+            logger.warning(f"editar_movimiento: Antes de revertir saldo_original_origen en {saldo_original_origen.nombre}: {saldo_original_origen.saldo}")
+
             saldo_original_origen.saldo += monto_original
             saldo_original_origen.save()
+
+            logger.warning(f"editar_movimiento: Después de revertir saldo_original_origen en {saldo_original_origen.nombre}: {saldo_original_origen.saldo}")
+
+            # Cuenta destino
+
+            logger.warning(f"editar_movimiento: Antes de revertir saldo_original_destino en {saldo_original_destino.nombre}: {saldo_original_destino.saldo}")
+
             saldo_original_destino.saldo -= monto_original
             saldo_original_destino.save()
 
+            logger.warning(f"editar_movimiento: Después de revertir saldo_original_destino en {saldo_original_destino.nombre}: {saldo_original_destino.saldo}")
+
+
             # Aplicar nuevo efecto
             movimiento_nuevo.usuario = request.user
             movimiento_nuevo.save()
-            movimiento_nuevo.cuenta_origen.saldo -= movimiento_nuevo.monto
-            movimiento_nuevo.cuenta_origen.save()
-            movimiento_nuevo.cuenta_destino.saldo += movimiento_nuevo.monto
-            movimiento_nuevo.cuenta_destino.save()
+
+            logger.warning(f"editar_movimiento: saldo_original_origen: {saldo_original_origen.saldo}")
+            logger.warning(f"editar_movimiento: Antes de aplicar movimiento_nuevo.cuenta_origen a {movimiento_nuevo.cuenta_origen.nombre}: {movimiento_nuevo.cuenta_origen.saldo}")
+
+            if movimiento_nuevo.cuenta_origen.nombre == saldo_original_origen.nombre: # si se trata de la misma cuenta...
+                logger.warning(f"Se trata de la misma cuenta origen {movimiento_nuevo.cuenta_origen.nombre} y {saldo_original_origen.nombre}")
+                saldo_original_origen.saldo -= movimiento_nuevo.monto # ...tomo como valor del saldo el ya revertido
+                saldo_original_origen.save()
+            else: # ...de lo contrario
+                logger.warning(f"Se trata de cuentas origen distintas {movimiento_nuevo.cuenta_origen.nombre} y {saldo_original_origen.nombre}")
+                movimiento_nuevo.cuenta_origen.saldo -= movimiento_nuevo.monto # ...tomar como valor del saldo el de la nueva cuenta.
+                movimiento_nuevo.cuenta_origen.save()
+
+            logger.warning(f"editar_movimiento: Después de aplicar movimiento_nuevo.cuenta_origen: {movimiento_nuevo.cuenta_origen.saldo}")
+
+            logger.warning(f"editar_movimiento: Antes de aplicar movimiento_nuevo.cuenta_destino a {movimiento_nuevo.cuenta_destino.nombre}: {movimiento_nuevo.cuenta_destino.saldo}")
+
+            if movimiento_nuevo.cuenta_destino.nombre == saldo_original_destino.nombre: # si se trata de la misma cuenta...
+                logger.warning(f"Se trata de la misma cuenta destino {movimiento_nuevo.cuenta_destino.nombre} y {saldo_original_destino.nombre}")
+                saldo_original_destino.saldo += movimiento_nuevo.monto # ...tomo como valor del saldo el ya revertido
+                saldo_original_destino.save()
+            else: # ...de lo contrario
+                logger.warning(f"Se trata de cuentas destino distintas {movimiento_nuevo.cuenta_destino.nombre} y {saldo_original_destino.nombre}")
+                movimiento_nuevo.cuenta_destino.saldo += movimiento_nuevo.monto # ...tomar como valor del saldo el de la nueva cuenta.
+                movimiento_nuevo.cuenta_destino.save()
+
+            logger.warning(f"editar_movimiento: Después de aplicar movimiento_nuevo.cuenta_destino a {movimiento_nuevo.cuenta_destino.nombre}: {movimiento_nuevo.cuenta_destino.saldo}")
 
             return redirect('dashboard')
     else:

+ 0 - 10
src/misfinanzas/settings.py

@@ -214,16 +214,6 @@ LOGGING = {
             'level': 'DEBUG',
             'propagate': True,
         },
-        'repostajes': {
-            'handlers': ['console', 'file'],
-            'level': 'DEBUG',
-            'propagate': False,
-        },
-        'lyrics': {
-            'handlers': ['console', 'file'],
-            'level': 'DEBUG',
-            'propagate': False,
-        },
         'django.request': {
             'handlers': ['mail_admins'],
             'level': 'ERROR',