diff --git a/ProyectoInventario-ejb/src/java/xforce/bl/ProveedorBL.java b/ProyectoInventario-ejb/src/java/xforce/bl/ProveedorBL.java
index 4d1a55e..99ea869 100644
--- a/ProyectoInventario-ejb/src/java/xforce/bl/ProveedorBL.java
+++ b/ProyectoInventario-ejb/src/java/xforce/bl/ProveedorBL.java
@@ -68,8 +68,10 @@ public class ProveedorBL implements ProveedorBLLocal {
         }
         proveedor.setEstado(2);
         Optional<Proveedor> pEncontradoOptional = pDAO.buscarproveedor(proveedor);
+        
         Optional<Usuarios> validarTel = usuariosdao.buscarTelefono(proveedor.getTelefono());
         Optional<Proveedor> validarTelPro = pDAO.buscarTelefono(proveedor.getTelefono());
+        
         Optional<Usuarios> validarEmail = usuariosdao.buscarCorreo(proveedor.getEmail());
         Optional<Proveedor> validarEmailpro = pDAO.buscarCorreo(proveedor.getEmail());
       
@@ -87,24 +89,27 @@ public class ProveedorBL implements ProveedorBLLocal {
         if (validarTel.isPresent()) {
             validarTel.get();
             System.out.println("Telefono duplicado");
-            addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Telefono duplicado");
+            addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Telefono duplicado en usuario");
             return Mensaje.ELEMENTO_DUPLICADO;
         }
-//validar telefono en proveedor       
-        /*if (validarTelPro.isPresent()) {
+        
+        if (validarTelPro.isPresent()){
             validarTelPro.get();
-            System.out.println("Telefono duplicado en proveedor");
-            addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Telefono duplicado en proveedor");
+            addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Telefono duplicado");
             return Mensaje.ELEMENTO_DUPLICADO;
-        }*/
-
-//validar email usuario
-        if (validarEmail.isPresent()) {
-            validarEmail.get();
-            System.out.println("Email duplicado");
+        }
+        
+        if(validarEmailpro.isPresent()){
+            validarEmailpro.get();
             addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Email duplicado");
             return Mensaje.ELEMENTO_DUPLICADO;
         }
+        if(validarEmail.isPresent()){
+            validarEmail.get();
+            addMessage(FacesMessage.SEVERITY_ERROR, "ERROR", "Email duplicado en usuario");
+            return Mensaje.ELEMENTO_DUPLICADO;
+        }
+
 
         
        
diff --git a/ProyectoInventario-war/web/proveedorLista.xhtml b/ProyectoInventario-war/web/proveedorLista.xhtml
index d427391..74732fe 100644
--- a/ProyectoInventario-war/web/proveedorLista.xhtml
+++ b/ProyectoInventario-war/web/proveedorLista.xhtml
@@ -36,11 +36,7 @@
                                 <p:menuitem value="Usuarios" action="usuariosLista.xhtml" ajax="false"/>
 
                                 <p:submenu label="Opciones" icon="pi pi-fw pi-list">
-                                    <p:menuitem value="Roles"
-                                                icon="pi pi-fw pi-users" 
-                                                oncomplete="PF('rol').show()" 
-                                                update=":rolD" 
-                                                actionListener="#{demoBeanRoles.prepararNuevo()}"/>
+                                    
 
 
                                     <p:menuitem value="Paises" 
@@ -120,7 +116,7 @@
                                                              style="margin-top: 5px"
                                                              value="Nuevo"
                                                              ajax="true"
-                                                             disabled="#{demoBeanLogin.permisos}"/>
+                                                             disabled="#{!demoBeanLogin.permisos}"/>
                                         </div>
 
                                         <div style="flex: 1; text-align: center;"> <!-- Div para el texto centrado -->
@@ -212,7 +208,7 @@
                                                          update=":dialogForm" 
                                                          icon="pi pi-pencil"
                                                          ajax="true"
-                                                         disabled="#{demoBeanLogin.permisos}"/>
+                                                         disabled="#{!demoBeanLogin.permisos}"/>
 
                                         <p:commandButton actionListener="#{demoBeanProveedor.prepararEliminar(item)}"
                                                          icon="pi pi-trash"
@@ -221,7 +217,7 @@
                                                          style="margin-left: 5px"
                                                          styleClass="ui-button-danger"
                                                          ajax="true"
-                                                         disabled="#{demoBeanLogin.permisos}"/>
+                                                         disabled="#{!demoBeanLogin.permisos}"/>
                                     </div>
                                 </p:column>
                             </p:dataTable>
@@ -236,7 +232,7 @@
                               responsive="true" 
                               showEffect="fade" 
                               modal="true"  
-                              resizable="false">
+                              resizable="true">
 
                         <h:form>
 
@@ -399,14 +395,14 @@
                               modal="true" 
                               resizable="true" 
                               width="auto"
-                              height="200">
+                              height="auto">
 
 
 
                         <h:form>
                             <p:growl id="messages" showDetail="true" for="mensajeProveedor"/>
                             <h1><h:outputText value=""/></h1>
-                            <p:panelGrid columns="2">
+                            <h:panelGrid columns="2">
 
                                 <h:outputText value="Nombre:"/>
                                 <h:outputText value="#{demoBeanProveedor.proveedor.nombre}" title="Nombre"/>
@@ -421,7 +417,7 @@
                                 <h:outputText value="marca:"/>
                                 <h:outputText value="#{demoBeanProveedor.proveedor.marcaId.nombre}" title="Estado"/>
 
-                            </p:panelGrid>
+                            </h:panelGrid>
 
                             <p:commandButton  action="#{demoBeanProveedor.eliminarProveedor()}"
                                               update="proveedorTabla:table, messages"
diff --git a/ProyectoInventario-war/web/ubicacionLista.xhtml b/ProyectoInventario-war/web/ubicacionLista.xhtml
index 3d977c3..78e21e3 100644
--- a/ProyectoInventario-war/web/ubicacionLista.xhtml
+++ b/ProyectoInventario-war/web/ubicacionLista.xhtml
@@ -33,11 +33,7 @@
 
                                 <p:submenu label="Opciones" icon="pi pi-fw pi-list">
 
-                                    <p:menuitem value="Roles"
-                                                icon="pi pi-fw pi-users" 
-                                                oncomplete="PF('rol').show()" 
-                                                update=":rolD" 
-                                                actionListener="#{demoBeanRoles.prepararNuevo()}"/>
+                                   
 
 
                                     <p:menuitem value="Paises" 
@@ -122,7 +118,7 @@
                                                           update=":dialogForm" 
                                                           icon="pi pi-plus" 
                                                           ajax="true"
-                                                          disabled="#{demoBeanLogin.permisos}"/>
+                                                          disabled="#{!demoBeanLogin.permisos}"/>
                                     </div>
                                     <div style="flex: 1; text-align: center;"> <!-- Div para el texto centrado -->
                                         <header style="display: flex; justify-content: center; font-size: 30px;"><strong>Ubicacion</strong></header>
@@ -167,7 +163,7 @@
                                                          update=":dialogForm" 
                                                          icon="pi pi-fw pi-pencil" 
                                                          styleClass="ui-button-secondary" ajax="true"
-                                                         disabled="#{demoBeanLogin.permisos}"
+                                                         disabled="#{!demoBeanLogin.permisos}"
                                                          />
                                         <p:commandButton actionListener="#{demoBeanUbicacion.prepararEliminar(item)}" 
                                                          ajax="true"
@@ -176,7 +172,7 @@
                                                          icon="pi pi-trash"
                                                          style="margin-left: 5px"
                                                          styleClass="ui-button-danger"
-                                                         disabled="#{demoBeanLogin.permisos}"/>
+                                                         disabled="#{!demoBeanLogin.permisos}"/>
                                     </div>
                                 </p:column>
                             </p:column>
@@ -271,12 +267,12 @@
                               modal="true" 
                               resizable="true" 
                               width="auto"
-                              height="200">
+                              height="auto">
 
                         <h:form>
                             <p:growl id="messages" showDetail="true" for="mensajeUbicacion"/>
 
-                            <p:panelGrid columns="2">
+                            <h:panelGrid columns="2">
 
                                 <h:outputText value="Bloque:"/>
                                 <h:outputText value="#{demoBeanUbicacion.ubi.bloque}" title="Bloque"/>
@@ -284,7 +280,7 @@
                                 <h:outputText value="#{demoBeanUbicacion.ubi.anaquel}" title="Anaquel"/>
                                 <h:outputText value="Estado:"/>
                                 <h:outputText value="#{demoBeanUbicacion.ubi.estado}" title="Estado"/>
-                            </p:panelGrid>
+                            </h:panelGrid>
                             <p:commandButton action="#{demoBeanUbicacion.eliminarUbicacion()}"  
                                              update="ubicacionTabla:table,messages"
                                              icon="pi pi-trash"