Merge pull request 'fixes bug#AL-2057' (!74) from AL-2057 into master

Reviewed-on: http://18.235.188.113:3000/adm/VentaBoletosAdm/pulls/74
Reviewed-by: Gleison da Cruz <gleison.cruz@totvs.com.br>
Reviewed-by: fabio <fabio.faria@rjconsultores.com.br>
master
Lucas Taiã 2023-02-23 14:53:39 +00:00
commit 4d0e38b162
1 changed files with 34 additions and 28 deletions

View File

@ -741,7 +741,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
if (puntoVenta.getPuntoventaId() != null) { if (puntoVenta.getPuntoventaId() != null) {
puntoVenta = puntoVentaService.obtenerID(puntoVenta.getPuntoventaId()); puntoVenta = puntoVentaService.obtenerID(puntoVenta.getPuntoventaId());
//chamadas de inicialização para evitar LazyInitializationException //chamadas de inicialização para evitar LazyInitializationException
puntoVenta.getLsFormaPagoDet().size(); puntoVenta.getLsFormaPagoDet().size();
puntoVenta.getLsParamRecoleccion().size(); puntoVenta.getLsParamRecoleccion().size();
puntoVenta.getLsPtovtaEmpresa().size(); puntoVenta.getLsPtovtaEmpresa().size();
@ -750,7 +750,6 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
puntoVenta.getPtovtaHorarioList().size(); puntoVenta.getPtovtaHorarioList().size();
puntoVenta.getLsPtovtaUsuarioBancario().size(); puntoVenta.getLsPtovtaUsuarioBancario().size();
puntoVentaList = (MyListbox) Executions.getCurrent().getArg().get("puntoVentaList");
lsEmpresaComissao = empresaService.buscarNotInPuntoVtaComissao(puntoVenta); lsEmpresaComissao = empresaService.buscarNotInPuntoVtaComissao(puntoVenta);
@ -784,7 +783,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
} }
}catch (WrongValueException e) { }catch (WrongValueException e) {
numtelefonouno.setValue(""); numtelefonouno.setValue("");
Messagebox.show("Telefone fora do padrão, Preencha novamente", Messagebox.show("Telefone fora do padrão, Preencha novamente",
Labels.getLabel(TITULO), Labels.getLabel(TITULO),
Messagebox.OK, Messagebox.EXCLAMATION); Messagebox.OK, Messagebox.EXCLAMATION);
} }
@ -795,7 +794,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
} }
}catch (WrongValueException e) { }catch (WrongValueException e) {
numtelefonodos.setValue(""); numtelefonodos.setValue("");
Messagebox.show("Celular fora do padrão, Preencha novamente", Messagebox.show("Celular fora do padrão, Preencha novamente",
Labels.getLabel(TITULO), Labels.getLabel(TITULO),
Messagebox.OK, Messagebox.EXCLAMATION); Messagebox.OK, Messagebox.EXCLAMATION);
} }
@ -842,6 +841,8 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
} }
} }
puntoVentaList = (MyListbox) Executions.getCurrent().getArg().get("puntoVentaList");
if (puntoVenta.getPuntoventaId() != null) { if (puntoVenta.getPuntoventaId() != null) {
if (puntoVenta.getStatusaprobacion() != null) { if (puntoVenta.getStatusaprobacion() != null) {
if (puntoVenta.getStatusaprobacion().equals("A")) { if (puntoVenta.getStatusaprobacion().equals("A")) {
@ -1136,8 +1137,8 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
} }
/** /**
* Desabilita todos os campos deixando apenas o status de aprova��o, aberto/fechado. * Desabilita todos os campos deixando apenas o status de aprova��o, aberto/fechado.
* Caso adicionar novos campos, favor tratar neste m�todo * Caso adicionar novos campos, favor tratar neste m�todo
*/ */
private void validaPerfilFuncionBloqueiaDesbloqueiaPontoVenda() { private void validaPerfilFuncionBloqueiaDesbloqueiaPontoVenda() {
if(isPerfilFuncionBloqueiaDesbloqueiaPontoVenda) { if(isPerfilFuncionBloqueiaDesbloqueiaPontoVenda) {
@ -1551,7 +1552,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
// try { // try {
// myConn.close(); // myConn.close();
// } catch (SQLException e) { // } 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(); myConn.close();
} }
} catch (SQLException e) { } catch (SQLException e) {
log.error("error fechar conex�o", e); log.error("error fechar conex�o", e);
throw new RuntimeException(e); throw new RuntimeException(e);
} }
} }
@ -1657,7 +1658,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
try { try {
myConn.close(); myConn.close();
} catch (SQLException e) { } 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<PtovtaEmpresaContaBancaria> excluir = new ArrayList<PtovtaEmpresaContaBancaria>(0); List<PtovtaEmpresaContaBancaria> excluir = new ArrayList<PtovtaEmpresaContaBancaria>(0);
for (PtovtaEmpresaContaBancaria pto : emp.getLsPtovtaEmpresaContaBancaria()) { for (PtovtaEmpresaContaBancaria pto : emp.getLsPtovtaEmpresaContaBancaria()) {
@ -2104,7 +2105,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
puntoVenta.setFecmodif(Calendar.getInstance().getTime()); puntoVenta.setFecmodif(Calendar.getInstance().getTime());
puntoVenta.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); puntoVenta.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId());
if(!isPermissaoMotivoFormaPago) { if(!isPermissaoMotivoFormaPago || puntoVenta.getPuntoventaId()==null) {
puntoVenta.setLsFormaPagoDet(lsFormaPagoDet); puntoVenta.setLsFormaPagoDet(lsFormaPagoDet);
}else { }else {
puntoVenta.setLsFormaPagoDet(formaPagoList.getListData()); puntoVenta.setLsFormaPagoDet(formaPagoList.getListData());
@ -2335,7 +2336,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
for (Object object : fechamentoParamptovtaList.getListData()) { for (Object object : fechamentoParamptovtaList.getListData()) {
encontrouEmpresaComFechamentoConfigurado = false; encontrouEmpresaComFechamentoConfigurado = false;
FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object; 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){ for(FechamentoParamptovta paramFechamento : paramsFechamento){
if(fechamentoParamptovta.getEmpresa().getEmpresaId() == paramFechamento.getEmpresa().getEmpresaId()){ if(fechamentoParamptovta.getEmpresa().getEmpresaId() == paramFechamento.getEmpresa().getEmpresaId()){
encontrouEmpresaComFechamentoConfigurado = true; encontrouEmpresaComFechamentoConfigurado = true;
@ -2346,7 +2347,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
break; break;
} }
} }
//Caso n�o esteja configurada insere no banco //Caso n�o esteja configurada insere no banco
if(!encontrouEmpresaComFechamentoConfigurado){ if(!encontrouEmpresaComFechamentoConfigurado){
fechamentoParamptovta.setPuntoventa(puntoVenta); fechamentoParamptovta.setPuntoventa(puntoVenta);
fechamentoParamptovtaService.suscribir(fechamentoParamptovta); fechamentoParamptovtaService.suscribir(fechamentoParamptovta);
@ -2358,7 +2359,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
boolean removerEmpresa = true; boolean removerEmpresa = true;
for(FechamentoParamptovta paramFechamento : paramsFechamento){ for(FechamentoParamptovta paramFechamento : paramsFechamento){
removerEmpresa = true; 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()) { for (Object object : fechamentoParamptovtaList.getListData()) {
FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object; FechamentoParamptovta fechamentoParamptovta = (FechamentoParamptovta) object;
if(paramFechamento.getEmpresa().getEmpresaId() == fechamentoParamptovta.getEmpresa().getEmpresaId()){ if(paramFechamento.getEmpresa().getEmpresaId() == fechamentoParamptovta.getEmpresa().getEmpresaId()){
@ -2366,7 +2367,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
break; 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){ if(removerEmpresa){
fechamentoParamptovtaService.borrar(paramFechamento); fechamentoParamptovtaService.borrar(paramFechamento);
} }
@ -2385,7 +2386,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
List<Object> empresasFechamento = Arrays.asList(fechamentoParamptovtaList.getData()); List<Object> empresasFechamento = Arrays.asList(fechamentoParamptovtaList.getData());
boolean validado = false; boolean validado = false;
try { 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++) { for (int i = 0; i < empresasVenda.size(); i++) {
PtovtaEmpresa empresaVenda = (PtovtaEmpresa) empresasVenda.get(i); PtovtaEmpresa empresaVenda = (PtovtaEmpresa) empresasVenda.get(i);
@ -2447,7 +2448,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
puntoVentaService.borrar(puntoVenta); 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( Messagebox.show(
Labels.getLabel("editarPuntoVentaController.MSG.borrarOK"), Labels.getLabel("editarPuntoVentaController.MSG.borrarOK"),
@ -2675,7 +2676,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
fpd.setActivo(Boolean.TRUE); fpd.setActivo(Boolean.TRUE);
fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId());
fpd.setFecmodif(Calendar.getInstance().getTime()); fpd.setFecmodif(Calendar.getInstance().getTime());
if(isPermissaoMotivoFormaPago) { if(isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago()) {
abrirMotivoFormaPagamento(fpd, Boolean.TRUE, ((Empresa) cmbEmpresaFormaPago.getSelectedItem().getValue()).getEmpresaId()) ; abrirMotivoFormaPagamento(fpd, Boolean.TRUE, ((Empresa) cmbEmpresaFormaPago.getSelectedItem().getValue()).getEmpresaId()) ;
}else { }else {
lsFormaPagoDet.add(fpd); lsFormaPagoDet.add(fpd);
@ -2716,7 +2717,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
fpd.setActivo(Boolean.FALSE); fpd.setActivo(Boolean.FALSE);
fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId()); fpd.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId());
fpd.setFecmodif(Calendar.getInstance().getTime()); fpd.setFecmodif(Calendar.getInstance().getTime());
if(isPermissaoMotivoFormaPago) { if(isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago()) {
abrirMotivoFormaPagamento(fpd, Boolean.FALSE, null) ; abrirMotivoFormaPagamento(fpd, Boolean.FALSE, null) ;
}else { }else {
formaPagoList.removeItem(fpd); formaPagoList.removeItem(fpd);
@ -3779,7 +3780,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
private boolean validaFechamentoParamptovta(FechamentoParamptovta fechamentoParamptovta) { private boolean validaFechamentoParamptovta(FechamentoParamptovta fechamentoParamptovta) {
boolean validado = true; boolean validado = true;
try { try {
// Valida Empresa Fechamento j� cadastrada. // Valida Empresa Fechamento j� cadastrada.
Object[] params = fechamentoParamptovtaList.getData(); Object[] params = fechamentoParamptovtaList.getData();
for (Object objectData : params) { for (Object objectData : params) {
@ -3986,7 +3987,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
} }
} catch (Exception e) { } 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){ } 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; 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() if(!( radioSegunda.isSelected() || radioTerca.isSelected() || radioQuarta.isSelected()
|| radioQuinta.isSelected() || radioSexta.isSelected())) { || radioQuinta.isSelected() || radioSexta.isSelected())) {
Messagebox.show(Labels.getLabel("editarPuntoVentaController.erroSemFrequencia.value"), Messagebox.show(Labels.getLabel("editarPuntoVentaController.erroSemFrequencia.value"),
@ -4564,7 +4565,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
return false; return false;
} }
} catch (Exception e) { } 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; return true;
@ -4573,7 +4574,7 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
public void onClick$btnApagarCobrancaAdicional(Event event) { public void onClick$btnApagarCobrancaAdicional(Event event) {
try{ try{
int resp = Messagebox.show( int resp = Messagebox.show(
Labels.getLabel("editarPuntoVentaController.MSG.apagarCobran�aAdicional"), Labels.getLabel("editarPuntoVentaController.MSG.apagarCobran�aAdicional"),
Labels.getLabel(TITULO), Labels.getLabel(TITULO),
Messagebox.YES | Messagebox.NO, Messagebox.QUESTION); Messagebox.YES | Messagebox.NO, Messagebox.QUESTION);
@ -4814,5 +4815,10 @@ public class EditarPuntoVentaController extends MyGenericForwardComposer {
return value; return value;
} }
} }
private Boolean isEdicaoDoPuntoVentaEisPermissaoMotivoFormaPago() {
if(puntoVenta.getPuntoventaId()!=null && isPermissaoMotivoFormaPago) {
return true;
}
return false;
}
} }