Open bigslycat opened 8 years ago
Yeah async setters would be useful. It's more difficult than you think, because the setter would need to resolve before doing validate()
, and you'd need to return a promise from .set()
because .set()
can throw an error if there's a cast error.
Would love to see that coming in Mongoose ! Do you plan to implement that in a next release @vkarpov15 ?
@maxs15 possibly. Still haven't investigated how we would implement this feature, so uncertain.
I've written a little plugin to mock async getters/setters until it lands in the core. It uses different properties (read
instead of get
and write
instead of set
) to simplify things and middleware (pre init
and post save
for getters and pre save
for setters). It is by no means perfect (I'm especially not happy how it interacts with validators) but it gets the job done.
I like this plugin but there's a few caveats, we'd also have to integrate this queries, because we also support doing things like findByIdAndUpdate(doc, doc)
I would be happy to see that feature.
What's the status?
+1
May be we ready for PR? Oh I wait this so long. =)
Hi what is status on this?
Library worked great with mongoose 4.
@Matzu89 what library?
Any update on this?
This would be a great feature to add.
Resurrecting this old thread since it is still the latest for this issue, and trying to bump it for visibility.
For my use case, I have a couple of fields that need to be encrypted with libsodium
before saving, and that needs to be done asynchronously. As of now I can't use neither get/set
, nor a virtual getter/setter
and had to settle on using a pre
save method which is far from ideal.
I have read the two discussions (#2571 and #517) that relate to this opportunity, and still do not see any reason to abandon it. Why not? Even if Mongoose should maintain compatibility with the ES5, this does not prevent to check that the setter returns to us. Watch this pseudo-code:
So why not? This is a very useful feature, and it will join naturally in existing.