exequiel / taller2013

Taller 2013 Red Educativa - Issue Tracking
4 stars 0 forks source link

Error al guardar un nuevo grupo #21

Open edittler opened 10 years ago

edittler commented 10 years ago

Intento guardar un nuevo grupo usando el método 'GuardarDatos' enviando el siguiente XML:

<WS><Grupo><actividadId>27</actividadId><usuarios><Usuario><id>85</id><username>Seba</username></Usuario><Usuario><id>62</id><username>testUser</username></Usuario></usuarios></Grupo></WS>

La respuesta obtenida es la siguiente:

No such field com.ws.pojos.Grupo.usuarios
---- Debugging information ----
field               : usuarios
class               : com.ws.pojos.Grupo
required-type       : com.ws.pojos.Grupo
converter-type      : com.thoughtworks.xstream.converters.reflection.ReflectionConverter
path                : /com.ws.pojos.Grupo/usuarios
line number         : 1
version             : 1.4.5
-------------------------------XML: <WS><Grupo><actividadId>27</actividadId><usuarios><Usuario><id>85</id><username>Seba</username></Usuario><Usuario><id>62</id><username>testUser</username></Usuario></usuarios></Grupo></WS>

Según la documentación y lo charlado, no se usa "list" sino, el campo "usuarios" para una lista de usuarios. Si estamos en lo equivocado, avisen.

exequiel commented 10 years ago

Ezequiel,

Lo que comentás es correcto (lo del tag "usuarios"), pero eso aplicaba a la clase Actividad no a Grupo. De igual manera estoy observando que en Grupo tenemos un problema diferente. Los Pojos de DB y tambien los nuestros no tienen un listado de usuarios... @asanguinetti me dará una mano validando si es correcto lo siguiente: La forma correcta de generar un grupo es asignarle al usuario ese mismo grupos, es decir, se navega desde usuario hacia grupo. Nos pasa lo mismo que con actividad, tenemos la referencia al revés digamos...

edittler commented 10 years ago

Para actividades individuales -> los usuarios se agregan a la actividad Para actividades grupales -> los usuarios se agregan a grupos Un grupo sin usuarios no es un grupo, jeje. Los grupos los debe armar un coordinador de la actividad, por lo que los mismos deber ser creados tal como envié. Un usuario no debería anotarse solo a un grupo. Al menos así es en los Grupos de actividades. Esperamos novedades.

exequiel commented 10 years ago

Claro, por eso... es lo mismo que pasó con actividades (antes un usuario tenia un conjunto de actividades, lo modificamos y quedó como debe, una actividad tiene un conjunto de usuarios). Hay que hacer las modificaciones desde abajo para que la lista de usuarios la tenga el grupo y no viceversa.

El cambio implica modificar los mapeos en DB, regenerar las clases cliente, reajustar las entidades que usamos y re deployar los WS de DB e integración.

Con la confimacion de Axel sobre el issue nos pondremos a trabajar en la reparacion.

2014-02-20 14:19 GMT-03:00 Ezequiel notifications@github.com:

Para actividades individuales -> los usuarios se agregan a la actividad Para actividades grupales -> los usuarios se agregan a grupos Un grupo sin usuarios no es un grupo, jeje. Los grupos los debe armar un coordinador de la actividad, por lo que los mismos deber ser creados tal como envié. Un usuario no debería anotarse solo a un grupo. Al menos así es en los Grupos de actividades. Esperamos novedades.

Reply to this email directly or view it on GitHubhttps://github.com/exequiel/taller2013/issues/21#issuecomment-35645506 .

asanguinetti commented 10 years ago

No, es más simple esta vez.

Había un typo mio que hacía que no se exportara el listado de Usuarios del Grupo en el WS. Todo el soporte abajo debería estar.

Ahora actualizo.

edittler commented 10 years ago

Buenisimo, estoy al tanto de lo que digan.

Lo que me quedó aclarar por las dudas, es que un grupo puede pertenecer a una sola actividad, por ello está el atributo "actividadId" en Grupo. Por lo que no haría falta "enlazar" las actividades con los grupos tal como se hace con los usuarios. Es decir, no necesito almacenar en Actividad una lista de "grupos".

asanguinetti commented 10 years ago

Listo Exequiel, ya está deployeado en la VM. Cuando quieras hacete un wsimport y lo vas a tener.

Las Actividades se pueden enlazar con los Grupos desde cualquiera de los dos extremos, al menos desde DB es así. Arreglen ustedes como les sea más simple o usen los dos según conveniencia. Eso sí, ojo si mandan una Actividad con un listado de Grupos vacío, porque me parece que se desvincularían los Grupos previamente vinculados

Flotristant commented 10 years ago

Acabo de probarlo y es asi como decis ;) De todas formas se pueden enalazar de las dos formas.

edittler commented 10 years ago

Que novedades hay de esto?

Recién probé si anda pero me muestra el mismo error, sigue sin reconocer el tag "usuarios".

exequiel commented 10 years ago

Ninguna,

Ayer nos juntamos en la Facultad y entre todos los grupos decidimos congelar el estado hasta la presentación de hoy, así que no se van a corregir bugs hasta próximo aviso.

Sent from my Nexus 4 El 21/02/2014 00:37, "Ezequiel" notifications@github.com escribió:

Que novedades hay de esto?

Recién probé si anda pero me muestra el mismo error, sigue sin reconocer el tag "usuarios".

Reply to this email directly or view it on GitHubhttps://github.com/exequiel/taller2013/issues/21#issuecomment-35696366 .