From ef856425f34f777e4909daa6257e515145cbbe4b Mon Sep 17 00:00:00 2001
From: erick.sato <erick.sato@centralit.com.br>
Date: Mon, 23 May 2016 15:28:55 -0300
Subject: [PATCH] [Redmine #4685]Correção find bugs

---
 citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java          | 14 +++++++-------
 citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java |  2 +-
 citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java         |  3 ++-
 citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java           |  4 ++--
 4 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java
index 9f0b89e..6e526e6 100644
--- a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java
+++ b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java
@@ -649,7 +649,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 		}
 
 		// Verifica se é uma alteração pelo inventário
-		if (UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && alteracaoBemVH.getDominioTipoAlteracaoIndividual().getCodigo().equals(Dominio.TIPO_ALTERACAO_BEM_PATRIMONIAL_INDIVIDUAL_INVENTARIO)) {
+		if (UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && alteracaoBemVH.getDominioTipoAlteracaoIndividual().getCodigo().equals(Dominio.TIPO_ALTERACAO_BEM_PATRIMONIAL_INDIVIDUAL_INVENTARIO)) {
 			this.alterarBemPorInventario(alteracaoBemVH);
 		}
 	}
@@ -696,7 +696,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 				BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString());
 				itensAlterados.add(itemValorLiquido);
 
-				if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) {
+				if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
 					Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
 
 					BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
@@ -1078,7 +1078,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 			itensAlterados = new ArrayList<BemPatrimonialAlteracaoItem>();
 		}
 
-		if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) {
+		if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
 			Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
 
 			BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
@@ -1141,13 +1141,13 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 						caracAnterior.setValorMaterialCaracteristica(caracNova.getValorMaterialCaracteristica());
 					}
 
-					if (caracAnterior.getArquivo() != null && !caracAnterior.getArquivo().equals(caracNova.getArquivo())) {
+					if (caracAnterior.getArquivo() != null && !java.util.Arrays.equals(caracAnterior.getArquivo(),caracNova.getArquivo())) {
 						BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo());
 						if (item != null) {
 							itensCarac.add(item);
 						}
 						caracAnterior.setArquivo(caracNova.getArquivo());
-					} else if (caracNova.getArquivo() != null && !caracNova.getArquivo().equals(caracAnterior.getArquivo())) {
+					} else if (caracNova.getArquivo() != null && !java.util.Arrays.equals(caracNova.getArquivo(),caracAnterior.getArquivo())) {
 						BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo());
 						if (item != null) {
 							itensCarac.add(item);
@@ -1377,7 +1377,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 
 					bemPatrimonial.setNumeroPatrimonial(novoNumeroPatrimonial);
 					for (BemPatrimonialAlteracaoItem bemPatrimonialAlteracaoItem : itensAlterados) {
-						if(bemPatrimonialAlteracaoItem.getBemPatrimonial() != null && bemPatrimonialAlteracaoItem.getBemPatrimonial().getId() == bemPatrimonial.getId()){
+						if(bemPatrimonialAlteracaoItem.getBemPatrimonial() != null && bemPatrimonialAlteracaoItem.getBemPatrimonial().getId().doubleValue() == bemPatrimonial.getId().doubleValue()){
 							bemPatrimonialAlteracaoItem.setBemPatrimonial(bemPatrimonial);
 						}
 					}
@@ -1464,7 +1464,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
 				BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString());
 				itensAlterados.add(itemValorLiquido);
 
-				if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) {
+				if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
 					Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
 
 					BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
diff --git a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java
index 893c6c5..ba746f3 100644
--- a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java
+++ b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java
@@ -1416,7 +1416,7 @@ public class HistoricoBemPatrimonialServiceImpl extends GenericServiceImpl<Histo
 
 		historicoBemPrincipal.setDominioClasseReferencia(classeReferencia);
 
-		historicoBemPrincipal.setCodigoOperacao(adicaoBemPrincipal.getCodigo());
+		historicoBemPrincipal.setCodigoOperacao(adicaoBemPrincipal != null ? adicaoBemPrincipal.getCodigo() : null);
 
 		return this.save(historicoBemPrincipal);
 
diff --git a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java
index 1946ac8..c87a7c1 100644
--- a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java
+++ b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java
@@ -487,9 +487,10 @@ public class SaidaTemporariaServiceImpl extends GenericServiceImpl<SaidaTemporar
 			for (Iterator<SaidaTemporariaItem> iterator = saida.getSaidaTemporariaItens().iterator(); iterator.hasNext();) {
 				this.saidaTemporariaItemService.removeById(iterator.next().getId());
 			}
+			return super.removeById(saida.getId());
 		}
+		return false;
 
-		return super.removeById(saida.getId());
 	}
 
 	/**
diff --git a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java
index 0b087f9..8e261c5 100644
--- a/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java
+++ b/citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java
@@ -249,7 +249,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl<Transferencia, 
 				} else {
 					boolean salvarNovaTransferencia = true;
 					for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) {
-						if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId() == transferencia.getEstruturaOrganizacionalDestino().getId()){
+						if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId().doubleValue() == transferencia.getEstruturaOrganizacionalDestino().getId().doubleValue()){
 							salvarNovaTransferencia = false;
 							break;
 						}
@@ -271,7 +271,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl<Transferencia, 
 			
 			boolean salvarNovaTransferencia = true;
 			for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) {
-				if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId() == transferencia.getEstruturaOrganizacionalDestino().getId()){
+				if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId().doubleValue() == transferencia.getEstruturaOrganizacionalDestino().getId().doubleValue()){
 					salvarNovaTransferencia = false;
 					break;
 				}
--
libgit2 0.21.2