Download as
Browse Code »

Commit ef856425f34f777e4909daa6257e515145cbbe4b

Authored by erick.sato
1 parent b05128c5
Exists in master

[Redmine #4685]Correção find bugs

Showing 4 changed files with 12 additions and 11 deletions   Show diff stats
citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/BemPatrimonialServiceImpl.java
  Diff comments   View file @ef85642
@@ -649,7 +649,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial @@ -649,7 +649,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
649 } 649 }
650 650
651 // Verifica se é uma alteração pelo inventário 651 // Verifica se é uma alteração pelo inventário
652 - if (UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && alteracaoBemVH.getDominioTipoAlteracaoIndividual().getCodigo().equals(Dominio.TIPO_ALTERACAO_BEM_PATRIMONIAL_INDIVIDUAL_INVENTARIO)) { 652 + if (UtilObjeto.isReferencia(alteracaoBemVH.getDominioTipoAlteracaoIndividual()) && alteracaoBemVH.getDominioTipoAlteracaoIndividual().getCodigo().equals(Dominio.TIPO_ALTERACAO_BEM_PATRIMONIAL_INDIVIDUAL_INVENTARIO)) {
653 this.alterarBemPorInventario(alteracaoBemVH); 653 this.alterarBemPorInventario(alteracaoBemVH);
654 } 654 }
655 } 655 }
@@ -696,7 +696,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial @@ -696,7 +696,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
696 BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString()); 696 BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString());
697 itensAlterados.add(itemValorLiquido); 697 itensAlterados.add(itemValorLiquido);
698 698
699 - if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) { 699 + if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
700 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO); 700 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
701 701
702 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString()); 702 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
@@ -1078,7 +1078,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial @@ -1078,7 +1078,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl<BemPatrimonial
1078 itensAlterados = new ArrayList<BemPatrimonialAlteracaoItem>(); 1078 itensAlterados = new ArrayList<BemPatrimonialAlteracaoItem>();
1079 } 1079 }
1080 1080
1081 - if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) { 1081 + if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
1082 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO); 1082 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
1083 1083
1084 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString()); 1084 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
@@ -1141,13 +1141,13 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial @@ -1141,13 +1141,13 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial
1141 caracAnterior.setValorMaterialCaracteristica(caracNova.getValorMaterialCaracteristica()); 1141 caracAnterior.setValorMaterialCaracteristica(caracNova.getValorMaterialCaracteristica());
1142 } 1142 }
1143 1143
1144 - if (caracAnterior.getArquivo() != null && !caracAnterior.getArquivo().equals(caracNova.getArquivo())) { 1144 + if (caracAnterior.getArquivo() != null && !java.util.Arrays.equals(caracAnterior.getArquivo(),caracNova.getArquivo())) {
1145 BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo()); 1145 BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo());
1146 if (item != null) { 1146 if (item != null) {
1147 itensCarac.add(item); 1147 itensCarac.add(item);
1148 } 1148 }
1149 caracAnterior.setArquivo(caracNova.getArquivo()); 1149 caracAnterior.setArquivo(caracNova.getArquivo());
1150 - } else if (caracNova.getArquivo() != null && !caracNova.getArquivo().equals(caracAnterior.getArquivo())) { 1150 + } else if (caracNova.getArquivo() != null && !java.util.Arrays.equals(caracNova.getArquivo(),caracAnterior.getArquivo())) {
1151 BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo()); 1151 BemPatrimonialAlteracaoItem item = new BemPatrimonialAlteracaoItem(campoCarac, caracDescricao, caracAnterior.getArquivo(), caracNova.getArquivo());
1152 if (item != null) { 1152 if (item != null) {
1153 itensCarac.add(item); 1153 itensCarac.add(item);
@@ -1377,7 +1377,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial @@ -1377,7 +1377,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial
1377 1377
1378 bemPatrimonial.setNumeroPatrimonial(novoNumeroPatrimonial); 1378 bemPatrimonial.setNumeroPatrimonial(novoNumeroPatrimonial);
1379 for (BemPatrimonialAlteracaoItem bemPatrimonialAlteracaoItem : itensAlterados) { 1379 for (BemPatrimonialAlteracaoItem bemPatrimonialAlteracaoItem : itensAlterados) {
1380 - if(bemPatrimonialAlteracaoItem.getBemPatrimonial() != null && bemPatrimonialAlteracaoItem.getBemPatrimonial().getId() == bemPatrimonial.getId()){ 1380 + if(bemPatrimonialAlteracaoItem.getBemPatrimonial() != null && bemPatrimonialAlteracaoItem.getBemPatrimonial().getId().doubleValue() == bemPatrimonial.getId().doubleValue()){
1381 bemPatrimonialAlteracaoItem.setBemPatrimonial(bemPatrimonial); 1381 bemPatrimonialAlteracaoItem.setBemPatrimonial(bemPatrimonial);
1382 } 1382 }
1383 } 1383 }
@@ -1464,7 +1464,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial @@ -1464,7 +1464,7 @@ public class BemPatrimonialServiceImpl extends GenericServiceImpl&lt;BemPatrimonial
1464 BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString()); 1464 BemPatrimonialAlteracaoItem itemValorLiquido = new BemPatrimonialAlteracaoItem(campoValorLiquido, bemPatrimonial.getValorLiquido().toString(), alteracaoBemVH.getNovoValorBruto().toString());
1465 itensAlterados.add(itemValorLiquido); 1465 itensAlterados.add(itemValorLiquido);
1466 1466
1467 - if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante() != alteracaoBemVH.getVidaUtil() )) { 1467 + if (alteracaoBemVH.getVidaUtil() != null && ( bemPatrimonial.getVidaUtilRestante().intValue() != alteracaoBemVH.getVidaUtil().intValue() )) {
1468 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO); 1468 Dominio campoVidaUtilRestante = this.dominioService.findByChaveAndCodigo(DominioMaterial.CAMPO_REFERENCIA, DominioMaterial.CAMPO_REFERENCIA_BEM_PATRIMONIAL_VIDA_UTIL_RESTANTE_CODIGO);
1469 1469
1470 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString()); 1470 BemPatrimonialAlteracaoItem itemVidaUtil = new BemPatrimonialAlteracaoItem(campoVidaUtilRestante, bemPatrimonial.getVidaUtilRestante().toString(), alteracaoBemVH.getVidaUtil().toString());
citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/HistoricoBemPatrimonialServiceImpl.java
  Diff comments   View file @ef85642
@@ -1416,7 +1416,7 @@ public class HistoricoBemPatrimonialServiceImpl extends GenericServiceImpl&lt;Histo @@ -1416,7 +1416,7 @@ public class HistoricoBemPatrimonialServiceImpl extends GenericServiceImpl&lt;Histo
1416 1416
1417 historicoBemPrincipal.setDominioClasseReferencia(classeReferencia); 1417 historicoBemPrincipal.setDominioClasseReferencia(classeReferencia);
1418 1418
1419 - historicoBemPrincipal.setCodigoOperacao(adicaoBemPrincipal.getCodigo()); 1419 + historicoBemPrincipal.setCodigoOperacao(adicaoBemPrincipal != null ? adicaoBemPrincipal.getCodigo() : null);
1420 1420
1421 return this.save(historicoBemPrincipal); 1421 return this.save(historicoBemPrincipal);
1422 1422
citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/SaidaTemporariaServiceImpl.java
  Diff comments   View file @ef85642
@@ -487,9 +487,10 @@ public class SaidaTemporariaServiceImpl extends GenericServiceImpl&lt;SaidaTemporar @@ -487,9 +487,10 @@ public class SaidaTemporariaServiceImpl extends GenericServiceImpl&lt;SaidaTemporar
487 for (Iterator<SaidaTemporariaItem> iterator = saida.getSaidaTemporariaItens().iterator(); iterator.hasNext();) { 487 for (Iterator<SaidaTemporariaItem> iterator = saida.getSaidaTemporariaItens().iterator(); iterator.hasNext();) {
488 this.saidaTemporariaItemService.removeById(iterator.next().getId()); 488 this.saidaTemporariaItemService.removeById(iterator.next().getId());
489 } 489 }
  490 + return super.removeById(saida.getId());
490 } 491 }
  492 + return false;
491 493
492 - return super.removeById(saida.getId());  
493 } 494 }
494 495
495 /** 496 /**
citgrp-patrimonio-api/src/main/java/br/com/centralit/api/service/impl/TransferenciaServiceImpl.java
  Diff comments   View file @ef85642
@@ -249,7 +249,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl&lt;Transferencia, @@ -249,7 +249,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl&lt;Transferencia,
249 } else { 249 } else {
250 boolean salvarNovaTransferencia = true; 250 boolean salvarNovaTransferencia = true;
251 for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) { 251 for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) {
252 - if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId() == transferencia.getEstruturaOrganizacionalDestino().getId()){ 252 + if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId().doubleValue() == transferencia.getEstruturaOrganizacionalDestino().getId().doubleValue()){
253 salvarNovaTransferencia = false; 253 salvarNovaTransferencia = false;
254 break; 254 break;
255 } 255 }
@@ -271,7 +271,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl&lt;Transferencia, @@ -271,7 +271,7 @@ public class TransferenciaServiceImpl extends GenericServiceImpl&lt;Transferencia,
271 271
272 boolean salvarNovaTransferencia = true; 272 boolean salvarNovaTransferencia = true;
273 for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) { 273 for (TransferenciaItem transferenciaItemTemp : transferenciaItemListTemp) {
274 - if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId() == transferencia.getEstruturaOrganizacionalDestino().getId()){ 274 + if(transferenciaItemTemp.getEstruturaOrganizacionalOrigem().getId().doubleValue() == transferencia.getEstruturaOrganizacionalDestino().getId().doubleValue()){
275 salvarNovaTransferencia = false; 275 salvarNovaTransferencia = false;
276 break; 276 break;
277 } 277 }