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
config/blueprints.js > defaultLimit doesn't affect populated associations #6829
Comments
@krawa76 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. |
Hi @krawa76 - thanks for the detailed info and workaround. |
Cool, didn't know that the parseBlueprintOptions existed. Works, thanks a lot! |
Node version: 10.13.0
Sails version (sails): 1.2.2
ORM hook version (sails-hook-orm): 2.1.1
Sockets hook version (sails-hook-sockets): 2.0.0
Organics hook version (sails-hook-organics): n/a
Grunt hook version (sails-hook-grunt): 4.0.0
Uploads hook version (sails-hook-uploads): n/a
DB adapter & version (e.g. sails-mysql@5.55.5): sails-mongo@1.0.1
Skipper adapter & version (e.g. skipper-s3@5.55.5): n/a
Model (Project.ts)
config/blueprints.js:
Issue: call to http://localhost:1337/project populates the apps collection with {limit: 30} condition, regardless of the number specified for defaultLimit.
Looks like the reason for this is:
sails/lib/hooks/blueprints/parse-blueprint-options.js
Line 48 in 8fb5774
The following code fixes the issue (same approach as
sails/lib/hooks/blueprints/actionUtil.js
Line 28 in 8fb5774
Could you include the fix to the next patch? Thanks.
The text was updated successfully, but these errors were encountered: