From 93c3039e206dfc8012d9d15e0d7d29bdddde8a76 Mon Sep 17 00:00:00 2001 From: "aristides@rjconsultores.com.br" Date: Fri, 17 Feb 2023 11:28:50 -0300 Subject: [PATCH 1/2] fixes bug#AL-2057 --- pom.xml | 2 +- .../catalogos/EditarPuntoVentaController.java | 62 ++++++++++--------- 2 files changed, 35 insertions(+), 29 deletions(-) diff --git a/pom.xml b/pom.xml index ee9b0a850..83af88142 100644 --- a/pom.xml +++ b/pom.xml @@ -4,7 +4,7 @@ 4.0.0 br.com.rjconsultores ventaboletosadm - 1.0.56 + 1.0.58 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 bdeffa3a0..153ad3d82 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 @@ -741,16 +741,15 @@ 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(); puntoVenta.getPtovtaAntecipacomissaoList().size(); puntoVenta.getPtovtaEstoqueList().size(); puntoVenta.getPtovtaHorarioList().size(); - puntoVenta.getLsPtovtaUsuarioBancario().size(); + puntoVenta.getLsPtovtaUsuarioBancario().size(); - puntoVentaList = (MyListbox) Executions.getCurrent().getArg().get("puntoVentaList"); lsEmpresaComissao = empresaService.buscarNotInPuntoVtaComissao(puntoVenta); @@ -784,7 +783,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); } @@ -795,7 +794,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); } @@ -841,6 +840,8 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { puntoVenta.setTitularId(ptovtaTitularService.obtenerID(puntoVenta.getTitularId().getPtovtaTitularId()));; } } + + puntoVentaList = (MyListbox) Executions.getCurrent().getArg().get("puntoVentaList"); if (puntoVenta.getPuntoventaId() != null) { if (puntoVenta.getStatusaprobacion() != null) { @@ -1136,8 +1137,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) { @@ -1551,7 +1552,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); // } // } } @@ -1606,7 +1607,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); } } @@ -1657,7 +1658,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); } } } @@ -2076,7 +2077,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } } - // Contas bancárias removidas + // Contas bancárias removidas List excluir = new ArrayList(0); for (PtovtaEmpresaContaBancaria pto : emp.getLsPtovtaEmpresaContaBancaria()) { @@ -2104,7 +2105,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { puntoVenta.setFecmodif(Calendar.getInstance().getTime()); puntoVenta.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); - if(!isPermissaoMotivoFormaPago) { + if(!isPermissaoMotivoFormaPago || puntoVenta.getPuntoventaId()==null) { puntoVenta.setLsFormaPagoDet(lsFormaPagoDet); }else { puntoVenta.setLsFormaPagoDet(formaPagoList.getListData()); @@ -2335,7 +2336,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; @@ -2346,7 +2347,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { break; } } - //Caso n�o esteja configurada insere no banco + //Caso n�o esteja configurada insere no banco if(!encontrouEmpresaComFechamentoConfigurado){ fechamentoParamptovta.setPuntoventa(puntoVenta); fechamentoParamptovtaService.suscribir(fechamentoParamptovta); @@ -2358,7 +2359,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 est�o no banco mas n�o est�o na lista for (Object object : fechamentoParamptovtaList.getListData()) { FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object; if(paramFechamento.getEmpresa().getEmpresaId() == fechamentoParamptovta.getEmpresa().getEmpresaId()){ @@ -2366,7 +2367,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { break; } } - //Remove do banco a empresa que n�o est� na lista + //Remove do banco a empresa que n�o est� na lista if(removerEmpresa){ fechamentoParamptovtaService.borrar(paramFechamento); } @@ -2385,7 +2386,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 s�o empresa para venda. for (int i = 0; i < empresasVenda.size(); i++) { PtovtaEmpresa empresaVenda = (PtovtaEmpresa) empresasVenda.get(i); @@ -2447,8 +2448,8 @@ 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"), Labels.getLabel(TITULO), @@ -2675,7 +2676,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { fpd.setActivo(Boolean.TRUE); fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); fpd.setFecmodif(Calendar.getInstance().getTime()); - if(isPermissaoMotivoFormaPago) { + if(isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago()) { abrirMotivoFormaPagamento(fpd, Boolean.TRUE, ((Empresa) cmbEmpresaFormaPago.getSelectedItem().getValue()).getEmpresaId()) ; }else { lsFormaPagoDet.add(fpd); @@ -2716,7 +2717,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { fpd.setActivo(Boolean.FALSE); fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); fpd.setFecmodif(Calendar.getInstance().getTime()); - if(isPermissaoMotivoFormaPago) { + if(isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago()) { abrirMotivoFormaPagamento(fpd, Boolean.FALSE, null) ; }else { formaPagoList.removeItem(fpd); @@ -3779,7 +3780,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { private boolean validaFechamentoParamptovta(FechamentoParamptovta fechamentoParamptovta) { boolean validado = true; try { - // Valida Empresa Fechamento j� cadastrada. + // Valida Empresa Fechamento j� cadastrada. Object[] params = fechamentoParamptovtaList.getData(); for (Object objectData : params) { @@ -3986,7 +3987,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { } } catch (Exception e) { - log.error("Erro ao adicionar Exce��o Multa Cancelamento", e); + log.error("Erro ao adicionar Exce��o Multa Cancelamento", e); } } @@ -4104,7 +4105,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 remo��o Exce��o Multa Cancelamento", e); } } @@ -4545,7 +4546,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 n�o escolher um dia if(!( radioSegunda.isSelected() || radioTerca.isSelected() || radioQuarta.isSelected() || radioQuinta.isSelected() || radioSexta.isSelected())) { Messagebox.show(Labels.getLabel("editarPuntoVentaController.erroSemFrequencia.value"), @@ -4564,7 +4565,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; @@ -4573,7 +4574,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); @@ -4814,5 +4815,10 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer { return value; } } - + private Boolean isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago() { + if(puntoVenta.getPuntoventaId()!=null && isPermissaoMotivoFormaPago) { + return true; + } + return false; + } } \ No newline at end of file From 56af5d1cbbd195b7fe69bed1f1949e34e79986d7 Mon Sep 17 00:00:00 2001 From: "aristides@rjconsultores.com.br" Date: Fri, 17 Feb 2023 14:31:47 -0300 Subject: [PATCH 2/2] fixes bug#AL-2057 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 5583e23a7..83af88142 100644 --- a/pom.xml +++ b/pom.xml @@ -4,7 +4,7 @@ 4.0.0 br.com.rjconsultores ventaboletosadm - 1.0.57 + 1.0.58 war