admin sin errores

Cuentas/cuentas.Designer.cs

@@ -117,6 +117,7 @@
             this.button3.TabIndex = 10;
             this.button3.Text = "Productos";
             this.button3.UseVisualStyleBackColor = false;
+            this.button3.Click += new System.EventHandler(this.button3_Click);
             // 
             // button5
             // 

Cuentas/cuentas.cs

@@ -9,6 +9,7 @@ using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 using System.Windows.Forms;
+using compabetov2.admin;
 
 namespace compabetov2
 {
@@ -55,7 +56,7 @@ namespace compabetov2
 
             foreach (Cuenta cuenta in cuentas)
             {
-                
+
                 TableLayoutPanel carta = new TableLayoutPanel();
                 carta.RowCount = 1;
                 carta.ColumnCount = 2;
@@ -84,7 +85,7 @@ namespace compabetov2
                 total.Font = new Font(total.Font.FontFamily, 12);
                 total.Width = 600;
 
-                Label estado = new Label(); 
+                Label estado = new Label();
                 estado.Text = $"Estado: {cuenta.estado}";
                 estado.Font = new Font(total.Font.FontFamily, 12);
                 estado.Width = 600;
@@ -221,16 +222,16 @@ namespace compabetov2
                         MessageBox.Show(mensaje);
                     });
 
-                    HUDPanel.Controls.Add(btnVer,2,0);
+                    HUDPanel.Controls.Add(btnVer, 2, 0);
 
                     carta.Controls.Add(HUDPanel, 1, 0);
                 }
 
-                carta.Controls.Add(infoPanel,0,0);
-                
+                carta.Controls.Add(infoPanel, 0, 0);
+
 
                 panelCuenta.Controls.Add(carta);
-                
+
             }
         }
 
@@ -246,8 +247,30 @@ namespace compabetov2
 
         private void button2_Click(object sender, EventArgs e)
         {
-            Estadiscticas verstats = new Estadiscticas();
-            verstats.Show();
+            if (loginPrincipal.tipo.Equals("empleado"))
+            {
+                MessageBox.Show("Usted no tiene permisos de acceder a este apartado");
+            }
+            else
+            {
+                Estadiscticas estadiscticas = new Estadiscticas();
+                estadiscticas.Owner = this;
+                estadiscticas.Show();
+            }
+        }
+
+        private void button3_Click(object sender, EventArgs e)
+        {
+            if (loginPrincipal.tipo.Equals("empleado"))
+            {
+                MessageBox.Show("Usted no tiene permisos de acceder a este apartado");
+            }
+            else
+            {
+                crud crud = new crud();
+                crud.Owner = this;
+                crud.Show();
+            }
         }
     }
 }

Envios/envios.cs

@@ -429,8 +429,16 @@ namespace compabetov2
 
         private void button2_Click(object sender, EventArgs e)
         {
-            Estadiscticas verstats = new Estadiscticas();
-            verstats.Show();
+            if (loginPrincipal.tipo.Equals("empleado"))
+            {
+                MessageBox.Show("Usted no tiene permisos de acceder a este apartado");
+            }
+            else
+            {
+                Estadiscticas estadiscticas = new Estadiscticas();
+                estadiscticas.Owner = this;
+                estadiscticas.Show();
+            };
         }
 
         private void txtTelefono_TextChanged(object sender, EventArgs e)

Ventas/vender.cs

@@ -866,9 +866,16 @@ namespace compabetov2
 
         private void button2_Click(object sender, EventArgs e)
         {
-            Estadiscticas estadiscticas = new Estadiscticas();
-            estadiscticas.Owner = this;
-            estadiscticas.Show();
+            if (loginPrincipal.tipo.Equals("empleado"))
+            {
+                MessageBox.Show("Usted no tiene permisos de acceder a este apartado");
+            }
+            else
+            {
+                Estadiscticas estadiscticas = new Estadiscticas();
+                estadiscticas.Owner = this;
+                estadiscticas.Show();
+            }
         }
 
         private void btnImporte_Click(object sender, EventArgs e)

admin/addvariante.cs

@@ -63,7 +63,7 @@ namespace compabetov2.admin
             }
             catch
             {
-                MessageBox.Show("Por favor introduzca un precio valido", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                MessageBox.Show("Por favor introduzca un numero valido en volumen", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 return false;
             }
         }

admin/editar.cs

@@ -63,37 +63,18 @@ namespace compabetov2.admin
                 //en caso de que descida cambiar la imagen
                 if (!imagePath.Equals(""))
                 {
-                    Producto productoNuevo = new Producto(producto.idProducto, nombre, "", precio, imagePath, 0);
+                    Producto productoNuevo = new Producto(producto.idProducto, nombre, "", precio, Path.GetFileName(imagePath), 0);
                     if (service.editarProducto(productoNuevo))
                     {
-                        //guardar imagen nueva
-                        string destinationFolder = Path.Combine(Application.StartupPath, "Resources");
-                        string destinationFile = Path.Combine(destinationFolder, Path.GetFileName(imagePath));
-                        File.Copy(imagePath, destinationFile, true);
+                        if(Path.GetFileName(imagePath) != producto.img)
+                        {
+                            //guardar imagen nueva
+                            string destinationFolder = Path.Combine(Application.StartupPath, "Resources");
+                            string destinationFile = Path.Combine(destinationFolder, Path.GetFileName(imagePath));
+                            File.Copy(imagePath, destinationFile, true);
+                        }
 
                         ventanaPadre.llenarLayout();
-                        
-
-                        //eliminar imagen antigua
-                        string imagenEliminada = Path.Combine(destinationFolder, producto.img);
-                        //se pone a esperar 1 segundo antes de eliminar la imagen
-                        //esto para que al programa le de tiempo de dejar de utilizar la imagen y no lance una excepcion
-                        Timer timer = new Timer();
-                        timer.Interval = 1000; // 1 segundos
-                        timer.Tick += delegate (object sender2, EventArgs e2)
-                        {
-                            try
-                            {
-                                File.Delete(imagenEliminada);
-                                timer.Stop();
-                            }
-                            catch (Exception ex)
-                            {
-                                // Manejar la excepción de eliminación de archivo temporal
-                                Console.WriteLine("Error al eliminar archivo temporal: " + ex.Message);
-                            }
-                        };
-
                         Close();
 
                     }
@@ -111,6 +92,7 @@ namespace compabetov2.admin
             }
             catch(Exception ex)
             {
+                MessageBox.Show(ex.Message);
                 MessageBox.Show("Por favor introduzca un precio valido", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
             }
         }

admin/editarvariante.cs

@@ -69,7 +69,7 @@ namespace compabetov2.admin
             }
             catch
             {
-                MessageBox.Show("Por favor introduzca un precio valido", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
+                MessageBox.Show("Por favor introduzca un numero valido en volumen", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                 return false;
             }
         }

database/DAO.cs

@@ -524,22 +524,42 @@ namespace compabetov2.database
 
         public static bool eliminarProductoDAO(Producto producto)
         {
+
             var db_conexion = new Conexion();
+
             try
             {
                 using (SQLiteConnection conexion = db_conexion.get_connection())
                 {
                     conexion.Open();
 
-                    string sql = "DELETE FROM Producto WHERE idproducto = @idproducto";
-
-                    using (SQLiteCommand command = new SQLiteCommand(sql, conexion))
+                    using (var transaccion = conexion.BeginTransaction())
                     {
-                        command.Parameters.AddWithValue("@idproducto", producto.idProducto);
-                        int filasAfectadas = command.ExecuteNonQuery();
+                        try
+                        {
+                            string sql = "DELETE FROM Producto WHERE idproducto = @idproducto";
+                            string sqlVariante = "DELETE FROM variantes WHERE fk_producto=@fk_producto;";
+
+                            SQLiteCommand commandProducto = new SQLiteCommand(sql, conexion);
+                            commandProducto.Parameters.AddWithValue("@idproducto", producto.idProducto);
+                            int filasAfectadas = commandProducto.ExecuteNonQuery();
+
+                            SQLiteCommand commandVariante = new SQLiteCommand(sqlVariante, conexion);
+                            commandVariante.Parameters.AddWithValue("@fk_producto", producto.idProducto);
+                            commandVariante.ExecuteNonQuery();
+
+                            transaccion.Commit();
+                            return true;
+
+                        }
+                        catch (SQLiteException ex)
+                        {
+                            transaccion.Rollback();
+                            MessageBox.Show(ex.Message);
+                            return false;
+                        }
+
 
-                        if (filasAfectadas > 0) { return true; }
-                        else { return false; }
                     }
                 }
             }