Fix: esto no va #3

New Issue

fede · 2024-08-11T19:51:53-03:00

fede commented

2024-08-11 19:51:53 -03:00

		Final_Das/Vista/FrmFactura.cs
		Line 160 in b19399b64c
	
				                        Subtotal = producto.Precio * Convert.ToInt32(numCantidad.Value),

esto tendria que ser un metodo dentro de detalles factura y no un campo porque es peligroso para la integridad de datos que este asi.

https://fedesrv.ddns.net/git/fede/Final_Das/src/commit/b19399b64c99fb765fe0e7e3dd4f34dd97e727e9/Vista/FrmFactura.cs#L160 esto tendria que ser un metodo dentro de detalles factura y no un campo porque es peligroso para la integridad de datos que este asi.

fede added the bug label 2024-08-11 19:51:53 -03:00

fede commented

2024-08-11 19:53:38 -03:00

osea algo así


namespace Entidades
{
    public class DetalleFactura: Detalle<Producto>
    {
        public int IdFactura { get; set; }

        public double Subtotal()
        {
             return Producto.Precio * Cantidad; 
        }
    }
}

osea algo así ```csharp namespace Entidades { public class DetalleFactura: Detalle<Producto> { public int IdFactura { get; set; } public double Subtotal() { return Producto.Precio * Cantidad; } } } ```

fede commented

2024-08-14 00:27:11 -03:00

3f526d09d6

3f526d09d6

fede closed this issue

2024-08-14 00:27:11 -03:00

fede added this to the goals project 2024-08-14 00:28:38 -03:00

fede added this to the Hacer Andar Entity milestone 2024-08-14 00:28:39 -03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: fede/Final_Das#3