Skip to content

fix: la ficha de incidencia muestra el Identificador primero

belen.varsi requested to merge fix/ocultar-comentarios-sin-permiso into dev

Mantis: 0044487

Qué

Corrige el Mantis 0044486: con un usuario que puede ver instancias pero no tiene VER_COMENTARIO_NORMAL_INSTANCIA, el detalle de la instancia mostraba igual la sección "Comentarios", con el error "No tienes permisos suficientes para realizar esta acción." y el texto "Todavía no hay comentarios.", aunque la instancia sí tenía comentarios.

Por qué

El bloque de comentarios normales se mostraba siempre y hacía la llamada al backend, que respondía 403. Solo el bloque de comentarios confidenciales revisaba el permiso antes. Además, ante cualquier error de carga, la lista vacía se interpretaba como "no hay comentarios".

Cambios

src/components/instancias/InstanciaComentarios.tsx:

  • Sin VER_COMENTARIO_NORMAL_INSTANCIA, la sección "Comentarios" no se muestra: ni el listado, ni el formulario para comentar, ni las acciones de editar y eliminar. Tampoco se hace la llamada al backend.
  • Si la carga de comentarios falla por otro motivo, ya no se muestra "Todavía no hay comentarios." debajo del error; queda solo el mensaje de error.

Aplica también al detalle de una Incidencia, que usa el mismo componente.

Los comentarios confidenciales no cambian: sin permiso siguen mostrando el aviso "No tenés permiso para ver los comentarios confidenciales de esta instancia."

Dependencias

Está apilado sobre fix/eliminar-comentario-normal-instancia, que a su vez está sobre !40 (merged) (editar comentario normal), porque los tres tocan el mismo bloque del componente. Orden de merge: !40 (merged), después eliminar, después este.

Cómo probar

  1. Armar un rol con VER_INSTANCIAS y sin VER_COMENTARIO_NORMAL_INSTANCIA, y asignarlo a un funcionario.
  2. Iniciar sesión con ese funcionario y abrir el detalle de una instancia que tenga al menos un comentario.
  3. La sección "Comentarios" no aparece, ni ningún error de permisos.
  4. Repetir en el detalle de una Incidencia.
  5. Iniciar sesión como Administrador y abrir la misma instancia: la sección se ve con sus comentarios y acciones, como antes.

Cambios en BD

Ninguno.

Edited by belen.varsi

Merge request reports