Ajustar consecutivo de las necesidades

[AlexFBP]

Consideraciones

• Hay una posible condición de carrera de la forma en que se están generando actualmente los consecutivos para las necesidades:

• consecutivos_crud --> ir al modelo SVG

• necesidades_crud --> ir al modelo SVG

Tareas

Unificar consecutivos de necesidad

• [x] Identificar todos los lados en que se manejan consecutivos para la necesidad
• [x] Quitar la columna duplicada del consecutivo de la necesidad (que solicitud y necesidad sean lo mismo - confirmar con Analista si no hay problema - no debería de haber problema porque se supone que se manejan estados)

Ajustar CRUD de necesidades:

• [x] Renombrar consecutivo_solicitud a consecutivo_id
• [x] Ajustar lo que implique lo anterior... (CRUD + MID + Cliente)
• [x] Revisar por qué existe la columna consecutivo_necesidad y quitarla
• [x] Ajustar lo que implique lo anterior...

Cambiar la generación actual de la necesidad

• [x] En los MID, crear helper que retorne el procesoId para usarlo posteriormente en el POST de consecutivos_crud. En principio sepuede crear un registro en consecutivos_crud/proceso, y retornar el id de ese proceso
• [x] En los MID, crear un helper para el POST de consecutivos_crud
• [x] En los MID, usar los helper anteriores para crear un consecutivo_crud, donde
  • contextoId vendrá del primer helper, será el procesoId (en lo posible)
  • year será la vigencia
• [x] Usar lo anterior para continuar el proceso...
• [x] Ajustar en Presupuesto para mostrar el consecutivo que es

[DanKazuky]

Se verifica como se asigna/crear el consecutivo necesidad y se identifica lo siguiente:

Este consecutivo se le asigna el mismo valor que posee la necesidad como ID, por lo que podríamos decir que tiene redundancia (este dato se repite), a continuación un pantallazo de como se asigna:

Este fragmento de código pertenece al proceso de aprobar la necesidad (Que es donde se asigna por primera y única vez el consecutivo de necesidad)

Conclusión: Considero que no habría problema de eliminar la columna o el dato de consecutivo de necesidad ya que no se pierde la información y tampoco tiene una funcionalidad importante en el proceso.

[DanKazuky]

Se realizo una reunión con el animo de exponer la propuesta ya descrita en el Issue, ademas realizar consulta de si los desarrollos que se encuentran actualmente, se debieron por alguna petición o gestión especifica que se ignore y se expone lo siguiente:

• Se quiere utilizar un consecutivo con el fin de asignar un identificador único a las solicitudes que ya han sido aprobadas.
• El Id principal se utiliza únicamente para consultar todas las solicitudes.

Se ve visto bueno para poder realizar cambios por lo que se procede a consultar con un dba, se realiza la reunión y se da visto bueno para realizar cambios ya que cumple con los requerimientos.

[AlexFBP]

Se encontró un problema en plan_cuentas_mid con la petición https://autenticacion.portaloas.udistrital.edu.co/apioas/plan_cuentas_mid/v1/crp/getFullCrp/

• [x] Agregar la función defer a todos los controladores, empezar con los relacionados con CDP/CRP - Se recomienda usar utils_oas
• [x] De paso, para el control de errores, ajustar los controladores que retornen un arreglo de resultados, revisar que cuando sean nil se retorne un []interface{}{} (o según el tipo de datos)

Referencias para el control de errores:

• Paquete
• Uso en controller
• Uso en helper

[DanKazuky]

Se ha evidenciado que los cambios para el consecutivo no fueron efectivos en ambientye de pruebas, ademas de eso se evidencia que hay un conflicto generando la aprobacion de la necesidad, se procede a revisar en contrando que al momento de solicitar la necesidad en consulta y solicitar la aprobacion se repiten datos (Rubros, Atividades, Marco Legal) en casos donde deberian haber dos o mas datos no repetidos, dando como resultado siempre el primer dato repetido multiples veces.

Se procede a revisar el cliente de necesidades y se encuentra que los datos se envian correctamente a guardar la necesidad, por lo que se descarta que se un mal envio de datos al guardar. Seguidamente se consulta la consulta de la nesidad y se identifica que esta llega con los valores repetidos desde el mid de plan de cuentas, por lo que se descarta fallo desde el cliente.

Se revisa el mid de plan de cuentas al momento de realizar la consulta y se determina que es un error de sobre-escritura en la variable que guarda temporalmente los datos de la necesidad que se retornan al cliente. Se ajusta para que el caso de sobreescribir no suceda y se realizan pruebas para confirmar funcionamiento.

[AlexFBP]

En la lista de necesidades:

• [x] La columna que actualmente tiene como título "Nº de Solicitud", está mostrando el consecutivo de la necesidad. Cambiarle el título por "Necesidad Nº" o similar
• [x] El título que anteriormente estaba en la columna del item anterior, deberá mostrarse en una columna, que le falta a la tabla; para visualizar el id de la necesidad. (@AlexFBP : Citar PR y/o commit en que se quitó esta columna)
• [x] Ordenar por fecha de modificación, en orden descendente

Al ver una necesidad:

• [x] Cuando se muestra la solicitud, en la parte de arriba en la parte grande en naranja; mostrar el id (el otro está OK, cuando se muestra Necesidad se está mostrando el consecutivo)
• [x] El botón del PDF debería poderse ver desde el estado Aprobado en adelante (Cuando se solicita el CDP, no se puede ver)
• [x] Ajustar el # de solicitud de CDP que se muestra arriba

En el PDF de la necesidad: (Las siguientes son de prioridad baja)

• [x] Meter el header en el cuerpo, para que no quede pegado al borde de la página
• [x] Quitar el borde de las tablas anidadas

En el cliente de Presupuesto, en ver solicitud de CDP

• [x] Ajustar "Nº de Necesidad" para que (actualmente está apareciendo en blanco) aparezca "{{SEQ}} de {{VIGENCIA}}" de la necesidad