New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FindOne() does not honor multiple where() #6830
Comments
@grd2345 Thanks for posting! We'll take a look as soon as possible. In the mean time, there are a few ways you can help speed things along:
Please remember: never post in a public forum if you believe you've found a genuine security vulnerability. Instead, disclose it responsibly. For help with questions about Sails, click here. |
Just a note, this issue is occuring also using Find() with multiple where(). It does not honor multiple where anymore.Is this by design or a bug. |
Hi @grd2345 honestly the word |
@grd2345 Thanks for bringing this to attention! @navicstein Thanks for the response as well! Sails console example: |
thanks, yes, I refactored to not use chain-able where clauses anymore. Thanks for everything. |
@grd2345 Ah, yes Sails 1.x has breaking changes for how waterline handles |
Node version: v8.16.0
Sails version (sails): 1.2.3
ORM hook version (sails-hook-orm): 2.1.1
Sockets hook version (sails-hook-sockets): 1.5.5
Grunt hook version (sails-hook-grunt): 1.0.8
DB adapter & version (e.g. sails-mysql@5.55.5): 1.0.2
I am in the process of migrating to version 1 and their is a breaking change that I did not see on the migration documents. This used to work in the prior version.
Agreement.findOne()
.where({id: agreementId})
.where({locationId: employee.currentLocation.id})
Now this fails with
More than one matching record found for
.findOne()
:I stepped through the adapter code and it ignores all .where() except the last where, which is an issue.
The text was updated successfully, but these errors were encountered: