Erro ao editar Perfil (fixes bug #6331)

Tempo: 02 horas

git-svn-id: http://desenvolvimento.rjconsultores.com.br/repositorio/sco/AdmVenta/Web/trunk/ventaboletos@44321 d1611594-4594-4d17-8e1d-87c2c4800839

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/seguridad/BusquedaPerfilController.java

@@ -24,6 +24,7 @@ import com.rjconsultores.ventaboletos.entidad.Usuario;
 import com.rjconsultores.ventaboletos.service.PerfilService;
 import com.rjconsultores.ventaboletos.service.UsuarioService;
 import com.rjconsultores.ventaboletos.utilerias.UsuarioLogado;
+import com.rjconsultores.ventaboletos.vo.segurida.PerfilJerarquia;
 import com.rjconsultores.ventaboletos.web.utilerias.MyGenericForwardComposer;
 import com.rjconsultores.ventaboletos.web.utilerias.MyListbox;
 import com.rjconsultores.ventaboletos.web.utilerias.paginacion.HibernateSearchObject;
@@ -105,10 +106,13 @@ public class BusquedaPerfilController extends MyGenericForwardComposer {
 
 		if (!usuarioJerarquia.equals(PerfilJerarquia.ADMIN)) {
 			if (!usuarioJerarquia.equals(PerfilJerarquia.NORMAL)) {
-				sistemaBusqueda.addFilterOr(new Filter("indJerarquia", usuarioJerarquia.getValor(), Filter.OP_GREATER_OR_EQUAL),
+				sistemaBusqueda.addFilterOr(
+						new Filter("indJerarquia", usuarioJerarquia.getValor(), Filter.OP_GREATER_OR_EQUAL),
+						new Filter("indJerarquia", null, Filter.OP_NULL),
 						new Filter("indJerarquia", PerfilJerarquia.NORMAL.getValor(), Filter.OP_LESS_OR_EQUAL));
 			} else {
-				sistemaBusqueda.addFilterOr(new Filter("indJerarquia", null, Filter.OP_NULL),
+				sistemaBusqueda.addFilterOr(
+						new Filter("indJerarquia", null, Filter.OP_NULL),
 						new Filter("indJerarquia", PerfilJerarquia.NORMAL.getValor(), Filter.OP_LESS_OR_EQUAL));
 			}
 		}

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/seguridad/EditarPerfilController.java

@@ -39,6 +39,7 @@ import com.rjconsultores.ventaboletos.service.FuncionSistemaService;
 import com.rjconsultores.ventaboletos.service.PerfilFuncionService;
 import com.rjconsultores.ventaboletos.service.PerfilService;
 import com.rjconsultores.ventaboletos.utilerias.UsuarioLogado;
+import com.rjconsultores.ventaboletos.vo.segurida.PerfilJerarquia;
 import com.rjconsultores.ventaboletos.web.utilerias.MyGenericForwardComposer;
 import com.rjconsultores.ventaboletos.web.utilerias.MyListbox;
 import com.rjconsultores.ventaboletos.web.utilerias.render.RenderPermiso;

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/seguridad/EditarUsuarioController.java

@@ -7,7 +7,6 @@ package com.rjconsultores.ventaboletos.web.gui.controladores.seguridad;
 import java.util.ArrayList;
 import java.util.Calendar;
 import java.util.Collections;
-import java.util.Iterator;
 import java.util.List;
 
 import org.apache.commons.collections.CollectionUtils;
@@ -65,7 +64,9 @@ import com.rjconsultores.ventaboletos.web.utilerias.render.RenderUsuarioEmpresa;
 @Scope("prototype")
 public class EditarUsuarioController extends MyGenericForwardComposer {
 
-	private static final String CONTRASENA_VACIA = "**********"; 
+	private static final long serialVersionUID = 1L;
+
+	private static final String CONTRASENA_VACIA = "**********";
 	@Autowired
 	private UsuarioService usuarioService;
 	@Autowired
@@ -344,7 +345,13 @@ public class EditarUsuarioController extends MyGenericForwardComposer {
 
 	@Override
 	public void doAfterCompose(Component comp) throws Exception {
-		lsPerfil = perfilService.obtenerTodos();
+
+		// lsPerfil = perfilService.obtenerTodos();
+
+		Usuario usuarioLogado = usuarioService.obtenerID(UsuarioLogado.getUsuarioLogado().getUsuarioId());
+		Perfil perfilUsuario = usuarioLogado.getUsuarioPerfilList().get(0).getPerfil();
+
+		lsPerfil = perfilService.buscaPerfilJerarquia(perfilUsuario);
 		lsEmpresa = empresaService.obtenerTodos();
 
 		super.doAfterCompose(comp);
@@ -586,8 +593,8 @@ public class EditarUsuarioController extends MyGenericForwardComposer {
 		if (!txtSenha.getValue().equals(CONTRASENA_VACIA)) {
 			senha = txtSenha.getText();
 			confirmSenha = txtConfirmSenha.getText();
-			
-			hayCambioContrasena=true;
+
+			hayCambioContrasena = true;
 		}
 
 		try {
@@ -613,9 +620,9 @@ public class EditarUsuarioController extends MyGenericForwardComposer {
 			Perfil perfil = (Perfil) cmbPerfil.getSelectedItem().getValue();
 			usuario.setUsuarioEmpresaList(lsUsuarioEmpresa);
 			usuario.setUsuarioUbicacionList(lsUsuarioUbicacion);
-			senha = hayCambioContrasena?senha:null;
+			senha = hayCambioContrasena ? senha : null;
 			usuarioService.suscribirActualizar(usuario, senha, perfil);
-			
+
 			if (usrAdmin) {
 				usuarioList.updateItem(usuario);
 			}
@@ -716,19 +723,19 @@ public class EditarUsuarioController extends MyGenericForwardComposer {
 
 		enableAddUbicacion();
 	}
-	
+
 	public void onClick$btnFinalizarSesion(Event ev) {
 		try {
 			List<UsuarioSesion> usuarioSesion = usuarioSesionService.buscar(usuario);
-			
+
 			for (UsuarioSesion u : usuarioSesion) {
 				u.setIndFirmado(Boolean.FALSE);
 				usuarioSesionService.actualizacion(u);
 			}
-			
+
 			Messagebox.show(Labels.getLabel("busquedaUsuarioSesionController.MSG.ok"),
 					Labels.getLabel("busquedaUsuarioSesionController.window.title"), Messagebox.OK, Messagebox.INFORMATION);
-			
+
 		} catch (Exception e) {
 			try {
 				Messagebox.show(Labels.getLabel("MSG.Error"), Labels.getLabel("busquedaUsuarioSesionController.window.title"),
@@ -736,6 +743,6 @@ public class EditarUsuarioController extends MyGenericForwardComposer {
 			} catch (InterruptedException e1) {
 			}
 		}
-		
+
 	}
 }

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/seguridad/PerfilJerarquia.java

@@ -1,29 +0,0 @@
-package com.rjconsultores.ventaboletos.web.gui.controladores.seguridad;
-
-public enum PerfilJerarquia {
-
-	NORMAL(0), ADMIN(1), JERARQUIA_2(2), JERARQUIA_3(3), JERARQUIA_4(4);
-
-	private Integer valor;
-
-	private PerfilJerarquia(Integer valor) {
-		this.valor = valor;
-	}
-
-	public Integer getValor() {
-		return valor;
-	}
-
-	public static PerfilJerarquia buscar(Integer valor) {
-		
-		if(valor == null)
-			return NORMAL;
-		
-		for (PerfilJerarquia jerarquia : PerfilJerarquia.values()) {
-			if (jerarquia.getValor().equals(valor)) {
-				return jerarquia;
-			}
-		}
-		return null;
-	}
-}