1era Tarea / Providers #133
Conversation
| /// <param name="value">Provider information.</param> | ||
| [HttpPut("{id}")] | ||
| public void Put(string id,[FromBody] ProviderDTO value) | ||
| { |
There was a problem hiding this comment.
Fijate que si el ID que se esta usando para actualizar la entidad no es el mismo que el que llega como parametro en la url. Si el json en el body del request no incluye el ID la actualizacion no se va a completar
There was a problem hiding this comment.
@miguel-rodriguez-cimino no entendí esto, me lo podrías ampliar un poco? Gracias!
There was a problem hiding this comment.
En Swagger UI cuando probas el metodo PUT tenes el textbox donde pones el ID de la entidad, y abajo vas a tener un editor donde pones en el body del request el json del objeto que queres actualizar. Ese json es algo asi como:
{
"id": "string",
"name": "string",
"phone": "string",
"email": "string",
...etc..
}
el Id que va aca, en el body, deberia ignorarse e incluso puede ser omitido porque tecnicamente el ID no es un valor que vos puedas actualizar. El ID que se usa para identificar el objeto que queres actualizar es el que pones en el textbox y que va a ir en la url del request, ej
http://localhost:49257/api/provider/fd665ad9-7966-4d45-9be8-d3c72de78043
Asi como esta el codigo, si probas mandar un request sin el id en el body, la entidad no se va a actualizar.
Como pista te puedo indicar que en realidad, el metodo Put del controller esta bien, lo que tendrias que corregir esta en otro lado, quizas debuggeandolo te podes dar cuenta, y si no avisame y te doy mas detalles.
Description
Issues
Ninguno
Opinion
Faltaría algún tutorial linkeado para establecer el entorno de desarrollo sin demasiado inconveniente, tanto en Windows como en otros sistemas operativos.
La tarea era bastante inmediata de extensión de código existente, por lo que no hay mucho que agregar por ahora.