Correção cadastro de ocupações. ( bug #5763 )

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

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/pricing/EditarPricingEspecificoController.java

@@ -128,7 +128,7 @@ public class EditarPricingEspecificoController extends MyGenericForwardComposer
 
 			pricingEspecifico = pricingEspecificoService.obtenerID(pricingEspecifico.getPricingespecificoId());
 
-			pricingListEspOcupacion.setData(pricingEspecifico.getPricingEspecificoOcupacionsList());
+			pricingListEspOcupacion.setData(pricingEspecifico.getPricingEspecificoOcupacionsListActivos());
 			pricingListEspOcupacion.addEventListener("onDoubleClick", new EventListener() {
 
 				@Override
@@ -159,31 +159,29 @@ public class EditarPricingEspecificoController extends MyGenericForwardComposer
 	}
 
 	public void onClick$btnApagarPricingEspOcup(Event ev) throws InterruptedException {
-		List<Object> list = pricingListEspOcupacion.getSelectedsItens();
+		if(pricingListEspOcupacion.getSelected() == null) {
-		if (list != null) {
+			Messagebox.show(Labels.getLabel("editarPricingController.MSG.selectItem"),
+					Labels.getLabel("editarPricingController.window.title"),
+					Messagebox.OK, Messagebox.EXCLAMATION);
+		} else {
 			int resp = Messagebox.show(Labels.getLabel("editarPricingEspecificoOcupacionController.MSG.borrarPergunta"),
 					Labels.getLabel("busquedaPricingEspecificoController.window.title"),
 					Messagebox.YES | Messagebox.NO, Messagebox.QUESTION);
 			if (resp == Messagebox.YES) {
-				for (Object o : list) {
+				PricingEspecificoOcupacion especificoOcupacion = (PricingEspecificoOcupacion) pricingListEspOcupacion.getSelected();
-
-					PricingEspecificoOcupacion especificoOcupacion = (PricingEspecificoOcupacion) o;
-					pricingEspecifico.getPricingEspecificoOcupacionsList().remove(especificoOcupacion);
 				
+				if(especificoOcupacion.getPricingEspecificoOcupacionId() != null) {
 					especificoOcupacion.setUsuarioId(UsuarioLogado.getUsuarioLogado().getUsuarioId());
 					especificoOcupacion.setFecmodif(Calendar.getInstance().getTime());
 					especificoOcupacion.setActivo(false);
 					
 					pricingListEspOcupacion.updateItem(especificoOcupacion);
-
+				} else {
+					pricingListEspOcupacion.removeSelectedItem();
+					pricingEspecifico.getPricingEspecificoOcupacionsList().remove(especificoOcupacion);
 				}
 
-				pricingListEspOcupacion.setData(pricingEspecifico.getPricingEspecificoOcupacionsList());
 			}
-		} else {
-			Messagebox.show(Labels.getLabel("editarPricingController.MSG.selectItem"),
-					Labels.getLabel("editarPricingController.window.title"),
-					Messagebox.OK, Messagebox.EXCLAMATION);
 		}
 	}
 
@@ -370,7 +368,7 @@ public class EditarPricingEspecificoController extends MyGenericForwardComposer
 	 * @throws InterruptedException
 	 */
 	private boolean validarPricingEspecificoOcupacion() throws InterruptedException {
-		if(pricingEspecifico.getPricingEspecificoOcupacionsList() == null || pricingEspecifico.getPricingEspecificoOcupacionsList().isEmpty()) {
+		if(pricingEspecifico.getPricingEspecificoOcupacionsList() == null || pricingEspecifico.getPricingEspecificoOcupacionsListActivos().isEmpty()) {
 			Messagebox.show(
 					Labels.getLabel("editarPricingEspecificoOcupacionController.MSG.noPricingEspecificoOcupacion"),
 					Labels.getLabel("busquedaPricingEspecificoController.window.title"),
@@ -378,7 +376,7 @@ public class EditarPricingEspecificoController extends MyGenericForwardComposer
 			return false;
 		}
 		
-		List<PricingEspecificoOcupacion> sOcupacionInicial = new ArrayList<PricingEspecificoOcupacion>(pricingEspecifico.getPricingEspecificoOcupacionsList());
+		List<PricingEspecificoOcupacion> sOcupacionInicial = new ArrayList<PricingEspecificoOcupacion>(pricingEspecifico.getPricingEspecificoOcupacionsListActivos());
 		Collections.sort(sOcupacionInicial,new Comparator<PricingEspecificoOcupacion>() {
 
 			@Override

src/java/com/rjconsultores/ventaboletos/web/gui/controladores/pricing/EditarPricingEspecificoOcupacionController.java

@@ -59,11 +59,9 @@ public class EditarPricingEspecificoOcupacionController extends MyGenericForward
 		ocupacionfinal.getValue();
 
 		try {
-			boolean esEdicion = false;
+			if (pricingEspecificoOcupacion.getPricingEspecificoOcupacionId() == null) {
-			if (pricingEspecificoOcupacion.getPricingEspecificoOcupacionId() == null)
 				pricingEspecificoOcupacion = new PricingEspecificoOcupacion();
-			else
+			}
-				esEdicion = true;
 
 			pricingEspecificoOcupacion.setTarifa(tarifa.getValueDecimal());
 			pricingEspecificoOcupacion.setOcupacioninicial(ocupacioninicial.getValueDecimal());
@@ -75,27 +73,28 @@ public class EditarPricingEspecificoOcupacionController extends MyGenericForward
 			pricingEspecificoOcupacion.setActivo(true);
 			
 			List<PricingEspecificoOcupacion> peoList = null;
-			if (pricingEspecifico.getPricingEspecificoOcupacionsList().isEmpty())
+			if (pricingEspecifico.getPricingEspecificoOcupacionsList().isEmpty()) {
 				peoList = new ArrayList<PricingEspecificoOcupacion>();
-			else
+			} else {
 				peoList = pricingEspecifico.getPricingEspecificoOcupacionsList();
+			}
 				
-			if (esEdicion) {
+			if (pricingEspecificoOcupacion.getPricingEspecificoOcupacionId() != null && peoList.contains(pricingEspecificoOcupacion)) {
-				int index = 0;
+				int index = peoList.indexOf(pricingEspecificoOcupacion);
-				for (PricingEspecificoOcupacion peo : peoList) {
+				if(index >= 0) {
-					if (peo.equals(pricingEspecificoOcupacion)) {
+					peoList.set(index, pricingEspecificoOcupacion);
-						index = peoList.indexOf(peo);
-						break;
-					}
 				}
-				peoList.set(index, pricingEspecificoOcupacion);
-
 			} else {
 				peoList.add(pricingEspecificoOcupacion);
 			}
 			
 			pricingEspecifico.setPricingEspecificoOcupacionsList(peoList);
-			pricingListEspOcupacion.setData(peoList);
+			
+			if(pricingEspecificoOcupacion.getPricingEspecificoOcupacionId() != null) {
+				pricingListEspOcupacion.updateItem(pricingEspecificoOcupacion);
+			} else {
+				pricingListEspOcupacion.addItemNovo(pricingEspecificoOcupacion);
+			}
 
 			closeWindow();
 

src/java/com/rjconsultores/ventaboletos/web/utilerias/render/RenderPricingEspecificoOcupacion.java

@@ -11,6 +11,10 @@ public class RenderPricingEspecificoOcupacion implements ListitemRenderer {
 	public void render(Listitem lstm, Object o) throws Exception {
 		PricingEspecificoOcupacion peo = (PricingEspecificoOcupacion) o;
 		
+		if (!peo.getActivo()) {
+			return;
+		}
+
 		Listcell lc = new Listcell(peo.getTarifa().toString());
 		lc.setParent(lstm);