Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
dresende
/
node-orm2
Public
Notifications
You must be signed in to change notification settings
Fork
368
Star
3k
Code
Issues
203
Pull requests
15
Actions
Projects
Wiki
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Wiki
Security and quality
Insights
With association autoFetch cache may return incomplete instance.
- #685
#685
Open
stueynz
wants to merge 11 commits into
dresende:master
dresende/node-orm2:master
from
stueynz:master
stueynz/node-orm2:master
Copy head branch name to clipboard
Conversation
Commits
11
(11)
Checks
Files changed
Open
With association autoFetch cache may return incomplete instance.
#685
stueynz
wants to merge 11 commits into
dresende:master
dresende/node-orm2:master
from
stueynz:master
stueynz/node-orm2:master
Copy head branch name to clipboard
Commits
Commits on Dec 8, 2015
Merge pull request #2 from stueynz/hasOne-with-zeroID
Show description for 5b368d8
Stuart McGrigor
committed
5b368d8
View commit details
Copy full SHA for 5b368d8
Browse repository at this point
Commits on Dec 9, 2015
When doing autofetch on associations, cached instance may not have all the associations populated
stueynz
committed
2e805b1
View commit details
Copy full SHA for 2e805b1
Browse repository at this point
Check cache returns correct objects, auto-fetch on & off
stueynz
committed
d267788
View commit details
Copy full SHA for d267788
Browse repository at this point
Merge complete
stueynz
committed
fc20525
View commit details
Copy full SHA for fc20525
Browse repository at this point
Ensure cache is still working as expected
stueynz
committed
c23f1c5
View commit details
Copy full SHA for c23f1c5
Browse repository at this point
Commits on Dec 17, 2015
Using the 'autoFetch' option on get() and find() will override the 'autoFetch' setting from the association definition.
stueynz
committed
350b4a4
View commit details
Copy full SHA for 350b4a4
Browse repository at this point
if get() or find() has locally set 'autoFetch' option, then don't use the cache
stueynz
committed
02824e2
View commit details
Copy full SHA for 02824e2
Browse repository at this point
renamed checkAssociations() to hasAutoFetchAssociationsLoaded() as its more descriptive, if a little long
stueynz
committed
51307fd
View commit details
Copy full SHA for 51307fd
Browse repository at this point
Whoops, we can't assume what the Id values will be when testing get() functions; particularly for mongodb
stueynz
committed
e38be11
View commit details
Copy full SHA for e38be11
Browse repository at this point
Commits on Jan 10, 2016
When hasMany relationship has extra data, then the UID used by Singleton for the associative entity needs to include both keys to be sure we get the right set of extra data
stueynz
committed
d753091
View commit details
Copy full SHA for d753091
Browse repository at this point
Merge pull request #4 from stueynz/hasmany-extra-cache
Show description for 4fe75f8
Stuart McGrigor
committed
4fe75f8
View commit details
Copy full SHA for 4fe75f8
Browse repository at this point
You can’t perform that action at this time.