search

civio / onodo

Network analysis for everyone
http://onodo.org/
GNU Affero General Public License v3.0
23 stars 10 forks source link

No se pueden añadir nodos #144

Closed esebastian closed 8 years ago

esebastian commented 8 years ago

Al añadir nodos desde la edición de una visualización se envía un registro vacío, que no se llega a insertar en la BD (fallan las validaciones).

Las actualizaciones posteriores no se envían a la BD.

dcabo commented 8 years ago

Ah, justo hace un rato me he chocado con esto, es lo primero que he sufrido cuando me he puesto a jugar con Onodo. Propongo desactivar la validación (que comprueba que el nodo tiene nombre) a corto plazo, de cara al martes, y hablamos con calma sobre si hay una solución mejor.

Por cierto, no sé si sería buena idea que las llamadas a la base de datos desde el controlador de la API fueran con ! para que en casos como estos no fallen silenciosamente, como está pasando ahora.

esebastian commented 8 years ago

A mi me da la impresión de que esto estaba funcionando antes, dejadme que le pegue un vistazo.

En cualquier caso, ¿qué os parecería si en lugar de desactivar la validación al crear una fila nueva le diésemos un nombre por defecto (tipo 'new node' o similar)?

Mi temor es que desactivando las validaciones se nos rompa la renderización de algunas vistas.

rauldiazpoblete commented 8 years ago

@esebastian Esto no podemos modificarlo ahora porque rompería la edición de las visualizaciones. Tenemos que plantearlo con calma más adelante para encontrar una solución alternativa. La cuestión es que la tabla de edición de nodos está sincronizada con la API, de manera que al crear una fila nueva en la tabla se crea un nodo nuevo vacío. Sé que desde el punto de vista de back esto no es deseable, pero nos resulta muy útil para facilitar la integración de HandsOnTable con el back. Si nos planteamos alguna validación podría ser en la renderización de las vistas, eliminando aquellos nodos o relaciones vacíos antes de devolver el listado de nodos/visualizaciones

dcabo commented 8 years ago

@rauldiazpoblete ¿lo que rompería la edición es lo de poner un nombre por defecto? ¿pero lo de quitar la validación en el modelo para no exigir que haya un nombre tú lo ves ok?

rauldiazpoblete commented 8 years ago

Lo que rompe la edición es impedir la creación de nodos sin nombre, porque siempre que creamos un nodo desde la tabla de edición primero se crea una fila vacía y por tanto sin nombre.

esebastian commented 8 years ago

La validación se ha eliminado en 89a671b.