GovernIB / gdib

Gestor Documental de les Illes Balears (Arxiu CAIB)
0 stars 1 forks source link

Cambio de comportamiento en el método getDocument() #40

Open atrobat-dgtic opened 3 years ago

atrobat-dgtic commented 3 years ago

Con los cambios en GDIB hemos detectado un cambio de comportamiento en el método getDocument() que afecta a las aplicaciones. Cuando llamas a este método con el parámetro content=false no retorna el contenido del documento, pero si que retorna el nodo content vacío dentro de la respuesta del mensaje, cuando antes no lo retornaba. ¿Se podria mantener el mismo comportamiento que antes y no retornar el elemento content si no se va a enviar el contenido del documento?

dsariego-memorandum commented 3 years ago

Buenas Toni,

Imagino que lo has probado en las máquinas 6 y 7, porque he lanzado una petición de prueba en la 4 a través del SoapUI y no devuelve el contenido cuando el content se pasa a false. ¿Puede ser? habría que deshacer los cambios que se hicieron para la anterior solución en el bus y aplicar unos nuevos que ya se detallaron en el documento de cambios del bus que pasé, de todas formas esta tarde os enviaré un nuevo correo con ese mismo documento actualizado, por los cambios de otras incidencias y cómo se harían los cambios de endpoints que pedisteis incluir.

Un saludo

atrobat-dgtic commented 3 years ago

He hecho la prueba en las máquinas 6 y 7. Actualizaremos el bus cuando nos paseis el documento con los ficheros que indiqueis en el.

atrobat-dgtic commented 3 years ago

Hola David,

He aplicado los cambios que comentas en el documento en el bus de servicios y el comportamiento de getDocument() sigue igual, al poner content=false retorna :

CONTENT
dsariego-memorandum commented 3 years ago

Hola,

¿Se deshicieron los anteriores cambios del bus antes de aplicar los nuevos? En un principio deshaciendo los anteriores cambios y añadiendo los nuevos indicados debería arreglarse, al menos yo no conseguí reproducir ninguna prueba que pase eso en la máquina 4, ¿Puedes pasarme una petición de ejemplo para intentar reproducirlo en esta máquina por si acaso?

atrobat-dgtic commented 3 years ago

No he deshecho los cambios, ya que en teoría los cambios que hicimos eran para arreglar cosas. Esto es lo que cambié https://github.com/GovernIB/gdib/commit/25f09201c73426d119216486179b8b7f9bd0bdc6.

Si me confirmas que debo deshacer todos estos cambios, de donde cojo los ficheros, supongo que los que hay en github son los que modifiqué.

Si quieres hacer una prueba lo puedes hacer des del soap ui, lanzando un getDocument con el parámetro content=false.

dsariego-memorandum commented 3 years ago

Buenas,

Esos cambios fueron hechos para la solución en la que se devolvía el mimeType y el tamaño del documento en el objeto del contenido, por lo tanto se tuvo que modificar tanto el código de gdib como el bus para que siempre devuelvan el objecto content donde estaban estos datos. Al aplicar la nueva solución en la que estos datos se envían en propiedades, no es necesario todos esos cambios en el bus. El problema que tenéis por lo que os pasa eso es que gdib ya no manda el objeto content cuando el booleano está a falso, pero el bus lo sigue esperando, en la línea 25 del getDocByCSVXMLResponse.xslt es donde pinta la salida, pero al no tener valores que meter porque gdib ya no lo devuelve, por eso os lo pinta vacío.

Debéis deshacer todos los cambios de ese commit, como hice yo:

https://github.com/GovernIB/gdib/commit/3822344f54ecd3ce850ac783c5cc4abfe6a9c9af

Conservo todos los archivos del bus antes de realizar cambios en ellos, si quieres te mando todos los archivos para deshacer estos cambios y con los nuevos cambios ya aplicados además, solo deberás sustituir en el bus todos los archivos por los que te pase y debería estar ya arreglado.

Un saludo,