fixes bug #7439

git-svn-id: http://desenvolvimento.rjconsultores.com.br/repositorio/sco/AdmVenta/Web/trunk/ventaboletos@56482 d1611594-4594-4d17-8e1d-87c2c4800839

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/gr/EditarAidfController.java

@@ -37,7 +37,6 @@ import com.rjconsultores.ventaboletos.service.AidfService;
 import com.rjconsultores.ventaboletos.service.DetAbastoBoletoService;
 import com.rjconsultores.ventaboletos.service.EmpresaService;
 import com.rjconsultores.ventaboletos.service.EstadoService;
-import com.rjconsultores.ventaboletos.service.MovimentacionBilhetesService;
 import com.rjconsultores.ventaboletos.utilerias.UsuarioLogado;
 import com.rjconsultores.ventaboletos.web.utilerias.MyGenericForwardComposer;
 import com.rjconsultores.ventaboletos.web.utilerias.MyListbox;
@@ -63,9 +62,6 @@ public class EditarAidfController extends MyGenericForwardComposer {
 	@Autowired
 	private EstadoService estadoService;
 
-	@Autowired
-	private MovimentacionBilhetesService movimentacionBilhetesService; 
-
 	@Autowired
 	private EmpresaService empresaService;
 
@@ -84,7 +80,6 @@ public class EditarAidfController extends MyGenericForwardComposer {
 	private Combobox cmbEmpresa;
 	private List<Empresa> lsEmpresas;
 
-//	private Textbox txtInscestadual;
 	private Combobox cmbInscestadual;
 	private List<InscricaoEstadual> lsInscestadual;
 
@@ -96,7 +91,6 @@ public class EditarAidfController extends MyGenericForwardComposer {
 	private Textbox txtFormFinal;
 	private Textbox txtFecAquisicao;
 	private Textbox txtFecVencimento;
-	private Textbox txtCnpj;
 
 	private Textbox txtForminicial;
 	private Textbox txtFormfinal;
@@ -110,7 +104,6 @@ public class EditarAidfController extends MyGenericForwardComposer {
 
 	private static Logger log = Logger.getLogger(EditarAidfController.class);
 
-	
 	@Autowired
 	private DataSource dataSourceRead;
 
@@ -134,8 +127,8 @@ public class EditarAidfController extends MyGenericForwardComposer {
 			btnApagar.setVisible(Boolean.FALSE);
 		}
 
-		if(aidf.getAidfId()!= null){
+		if (aidf.getAidfId() != null) {
-			if(detAbastoBoletoService.buscabDetAbastoBoletoToVerifyIfAidfCanBeEditable(aidf).size() >0){
+			if (detAbastoBoletoService.buscabDetAbastoBoletoToVerifyIfAidfCanBeEditable(aidf).size() > 0) {
 				Messagebox.show(
 						Labels.getLabel("editarAidfController.MSG.noWrite"),
 						Labels.getLabel("editarAidfController.window.title"),
@@ -147,6 +140,9 @@ public class EditarAidfController extends MyGenericForwardComposer {
 	}
 
 	public void onClick$btnApagar(Event ev) throws InterruptedException {
+
+		try {
+
 			int opcao = Messagebox.show(
 					Labels.getLabel("editarAidfController.btnApagar.MSG.Deseja.Borrar"),
 					Labels.getLabel("editarAidfController.window.title"),
@@ -169,6 +165,16 @@ public class EditarAidfController extends MyGenericForwardComposer {
 						Messagebox.OK, Messagebox.INFORMATION);
 				closeWindow();
 			}
+
+		} catch (Exception ex) {
+			log.error("", ex);
+
+			Messagebox.show(
+					Labels.getLabel("MSG.Error"),
+					Labels.getLabel("editarAidfController.window.title"),
+					Messagebox.OK, Messagebox.EXCLAMATION);
+		}
+
 	}
 
 	public void onClick$btnSalvar(Event ev) throws InterruptedException {
@@ -182,33 +188,24 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		txtForminicial.getValue();
 		txtFormfinal.getValue();
 		txtDocFiscal.getValue();
+		cmbTipoValidaVenta.getValue();
+
+		try {
 
 			Comboitem inscEstadualSeleccionada = cmbInscestadual.getSelectedItem();
-		String _inscestadual = inscEstadualSeleccionada != null ? inscEstadualSeleccionada.getValue().toString() : "";
+			aidf.setInscestadual(inscEstadualSeleccionada != null ? inscEstadualSeleccionada.getValue().toString() : "");
-		aidf.setInscestadual(_inscestadual);
 
-		Long x ;
+			Long idIdf = aidfService.validarDocFiscalSerieFormInicialFinal(aidf, dataSourceRead.getConnection());
-		try {
+
-			x = aidfService.validarDocFiscalSerieFormInicialFinal(aidf, dataSourceRead.getConnection());
+			if (idIdf != -1) {
-			if(x != -1){
 				Messagebox.show(
-	                    Labels.getLabel("editarAidfController.btnSalvar.MSG.conflitoFormIniFin", new Object[] { x }),
+						Labels.getLabel("editarAidfController.btnSalvar.MSG.conflitoFormIniFin", new Object[] { idIdf }),
 						Labels.getLabel("editarAidfController.window.title"),
 						Messagebox.OK, Messagebox.EXCLAMATION);
 				return;
 			}
-		} catch (SQLException e) {
 
-			Messagebox.show(
+			if (!validarDocFiscalEstado(aidf.getAidfId(), txtDocFiscal.getValue(), aidf.getEstado())) {
-                    Labels.getLabel("MSG.Error"),
-                    Labels.getLabel("editarAidfController.window.title"),
-                    Messagebox.OK, Messagebox.EXCLAMATION);
-			
-			log.error("",e);
-			
-		}
-		
-		if(!validarDocFiscalEstado(aidf.getAidfId(), txtDocFiscal.getValue(), aidf.getEstado())){
 				Messagebox.show(
 						Labels.getLabel("editarAidfController.btnSalvar.MSG.conflitoDocFiscalEstado"),
 						Labels.getLabel("editarAidfController.window.title"),
@@ -224,7 +221,7 @@ public class EditarAidfController extends MyGenericForwardComposer {
 				return;
 			}
 
-		if(!validarVendaManual(aidf.getAidfTipo().getAidftipoId(), aidf.getSerie())){
+			if (!validarVendaManual(aidf.getAidfTipo().getAidftipoId(), aidf.getSerie())) {
 				Messagebox.show(
 						Labels.getLabel("editarAidfController.btnSalvar.MSG.serieUNICTipoVendaManualNaoPermitido"),
 						Labels.getLabel("editarAidfController.window.title"),
@@ -236,7 +233,7 @@ public class EditarAidfController extends MyGenericForwardComposer {
 			aidf.setFecmodif(Calendar.getInstance().getTime());
 			aidf.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId());
 
-		if(aidf.getAidfId() == null){
+			if (aidf.getAidfId() == null) {
 				aidfService.suscribir(aidf);
 				aidfList.addItemNovo(aidf);
 			} else {
@@ -250,18 +247,36 @@ public class EditarAidfController extends MyGenericForwardComposer {
 					Messagebox.OK, Messagebox.INFORMATION);
 
 			closeWindow();
+
+		} catch (SQLException e) {
+
+			Messagebox.show(
+					Labels.getLabel("MSG.Error"),
+					Labels.getLabel("editarAidfController.window.title"),
+					Messagebox.OK, Messagebox.EXCLAMATION);
+
+			log.error("", e);
+
+		} catch (Exception ex) {
+			log.error("", ex);
+
+			Messagebox.show(
+					Labels.getLabel("MSG.Error"),
+					Labels.getLabel("editarAidfController.window.title"),
+					Messagebox.OK, Messagebox.EXCLAMATION);
 		}
 
+	}
 
-	private Boolean validarDocFiscalEstado(Long idAidf, String docFiscal, Estado estado){
+	private Boolean validarDocFiscalEstado(Long idAidf, String docFiscal, Estado estado) {
 		return aidfService.validarDocFiscalPorEstado(idAidf, docFiscal, estado);
 	}
 
-	private Boolean validarTipoVenda(String tipoVenda, String serie, Integer tipo){
+	private Boolean validarTipoVenda(String tipoVenda, String serie, Integer tipo) {
 		return aidfService.validarTipoVenda(tipoVenda, serie, tipo);
 	}
 
-	private Boolean validarVendaManual(Integer tipo, String serie){
+	private Boolean validarVendaManual(Integer tipo, String serie) {
 		return aidfService.validarVendaManual(tipo, serie);
 	}
 
@@ -276,53 +291,38 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		}
 	}
 
-	public void onBlur$txtSerie(Event ev){
+	public void onBlur$txtSerie(Event ev) {
-		try{
+		try {
 			txtSerie.getValue();
-		}catch(WrongValueException e){
+		} catch (WrongValueException e) {
 			aidf.setSerie("");
 			txtSerie.setValue("");
 		}
 	}
 
-	public Combobox getCmbInscestadual() {				
+	public class OnlyNumber implements Constraint {
-		return cmbInscestadual;
+		public void validate(Component comp, Object value) throws WrongValueException {
+			if (!value.toString().matches("[0-9]+")) {
+				aidf.setDocfiscal(value.toString());
+				throw new WrongValueException(comp, Labels.getLabel("editarAidfController.btnSalvar.MSG.apenasValoresNumericosPermitidos"));
+			}
+		}
 	}
 
-	public void setCmbInscestadual(Combobox cmbInscestadual) {
+	public Aidf getAidf() {
-		this.cmbInscestadual = cmbInscestadual;
+		return aidf;
 	}
 
-	public List<InscricaoEstadual> getLsInscestadual() {
+	public void setAidf(Aidf aidf) {
-		return lsInscestadual;
+		this.aidf = aidf;
 	}
 
-	public void setLsInscestadual(List<InscricaoEstadual> lsInscestadual) {
+	public MyListbox getAidfList() {
-		this.lsInscestadual = lsInscestadual;
+		return aidfList;
 	}
 
-	public Combobox getCmbTipoValidaVenta() {
+	public void setAidfList(MyListbox aidfList) {
-		return cmbTipoValidaVenta;
+		this.aidfList = aidfList;
-	}
-
-	public void setCmbTipoValidaVenta(Combobox cmbTipoValidaVenta) {
-		this.cmbTipoValidaVenta = cmbTipoValidaVenta;
-	}		
-	
-	public Button getBtnApagar() {
-		return btnApagar;
-	}
-
-	public void setBtnApagar(Button btnApagar) {
-		this.btnApagar = btnApagar;
-	}
-
-	public EstadoService getEstadoService() {
-		return estadoService;
-	}
-
-	public void setEstadoService(EstadoService estadoService) {
-		this.estadoService = estadoService;
 	}
 
 	public List<AidfEspecie> getLsAidfEspecie() {
@@ -333,6 +333,14 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.lsAidfEspecie = lsAidfEspecie;
 	}
 
+	public Combobox getCmbAidfEspecie() {
+		return cmbAidfEspecie;
+	}
+
+	public void setCmbAidfEspecie(Combobox cmbAidfEspecie) {
+		this.cmbAidfEspecie = cmbAidfEspecie;
+	}
+
 	public List<Estado> getLsEstado() {
 		return lsEstado;
 	}
@@ -341,6 +349,14 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.lsEstado = lsEstado;
 	}
 
+	public Combobox getCmbEstado() {
+		return cmbEstado;
+	}
+
+	public void setCmbEstado(Combobox cmbEstado) {
+		this.cmbEstado = cmbEstado;
+	}
+
 	public List<AidfTipo> getLsAidfTipo() {
 		return lsAidfTipo;
 	}
@@ -349,6 +365,14 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.lsAidfTipo = lsAidfTipo;
 	}
 
+	public Combobox getCmbAidfTipo() {
+		return cmbAidfTipo;
+	}
+
+	public void setCmbAidfTipo(Combobox cmbAidfTipo) {
+		this.cmbAidfTipo = cmbAidfTipo;
+	}
+
 	public Combobox getCmbEmpresa() {
 		return cmbEmpresa;
 	}
@@ -365,6 +389,22 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.lsEmpresas = lsEmpresas;
 	}
 
+	public Combobox getCmbInscestadual() {
+		return cmbInscestadual;
+	}
+
+	public void setCmbInscestadual(Combobox cmbInscestadual) {
+		this.cmbInscestadual = cmbInscestadual;
+	}
+
+	public List<InscricaoEstadual> getLsInscestadual() {
+		return lsInscestadual;
+	}
+
+	public void setLsInscestadual(List<InscricaoEstadual> lsInscestadual) {
+		this.lsInscestadual = lsInscestadual;
+	}
+
 	public Textbox getTxtDocFiscal() {
 		return txtDocFiscal;
 	}
@@ -429,36 +469,36 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.txtFecVencimento = txtFecVencimento;
 	}
 
-	public Combobox getCmbAidfTipo() {
+	public Textbox getTxtForminicial() {
-		return cmbAidfTipo;
+		return txtForminicial;
 	}
 
-	public void setCmbAidfTipo(Combobox _cmbAidfTipo) {
+	public void setTxtForminicial(Textbox txtForminicial) {
-		cmbAidfTipo = _cmbAidfTipo;
+		this.txtForminicial = txtForminicial;
 	}
 
-	public Combobox getCmbAidfEspecie() {
+	public Textbox getTxtFormfinal() {
-		return cmbAidfEspecie;
+		return txtFormfinal;
 	}
 
-	public void setCmbAidfEspecie(Combobox _cmbAidfEspecie) {
+	public void setTxtFormfinal(Textbox txtFormfinal) {
-		cmbAidfEspecie = _cmbAidfEspecie;
+		this.txtFormfinal = txtFormfinal;
 	}
 
-	public Combobox getCmbEstado() {
+	public Button getBtnApagar() {
-		return cmbEstado;
+		return btnApagar;
 	}
 
-	public void setCmbEstado(Combobox _cmbEstado) {
+	public void setBtnApagar(Button btnApagar) {
-		cmbEstado = _cmbEstado;
+		this.btnApagar = btnApagar;
 	}
 
-	public Aidf getAidf() {
+	public Button getBtnSalvar() {
-		return aidf;
+		return btnSalvar;
 	}
 
-	public void setAidf(Aidf aidf) {
+	public void setBtnSalvar(Button btnSalvar) {
-		this.aidf = aidf;
+		this.btnSalvar = btnSalvar;
 	}
 
 	public Datebox getDtbFecadquisicion() {
@@ -477,36 +517,12 @@ public class EditarAidfController extends MyGenericForwardComposer {
 		this.dtbFecvencimiento = dtbFecvencimiento;
 	}
 
-	public Textbox getTxtForminicial() {
+	public Combobox getCmbTipoValidaVenta() {
-		return txtForminicial;
+		return cmbTipoValidaVenta;
 	}
 
-	public void setTxtForminicial(Textbox txtForminicial) {
+	public void setCmbTipoValidaVenta(Combobox cmbTipoValidaVenta) {
-		this.txtForminicial = txtForminicial;
+		this.cmbTipoValidaVenta = cmbTipoValidaVenta;
 	}
 
-	public Textbox getTxtFormfinal() {
-		return txtFormfinal;
-	}
-
-	public void setTxtFormfinal(Textbox txtFormfinal) {
-		this.txtFormfinal = txtFormfinal;
-	}
-
-	public Button getBtnSalvar() {
-		return btnSalvar;
-	}
-
-	public void setBtnSalvar(Button btnSalvar) {
-		this.btnSalvar = btnSalvar;
-	}
-	
-	public class OnlyNumber implements Constraint {
-	    public void validate(Component comp, Object value) throws WrongValueException {
-	        if (!value.toString().matches("[0-9]+")){
-	        	aidf.setDocfiscal(value.toString());
-	            throw new WrongValueException(comp, Labels.getLabel("editarAidfController.btnSalvar.MSG.apenasValoresNumericosPermitidos"));
-	        }
-	    }
-	}
 }

web/gui/gr/editarAidf.zul

@@ -33,7 +33,7 @@
 						value="${c:l('busquedaAidfController.aidfEspecie.label')}" />
 					<combobox id="cmbAidfEspecie"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						mold="rounded" buttonVisible="true" width="70%"
+						constraint="no empty" mold="rounded" buttonVisible="true" width="70%"
 						model="@{winEditarAidf$composer.lsAidfEspecie}"
 						selectedItem="@{winEditarAidf$composer.aidf.aidfEspecie}" />
 				</row>
@@ -42,7 +42,7 @@
 						value="${c:l('busquedaAidfController.aidfTipo.label')}" />
 					<combobox id="cmbAidfTipo"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						mold="rounded" buttonVisible="true" width="70%"
+						constraint="no empty" mold="rounded" buttonVisible="true" width="70%"
 						model="@{winEditarAidf$composer.lsAidfTipo}"
 						selectedItem="@{winEditarAidf$composer.aidf.aidfTipo}" />
 				</row>
@@ -51,7 +51,7 @@
 						value="${c:l('busquedaAidfController.estado.label')}" />
 					<combobox id="cmbEstado"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						mold="rounded" buttonVisible="true" width="70%"
+						constraint="no empty" mold="rounded" buttonVisible="true" width="70%"
 						model="@{winEditarAidf$composer.lsEstado}"
 						selectedItem="@{winEditarAidf$composer.aidf.estado}" />
 				</row>
@@ -106,7 +106,7 @@
 						value="${c:l('busquedaAidfController.empresa.label')}" />
 					<combobox id="cmbEmpresa"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						width="70%" mold="rounded" buttonVisible="true"
+						constraint="no empty" mold="rounded" buttonVisible="true" width="70%"
 						model="@{winEditarAidf$composer.lsEmpresas}"
 						selectedItem="@{winEditarAidf$composer.aidf.empresa}" />
 				</row>
@@ -115,7 +115,7 @@
 						value="${c:l('busquedaAidfController.inscestadual.label')}" />
 					<combobox id="cmbInscestadual"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						width="70%" mold="rounded" buttonVisible="true"
+						mold="rounded" buttonVisible="true" width="70%"
 						model="@{winEditarAidf$composer.lsInscestadual}"
 						selectedItem="@{winEditarAidf$composer.aidf.getInscestadual()}" />
 
@@ -146,7 +146,7 @@
 					<label	value="${c:l('busquedaAidfController.lbTipoValidaAIDF.value')}" />
 					<combobox id="cmbTipoValidaVenta"
 						use="com.rjconsultores.ventaboletos.web.utilerias.MyComboboxEstandar"
-						mold="rounded" buttonVisible="true" width="70%" 
+						constraint="no empty" mold="rounded" buttonVisible="true" width="70%"
 						selectedItem="@{winEditarAidf$composer.aidf.tipoValidaVenta}">
 					    <comboitem value=" " label=" " />
                         <comboitem value="A" label="AGENCIA" />