search

bluemixmg / Ideation-Toolchain

Other
0 stars 0 forks source link

Campo de consulta coddesafio en el SELECT de contar ideas por desafio (DesafioDAO) debe cambiarse a id #3

Open bgutierrez600 opened 7 years ago

bgutierrez600 commented 7 years ago

Por favor, realizar el cambio correspondiente en los metodos que corresponda de DesafioDAO. A medida que se vayan encontrando estos problemas en otras clases se ira indicando como issues.

Ubicacion: Clase DesafioDAO linea 279.

String sql = "SELECT count(*) as cantideas FROM idea WHERE coddesafio='" + id + "'";

Narvis, julio, por favor si pueden comenten este issue, indicandole a luis problemas similares (y su ubicacion en el codigo) de los campos de la tabla desafio (por ahora, para llevar un orden del codigo que se va modificando) y luis por favor ve modificandolos cuando puedas. Tambien comenten este issue a medida que lo vayan completando.

/cc @lfreitez @njarvis93 @jparedes-marnagarage

jparedes-marnagarage commented 7 years ago

Cambiar nombre de la tabla aredepericia por areadepericia.

Cambiar los siguientes campos de clave primaria a tipo serial (entero autoincrementable), excepto los que sean claves foráneas (en tal caso cambiarlos a integer):

  1. codareaest en la tabla areaestrategica
  2. codarea en la tabla areasporevaluador
  3. codigoarea en la tabla areadepericia
  4. codigo en la tabla categoria
  5. codigo_idea en la tabla evaluacion
  6. codigo en la tabla idea
  7. codDesafio en la tabla idea

Creo que ideador y asociado no necesitan el campo email, ya que lo obtiene a través de su relación con usuario

@lfreitez CC: @bgutierrez600 @njarvis93

njarvis93 commented 7 years ago

Correcciones a los modelos con respecto a las tablas: Corregidos los items 2, 3, 4 y 7. El item 1 estaba corregido cuando llegue. Con respecto al item 6, no entiendo porque se cambiara el código de la idea a serial, necesito contexto. Queda pendiente el item 5 que esta relacionado con el item 6. Concuerdo con Julio en el punto del email en ideador y asociado.

@bgutierrez600 no entiendo tu requerimiento con respecto al issue. La linea que indicas esta en el DAO y esta una comentarizada donde se usa el codDesafio de tipo String. Se manejaran ambos campos? O ahora el id será equivalente a codDesafio?

njarvis93 commented 7 years ago

Por ahora los cambios se encuentran en la rama ideas, hasta que no se modifiquen los campos en las tablas no haré el merge

bgutierrez600 commented 7 years ago

Listas las modificaciones en la BD respecto a los 7 items propuestos por @jparedes-marnagarage.

Por favor avisarme cualquier error o cambio despues de las pruebas.

/cc @njarvis93