[Delivers #66470554] Se subio a session el colportor y el asociado, y…

… se corrigio combo que lista temporadas

src/main/java/mx/edu/um/mateo/colportor/web/TemporadaColportorController.java

@@ -251,39 +251,34 @@ public String crea(HttpServletRequest request, HttpServletResponse response, @Va
  }
 
  @RequestMapping("/edita/{id}")
- public String edita(@PathVariable Long id, Model modelo) {
+ public String edita(HttpServletRequest request, @PathVariable Long id, Model modelo) {
  log.debug("Edita Temporada {}", id);
 
  TemporadaColportor temporadaColportor = temporadaColportorDao.obtiene(id);
  modelo.addAttribute(Constantes.TEMPORADACOLPORTOR, temporadaColportor);
 
+ request.getSession().setAttribute(Constantes.COLPORTOR, temporadaColportor.getColportor());
+ request.getSession().setAttribute(Constantes.ASOCIADO_COLPORTOR, temporadaColportor.getAsociado());
+
  Map<String, Object> params = new HashMap<>();
  params.put("organizacion", ambiente.obtieneUsuario().getEmpresa().getOrganizacion().getId());
+ params.put("reporte","");
  params = temporadaDao.lista(params); 
  modelo.addAttribute(Constantes.TEMPORADA_LIST, (List) params.get(Constantes.TEMPORADA_LIST));
-
+
+ params = new HashMap<>();
+ params.put("reporte","");
  params = colegioDao.lista(params);
  modelo.addAttribute(Constantes.CONTAINSKEY_COLEGIOS, (List) params.get(Constantes.CONTAINSKEY_COLEGIOS));
 
- params.put("empresa", ambiente.obtieneUsuario().getEmpresa().getId());
- params = colportorDao.lista(params);
- modelo.addAttribute(Constantes.COLPORTOR_LIST, (List) params.get(Constantes.COLPORTOR_LIST));
-
- params = asociadoDao.lista(params);
- modelo.addAttribute(Constantes.ASOCIADO_LIST, (List) params.get(Constantes.ASOCIADO_LIST));
-
- Map<String, Object> temporadas = temporadaDao.lista(null);
- params = temporadaDao.lista(params);
- modelo.addAttribute(Constantes.TEMPORADA_LIST, (List)params.get(Constantes.TEMPORADA_LIST));
-
-
  return Constantes.TEMPORADACOLPORTOR_PATH_EDITA;
  }
 
  @Transactional
  @RequestMapping(value = "/actualiza", method = RequestMethod.POST)
  public String actualiza(HttpServletRequest request, @Valid TemporadaColportor temporadaColportor, 
  BindingResult bindingResult, Errors errors, Model modelo, RedirectAttributes redirectAttributes) throws ParseException {
+
  if (bindingResult.hasErrors()) {
  log.error("Hubo algun error en la forma, regresando");
  despliegaBindingResultErrors(bindingResult);
@@ -300,16 +295,20 @@ public String actualiza(HttpServletRequest request, @Valid TemporadaColportor te
  try {
  Usuario usuario = ambiente.obtieneUsuario();
 
+ request.getSession().setAttribute(Constantes.COLPORTOR, temporadaColportor.getColportor());
+ request.getSession().setAttribute(Constantes.ASOCIADO_COLPORTOR, temporadaColportor.getAsociado());
+
  Temporada temporada = temporadaDao.obtiene(temporadaColportor.getTemporada().getId());
  temporadaColportor.setTemporada(temporada);
- Asociado asociado = asociadoDao.obtiene(ambiente.obtieneUsuario().getId());
- temporadaColportor.setAsociado(asociado);
- Colportor colportor = colportorDao.obtiene(temporadaColportor.getColportor().getId());
- temporadaColportor.setColportor(colportor);
+ //Asociado asociado = asociadoDao.obtiene(temporadaColportor.getAsociado().getId());
+ temporadaColportor.setAsociado((Asociado)request.getSession().getAttribute(Constantes.ASOCIADO_COLPORTOR));
+ //Colportor colportor = colportorDao.obtiene(temporadaColportor.getColportor().getId());
+ temporadaColportor.setColportor((Colportor)request.getSession().getAttribute(Constantes.COLPORTOR));
 
  Colegio colegio = colegioDao.obtiene(temporadaColportor.getColegio().getId());
  temporadaColportor.setColegio(colegio);
  temporadaColportor = temporadaColportorDao.actualiza(temporadaColportor);
+
  } catch (ConstraintViolationException e) {
  log.error("No se pudo crear al Asociacion", e);
  return Constantes.TEMPORADACOLPORTOR_PATH_NUEVA;

src/main/webapp/WEB-INF/jsp/colportaje/temporadaColportor/edita.jsp

@@ -39,25 +39,16 @@
  <label for="temporada">
  <s:message code="temporada.label" />
  <span class="required-indicator">*</span>
- <form:select id="temporadaId" path="temporada.id" items="${temporadas}" itemLabel="nombre" itemValue="id" />
+ <form:select id="temporada.id" path="temporada.id" items="${temporadas}" itemLabel="nombre" itemValue="id" />
  <form:errors path="temporada" cssClass="alert alert-error" />
  </div>
- </s:bind> 
- <s:bind path="temporadaColportor.colportor">
- <div class="control-group <c:if test='${not empty status.errorMessages}'>error</c:if>">
- <label for="colportor">
- <s:message code="colportor.label" />
- <span class="required-indicator">*</span>
- <form:select id="colportorId" path="colportor.id" items="${colportores}" itemLabel="calle" itemValue="id" />
- <form:errors path="colportor" cssClass="alert alert-error" />
- </div>
  </s:bind>
  <s:bind path="temporadaColportor.colegio">
  <div class="control-group <c:if test='${not empty status.errorMessages}'>error</c:if>">
  <label for="colegio">
  <s:message code="colegio.label" />
  <span class="required-indicator">*</span>
- <form:select id="colegioId" path="colegio.id" items="${colegios}" itemLabel="nombre" itemValue="id" />
+ <form:select id="colegio.id" path="colegio.id" items="${colegios}" itemLabel="nombre" itemValue="id" />
  <form:errors path="colegio" cssClass="alert alert-error" />
  </div>
  </s:bind>
@@ -75,8 +66,8 @@
  <label for="status">
  <s:message code="status.label" />
  <span class="required-indicator">*</span>
- <form:radiobutton path="status" value="A" cssClass="span3" id="status"/><s:message code="activo.label"/>
- <form:radiobutton path="status" value="I" cssClass="span3" id="status"/><s:message code="inactivo.label"/>
+ <form:radiobutton path="status" value="A" cssClass="span3" id="activo"/><s:message code="activo.label"/>
+ <form:radiobutton path="status" value="I" cssClass="span3" id="inactivo"/><s:message code="inactivo.label"/>
  <form:errors path="status" cssClass="alert alert-error" />
  </div>
  </s:bind>