Project

General

Profile

Errores #5281

[Acuerdo][API Reports] Corregir incoherencia en interfaz de Acuerdos entre el filtro y los datos que se visualizan

Added by Oscar Abad about 2 months ago. Updated about 1 month ago.

Status:
Resuelta
Priority:
Normal
Assignee:
Category:
Desarrollo
Target version:
Start date:
02/27/2024
Due date:
% Done:

100%


Description

Especificación

Cuando se realiza un filtrado por nombre en la vista de Acuerdos, lo hace por el campo "acuerdo_datos.acuerdo_actual" = true, cuyo nombre no coincide con el que se muestra cuando hay más de una versión del acuerdo, ya que accede a través de la propiedad :ultimos_datos, que obtiene la primera versión y no la última [order('acuerdo_datos.created_at asc')].

Además, los datos que se visualizan en la interfaz no coinciden con los obtenidos por la API, que proceden de la propiedad :actual_ultimos_datos, que obtiene la primera versión con "acuerdo_datos.acuerdo_actual" = true

Ejemplo: Acuerdos filtrado por nombre: " CONVENIO ESPECÍFICO DE COLABORACION INTERINSTITUCIONAL ENTRE MINERD Y LA OEI"

Al incluir todos los estados de un acuerdo, también aparecen acuerdos que no tienen estado, que no se deberían obtener por la petición API.

Análisis

Relaciones entre acuerdo y acuerdo_datos en el modelo:

  has_one :ultimos_datos, -> { order('acuerdo_datos.created_at asc') }, foreign_key: :acuerdo_id, class_name: 'AcuerdoDatos'
  has_one :actual_ultimos_datos, -> { where("acuerdo_datos.acuerdo_actual": true).order('acuerdo_datos.created_at asc') }, foreign_key: :acuerdo_id, class_name: 'AcuerdoDatos'

Selector de los acuerdos en la acción listado del controlador AcuerdoController a partir de "acuerdo_datos.acuerdo_actual" => true":

    @acuerdos = @paginado = Acuerdo.includes({acuerdo_datos: [:acuerdos_contraparte, :acuerdos_aportacion, :acuerdos_area_actuacion, :acuerdos_pais]}, :acuerdos_estado).
                                    where("acuerdo_datos.acuerdo_actual" => true, "acuerdos_estado.estado_vigente" => true).
                                    joins(join_tables).where(filters).where(filters_name).where(filters_year).
                                    order(session[:acuerdo_orden] + ' ' + session[:acuerdo_asc_desc]).
                                    paginate(page: (params[:format]=='xls' ? nil : params[:page]),
                                             per_page: (params[:format_xlsx_count] || session[:por_pagina]) )

Las diferentes versiones de un acuerdo aparecen mediante esta acción del controlador AcuerdoController:

  def versiones
    @acuerdo = Acuerdo.find_by_id params[:id]
    versiones = @acuerdo.acuerdo_datos.order('created_at DESC')
    # Le quitamos la ultima version
    @versiones = versiones.to_ary - [versiones.first]
    render(:update) do  |page|
      page.replace_html params[:update], partial: 'versiones'
    end
  end

Función del modelo Acuerdo donde se crea una nueva versión del acuerdo al pasar a un metaestado 'firmado' desde un metaestado 'definicion':

  def crear_nueva_version
    antigua_version = ultimos_datos
    nueva_version = antigua_version.dup
    nueva_version.save
    antigua_version.update_attribute :acuerdo_actual, false
    nueva_version.update_attribute :acuerdo_actual, true
    antigua_version.acuerdos_contraparte.each do |contraparte|
      nueva = contraparte.dup
      nueva.acuerdo_datos_id = nueva_version.id
      nueva.save
    end
    antigua_version.acuerdos_aportacion.each do |aportacion|
      nueva = aportacion.dup
      nueva.acuerdo_datos_id = nueva_version.id
      nueva.save
    end
    antigua_version.acuerdos_pago.each do |pago|
      nuevo = pago.dup
      nuevo.acuerdo_datos_id = nueva_version.id
      nuevo.save
    end
    antigua_version.acuerdos_area_actuacion.each do |area_actuacion|
      nueva = area_actuacion.dup
      nueva.acuerdo_datos_id = nueva_version.id
      nueva.save
    end
    antigua_version.acuerdos_pais.each do |pais|
      nuevo = pais.dup
      nuevo.acuerdo_datos_id = nueva_version.id
      nuevo.save
    end 
  end

Diagrama de clases desde la perspectiva de la entidad Acuerdo:

Implementación

Hay que utilizar [versiones.last] en lugar de [versiones.first] en la acción versiones del controlador AcuerdoController:
https://git.semillasl.com/gong/gor/-/commit/cf08cca05ca7eb122b6a40d32a2086cad40c96d1
De esta forma, pulsando el botón de versiones del acuerdo veremos, como primer registro aparecerá la foto del acuerdo cuyos datos son los que aparecen en API informes.

También hay que enviar en la API de informes actual_ultimos_datos del acuerdo sólo si el acuerdo está en metaestado definición, en caso contrario, enviar ultimos_datos.
https://git.semillasl.com/gong/gor/-/commit/e447b9d4d0c413bc32289be0bef522cd8b685246

Se ha optimizado la generación del fichero .rabl unificando los nodos que lo componían:
https://git.semillasl.com/gong/gor/-/commit/202f47698ae2eee2a37dc0f524a73dd106c96ee6

Aquí se filtra para que no se devuelva ningún acuerdo sin estado vigente:
https://git.semillasl.com/gong/gor/-/commit/3b911dc290caf8660d3b0f69cac5bcbd5532ea99

Pruebas

Versión antes de modificación KO al buscar Acuerdo por nombre " CONVENIO ESPECÍFICO DE COLABORACION INTERINSTITUCIONAL ENTRE MINERD Y LA OEI":

Versión después de modificación OK al buscar Acuerdo por nombre " CONVENIO ESPECÍFICO DE COLABORACION INTERINSTITUCIONAL ENTRE MINERD Y LA OEI":

VersionesAcuerdoKO.png (45.4 KB) Oscar Abad, 02/27/2024 05:16 PM

VersionesAcuerdoOK.png (41.3 KB) Oscar Abad, 02/28/2024 09:06 AM

DiagramaClasesPerspectivaAcuerdo.png (22.9 KB) Oscar Abad, 02/28/2024 09:45 AM

History

#1 Updated by Oscar Abad about 2 months ago

  • Description updated (diff)

#2 Updated by Oscar Abad about 2 months ago

  • Description updated (diff)

#3 Updated by Oscar Abad about 2 months ago

  • Description updated (diff)
  • Status changed from Nueva to En Curso

#4 Updated by Oscar Abad about 2 months ago

  • Description updated (diff)

#5 Updated by Oscar Abad about 2 months ago

#6 Updated by Oscar Abad about 1 month ago

  • File deleted (VersionesAcuerdoOK.png)

#8 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#9 Updated by Oscar Abad about 1 month ago

#10 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#11 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#12 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#13 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#14 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#15 Updated by Oscar Abad about 1 month ago

  • Subject changed from [Acuerdo] Corregir incoherencia en interfaz de Acuerdos entre el filtro y los datos que se visualizan to [Acuerdo][API Reports] Corregir incoherencia en interfaz de Acuerdos entre el filtro y los datos que se visualizan

#16 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#17 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#18 Updated by Oscar Abad about 1 month ago

  • Description updated (diff)

#19 Updated by Oscar Abad about 1 month ago

  • Status changed from En Curso to Resuelta
  • % Done changed from 0 to 100

Also available in: Atom PDF

Financiado por:

Desarrollado por:
Software libre forjado en: