From d0f10c832bb6ef2ebb8f2069a2800938ec3a2dc9 Mon Sep 17 00:00:00 2001 From: "gleison.cruz" Date: Mon, 3 Apr 2023 14:50:35 -0300 Subject: [PATCH] fixes bug#AL-2449 --- pom.xml | 2 +- .../catalogos/EditarPuntoVentaController.java | 50 ++++++++++--------- web/gui/catalogos/editarPuntoVenta.zul | 3 +- 3 files changed, 29 insertions(+), 26 deletions(-) diff --git a/pom.xml b/pom.xml index 5516082d7..20e44981d 100644 --- a/pom.xml +++ b/pom.xml @@ -4,7 +4,7 @@ 4.0.0 br.com.rjconsultores ventaboletosadm - 1.0.92 + 1.0.93 war diff --git a/src/java/com/rjconsultores/ventaboletos/web/gui/controladores/catalogos/EditarPuntoVentaController.java b/src/java/com/rjconsultores/ventaboletos/web/gui/controladores/catalogos/EditarPuntoVentaController.java index 1e03a85bf..f15679d51 100644 --- a/src/java/com/rjconsultores/ventaboletos/web/gui/controladores/catalogos/EditarPuntoVentaController.java +++ b/src/java/com/rjconsultores/ventaboletos/web/gui/controladores/catalogos/EditarPuntoVentaController.java @@ -744,7 +744,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { if (puntoVenta.getPuntoventaId() != null) { puntoVenta = puntoVentaService.obtenerID(puntoVenta.getPuntoventaId()); - //chamadas de inicialização para evitar LazyInitializationException + //chamadas de inicialização para evitar LazyInitializationException puntoVenta.getLsFormaPagoDet().size(); puntoVenta.getLsParamRecoleccion().size(); puntoVenta.getLsPtovtaEmpresa().size(); @@ -786,7 +786,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } }catch (WrongValueException e) { numtelefonouno.setValue(""); - Messagebox.show("Telefone fora do padrão, Preencha novamente", + Messagebox.show("Telefone fora do padrão, Preencha novamente", Labels.getLabel(TITULO), Messagebox.OK, Messagebox.EXCLAMATION); } @@ -797,7 +797,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } }catch (WrongValueException e) { numtelefonodos.setValue(""); - Messagebox.show("Celular fora do padrão, Preencha novamente", + Messagebox.show("Celular fora do padrão, Preencha novamente", Labels.getLabel(TITULO), Messagebox.OK, Messagebox.EXCLAMATION); } @@ -1140,8 +1140,8 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } /** - * Desabilita todos os campos deixando apenas o status de aprova��o, aberto/fechado. - * Caso adicionar novos campos, favor tratar neste m�todo + * Desabilita todos os campos deixando apenas o status de aprovação, aberto/fechado. + * Caso adicionar novos campos, favor tratar neste método */ private void validaPerfilFuncionBloqueiaDesbloqueiaPontoVenda() { if(isPerfilFuncionBloqueiaDesbloqueiaPontoVenda) { @@ -1557,7 +1557,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { // try { // myConn.close(); // } catch (SQLException e) { - // log.error("Erro ao fechar a conex�o", e); + // log.error("Erro ao fechar a conexão", e); // } // } } @@ -1612,7 +1612,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { myConn.close(); } } catch (SQLException e) { - log.error("error fechar conex�o", e); + log.error("error fechar conexão", e); throw new RuntimeException(e); } } @@ -1663,7 +1663,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { try { myConn.close(); } catch (SQLException e) { - log.error("Erro ao fechar a conex�o", e); + log.error("Erro ao fechar a conexão", e); } } } @@ -1959,8 +1959,10 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } else { puntoVenta.setTitularId(null); } - - validarDiversos(); + + /*Comentanddo por que o cliente que pediu não existe mais + validarDiversos(); + */ String numagencia = txtNumAg.getValue() + (StringUtils.isBlank(txtDigitoAg.getValue()) ? "" : "-" + txtDigitoAg.getValue()); @@ -2026,7 +2028,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { /* * } else if (puntoVenta.getNodo() == null) { Messagebox.show(Labels.getLabel("editarPuntoVentaController.MSG.nodo"), Labels.getLabel(TITULO), Messagebox.OK, Messagebox.EXCLAMATION); */ - } else if (lsFormaPagoDet.isEmpty()) { + } else if ((lsFormaPagoDet == null) || (lsFormaPagoDet.isEmpty())) { Messagebox.show(Labels.getLabel("editarPuntoVentaController.MSG.formaDePago"), Labels.getLabel(TITULO), Messagebox.OK, Messagebox.EXCLAMATION); @@ -2082,7 +2084,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } } - // Contas bancárias removidas + // Contas bancárias removidas List excluir = new ArrayList(0); for (PtovtaEmpresaContaBancaria pto : emp.getLsPtovtaEmpresaContaBancaria()) { @@ -2341,7 +2343,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { for (Object object : fechamentoParamptovtaList.getListData()) { encontrouEmpresaComFechamentoConfigurado = false; FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object; - //verifica se empresa j� est� configurada com fechamento de conta corrente + //verifica se empresa já está configurada com fechamento de conta corrente for(FechamentoParamptovta paramFechamento : paramsFechamento){ if(fechamentoParamptovta.getEmpresa().getEmpresaId() == paramFechamento.getEmpresa().getEmpresaId()){ encontrouEmpresaComFechamentoConfigurado = true; @@ -2352,7 +2354,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { break; } } - //Caso n�o esteja configurada insere no banco + //Caso nao esteja configurada insere no banco if(!encontrouEmpresaComFechamentoConfigurado){ fechamentoParamptovta.setPuntoventa(puntoVenta); fechamentoParamptovtaService.suscribir(fechamentoParamptovta); @@ -2364,7 +2366,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { boolean removerEmpresa = true; for(FechamentoParamptovta paramFechamento : paramsFechamento){ removerEmpresa = true; - //Procura por empresas que est�o no banco mas n�o est�o na lista + //Procura por empresas que estao no banco mas nao estao na lista for (Object object : fechamentoParamptovtaList.getListData()) { FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object; if(paramFechamento.getEmpresa().getEmpresaId() == fechamentoParamptovta.getEmpresa().getEmpresaId()){ @@ -2372,7 +2374,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { break; } } - //Remove do banco a empresa que n�o est� na lista + //Remove do banco a empresa que nao estao na lista if(removerEmpresa){ fechamentoParamptovtaService.borrar(paramFechamento); } @@ -2391,7 +2393,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { List empresasFechamento = Arrays.asList(fechamentoParamptovtaList.getData()); boolean validado = false; try { - //Valida que todas empresas de Fechamento s�o empresa para venda. + //Valida que todas empresas de Fechamento sao empresa para venda. for (int i = 0; i < empresasVenda.size(); i++) { PtovtaEmpresa empresaVenda = (PtovtaEmpresa) empresasVenda.get(i); @@ -2454,7 +2456,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { puntoVentaService.borrar(puntoVenta); - log.warn("Usuário " + UsuarioLogado.getUsuarioLogado().getNombusuario() + " excluiu o ponto de venda " + puntoVenta.getPuntoventaId()); + log.warn("Usuário " + UsuarioLogado.getUsuarioLogado().getNombusuario() + " excluiu o ponto de venda " + puntoVenta.getPuntoventaId()); Messagebox.show( Labels.getLabel("editarPuntoVentaController.MSG.borrarOK"), @@ -3786,7 +3788,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { private boolean validaFechamentoParamptovta(FechamentoParamptovta fechamentoParamptovta) { boolean validado = true; try { - // Valida Empresa Fechamento j� cadastrada. + // Valida Empresa Fechamento ja cadastrada. Object[] params = fechamentoParamptovtaList.getData(); for (Object objectData : params) { @@ -3993,7 +3995,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } } catch (Exception e) { - log.error("Erro ao adicionar Exce��o Multa Cancelamento", e); + log.error("Erro ao adicionar Excecao Multa Cancelamento", e); } } @@ -4111,7 +4113,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } } } catch (Exception e){ - log.error("Erro ao processar remo��o Exce��o Multa Cancelamento", e); + log.error("Erro ao processar remocao Exception Multa Cancelamento", e); } } @@ -4552,7 +4554,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { return false; - } else if(radioSemanal.isSelected()) { //Se escolher semana e n�o escolher um dia + } else if(radioSemanal.isSelected()) { //Se escolher semana e nao escolher um dia if(!( radioSegunda.isSelected() || radioTerca.isSelected() || radioQuarta.isSelected() || radioQuinta.isSelected() || radioSexta.isSelected())) { Messagebox.show(Labels.getLabel("editarPuntoVentaController.erroSemFrequencia.value"), @@ -4571,7 +4573,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { return false; } } catch (Exception e) { - log.error("Erro ao validar a inser��o da cobranca adicional no ponto de venda: ", e); + log.error("Erro ao validar a inserção da cobranca adicional no ponto de venda: ", e); } return true; @@ -4580,7 +4582,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { public void onClick$btnApagarCobrancaAdicional(Event event) { try{ int resp = Messagebox.show( - Labels.getLabel("editarPuntoVentaController.MSG.apagarCobran�aAdicional"), + Labels.getLabel("editarPuntoVentaController.MSG.apagarCobran�aAdicional"), Labels.getLabel(TITULO), Messagebox.YES | Messagebox.NO, Messagebox.QUESTION); diff --git a/web/gui/catalogos/editarPuntoVenta.zul b/web/gui/catalogos/editarPuntoVenta.zul index 1335fd640..f286cabb6 100644 --- a/web/gui/catalogos/editarPuntoVenta.zul +++ b/web/gui/catalogos/editarPuntoVenta.zul @@ -50,7 +50,8 @@ + label="${c:l('editarPuntoVentaController.tab.label.diversos')}" + visible="false" />