Cria novas constantes

- Novas constantes criadas para valores utilizados com frequência. - Criação de atributo estático para ObjectMapper

Cria novas constantes
- Novas constantes criadas para valores utilizados com frequência. - Criação de atributo estático para ObjectMapper
andre.guimaraes
1 parent 7e10bf4f
Showing 9 changed files with 41 additions and 18 deletions Show diff stats
src/main/java/br/gov/ans/integracao/sei/rest/DocumentoResource.java
src/main/java/br/gov/ans/integracao/sei/rest/InfoResource.java
src/main/java/br/gov/ans/integracao/sei/rest/MessagesKeys.java
src/main/java/br/gov/ans/integracao/sei/rest/UnidadeResource.java
src/main/java/br/gov/ans/integracao/sei/utils/Constantes.java
src/main/java/br/gov/ans/integracao/sei/utils/ContatoHelper.java
src/main/java/br/gov/ans/integracao/sei/utils/Util.java
src/main/java/br/gov/ans/integracao/templates/client/ClientTemplatesBroker.java
src/main/java/br/gov/ans/utils/HttpHeadersUtil.java
@@ -329,9 +329,11 @@ public class DocumentoResource {
 				documento.setConteudo(conteudoHTML);
 			}
-			logger.debug(messages.getMessage("debug.novo.documento.enviado"));
+			logger.debug(messages.getMessage(MessagesKeys.DEBUG_NOVO_DOCUMENTO_ENVIADO));
+			
 			retorno = seiNativeService.incluirDocumento(Constantes.SEI_BROKER, Operacao.INCLUIR_DOCUMENTO,  unidadeResource.consultarCodigo(unidade), documento);
-			logger.debug(messages.getMessage("debug.novo.documento.processado"));
+			
+			logger.debug(messages.getMessage(MessagesKeys.DEBUG_NOVO_DOCUMENTO_PROCESSADO));
 		}catch(Exception ex){
 			registrarProblemaInclusao(inclusaoDocumento);
@@ -710,9 +712,10 @@ public class DocumentoResource {
 	}
 	public String transformarConteudoDocumentoInterno(String conteudo, String template) throws RemoteException, Exception{
+		StringWriter writer = new StringWriter();
+
 		try{
 			Mustache mustache = mustacheUtils.compile(removeExtensaoLegado(template));	
-			StringWriter writer = new StringWriter();
 			Map<String, Object> model = decodeConteudoMustache(conteudo);
@@ -723,6 +726,8 @@ public class DocumentoResource {
 		}catch(JsonParseException ex){
 			logger.debug(conteudo);
 			throw new BusinessException(messages.getMessage("erro.processar.conteudo.json"));
+		}finally{
+			writer.close();
 		}
 	}	
@@ -99,7 +99,7 @@ public class InfoResource {
 	@Path("/conexoes/mysql")
 	@Produces(MediaType.TEXT_PLAIN)
 	public String testMySQLConnection() throws Exception{
-		Query query = emMySQL.createNativeQuery("SELECT version()");
+		Query query = emMySQL.createNativeQuery(Constantes.MYSQL_SQL_TEST_CONECTION);
 		try{
 			return((String) query.getSingleResult());
@@ -133,7 +133,7 @@ public class InfoResource {
 	@Path("/conexoes/oracle")
 	@Produces(MediaType.TEXT_PLAIN)
 	public String testOracleConnection() throws Exception{
-		Query query = emOracle.createNativeQuery("SELECT BANNER FROM V$VERSION WHERE ROWNUM = 1");
+		Query query = emOracle.createNativeQuery(Constantes.ORACLE_SQL_TEST_CONECTION);
 		try{
 			return ((String) query.getSingleResult());
@@ -170,7 +170,7 @@ public class InfoResource {
 		try{
 			seiNativeService.listarUnidades(Constantes.SEI_BROKER, Operacao.LISTAR_UNIDADES, null, null);
-			return "SEI respondeu com sucesso.";
+			return Constantes.SEI_RESPONDEU_COM_SUCESSO;
 		}catch(Exception ex){
 			logger.error(messages.getMessage("erro.testar.sei"),ex);
 			throw new Exception(messages.getMessage("erro.testar.sei"));
@@ -0,0 +1,6 @@
+package br.gov.ans.integracao.sei.rest;
+
+public class MessagesKeys {
+	public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
+	public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
+}
@@ -67,8 +67,10 @@ public class UnidadeResource {
 	@Produces({MediaType.APPLICATION_JSON, MediaType.APPLICATION_XML})
 	public br.gov.ans.integracao.sei.client.Unidade[] listarUnidades() throws Exception{
 		Unidade[] lista = seiNativeService.listarUnidades(Constantes.SEI_BROKER, Operacao.LISTAR_UNIDADES, null, null);
-				
-		carregarMapUnidades(lista);
+
+		if(isMapUnidadesExpirado()){
+			carregarMapUnidades(lista);
+		}
 		return lista;
 	}
@@ -6,22 +6,28 @@ import javax.inject.Named;
 @Named
 @ApplicationScoped
 public class Constantes {
+	public static final String ACCEPT_HEADER_KEY = "Accept";
 	private static final int AXIS_TIMEOUT_IN_MINUTES = 10;
 	public static final int AXIS_TIMEOUT = 1000 * (60 * AXIS_TIMEOUT_IN_MINUTES);
 	public static final String CODIGO_BRASIL = "76";
 	public static final String CODIGO_ORGAO_ANS = "0";
+	public static final String  CONTENT_TYPE_HEADER_KEY= "Content-Type";
 	public static final String DATE_PATTERN = "dd/MM/yyyy";
 	public static final String DOCUMENTO_GERAL = "G";
 	public static final String DOCUMENTO_RECEBIDO = "R";
+	public static final String JBOSS_HOME = System.getProperty("jboss.home.dir");
 	public static final String MASCARA_PROCESSO_17 = "#####.######/####-##";
 	public static final String MASCARA_PROCESSO_21 = "#######.########/####-##";
+	public static final String MYSQL_SQL_TEST_CONECTION = "SELECT version()";
 	public static String NAO = "N";
-	public static final String JBOSS_HOME = System.getProperty("jboss.home.dir");
 	public static final String NOME_SISTEMA = "sei-broker";
+	public static final String ORACLE_SQL_TEST_CONECTION = "SELECT BANNER FROM V$VERSION WHERE ROWNUM = 1";
 	public static final String REGEX_MASCARA_PROCESSO = "(\\d)(\\d)(\\d)(\\d)(\\d)(\\.)(\\d)(\\d)(\\d)(\\d)(\\d)(\\d)(\\/)(\\d)(\\d)(\\d)(\\d)(-)(\\d)(\\d)";
 	public static final String REGEX_SOMENTE_NUMEROS = "\\D+";
 	public static final String RESOURCE_METHOD_INVOKER = "org.jboss.resteasy.core.ResourceMethodInvoker";
 	public static final String SEI_BROKER = "SEI-Broker";
+	public static final String SEI_RESPONDEU_COM_SUCESSO = "SEI respondeu com sucesso.";
+	public static String SIM = "S";
 	public static final int TAMANHO_MAXIMO_ARQUIVO = 16777216;
 	public static final int TAMANHO_MAXIMO_DOCUMENTO = 22020096;
 	public static final Integer TAMANHO_PAGINA_PADRAO = 50; 
@@ -110,9 +110,9 @@ public class ContatoHelper {
     private void preencherDadosAssociado(Pessoa pessoa, Contato contato) throws RemoteException, BusinessException, Exception{
 		if(pessoa.getAssociado() == null){
-			contato.setSinEnderecoAssociado("N");
+			contato.setSinEnderecoAssociado(Constantes.NAO);
 		}else{
-			contato.setSinEnderecoAssociado("S");
+			contato.setSinEnderecoAssociado(Constantes.SIM);
 			contato.setIdContatoAssociado(getIdContato(pessoa.getAssociado().getSigla())+"");
 			contato.setNomeContatoAssociado(pessoa.getAssociado().getNome());
@@ -21,7 +21,9 @@ import br.gov.ans.exceptions.BusinessException;
 public class Util {
 	private static final FastDateFormat dateFormater = FastDateFormat.getInstance(Constantes.DATE_PATTERN);
-				
+
+	private static final ObjectMapper mapper = new ObjectMapper();
+	
 	public static String getSOuN(String valor){			
 		if("S".equals(valor) || "s".equals(valor)){
 			return "S";
@@ -110,8 +112,6 @@ public class Util {
 	public static Map<String, Object> jsonToMap(String json) throws JsonParseException,JsonMappingException, IOException {
 		Map<String,Object> out = new HashMap<String,Object>();
-		ObjectMapper mapper = new ObjectMapper();
-		
 		out = mapper.readValue(json, new TypeReference<Map<String, Object>>(){});
 		return  out;
@@ -13,6 +13,7 @@ import br.gov.ans.exceptions.BusinessException;
 import br.gov.ans.exceptions.ErrorMessage;
 import br.gov.ans.factories.qualifiers.Autenticado;
 import br.gov.ans.factories.qualifiers.SeiQualifiers.TemplatesBrokerParameter;
+import br.gov.ans.integracao.sei.utils.Constantes;
 public class ClientTemplatesBroker {
@@ -44,13 +45,14 @@ public class ClientTemplatesBroker {
 			return corpoTemplate;
 		}catch(ProcessingException ex){
-			logger.error(ex, ex);
+			logger.error(ex);
+			logger.debug(ex, ex);
 			throw new Exception("Erro ao carregar o template, contacte a equipe responsável.");
 		}
 	}
 	public void errorHandling(Response response) throws Exception{
-		if(!response.getHeaderString("Content-Type").equals(MediaType.APPLICATION_JSON)){
+		if(!response.getHeaderString(Constantes.CONTENT_TYPE_HEADER_KEY).equals(MediaType.APPLICATION_JSON)){
 			logger.error(response.readEntity(String.class));
 			response.close();
@@ -7,13 +7,15 @@ import javax.ws.rs.core.MediaType;
 import org.apache.commons.lang3.StringUtils;
 import org.jboss.logging.Logger;
+import br.gov.ans.integracao.sei.utils.Constantes;
+
 public class HttpHeadersUtil {
 	private static Logger logger = Logger.getLogger(HttpHeadersUtil.class);
 	public static String getAcceptType(HttpHeaders headers){
 		try{
-			String accept = headers.getRequestHeader("Accept").get(0);
+			String accept = headers.getRequestHeader(Constantes.ACCEPT_HEADER_KEY).get(0);
 			if(!StringUtils.isBlank(accept)&& StringUtils.contains(accept, MediaType.APPLICATION_XML)){
 				return MediaType.APPLICATION_XML;		
@@ -27,7 +29,7 @@ public class HttpHeadersUtil {
 	public static String getAcceptType(HttpServletRequest request){
 		try{
-			String accept = request.getHeader("Accept");
+			String accept = request.getHeader(Constantes.ACCEPT_HEADER_KEY);
 			if(!StringUtils.isBlank(accept)&& StringUtils.contains(accept, MediaType.APPLICATION_XML)){
 				return MediaType.APPLICATION_XML;
	@@ -99,7 +99,7 @@ public class InfoResource {		@@ -99,7 +99,7 @@ public class InfoResource {
99	@Path("/conexoes/mysql")	99	@Path("/conexoes/mysql")
100	@Produces(MediaType.TEXT_PLAIN)	100	@Produces(MediaType.TEXT_PLAIN)
101	public String testMySQLConnection() throws Exception{	101	public String testMySQLConnection() throws Exception{
102	- Query query = emMySQL.createNativeQuery("SELECT version()");	102	+ Query query = emMySQL.createNativeQuery(Constantes.MYSQL_SQL_TEST_CONECTION);
103		103
104	try{	104	try{
105	return((String) query.getSingleResult());	105	return((String) query.getSingleResult());
	@@ -133,7 +133,7 @@ public class InfoResource {		@@ -133,7 +133,7 @@ public class InfoResource {
133	@Path("/conexoes/oracle")	133	@Path("/conexoes/oracle")
134	@Produces(MediaType.TEXT_PLAIN)	134	@Produces(MediaType.TEXT_PLAIN)
135	public String testOracleConnection() throws Exception{	135	public String testOracleConnection() throws Exception{
136	- Query query = emOracle.createNativeQuery("SELECT BANNER FROM V$VERSION WHERE ROWNUM = 1");	136	+ Query query = emOracle.createNativeQuery(Constantes.ORACLE_SQL_TEST_CONECTION);
137		137
138	try{	138	try{
139	return ((String) query.getSingleResult());	139	return ((String) query.getSingleResult());
	@@ -170,7 +170,7 @@ public class InfoResource {		@@ -170,7 +170,7 @@ public class InfoResource {
170	try{	170	try{
171	seiNativeService.listarUnidades(Constantes.SEI_BROKER, Operacao.LISTAR_UNIDADES, null, null);	171	seiNativeService.listarUnidades(Constantes.SEI_BROKER, Operacao.LISTAR_UNIDADES, null, null);
172		172
173	- return "SEI respondeu com sucesso.";	173	+ return Constantes.SEI_RESPONDEU_COM_SUCESSO;
174	}catch(Exception ex){	174	}catch(Exception ex){
175	logger.error(messages.getMessage("erro.testar.sei"),ex);	175	logger.error(messages.getMessage("erro.testar.sei"),ex);
176	throw new Exception(messages.getMessage("erro.testar.sei"));	176	throw new Exception(messages.getMessage("erro.testar.sei"));
@@ -0,0 +1,6 @@		@@ -0,0 +1,6 @@
	1	+package br.gov.ans.integracao.sei.rest;
	2	+
	3	+public class MessagesKeys {
	4	+ public static final String DEBUG_NOVO_DOCUMENTO_ENVIADO = "debug.novo.documento.enviado";
	5	+ public static final String DEBUG_NOVO_DOCUMENTO_PROCESSADO = "debug.novo.documento.processado";
	6	+}
	@@ -110,9 +110,9 @@ public class ContatoHelper {		@@ -110,9 +110,9 @@ public class ContatoHelper {
110		110
111	private void preencherDadosAssociado(Pessoa pessoa, Contato contato) throws RemoteException, BusinessException, Exception{	111	private void preencherDadosAssociado(Pessoa pessoa, Contato contato) throws RemoteException, BusinessException, Exception{
112	if(pessoa.getAssociado() == null){	112	if(pessoa.getAssociado() == null){
113	- contato.setSinEnderecoAssociado("N");	113	+ contato.setSinEnderecoAssociado(Constantes.NAO);
114	}else{	114	}else{
115	- contato.setSinEnderecoAssociado("S");	115	+ contato.setSinEnderecoAssociado(Constantes.SIM);
116		116
117	contato.setIdContatoAssociado(getIdContato(pessoa.getAssociado().getSigla())+"");	117	contato.setIdContatoAssociado(getIdContato(pessoa.getAssociado().getSigla())+"");
118	contato.setNomeContatoAssociado(pessoa.getAssociado().getNome());	118	contato.setNomeContatoAssociado(pessoa.getAssociado().getNome());