Errores #2292
Error en resumen ejecutivo de agente
Estado:
Resuelta
Prioridad:
Normal
Asignado a:
José Juan Pardo
Categoría:
-
Versión prevista:
-
Fecha de inicio:
2015-06-29
Fecha fin:
% Realizado:
0%
Version:
Descripción
En preproduccion se produce un error en la comprobacion de la divisa intermedia:
app/controllers/resumen_agente_controller.rb:485
Histórico
#1 Actualizado por Jaime Ortiz hace casi 9 años
Se trata de cambiar el estilo del codigo:
moneda_agente = Moneda.find(@agente.moneda_id)
divisa_agente = Moneda.find(@agente.moneda_intermedia_id)
Por:
moneda_agente = @agente.moneda_principal
divisa_agente = @agente.moneda_intermedia
Lo hemos corregido para la versión 2.48.2
Podrías corregirlo en el trunk ?
NOTA: Por otro lado seria interesante acordar unas normas de estilo de codificación y de pruebas necesarias para validar las cosas.
#2 Actualizado por Santiago Ramos hace más de 8 años
- Estado cambiado Nueva por Resuelta
Veo en el código que está cambiado por:
moneda_agente = Moneda.find_by_id(@agente.moneda_id) divisa_agente = Moneda.find_by_id(@agente.moneda_intermedia_id)
(es un poco más retorcido, pero igual a nivel de querys y al usar "find_by_id" no casca si el parametro es nil)
Cierro el ticket.