search

fos-alex / estamos-para-ayudarlo

1 stars 2 forks source link

ABM Listas de Compra - Editar #34

Closed jbichman closed 10 years ago

jbichman commented 10 years ago

Luego de hacer todo el circuito para editar listas de compras, se llega a la última pantalla de confirmación y el saveList() termina devolviendo un error en el request PUT. Habría que ver por qué está fallando y debería poder capturarse el error.

En nuevaLista.html agrego el código para capturar este tipo de errores del saveList y mostrar la notificación. Ver de completarlo desde el lado del controlador o lo que sea que esté fallando.

erroreditar

Alex, ayudame a arreglarlo. Gracias!

pdbenitez commented 10 years ago

Juli, x lo q estuve viendo ese error te devuelve cuando falla al hacer el update en la base de datos posta.

if($this->lista_model->update($id,$lista)) { //aca entra si el update en la base si hizo correctamente }else { $this->error(406,"Error modificando la lista $id");
}

Voy a hacer el circuito de editar a ver si me tira ese error.

pdbenitez commented 10 years ago

Hay algo q no entiendo bien. Cada vez que quiero editar una lista tira un OPTIONS primero y dsp un PUT. Cuando lo pruebo en forma local se queda colgado esperando una respuesta en el OPTIONS. Cuando lo pruebo apuntando al server amazon pasa el OPTIONS (obtiene una respuesta de metodo no implementado) y a continuacion manda el PUT y ahi es donde devuelve el msj q dice Juli y falla.

image

pmussari commented 10 years ago

Dani, necesitas ayuda con esto?

pdbenitez commented 10 years ago

No no pasa nada pablito. Juli, recien probe crear y modificar una lista y lo hace sin problemas (tanto localmente como al server amazon). Lo unico q veo q esta mal es cuando recupera las cantidades en el editar q vuelven a estar en 0. Habria q ver si podemos reproducir el error q te paso.

jbichman commented 10 years ago

DANIIII, ahí quedó!!! qué tocaste? jajaja Bueno, con respecto a que no levanta las cantidades en el editar creo que ya habia un ticket por eso. Lo del OPTIONS yo tmb lo vi pero no lo entendía y supuse que no era relevante para la operación. Mill gracias. Lo dejo abierto porque voy a probar un toque más, luego lo cierro

pdbenitez commented 10 years ago

Lo unico q hice fue boletear todas las listas y sus productos q habian en la base xq me molestaba ver como 20 listas al pedo jajaj Dsp le agregue a las listas el campo fecha_modificacion x el tema q necesitaban saber la ultima lista vigente o algo asi si mal no recuerdo

jbichman commented 10 years ago

Bueno genial Dani, funciona bien. Queda por ver el tema de las cantidades que estaba en otro issue Muchas gracias por la ayuda! lo cierro!