Params.to_map behaviour for missing fields. about params HOT 4 CLOSED

Hey @lasseebert,

Definitely your use case seems valid to me, there should be a way to determine if some value was given on the request or completely missed. And as you mention I think the behaviour of to_map has to change, having nil values only if the user explicitly sent nil for that field or if the developer specified a default nil value for it. Please go ahead implementing it.

I've just added you as collaborator so you can directly merge things or send PR if you want some code review. Also merged ecto-2.x branch into master as I think we'll be adding more features on it and ecto-1.x branch will be kept behind.

Thanks again! :)

from params.

Oh, just to make it clear, I don't think we need a separate function besides to_map, so feel free to change it as you said.

from params.

We should update the documentation after this, IIRC it still mentions the old changes function. And we should mention the new behaviour of to_map and how it contrasts with data.

from params.

Thanks for making me a collaborator :)
I'll make the changes to to_map as discussed above.

from params.